Re: [PATCH v6 2/2] intel-ci: Blocklist compression PAT index for LNL

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

On Mon, 15 Sep 2025 09:39:41 -0700, Dixit, Ashutosh wrote:
>
> On Mon, 15 Sep 2025 05:08:29 -0700, Kamil Konieczny wrote:
> >
> > Hi pravalika.gurram,
> > On 2025-09-15 at 16:20:57 +0530, pravalika.gurram@intel.com wrote:
> > > From: Pravalika Gurram <pravalika.gurram@intel.com>
> > >
> > > Madvise does not support compression on LNL.
> > > Therefore, compression-related PAT index
> > > test cases have been blocklisted for this platform.
> > >
> > > Signed-off-by: Pravalika Gurram <pravalika.gurram@intel.com>
> >
> > Acked-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
>
> What is the guidance for when we blocklist a test vs. when we skip it,
> using igt_skip or igt_require?

IMO we should skip it in the code, so anyone looking at the code will know
the test doesn't work on LNL, without having to look at the blocklists.

Blocklisting concerns CI, so we may or may not want to blocklist it for CI
(in addition to skipping in code).

>
> >
> > > ---
> > >  tests/intel-ci/xe.lnl.core.blocklist.txt | 6 ++++++
> > >  1 file changed, 6 insertions(+)
> > >
> > > diff --git a/tests/intel-ci/xe.lnl.core.blocklist.txt b/tests/intel-ci/xe.lnl.core.blocklist.txt
> > > index 1e69170d8..3a21d7263 100644
> > > --- a/tests/intel-ci/xe.lnl.core.blocklist.txt
> > > +++ b/tests/intel-ci/xe.lnl.core.blocklist.txt
> > > @@ -83,3 +83,9 @@ igt@xe_live_ktest@xe_eudebug
> > >  igt@xe_oa@oa-tlb-invalidate
> > >  igt@xe_oa@unprivileged-single-ctx-counters
> > >  ##################################################################
> > > +# Expected fail as per the current behaviour compression
> > > +# for madvise needs to skipped on LNL
> > > +##################################################################
> > > +igt@xe_exec_system_allocator@pat-index-madvise-pat-idx-wt-.*
> > > +igt@xe_exec_system_allocator@pat-index-madvise-pat-idx-uc-comp-.*
> > > +##################################################################
> > > --
> > > 2.34.1
> > >