[igt-dev] [PATCH i-g-t] intel-ci: Remove pixel-format-source-clamping@* from the pre-merge blacklist

[Patnana, Venkata Sai]

I ll change commit text test as you suggested.

Thanks
Sai

> -----Original Message-----
> From: Latvala, Petri <petri.latvala at intel.com>
> Sent: Wednesday, June 9, 2021 10:19 AM
> To: Patnana, Venkata Sai <venkata.sai.patnana at intel.com>
> Cc: igt-dev at lists.freedesktop.org; B S, Karthik <karthik.b.s at intel.com>
> Subject: Re: [PATCH i-g-t] intel-ci: Remove pixel-format-source-clamping@*
> from the pre-merge blacklist
> 
> On Tue, Jun 08, 2021 at 08:45:52PM +0530, venkata.sai.patnana at intel.com
> wrote:
> > From: Patnana Venkata Sai <venkata.sai.patnana at intel.com>
> >
> > This subtest currently taking less time ~20s. So removing this subtest
> > from pre-merge ci blacklist file.
> >
> > Cc: Karthik B S <karthik.b.s at intel.com>
> > Cc: Petri Latvala <petri.latvala at intel.com>
> > Signed-off-by: Patnana Venkata Sai <venkata.sai.patnana at intel.com>
> > ---
> >  tests/intel-ci/blacklist-pre-merge.txt | 20 --------------------
> >  1 file changed, 20 deletions(-)
> >
> > diff --git a/tests/intel-ci/blacklist-pre-merge.txt
> > b/tests/intel-ci/blacklist-pre-merge.txt
> > index 6363a7ac86..115474d987 100644
> > --- a/tests/intel-ci/blacklist-pre-merge.txt
> > +++ b/tests/intel-ci/blacklist-pre-merge.txt
> > @@ -153,26 +153,6 @@ igt at gem_tiled_blits@normal  igt at gem_tiled_wc
> >
> >
> > -
> #################################################################
> ####
> > ########## -# This is a useful test, but it mostly tests the HW rather
> > than the driver.
> > -# Very few regressions should be caught by this test as the driver
> > code should -# be relatively left untouched. Hopefully, it will get
> > optimized to be made -# useful in pre-merge as well:
> > -#
> > -# - shard-skl: 1% (~2.5 minutes)
> > -# - shard-kbl: 1.5% (~2 minutes)
> > -# - shard-apl: 1.4% (~2.5 minutes)
> > -# - shard-glk: 2% (~4.5 minutes)
> > -# - shard-icl: 2.7% (~5.5 minutes)
> > -# - shard-tgl: 2.3% (~4.5 minutes)
> > -#
> > -# Issue: https://gitlab.freedesktop.org/drm/intel/issues/1287
> > -#
> > -# Data acquired on 2020-02-20 by Martin Peres
> > -
> #################################################################
> ####
> > ##########
> > -igt at kms_plane@pixel-format-pipe-[b-d]-planes(-source-clamping)?
> > -
> > -
> 
> 
> Well now.
> 
> The change itself is
> 
> Acked-by: Petri Latvala <petri.latvala at intel.com>
> 
> But the commit message needs changing to something like
> 
> ===
> After commit d28aee5c5f52 ("tests/kms_plane: Restrict the test execution to
> two pipes") these subtests got renamed, and along with the rename no longer
> takes entire minutes to execute. Enable their execution on premerge.
> ===
> 
> 
> --
> Petri Latvala