Re: [PATCH v3] drm/xe: Fix a bug in pc_adjust_freq_bounds()

On Wed, 2026-08-05 at 16:46 -0700, Vinay Belgaumkar wrote:
> In cases where min frequency was actually greater than BMG_MIN_FREQ,
> we were not using the updated min frequency as there was a missing
> call to pc_action_query_task_state() between the two settings of
> min frequency. Since we know what min_freq was last set, use that
> cached value while comparing to BMG_MIN_FREQ to fix this issue.
> 
> v2: pc->freq_ready is not set until after pc_adjust_freq_bounds().
> Stay
> with pc_action_query_task_state() instead.
> 
> v3: Update commit message (Stuart)
> 
> Fixes: bdde16c9ac5c ("drm/xe/bmg: Update Wa_14022085890")
> Signed-off-by: Balasubramani Vivekanandan
> <[email protected]>
> Signed-off-by: Vinay Belgaumkar <[email protected]>

Reviewed-by: Stuart Summers <[email protected]>

Thanks for the change here!

> ---
>  drivers/gpu/drm/xe/xe_guc_pc.c | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_guc_pc.c
> b/drivers/gpu/drm/xe/xe_guc_pc.c
> index 59f2fa79ad42..7cf8f4858598 100644
> --- a/drivers/gpu/drm/xe/xe_guc_pc.c
> +++ b/drivers/gpu/drm/xe/xe_guc_pc.c
> @@ -911,6 +911,7 @@ static bool pc_needs_min_freq_change(struct
> xe_guc_pc *pc)
>  static int pc_adjust_freq_bounds(struct xe_guc_pc *pc)
>  {
>         int ret;
> +       u32 min_freq;
>  
>         lockdep_assert_held(&pc->freq_lock);
>  
> @@ -933,8 +934,14 @@ static int pc_adjust_freq_bounds(struct
> xe_guc_pc *pc)
>          * Same thing happens for Server platforms where min is
> listed as
>          * RPMax
>          */
> -       if (pc_get_min_freq(pc) > pc->rp0_freq)
> +       min_freq = pc_get_min_freq(pc);
> +       if (min_freq > pc->rp0_freq) {
>                 ret = pc_set_min_freq(pc, pc->rp0_freq);
> +               if (ret)
> +                       goto out;
> +
> +               min_freq = pc->rp0_freq;
> +       }
>  
>         /*
>          * Setting GT RP min frequency to 1.2GHz by default for
> @@ -947,8 +954,8 @@ static int pc_adjust_freq_bounds(struct xe_guc_pc
> *pc)
>          * we aren't expecting high power output across board
>          *
>          */
> -       if (pc_needs_min_freq_change(pc))
> -               ret = pc_set_min_freq(pc, max(BMG_MIN_FREQ,
> pc_get_min_freq(pc)));
> +       if (pc_needs_min_freq_change(pc) && min_freq < BMG_MIN_FREQ)
> +               ret = pc_set_min_freq(pc, BMG_MIN_FREQ);
>  
>  out:
>         return ret;