[PATCH i-g-t v2 2/2] tests/xe_ccs: Simplify subtest skip condition

Zbigniew Kempczyński zbigniew.kempczynski at intel.com
Wed Mar 13 10:45:20 UTC 2024


Newer platforms (Xe2) supports compression on system memory so subtests
which might be executed are not only limited to discrete/vram. Enabling
dynamic subtest in ctrl-surf-copy test depends on surface compression
property. Lets use introduced in previous patch helper which returns
does region supports surface compression.

Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
Cc: Matthew Auld <matthew.auld at intel.com>
---
 tests/intel/xe_ccs.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/intel/xe_ccs.c b/tests/intel/xe_ccs.c
index 5a79642c9d..d6f5408a78 100644
--- a/tests/intel/xe_ccs.c
+++ b/tests/intel/xe_ccs.c
@@ -12,6 +12,7 @@
 #include "igt.h"
 #include "igt_syncobj.h"
 #include "intel_blt.h"
+#include "intel_common.h"
 #include "intel_mocs.h"
 #include "intel_pat.h"
 #include "xe/xe_ioctl.h"
@@ -618,8 +619,7 @@ static void block_copy_test(int xe,
 			region2 = igt_collection_get_value(regions, 1);
 
 			/* if not XE2, then Compressed surface must be in device memory */
-			if (config->compression && !(AT_LEAST_GEN((intel_get_drm_devid(xe)), 20)) &&
-									!XE_IS_VRAM_MEMORY_REGION(xe, region2))
+			if (config->compression && !is_intel_region_compressible(xe, region2))
 				continue;
 
 			regtxt = xe_memregion_dynamic_subtest_name(xe, regions);
-- 
2.34.1



More information about the igt-dev mailing list