Re: [PATCH v4 1/4] drm/xe/reg_sr: Don't process gt/hwe lists in VF

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

On Wed, 18 Feb 2026 14:09:12 -0800, Matt Roper wrote:
>
> diff --git a/drivers/gpu/drm/xe/xe_rtp.c b/drivers/gpu/drm/xe/xe_rtp.c
> index b7c26e2fb411ea75fbd484b55f48019fc09f0581..7bfdc6795ce6fd3d2c8c9d1398c7b72ff476b96d 100644
> --- a/drivers/gpu/drm/xe/xe_rtp.c
> +++ b/drivers/gpu/drm/xe/xe_rtp.c
> @@ -270,6 +270,8 @@ static void rtp_mark_active(struct xe_device *xe,
>   * @sr: Save-restore struct where matching rules execute the action. This can be
>   *      viewed as the "coalesced view" of multiple the tables. The bits for each
>   *      register set are expected not to collide with previously added entries
> + * @process_in_vf: Whether this RTP table should get processed for SR-IOV VF
> + *      devices.  Should generally only be 'true' for LRC tables.
>   *
>   * Walk the table pointed by @entries (with an empty sentinel) and add all
>   * entries with matching rules to @sr. If @hwe is not NULL, its mmio_base is
> @@ -278,7 +280,8 @@ static void rtp_mark_active(struct xe_device *xe,
>  void xe_rtp_process_to_sr(struct xe_rtp_process_ctx *ctx,
>			  const struct xe_rtp_entry_sr *entries,
>			  size_t n_entries,
> -			  struct xe_reg_sr *sr)
> +			  struct xe_reg_sr *sr,
> +			  bool process_in_vf)

Another idea would be to change 'process_in_vf' to something like
'is_lrc'. But otherwise ok as is too (specially if some non-lrc cases need
to be processed in VF's in the future):

Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>

>  {
>	const struct xe_rtp_entry_sr *entry;
>	struct xe_hw_engine *hwe = NULL;
> @@ -287,6 +290,9 @@ void xe_rtp_process_to_sr(struct xe_rtp_process_ctx *ctx,
>
>	rtp_get_context(ctx, &hwe, &gt, &xe);
>
> +	if (!process_in_vf && IS_SRIOV_VF(xe))
> +		return;
> +