Re: [LTP] cpufreq.c: add new test for cpufreq sysfs interface validation

[linuxtestproject.agent@gmail.com]

Hi Piotr,

On Wed, 6 May 2026, Piotr Kubaj wrote:
> cpufreq.c: add new test for cpufreq sysfs interface validation

> +	tst_res(TDEBUG, "Checking whether scaling_available_governors contains \"performance\" and \"schedutil\" after switching /sys/devices/system/cpu/intel_pstate/status to \"passive\"");
> +	snprintf(path, sizeof(path), "/sys/devices/system/cpu/intel_pstate/status");
> +	SAFE_FILE_PRINTF(path, "passive");
> +
> +	for (int i = 0; i < nproc; i++) {
> +		...
> +		SAFE_FILE_PRINTF(path, "schedutil");
> +		...
> +	}
> +}

run() exits with intel_pstate/status still "passive" and each CPU's
scaling_governor still "schedutil". On the second iteration (-i 2) the
status check fires TFAIL immediately, and the governor branch hits
tst_brk(TBROK, "Unknown scaling_governor: schedutil").

Restore the status to "active" and the governors to previous_scaling_governor[i]
at the end of run() before it returns.

[...]

---
Note:

Our agent completed the review of the patch. The full review can be
found at: https://patchwork.ozlabs.org/project/ltp/list/?series=502943

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

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp