linux-kernel - Re: [PATCH] drm: ci: fix the xfails for apq8016

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date: Mon, 29 Apr 2024 17:06:28 -0300
From: Helen Koike <helen.koike@...labora.com>
To: Abhinav Kumar <quic_abhinavk@...cinc.com>, Rob Clark
 <robdclark@...il.com>, Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
 Sean Paul <sean@...rly.run>, Marijn Suijten <marijn.suijten@...ainline.org>,
 Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
 Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
 David Airlie <airlied@...il.com>, Daniel Vetter <daniel@...ll.ch>
Cc: linux-arm-msm@...r.kernel.org, dri-devel@...ts.freedesktop.org,
 freedreno@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drm: ci: fix the xfails for apq8016



On 25/04/2024 11:01, Helen Koike wrote:
> 
> 
> On 08/04/2024 14:04, Abhinav Kumar wrote:
>> Hi Helen
>>
>> Gentle reminder on this.
>>
>> If you are okay, we can land it via msm-next tree...
>>
>> Thanks
>>
>> Abhinav
>>
>> On 4/1/2024 1:48 PM, Abhinav Kumar wrote:
>>> After IGT migrating to dynamic sub-tests, the pipe prefixes
>>> in the expected fails list are incorrect. Lets drop those
>>> to accurately match the expected fails.
>>>
>>> In addition, update the xfails list to match the current passing
>>> list. This should have ideally failed in the CI run because some
>>> tests were marked as fail even though they passed but due to the
>>> mismatch in test names, the matching didn't correctly work and was
>>> resulting in those failures not being seen.
>>>
>>> Here is the passing pipeline for apq8016 with this change:
>>>
>>> https://gitlab.freedesktop.org/drm/msm/-/jobs/57050562
>>>
>>> Signed-off-by: Abhinav Kumar <quic_abhinavk@...cinc.com>
> 
> I'm sorry about my delay.
> 
> Acked-by: Helen Koike <helen.koike@...labora.com>
> 
> I'm also merging it to msm-misc-next.

Applied, thanks.

Helen

> 
> Regards,
> Helen
> 
>>> ---
>>>   drivers/gpu/drm/ci/xfails/msm-apq8016-fails.txt | 13 +------------
>>>   1 file changed, 1 insertion(+), 12 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/ci/xfails/msm-apq8016-fails.txt 
>>> b/drivers/gpu/drm/ci/xfails/msm-apq8016-fails.txt
>>> index 44a5c62dedad..b14d4e884971 100644
>>> --- a/drivers/gpu/drm/ci/xfails/msm-apq8016-fails.txt
>>> +++ b/drivers/gpu/drm/ci/xfails/msm-apq8016-fails.txt
>>> @@ -1,17 +1,6 @@
>>>   kms_3d,Fail
>>>   kms_addfb_basic@...fb25-bad-modifier,Fail
>>> -kms_cursor_legacy@...-pipes-forked-bo,Fail
>>> -kms_cursor_legacy@...-pipes-forked-move,Fail
>>> -kms_cursor_legacy@...-pipes-single-bo,Fail
>>> -kms_cursor_legacy@...-pipes-single-move,Fail
>>> -kms_cursor_legacy@...-pipes-torture-bo,Fail
>>> -kms_cursor_legacy@...-pipes-torture-move,Fail
>>> -kms_cursor_legacy@...e-A-forked-bo,Fail
>>> -kms_cursor_legacy@...e-A-forked-move,Fail
>>> -kms_cursor_legacy@...e-A-single-bo,Fail
>>> -kms_cursor_legacy@...e-A-single-move,Fail
>>> -kms_cursor_legacy@...e-A-torture-bo,Fail
>>> -kms_cursor_legacy@...e-A-torture-move,Fail
>>> +kms_cursor_legacy@...ture-bo,Fail
>>>   kms_force_connector_basic@...ce-edid,Fail
>>>   kms_hdmi_inject@...ect-4k,Fail
>>>   kms_selftest@..._format,Timeout