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

[Ch, Sai Gowtham]

>-----Original Message-----
>From: igt-dev <igt-dev-bounces at lists.freedesktop.org> On Behalf Of
>priyanka.dandamudi at intel.com
>Sent: Tuesday, March 12, 2024 12:45 PM
>To: Dandamudi, Priyanka <priyanka.dandamudi at intel.com>; Nayana, Venkata
>Ramana <venkata.ramana.nayana at intel.com>; Brost, Matthew
><matthew.brost at intel.com>; igt-dev at lists.freedesktop.org
>Subject: [PATCH i-g-t] tests/intel/xe_evict: Skip test if smem less than lmem
>
>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");
Can you elaborate the commit message a bit more ?
Looks good to me
Reviewed-by: Sai Gowtham Ch <sai.gowtham.ch at intel.com>
>
> 		xe_for_each_engine(fd, hwe)
> 			if (hwe->engine_class != DRM_XE_ENGINE_CLASS_COPY)
>--
>2.25.1