[igt-dev] [i-g-t] intel-ci/xe: Update kms blocklist documentation

[Sharma, Swati2]

Hi Bhanu,

Please find my comments inline.

On 03-Aug-23 5:53 PM, Bhanuprakash Modem wrote:
> Add justification to kms blocklist.
> 
> Cc: Swati Sharma <swati2.sharma at intel.com>
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>
> ---
>   tests/intel-ci/xe.blocklist.txt | 41 ++++++++++++++++++---------------
>   1 file changed, 22 insertions(+), 19 deletions(-)
> 
> diff --git a/tests/intel-ci/xe.blocklist.txt b/tests/intel-ci/xe.blocklist.txt
> index b40dd4f70..df7bb4149 100644
> --- a/tests/intel-ci/xe.blocklist.txt
> +++ b/tests/intel-ci/xe.blocklist.txt
> @@ -44,41 +44,44 @@ igt at syncobj_.*
>   igt at template
>   igt at tools_test
>   ##################################################################
> -# Pipe E, F, G and H
> +# KMS: Pipe E, F, G and H are not available on Intel hardware
>   ##################################################################
>   igt at .*@.*pipe-(e|f|g|h).*
> -##################################################################
> -# Multidisplay
> -##################################################################
>   igt at kms_bw@.*-(5|6|7|8)-displays-.*
>   ##################################################################
> +# KMS: Tests those are specific to i915 driver

 >>Nitpick: Tests specific to i915 driver

Also, can we precisely mention which all features are not supported
in xe in contrast to i915 and why we had to blacklist all these testcases.

> +##################################################################
> +igt at kms_busy@.*
>   igt at kms_draw_crc@draw-method-(mmap-cpu|mmap-gtt|pwrite)
> -igt at kms_draw_crc@fill-fb
> -igt at kms_pipe_b_c_ivb@.*
> -igt at kms_pwrite_crc
> -igt at kms_ccs@.*
> -igt at kms_frontbuffer_tracking@.*
> -igt at kms_fbcon_fbt@(fbc|psr)(-suspend)
>   igt at kms_fence_pin_leak
> -igt at kms_busy@.*
> -igt at kms_psr@.*_mmap_(cpu|gtt)
> -igt at kms_legacy_colorkey@basic
> -igt at kms_legacy_colorkey@invalid-plane
> -igt at kms_flip_scaled_crc@flip-(32|64)bpp-(4|x|y|yf|tileccs)tile-.*
> -igt at kms_big_fb@(4|x|y|yf)-tiled-.*
>   igt at kms_flip@2x-flip-vs-fences
>   igt at kms_flip@flip-vs-fences(-interruptible)
> +igt at kms_legacy_colorkey@basic
> +igt at kms_legacy_colorkey@invalid-plane

 >> These 2 can be combined together.

> +igt at kms_pipe_b_c_ivb@.*
> +igt at kms_psr@.*_mmap_(cpu|gtt)
> +igt at kms_pwrite_crc
> +igt at kms_rotation_crc@exhaust-fences
> +##################################################################
> +# KMS: IGT HANG & Tiling support is not ready.
> +# To be removed once IGT started supporting these features.
> +##################################################################

 >> Make as 2 categories (i) Hang (ii) Tiling

>   igt at kms_addfb_basic@clobberred-modifier
>   igt at kms_addfb_basic@.*-tiling
>   igt at kms_addfb_basic@.*-(4|x|y|yf)-.*
>   igt at kms_async_flips@.*
> -igt at kms_plane_multiple@tiling-(4|x|y|yf)
> +igt at kms_big_fb@(4|x|y|yf)-tiled-.*
> +igt at kms_ccs@.*
>   igt at kms_cursor_legacy@flip-vs-cursor-busy-crc-(atomic|legacy)
> -igt at kms_rotation_crc@primary-(4|x|y|yf)-tiled-.*
> -igt at kms_rotation_crc@exhaust-fences
> +igt at kms_draw_crc@fill-fb
> +igt at kms_fbcon_fbt@(fbc|psr)(-suspend)

 >> Specific suspend tests are dependent on tiling?

> +igt at kms_flip_scaled_crc@flip-(32|64)bpp-(4|x|y|yf|tileccs)tile-.*
>   igt at kms_force_connector_basic@force-load-detect
> +igt at kms_frontbuffer_tracking@.*
>   igt at kms_getfb@.*-ccs
>   igt at kms_plane_lowres@tiling-(4|x|y|yf)
> +igt at kms_plane_multiple@tiling-(4|x|y|yf)
> +igt at kms_rotation_crc@primary-(4|x|y|yf)-tiled-.*
>   igt at kms_.*@.*hang.*
>   ##################################################################
>   igt at xe_sysfs*