[igt-dev] [PATCH v1 0/1] test/gem_create: limit set_pat to GEN12+ platforms

[igt-dev] [PATCH v1 0/1] test/gem_create: limit set_pat to GEN12+ platforms

[Andi Shyti]

From: Fei Yang <fei.yang@intel.com>

An agreement was reached to expose gem_create_ext_set_pat extension
for GEN12+ platforms only. For older platforms the extension would
return -ENODEV.

Fei Yang (1):
  test/gem_create: limit set_pat to GEN12+ platforms

 tests/i915/gem_create.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

-- 
2.25.1

[igt-dev] [PATCH v1 1/1] test/gem_create: limit set_pat to GEN12+ platforms

[Andi Shyti]

From: Fei Yang <fei.yang@intel.com>

For platforms prior to GEN12 the set_pat extension should
return -ENODEV.

Signed-off-by: Fei Yang <fei.yang@intel.com>
Signed-off-by: Andi Shyti <andi.shyti@linux.intel.com>
---
 tests/i915/gem_create.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/tests/i915/gem_create.c b/tests/i915/gem_create.c
index e9e8216d..1553512f 100644
--- a/tests/i915/gem_create.c
+++ b/tests/i915/gem_create.c
@@ -657,6 +657,7 @@ static void create_ext_set_pat(int fd)
 		.base = { .name = I915_GEM_CREATE_EXT_SET_PAT },
 		.pat_index = 65,
 	};
+	uint32_t devid = intel_get_drm_devid(fd);
 	struct drm_i915_gem_caching arg;
 	uint64_t size;
 	uint32_t handle;
@@ -670,7 +671,11 @@ static void create_ext_set_pat(int fd)
 	 * With a valid PAT index specified, returning -EINVAL here
 	 * indicates set_pat extension is not supported
 	 */
-	if (ret == -EINVAL)
+	if (intel_graphics_ver(devid) < IP_VER(12, 0)) {
+		igt_assert(ret == -ENODEV);
+		return;
+	}
+	else if (ret == -EINVAL)
 		igt_skip("I915_GEM_CREATE_EXT_SET_PAT is not supported\n");
 	igt_assert(ret == 0);
 
-- 
2.25.1