Re: [PATCH i-g-t] tests/kms_getfb: Skip getfb2-accept-ccs subtest on Xe2 platforms

[Karthik B S <karthik.b.s@intel.com>]

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


On 6/21/2024 10:02 AM, Pranay Samala wrote:
> The getfb2-accept-ccs subtest tests on ccs modifiers.
> With Xe2 platforms we no more have the ccs modifiers.
>
> Adding a check to verify the platform. If it is
> Xe2 platform then test will skip, otherwise it
> will execute further.
>
> Signed-off-by: Pranay Samala<pranay.samala@intel.com>
> ---
>   tests/kms_getfb.c | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/tests/kms_getfb.c b/tests/kms_getfb.c
> index bb9679bc9..81829ca3b 100644
> --- a/tests/kms_getfb.c
> +++ b/tests/kms_getfb.c
> @@ -161,6 +161,9 @@ static void get_ccs_fb(int fd, struct drm_mode_fb_cmd2 *ret)
>   	igt_require(has_addfb2_iface(fd));
>   	devid = intel_get_drm_devid(fd);
>   
> +	igt_require_f(intel_graphics_ver(devid) < 20,
> +		      "No ccs modifiers on Xe2\n");
> +

Hi Pranay,

I see that getfb tests are skipping on legacy platforms with this patch.
https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_11292/index.html?testfilter=kms_getfb

This should be resolved by replacing 'intel_graphics_ver' with 
'intel_display_ver'. Could you please update this.

Thanks,
Karthik.B.S
>   	if (HAS_FLATCCS(devid)) {
>   		add.modifier[0] = I915_FORMAT_MOD_4_TILED_DG2_RC_CCS;
>   		add.pitches[0] = ALIGN(add.width * 4, 4 * 512);

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