danvics/R710-Fan-Control
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

make more robust against ipmitool failing

Browse source 
in default fallback case (which includes when a fault has occurred or
when the script exits) - keep retrying for a bit, and in servo case,
just allow the next loop to proceed ASAP, without resetting variables
and hysteresis
This commit is contained in:
Tim Connors 2020-11-16 12:04:56 +11:00
parent e41c1aa38e
commit cf3a9c347c
1 changed files with 23 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
R710-IPMI-TEMP/fan-speed-control.pl
View file

@ -74,8 +74,15 @@ sub set_fans_default {
$current_mode="default"; $current_mode="default";
$lastfan=undef; $lastfan=undef;
print "--> enable dynamic fan control\n"; print "--> enable dynamic fan control\n";
system("ipmitool raw 0x30 0x30 0x01 0x01"); foreach my $attempt (1..10) {
system("ipmitool raw 0x30 0x30 0x01 0x01") == 0 and return 1;
sleep 1;
print " Retrying dynamic control $attempt\n";
}
print "Retries of dynamic control all failed\n";
return 0;
} }
return 1;
} }
sub set_fans_servo { sub set_fans_servo {
@ -101,7 +108,9 @@ sub set_fans_servo {
if (!defined $current_mode or $current_mode ne "set") { if (!defined $current_mode or $current_mode ne "set") {
$current_mode="set"; $current_mode="set";
print "--> disable dynamic fan control\n"; print "--> disable dynamic fan control\n";
system("ipmitool raw 0x30 0x30 0x01 0x00"); system("ipmitool raw 0x30 0x30 0x01 0x00") == 0 or return 0;
# if this fails, want to return telling caller not to think weve
# made a change
} }
# FIXME: probably want to take into account ambient temperature - if # FIXME: probably want to take into account ambient temperature - if
@ -138,8 +147,11 @@ sub set_fans_servo {
$demand = sprintf("0x%x", $demand); $demand = sprintf("0x%x", $demand);
# print "demand = $demand\n"; # print "demand = $demand\n";
print "--> ipmitool raw 0x30 0x30 0x02 0xff $demand\n"; print "--> ipmitool raw 0x30 0x30 0x02 0xff $demand\n";
system("ipmitool raw 0x30 0x30 0x02 0xff $demand"); system("ipmitool raw 0x30 0x30 0x02 0xff $demand") == 0 or return 0;
# if this fails, want to return telling caller not to think weve
# made a change
} }
return 1;
} }
my ($tempfh, $tempfilename) = tempfile("fan-speed-control.XXXXX"); my ($tempfh, $tempfilename) = tempfile("fan-speed-control.XXXXX");
@ -195,9 +207,15 @@ while () {
# FIXME: hysteresis # FIXME: hysteresis
if ($ambient_temp > $default_threshold) { if ($ambient_temp > $default_threshold) {
print "fallback because of high ambient temperature $ambient_temp > $default_threshold\n"; print "fallback because of high ambient temperature $ambient_temp > $default_threshold\n";
set_fans_default(); if (!set_fans_default()) {
# return for next loop without resetting timers and delta change if that fails
next;
}
} else { } else {
set_fans_servo($ambient_temp, \@cputemps, \@coretemps, \@hddtemps); if (!set_fans_servo($ambient_temp, \@cputemps, \@coretemps, \@hddtemps)) {
# return for next loop without resetting timers and delta change if that fails
next;
}
} }
# every 20 minutes (enough to establish spin-down), invalidate the # every 20 minutes (enough to establish spin-down), invalidate the