Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7;
Message-ID: <836c7166-0a42-4fdc-acf4-65e0a096c41f@collabora.com>
Date:   Sat, 4 Nov 2023 10:19:17 -0300
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 3/3] drm/ci: Add skips, fails and flakes for SM8250
Content-Language: en-US
To:     Jessica Zhang <quic_jesszhan@quicinc.com>,
        David Airlie <airlied@gmail.com>,
        Daniel Vetter <daniel@ffwll.ch>
Cc:     robdclark@chromium.org, quic_abhinavk@quicinc.com,
        freedreno@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
        linux-kernel@vger.kernel.org
References: <20231010-rb5-runner-v1-0-aba1fcc6e3aa@quicinc.com>
 <20231010-rb5-runner-v1-3-aba1fcc6e3aa@quicinc.com>
From:   Helen Koike <helen.koike@collabora.com>
In-Reply-To: <20231010-rb5-runner-v1-3-aba1fcc6e3aa@quicinc.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


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@quicinc.com>
> Signed-off-by: Jessica Zhang <quic_jesszhan@quicinc.com>
> ---
>   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.*
>