Skip to content

Commit 073b42f

Browse files
authored
system: don't short circuit merging cpu and power deltas (#50)
1 parent 7c568e8 commit 073b42f

1 file changed

Lines changed: 6 additions & 8 deletions

File tree

watt/system.rs

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -838,38 +838,36 @@ pub fn run_daemon(config: config::DaemonConfig) -> anyhow::Result<()> {
838838
let cpu_some = {
839839
let (cpu_deltas_lo, cpu_turbo_lo) = rule.cpu.eval(&state)?;
840840

841-
let deltas_some = cpu_deltas.iter_mut().all(|(cpu, delta)| {
841+
for (cpu, delta) in cpu_deltas.iter_mut() {
842842
let delta_lo = cpu_deltas_lo
843843
.get(cpu)
844844
.expect("cpu deltas and cpus should match");
845845

846846
*delta = mem::take(delta).or(delta_lo);
847-
848-
delta.is_some()
849-
});
847+
}
850848

851849
cpu_turbo = cpu_turbo.or(cpu_turbo_lo);
852850

851+
let deltas_some = cpu_deltas.values().all(|delta| delta.is_some());
853852
deltas_some && cpu_turbo.is_some()
854853
};
855854

856855
let power_some = {
857856
let (power_deltas_lo, power_platform_profile_lo) =
858857
rule.power.eval(&state)?;
859858

860-
let deltas_some = power_deltas.iter_mut().all(|(power, delta)| {
859+
for (power, delta) in power_deltas.iter_mut() {
861860
let delta_lo = power_deltas_lo
862861
.get(power)
863862
.expect("power deltas and power supplies should match");
864863

865864
*delta = mem::take(delta).or(delta_lo);
866-
867-
delta.is_some()
868-
});
865+
}
869866

870867
power_platform_profile =
871868
power_platform_profile.or(power_platform_profile_lo);
872869

870+
let deltas_some = power_deltas.values().all(|delta| delta.is_some());
873871
deltas_some && power_platform_profile.is_some()
874872
};
875873

0 commit comments

Comments
 (0)