[PATCH i-g-t] tests/intel/xe_evict: Skip test if smem less than lmem

[Kamil Konieczny]

Hi priyanka.dandamudi,
On 2024-03-12 at 12:45:02 +0530, priyanka.dandamudi at intel.com wrote:
> From: Priyanka Dandamudi <priyanka.dandamudi at intel.com>
> 
> Currently the tests are failing with OOO memory error.
> Added code to skip the test if the system memory size is less than local
> memory size.
> 
> Cc: Matthew Brost <matthew.brost at intel.com>
> Cc: Venkata Ramana Nayana <venkata.ramana.nayana at intel.com>
> Signed-off-by: Priyanka Dandamudi <priyanka.dandamudi at intel.com>
> ---
>  tests/intel/xe_evict.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/intel/xe_evict.c b/tests/intel/xe_evict.c
> index 4de82c3bd..2dc3fe542 100644
> --- a/tests/intel/xe_evict.c
> +++ b/tests/intel/xe_evict.c
> @@ -753,6 +753,8 @@ igt_main
>  		igt_require(xe_has_vram(fd));
>  		vram_size = xe_visible_vram_size(fd, 0);
>  		igt_assert(vram_size);
> +		igt_skip_on_f(igt_get_avail_ram_mb() < (vram_size >> 20),
> +			      "System memory is less than local memory \n");

Could you add here actual numbers, possibly in GB (if they diff in GB)?
Also, maybe it is worth add a TODO and restrict memory tested in such
a case? I mean that only part which will fit in SMEM will be evicted?

Regards,
Kamil
>  
>  		xe_for_each_engine(fd, hwe)
>  			if (hwe->engine_class != DRM_XE_ENGINE_CLASS_COPY)
> -- 
> 2.25.1
>