[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