[LTP] thermal: add new test group

linuxtestproject.agent@gmail.com linuxtestproject.agent@gmail.com
Thu May 14 14:00:04 CEST 2026


Hi Piotr,

On 2026-05-14, Piotr Kubaj wrote:
> thermal: add new test group

> +static void run(void)
> +{
> +	static bool first_zone = true;
> +	...
> +		if (!first_zone) {
> +			tst_res(TINFO, "Cooling down for %d seconds", COOLDOWN);
> +			sleep(COOLDOWN);
> +		}
> +		first_zone = false;

The `static` qualifier means `first_zone` is initialized only once, at
program start. On the second iteration (`-i 2`), it will already be
`false`, causing the first zone to incur an unnecessary 300-second
cooldown. Drop `static` so it resets to `true` on every call.

[...]

---
Note:

Our agent completed the review of the patch. The agent can sometimes
produce false positives although often its findings are genuine. If you
find issues with the review, please comment this email or ignore the
suggestions.

Regards,
LTP AI Reviewer


More information about the ltp mailing list