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

Kamil Konieczny kamil.konieczny at linux.intel.com
Wed Dec 18 14:18:39 UTC 2024


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?

Regards,
Kamil

> +##################################################################
> -- 
> 2.47.1
> 


More information about the igt-dev mailing list