Re: [igt-dev] [PATCH i-g-t 3/6] RFC tests/i915/kms_frontbuffer_tracking: xe doesn't support tiling as of now

["Hogander, Jouni" <jouni.hogander@intel.com>]

On Thu, 2023-06-01 at 10:39 +0530, Kunal Joshi wrote:
> XE doesn't support tiling as of now, so set tiling to linear

Maybe this should be taken into account in opt_handler as well? I.e.
Trying to use tiling other than liner for Xe should result error.

> 
> v2: missed one tiling case
> 
> Signed-off-by: Kunal Joshi <kunal1.joshi@intel.com>
> ---
>  tests/i915/kms_frontbuffer_tracking.c | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/tests/i915/kms_frontbuffer_tracking.c
> b/tests/i915/kms_frontbuffer_tracking.c
> index 3bef46b6..de1a82a9 100644
> --- a/tests/i915/kms_frontbuffer_tracking.c
> +++ b/tests/i915/kms_frontbuffer_tracking.c
> @@ -1349,6 +1349,12 @@ static void setup_drm(void)
>          */
>         is_xe = is_xe_device(drm.fd);
>  
> +       /*
> +        * XE only support linear tiling
> +        */
> +       if (is_xe)
> +               opt.tiling = TILING_LINEAR;
> +
>         kmstest_set_vt_graphics_mode();
>         igt_display_require(&drm.display, drm.fd);
>  
> @@ -3642,6 +3648,9 @@ igt_main_args("", long_options, help_str,
> opt_handler, NULL)
>  
>                         for (t.tiling = TILING_LINEAR; t.tiling <
> TILING_COUNT;
>                              t.tiling++) {
> +                               if (is_xe && t.tiling !=
> TILING_LINEAR)
> +                                       continue;
> +
>                                 if (t.tiling == TILING_X)
>                                         continue;
>