From: "Saarinen, Jani" <jani.saarinen@intel.com>
To: "Deak, Imre" <imre.deak@intel.com>,
"Ville Syrjälä" <ville.syrjala@linux.intel.com>,
"Mun, Gwan-gyeong" <gwan-gyeong.mun@intel.com>,
"Souza, Jose" <jose.souza@intel.com>
Cc: "igt-dev@lists.freedesktop.org" <igt-dev@lists.freedesktop.org>
Subject: Re: [igt-dev] ✗ Fi.CI.IGT: failure for lib/igt_fb: A few changes to fix test runs on ADL_P
Date: Wed, 5 May 2021 08:06:29 +0000 [thread overview]
Message-ID: <5e2745418cd04838960f6d85014bfd3f@intel.com> (raw)
In-Reply-To: <20210504185341.GC30246@ideak-desk.fi.intel.com>
+@Mun, Gwan-gyeong, @Souza, Jose. See below from Imre.
> -----Original Message-----
> From: igt-dev <igt-dev-bounces@lists.freedesktop.org> On Behalf Of Imre Deak
> Sent: tiistai 4. toukokuuta 2021 21.54
> To: Ville Syrjälä <ville.syrjala@linux.intel.com>
> Cc: igt-dev@lists.freedesktop.org
> Subject: Re: [igt-dev] ✗ Fi.CI.IGT: failure for lib/igt_fb: A few changes to fix test runs
> on ADL_P
>
> On Tue, May 04, 2021 at 06:13:11PM +0300, Ville Syrjälä wrote:
> > On Tue, May 04, 2021 at 06:00:58PM +0300, Imre Deak wrote:
> > > On Fri, Apr 30, 2021 at 12:55:28AM +0000, Patchwork wrote:
> > > > == Series Details ==
> > > >
> > > > Series: lib/igt_fb: A few changes to fix test runs on ADL_P
> > > > URL : https://patchwork.freedesktop.org/series/89668/
> > > > State : failure
> > > >
> > > > == Summary ==
> > > >
> > > > CI Bug Log - changes from CI_DRM_10028_full -> IGTPW_5771_full
> > > > ====================================================
> > > >
> > > > Summary
> > > > -------
> > > >
> > > > **FAILURE**
> > > >
> > > > Serious unknown changes coming with IGTPW_5771_full absolutely need to
> be
> > > > verified manually.
> > > >
> > > > If you think the reported changes have nothing to do with the changes
> > > > introduced in IGTPW_5771_full, 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_5771/index.html
> > > >
> > > > Possible new issues
> > > > -------------------
> > > >
> > > > Here are the unknown changes that may have been introduced in
> IGTPW_5771_full:
> > > >
> > > > ### IGT changes ###
> > > >
> > > > #### Possible regressions ####
> > > >
> > > > * igt@kms_plane@pixel-format-pipe-a-planes:
> > > > - shard-tglb: [PASS][1] -> [SKIP][2]
> > > > [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10028/shard-
> tglb3/igt@kms_plane@pixel-format-pipe-a-planes.html
> > > > [2]:
> > > > https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5771/shard-tglb7/ig
> > > > t@kms_plane@pixel-format-pipe-a-planes.html
> > >
> > > Test requirement not met in function test_format_plane, file
> ../tests/kms_plane.c:912:
> > > Test requirement: num_unique_crcs(ref_crc[PLANAR_CRC_SET],
> > > data->num_colors) > 1
> > >
> > > after a bunch of CRC mismatches, so I suppose the CRC generation
> > > gets stuck at one point.
> > >
> > > It's on the same path this patchset changes, however on TGL the
> > > patchset should have no functional change: the stride calculated by
> > > __intel_buf_init() and the stride passed to it by
> > > igt_fb_create_intel_buf() always match based on my tests. After
> > > testing the same CI testlist locally on 2 TGLs, I also rerun the
> > > testlist where this problem didn't happen:
> >
> > There's definitely something hinky going on with tgl since kms_big_fb
> > is also failing atm in ci. But dunno if that's related or not.
>
> That seems to get triggered by kms_psr2_sf followed by kms_big_fb. I could repro
> this only (consistently) on TGL/A0 (all TGL shards are like
> this) either running kms_psr2_sf/plane-move-sf-dmg-area-1 or
> kms_psr2_sf/overlay-primary-update-sf-dmg-area-4 followed by
> kms_big_fb/x-tiled-64bpp-rotate-180 .
>
> The same doesn't happen on TGL/B0 with the same panel as used by by TGL shards,
> though PSR got disabled on B0 due to some PSR RFB error.
>
> Since this problem seems to happen consistently on A0, affecting later tests I suggest
> blacklisting kms_psr2_sf at least on those machines, until the problem is root
> caused, or at least to see how the kms_big_fb results change.
>
> > --
> > Ville Syrjälä
> > Intel
> _______________________________________________
> igt-dev mailing list
> igt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
next prev parent reply other threads:[~2021-05-05 8:06 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-29 19:26 [igt-dev] [PATCH i-g-t 0/3] lib/igt_fb: A few changes to fix test runs on ADL_P Imre Deak
2021-04-29 19:26 ` [igt-dev] [PATCH i-g-t 1/3] tests/kms_ccs: Skip CCS color clear value check for randomizing subtests Imre Deak
2021-05-05 0:23 ` Rodrigo Siqueira
2021-05-05 10:27 ` Imre Deak
2021-05-05 23:07 ` Rodrigo Siqueira
2021-04-29 19:26 ` [igt-dev] [PATCH i-g-t 2/3] lib/igt_fb: Fix stride for render_copy based FB conversions Imre Deak
2021-04-29 19:51 ` Ville Syrjälä
2021-04-30 5:53 ` Zbigniew Kempczyński
2021-04-30 10:29 ` Imre Deak
2021-04-29 19:26 ` [igt-dev] [PATCH i-g-t 3/3] lib/igt_fb: Remove redundant VC4 SAND plane size alignment Imre Deak
2021-04-29 20:42 ` [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_fb: A few changes to fix test runs on ADL_P Patchwork
2021-04-30 0:55 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
2021-05-04 15:00 ` Imre Deak
2021-05-04 15:13 ` Ville Syrjälä
2021-05-04 18:53 ` Imre Deak
2021-05-05 8:06 ` Saarinen, Jani [this message]
2021-05-05 21:03 ` Souza, Jose
2021-05-04 9:39 ` [igt-dev] [PATCH i-g-t 0/3] " Juha-Pekka Heikkila
2021-05-04 12:07 ` [igt-dev] ✓ Fi.CI.BAT: success for lib/igt_fb: A few changes to fix test runs on ADL_P (rev2) Patchwork
2021-05-04 13:45 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5e2745418cd04838960f6d85014bfd3f@intel.com \
--to=jani.saarinen@intel.com \
--cc=gwan-gyeong.mun@intel.com \
--cc=igt-dev@lists.freedesktop.org \
--cc=imre.deak@intel.com \
--cc=jose.souza@intel.com \
--cc=ville.syrjala@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox