[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