[Intel-gfx] [PATCH i-g-t] i915: Remove gem_mocs_settings

Chris Wilson chris at chris-wilson.co.uk
Fri Nov 15 10:32:16 UTC 2019


This igt relies on using privileged batches to read and modify protected
registers. This is no longer supported and so the test is entirely
defunct, and for as much as we can has been subsumed into a kernel
selftest.

Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
Cc: Martin Peres <martin.peres at linux.intel.com>
---
 tests/Makefile.sources         |   3 -
 tests/i915/gem_mocs_settings.c | 641 ---------------------------------
 tests/intel-ci/blacklist.txt   |   1 -
 tests/meson.build              |   8 -
 4 files changed, 653 deletions(-)
 delete mode 100644 tests/i915/gem_mocs_settings.c

diff --git a/tests/Makefile.sources b/tests/Makefile.sources
index 27801c89c..f2dcca6d7 100644
--- a/tests/Makefile.sources
+++ b/tests/Makefile.sources
@@ -326,9 +326,6 @@ gem_mmap_offset_exhaustion_SOURCES = i915/gem_mmap_offset_exhaustion.c
 TESTS_progs += gem_mmap_wc
 gem_mmap_wc_SOURCES = i915/gem_mmap_wc.c
 
-TESTS_progs += gem_mocs_settings
-gem_mocs_settings_SOURCES = i915/gem_mocs_settings.c
-
 TESTS_progs += gem_partial_pwrite_pread
 gem_partial_pwrite_pread_SOURCES = i915/gem_partial_pwrite_pread.c
 
diff --git a/tests/i915/gem_mocs_settings.c b/tests/i915/gem_mocs_settings.c
deleted file mode 100644
index 60f7037fc..000000000
--- a/tests/i915/gem_mocs_settings.c
+++ /dev/null
@@ -1,641 +0,0 @@
-/*
- * Copyright © 2016 Intel Corporation
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice (including the next
- * paragraph) shall be included in all copies or substantial portions of the
- * Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
- * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
- * IN THE SOFTWARE.
- *
- */
-
-/** @file gem_mocs_settings.c
- *
- * Check that the MOCs cache settings are valid.
- */
-
-#include "igt.h"
-#include "igt_gt.h"
-#include "igt_perf.h"
-#include "igt_sysfs.h"
-
-#define GEN9_NUM_MOCS_ENTRIES   62  /* 62 out of 64 - 63 & 64 are reserved. */
-#define GEN11_NUM_MOCS_ENTRIES  64  /* 63-64 are reserved, but configured. */
-
-enum {
-	NONE,
-	RESET,
-	RC6,
-	SUSPEND,
-	HIBERNATE,
-	MAX_MOCS_TEST_MODES
-};
-
-static const char * const test_modes[] = {
-	[NONE] = "settings",
-	[RESET] = "reset",
-	[RC6] = "rc6",
-	[SUSPEND] = "suspend",
-	[HIBERNATE] = "hibernate"
-};
-
-#define MOCS_NON_DEFAULT_CTX	(1<<0)
-#define MOCS_DIRTY_VALUES	(1<<1)
-#define ALL_MOCS_FLAGS		(MOCS_NON_DEFAULT_CTX | \
-				 MOCS_DIRTY_VALUES)
-
-#define GEN9_LNCFCMOCS0		(0xB020)	/* L3 Cache Control base */
-#define GEN9_GFX_MOCS_0		(0xc800)	/* Graphics MOCS base register*/
-#define GEN9_MFX0_MOCS_0	(0xc900)	/* Media 0 MOCS base register*/
-#define GEN9_MFX1_MOCS_0	(0xcA00)	/* Media 1 MOCS base register*/
-#define GEN9_VEBOX_MOCS_0	(0xcB00)	/* Video MOCS base register*/
-#define GEN9_BLT_MOCS_0		(0xcc00)	/* Blitter MOCS base register*/
-#define GEN12_GLOBAL_MOCS	(0x4000)
-#define ICELAKE_MOCS_PTE	{0x00000004, 0x0030, 0x1}
-#define MOCS_PTE		{0x00000038, 0x0030, 0x1}
-
-struct mocs_entry {
-	uint32_t	control_value;
-	uint16_t	l3cc_value;
-	uint8_t 	used;
-};
-
-struct mocs_table {
-	uint32_t		size;
-	const struct mocs_entry	*table;
-};
-
-/* The first entries in the MOCS tables are defined by uABI */
-
-static const struct mocs_entry tigerlake_mocs_table[GEN11_NUM_MOCS_ENTRIES] = {
-	[2]  = { 0x00000037, 0x0030, 0x1},
-	[3]  = { 0x00000005, 0x0010, 0x1},
-	[4]  = { 0x00000005, 0x0030, 0x1},
-	[5]  = { 0x00000037, 0x0010, 0x1},
-	[6]  = { 0x00000017, 0x0010, 0x1},
-	[7]  = { 0x00000017, 0x0030, 0x1},
-	[8]  = { 0x00000027, 0x0010, 0x1},
-	[9]  = { 0x00000027, 0x0030, 0x1},
-	[10] = { 0x00000077, 0x0010, 0x1},
-	[11] = { 0x00000077, 0x0030, 0x1},
-	[12] = { 0x00000057, 0x0010, 0x1},
-	[13] = { 0x00000057, 0x0030, 0x1},
-	[14] = { 0x00000067, 0x0010, 0x1},
-	[15] = { 0x00000067, 0x0030, 0x1},
-	[18] = { 0x00060037, 0x0030, 0x1},
-	[19] = { 0x00000737, 0x0030, 0x1},
-	[20] = { 0x00000337, 0x0030, 0x1},
-	[21] = { 0x00000137, 0x0030, 0x1},
-	[22] = { 0x000003b7, 0x0030, 0x1},
-	[23] = { 0x000007b7, 0x0030, 0x1},
-	[48] = { 0x00000037, 0x0030, 0x1},
-	[49] = { 0x00000005, 0x0030, 0x1},
-	[50] = { 0x00000037, 0x0010, 0x1},
-	[51] = { 0x00000005, 0x0010, 0x1},
-	[60] = { 0x00000037, 0x0010, 0x1},
-	[61] = { 0x00000005, 0x0030, 0x1},
-	[62] = { 0x00000037, 0x0010, 0x1},
-	[63] = { 0x00000037, 0x0010, 0x1},
-};
-
-static const struct mocs_entry icelake_mocs_table[GEN11_NUM_MOCS_ENTRIES] = {
-	[0]  = { 0x00000005, 0x0010, 0x1},
-	[1]  = ICELAKE_MOCS_PTE,
-	[2]  = { 0x00000037, 0x0030, 0x1},
-	[3]  = { 0x00000005, 0x0010, 0x1},
-	[4]  = { 0x00000005, 0x0030, 0x1},
-	[5]  = { 0x00000037, 0x0010, 0x1},
-	[6]  = { 0x00000017, 0x0010, 0x1},
-	[7]  = { 0x00000017, 0x0030, 0x1},
-	[8]  = { 0x00000027, 0x0010, 0x1},
-	[9]  = { 0x00000027, 0x0030, 0x1},
-	[10] = { 0x00000077, 0x0010, 0x1},
-	[11] = { 0x00000077, 0x0030, 0x1},
-	[12] = { 0x00000057, 0x0010, 0x1},
-	[13] = { 0x00000057, 0x0030, 0x1},
-	[14] = { 0x00000067, 0x0010, 0x1},
-	[15] = { 0x00000067, 0x0030, 0x1},
-	[18] = { 0x00060037, 0x0030, 0x1},
-	[19] = { 0x00000737, 0x0030, 0x1},
-	[20] = { 0x00000337, 0x0030, 0x1},
-	[21] = { 0x00000137, 0x0030, 0x1},
-	[22] = { 0x000003b7, 0x0030, 0x1},
-	[23] = { 0x000007b7, 0x0030, 0x1},
-	[62] = { 0x00000037, 0x0010, 0x1},
-	[63] = { 0x00000037, 0x0010, 0x1},
-};
-
-static const struct mocs_entry skylake_mocs_table[GEN9_NUM_MOCS_ENTRIES] = {
-	[0] = { 0x00000009, 0x0010, 0x1},
-	[1] = MOCS_PTE,
-	[2] = { 0x0000003b, 0x0030, 0x1},
-	[3 ... GEN9_NUM_MOCS_ENTRIES - 1] = MOCS_PTE,
-};
-
-static const struct mocs_entry dirty_skylake_mocs_table[GEN9_NUM_MOCS_ENTRIES] = {
-	[0 ... GEN9_NUM_MOCS_ENTRIES - 1] = { 0x00003FFF, 0x003F, 0x1 },
-};
-
-static const struct mocs_entry broxton_mocs_table[GEN9_NUM_MOCS_ENTRIES] = {
-	[0] = { 0x00000009, 0x0010, 0x1},
-	[1] = MOCS_PTE,
-	[2] = { 0x00000039, 0x0030, 0x1},
-	[3 ... GEN9_NUM_MOCS_ENTRIES - 1] = MOCS_PTE,
-};
-
-static const struct mocs_entry dirty_broxton_mocs_table[GEN9_NUM_MOCS_ENTRIES] = {
-	[0 ... GEN9_NUM_MOCS_ENTRIES - 1] = { 0x00007FFF, 0x003F, 0x1 },
-};
-
-static const uint32_t write_values[GEN9_NUM_MOCS_ENTRIES] = {
-	[0 ... GEN9_NUM_MOCS_ENTRIES - 1] = 0xFFFFFFFF,
-};
-
-static bool has_global_mocs(int fd)
-{
-	return intel_gen(intel_get_drm_devid(fd)) >= 12;
-}
-
-static bool get_mocs_settings(int fd, struct mocs_table *table, bool dirty)
-{
-	uint32_t devid = intel_get_drm_devid(fd);
-	bool result = false;
-
-	if (IS_SKYLAKE(devid) || IS_KABYLAKE(devid) || IS_COMETLAKE(devid)) {
-		if (dirty) {
-			table->size  = ARRAY_SIZE(dirty_skylake_mocs_table);
-			table->table = dirty_skylake_mocs_table;
-		} else {
-			table->size  = ARRAY_SIZE(skylake_mocs_table);
-			table->table = skylake_mocs_table;
-		}
-		result = true;
-	} else if (IS_BROXTON(devid)) {
-		if (dirty) {
-			table->size  = ARRAY_SIZE(dirty_broxton_mocs_table);
-			table->table = dirty_broxton_mocs_table;
-		} else {
-			table->size  = ARRAY_SIZE(broxton_mocs_table);
-			table->table = broxton_mocs_table;
-		}
-		result = true;
-	} else if (IS_ICELAKE(devid)) {
-		table->size  = ARRAY_SIZE(icelake_mocs_table);
-		table->table = icelake_mocs_table;
-		result = true;
-	} else if (IS_TIGERLAKE(devid)) {
-		table->size  = ARRAY_SIZE(tigerlake_mocs_table);
-		table->table = tigerlake_mocs_table;
-		result = true;
-	}
-
-	return result;
-}
-
-#define LOCAL_I915_EXEC_BSD1 (I915_EXEC_BSD | (1<<13))
-#define LOCAL_I915_EXEC_BSD2 (I915_EXEC_BSD | (2<<13))
-
-static uint32_t get_engine_base(int fd, uint32_t engine)
-{
-	if (has_global_mocs(fd))
-		return GEN12_GLOBAL_MOCS;
-
-	switch (engine) {
-	case LOCAL_I915_EXEC_BSD1:	return GEN9_MFX0_MOCS_0;
-	case LOCAL_I915_EXEC_BSD2:	return GEN9_MFX1_MOCS_0;
-	case I915_EXEC_RENDER:		return GEN9_GFX_MOCS_0;
-	case I915_EXEC_BLT:		return GEN9_BLT_MOCS_0;
-	case I915_EXEC_VEBOX:		return GEN9_VEBOX_MOCS_0;
-	default:			return 0;
-	}
-}
-
-static int has_secure_batches(const int fd)
-{
-	int v = -1;
-	drm_i915_getparam_t gp = {
-		.param = I915_PARAM_HAS_SECURE_BATCHES,
-		.value = &v,
-	};
-
-	drmIoctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
-
-	return v > 0;
-}
-
-#define MI_STORE_REGISTER_MEM_64_BIT_ADDR	((0x24 << 23) | 2)
-
-static int create_read_batch(struct drm_i915_gem_relocation_entry *reloc,
-			     uint32_t *batch,
-			     uint32_t dst_handle,
-			     uint32_t size,
-			     uint32_t reg_base)
-{
-	unsigned int offset = 0;
-
-	for (uint32_t index = 0; index < size; index++, offset += 4) {
-		batch[offset]   = MI_STORE_REGISTER_MEM_64_BIT_ADDR;
-		batch[offset+1] = reg_base + (index * sizeof(uint32_t));
-		batch[offset+2] = index * sizeof(uint32_t);	/* reloc */
-		batch[offset+3] = 0;
-
-		reloc[index].offset = (offset + 2) * sizeof(uint32_t);
-		reloc[index].delta = index * sizeof(uint32_t);
-		reloc[index].target_handle = dst_handle;
-		reloc[index].write_domain = I915_GEM_DOMAIN_RENDER;
-		reloc[index].read_domains = I915_GEM_DOMAIN_RENDER;
-	}
-
-	batch[offset++] = MI_BATCH_BUFFER_END;
-	batch[offset++] = 0;
-
-	return offset * sizeof(uint32_t);
-}
-
-static void do_read_registers(int fd,
-			      uint32_t ctx_id,
-			      uint32_t dst_handle,
-			      uint32_t reg_base,
-			      uint32_t size,
-			      uint32_t engine_id)
-{
-	struct drm_i915_gem_execbuffer2 execbuf;
-	struct drm_i915_gem_exec_object2 obj[2];
-	struct drm_i915_gem_relocation_entry reloc[size];
-	uint32_t batch[size * 4 + 4];
-	uint32_t handle = gem_create(fd, 4096);
-
-	memset(reloc, 0, sizeof(reloc));
-	memset(obj, 0, sizeof(obj));
-	memset(&execbuf, 0, sizeof(execbuf));
-
-	obj[0].handle = dst_handle;
-
-	obj[1].handle = handle;
-	obj[1].relocation_count = size;
-	obj[1].relocs_ptr = to_user_pointer(reloc);
-
-	execbuf.buffers_ptr = to_user_pointer(obj);
-	execbuf.buffer_count = 2;
-	execbuf.batch_len =
-		create_read_batch(reloc, batch, dst_handle, size, reg_base);
-	i915_execbuffer2_set_context_id(execbuf, ctx_id);
-	execbuf.flags = I915_EXEC_SECURE | engine_id;
-
-	gem_write(fd, handle, 0, batch, execbuf.batch_len);
-	gem_execbuf(fd, &execbuf);
-	gem_close(fd, handle);
-}
-
-#define LOCAL_MI_LOAD_REGISTER_IMM	(0x22 << 23)
-
-static int create_write_batch(uint32_t *batch,
-			      const uint32_t *values,
-			      uint32_t size,
-			      uint32_t reg_base)
-{
-	unsigned int i;
-	unsigned int offset = 0;
-
-	batch[offset++] = LOCAL_MI_LOAD_REGISTER_IMM | (size * 2 - 1);
-
-	for (i = 0; i < size; i++) {
-		batch[offset++] = reg_base + (i * 4);
-		batch[offset++] = values[i];
-	}
-
-	batch[offset++] = MI_BATCH_BUFFER_END;
-
-	return offset * sizeof(uint32_t);
-}
-
-static void write_registers(int fd,
-			    uint32_t ctx_id,
-			    uint32_t reg_base,
-			    const uint32_t *values,
-			    uint32_t size,
-			    uint32_t engine_id,
-			    bool privileged)
-{
-	struct drm_i915_gem_exec_object2 obj;
-	struct drm_i915_gem_execbuffer2 execbuf;
-	uint32_t batch[size * 4 + 2];
-	uint32_t handle = gem_create(fd, 4096);
-
-	memset(&obj, 0, sizeof(obj));
-	memset(&execbuf, 0, sizeof(execbuf));
-
-	obj.handle = handle;
-
-	execbuf.buffers_ptr = to_user_pointer(&obj);
-	execbuf.buffer_count = 1;
-	execbuf.batch_len = create_write_batch(batch, values, size, reg_base);
-	i915_execbuffer2_set_context_id(execbuf, ctx_id);
-	if (privileged)
-		execbuf.flags = I915_EXEC_SECURE | engine_id;
-	else
-		execbuf.flags = engine_id;
-
-	gem_write(fd, handle, 0, batch, execbuf.batch_len);
-	gem_execbuf(fd, &execbuf);
-	gem_close(fd, handle);
-}
-
-static void check_control_registers(int fd,
-				    unsigned engine,
-				    uint32_t ctx_id,
-				    bool dirty)
-{
-	const uint32_t reg_base = get_engine_base(fd, engine);
-	uint32_t dst_handle = gem_create(fd, 4096);
-	uint32_t *read_regs;
-	struct mocs_table table;
-
-	igt_assert(get_mocs_settings(fd, &table, dirty));
-
-	do_read_registers(fd,
-			  ctx_id,
-			  dst_handle,
-			  reg_base,
-			  table.size,
-			  engine);
-
-	read_regs = gem_mmap__cpu(fd, dst_handle, 0, 4096, PROT_READ);
-
-	gem_set_domain(fd, dst_handle, I915_GEM_DOMAIN_CPU, 0);
-	for (int index = 0; index < table.size; index++) {
-		uint32_t val, read_val;
-
-		if (!table.table[index].used)
-			continue;
-
-		read_val = read_regs[index];
-		val = table.table[index].control_value;
-		igt_assert_f(read_val == val,
-			     "engine=%u index=%u read_value=0x%08x value=0x%08x\n",
-			     engine, index, read_val, val);
-	}
-
-	munmap(read_regs, 4096);
-	gem_close(fd, dst_handle);
-}
-
-static void check_l3cc_registers(int fd,
-				 unsigned engine,
-				 uint32_t ctx_id,
-				 bool dirty)
-{
-	struct mocs_table table;
-	uint32_t dst_handle = gem_create(fd, 4096);
-	uint32_t *read_regs;
-	int index;
-
-	igt_assert(get_mocs_settings(fd, &table, dirty));
-
-	do_read_registers(fd,
-			  ctx_id,
-			  dst_handle,
-			  GEN9_LNCFCMOCS0,
-			  (table.size + 1) / 2,
-			  engine);
-
-	read_regs = gem_mmap__cpu(fd, dst_handle, 0, 4096, PROT_READ);
-
-	gem_set_domain(fd, dst_handle, I915_GEM_DOMAIN_CPU, 0);
-
-	for (index = 0; index < table.size / 2; index++) {
-		if (table.table[index * 2].used) {
-			igt_assert_eq_u32(read_regs[index] & 0xffff,
-					  table.table[index * 2].l3cc_value);
-		}
-		if (table.table[index * 2 + 1].used) {
-			igt_assert_eq_u32(read_regs[index] >> 16,
-					  table.table[index * 2 + 1].l3cc_value);
-		}
-	}
-
-	if (table.size & 1)
-		igt_assert_eq_u32(read_regs[index] & 0xffff,
-				  table.table[index * 2].l3cc_value);
-
-	munmap(read_regs, 4096);
-	gem_close(fd, dst_handle);
-}
-
-static void rc6_wait(int i915)
-{
-	uint64_t start[2], now[2], prev;
-	bool rc6 = false;
-	int fd;
-
-	fd = perf_i915_open(I915_PMU_RC6_RESIDENCY);
-	igt_require(fd != -1);
-
-	/* First wait for roughly an RC6 Evaluation Interval */
-	gem_quiescent_gpu(i915);
-	usleep(320e3);
-
-	/* Then poll for RC6 to start ticking */
-	igt_assert_eq(read(fd, start, sizeof(start)), sizeof(start));
-	prev = start[1];
-	do {
-		usleep(5e3);
-		igt_assert_eq(read(fd, now, sizeof(now)), sizeof(now));
-		if (now[1] - prev > 1e6) {
-			rc6 = true;
-			break;
-		}
-		prev = now[1];
-	} while (now[0] - start[0] < 1e9);
-
-	close(fd);
-
-	igt_debug("rc6 residency %.2fms (delta %.1fms over 5ms), elapsed %.2fms\n",
-		  1e-6 * (now[1] - start[1]),
-		  1e-6 * (now[1] - prev),
-		  1e-6 * (now[0] - start[0]));
-	igt_require(rc6);
-}
-
-static void check_mocs_values(int fd,
-			      unsigned engine, uint32_t ctx_id,
-			      bool dirty)
-{
-	check_control_registers(fd, engine, ctx_id, dirty);
-
-	if (engine == I915_EXEC_RENDER)
-		check_l3cc_registers(fd, engine, ctx_id, dirty);
-}
-
-static void write_dirty_mocs(int fd,
-			     unsigned engine, uint32_t ctx_id,
-			     bool privileged)
-{
-	int num_of_mocs_entries;
-
-	if (intel_gen(intel_get_drm_devid(fd)) >= 11)
-		num_of_mocs_entries = GEN11_NUM_MOCS_ENTRIES;
-	else
-		num_of_mocs_entries = GEN9_NUM_MOCS_ENTRIES;
-
-	write_registers(fd, ctx_id, get_engine_base(fd, engine),
-			write_values, num_of_mocs_entries,
-			engine, privileged);
-
-	if (engine == I915_EXEC_RENDER)
-		write_registers(fd, ctx_id, GEN9_LNCFCMOCS0,
-				write_values, num_of_mocs_entries/2,
-				engine, privileged);
-}
-
-static void run_test(int fd, unsigned engine, unsigned flags, unsigned mode)
-{
-	uint32_t ctx_id = 0;
-	uint32_t ctx_clean_id;
-	uint32_t ctx_dirty_id;
-
-	/* As mocs is global for GEN11+, trying privileged write to dirty
-	 * the mocs and testing context save and restore of mocs between
-	 * contexts is bound to fail.
-	 */
-	if (flags & MOCS_DIRTY_VALUES)
-		igt_skip_on(intel_gen(intel_get_drm_devid(fd)) >= 11);
-
-	gem_require_ring(fd, engine);
-
-	/* Skip if we don't know where the registers are for this engine */
-	igt_require(get_engine_base(fd, engine));
-
-	if (flags & MOCS_NON_DEFAULT_CTX)
-		ctx_id = gem_context_create(fd);
-
-	if (flags & MOCS_DIRTY_VALUES) {
-		ctx_dirty_id = gem_context_create(fd);
-		write_dirty_mocs(fd, engine, ctx_dirty_id, true);
-		check_mocs_values(fd, engine, ctx_dirty_id, true);
-	}
-
-	check_mocs_values(fd, engine, ctx_id, false);
-
-	switch (mode) {
-	case NONE:	break;
-	case RESET:	igt_force_gpu_reset(fd);	break;
-	case SUSPEND:	igt_system_suspend_autoresume(SUSPEND_STATE_MEM,
-						      SUSPEND_TEST_NONE); break;
-	case HIBERNATE:	igt_system_suspend_autoresume(SUSPEND_STATE_DISK,
-						      SUSPEND_TEST_NONE); break;
-	case RC6:	rc6_wait(fd);			break;
-	}
-
-	check_mocs_values(fd, engine, ctx_id, false);
-
-	if (flags & MOCS_DIRTY_VALUES) {
-		ctx_clean_id = gem_context_create(fd);
-		check_mocs_values(fd, engine, ctx_dirty_id, true);
-		check_mocs_values(fd, engine, ctx_clean_id, false);
-		gem_context_destroy(fd, ctx_dirty_id);
-		gem_context_destroy(fd, ctx_clean_id);
-	}
-
-	if (ctx_id)
-		gem_context_destroy(fd, ctx_id);
-}
-
-static void isolation_test(int fd, unsigned engine)
-{
-	uint32_t ctx[2] = { gem_context_create(fd), gem_context_create(fd) };
-
-	/* Any writes by one normal client should not affect a second client */
-	write_dirty_mocs(fd, engine, ctx[0], false);
-	check_mocs_values(fd, engine, ctx[1], false);
-
-	for (int i = 0; i < ARRAY_SIZE(ctx); i++)
-		gem_context_destroy(fd, ctx[i]);
-}
-
-igt_main
-{
-	const struct intel_execution_engine *e;
-	struct mocs_table table;
-	int fd = -1;
-
-	igt_fixture {
-		fd = drm_open_driver_master(DRIVER_INTEL); /* for SECURE */
-		igt_require_gem(fd);
-		gem_require_mocs_registers(fd);
-		igt_require(get_mocs_settings(fd, &table, false));
-		igt_require(has_secure_batches(fd));
-	}
-
-	for (e = intel_execution_engines; e->name; e++) {
-		/* We don't know which engine will be assigned to us if we're
-		 * using plain I915_EXEC_BSD, I915_EXEC_DEFAULT is just
-		 * duplicating render
-		 */
-		if ((e->exec_id == I915_EXEC_BSD && !e->flags) ||
-		    e->exec_id == I915_EXEC_DEFAULT)
-			continue;
-
-		for (unsigned mode = NONE; mode < MAX_MOCS_TEST_MODES; mode++) {
-			igt_subtest_group {
-				igt_hang_t hang = {};
-
-				igt_fixture {
-					if (mode == RESET)
-						hang = igt_allow_hang(fd, 0, 0);
-				}
-
-				for (unsigned flags = 0; flags < ALL_MOCS_FLAGS + 1; flags++) {
-					/* Trying to test non-render engines for dirtying MOCS
-					 * values from one context having effect on different
-					 * context is bound to fail - only render engine is
-					 * doing context save/restore of MOCS registers.
-					 * Let's also limit testing values on non-default
-					 * contexts to render-only.
-					 */
-					if (flags && e->exec_id != I915_EXEC_RENDER)
-						continue;
-
-					igt_subtest_f("mocs-%s%s%s-%s",
-						      test_modes[mode],
-						      flags & MOCS_NON_DEFAULT_CTX ? "-ctx": "",
-						      flags & MOCS_DIRTY_VALUES ? "-dirty" : "",
-						      e->name) {
-						if (flags & (MOCS_NON_DEFAULT_CTX | MOCS_DIRTY_VALUES))
-							gem_require_contexts(fd);
-
-						run_test(fd, eb_ring(e), flags, mode);
-					}
-				}
-
-				igt_fixture {
-					if (mode == RESET)
-						igt_disallow_hang(fd, hang);
-				}
-			}
-		}
-
-		igt_subtest_f("mocs-isolation-%s", e->name) {
-			gem_require_ring(fd, eb_ring(e));
-			gem_require_contexts(fd);
-
-			isolation_test(fd, eb_ring(e));
-		}
-	}
-
-	igt_fixture
-		close(fd);
-}
diff --git a/tests/intel-ci/blacklist.txt b/tests/intel-ci/blacklist.txt
index 4d1813e50..056ad7324 100644
--- a/tests/intel-ci/blacklist.txt
+++ b/tests/intel-ci/blacklist.txt
@@ -49,7 +49,6 @@ igt at gem_hangcheck_forcewake(@.*)?
 igt at gem_lut_handle(@.*)?
 igt at gem_mmap_gtt@.*(huge|swap|clflush|forked).*
 igt at gem_mmap@.*(swap|huge).*
-igt at gem_mocs_settings@.*(suspend|hibernate).*
 igt at gem_pin(@.*)?
 igt at gem_pread_after_blit(@.*)?
 igt at gem_pwrite_pread@.*performance.*
diff --git a/tests/meson.build b/tests/meson.build
index 755fc9e6f..ca2b8b62d 100644
--- a/tests/meson.build
+++ b/tests/meson.build
@@ -314,14 +314,6 @@ test_executables += executable('gem_exec_balancer', 'i915/gem_exec_balancer.c',
 	   install : true)
 test_list += 'gem_exec_balancer'
 
-test_executables += executable('gem_mocs_settings',
-	   join_paths('i915', 'gem_mocs_settings.c'),
-	   dependencies : test_deps + [ lib_igt_perf ],
-	   install_dir : libexecdir,
-	   install_rpath : libexecdir_rpathdir,
-	   install : true)
-test_list += 'gem_mocs_settings'
-
 test_executables += executable('perf_pmu', 'perf_pmu.c',
 	   dependencies : test_deps + [ lib_igt_perf ],
 	   install_dir : libexecdir,
-- 
2.24.0



More information about the Intel-gfx mailing list