Re: [PATCH 3/3] drm/ci: Add skips, fails and flakes for SM8250

[Helen Koike]

On 10/10/2023 19:25, Jessica Zhang wrote:
> Add skips, fails and flakes for the SM8250 test.
>
> Generated using update-xfails.py [1]
>
> [1] https://patchwork.freedesktop.org/patch/561453/?series=124793&rev=1
>
> Signed-off-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>
> Signed-off-by: Jessica Zhang <quic_jesszhan@xxxxxxxxxxx>
> ---
>   drivers/gpu/drm/ci/xfails/msm-sm8250-fails.txt  | 29 +++++++++++++++++++++++++
>   drivers/gpu/drm/ci/xfails/msm-sm8250-flakes.txt |  3 +++
>   drivers/gpu/drm/ci/xfails/msm-sm8250-skips.txt  |  8 +++++++
>   3 files changed, 40 insertions(+)
>
> diff --git a/drivers/gpu/drm/ci/xfails/msm-sm8250-fails.txt b/drivers/gpu/drm/ci/xfails/msm-sm8250-fails.txt
> new file mode 100644
> index 000000000000..cc8ae32e90e7
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/xfails/msm-sm8250-fails.txt
> @@ -0,0 +1,29 @@
> +kms_3d,Fail
> +kms_atomic_transition@plane-all-modeset-transition,Timeout
> +kms_color@ctm-0-25,Fail
> +kms_color@ctm-0-50,Fail
> +kms_color@ctm-0-75,Fail
> +kms_color@ctm-blue-to-red,Fail
> +kms_color@ctm-negative,Fail
> +kms_color@ctm-red-to-blue,Fail
> +kms_color@ctm-signed,Fail
> +kms_cursor_legacy@basic-flip-after-cursor-varying-size,Fail
> +kms_cursor_legacy@basic-flip-before-cursor-varying-size,Fail
> +kms_cursor_legacy@cursor-vs-flip-atomic-transitions-varying-size,Fail
> +kms_cursor_legacy@cursor-vs-flip-toggle,Fail
> +kms_cursor_legacy@cursor-vs-flip-varying-size,Fail
> +kms_cursor_legacy@short-flip-after-cursor-atomic-transitions-varying-size,Fail
> +kms_cursor_legacy@short-flip-before-cursor-atomic-transitions-varying-size,Fail
> +kms_cursor_legacy@short-flip-before-cursor-toggle,Fail
> +kms_hdmi_inject@inject-4k,Fail
> +kms_pipe_crc_basic@compare-crc-sanitycheck-nv12,Fail
> +kms_plane@pixel-format,Fail
> +kms_plane@pixel-format-source-clamping,Fail
> +kms_plane@plane-position-covered,Fail
> +kms_plane@plane-position-hole,Fail
> +kms_plane@plane-position-hole-dpms,Fail
> +kms_plane_alpha_blend@alpha-7efc,Fail
> +kms_plane_alpha_blend@coverage-7efc,Fail
> +kms_plane_alpha_blend@coverage-vs-premult-vs-constant,Fail
> +kms_plane_cursor@overlay,Fail
> +kms_rmfb@close-fd,Fail
> diff --git a/drivers/gpu/drm/ci/xfails/msm-sm8250-flakes.txt b/drivers/gpu/drm/ci/xfails/msm-sm8250-flakes.txt
> new file mode 100644
> index 000000000000..0b55665184c1
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/xfails/msm-sm8250-flakes.txt
> @@ -0,0 +1,3 @@
> +kms_cursor_legacy@flip-vs-cursor-atomic-transitions-varying-size
> +kms_cursor_legacy@flip-vs-cursor-varying-size
> +kms_plane_cursor@viewport

We are trying to add some docs, specially to the flakes, please check: 
https://lists.freedesktop.org/archives/dri-devel/2023-October/427982.html

Could you add it for those? (I'm glad to see there are just a few flakes).

Thanks
Helen

> diff --git a/drivers/gpu/drm/ci/xfails/msm-sm8250-skips.txt b/drivers/gpu/drm/ci/xfails/msm-sm8250-skips.txt
> new file mode 100644
> index 000000000000..c20422c58e4d
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/xfails/msm-sm8250-skips.txt
> @@ -0,0 +1,8 @@
> +# Suspend to RAM seems to be broken on this machine
> +.*suspend.*
> +
> +# reboots device
> +kms_plane_scaling.*
> +
> +# long execution time
> +kms_flip.*