All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t 0/2] Upstream gem_lmem_swapping
@ 2021-11-15  7:16 Zbigniew Kempczyński
  2021-11-15  7:16 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_lmem_swapping: Add the gem_lmem_swapping test Zbigniew Kempczyński
                   ` (3 more replies)
  0 siblings, 4 replies; 11+ messages in thread
From: Zbigniew Kempczyński @ 2021-11-15  7:16 UTC (permalink / raw)
  To: igt-dev

Address review comments from Maarten + add "basic" to BAT

v2: fix seed handling when more process are involved in creating
    and filling objects
    add igt_describe()

v3: remove unnecessary empty line
    alter s-o-b list
    add additional 3 subtests to fast-feedback

CQ Tang (1):
  tests/i915/gem_lmem_swapping: Add the gem_lmem_swapping test

Zbigniew Kempczyński (1):
  tests/intel-ci/fast-feedback: Add four subtests of gem_lmem_swapping
    to BAT

 tests/i915/gem_lmem_swapping.c        | 550 ++++++++++++++++++++++++++
 tests/intel-ci/fast-feedback.testlist |   4 +
 tests/meson.build                     |   1 +
 3 files changed, 555 insertions(+)
 create mode 100644 tests/i915/gem_lmem_swapping.c

-- 
2.26.0

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_lmem_swapping: Add the gem_lmem_swapping test
  2021-11-15  7:16 [igt-dev] [PATCH i-g-t 0/2] Upstream gem_lmem_swapping Zbigniew Kempczyński
@ 2021-11-15  7:16 ` Zbigniew Kempczyński
  2021-11-16 10:31   ` Matthew Auld
  2021-11-15  7:16 ` [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add four subtests of gem_lmem_swapping to BAT Zbigniew Kempczyński
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 11+ messages in thread
From: Zbigniew Kempczyński @ 2021-11-15  7:16 UTC (permalink / raw)
  To: igt-dev

From: CQ Tang <cq.tang@intel.com>

This test exercises the lmem eviction functionality with one
dynamic subtest per lmem region.

In essence the various subtests create lmem buffers objects consuming more
lmem memory than available and check that they are successful submitting
workload subsets whose lmem size requirements are smaller than the total
amount of lmem available, forcing eviction of buffer objects under
various conditions.

v2: - addressing review comments (handling seed, minor cleanups) (Zbigniew)
    - add dynamic_lmem_subtest() macro (Maarten)

v3: - make seed unique for each process (Zbigniew)
    - add simple igt_describe() (Petri)

Signed-off-by: CQ Tang <cq.tang@intel.com>
Co-developed-by: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Co-developed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Co-developed-by: Ayaz A Siddiqui <ayaz.siddiqui@intel.com>
Signed-off-by: Ayaz A Siddiqui <ayaz.siddiqui@intel.com>
Co-developed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Co-developed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Co-developed-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Acked-by: Petri Latvala <petri.latvala@intel.com>
---
 tests/i915/gem_lmem_swapping.c | 550 +++++++++++++++++++++++++++++++++
 tests/meson.build              |   1 +
 2 files changed, 551 insertions(+)
 create mode 100644 tests/i915/gem_lmem_swapping.c

diff --git a/tests/i915/gem_lmem_swapping.c b/tests/i915/gem_lmem_swapping.c
new file mode 100644
index 000000000..786110338
--- /dev/null
+++ b/tests/i915/gem_lmem_swapping.c
@@ -0,0 +1,550 @@
+// SPDX-License-Identifier: MIT
+/*
+ * Copyright © 2021 Intel Corporation
+ */
+
+#include "i915/gem.h"
+#include "i915/gem_create.h"
+#include "i915/gem_vm.h"
+#include "i915/intel_memory_region.h"
+#include "igt.h"
+#include "igt_kmod.h"
+#include <unistd.h>
+#include <stdlib.h>
+#include <stdint.h>
+#include <stdio.h>
+#include <string.h>
+#include <fcntl.h>
+#include <inttypes.h>
+#include <errno.h>
+#include <sys/stat.h>
+#include <sys/ioctl.h>
+#include <sys/time.h>
+#include <sys/wait.h>
+#include "drm.h"
+
+IGT_TEST_DESCRIPTION("Exercise local memory swapping.");
+
+#define __round_mask(x, y) ((__typeof__(x))((y) - 1))
+#define round_up(x, y) ((((x) - 1) | __round_mask(x, y)) + 1)
+
+#define PAGE_SIZE  (1ULL << 12)
+#define SZ_64K	   (16 * PAGE_SIZE)
+
+static const char *readable_unit(uint64_t size)
+{
+	return size >> 20 ? "MiB" : size >> 10 ? "KiB" : "Bytes";
+}
+
+static uint64_t readable_size(uint64_t size)
+{
+	return size >> 20 ? size >> 20 : size >> 10 ? size >> 10 : size;
+}
+
+struct {
+	unsigned int seed;
+	bool user_seed;
+} opt;
+
+struct params {
+	struct {
+		uint64_t min;
+		uint64_t max;
+	} size;
+	unsigned int count;
+	unsigned int loops;
+	unsigned int mem_limit;
+#define TEST_VERIFY	(1 << 0)
+#define TEST_PARALLEL	(1 << 1)
+#define TEST_HEAVY	(1 << 2)
+#define TEST_RANDOM	(1 << 3)
+#define TEST_ENGINES	(1 << 4)
+#define TEST_MULTI	(1 << 5)
+	unsigned int flags;
+	unsigned int seed;
+	bool oom_test;
+};
+
+struct object {
+	uint64_t size;
+	uint32_t seed;
+	uint32_t handle;
+};
+
+static uint32_t create_bo(int i915,
+			  uint64_t size,
+			  struct drm_i915_gem_memory_class_instance *region,
+			  bool do_oom_test)
+{
+	uint32_t handle;
+	int ret;
+
+retry:
+	ret = __gem_create_in_memory_region_list(i915, &handle, &size, region, 1);
+	if (do_oom_test && (ret == -ENOMEM || ret == -ENXIO))
+		goto retry;
+	igt_assert_eq(ret, 0);
+	return handle;
+}
+
+static unsigned int __num_engines__;
+
+static void
+init_object(int i915, struct object *obj, unsigned long seed, unsigned int flags)
+{
+	unsigned int j;
+	uint32_t *buf;
+
+	obj->seed = seed;
+
+	buf = gem_mmap_offset__fixed(i915, obj->handle, 0, obj->size, PROT_WRITE);
+
+	for (j = 0; j < obj->size / sizeof(*buf); j++)
+		buf[j] = seed++;
+
+	munmap(buf, obj->size);
+}
+
+static void
+verify_object(int i915, const struct object *obj,  unsigned int flags)
+{
+	unsigned long j;
+	uint32_t *buf;
+
+	buf = gem_mmap_offset__fixed(i915, obj->handle, 0, obj->size, PROT_READ);
+
+	for (j = 0; j < obj->size / PAGE_SIZE; j++) {
+		unsigned long x = (j * PAGE_SIZE + rand() % PAGE_SIZE) / sizeof(*buf);
+		uint32_t val = obj->seed + x;
+
+		igt_assert_f(buf[x] == val,
+			     "Object mismatch at offset %zu - found %08x, expected %08x; difference:%08x!\n",
+			     x * sizeof(*buf), buf[x], val, buf[x] ^ val);
+	}
+
+	munmap(buf, obj->size);
+}
+
+static void move_to_lmem(int i915,
+			 struct object *list,
+			 unsigned int num,
+			 uint32_t batch,
+			 unsigned int engine,
+			 bool do_oom_test)
+{
+	struct drm_i915_gem_exec_object2 obj[1 + num];
+	struct drm_i915_gem_execbuffer2 eb = {
+		.buffers_ptr = to_user_pointer(obj),
+		.buffer_count = 1 + num,
+		.flags = I915_EXEC_NO_RELOC | I915_EXEC_HANDLE_LUT | engine,
+	};
+	unsigned int i, ret;
+
+	memset(obj, 0, sizeof(obj));
+
+	for (i = 0; i < num; i++) {
+		obj[i].handle = list[i].handle;
+		obj[i].flags = EXEC_OBJECT_SUPPORTS_48B_ADDRESS;
+	}
+
+	obj[i].handle = batch;
+retry:
+	ret = __gem_execbuf(i915, &eb);
+	if (do_oom_test && (ret == -ENOMEM || ret == -ENXIO))
+		goto retry;
+	igt_assert_eq(ret, 0);
+}
+
+static void __do_evict(int i915,
+		       struct drm_i915_gem_memory_class_instance *region,
+		       struct params *params,
+		       unsigned int seed)
+{
+	const unsigned int max_swap_in = params->count / 100 + 1;
+	const uint32_t bbe = MI_BATCH_BUFFER_END;
+	struct object *objects, *obj, *list;
+	uint32_t batch;
+	unsigned int engine = 0;
+	unsigned int i, l;
+	uint64_t size;
+	struct timespec t = {};
+	unsigned int num;
+
+	__gem_context_set_persistence(i915, 0, false);
+	size = 4096;
+	batch = create_bo(i915, size, region, params->oom_test);
+
+	gem_write(i915, batch, 0, &bbe, sizeof(bbe));
+
+	objects = calloc(params->count, sizeof(*objects));
+	igt_assert(objects);
+
+	list = calloc(max_swap_in, sizeof(*list));
+	igt_assert(list);
+
+	srand(seed);
+
+	/* Create the initial working set of objects. */
+
+	size = 0;
+	for (i = 0, obj = objects; i < params->count; i++, obj++) {
+		if (params->flags & TEST_RANDOM)
+			obj->size = rand() %
+				(params->size.max - params->size.min) +
+				params->size.min;
+		else
+			obj->size = params->size.min;
+
+		size += obj->size;
+		if ((size >> 20) > params->mem_limit) {
+			params->count = i;
+			break;
+		}
+		obj->handle = create_bo(i915, obj->size, region, params->oom_test);
+
+		if (params->flags & TEST_VERIFY)
+			init_object(i915, obj, rand(), params->flags);
+		else
+			move_to_lmem(i915, objects + i, 1, batch, engine,
+				     params->oom_test);
+	}
+
+	igt_debug("obj size min/max=%lu %s/%lu %s, count=%u, seed: %u\n",
+		  readable_size(params->size.min), readable_unit(params->size.min),
+		  readable_size(params->size.max), readable_unit(params->size.max),
+		  params->count, seed);
+
+	/*
+	 * Move random objects back into lmem.
+	 * for TEST_MULTI runs, make each object count s a loop to
+	 * avoid excessive run times
+	 */
+	for (l = 0; l < params->loops && igt_seconds_elapsed(&t) < 300; l += num) {
+		unsigned int idx = rand() % params->count;
+
+		num = params->flags & TEST_MULTI ? rand() % max_swap_in + 1 : 1;
+		for (i = 0; i < num; i++) {
+			list[i] = objects[idx];
+			idx = (idx + 1) % params->count;
+		}
+
+		move_to_lmem(i915, list, num, batch, engine, params->oom_test);
+
+		if (params->flags & TEST_ENGINES)
+			engine = (engine + 1) % __num_engines__;
+
+		if (params->flags & TEST_VERIFY) {
+			for (i = 0; i < num; i++)
+				verify_object(i915, &list[i], params->flags);
+
+			/* Update random object - may swap it back in. */
+			i = rand() % params->count;
+			init_object(i915, &objects[i], rand(), params->flags);
+		}
+	}
+
+	for (i = 0; i < params->count; i++)
+		gem_close(i915, objects[i].handle);
+
+	free(list);
+	free(objects);
+
+	gem_close(i915, batch);
+}
+
+static void fill_params(int i915, struct params *params,
+			struct drm_i915_memory_region_info *region,
+			unsigned int flags,
+			unsigned int nproc,
+			bool do_oom_test)
+{
+	const int swap_mb = /* For lmem, swap is total of smem + swap. */
+		intel_get_total_ram_mb() + intel_get_total_swap_mb();
+	const unsigned int size = 1 << 20;
+	const int max_swap_pct = 75;
+	/*
+	 * In random mode, add 85% hard limit to use system memory.
+	 * noticed that 88.8% can trigger OOM on some system.
+	 */
+	const int mem_limit_pct = 85;
+	int spill_mb;
+	uint32_t handle;
+
+	if (flags & TEST_RANDOM) {
+		params->size.min = 4096;
+		handle = create_bo(i915, params->size.min, &region->region,
+				   do_oom_test);
+		gem_close(i915, handle);
+		params->size.max = 2 * size + params->size.min;
+	} else {
+		params->size.min = size;
+		params->size.max = size;
+	}
+
+	params->count = (region->probed_size + (size - 1)) / size * 3 / 2;
+	spill_mb = (size >> 20) * params->count - (region->probed_size >> 20);
+	/* Don't use all RAM for swapout. */
+	igt_require(spill_mb <= swap_mb * max_swap_pct / 100);
+
+	if (flags & TEST_HEAVY) {
+		params->count *= 2;
+		spill_mb = (size >> 20) * params->count -
+			(region->probed_size >> 20);
+
+		if (spill_mb > swap_mb * max_swap_pct / 100) {
+			unsigned int count;
+			unsigned long set;
+
+			igt_warn("Reducing working set due low RAM + swap! (Need %d MiB, have %d MiB.)\n",
+				 spill_mb, swap_mb);
+			set = region->probed_size +
+				(((unsigned long)swap_mb * max_swap_pct / 100) << 20);
+			count = set / size;
+			/* No point if heavy test is too similar to normal. */
+			igt_require(count > (params->count / 2) * 133 / 100);
+			params->count = count;
+		}
+	}
+
+	params->loops = params->count;
+	params->seed = opt.user_seed ? opt.seed : time(NULL);
+
+	/*
+	 * If run in parallel, reduce per process buffer count to keep the
+	 * total the same, but don't reduce loops since we gain some
+	 * efficiency by the parallel execution
+	 */
+	if (flags & TEST_PARALLEL)
+		params->count /= nproc;
+
+	/*
+	 * For heavy tests, reduce the loop count to avoid excessive
+	 * run-times
+	 */
+	if (flags & TEST_HEAVY)
+		params->loops = params->loops / 2 + 1;
+
+	params->flags = flags;
+	params->oom_test = do_oom_test;
+
+	params->mem_limit = swap_mb * mem_limit_pct / 100 +
+		(region->probed_size >> 20);
+	igt_info("Memory: system-total %dMiB, lmem-region %lldMiB, usage-limit %dMiB\n",
+		 swap_mb, (region->probed_size >> 20), params->mem_limit);
+	igt_info("Using %u thread(s), %u loop(s), %u objects of %lu %s - %lu %s, seed: %u, oom: %s\n",
+		 params->flags & TEST_PARALLEL ? nproc : 1,
+		 params->loops,
+		 params->count,
+		 readable_size(params->size.min),
+		 readable_unit(params->size.min),
+		 readable_size(params->size.max),
+		 readable_unit(params->size.max),
+		 params->seed,
+		 do_oom_test ? "yes" : "no");
+}
+
+static void test_evict(int i915,
+		       struct drm_i915_memory_region_info *region,
+		       unsigned int flags)
+{
+	const unsigned int nproc = sysconf(_SC_NPROCESSORS_ONLN) + 1;
+	struct params params;
+
+	fill_params(i915, &params, region, flags, nproc, false);
+
+	if (flags & TEST_PARALLEL) {
+		int fd = gem_reopen_driver(i915);
+
+		igt_fork(child, nproc)
+			__do_evict(fd, &region->region, &params,
+				   params.seed + child + 1);
+
+		igt_waitchildren();
+		close(fd);
+	} else {
+		__do_evict(i915, &region->region, &params, params.seed);
+	}
+}
+
+static void leak(uint64_t alloc)
+{
+	char *ptr;
+
+	ptr = mmap(NULL, alloc, PROT_READ | PROT_WRITE,
+		   MAP_ANON | MAP_PRIVATE | MAP_POPULATE, -1, 0);
+	if (ptr == MAP_FAILED)
+		return;
+
+	while (alloc) {
+		alloc -= 4096;
+		ptr[alloc] = 0;
+	}
+}
+
+static void gem_leak(int fd, uint64_t alloc)
+{
+	uint32_t handle = gem_create(fd, alloc);
+	void *buf;
+
+	buf = gem_mmap_offset__fixed(fd, handle, 0, PAGE_SIZE, PROT_WRITE);
+	memset(buf, 0, PAGE_SIZE);
+	munmap(buf, PAGE_SIZE);
+
+	gem_madvise(fd, handle, I915_MADV_DONTNEED);
+}
+
+static int *lmem_done;
+
+static void smem_oom_exit_handler(int sig)
+{
+	(*lmem_done)++;
+}
+
+static void test_smem_oom(int i915,
+			  struct drm_i915_memory_region_info *region)
+{
+	const uint64_t smem_size = intel_get_total_ram_mb() +
+		intel_get_total_swap_mb();
+	const unsigned int alloc = 256 * 1024 * 1024;
+	const unsigned int num_alloc = 1 + smem_size / (alloc >> 20);
+	struct igt_helper_process smem_proc = {};
+	unsigned int n;
+
+	lmem_done = mmap(0, sizeof(*lmem_done), PROT_WRITE,
+			 MAP_SHARED | MAP_ANON, -1, 0);
+	igt_assert(lmem_done != MAP_FAILED);
+	*lmem_done = 0;
+
+	/* process for testing lmem eviction */
+	igt_fork(child, 1) {
+		int fd = gem_reopen_driver(i915);
+		struct params params;
+
+		fill_params(i915, &params, region, 0, 1, true);
+
+		igt_install_exit_handler(smem_oom_exit_handler);
+		__do_evict(fd, &region->region, &params,
+			   params.seed + child + 1);
+
+		close(fd);
+	}
+
+	/* smem memory hog process, respawn till the lmem process completes */
+	while (!READ_ONCE(*lmem_done)) {
+		igt_fork_helper(&smem_proc) {
+			igt_fork(child, 1) {
+				for (int pass = 0; pass < num_alloc; pass++) {
+					if (READ_ONCE(*lmem_done))
+						break;
+					leak(alloc);
+				}
+			}
+			igt_fork(child, 1) {
+				int fd = gem_reopen_driver(i915);
+
+				for (int pass = 0; pass < num_alloc; pass++) {
+					if (READ_ONCE(*lmem_done))
+						break;
+					gem_leak(fd, alloc);
+				}
+				close(fd);
+			}
+			/*
+			 * Wait for grand-child processes to finish or be
+			 * killed by the oom killer, don't call
+			 * igt_waitchildren because of the noise
+			 */
+			for (n = 0; n < 2; n++)
+				wait(NULL);
+		}
+		igt_wait_helper(&smem_proc);
+	}
+	munmap(lmem_done, sizeof(*lmem_done));
+	/* Reap exit status of the lmem process */
+	igt_waitchildren();
+}
+
+#define dynamic_lmem_subtest(reg, regs, subtest_name...) \
+	igt_subtest_with_dynamic(subtest_name) \
+		for (unsigned int i = 0; i < (regs)->num_regions; i++) \
+			for_each_if (((reg) = &(regs)->regions[i])->region.memory_class == I915_MEMORY_CLASS_DEVICE) \
+				igt_dynamic_f("lmem%u", (reg)->region.memory_instance)
+
+static int opt_handler(int option, int option_index, void *input)
+{
+	switch (option) {
+	case 's':
+		opt.user_seed = true;
+		opt.seed = strtoul(optarg, NULL, 0);
+		break;
+	default:
+		return IGT_OPT_HANDLER_ERROR;
+	}
+
+	return IGT_OPT_HANDLER_SUCCESS;
+}
+
+const char *help_str =
+	"  --seed       Seed for random number generator";
+
+struct option long_options[] = {
+	{ "seed",    required_argument, NULL, 's'},
+	{ 0, 0, 0, 0 }
+};
+
+igt_main_args("", long_options, help_str, opt_handler, NULL)
+{
+	struct drm_i915_query_memory_regions *regions;
+	struct drm_i915_memory_region_info *region;
+	struct test {
+		const char *name;
+		unsigned int flags;
+	} *test, tests[] = {
+		{ "basic", 0 },
+		{ "random", TEST_RANDOM },
+		{ "random-engines", TEST_RANDOM | TEST_ENGINES },
+		{ "heavy-random", TEST_RANDOM | TEST_HEAVY },
+		{ "heavy-multi", TEST_RANDOM | TEST_HEAVY | TEST_ENGINES | TEST_MULTI },
+		{ "verify", TEST_VERIFY},
+		{ "verify-random", TEST_VERIFY | TEST_RANDOM},
+		{ "heavy-verify-random", TEST_VERIFY | TEST_RANDOM | TEST_HEAVY },
+		{ "heavy-verify-multi", TEST_VERIFY | TEST_RANDOM | TEST_HEAVY | TEST_ENGINES | TEST_MULTI },
+		{ "parallel-random", TEST_PARALLEL | TEST_RANDOM },
+		{ "parallel-random-engines", TEST_PARALLEL | TEST_RANDOM | TEST_ENGINES },
+		{ "parallel-random-verify", TEST_PARALLEL | TEST_RANDOM | TEST_VERIFY },
+		{ "parallel-multi", TEST_PARALLEL | TEST_RANDOM | TEST_VERIFY | TEST_ENGINES | TEST_MULTI },
+		{ }
+	};
+	int i915 = -1;
+
+	igt_fixture {
+		struct intel_execution_engine2 *e;
+
+		i915 = drm_open_driver(DRIVER_INTEL);
+		igt_require_gem(i915);
+
+		regions = gem_get_query_memory_regions(i915);
+		igt_require(regions);
+
+		for_each_physical_engine(i915, e)
+			__num_engines__++;
+		igt_require(__num_engines__);
+	}
+
+	for (test = tests; test->name; test++) {
+		igt_describe("Exercise local memory swapping to system memory");
+		dynamic_lmem_subtest(region, regions, test->name)
+			test_evict(i915, region, test->flags);
+	}
+
+	igt_describe("Exercise local memory swapping during exhausting system memory");
+	dynamic_lmem_subtest(region, regions, "smem-oom")
+		test_smem_oom(i915, region);
+
+	igt_fixture {
+		free(regions);
+		close(i915);
+	}
+
+	igt_exit();
+}
diff --git a/tests/meson.build b/tests/meson.build
index 0af3e03a3..49c0f2d24 100644
--- a/tests/meson.build
+++ b/tests/meson.build
@@ -156,6 +156,7 @@ i915_progs = [
 	'gem_gtt_speed',
 	'gem_huc_copy',
 	'gem_linear_blits',
+	'gem_lmem_swapping',
 	'gem_lut_handle',
 	'gem_madvise',
 	'gem_media_fill',
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add four subtests of gem_lmem_swapping to BAT
  2021-11-15  7:16 [igt-dev] [PATCH i-g-t 0/2] Upstream gem_lmem_swapping Zbigniew Kempczyński
  2021-11-15  7:16 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_lmem_swapping: Add the gem_lmem_swapping test Zbigniew Kempczyński
@ 2021-11-15  7:16 ` Zbigniew Kempczyński
  2021-11-16 11:07   ` Matthew Auld
  2021-11-15  8:05 ` [igt-dev] ✓ Fi.CI.BAT: success for Upstream gem_lmem_swapping (rev4) Patchwork
  2021-11-15 11:17 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
  3 siblings, 1 reply; 11+ messages in thread
From: Zbigniew Kempczyński @ 2021-11-15  7:16 UTC (permalink / raw)
  To: igt-dev

Add basic and some random subtests to BAT to verify early lmem swapping
is working on discrete.

Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Cc: Petri Latvala <petri.latvala@intel.com>
Cc: Tomi Sarvela <tomi.p.sarvela@intel.com>
---
 tests/intel-ci/fast-feedback.testlist | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
index fa5006d2e..de1c6cb46 100644
--- a/tests/intel-ci/fast-feedback.testlist
+++ b/tests/intel-ci/fast-feedback.testlist
@@ -34,6 +34,10 @@ igt@gem_flink_basic@double-flink
 igt@gem_flink_basic@flink-lifetime
 igt@gem_huc_copy@huc-copy
 igt@gem_linear_blits@basic
+igt@gem_lmem_swapping@basic
+igt@gem_lmem_swapping@parallel-random-engines
+igt@gem_lmem_swapping@random-engines
+igt@gem_lmem_swapping@verify-random
 igt@gem_mmap@basic
 igt@gem_mmap_gtt@basic
 igt@gem_render_linear_blits@basic
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [igt-dev] ✓ Fi.CI.BAT: success for Upstream gem_lmem_swapping (rev4)
  2021-11-15  7:16 [igt-dev] [PATCH i-g-t 0/2] Upstream gem_lmem_swapping Zbigniew Kempczyński
  2021-11-15  7:16 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_lmem_swapping: Add the gem_lmem_swapping test Zbigniew Kempczyński
  2021-11-15  7:16 ` [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add four subtests of gem_lmem_swapping to BAT Zbigniew Kempczyński
@ 2021-11-15  8:05 ` Patchwork
  2021-11-15 11:17 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
  3 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2021-11-15  8:05 UTC (permalink / raw)
  To: Zbigniew Kempczyński; +Cc: igt-dev

[-- Attachment #1: Type: text/plain, Size: 7862 bytes --]

== Series Details ==

Series: Upstream gem_lmem_swapping (rev4)
URL   : https://patchwork.freedesktop.org/series/96707/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_10879 -> IGTPW_6397
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/index.html

Participating hosts (29 -> 27)
------------------------------

  Additional (1): fi-pnv-d510 
  Missing    (3): fi-bsw-cyan bat-dg1-6 bat-dg1-5 

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in IGTPW_6397:

### IGT changes ###

#### Possible regressions ####

  * {igt@gem_lmem_swapping@basic} (NEW):
    - {fi-ehl-2}:         NOTRUN -> [SKIP][1] +3 similar issues
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-ehl-2/igt@gem_lmem_swapping@basic.html

  * {igt@gem_lmem_swapping@parallel-random-engines} (NEW):
    - {fi-tgl-dsi}:       NOTRUN -> [SKIP][2] +3 similar issues
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-tgl-dsi/igt@gem_lmem_swapping@parallel-random-engines.html
    - {fi-jsl-1}:         NOTRUN -> [SKIP][3] +3 similar issues
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-jsl-1/igt@gem_lmem_swapping@parallel-random-engines.html
    - fi-cml-u2:          NOTRUN -> [SKIP][4] +3 similar issues
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-cml-u2/igt@gem_lmem_swapping@parallel-random-engines.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@i915_selftest@live@execlists:
    - {fi-tgl-dsi}:       [PASS][5] -> [INCOMPLETE][6]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/fi-tgl-dsi/igt@i915_selftest@live@execlists.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-tgl-dsi/igt@i915_selftest@live@execlists.html

  
New tests
---------

  New tests have been introduced between CI_DRM_10879 and IGTPW_6397:

### New IGT tests (4) ###

  * igt@gem_lmem_swapping@basic:
    - Statuses : 21 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@parallel-random-engines:
    - Statuses : 21 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@random-engines:
    - Statuses : 21 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@verify-random:
    - Statuses : 21 skip(s)
    - Exec time: [0.0] s

  

Known issues
------------

  Here are the changes found in IGTPW_6397 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * {igt@gem_lmem_swapping@basic} (NEW):
    - fi-kbl-7567u:       NOTRUN -> [SKIP][7] ([fdo#109271]) +3 similar issues
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-kbl-7567u/igt@gem_lmem_swapping@basic.html
    - fi-apl-guc:         NOTRUN -> [SKIP][8] ([fdo#109271]) +3 similar issues
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-apl-guc/igt@gem_lmem_swapping@basic.html
    - fi-glk-dsi:         NOTRUN -> [SKIP][9] ([fdo#109271]) +3 similar issues
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-glk-dsi/igt@gem_lmem_swapping@basic.html
    - fi-kbl-guc:         NOTRUN -> [SKIP][10] ([fdo#109271]) +3 similar issues
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-kbl-guc/igt@gem_lmem_swapping@basic.html
    - fi-kbl-7500u:       NOTRUN -> [SKIP][11] ([fdo#109271]) +3 similar issues
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-kbl-7500u/igt@gem_lmem_swapping@basic.html
    - fi-bdw-5557u:       NOTRUN -> [SKIP][12] ([fdo#109271]) +3 similar issues
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-bdw-5557u/igt@gem_lmem_swapping@basic.html
    - fi-kbl-soraka:      NOTRUN -> [SKIP][13] ([fdo#109271]) +3 similar issues
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-kbl-soraka/igt@gem_lmem_swapping@basic.html

  * {igt@gem_lmem_swapping@parallel-random-engines} (NEW):
    - fi-cfl-8700k:       NOTRUN -> [SKIP][14] ([fdo#109271]) +3 similar issues
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-cfl-8700k/igt@gem_lmem_swapping@parallel-random-engines.html
    - {fi-hsw-gt1}:       NOTRUN -> [SKIP][15] ([fdo#109271]) +3 similar issues
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-hsw-gt1/igt@gem_lmem_swapping@parallel-random-engines.html
    - fi-cfl-guc:         NOTRUN -> [SKIP][16] ([fdo#109271]) +3 similar issues
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-cfl-guc/igt@gem_lmem_swapping@parallel-random-engines.html
    - fi-hsw-4770:        NOTRUN -> [SKIP][17] ([fdo#109271]) +3 similar issues
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-hsw-4770/igt@gem_lmem_swapping@parallel-random-engines.html

  * {igt@gem_lmem_swapping@random-engines} (NEW):
    - fi-skl-6700k2:      NOTRUN -> [SKIP][18] ([fdo#109271]) +3 similar issues
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-skl-6700k2/igt@gem_lmem_swapping@random-engines.html
    - fi-blb-e6850:       NOTRUN -> [SKIP][19] ([fdo#109271]) +3 similar issues
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-blb-e6850/igt@gem_lmem_swapping@random-engines.html
    - fi-kbl-8809g:       NOTRUN -> [SKIP][20] ([fdo#109271]) +3 similar issues
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-kbl-8809g/igt@gem_lmem_swapping@random-engines.html
    - fi-cfl-8109u:       NOTRUN -> [SKIP][21] ([fdo#109271]) +3 similar issues
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-cfl-8109u/igt@gem_lmem_swapping@random-engines.html

  * {igt@gem_lmem_swapping@verify-random} (NEW):
    - fi-bsw-kefka:       NOTRUN -> [SKIP][22] ([fdo#109271]) +3 similar issues
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-bsw-kefka/igt@gem_lmem_swapping@verify-random.html

  * igt@prime_vgem@basic-userptr:
    - fi-pnv-d510:        NOTRUN -> [SKIP][23] ([fdo#109271]) +57 similar issues
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-pnv-d510/igt@prime_vgem@basic-userptr.html

  
#### Possible fixes ####

  * igt@i915_selftest@live@hangcheck:
    - {fi-hsw-gt1}:       [DMESG-WARN][24] ([i915#3303]) -> [PASS][25]
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/fi-hsw-gt1/igt@i915_selftest@live@hangcheck.html
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/fi-hsw-gt1/igt@i915_selftest@live@hangcheck.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [i915#3303]: https://gitlab.freedesktop.org/drm/intel/issues/3303


Build changes
-------------

  * CI: CI-20190529 -> None
  * IGT: IGT_6280 -> IGTPW_6397

  CI-20190529: 20190529
  CI_DRM_10879: d65e2bf0ad52214d54407c4e344333fde5e53b1f @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_6397: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/index.html
  IGT_6280: 246bfd31dba6bf184b26b170d91d72c90a54be6b @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git



== Testlist changes ==

+igt@gem_lmem_swapping@basic
+igt@gem_lmem_swapping@heavy-multi
+igt@gem_lmem_swapping@heavy-random
+igt@gem_lmem_swapping@heavy-verify-multi
+igt@gem_lmem_swapping@heavy-verify-random
+igt@gem_lmem_swapping@parallel-multi
+igt@gem_lmem_swapping@parallel-random
+igt@gem_lmem_swapping@parallel-random-engines
+igt@gem_lmem_swapping@parallel-random-verify
+igt@gem_lmem_swapping@random
+igt@gem_lmem_swapping@random-engines
+igt@gem_lmem_swapping@smem-oom
+igt@gem_lmem_swapping@verify
+igt@gem_lmem_swapping@verify-random

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/index.html

[-- Attachment #2: Type: text/html, Size: 10188 bytes --]

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [igt-dev] ✓ Fi.CI.IGT: success for Upstream gem_lmem_swapping (rev4)
  2021-11-15  7:16 [igt-dev] [PATCH i-g-t 0/2] Upstream gem_lmem_swapping Zbigniew Kempczyński
                   ` (2 preceding siblings ...)
  2021-11-15  8:05 ` [igt-dev] ✓ Fi.CI.BAT: success for Upstream gem_lmem_swapping (rev4) Patchwork
@ 2021-11-15 11:17 ` Patchwork
  3 siblings, 0 replies; 11+ messages in thread
From: Patchwork @ 2021-11-15 11:17 UTC (permalink / raw)
  To: Zbigniew Kempczyński; +Cc: igt-dev

[-- Attachment #1: Type: text/plain, Size: 30251 bytes --]

== Series Details ==

Series: Upstream gem_lmem_swapping (rev4)
URL   : https://patchwork.freedesktop.org/series/96707/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_10879_full -> IGTPW_6397_full
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/index.html

Participating hosts (11 -> 7)
------------------------------

  Missing    (4): pig-skl-6260u pig-kbl-iris shard-rkl pig-glk-j5005 

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in IGTPW_6397_full:

### IGT changes ###

#### Possible regressions ####

  * {igt@gem_lmem_swapping@heavy-random} (NEW):
    - shard-iclb:         NOTRUN -> [SKIP][1] +13 similar issues
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb8/igt@gem_lmem_swapping@heavy-random.html

  * {igt@gem_lmem_swapping@verify-random} (NEW):
    - shard-tglb:         NOTRUN -> [SKIP][2] +13 similar issues
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb1/igt@gem_lmem_swapping@verify-random.html

  
New tests
---------

  New tests have been introduced between CI_DRM_10879_full and IGTPW_6397_full:

### New IGT tests (14) ###

  * igt@gem_lmem_swapping@basic:
    - Statuses : 3 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@heavy-multi:
    - Statuses : 5 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@heavy-random:
    - Statuses : 4 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@heavy-verify-multi:
    - Statuses : 5 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@heavy-verify-random:
    - Statuses : 5 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@parallel-multi:
    - Statuses : 5 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@parallel-random:
    - Statuses : 5 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@parallel-random-engines:
    - Statuses : 5 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@parallel-random-verify:
    - Statuses : 5 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@random:
    - Statuses : 5 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@random-engines:
    - Statuses : 5 skip(s)
    - Exec time: [0.0, 0.00] s

  * igt@gem_lmem_swapping@smem-oom:
    - Statuses : 4 skip(s)
    - Exec time: [0.0] s

  * igt@gem_lmem_swapping@verify:
    - Statuses : 5 skip(s)
    - Exec time: [0.0, 0.00] s

  * igt@gem_lmem_swapping@verify-random:
    - Statuses : 5 skip(s)
    - Exec time: [0.0] s

  

Known issues
------------

  Here are the changes found in IGTPW_6397_full that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@gem_ctx_isolation@preservation-s3@vcs0:
    - shard-kbl:          [PASS][3] -> [DMESG-WARN][4] ([i915#180]) +3 similar issues
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-kbl7/igt@gem_ctx_isolation@preservation-s3@vcs0.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl3/igt@gem_ctx_isolation@preservation-s3@vcs0.html

  * igt@gem_ctx_persistence@many-contexts:
    - shard-tglb:         [PASS][5] -> [FAIL][6] ([i915#2410])
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-tglb3/igt@gem_ctx_persistence@many-contexts.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb1/igt@gem_ctx_persistence@many-contexts.html

  * igt@gem_exec_fair@basic-none@vcs0:
    - shard-kbl:          [PASS][7] -> [FAIL][8] ([i915#2842])
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-kbl7/igt@gem_exec_fair@basic-none@vcs0.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl7/igt@gem_exec_fair@basic-none@vcs0.html

  * igt@gem_exec_fair@basic-none@vecs0:
    - shard-apl:          [PASS][9] -> [FAIL][10] ([i915#2842] / [i915#3468])
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-apl4/igt@gem_exec_fair@basic-none@vecs0.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl3/igt@gem_exec_fair@basic-none@vecs0.html

  * igt@gem_exec_fair@basic-pace-share@rcs0:
    - shard-tglb:         [PASS][11] -> [FAIL][12] ([i915#2842])
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-tglb5/igt@gem_exec_fair@basic-pace-share@rcs0.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb3/igt@gem_exec_fair@basic-pace-share@rcs0.html

  * igt@gem_exec_fair@basic-sync@rcs0:
    - shard-kbl:          [PASS][13] -> [SKIP][14] ([fdo#109271])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-kbl2/igt@gem_exec_fair@basic-sync@rcs0.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl6/igt@gem_exec_fair@basic-sync@rcs0.html

  * igt@gem_exec_whisper@basic-queues-forked-all:
    - shard-iclb:         [PASS][15] -> [INCOMPLETE][16] ([i915#1895])
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-iclb8/igt@gem_exec_whisper@basic-queues-forked-all.html
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb8/igt@gem_exec_whisper@basic-queues-forked-all.html

  * igt@gem_huc_copy@huc-copy:
    - shard-kbl:          NOTRUN -> [SKIP][17] ([fdo#109271] / [i915#2190])
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl1/igt@gem_huc_copy@huc-copy.html

  * {igt@gem_lmem_swapping@random} (NEW):
    - shard-apl:          NOTRUN -> [SKIP][18] ([fdo#109271]) +146 similar issues
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl3/igt@gem_lmem_swapping@random.html

  * igt@gem_pwrite@basic-exhaustion:
    - shard-apl:          NOTRUN -> [WARN][19] ([i915#2658])
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl1/igt@gem_pwrite@basic-exhaustion.html

  * igt@gem_pxp@verify-pxp-stale-ctx-execution:
    - shard-tglb:         NOTRUN -> [SKIP][20] ([i915#4270]) +1 similar issue
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb8/igt@gem_pxp@verify-pxp-stale-ctx-execution.html

  * igt@gem_userptr_blits@input-checking:
    - shard-apl:          NOTRUN -> [DMESG-WARN][21] ([i915#3002])
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl1/igt@gem_userptr_blits@input-checking.html
    - shard-kbl:          NOTRUN -> [DMESG-WARN][22] ([i915#3002])
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl4/igt@gem_userptr_blits@input-checking.html

  * igt@gem_userptr_blits@vma-merge:
    - shard-apl:          NOTRUN -> [FAIL][23] ([i915#3318])
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl6/igt@gem_userptr_blits@vma-merge.html
    - shard-kbl:          NOTRUN -> [FAIL][24] ([i915#3318])
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl6/igt@gem_userptr_blits@vma-merge.html

  * igt@gem_workarounds@suspend-resume:
    - shard-tglb:         NOTRUN -> [INCOMPLETE][25] ([i915#456])
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb7/igt@gem_workarounds@suspend-resume.html

  * igt@gem_workarounds@suspend-resume-context:
    - shard-apl:          [PASS][26] -> [DMESG-WARN][27] ([i915#180]) +1 similar issue
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-apl7/igt@gem_workarounds@suspend-resume-context.html
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl1/igt@gem_workarounds@suspend-resume-context.html

  * igt@gen7_exec_parse@basic-rejected:
    - shard-tglb:         NOTRUN -> [SKIP][28] ([fdo#109289]) +2 similar issues
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb2/igt@gen7_exec_parse@basic-rejected.html

  * igt@gen9_exec_parse@bb-start-param:
    - shard-tglb:         NOTRUN -> [SKIP][29] ([i915#2856]) +1 similar issue
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb7/igt@gen9_exec_parse@bb-start-param.html

  * igt@i915_pm_dc@dc6-psr:
    - shard-iclb:         [PASS][30] -> [FAIL][31] ([i915#454])
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-iclb2/igt@i915_pm_dc@dc6-psr.html
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb3/igt@i915_pm_dc@dc6-psr.html

  * igt@i915_pm_lpsp@screens-disabled:
    - shard-tglb:         NOTRUN -> [SKIP][32] ([i915#1902])
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb5/igt@i915_pm_lpsp@screens-disabled.html

  * igt@i915_suspend@forcewake:
    - shard-apl:          NOTRUN -> [DMESG-WARN][33] ([i915#180])
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl8/igt@i915_suspend@forcewake.html

  * igt@kms_big_fb@linear-32bpp-rotate-180:
    - shard-glk:          NOTRUN -> [DMESG-WARN][34] ([i915#118]) +1 similar issue
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk8/igt@kms_big_fb@linear-32bpp-rotate-180.html

  * igt@kms_big_fb@x-tiled-32bpp-rotate-0:
    - shard-glk:          [PASS][35] -> [DMESG-WARN][36] ([i915#118]) +1 similar issue
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-glk3/igt@kms_big_fb@x-tiled-32bpp-rotate-0.html
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk4/igt@kms_big_fb@x-tiled-32bpp-rotate-0.html

  * igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-180-hflip:
    - shard-kbl:          NOTRUN -> [SKIP][37] ([fdo#109271] / [i915#3777])
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl1/igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-180-hflip.html

  * igt@kms_big_fb@y-tiled-64bpp-rotate-90:
    - shard-tglb:         NOTRUN -> [SKIP][38] ([fdo#111614])
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb1/igt@kms_big_fb@y-tiled-64bpp-rotate-90.html

  * igt@kms_ccs@pipe-a-crc-primary-rotation-180-y_tiled_gen12_mc_ccs:
    - shard-tglb:         NOTRUN -> [SKIP][39] ([i915#3689] / [i915#3886]) +2 similar issues
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb8/igt@kms_ccs@pipe-a-crc-primary-rotation-180-y_tiled_gen12_mc_ccs.html

  * igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc:
    - shard-apl:          NOTRUN -> [SKIP][40] ([fdo#109271] / [i915#3886]) +7 similar issues
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl6/igt@kms_ccs@pipe-a-missing-ccs-buffer-y_tiled_gen12_rc_ccs_cc.html

  * igt@kms_ccs@pipe-b-bad-pixel-format-y_tiled_gen12_rc_ccs_cc:
    - shard-iclb:         NOTRUN -> [SKIP][41] ([fdo#109278] / [i915#3886]) +1 similar issue
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb4/igt@kms_ccs@pipe-b-bad-pixel-format-y_tiled_gen12_rc_ccs_cc.html

  * igt@kms_ccs@pipe-c-ccs-on-another-bo-y_tiled_gen12_mc_ccs:
    - shard-kbl:          NOTRUN -> [SKIP][42] ([fdo#109271] / [i915#3886]) +8 similar issues
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl4/igt@kms_ccs@pipe-c-ccs-on-another-bo-y_tiled_gen12_mc_ccs.html

  * igt@kms_ccs@pipe-c-crc-primary-rotation-180-y_tiled_gen12_rc_ccs_cc:
    - shard-glk:          NOTRUN -> [SKIP][43] ([fdo#109271] / [i915#3886]) +2 similar issues
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk4/igt@kms_ccs@pipe-c-crc-primary-rotation-180-y_tiled_gen12_rc_ccs_cc.html

  * igt@kms_ccs@pipe-c-crc-primary-rotation-180-yf_tiled_ccs:
    - shard-tglb:         NOTRUN -> [SKIP][44] ([i915#3689]) +5 similar issues
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb6/igt@kms_ccs@pipe-c-crc-primary-rotation-180-yf_tiled_ccs.html

  * igt@kms_chamelium@dp-hpd-storm-disable:
    - shard-glk:          NOTRUN -> [SKIP][45] ([fdo#109271] / [fdo#111827]) +1 similar issue
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk5/igt@kms_chamelium@dp-hpd-storm-disable.html

  * igt@kms_chamelium@vga-hpd-fast:
    - shard-tglb:         NOTRUN -> [SKIP][46] ([fdo#109284] / [fdo#111827]) +10 similar issues
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb1/igt@kms_chamelium@vga-hpd-fast.html

  * igt@kms_color_chamelium@pipe-c-ctm-0-25:
    - shard-apl:          NOTRUN -> [SKIP][47] ([fdo#109271] / [fdo#111827]) +12 similar issues
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl3/igt@kms_color_chamelium@pipe-c-ctm-0-25.html
    - shard-kbl:          NOTRUN -> [SKIP][48] ([fdo#109271] / [fdo#111827]) +6 similar issues
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl3/igt@kms_color_chamelium@pipe-c-ctm-0-25.html

  * igt@kms_content_protection@atomic:
    - shard-kbl:          NOTRUN -> [TIMEOUT][49] ([i915#1319])
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl1/igt@kms_content_protection@atomic.html
    - shard-apl:          NOTRUN -> [TIMEOUT][50] ([i915#1319])
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl7/igt@kms_content_protection@atomic.html

  * igt@kms_content_protection@uevent:
    - shard-kbl:          NOTRUN -> [FAIL][51] ([i915#2105])
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl3/igt@kms_content_protection@uevent.html
    - shard-apl:          NOTRUN -> [FAIL][52] ([i915#2105])
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl6/igt@kms_content_protection@uevent.html

  * igt@kms_cursor_crc@pipe-a-cursor-32x10-offscreen:
    - shard-tglb:         NOTRUN -> [SKIP][53] ([i915#3359]) +4 similar issues
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb8/igt@kms_cursor_crc@pipe-a-cursor-32x10-offscreen.html

  * igt@kms_cursor_crc@pipe-b-cursor-32x32-sliding:
    - shard-tglb:         NOTRUN -> [SKIP][54] ([i915#3319]) +1 similar issue
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb8/igt@kms_cursor_crc@pipe-b-cursor-32x32-sliding.html

  * igt@kms_cursor_crc@pipe-d-cursor-512x512-offscreen:
    - shard-tglb:         NOTRUN -> [SKIP][55] ([fdo#109279] / [i915#3359]) +3 similar issues
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb3/igt@kms_cursor_crc@pipe-d-cursor-512x512-offscreen.html

  * igt@kms_cursor_crc@pipe-d-cursor-alpha-transparent:
    - shard-iclb:         NOTRUN -> [SKIP][56] ([fdo#109278]) +3 similar issues
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb2/igt@kms_cursor_crc@pipe-d-cursor-alpha-transparent.html

  * igt@kms_cursor_legacy@cursorb-vs-flipb-toggle:
    - shard-iclb:         NOTRUN -> [SKIP][57] ([fdo#109274] / [fdo#109278])
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb8/igt@kms_cursor_legacy@cursorb-vs-flipb-toggle.html

  * igt@kms_cursor_legacy@flip-vs-cursor-toggle:
    - shard-iclb:         [PASS][58] -> [FAIL][59] ([i915#2346])
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-iclb5/igt@kms_cursor_legacy@flip-vs-cursor-toggle.html
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb7/igt@kms_cursor_legacy@flip-vs-cursor-toggle.html

  * igt@kms_cursor_legacy@pipe-d-torture-bo:
    - shard-kbl:          NOTRUN -> [SKIP][60] ([fdo#109271] / [i915#533]) +1 similar issue
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl4/igt@kms_cursor_legacy@pipe-d-torture-bo.html

  * igt@kms_dsc@xrgb8888-dsc-compression:
    - shard-tglb:         NOTRUN -> [SKIP][61] ([i915#3828])
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb6/igt@kms_dsc@xrgb8888-dsc-compression.html

  * igt@kms_fbcon_fbt@fbc-suspend:
    - shard-kbl:          [PASS][62] -> [INCOMPLETE][63] ([i915#180] / [i915#636])
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-kbl1/igt@kms_fbcon_fbt@fbc-suspend.html
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl3/igt@kms_fbcon_fbt@fbc-suspend.html

  * igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytile:
    - shard-iclb:         NOTRUN -> [SKIP][64] ([i915#3701])
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb2/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-32bpp-ytile.html

  * igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-indfb-draw-mmap-gtt:
    - shard-iclb:         NOTRUN -> [SKIP][65] ([fdo#109280])
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb8/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-pri-indfb-draw-mmap-gtt.html

  * igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-mmap-cpu:
    - shard-glk:          NOTRUN -> [SKIP][66] ([fdo#109271]) +46 similar issues
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk3/igt@kms_frontbuffer_tracking@fbcpsr-rgb565-draw-mmap-cpu.html

  * igt@kms_frontbuffer_tracking@psr-2p-scndscrn-cur-indfb-draw-mmap-gtt:
    - shard-tglb:         NOTRUN -> [SKIP][67] ([fdo#111825]) +31 similar issues
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb2/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-cur-indfb-draw-mmap-gtt.html

  * igt@kms_hdr@static-toggle-suspend:
    - shard-tglb:         NOTRUN -> [SKIP][68] ([i915#1187])
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb3/igt@kms_hdr@static-toggle-suspend.html

  * igt@kms_plane_alpha_blend@pipe-b-alpha-7efc:
    - shard-kbl:          NOTRUN -> [FAIL][69] ([fdo#108145] / [i915#265]) +1 similar issue
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl7/igt@kms_plane_alpha_blend@pipe-b-alpha-7efc.html

  * igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max:
    - shard-apl:          NOTRUN -> [FAIL][70] ([fdo#108145] / [i915#265]) +3 similar issues
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl8/igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max.html

  * igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-1:
    - shard-tglb:         NOTRUN -> [SKIP][71] ([i915#2920]) +2 similar issues
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb3/igt@kms_psr2_sf@overlay-primary-update-sf-dmg-area-1.html

  * igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-2:
    - shard-apl:          NOTRUN -> [SKIP][72] ([fdo#109271] / [i915#658]) +1 similar issue
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl6/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-2.html

  * igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-4:
    - shard-kbl:          NOTRUN -> [SKIP][73] ([fdo#109271] / [i915#658]) +2 similar issues
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl4/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-4.html

  * igt@kms_psr2_su@frontbuffer:
    - shard-glk:          NOTRUN -> [SKIP][74] ([fdo#109271] / [i915#658])
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk6/igt@kms_psr2_su@frontbuffer.html

  * igt@kms_psr@psr2_basic:
    - shard-iclb:         [PASS][75] -> [SKIP][76] ([fdo#109441]) +1 similar issue
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-iclb2/igt@kms_psr@psr2_basic.html
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb3/igt@kms_psr@psr2_basic.html

  * igt@kms_psr@psr2_cursor_plane_onoff:
    - shard-tglb:         NOTRUN -> [FAIL][77] ([i915#132] / [i915#3467]) +1 similar issue
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb3/igt@kms_psr@psr2_cursor_plane_onoff.html

  * igt@kms_rotation_crc@exhaust-fences:
    - shard-iclb:         [PASS][78] -> [DMESG-WARN][79] ([i915#4391])
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-iclb1/igt@kms_rotation_crc@exhaust-fences.html
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb7/igt@kms_rotation_crc@exhaust-fences.html

  * igt@kms_rotation_crc@primary-yf-tiled-reflect-x-0:
    - shard-tglb:         NOTRUN -> [SKIP][80] ([fdo#111615]) +2 similar issues
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb6/igt@kms_rotation_crc@primary-yf-tiled-reflect-x-0.html

  * igt@kms_writeback@writeback-check-output:
    - shard-apl:          NOTRUN -> [SKIP][81] ([fdo#109271] / [i915#2437])
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl4/igt@kms_writeback@writeback-check-output.html
    - shard-kbl:          NOTRUN -> [SKIP][82] ([fdo#109271] / [i915#2437])
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl1/igt@kms_writeback@writeback-check-output.html

  * igt@prime_nv_pcopy@test2:
    - shard-kbl:          NOTRUN -> [SKIP][83] ([fdo#109271]) +120 similar issues
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl2/igt@prime_nv_pcopy@test2.html
    - shard-iclb:         NOTRUN -> [SKIP][84] ([fdo#109291])
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb4/igt@prime_nv_pcopy@test2.html

  * igt@prime_nv_pcopy@test3_1:
    - shard-tglb:         NOTRUN -> [SKIP][85] ([fdo#109291]) +2 similar issues
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb6/igt@prime_nv_pcopy@test3_1.html

  * igt@sysfs_clients@pidname:
    - shard-apl:          NOTRUN -> [SKIP][86] ([fdo#109271] / [i915#2994]) +1 similar issue
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl4/igt@sysfs_clients@pidname.html
    - shard-tglb:         NOTRUN -> [SKIP][87] ([i915#2994])
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-tglb5/igt@sysfs_clients@pidname.html

  
#### Possible fixes ####

  * igt@gem_eio@in-flight-suspend:
    - shard-kbl:          [DMESG-WARN][88] ([i915#180]) -> [PASS][89] +1 similar issue
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-kbl3/igt@gem_eio@in-flight-suspend.html
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl4/igt@gem_eio@in-flight-suspend.html

  * igt@gem_exec_fair@basic-deadline:
    - shard-glk:          [FAIL][90] ([i915#2846]) -> [PASS][91]
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-glk2/igt@gem_exec_fair@basic-deadline.html
   [91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk2/igt@gem_exec_fair@basic-deadline.html

  * igt@gem_exec_fair@basic-throttle@rcs0:
    - shard-glk:          [FAIL][92] ([i915#2842]) -> [PASS][93] +3 similar issues
   [92]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-glk8/igt@gem_exec_fair@basic-throttle@rcs0.html
   [93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk5/igt@gem_exec_fair@basic-throttle@rcs0.html

  * igt@gem_exec_whisper@basic-queues-forked-all:
    - shard-glk:          [DMESG-WARN][94] ([i915#118]) -> [PASS][95]
   [94]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-glk8/igt@gem_exec_whisper@basic-queues-forked-all.html
   [95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk4/igt@gem_exec_whisper@basic-queues-forked-all.html

  * igt@i915_pm_dc@dc9-dpms:
    - shard-apl:          [SKIP][96] ([fdo#109271]) -> [PASS][97]
   [96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-apl7/igt@i915_pm_dc@dc9-dpms.html
   [97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl2/igt@i915_pm_dc@dc9-dpms.html

  * igt@kms_cursor_crc@pipe-a-cursor-128x42-offscreen:
    - shard-iclb:         [FAIL][98] ([i915#3444]) -> [PASS][99]
   [98]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-iclb8/igt@kms_cursor_crc@pipe-a-cursor-128x42-offscreen.html
   [99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb5/igt@kms_cursor_crc@pipe-a-cursor-128x42-offscreen.html
    - shard-glk:          [FAIL][100] ([i915#1888] / [i915#3444]) -> [PASS][101]
   [100]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-glk8/igt@kms_cursor_crc@pipe-a-cursor-128x42-offscreen.html
   [101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk8/igt@kms_cursor_crc@pipe-a-cursor-128x42-offscreen.html

  * igt@kms_cursor_legacy@2x-long-flip-vs-cursor-legacy:
    - shard-glk:          [FAIL][102] ([i915#72]) -> [PASS][103]
   [102]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-glk1/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-legacy.html
   [103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk1/igt@kms_cursor_legacy@2x-long-flip-vs-cursor-legacy.html

  * igt@kms_flip@flip-vs-expired-vblank@c-hdmi-a1:
    - shard-glk:          [FAIL][104] ([i915#79]) -> [PASS][105]
   [104]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-glk8/igt@kms_flip@flip-vs-expired-vblank@c-hdmi-a1.html
   [105]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk7/igt@kms_flip@flip-vs-expired-vblank@c-hdmi-a1.html

  * igt@kms_flip@flip-vs-suspend-interruptible@c-dp1:
    - shard-apl:          [DMESG-WARN][106] ([i915#180]) -> [PASS][107] +3 similar issues
   [106]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-apl2/igt@kms_flip@flip-vs-suspend-interruptible@c-dp1.html
   [107]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-apl7/igt@kms_flip@flip-vs-suspend-interruptible@c-dp1.html

  * igt@kms_flip@flip-vs-suspend@c-dp1:
    - shard-kbl:          [INCOMPLETE][108] ([i915#636]) -> [PASS][109] +1 similar issue
   [108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-kbl2/igt@kms_flip@flip-vs-suspend@c-dp1.html
   [109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl7/igt@kms_flip@flip-vs-suspend@c-dp1.html

  * igt@kms_psr@psr2_primary_page_flip:
    - shard-iclb:         [SKIP][110] ([fdo#109441]) -> [PASS][111] +1 similar issue
   [110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-iclb4/igt@kms_psr@psr2_primary_page_flip.html
   [111]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb2/igt@kms_psr@psr2_primary_page_flip.html

  * igt@kms_setmode@basic:
    - shard-glk:          [FAIL][112] ([i915#31]) -> [PASS][113]
   [112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-glk3/igt@kms_setmode@basic.html
   [113]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-glk5/igt@kms_setmode@basic.html

  
#### Warnings ####

  * igt@gem_exec_fair@basic-pace@vcs0:
    - shard-kbl:          [SKIP][114] ([fdo#109271]) -> [FAIL][115] ([i915#2842])
   [114]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-kbl6/igt@gem_exec_fair@basic-pace@vcs0.html
   [115]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl2/igt@gem_exec_fair@basic-pace@vcs0.html

  * igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-4:
    - shard-iclb:         [SKIP][116] ([i915#2920]) -> [SKIP][117] ([i915#658]) +1 similar issue
   [116]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-iclb2/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-4.html
   [117]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb3/igt@kms_psr2_sf@overlay-plane-update-sf-dmg-area-4.html

  * igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1:
    - shard-iclb:         [SKIP][118] ([i915#658]) -> [SKIP][119] ([i915#2920]) +1 similar issue
   [118]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-iclb7/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1.html
   [119]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-iclb2/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-1.html

  * igt@runner@aborted:
    - shard-kbl:          ([FAIL][120], [FAIL][121], [FAIL][122]) ([i915#1436] / [i915#1814] / [i915#3002] / [i915#3363] / [i915#4312]) -> ([FAIL][123], [FAIL][124], [FAIL][125], [FAIL][126]) ([i915#1436] / [i915#180] / [i915#3002] / [i915#3363] / [i915#4312] / [i915#92])
   [120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-kbl3/igt@runner@aborted.html
   [121]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-kbl3/igt@runner@aborted.html
   [122]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10879/shard-kbl6/igt@runner@aborted.html
   [123]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl4/igt@runner@aborted.html
   [124]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl3/igt@runner@aborted.html
   [125]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl3/igt@runner@aborted.html
   [126]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/shard-kbl3/igt@runner@aborted.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145
  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [fdo#109274]: https://bugs.freedesktop.org/show_bug.cgi?id=109274
  [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
  [fdo#109279]: https://bugs.freedesktop.org/show_bug.cgi?id=109279
  [fdo#109280]: https://bugs.freedesktop.org/show_bug.cgi?id=109280
  [fdo#109284]: https://bugs.freedesktop.org/show_bug.cgi?id=109284
  [fdo#109289]: https://bugs.freedesktop.org/show_bug.cgi?id=109289
  [fdo#109291]: https://bugs.freedesktop.org/show_bug.cgi?id=109291
  [fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
  [fdo#111614]: https://bugs.freedesktop.org/show_bug.cgi?id=111614
  [fdo#111615]: https://bugs.freedesktop.org/show_bug.cgi?id=111615
  [fdo#111825]: https://bugs.freedesktop.org/show_bug.cgi?id=111825
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#118]: https://gitlab.freedesktop.org/drm/intel/issues/118
  [i915#1187]: https://gitlab.freedesktop.org/drm/intel/issues/1187
  [i915#1319]: https://gitlab.freedesktop.org/drm/intel/issues/1319
  [i915#132]: https://gitlab.freedesktop.org/drm/intel/issues/132
  [i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
  [i915#180]: https://gitlab.freedesktop.org/drm/intel/issues/180
  [i915#1814]: https://gitlab.freedesktop.org/drm/intel/issues/1814
  [i915#1888]: https://gitlab.freedesktop.org/drm/intel/issues/1888
  [i915#1895]: https://gitlab.freedesktop.org/drm/intel/issues/1895
  [i915#1902]: https://gitlab.freedesktop.org/drm/intel/issues/1902
  [i915#2105]: https://gitlab.freedesktop.org/drm/intel/issues/2105
  [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
  [i915#2346]: https://gitlab.freedesktop.org/drm/intel/issues/2346
  [i915#2410]: https://gitlab.freedesktop.org/drm/intel/issues/

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6397/index.html

[-- Attachment #2: Type: text/html, Size: 36800 bytes --]

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_lmem_swapping: Add the gem_lmem_swapping test
  2021-11-15  7:16 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_lmem_swapping: Add the gem_lmem_swapping test Zbigniew Kempczyński
@ 2021-11-16 10:31   ` Matthew Auld
  0 siblings, 0 replies; 11+ messages in thread
From: Matthew Auld @ 2021-11-16 10:31 UTC (permalink / raw)
  To: Zbigniew Kempczyński; +Cc: igt-dev

On Mon, 15 Nov 2021 at 07:16, Zbigniew Kempczyński
<zbigniew.kempczynski@intel.com> wrote:
>
> From: CQ Tang <cq.tang@intel.com>
>
> This test exercises the lmem eviction functionality with one
> dynamic subtest per lmem region.
>
> In essence the various subtests create lmem buffers objects consuming more
> lmem memory than available and check that they are successful submitting
> workload subsets whose lmem size requirements are smaller than the total
> amount of lmem available, forcing eviction of buffer objects under
> various conditions.
>
> v2: - addressing review comments (handling seed, minor cleanups) (Zbigniew)
>     - add dynamic_lmem_subtest() macro (Maarten)
>
> v3: - make seed unique for each process (Zbigniew)
>     - add simple igt_describe() (Petri)
>
> Signed-off-by: CQ Tang <cq.tang@intel.com>
> Co-developed-by: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
> Co-developed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
> Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
> Co-developed-by: Ayaz A Siddiqui <ayaz.siddiqui@intel.com>
> Signed-off-by: Ayaz A Siddiqui <ayaz.siddiqui@intel.com>
> Co-developed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> Co-developed-by: Chris Wilson <chris@chris-wilson.co.uk>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
> Co-developed-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> Acked-by: Petri Latvala <petri.latvala@intel.com>
> ---
>  tests/i915/gem_lmem_swapping.c | 550 +++++++++++++++++++++++++++++++++
>  tests/meson.build              |   1 +
>  2 files changed, 551 insertions(+)
>  create mode 100644 tests/i915/gem_lmem_swapping.c
>
> diff --git a/tests/i915/gem_lmem_swapping.c b/tests/i915/gem_lmem_swapping.c
> new file mode 100644
> index 000000000..786110338
> --- /dev/null
> +++ b/tests/i915/gem_lmem_swapping.c
> @@ -0,0 +1,550 @@
> +// SPDX-License-Identifier: MIT
> +/*
> + * Copyright © 2021 Intel Corporation
> + */
> +
> +#include "i915/gem.h"
> +#include "i915/gem_create.h"
> +#include "i915/gem_vm.h"
> +#include "i915/intel_memory_region.h"
> +#include "igt.h"
> +#include "igt_kmod.h"
> +#include <unistd.h>
> +#include <stdlib.h>
> +#include <stdint.h>
> +#include <stdio.h>
> +#include <string.h>
> +#include <fcntl.h>
> +#include <inttypes.h>
> +#include <errno.h>
> +#include <sys/stat.h>
> +#include <sys/ioctl.h>
> +#include <sys/time.h>
> +#include <sys/wait.h>
> +#include "drm.h"
> +
> +IGT_TEST_DESCRIPTION("Exercise local memory swapping.");
> +
> +#define __round_mask(x, y) ((__typeof__(x))((y) - 1))
> +#define round_up(x, y) ((((x) - 1) | __round_mask(x, y)) + 1)
> +
> +#define PAGE_SIZE  (1ULL << 12)
> +#define SZ_64K    (16 * PAGE_SIZE)
> +
> +static const char *readable_unit(uint64_t size)
> +{
> +       return size >> 20 ? "MiB" : size >> 10 ? "KiB" : "Bytes";
> +}
> +
> +static uint64_t readable_size(uint64_t size)
> +{
> +       return size >> 20 ? size >> 20 : size >> 10 ? size >> 10 : size;
> +}
> +
> +struct {
> +       unsigned int seed;
> +       bool user_seed;
> +} opt;
> +
> +struct params {
> +       struct {
> +               uint64_t min;
> +               uint64_t max;
> +       } size;
> +       unsigned int count;
> +       unsigned int loops;
> +       unsigned int mem_limit;
> +#define TEST_VERIFY    (1 << 0)
> +#define TEST_PARALLEL  (1 << 1)
> +#define TEST_HEAVY     (1 << 2)
> +#define TEST_RANDOM    (1 << 3)
> +#define TEST_ENGINES   (1 << 4)
> +#define TEST_MULTI     (1 << 5)
> +       unsigned int flags;
> +       unsigned int seed;
> +       bool oom_test;
> +};
> +
> +struct object {
> +       uint64_t size;
> +       uint32_t seed;
> +       uint32_t handle;
> +};
> +
> +static uint32_t create_bo(int i915,
> +                         uint64_t size,
> +                         struct drm_i915_gem_memory_class_instance *region,
> +                         bool do_oom_test)
> +{
> +       uint32_t handle;
> +       int ret;
> +
> +retry:
> +       ret = __gem_create_in_memory_region_list(i915, &handle, &size, region, 1);
> +       if (do_oom_test && (ret == -ENOMEM || ret == -ENXIO))

AFAIK __gem_create doesn't allocate any pages(at least on drm-tip),
just a "shell" object, so we shouldn't expect to see something like
-ENXIO here. Hopefully this doesn't change the meaning of some test
below using this?

> +               goto retry;
> +       igt_assert_eq(ret, 0);
> +       return handle;
> +}
> +
> +static unsigned int __num_engines__;
> +
> +static void
> +init_object(int i915, struct object *obj, unsigned long seed, unsigned int flags)
> +{
> +       unsigned int j;
> +       uint32_t *buf;
> +
> +       obj->seed = seed;
> +
> +       buf = gem_mmap_offset__fixed(i915, obj->handle, 0, obj->size, PROT_WRITE);
> +
> +       for (j = 0; j < obj->size / sizeof(*buf); j++)
> +               buf[j] = seed++;
> +
> +       munmap(buf, obj->size);
> +}
> +
> +static void
> +verify_object(int i915, const struct object *obj,  unsigned int flags)
> +{
> +       unsigned long j;
> +       uint32_t *buf;
> +
> +       buf = gem_mmap_offset__fixed(i915, obj->handle, 0, obj->size, PROT_READ);
> +
> +       for (j = 0; j < obj->size / PAGE_SIZE; j++) {
> +               unsigned long x = (j * PAGE_SIZE + rand() % PAGE_SIZE) / sizeof(*buf);
> +               uint32_t val = obj->seed + x;
> +
> +               igt_assert_f(buf[x] == val,
> +                            "Object mismatch at offset %zu - found %08x, expected %08x; difference:%08x!\n",
> +                            x * sizeof(*buf), buf[x], val, buf[x] ^ val);
> +       }
> +
> +       munmap(buf, obj->size);
> +}
> +
> +static void move_to_lmem(int i915,
> +                        struct object *list,
> +                        unsigned int num,
> +                        uint32_t batch,
> +                        unsigned int engine,
> +                        bool do_oom_test)
> +{
> +       struct drm_i915_gem_exec_object2 obj[1 + num];
> +       struct drm_i915_gem_execbuffer2 eb = {
> +               .buffers_ptr = to_user_pointer(obj),
> +               .buffer_count = 1 + num,
> +               .flags = I915_EXEC_NO_RELOC | I915_EXEC_HANDLE_LUT | engine,
> +       };
> +       unsigned int i, ret;
> +
> +       memset(obj, 0, sizeof(obj));
> +
> +       for (i = 0; i < num; i++) {
> +               obj[i].handle = list[i].handle;
> +               obj[i].flags = EXEC_OBJECT_SUPPORTS_48B_ADDRESS;
> +       }
> +
> +       obj[i].handle = batch;
> +retry:
> +       ret = __gem_execbuf(i915, &eb);
> +       if (do_oom_test && (ret == -ENOMEM || ret == -ENXIO))
> +               goto retry;
> +       igt_assert_eq(ret, 0);
> +}
> +
> +static void __do_evict(int i915,
> +                      struct drm_i915_gem_memory_class_instance *region,
> +                      struct params *params,
> +                      unsigned int seed)
> +{
> +       const unsigned int max_swap_in = params->count / 100 + 1;
> +       const uint32_t bbe = MI_BATCH_BUFFER_END;
> +       struct object *objects, *obj, *list;
> +       uint32_t batch;
> +       unsigned int engine = 0;
> +       unsigned int i, l;
> +       uint64_t size;
> +       struct timespec t = {};
> +       unsigned int num;
> +
> +       __gem_context_set_persistence(i915, 0, false);
> +       size = 4096;
> +       batch = create_bo(i915, size, region, params->oom_test);
> +
> +       gem_write(i915, batch, 0, &bbe, sizeof(bbe));
> +
> +       objects = calloc(params->count, sizeof(*objects));
> +       igt_assert(objects);
> +
> +       list = calloc(max_swap_in, sizeof(*list));
> +       igt_assert(list);
> +
> +       srand(seed);
> +
> +       /* Create the initial working set of objects. */
> +
> +       size = 0;
> +       for (i = 0, obj = objects; i < params->count; i++, obj++) {
> +               if (params->flags & TEST_RANDOM)
> +                       obj->size = rand() %
> +                               (params->size.max - params->size.min) +
> +                               params->size.min;
> +               else
> +                       obj->size = params->size.min;
> +
> +               size += obj->size;
> +               if ((size >> 20) > params->mem_limit) {
> +                       params->count = i;
> +                       break;
> +               }
> +               obj->handle = create_bo(i915, obj->size, region, params->oom_test);
> +
> +               if (params->flags & TEST_VERIFY)
> +                       init_object(i915, obj, rand(), params->flags);
> +               else
> +                       move_to_lmem(i915, objects + i, 1, batch, engine,
> +                                    params->oom_test);
> +       }
> +
> +       igt_debug("obj size min/max=%lu %s/%lu %s, count=%u, seed: %u\n",
> +                 readable_size(params->size.min), readable_unit(params->size.min),
> +                 readable_size(params->size.max), readable_unit(params->size.max),
> +                 params->count, seed);
> +
> +       /*
> +        * Move random objects back into lmem.
> +        * for TEST_MULTI runs, make each object count s a loop to
> +        * avoid excessive run times

Strange formatting?

> +        */
> +       for (l = 0; l < params->loops && igt_seconds_elapsed(&t) < 300; l += num) {
> +               unsigned int idx = rand() % params->count;
> +
> +               num = params->flags & TEST_MULTI ? rand() % max_swap_in + 1 : 1;
> +               for (i = 0; i < num; i++) {
> +                       list[i] = objects[idx];
> +                       idx = (idx + 1) % params->count;
> +               }
> +
> +               move_to_lmem(i915, list, num, batch, engine, params->oom_test);
> +
> +               if (params->flags & TEST_ENGINES)
> +                       engine = (engine + 1) % __num_engines__;
> +
> +               if (params->flags & TEST_VERIFY) {
> +                       for (i = 0; i < num; i++)
> +                               verify_object(i915, &list[i], params->flags);
> +
> +                       /* Update random object - may swap it back in. */
> +                       i = rand() % params->count;
> +                       init_object(i915, &objects[i], rand(), params->flags);
> +               }
> +       }
> +
> +       for (i = 0; i < params->count; i++)
> +               gem_close(i915, objects[i].handle);
> +
> +       free(list);
> +       free(objects);
> +
> +       gem_close(i915, batch);
> +}
> +
> +static void fill_params(int i915, struct params *params,
> +                       struct drm_i915_memory_region_info *region,
> +                       unsigned int flags,
> +                       unsigned int nproc,
> +                       bool do_oom_test)
> +{
> +       const int swap_mb = /* For lmem, swap is total of smem + swap. */
> +               intel_get_total_ram_mb() + intel_get_total_swap_mb();
> +       const unsigned int size = 1 << 20;
> +       const int max_swap_pct = 75;
> +       /*
> +        * In random mode, add 85% hard limit to use system memory.
> +        * noticed that 88.8% can trigger OOM on some system.
> +        */
> +       const int mem_limit_pct = 85;
> +       int spill_mb;
> +       uint32_t handle;
> +
> +       if (flags & TEST_RANDOM) {
> +               params->size.min = 4096;
> +               handle = create_bo(i915, params->size.min, &region->region,
> +                                  do_oom_test);
> +               gem_close(i915, handle);
> +               params->size.max = 2 * size + params->size.min;
> +       } else {
> +               params->size.min = size;
> +               params->size.max = size;
> +       }
> +
> +       params->count = (region->probed_size + (size - 1)) / size * 3 / 2;
> +       spill_mb = (size >> 20) * params->count - (region->probed_size >> 20);
> +       /* Don't use all RAM for swapout. */
> +       igt_require(spill_mb <= swap_mb * max_swap_pct / 100);
> +
> +       if (flags & TEST_HEAVY) {
> +               params->count *= 2;
> +               spill_mb = (size >> 20) * params->count -
> +                       (region->probed_size >> 20);
> +
> +               if (spill_mb > swap_mb * max_swap_pct / 100) {
> +                       unsigned int count;
> +                       unsigned long set;
> +
> +                       igt_warn("Reducing working set due low RAM + swap! (Need %d MiB, have %d MiB.)\n",
> +                                spill_mb, swap_mb);
> +                       set = region->probed_size +
> +                               (((unsigned long)swap_mb * max_swap_pct / 100) << 20);
> +                       count = set / size;
> +                       /* No point if heavy test is too similar to normal. */
> +                       igt_require(count > (params->count / 2) * 133 / 100);
> +                       params->count = count;
> +               }
> +       }
> +
> +       params->loops = params->count;
> +       params->seed = opt.user_seed ? opt.seed : time(NULL);
> +
> +       /*
> +        * If run in parallel, reduce per process buffer count to keep the
> +        * total the same, but don't reduce loops since we gain some
> +        * efficiency by the parallel execution
> +        */
> +       if (flags & TEST_PARALLEL)
> +               params->count /= nproc;
> +
> +       /*
> +        * For heavy tests, reduce the loop count to avoid excessive
> +        * run-times
> +        */
> +       if (flags & TEST_HEAVY)
> +               params->loops = params->loops / 2 + 1;
> +
> +       params->flags = flags;
> +       params->oom_test = do_oom_test;
> +
> +       params->mem_limit = swap_mb * mem_limit_pct / 100 +
> +               (region->probed_size >> 20);
> +       igt_info("Memory: system-total %dMiB, lmem-region %lldMiB, usage-limit %dMiB\n",
> +                swap_mb, (region->probed_size >> 20), params->mem_limit);
> +       igt_info("Using %u thread(s), %u loop(s), %u objects of %lu %s - %lu %s, seed: %u, oom: %s\n",
> +                params->flags & TEST_PARALLEL ? nproc : 1,
> +                params->loops,
> +                params->count,
> +                readable_size(params->size.min),
> +                readable_unit(params->size.min),
> +                readable_size(params->size.max),
> +                readable_unit(params->size.max),
> +                params->seed,
> +                do_oom_test ? "yes" : "no");
> +}
> +
> +static void test_evict(int i915,
> +                      struct drm_i915_memory_region_info *region,
> +                      unsigned int flags)
> +{
> +       const unsigned int nproc = sysconf(_SC_NPROCESSORS_ONLN) + 1;
> +       struct params params;
> +
> +       fill_params(i915, &params, region, flags, nproc, false);
> +
> +       if (flags & TEST_PARALLEL) {
> +               int fd = gem_reopen_driver(i915);
> +
> +               igt_fork(child, nproc)
> +                       __do_evict(fd, &region->region, &params,
> +                                  params.seed + child + 1);
> +
> +               igt_waitchildren();
> +               close(fd);
> +       } else {
> +               __do_evict(i915, &region->region, &params, params.seed);
> +       }
> +}
> +
> +static void leak(uint64_t alloc)
> +{
> +       char *ptr;
> +
> +       ptr = mmap(NULL, alloc, PROT_READ | PROT_WRITE,
> +                  MAP_ANON | MAP_PRIVATE | MAP_POPULATE, -1, 0);
> +       if (ptr == MAP_FAILED)
> +               return;
> +
> +       while (alloc) {
> +               alloc -= 4096;
> +               ptr[alloc] = 0;
> +       }
> +}
> +
> +static void gem_leak(int fd, uint64_t alloc)
> +{
> +       uint32_t handle = gem_create(fd, alloc);
> +       void *buf;
> +
> +       buf = gem_mmap_offset__fixed(fd, handle, 0, PAGE_SIZE, PROT_WRITE);
> +       memset(buf, 0, PAGE_SIZE);
> +       munmap(buf, PAGE_SIZE);
> +
> +       gem_madvise(fd, handle, I915_MADV_DONTNEED);

Don't know if it matters much, but this doesn't really "leak" anything
AFAIK. It just allocates the pages and then drops them immediately in
the madvise, due to differences in how the mmap works in the ttm vs
the old stuff(the ttm fault handler doesn't currently set the
mm.pages). I guess it depends on what the original meaning of this
"gem_leak" was.

Anyway,
Reviewed-by: Matthew Auld <matthew.auld@intel.com>

> +}
> +
> +static int *lmem_done;
> +
> +static void smem_oom_exit_handler(int sig)
> +{
> +       (*lmem_done)++;
> +}
> +
> +static void test_smem_oom(int i915,
> +                         struct drm_i915_memory_region_info *region)
> +{
> +       const uint64_t smem_size = intel_get_total_ram_mb() +
> +               intel_get_total_swap_mb();
> +       const unsigned int alloc = 256 * 1024 * 1024;
> +       const unsigned int num_alloc = 1 + smem_size / (alloc >> 20);
> +       struct igt_helper_process smem_proc = {};
> +       unsigned int n;
> +
> +       lmem_done = mmap(0, sizeof(*lmem_done), PROT_WRITE,
> +                        MAP_SHARED | MAP_ANON, -1, 0);
> +       igt_assert(lmem_done != MAP_FAILED);
> +       *lmem_done = 0;
> +
> +       /* process for testing lmem eviction */
> +       igt_fork(child, 1) {
> +               int fd = gem_reopen_driver(i915);
> +               struct params params;
> +
> +               fill_params(i915, &params, region, 0, 1, true);
> +
> +               igt_install_exit_handler(smem_oom_exit_handler);
> +               __do_evict(fd, &region->region, &params,
> +                          params.seed + child + 1);
> +
> +               close(fd);
> +       }
> +
> +       /* smem memory hog process, respawn till the lmem process completes */
> +       while (!READ_ONCE(*lmem_done)) {
> +               igt_fork_helper(&smem_proc) {
> +                       igt_fork(child, 1) {
> +                               for (int pass = 0; pass < num_alloc; pass++) {
> +                                       if (READ_ONCE(*lmem_done))
> +                                               break;
> +                                       leak(alloc);
> +                               }
> +                       }
> +                       igt_fork(child, 1) {
> +                               int fd = gem_reopen_driver(i915);
> +
> +                               for (int pass = 0; pass < num_alloc; pass++) {
> +                                       if (READ_ONCE(*lmem_done))
> +                                               break;
> +                                       gem_leak(fd, alloc);
> +                               }
> +                               close(fd);
> +                       }
> +                       /*
> +                        * Wait for grand-child processes to finish or be
> +                        * killed by the oom killer, don't call
> +                        * igt_waitchildren because of the noise
> +                        */
> +                       for (n = 0; n < 2; n++)
> +                               wait(NULL);
> +               }
> +               igt_wait_helper(&smem_proc);
> +       }
> +       munmap(lmem_done, sizeof(*lmem_done));
> +       /* Reap exit status of the lmem process */
> +       igt_waitchildren();
> +}
> +
> +#define dynamic_lmem_subtest(reg, regs, subtest_name...) \
> +       igt_subtest_with_dynamic(subtest_name) \
> +               for (unsigned int i = 0; i < (regs)->num_regions; i++) \
> +                       for_each_if (((reg) = &(regs)->regions[i])->region.memory_class == I915_MEMORY_CLASS_DEVICE) \
> +                               igt_dynamic_f("lmem%u", (reg)->region.memory_instance)
> +
> +static int opt_handler(int option, int option_index, void *input)
> +{
> +       switch (option) {
> +       case 's':
> +               opt.user_seed = true;
> +               opt.seed = strtoul(optarg, NULL, 0);
> +               break;
> +       default:
> +               return IGT_OPT_HANDLER_ERROR;
> +       }
> +
> +       return IGT_OPT_HANDLER_SUCCESS;
> +}
> +
> +const char *help_str =
> +       "  --seed       Seed for random number generator";
> +
> +struct option long_options[] = {
> +       { "seed",    required_argument, NULL, 's'},
> +       { 0, 0, 0, 0 }
> +};
> +
> +igt_main_args("", long_options, help_str, opt_handler, NULL)
> +{
> +       struct drm_i915_query_memory_regions *regions;
> +       struct drm_i915_memory_region_info *region;
> +       struct test {
> +               const char *name;
> +               unsigned int flags;
> +       } *test, tests[] = {
> +               { "basic", 0 },
> +               { "random", TEST_RANDOM },
> +               { "random-engines", TEST_RANDOM | TEST_ENGINES },
> +               { "heavy-random", TEST_RANDOM | TEST_HEAVY },
> +               { "heavy-multi", TEST_RANDOM | TEST_HEAVY | TEST_ENGINES | TEST_MULTI },
> +               { "verify", TEST_VERIFY},
> +               { "verify-random", TEST_VERIFY | TEST_RANDOM},
> +               { "heavy-verify-random", TEST_VERIFY | TEST_RANDOM | TEST_HEAVY },
> +               { "heavy-verify-multi", TEST_VERIFY | TEST_RANDOM | TEST_HEAVY | TEST_ENGINES | TEST_MULTI },
> +               { "parallel-random", TEST_PARALLEL | TEST_RANDOM },
> +               { "parallel-random-engines", TEST_PARALLEL | TEST_RANDOM | TEST_ENGINES },
> +               { "parallel-random-verify", TEST_PARALLEL | TEST_RANDOM | TEST_VERIFY },
> +               { "parallel-multi", TEST_PARALLEL | TEST_RANDOM | TEST_VERIFY | TEST_ENGINES | TEST_MULTI },
> +               { }
> +       };
> +       int i915 = -1;
> +
> +       igt_fixture {
> +               struct intel_execution_engine2 *e;
> +
> +               i915 = drm_open_driver(DRIVER_INTEL);
> +               igt_require_gem(i915);
> +
> +               regions = gem_get_query_memory_regions(i915);
> +               igt_require(regions);
> +
> +               for_each_physical_engine(i915, e)
> +                       __num_engines__++;
> +               igt_require(__num_engines__);
> +       }
> +
> +       for (test = tests; test->name; test++) {
> +               igt_describe("Exercise local memory swapping to system memory");
> +               dynamic_lmem_subtest(region, regions, test->name)
> +                       test_evict(i915, region, test->flags);
> +       }
> +
> +       igt_describe("Exercise local memory swapping during exhausting system memory");
> +       dynamic_lmem_subtest(region, regions, "smem-oom")
> +               test_smem_oom(i915, region);
> +
> +       igt_fixture {
> +               free(regions);
> +               close(i915);
> +       }
> +
> +       igt_exit();
> +}
> diff --git a/tests/meson.build b/tests/meson.build
> index 0af3e03a3..49c0f2d24 100644
> --- a/tests/meson.build
> +++ b/tests/meson.build
> @@ -156,6 +156,7 @@ i915_progs = [
>         'gem_gtt_speed',
>         'gem_huc_copy',
>         'gem_linear_blits',
> +       'gem_lmem_swapping',
>         'gem_lut_handle',
>         'gem_madvise',
>         'gem_media_fill',
> --
> 2.26.0
>

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add four subtests of gem_lmem_swapping to BAT
  2021-11-15  7:16 ` [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add four subtests of gem_lmem_swapping to BAT Zbigniew Kempczyński
@ 2021-11-16 11:07   ` Matthew Auld
  2021-11-16 12:10     ` Petri Latvala
  0 siblings, 1 reply; 11+ messages in thread
From: Matthew Auld @ 2021-11-16 11:07 UTC (permalink / raw)
  To: Zbigniew Kempczyński; +Cc: igt-dev

On Mon, 15 Nov 2021 at 07:16, Zbigniew Kempczyński
<zbigniew.kempczynski@intel.com> wrote:
>
> Add basic and some random subtests to BAT to verify early lmem swapping
> is working on discrete.
>
> Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> Cc: Petri Latvala <petri.latvala@intel.com>
> Cc: Tomi Sarvela <tomi.p.sarvela@intel.com>

Acked-by: Matthew Auld <matthew.auld@intel.com>

According to the logs the below adds ~100+ seconds to BAT on DG1,
hopefully that is acceptable.

> ---
>  tests/intel-ci/fast-feedback.testlist | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
> index fa5006d2e..de1c6cb46 100644
> --- a/tests/intel-ci/fast-feedback.testlist
> +++ b/tests/intel-ci/fast-feedback.testlist
> @@ -34,6 +34,10 @@ igt@gem_flink_basic@double-flink
>  igt@gem_flink_basic@flink-lifetime
>  igt@gem_huc_copy@huc-copy
>  igt@gem_linear_blits@basic
> +igt@gem_lmem_swapping@basic
> +igt@gem_lmem_swapping@parallel-random-engines
> +igt@gem_lmem_swapping@random-engines
> +igt@gem_lmem_swapping@verify-random
>  igt@gem_mmap@basic
>  igt@gem_mmap_gtt@basic
>  igt@gem_render_linear_blits@basic
> --
> 2.26.0
>

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add four subtests of gem_lmem_swapping to BAT
  2021-11-16 11:07   ` Matthew Auld
@ 2021-11-16 12:10     ` Petri Latvala
  0 siblings, 0 replies; 11+ messages in thread
From: Petri Latvala @ 2021-11-16 12:10 UTC (permalink / raw)
  To: Matthew Auld; +Cc: igt-dev

On Tue, Nov 16, 2021 at 11:07:46AM +0000, Matthew Auld wrote:
> On Mon, 15 Nov 2021 at 07:16, Zbigniew Kempczyński
> <zbigniew.kempczynski@intel.com> wrote:
> >
> > Add basic and some random subtests to BAT to verify early lmem swapping
> > is working on discrete.
> >
> > Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
> > Cc: Petri Latvala <petri.latvala@intel.com>
> > Cc: Tomi Sarvela <tomi.p.sarvela@intel.com>
> 
> Acked-by: Matthew Auld <matthew.auld@intel.com>
> 
> According to the logs the below adds ~100+ seconds to BAT on DG1,
> hopefully that is acceptable.

That doesn't make DG1 the slowest BAT machine (chromebooks with their
eMMC disks still have that crown) and if the situation changes we can
revisit.

My

Acked-by: Petri Latvala <petri.latvala@intel.com>

from before still stands.

> 
> > ---
> >  tests/intel-ci/fast-feedback.testlist | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
> > index fa5006d2e..de1c6cb46 100644
> > --- a/tests/intel-ci/fast-feedback.testlist
> > +++ b/tests/intel-ci/fast-feedback.testlist
> > @@ -34,6 +34,10 @@ igt@gem_flink_basic@double-flink
> >  igt@gem_flink_basic@flink-lifetime
> >  igt@gem_huc_copy@huc-copy
> >  igt@gem_linear_blits@basic
> > +igt@gem_lmem_swapping@basic
> > +igt@gem_lmem_swapping@parallel-random-engines
> > +igt@gem_lmem_swapping@random-engines
> > +igt@gem_lmem_swapping@verify-random
> >  igt@gem_mmap@basic
> >  igt@gem_mmap_gtt@basic
> >  igt@gem_render_linear_blits@basic
> > --
> > 2.26.0
> >

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [igt-dev] [PATCH i-g-t 0/2] Upstream gem_lmem_swapping
@ 2021-11-10 13:17 Zbigniew Kempczyński
  0 siblings, 0 replies; 11+ messages in thread
From: Zbigniew Kempczyński @ 2021-11-10 13:17 UTC (permalink / raw)
  To: igt-dev

Address review comments from Maarten + add "basic" to BAT

v2: fix seed handling when more process are involved in creating
    and filling objects
    add igt_describe()

CQ Tang (1):
  tests/i915/gem_lmem_swapping: Add the gem_lmem_swapping test

Zbigniew Kempczyński (1):
  tests/intel-ci/fast-feedback: Add gem_lmem_swapping@basic to BAT

 tests/i915/gem_lmem_swapping.c        | 551 ++++++++++++++++++++++++++
 tests/intel-ci/fast-feedback.testlist |   1 +
 tests/meson.build                     |   1 +
 3 files changed, 553 insertions(+)
 create mode 100644 tests/i915/gem_lmem_swapping.c

-- 
2.26.0

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [igt-dev] [PATCH i-g-t 0/2] Upstream gem_lmem_swapping
@ 2021-11-09 10:33 Zbigniew Kempczyński
  0 siblings, 0 replies; 11+ messages in thread
From: Zbigniew Kempczyński @ 2021-11-09 10:33 UTC (permalink / raw)
  To: igt-dev

Address review comments from Maarten + add "basic" to BAT

CQ Tang (1):
  tests/i915/gem_lmem_swapping: Add the gem_lmem_swapping test

Zbigniew Kempczyński (1):
  tests/intel-ci/fast-feedback: Add gem_lmem_swapping@basic to BAT

 tests/i915/gem_lmem_swapping.c        | 543 ++++++++++++++++++++++++++
 tests/intel-ci/fast-feedback.testlist |   1 +
 tests/meson.build                     |   1 +
 3 files changed, 545 insertions(+)
 create mode 100644 tests/i915/gem_lmem_swapping.c

-- 
2.26.0

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [igt-dev] [PATCH i-g-t 0/2] Upstream gem_lmem_swapping
@ 2021-11-09 10:33 Zbigniew Kempczyński
  0 siblings, 0 replies; 11+ messages in thread
From: Zbigniew Kempczyński @ 2021-11-09 10:33 UTC (permalink / raw)
  To: igt-dev

Address review comments from Maarten + add "basic" to BAT

CQ Tang (1):
  tests/i915/gem_lmem_swapping: Add the gem_lmem_swapping test

Zbigniew Kempczyński (1):
  tests/intel-ci/fast-feedback: Add gem_lmem_swapping@basic to BAT

 tests/i915/gem_lmem_swapping.c        | 543 ++++++++++++++++++++++++++
 tests/intel-ci/fast-feedback.testlist |   1 +
 tests/meson.build                     |   1 +
 3 files changed, 545 insertions(+)
 create mode 100644 tests/i915/gem_lmem_swapping.c

-- 
2.26.0

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2021-11-16 12:12 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-15  7:16 [igt-dev] [PATCH i-g-t 0/2] Upstream gem_lmem_swapping Zbigniew Kempczyński
2021-11-15  7:16 ` [igt-dev] [PATCH i-g-t 1/2] tests/i915/gem_lmem_swapping: Add the gem_lmem_swapping test Zbigniew Kempczyński
2021-11-16 10:31   ` Matthew Auld
2021-11-15  7:16 ` [igt-dev] [PATCH i-g-t 2/2] tests/intel-ci/fast-feedback: Add four subtests of gem_lmem_swapping to BAT Zbigniew Kempczyński
2021-11-16 11:07   ` Matthew Auld
2021-11-16 12:10     ` Petri Latvala
2021-11-15  8:05 ` [igt-dev] ✓ Fi.CI.BAT: success for Upstream gem_lmem_swapping (rev4) Patchwork
2021-11-15 11:17 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
  -- strict thread matches above, loose matches on Subject: below --
2021-11-10 13:17 [igt-dev] [PATCH i-g-t 0/2] Upstream gem_lmem_swapping Zbigniew Kempczyński
2021-11-09 10:33 Zbigniew Kempczyński
2021-11-09 10:33 Zbigniew Kempczyński

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.