[PATCH i-g-t] xe-ci: Disable xe_evict subtests that are expected to possibly fail

[Thomas Hellström]

On Wed, 2024-12-18 at 15:18 +0100, Kamil Konieczny wrote:
> Hi Thomas,
> On 2024-12-18 at 13:45:19 +0100, Thomas Hellström wrote:
> > Threads may compete over VRAM resources, and since there is no
> > serialization in TTM, threaded evict tests may have a high
> > possibility of failing.
> > 
> > Disable such tests until we have a working exhaustive eviction
> > implementation in place.
> > 
> > Signed-off-by: Thomas Hellström <thomas.hellstrom at linux.intel.com>
> > Cc: Matthew Brost <matthew.brost at intel.com>
> > Cc: <akshata.jahagirdar at intel.com>
> 
> Looks reasonable, please only change prefix into
> tests/intel-ci/xe*:
> 
> also consider writing it a little shorter, like:
> [PATCH i-g-t] tests/intel-ci/xe*: Disable few xe_evict subtests
> 
> With this
> Acked-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> 
> Please also see one nit below.
> 
> > ---
> >  tests/intel-ci/xe-fast-feedback.testlist | 5 -----
> >  tests/intel-ci/xe.blocklist.txt          | 7 +++++++
> >  2 files changed, 7 insertions(+), 5 deletions(-)
> > 
> > diff --git a/tests/intel-ci/xe-fast-feedback.testlist
> > b/tests/intel-ci/xe-fast-feedback.testlist
> > index 727be9230..f16ca9188 100644
> > --- a/tests/intel-ci/xe-fast-feedback.testlist
> > +++ b/tests/intel-ci/xe-fast-feedback.testlist
> > @@ -263,19 +263,14 @@
> > igt at xe_exec_threads@threads-mixed-shared-vm-basic
> >  igt at xe_exec_threads@threads-mixed-fd-basic
> >  igt at xe_exec_threads@threads-mixed-userptr-invalidate
> >  
> > igt at xe_exec_threads@threads-mixed-shared-vm-userptr-invalidate-race
> > -igt at xe_evict@evict-beng-mixed-threads-small-multi-vm
> >  igt at xe_evict@evict-beng-small
> >  igt at xe_evict@evict-beng-small-cm
> >  igt at xe_evict@evict-beng-small-external
> >  igt at xe_evict@evict-beng-small-external-cm
> >  igt at xe_evict@evict-beng-small-multi-vm
> >  igt at xe_evict@evict-cm-threads-small
> > -igt at xe_evict@evict-mixed-threads-small
> > -igt at xe_evict@evict-mixed-threads-small-multi-vm
> >  igt at xe_evict@evict-small
> >  igt at xe_evict@evict-small-cm
> >  igt at xe_evict@evict-small-external
> >  igt at xe_evict@evict-small-external-cm
> >  igt at xe_evict@evict-small-multi-vm
> > -igt at xe_evict@evict-small-multi-vm-cm
> > -igt at xe_evict@evict-threads-small
> > diff --git a/tests/intel-ci/xe.blocklist.txt b/tests/intel-
> > ci/xe.blocklist.txt
> > index e29e745a0..ed9e572a5 100644
> > --- a/tests/intel-ci/xe.blocklist.txt
> > +++ b/tests/intel-ci/xe.blocklist.txt
> > @@ -90,3 +90,10 @@ igt at sriov_basic@enable-vfs-autoprobe-on$
> >  igt at sriov_basic@enable-vfs-bind-unbind-each$
> >  igt at sriov_basic@bind-unbind-vf$
> >  ##################################################################
> > +# Exclude xe_evict tests that are not expected to succeed
> > +# due to missing TTM exhaustive eviction
> > +##################################################################
> > +igt at xe_evict@*-multi-vm-cm*
> > +igt at xe_evict@*-threads-*
> > +# igt at xe_evict@*-cm* # Rebind and VM_BIND may race.
> 
> Did you mean that
> igt at xe_evict@evict-cm-threads-small
> 
> should stay in BAT?

No it shouldn't. Good catch. I'll update.

> 
> Regards,
> Kamil
> 
> > +##################################################################
> > -- 
> > 2.47.1
> >