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

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

[venkata.sai.patnana]

From: Patnana Venkata Sai <venkata.sai.patnana@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@intel.com>
Cc: Petri Latvala <petri.latvala@intel.com>
Signed-off-by: Patnana Venkata Sai <venkata.sai.patnana@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@gem_tiled_blits@normal
 igt@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@kms_plane@pixel-format-pipe-[b-d]-planes(-source-clamping)?
-
-
 ###############################################################################
 # This test is doing nothing more than waiting for the driver to be suspended
 # before issueing a modeset. However, it never failed while testing for this
-- 
2.25.1

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

[igt-dev] ✗ Fi.CI.BAT: failure for intel-ci: Remove pixel-format-source-clamping@* from the pre-merge blacklist

[Patchwork]

[-- Attachment #1.1: Type: text/plain, Size: 6416 bytes --]

== Series Details ==

Series: intel-ci: Remove pixel-format-source-clamping@* from the pre-merge blacklist
URL   : https://patchwork.freedesktop.org/series/91180/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_10193 -> IGTPW_5900
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_5900 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_5900, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/index.html

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in IGTPW_5900:

### IGT changes ###

#### Possible regressions ####

  * igt@dmabuf@all@dma_fence_chain:
    - fi-hsw-4770:        [PASS][1] -> [FAIL][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10193/fi-hsw-4770/igt@dmabuf@all@dma_fence_chain.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-hsw-4770/igt@dmabuf@all@dma_fence_chain.html

  
Known issues
------------

  Here are the changes found in IGTPW_5900 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@runner@aborted:
    - fi-hsw-4770:        NOTRUN -> [FAIL][3] ([i915#2505])
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-hsw-4770/igt@runner@aborted.html

  
#### Possible fixes ####

  * igt@kms_chamelium@dp-crc-fast:
    - fi-kbl-7500u:       [FAIL][4] ([i915#1372]) -> [PASS][5]
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10193/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-kbl-7500u/igt@kms_chamelium@dp-crc-fast.html

  
#### Warnings ####

  * igt@i915_selftest@live@execlists:
    - fi-icl-u2:          [DMESG-FAIL][6] ([i915#3462]) -> [INCOMPLETE][7] ([i915#2782] / [i915#3462])
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10193/fi-icl-u2/igt@i915_selftest@live@execlists.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-icl-u2/igt@i915_selftest@live@execlists.html
    - fi-tgl-u2:          [DMESG-FAIL][8] ([i915#3462]) -> [INCOMPLETE][9] ([i915#3462])
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10193/fi-tgl-u2/igt@i915_selftest@live@execlists.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-tgl-u2/igt@i915_selftest@live@execlists.html
    - fi-cml-s:           [DMESG-FAIL][10] ([i915#3462]) -> [INCOMPLETE][11] ([i915#3462])
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10193/fi-cml-s/igt@i915_selftest@live@execlists.html
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-cml-s/igt@i915_selftest@live@execlists.html

  * igt@runner@aborted:
    - fi-skl-6600u:       [FAIL][12] ([i915#1436] / [i915#3363]) -> [FAIL][13] ([i915#1436] / [i915#2426] / [i915#3363])
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10193/fi-skl-6600u/igt@runner@aborted.html
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-skl-6600u/igt@runner@aborted.html
    - fi-icl-u2:          [FAIL][14] ([i915#2426] / [i915#2782] / [i915#3363]) -> [FAIL][15] ([i915#2782] / [i915#3363])
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10193/fi-icl-u2/igt@runner@aborted.html
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-icl-u2/igt@runner@aborted.html
    - fi-glk-dsi:         [FAIL][16] ([i915#3363] / [k.org#202321]) -> [FAIL][17] ([i915#2426] / [i915#3363] / [k.org#202321])
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10193/fi-glk-dsi/igt@runner@aborted.html
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-glk-dsi/igt@runner@aborted.html
    - fi-kbl-7500u:       [FAIL][18] ([i915#1436] / [i915#3363]) -> [FAIL][19] ([i915#1436] / [i915#2426] / [i915#3363])
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10193/fi-kbl-7500u/igt@runner@aborted.html
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-kbl-7500u/igt@runner@aborted.html
    - fi-cml-u2:          [FAIL][20] ([i915#2082] / [i915#2426] / [i915#3363] / [i915#3462]) -> [FAIL][21] ([i915#3363] / [i915#3462])
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10193/fi-cml-u2/igt@runner@aborted.html
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-cml-u2/igt@runner@aborted.html
    - fi-cfl-guc:         [FAIL][22] ([i915#3363]) -> [FAIL][23] ([i915#2426] / [i915#3363])
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10193/fi-cfl-guc/igt@runner@aborted.html
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/fi-cfl-guc/igt@runner@aborted.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [i915#1222]: https://gitlab.freedesktop.org/drm/intel/issues/1222
  [i915#1372]: https://gitlab.freedesktop.org/drm/intel/issues/1372
  [i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
  [i915#2082]: https://gitlab.freedesktop.org/drm/intel/issues/2082
  [i915#2426]: https://gitlab.freedesktop.org/drm/intel/issues/2426
  [i915#2505]: https://gitlab.freedesktop.org/drm/intel/issues/2505
  [i915#2782]: https://gitlab.freedesktop.org/drm/intel/issues/2782
  [i915#2932]: https://gitlab.freedesktop.org/drm/intel/issues/2932
  [i915#2966]: https://gitlab.freedesktop.org/drm/intel/issues/2966
  [i915#3363]: https://gitlab.freedesktop.org/drm/intel/issues/3363
  [i915#3462]: https://gitlab.freedesktop.org/drm/intel/issues/3462
  [k.org#202321]: https://bugzilla.kernel.org/show_bug.cgi?id=202321


Participating hosts (46 -> 40)
------------------------------

  Missing    (6): fi-rkl-11500t fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-dg1-1 fi-bdw-samus 


Build changes
-------------

  * CI: CI-20190529 -> None
  * IGT: IGT_6102 -> IGTPW_5900

  CI-20190529: 20190529
  CI_DRM_10193: 05a52983a3bd69a42157b818d3d83f457f26712a @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_5900: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/index.html
  IGT_6102: 6e67969bf93dda8f22773ccae362f757fce25c3d @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5900/index.html

[-- Attachment #1.2: Type: text/html, Size: 8910 bytes --]

[-- Attachment #2: Type: text/plain, Size: 154 bytes --]

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [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@intel.com wrote:
> From: Patnana Venkata Sai <venkata.sai.patnana@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@intel.com>
> Cc: Petri Latvala <petri.latvala@intel.com>
> Signed-off-by: Patnana Venkata Sai <venkata.sai.patnana@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@gem_tiled_blits@normal
>  igt@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@kms_plane@pixel-format-pipe-[b-d]-planes(-source-clamping)?
> -
> -


Well now.

The change itself is

Acked-by: Petri Latvala <petri.latvala@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
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

Re: [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@intel.com>
> Sent: Wednesday, June 9, 2021 10:19 AM
> To: Patnana, Venkata Sai <venkata.sai.patnana@intel.com>
> Cc: igt-dev@lists.freedesktop.org; B S, Karthik <karthik.b.s@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@intel.com
> wrote:
> > From: Patnana Venkata Sai <venkata.sai.patnana@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@intel.com>
> > Cc: Petri Latvala <petri.latvala@intel.com>
> > Signed-off-by: Patnana Venkata Sai <venkata.sai.patnana@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@gem_tiled_blits@normal  igt@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@kms_plane@pixel-format-pipe-[b-d]-planes(-source-clamping)?
> > -
> > -
> 
> 
> Well now.
> 
> The change itself is
> 
> Acked-by: Petri Latvala <petri.latvala@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
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

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

[Petri Latvala]

On Wed, Jun 09, 2021 at 07:48:38AM +0300, Petri Latvala wrote:
> On Tue, Jun 08, 2021 at 08:45:52PM +0530, venkata.sai.patnana@intel.com wrote:
> > From: Patnana Venkata Sai <venkata.sai.patnana@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@intel.com>
> > Cc: Petri Latvala <petri.latvala@intel.com>
> > Signed-off-by: Patnana Venkata Sai <venkata.sai.patnana@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@gem_tiled_blits@normal
> >  igt@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@kms_plane@pixel-format-pipe-[b-d]-planes(-source-clamping)?
> > -
> > -
> 
> 
> Well now.
> 
> The change itself is
> 
> Acked-by: Petri Latvala <petri.latvala@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.
> ===


Verified offline that this text is good. Patch is merged, thanks!


-- 
Petri Latvala
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev