From 595461f7254f1d5cba8d0772cad1f8eac6accd94 Mon Sep 17 00:00:00 2001 From: Andrea Mazzoleni Date: Mon, 7 Sep 2026 00:20:02 +0200 Subject: [PATCH] Document thermal power management as best effort --- cmdline/thermal.c | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/cmdline/thermal.c b/cmdline/thermal.c index a306fc78..4a4c29d6 100644 --- a/cmdline/thermal.c +++ b/cmdline/thermal.c @@ -345,6 +345,20 @@ void state_thermal_cooldown(struct snapraid_state* state) int sleep_time = cooldown_time; log_tag("thermal:spindown\n"); + + /* + * Power management is best effort. + * + * Some devices may not support explicit spindown/spinup operations, or an + * operation may fail only for a subset of the devices. This must not make the + * whole thermal cooldown fail. + * + * Thermal safety is enforced by measuring the temperatures again after the + * cooldown. If any disk is still above the configured limit, the cooldown is + * repeated. Therefore failure to spin down a device only makes cooling less + * effective; it does not allow the operation to resume while a measurable disk + * is still above the temperature limit. + */ state_device(state, DEVICE_DOWN, 0); msg_progress("Cooldown...\n");