Re: [PATCH i-g-t] tests/intel/xe_sriov_flr: Skip media GTs when reading LMEM provisioning

["Laguna, Lukasz" <lukasz.laguna@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 1439 bytes --]


On 8/26/2025 21:25, Marcin Bernatowicz wrote:
> LMEM provisioning is exposed only on the primary GT.
> Media GTs do not provide LMEM provisioning data,
> so attempting to read it there caused the subtest to skip.
> Restrict iteration to non-media GTs before reading LMEM provisioning.
>
> Signed-off-by: Marcin Bernatowicz<marcin.bernatowicz@linux.intel.com>
> Cc: Adam Miszczak<adam.miszczak@linux.intel.com>
> Cc: Jakub Kolakowski<jakub1.kolakowski@intel.com>
> Cc: Lukasz Laguna<lukasz.laguna@intel.com>
> Cc: Michal Wajdeczko<michal.wajdeczko@intel.com>
> Cc: Satyanarayana K V P<satyanarayana.k.v.p@intel.com>
> ---
>   tests/intel/xe_sriov_flr.c | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/tests/intel/xe_sriov_flr.c b/tests/intel/xe_sriov_flr.c
> index fc08703c0..248dcf6fb 100644
> --- a/tests/intel/xe_sriov_flr.c
> +++ b/tests/intel/xe_sriov_flr.c
> @@ -813,6 +813,9 @@ static int populate_vf_lmem_sizes(struct subcheck_data *data)
>   	igt_assert(ldata->vf_lmem_size);
>   
>   	xe_for_each_gt(data->pf_fd, gt) {
> +		if (xe_is_media_gt(data->pf_fd, gt))

This data is available only for the main GT, so I would suggest changing 
the condition to |!xe_is_main_gt()|. That way it also covers potential 
other non-main GT types in the future.

> +			continue;
> +
>   		ret = xe_sriov_pf_debugfs_read_provisioned_ranges(data->pf_fd,
>   								  XE_SRIOV_SHARED_RES_LMEM,
>   								  gt, &ranges, &nr_ranges);

[-- Attachment #2: Type: text/html, Size: 2532 bytes --]