Re: [PATCH i-g-t 0/2] tests/intel/xe_evict: Adapt the working set to memory size

[Matthew Brost <matthew.brost@intel.com>]

On Wed, Jun 26, 2024 at 02:38:31PM +0200, Thomas Hellström wrote:
> It turns out that xe_evict in the cm and thread mode is a very good test for
> exhaustive eviction. But the test sometimes uses a working set that doesn't fit

Do to hear that is it is a good test.

Have you tried [1], in particular *evict* sections. I wrote this as an
additional test which also fully checks the integrity of memory as it
gets bounced around. I think this test has value too. It might have a
similar problem to what you fixing in this series though. I think it
worth fixing and getting merged if you may review. I can work on fixing
it if think this has value too so it can be merged.

Anyways will review this series as soon as I can.

Matt

[1] https://patchwork.freedesktop.org/patch/588613/?series=132251&rev=1

> in gpu-accessible memory and a total memory size that exceeds the available
> system size. Attempt to fix that to avoid false failures. With these fixes
> the test should pass unless in -cm mode or multithreaded mode. These require
> fixes for exhaustive eviction in the xe kernel driver.
> 
> Thomas Hellström (2):
>   tests/intel/xe_evict: Reduce allocations to maximum working set
>   tests/intel/xe_evict: Reduce the "large" bo size for threaded eviction
> 
>  tests/intel/xe_evict.c | 126 ++++++++++++++++++++++++++++++++---------
>  1 file changed, 99 insertions(+), 27 deletions(-)
> 
> -- 
> 2.44.0
>