Re: [PATCH] drm/xe/uc: Apply RCS/CCS yield policy to SR-IOV VFs

[Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>]

On 7/8/2026 3:06 AM, Marcin Bernatowicz wrote:
> VFs were missing the call to apply the global scheduling policy.
> Call xe_guc_submit_enable() during vf_uc_load_hw() to ensure VFs
> get the same policy enforcement as PF.
>
> Fixes: 26caeae9fb48 ("drm/xe/guc: Set RCS/CCS yield policy")
> Suggested-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
> Signed-off-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
> Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> ---
>   drivers/gpu/drm/xe/xe_uc.c | 5 ++++-
>   1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_uc.c b/drivers/gpu/drm/xe/xe_uc.c
> index 75091bde0d50..c6f5fd966afb 100644
> --- a/drivers/gpu/drm/xe/xe_uc.c
> +++ b/drivers/gpu/drm/xe/xe_uc.c
> @@ -15,6 +15,7 @@
>   #include "xe_guc_pc.h"
>   #include "xe_guc_rc.h"
>   #include "xe_guc_engine_activity.h"
> +#include "xe_guc_submit.h"
>   #include "xe_huc.h"
>   #include "xe_sriov.h"
>   #include "xe_wopcm.h"
> @@ -159,7 +160,9 @@ static int vf_uc_load_hw(struct xe_uc *uc)
>   	if (err)
>   		return err;
>   
> -	uc->guc.submission_state.enabled = true;
> +	err = xe_guc_submit_enable(&uc->guc);
> +	if (err)
> +		return err;

On PF we call xe_guc_submit_enable() after 
xe_guc_opt_in_features_enable(). It doesn't make any difference right 
now, but IMO we should still make the VF ordering match PF in case it 
matters in the future.
With that:

Reviewed-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>

Thanks for the fix!
Daniele

>   
>   	err = xe_guc_opt_in_features_enable(&uc->guc);
>   	if (err)