Re: [PATCH] cpufreq: schedutil: Don't ignore limit changes when util is unchanged

From: Rafael J. Wysocki
Date: Fri Apr 11 2025 - 06:31:08 EST


On Fri, Apr 11, 2025 at 10:22 AM Xuewen Yan <xuewen.yan94@xxxxxxxxx> wrote:
>
> ...
> >
> > AFAICS, after this code modification, a limit change may be missed due
> > to a possible race with sugov_limits() which cannot happen if
> > sg_policy->limits_changed is only cleared when it is set before
> > updating sg_policy->need_freq_update.
> >
> could the following patch prevent the race?
>
> https://lore.kernel.org/all/CAB8ipk_Ayqmh=Ch2aH2c+i-q+qdiQ317VBH1kOHYN=R9dt6LOw@xxxxxxxxxxxxxx/

The first hunk is essentially a partial revert of the problematic
commit, but I'm not sure what you want to achieve with the second one.