Re: [PATCH v1 3/4] drm/ci: uprev IGT and generate testlist from build

From: Vignesh Raman
Date: Tue Apr 23 2024 - 08:57:18 EST


Hi,

On 23/04/24 17:53, Dmitry Baryshkov wrote:
On Tue, 23 Apr 2024 at 13:24, Maíra Canal <mcanal@xxxxxxxxxx> wrote:

On 4/23/24 01:02, Vignesh Raman wrote:
Uprev IGT to the latest version and stop vendoring the
testlist into the kernel. Instead, use the testlist from
the IGT build to ensure we do not miss renamed or newly
added tests.

Nitpick: wouldn't it be better to (1) stop vendoring the
testlist into the kernel in one patch and then (2) uprev
IGT to the latest version? I feel that this patch is changing
a lot of different stuff.

Definitely. Otherwise it's hard to understand whether a change to
fails/flakes is caused by the uprev or by the testlist change. E.g. in
several cases the list of failing subtests seems to be replaced with
the test itself.


Yes, will follow the below approach.
1) Use the testlist from build and update fails/flakes/skips
2) Uprev IGT and update fails/flakes/skips

Regards,
Vignesh



Skip kms tests for panfrost driver since it is not a kms
driver and skip driver-specific tests. Update xfails with
the latest testlist.

Increase the timoout of i915 jobs to 2h30m since some jobs
takes more than 2 hours to complete.

Signed-off-by: Vignesh Raman <vignesh.raman@xxxxxxxxxxxxx>
---