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

[Petri Latvala]

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