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

[Vignesh Raman]

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>
> > > ---