[igt-dev] [PATCH i-g-t] i915: Remove gem_mocs_settings
Mika Kuoppala
mika.kuoppala at linux.intel.com
Tue Nov 26 08:01:03 UTC 2019
Chris Wilson <chris at chris-wilson.co.uk> writes:
> 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>
Acked-by: Mika Kuoppala <mika.kuoppala 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
>
> _______________________________________________
> igt-dev mailing list
> igt-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev
More information about the igt-dev
mailing list