* [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests
@ 2021-06-08 9:40 Daniel Vetter
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 2/7] Revert "test/i915/gem_exec_reloc: Restore interclient testings" Daniel Vetter
` (11 more replies)
0 siblings, 12 replies; 20+ messages in thread
From: Daniel Vetter @ 2021-06-08 9:40 UTC (permalink / raw)
To: IGT development; +Cc: Petri Latvala, Daniel Vetter
These have been banned across all CI for over 2 years, realistically
that's not going to change and they're just in the way.
Cc: Jason Ekstrand <jason@jlekstrand.net>
Cc: Petri Latvala <petri.latvala@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
tests/i915/gem_exec_reloc.c | 31 +++++--------------------------
tests/intel-ci/blacklist.txt | 1 -
2 files changed, 5 insertions(+), 27 deletions(-)
diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
index 397c335e4d3c..cb4899fe6e1e 100644
--- a/tests/i915/gem_exec_reloc.c
+++ b/tests/i915/gem_exec_reloc.c
@@ -614,8 +614,6 @@ static bool has_64b_reloc(int fd)
#define NORELOC 1
#define ACTIVE 2
-#define INTERRUPTIBLE 4
-#define HANG 8
static void basic_reloc(int fd, unsigned before, unsigned after, unsigned flags)
{
#define OBJSZ 8192
@@ -678,8 +676,7 @@ static void basic_reloc(int fd, unsigned before, unsigned after, unsigned flags)
spin = igt_spin_new(fd,
.engine = I915_EXEC_DEFAULT,
.dependency = obj.handle);
- if (!(flags & HANG))
- igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
+
igt_assert(gem_bo_busy(fd, obj.handle));
}
@@ -746,8 +743,6 @@ static void basic_reloc(int fd, unsigned before, unsigned after, unsigned flags)
spin = igt_spin_new(fd,
.engine = I915_EXEC_DEFAULT,
.dependency = obj.handle);
- if (!(flags & HANG))
- igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
igt_assert(gem_bo_busy(fd, obj.handle));
}
@@ -873,8 +868,6 @@ static void basic_range(int fd, unsigned flags)
if (flags & ACTIVE) {
spin = igt_spin_new(fd, .dependency = obj[n].handle);
- if (!(flags & HANG))
- igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
igt_assert(gem_bo_busy(fd, obj[n].handle));
}
@@ -1468,8 +1461,6 @@ igt_main
{ "", 0 , true},
{ "-noreloc", NORELOC, true },
{ "-active", ACTIVE, true },
- { "-interruptible", ACTIVE | INTERRUPTIBLE },
- { "-hang", ACTIVE | HANG },
{ },
}, *f;
uint64_t size;
@@ -1483,14 +1474,7 @@ igt_main
}
for (f = flags; f->name; f++) {
- igt_hang_t hang;
-
igt_subtest_group {
- igt_fixture {
- if (f->flags & HANG)
- hang = igt_allow_hang(fd, 0, 0);
- }
-
for (m = modes; m->name; m++) {
igt_subtest_f("%s%s%s",
f->basic ? "basic-" : "",
@@ -1498,22 +1482,17 @@ igt_main
f->name) {
if ((m->before | m->after) & I915_GEM_DOMAIN_WC)
igt_require(gem_mmap__has_wc(fd));
- igt_while_interruptible(f->flags & INTERRUPTIBLE)
- basic_reloc(fd, m->before, m->after, f->flags);
+
+ basic_reloc(fd, m->before, m->after, f->flags);
}
}
if (!(f->flags & NORELOC)) {
igt_subtest_f("%srange%s",
f->basic ? "basic-" : "", f->name) {
- igt_while_interruptible(f->flags & INTERRUPTIBLE)
- basic_range(fd, f->flags);
- }
- }
- igt_fixture {
- if (f->flags & HANG)
- igt_disallow_hang(fd, hang);
+ basic_range(fd, f->flags);
+ }
}
}
}
diff --git a/tests/intel-ci/blacklist.txt b/tests/intel-ci/blacklist.txt
index 33f92e37f602..4d2555f8adb3 100644
--- a/tests/intel-ci/blacklist.txt
+++ b/tests/intel-ci/blacklist.txt
@@ -30,7 +30,6 @@ igt@gem_exec_flush@(?!.*basic).*
igt@gem_exec_latency(@.*)?
igt@gem_exec_lut_handle(@.*)?
igt@gem_exec_nop@(?!.*basic).*
-igt@gem_exec_reloc@(?!.*basic).*
igt@gem_exec_suspend@(?!.*basic).*
igt@gem_exec_whisper@(?!basic).*
igt@gem_fd_exhaustion(@.*)?
--
2.24.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [igt-dev] [PATCH i-g-t 2/7] Revert "test/i915/gem_exec_reloc: Restore interclient testings"
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
@ 2021-06-08 9:40 ` Daniel Vetter
2021-06-09 17:56 ` Jason Ekstrand
2021-06-14 10:40 ` Arkadiusz Hiler
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 3/7] Revert "i915/gem_exec_reloc: Verify engine isolation" Daniel Vetter
` (10 subsequent siblings)
11 siblings, 2 replies; 20+ messages in thread
From: Daniel Vetter @ 2021-06-08 9:40 UTC (permalink / raw)
To: IGT development; +Cc: Petri Latvala, Daniel Vetter, Dave Airlie
This reverts commit cf2f41b3d3dfabaf3a4837062f996f3491a350b1.
And I mean revert with extreme prejudice because this was already
thrown out in
commit 39e9aa1032a4e60f776f34b3ccf4fb728abbfe5c
Author: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Date: Mon Aug 10 11:00:34 2020 +0200
tests/i915: Remove subtests that rely on async relocation behavior
and Chris then decided to undo that without any acks from the people
who've removed these tests. Or well anyone else. Commit fights in
upstream repositories are not acceptable.
Cc: Jason Ekstrand <jason@jlekstrand.net>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Dave Airlie <airlied@redhat.com>
Cc: Petri Latvala <petri.latvala@intel.com>
Cc: Arkadiusz Hiler <arek@hiler.eu>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
tests/i915/gem_exec_reloc.c | 219 ------------------------------------
1 file changed, 219 deletions(-)
diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
index cb4899fe6e1e..efe6e2e02c52 100644
--- a/tests/i915/gem_exec_reloc.c
+++ b/tests/i915/gem_exec_reloc.c
@@ -426,76 +426,6 @@ static void many_active(int i915, unsigned engine)
}
}
-static void __wide_active(int i915, unsigned engine, unsigned long count)
-{
- struct drm_i915_gem_relocation_entry *reloc =
- calloc(count, sizeof(*reloc));
- struct drm_i915_gem_exec_object2 *obj =
- calloc(count + 1, sizeof(*obj));
- struct drm_i915_gem_execbuffer2 execbuf = {
- .buffers_ptr = to_user_pointer(obj),
- .buffer_count = count + 1,
- .flags = engine | I915_EXEC_HANDLE_LUT,
- };
- igt_spin_t *spin;
-
- for (unsigned long i = 0; i < count; i++) {
- obj[i].handle = gem_create(i915, 4096);
- obj[i].flags = EXEC_OBJECT_WRITE;
- obj[i].flags |= EXEC_OBJECT_SUPPORTS_48B_ADDRESS;
- }
-
- spin = __igt_spin_new(i915,
- .engine = engine,
- .flags = (IGT_SPIN_FENCE_OUT |
- IGT_SPIN_NO_PREEMPTION));
- obj[count] = spin->obj[1];
- gem_execbuf(i915, &execbuf); /* mark all the objects as active */
-
- for (unsigned long i = 0; i < count; i++) {
- reloc[i].target_handle = i;
- reloc[i].presumed_offset = ~0ull;
- obj[i].relocs_ptr = to_user_pointer(&reloc[i]);
- obj[i].relocation_count = 1;
- }
- gem_execbuf(i915, &execbuf); /* relocation onto active objects */
-
- igt_assert_eq(sync_fence_status(spin->out_fence), 0);
- igt_spin_free(i915, spin);
-
- for (unsigned long i = 0; i < count; i++) {
- uint64_t addr;
-
- gem_read(i915, obj[i].handle, 0, &addr, sizeof(addr));
- igt_assert_eq_u64(addr, obj[i].offset);
-
- gem_close(i915, obj[i].handle);
- }
- free(obj);
- free(reloc);
-}
-
-static void wide_active(int i915, unsigned engine)
-{
- const uint64_t max = gem_aperture_size(i915) / 4096 / 2;
- unsigned long count = 256;
-
- igt_until_timeout(2) {
- uint64_t required, total;
-
- if (!__intel_check_memory(count, 4096, CHECK_RAM,
- &required, &total))
- break;
-
- igt_debug("Testing count:%lu\n", count);
- __wide_active(i915, engine, count);
-
- count <<= 1;
- if (count >= max)
- break;
- }
-}
-
static unsigned int offset_in_page(void *addr)
{
return (uintptr_t)addr & 4095;
@@ -932,145 +862,6 @@ static void basic_softpin(int fd)
gem_close(fd, obj[1].handle);
}
-static struct drm_i915_gem_relocation_entry *
-parallel_relocs(int count, unsigned long *out)
-{
- struct drm_i915_gem_relocation_entry *reloc;
- unsigned long sz;
- int i;
-
- sz = count * sizeof(*reloc);
- sz = ALIGN(sz, 4096);
-
- reloc = mmap(0, sz, PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0);
- igt_assert(reloc != MAP_FAILED);
- for (i = 0; i < count; i++) {
- reloc[i].target_handle = 0;
- reloc[i].presumed_offset = ~0ull;
- reloc[i].offset = 8 * i;
- reloc[i].delta = i;
- reloc[i].read_domains = I915_GEM_DOMAIN_INSTRUCTION;
- reloc[i].write_domain = 0;
- }
- mprotect(reloc, sz, PROT_READ);
-
- *out = sz;
- return reloc;
-}
-
-static int __execbuf(int i915, struct drm_i915_gem_execbuffer2 *execbuf)
-{
- int err;
-
- err = 0;
- if (ioctl(i915, DRM_IOCTL_I915_GEM_EXECBUFFER2, execbuf)) {
- err = -errno;
- igt_assume(err);
- }
-
- errno = 0;
- return err;
-}
-
-static int stop;
-static void sighandler(int sig)
-{
- stop = 1;
-}
-
-static void parallel_child(int i915,
- const struct intel_execution_engine2 *engine,
- struct drm_i915_gem_relocation_entry *reloc,
- uint32_t common)
-{
- igt_spin_t *spin = __igt_spin_new(i915, .engine = engine->flags);
- struct drm_i915_gem_exec_object2 reloc_target = {
- .handle = gem_create(i915, 32 * 1024 * 8),
- .relocation_count = 32 * 1024,
- .relocs_ptr = to_user_pointer(reloc),
- };
- struct drm_i915_gem_exec_object2 obj[3] = {
- reloc_target,
- { .handle = common },
- spin->obj[1],
- };
- struct drm_i915_gem_execbuffer2 execbuf = {
- .buffers_ptr = to_user_pointer(obj),
- .buffer_count = ARRAY_SIZE(obj),
- .flags = engine->flags | I915_EXEC_HANDLE_LUT,
- };
- struct sigaction act = {
- .sa_handler = sighandler,
- };
- unsigned long count = 0;
-
- sigaction(SIGINT, &act, NULL);
- while (!READ_ONCE(stop)) {
- int err = __execbuf(i915, &execbuf);
- if (err == -EINTR)
- break;
-
- igt_assert_eq(err, 0);
- count++;
- }
-
- igt_info("%s: count %lu\n", engine->name, count);
- igt_spin_free(i915, spin);
-}
-
-static void kill_children(int sig)
-{
- signal(sig, SIG_IGN);
- kill(-getpgrp(), SIGINT);
- signal(sig, SIG_DFL);
-}
-
-static void parallel(int i915)
-{
- const struct intel_execution_engine2 *e;
- struct drm_i915_gem_relocation_entry *reloc;
- uint32_t common = gem_create(i915, 4096);
- uint32_t batch = batch_create(i915);
- unsigned long reloc_sz;
-
- reloc = parallel_relocs(32 * 1024, &reloc_sz);
-
- stop = 0;
- __for_each_physical_engine(i915, e) {
- igt_fork(child, 1)
- parallel_child(i915, e, reloc, common);
- }
- sleep(2);
-
- if (gem_scheduler_has_preemption(i915)) {
- uint32_t ctx = gem_context_clone_with_engines(i915, 0);
-
- __for_each_physical_engine(i915, e) {
- struct drm_i915_gem_exec_object2 obj[2] = {
- { .handle = common },
- { .handle = batch },
- };
- struct drm_i915_gem_execbuffer2 execbuf = {
- .buffers_ptr = to_user_pointer(obj),
- .buffer_count = ARRAY_SIZE(obj),
- .flags = e->flags,
- .rsvd1 = ctx,
- };
- gem_execbuf(i915, &execbuf);
- }
-
- gem_context_destroy(i915, ctx);
- }
- gem_sync(i915, batch);
- gem_close(i915, batch);
-
- kill_children(SIGINT);
- igt_waitchildren();
-
- gem_close(i915, common);
- munmap(reloc, reloc_sz);
-}
-
#define CONCURRENT 1024
static uint64_t concurrent_relocs(int i915, int idx, int count)
@@ -1552,16 +1343,6 @@ igt_main
}
}
- igt_subtest_with_dynamic("basic-wide-active") {
- __for_each_physical_engine(fd, e) {
- igt_dynamic_f("%s", e->name)
- wide_active(fd, e->flags);
- }
- }
-
- igt_subtest("basic-parallel")
- parallel(fd);
-
igt_subtest("basic-concurrent0")
concurrent(fd, 0);
igt_subtest("basic-concurrent16")
--
2.24.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [igt-dev] [PATCH i-g-t 3/7] Revert "i915/gem_exec_reloc: Verify engine isolation"
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 2/7] Revert "test/i915/gem_exec_reloc: Restore interclient testings" Daniel Vetter
@ 2021-06-08 9:40 ` Daniel Vetter
2021-06-09 17:59 ` Jason Ekstrand
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 4/7] Revert "i915/gem_exec_reloc: Exercise concurrent relocations" Daniel Vetter
` (9 subsequent siblings)
11 siblings, 1 reply; 20+ messages in thread
From: Daniel Vetter @ 2021-06-08 9:40 UTC (permalink / raw)
To: IGT development; +Cc: Daniel Vetter, Dave Airlie
This reverts commit 9fe244cb751c9d3be0581a943bb9baa8651d8d29.
This validates gpu relocations, which we're about to delete.
Cc: Jason Ekstrand <jason@jlekstrand.net>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Dave Airlie <airlied@redhat.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
tests/i915/gem_exec_reloc.c | 73 -------------------------------------
1 file changed, 73 deletions(-)
diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
index efe6e2e02c52..3b200f557b2c 100644
--- a/tests/i915/gem_exec_reloc.c
+++ b/tests/i915/gem_exec_reloc.c
@@ -471,72 +471,6 @@ static void active_spin(int fd, unsigned engine)
igt_spin_free(fd, spin);
}
-static void others_spin(int i915, unsigned engine)
-{
- struct drm_i915_gem_relocation_entry reloc = {};
- struct drm_i915_gem_exec_object2 obj = {
- .relocs_ptr = to_user_pointer(&reloc),
- .relocation_count = 1,
- };
- struct drm_i915_gem_execbuffer2 execbuf = {
- .buffers_ptr = to_user_pointer(&obj),
- .buffer_count = 1,
- .flags = engine,
- };
- const struct intel_execution_engine2 *e;
- igt_spin_t *spin = NULL;
- uint64_t addr;
- int fence;
-
- __for_each_physical_engine(i915, e) {
- if (e->flags == engine)
- continue;
-
- if (!spin) {
- spin = igt_spin_new(i915,
- .engine = e->flags,
- .flags = IGT_SPIN_FENCE_OUT);
- fence = dup(spin->out_fence);
- } else {
- int old_fence;
-
- spin->execbuf.flags &= ~I915_EXEC_RING_MASK;
- spin->execbuf.flags |= e->flags;
- gem_execbuf_wr(i915, &spin->execbuf);
-
- old_fence = fence;
- fence = sync_fence_merge(old_fence,
- spin->execbuf.rsvd2 >> 32);
- close(spin->execbuf.rsvd2 >> 32);
- close(old_fence);
- }
- }
- igt_require(spin);
-
- /* All other engines are busy, let's relocate! */
- obj.handle = batch_create(i915);
- reloc.target_handle = obj.handle;
- reloc.presumed_offset = -1;
- reloc.offset = 64;
- gem_execbuf(i915, &execbuf);
-
- /* Verify the relocation took place */
- gem_read(i915, obj.handle, 64, &addr, sizeof(addr));
- igt_assert_eq_u64(addr, obj.offset);
- gem_close(i915, obj.handle);
-
- /* Even if the spinner was harmed in the process */
- igt_spin_end(spin);
- igt_assert_eq(sync_fence_wait(fence, 200), 0);
- igt_assert_neq(sync_fence_status(fence), 0);
- if (sync_fence_status(fence) < 0)
- igt_warn("Spinner was cancelled, %s\n",
- strerror(-sync_fence_status(fence)));
- close(fence);
-
- igt_spin_free(i915, spin);
-}
-
static bool has_64b_reloc(int fd)
{
return intel_gen(intel_get_drm_devid(fd)) >= 8;
@@ -1329,13 +1263,6 @@ igt_main
}
}
- igt_subtest_with_dynamic("basic-spin-others") {
- __for_each_physical_engine(fd, e) {
- igt_dynamic_f("%s", e->name)
- others_spin(fd, e->flags);
- }
- }
-
igt_subtest_with_dynamic("basic-many-active") {
__for_each_physical_engine(fd, e) {
igt_dynamic_f("%s", e->name)
--
2.24.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [igt-dev] [PATCH i-g-t 4/7] Revert "i915/gem_exec_reloc: Exercise concurrent relocations"
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 2/7] Revert "test/i915/gem_exec_reloc: Restore interclient testings" Daniel Vetter
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 3/7] Revert "i915/gem_exec_reloc: Verify engine isolation" Daniel Vetter
@ 2021-06-08 9:40 ` Daniel Vetter
2021-06-09 18:12 ` Jason Ekstrand
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 5/7] Revert "i915/gem_exec_reloc: Flood the ring with GPU relocs" Daniel Vetter
` (8 subsequent siblings)
11 siblings, 1 reply; 20+ messages in thread
From: Daniel Vetter @ 2021-06-08 9:40 UTC (permalink / raw)
To: IGT development; +Cc: Daniel Vetter, Dave Airlie
This reverts commit c1f30ee09ac2e7eb3e8e90245239731a169a6050.
This validates gpu relocations, which we're about to delete.
Cc: Jason Ekstrand <jason@jlekstrand.net>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Dave Airlie <airlied@redhat.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
tests/i915/gem_exec_reloc.c | 215 ------------------------------------
1 file changed, 215 deletions(-)
diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
index 3b200f557b2c..c3f42aff9c9a 100644
--- a/tests/i915/gem_exec_reloc.c
+++ b/tests/i915/gem_exec_reloc.c
@@ -796,216 +796,6 @@ static void basic_softpin(int fd)
gem_close(fd, obj[1].handle);
}
-#define CONCURRENT 1024
-
-static uint64_t concurrent_relocs(int i915, int idx, int count)
-{
- struct drm_i915_gem_relocation_entry *reloc;
- const unsigned int gen = intel_gen(intel_get_drm_devid(i915));
- unsigned long sz;
- int offset;
-
- sz = count * sizeof(*reloc);
- sz = ALIGN(sz, 4096);
-
- reloc = mmap(0, sz, PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0);
- igt_assert(reloc != MAP_FAILED);
-
- offset = 1;
- if (gen >= 4 && gen < 8)
- offset += 1;
-
- for (int n = 0; n < count; n++) {
- reloc[n].presumed_offset = ~0ull;
- reloc[n].offset = (4 * n + offset) * sizeof(uint32_t);
- reloc[n].delta = (count * idx + n) * sizeof(uint32_t);
- }
- mprotect(reloc, sz, PROT_READ);
-
- return to_user_pointer(reloc);
-}
-
-static int flags_to_index(const struct intel_execution_engine2 *e)
-{
- return (e->flags & 63) | ((e->flags >> 13) & 3) << 4;
-}
-
-static void xchg_u32(void *array, unsigned i, unsigned j)
-{
- uint32_t *u32 = array;
- uint32_t tmp = u32[i];
- u32[i] = u32[j];
- u32[j] = tmp;
-}
-
-static void concurrent_child(int i915,
- const struct intel_execution_engine2 *e,
- uint32_t *common, int num_common,
- int in, int out)
-{
- const unsigned int gen = intel_gen(intel_get_drm_devid(i915));
- int idx = flags_to_index(e);
- uint64_t relocs = concurrent_relocs(i915, idx, CONCURRENT);
- struct drm_i915_gem_exec_object2 obj[num_common + 2];
- struct drm_i915_gem_execbuffer2 execbuf = {
- .buffers_ptr = to_user_pointer(obj),
- .buffer_count = ARRAY_SIZE(obj),
- .flags = e->flags | I915_EXEC_HANDLE_LUT | (gen < 6 ? I915_EXEC_SECURE : 0),
- };
- uint32_t *batch = &obj[num_common + 1].handle;
- unsigned long count = 0;
- uint32_t *x;
- int err = 0;
-
- memset(obj, 0, sizeof(obj));
- obj[0].handle = gem_create(i915, 64 * CONCURRENT * 4);
-
- igt_permute_array(common, num_common, xchg_u32);
- for (int n = 1; n <= num_common; n++) {
- obj[n].handle = common[n - 1];
- obj[n].relocation_count = CONCURRENT;
- obj[n].relocs_ptr = relocs;
- }
-
- obj[num_common + 1].relocation_count = CONCURRENT;
- obj[num_common + 1].relocs_ptr = relocs;
-
- x = gem_mmap__device_coherent(i915, obj[0].handle,
- 0, 64 * CONCURRENT * 4, PROT_READ);
- x += idx * CONCURRENT;
-
- do {
- read(in, batch, sizeof(*batch));
- if (!*batch)
- break;
-
- gem_execbuf(i915, &execbuf);
- gem_sync(i915, *batch); /* write hazards lies */
-
- for (int n = 0; n < CONCURRENT; n++) {
- if (x[n] != *batch) {
- igt_warn("%s: Invalid store [bad reloc] found:%08x at index %d, expected %08x\n",
- e->name, x[n], n, *batch);
- err = -EINVAL;
- break;
- }
- }
-
- write(out, &err, sizeof(err));
- count++;
- } while (err == 0);
-
- gem_close(i915, obj[0].handle);
- igt_info("%s: completed %ld cycles\n", e->name, count);
-}
-
-static uint32_t create_concurrent_batch(int i915, unsigned int count)
-{
- const unsigned int gen = intel_gen(intel_get_drm_devid(i915));
- size_t sz = ALIGN(4 * (1 + 4 * count), 4096);
- uint32_t handle = gem_create(i915, sz);
- uint32_t *map, *cs;
- uint32_t cmd;
-
- cmd = MI_STORE_DWORD_IMM;
- if (gen < 6)
- cmd |= 1 << 22;
- if (gen < 4)
- cmd--;
-
- cs = map = gem_mmap__device_coherent(i915, handle, 0, sz, PROT_WRITE);
- for (int n = 0; n < count; n++) {
- *cs++ = cmd;
- *cs++ = 0;
- if (gen >= 4) {
- *cs++ = 0;
- *cs++ = handle;
- } else {
- *cs++ = handle;
- *cs++ = 0;
- }
- }
- *cs++ = MI_BATCH_BUFFER_END;
- munmap(map, sz);
-
- return handle;
-}
-
-static void concurrent(int i915, int num_common)
-{
- const struct intel_execution_engine2 *e;
- int in[2], out[2];
- uint32_t common[16];
- int result = -1;
- uint32_t batch;
- int nchild;
-
- /*
- * Exercise a few clients all trying to submit the same batch
- * buffer writing to different locations. This exercises that the
- * relocation handling within the gem_execbuf() ioctl is atomic
- * with respect to the batch -- that is this call to execbuf only
- * uses the relocations as supplied with the ioctl and does not
- * use any of the conflicting relocations from the concurrent
- * submissions.
- */
-
- pipe(in);
- pipe(out);
-
- for (int n = 0; n < num_common; n++)
- common[n] = gem_create(i915, 4 * 4 * CONCURRENT);
-
- nchild = 0;
- __for_each_physical_engine(i915, e) {
- if (!gem_class_can_store_dword(i915, e->class))
- continue;
-
- igt_fork(child, 1)
- concurrent_child(i915, e,
- common, num_common,
- in[0], out[1]);
-
- if (++nchild == 64)
- break;
- }
- close(in[0]);
- close(out[1]);
- igt_require(nchild > 1);
-
- igt_until_timeout(5) {
- batch = create_concurrent_batch(i915, CONCURRENT);
-
- for (int n = 0; n < nchild; n++)
- write(in[1], &batch, sizeof(batch));
-
- for (int n = 0; n < nchild; n++) {
- result = -1;
- read(out[0], &result, sizeof(result));
- if (result < 0)
- break;
- }
-
- gem_close(i915, batch);
- if (result < 0)
- break;
- }
-
- batch = 0;
- for (int n = 0; n < nchild; n++)
- write(in[1], &batch, sizeof(batch));
-
- close(in[1]);
- close(out[0]);
-
- igt_waitchildren();
-
- for (int n = 0; n < num_common; n++)
- gem_close(i915, common[n]);
-
- igt_assert_eq(result, 0);
-}
-
static uint32_t
pin_scanout(igt_display_t *dpy, igt_output_t *output, struct igt_fb *fb)
{
@@ -1270,11 +1060,6 @@ igt_main
}
}
- igt_subtest("basic-concurrent0")
- concurrent(fd, 0);
- igt_subtest("basic-concurrent16")
- concurrent(fd, 16);
-
igt_subtest("invalid-domains")
invalid_domains(fd);
--
2.24.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [igt-dev] [PATCH i-g-t 5/7] Revert "i915/gem_exec_reloc: Flood the ring with GPU relocs"
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
` (2 preceding siblings ...)
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 4/7] Revert "i915/gem_exec_reloc: Exercise concurrent relocations" Daniel Vetter
@ 2021-06-08 9:40 ` Daniel Vetter
2021-06-09 18:05 ` Jason Ekstrand
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 6/7] Revert "i915/gem_exec_reloc: Check that relocations do not block" Daniel Vetter
` (7 subsequent siblings)
11 siblings, 1 reply; 20+ messages in thread
From: Daniel Vetter @ 2021-06-08 9:40 UTC (permalink / raw)
To: IGT development; +Cc: Daniel Vetter, Dave Airlie
This reverts commit 57ee41f12b7a53283fd812c5f72fcb39e6ad2197.
This validates gpu relocations, which we're about to delete.
Cc: Jason Ekstrand <jason@jlekstrand.net>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Dave Airlie <airlied@redhat.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
tests/i915/gem_exec_reloc.c | 89 -------------------------------------
1 file changed, 89 deletions(-)
diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
index c3f42aff9c9a..cbd40962c610 100644
--- a/tests/i915/gem_exec_reloc.c
+++ b/tests/i915/gem_exec_reloc.c
@@ -344,88 +344,6 @@ static void active(int fd, unsigned engine)
gem_close(fd, obj[0].handle);
}
-static uint64_t many_relocs(unsigned long count, unsigned long *out)
-{
- struct drm_i915_gem_relocation_entry *reloc;
- unsigned long sz;
- int i;
-
- sz = count * sizeof(*reloc);
- sz = ALIGN(sz, 4096);
-
- reloc = mmap(0, sz, PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0);
- igt_assert(reloc != MAP_FAILED);
- for (i = 0; i < count; i++) {
- reloc[i].target_handle = 0;
- reloc[i].presumed_offset = ~0ull;
- reloc[i].offset = 8 * i;
- reloc[i].delta = 8 * i;
- }
- mprotect(reloc, sz, PROT_READ);
-
- *out = sz;
- return to_user_pointer(reloc);
-}
-
-static void __many_active(int i915, unsigned engine, unsigned long count)
-{
- unsigned long reloc_sz;
- struct drm_i915_gem_exec_object2 obj[2] = {{
- .handle = gem_create(i915, count * sizeof(uint64_t)),
- .relocs_ptr = many_relocs(count, &reloc_sz),
- .relocation_count = count,
- }};
- struct drm_i915_gem_execbuffer2 execbuf = {
- .buffers_ptr = to_user_pointer(obj),
- .buffer_count = ARRAY_SIZE(obj),
- .flags = engine | I915_EXEC_HANDLE_LUT,
- };
- igt_spin_t *spin;
-
- spin = __igt_spin_new(i915,
- .engine = engine,
- .dependency = obj[0].handle,
- .flags = (IGT_SPIN_FENCE_OUT |
- IGT_SPIN_NO_PREEMPTION));
- obj[1] = spin->obj[1];
- gem_execbuf(i915, &execbuf);
- igt_assert_eq(sync_fence_status(spin->out_fence), 0);
- igt_spin_free(i915, spin);
-
- for (unsigned long i = 0; i < count; i++) {
- uint64_t addr;
-
- gem_read(i915, obj[0].handle, i * sizeof(addr),
- &addr, sizeof(addr));
-
- igt_assert_eq_u64(addr, obj[0].offset + i * sizeof(addr));
- }
-
- munmap(from_user_pointer(obj[0].relocs_ptr), reloc_sz);
- gem_close(i915, obj[0].handle);
-}
-
-static void many_active(int i915, unsigned engine)
-{
- const uint64_t max = 2048;
- unsigned long count = 256;
-
- igt_until_timeout(2) {
- uint64_t required, total;
-
- if (!__intel_check_memory(1, 8 * count, CHECK_RAM,
- &required, &total))
- break;
-
- igt_debug("Testing count:%lu\n", count);
- __many_active(i915, engine, count);
-
- count <<= 1;
- if (count >= max)
- break;
- }
-}
-
static unsigned int offset_in_page(void *addr)
{
return (uintptr_t)addr & 4095;
@@ -1053,13 +971,6 @@ igt_main
}
}
- igt_subtest_with_dynamic("basic-many-active") {
- __for_each_physical_engine(fd, e) {
- igt_dynamic_f("%s", e->name)
- many_active(fd, e->flags);
- }
- }
-
igt_subtest("invalid-domains")
invalid_domains(fd);
--
2.24.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [igt-dev] [PATCH i-g-t 6/7] Revert "i915/gem_exec_reloc: Check that relocations do not block"
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
` (3 preceding siblings ...)
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 5/7] Revert "i915/gem_exec_reloc: Flood the ring with GPU relocs" Daniel Vetter
@ 2021-06-08 9:40 ` Daniel Vetter
2021-06-09 18:03 ` Jason Ekstrand
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 7/7] tests/gem_exec_schedule: Use store_dword_plug again Daniel Vetter
` (6 subsequent siblings)
11 siblings, 1 reply; 20+ messages in thread
From: Daniel Vetter @ 2021-06-08 9:40 UTC (permalink / raw)
To: IGT development; +Cc: Daniel Vetter, Dave Airlie
This reverts commit 5343ca6ad8fac39fe4d468f771af72c968404bea.
This validates gpu relocations, which we're about to delete.
Cc: Jason Ekstrand <jason@jlekstrand.net>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Dave Airlie <airlied@redhat.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
tests/i915/gem_exec_reloc.c | 52 -------------------------------------
1 file changed, 52 deletions(-)
diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
index cbd40962c610..5b15bcdc3a1f 100644
--- a/tests/i915/gem_exec_reloc.c
+++ b/tests/i915/gem_exec_reloc.c
@@ -344,51 +344,6 @@ static void active(int fd, unsigned engine)
gem_close(fd, obj[0].handle);
}
-static unsigned int offset_in_page(void *addr)
-{
- return (uintptr_t)addr & 4095;
-}
-
-static void active_spin(int fd, unsigned engine)
-{
- const uint32_t bbe = MI_BATCH_BUFFER_END;
- struct drm_i915_gem_relocation_entry reloc;
- struct drm_i915_gem_exec_object2 obj[2];
- struct drm_i915_gem_execbuffer2 execbuf;
- igt_spin_t *spin;
-
- spin = igt_spin_new(fd,
- .engine = engine,
- .flags = IGT_SPIN_NO_PREEMPTION);
-
- memset(obj, 0, sizeof(obj));
- obj[0] = spin->obj[IGT_SPIN_BATCH];
- obj[0].relocs_ptr = to_user_pointer(&reloc);
- obj[0].relocation_count = 1;
- obj[1].handle = gem_create(fd, 4096);
- gem_write(fd, obj[1].handle, 0, &bbe, sizeof(bbe));
-
- memset(&reloc, 0, sizeof(reloc));
- reloc.presumed_offset = -1;
- reloc.offset = offset_in_page(spin->condition);
- reloc.target_handle = obj[0].handle;
-
- memset(&execbuf, 0, sizeof(execbuf));
- execbuf.buffers_ptr = to_user_pointer(obj);
- execbuf.buffer_count = 2;
- execbuf.flags = engine;
-
- gem_execbuf(fd, &execbuf);
- gem_close(fd, obj[1].handle);
- igt_assert_eq(*spin->condition, spin->cmd_precondition);
-
- igt_spin_end(spin);
- gem_sync(fd, spin->handle);
-
- igt_assert_eq(*spin->condition, obj[0].offset);
- igt_spin_free(fd, spin);
-}
-
static bool has_64b_reloc(int fd)
{
return intel_gen(intel_get_drm_devid(fd)) >= 8;
@@ -964,13 +919,6 @@ igt_main
}
}
- igt_subtest_with_dynamic("basic-spin") {
- __for_each_physical_engine(fd, e) {
- igt_dynamic_f("%s", e->name)
- active_spin(fd, e->flags);
- }
- }
-
igt_subtest("invalid-domains")
invalid_domains(fd);
--
2.24.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [igt-dev] [PATCH i-g-t 7/7] tests/gem_exec_schedule: Use store_dword_plug again
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
` (4 preceding siblings ...)
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 6/7] Revert "i915/gem_exec_reloc: Check that relocations do not block" Daniel Vetter
@ 2021-06-08 9:40 ` Daniel Vetter
2021-06-08 11:22 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/7] tests/gem_exec_reloc: Remove banned tests Patchwork
` (5 subsequent siblings)
11 siblings, 0 replies; 20+ messages in thread
From: Daniel Vetter @ 2021-06-08 9:40 UTC (permalink / raw)
To: IGT development; +Cc: Daniel Vetter, Dave Airlie
In
commit 2884f91dd6d7682ea738ef6f0943cc591643dda2
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date: Fri Jul 20 12:29:30 2018 +0100
igt/gem_exec_schedule: Trim deep runtime
this was open-coded, I guess to avoid having to allocate a ton of
batchbuffers. Unfortunately this relies on being able to rewrite
batchbuffer relocations while the batch is in-flight (otherwise
everything stalls and our setup is for nothing), and we're removing
gpu relocations from upstream.
This problem was realized for the testcase in general in
commit f1e62e330a6e2de7b3cbf7cf02d71ae00cc6adcc
Author: Chris Wilson <chris@chris-wilson.co.uk>
Date: Tue May 26 15:22:38 2020 +0100
i915/gem_exec_schedule: Reduce relocation risk for store-dword
but the fix in there is only stochastic, there's still a chance of
failure and hence unsightly noise in CI. The proper fix is to use
softpin for this testcase unconditionally (not just when relocations
aren't available), so that we clearly disentangle the concerns here
and really only test the scheduler. And not some legacy features that
are on the way out to their deserved retirement like relocations.
Zbyscek and Ashotush cc'ed so they're aware that this testcase must be
switched over to softping uncondtionally, for correctness reasons.
But that's a pile more work, so as an interim solution go back to
__store_dword helper.
Cc: Jason Ekstrand <jason@jlekstrand.net>
Cc: Dixit Ashutosh <ashutosh.dixit@intel.com>
Cc: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Dave Airlie <airlied@redhat.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
tests/i915/gem_exec_schedule.c | 52 +++-------------------------------
1 file changed, 4 insertions(+), 48 deletions(-)
diff --git a/tests/i915/gem_exec_schedule.c b/tests/i915/gem_exec_schedule.c
index 9585059d8ded..b5de1ab4eeae 100644
--- a/tests/i915/gem_exec_schedule.c
+++ b/tests/i915/gem_exec_schedule.c
@@ -1846,55 +1846,11 @@ static void deep(int fd, unsigned ring)
/* Create a deep dependency chain, with a few branches */
for (n = 0; n < nreq && igt_seconds_elapsed(&tv) < 2; n++) {
- const unsigned int gen = intel_gen(intel_get_drm_devid(fd));
- struct drm_i915_gem_exec_object2 obj[3];
- struct drm_i915_gem_relocation_entry reloc;
- struct drm_i915_gem_execbuffer2 eb = {
- .buffers_ptr = to_user_pointer(obj),
- .buffer_count = 3,
- .flags = ring | (gen < 6 ? I915_EXEC_SECURE : 0),
- .rsvd1 = ctx[n % MAX_CONTEXTS],
- };
- uint32_t batch[16];
- int i;
-
- memset(obj, 0, sizeof(obj));
- obj[0].handle = plug;
-
- memset(&reloc, 0, sizeof(reloc));
- reloc.presumed_offset = 0;
- reloc.offset = sizeof(uint32_t);
- reloc.delta = sizeof(uint32_t) * n;
- reloc.read_domains = I915_GEM_DOMAIN_RENDER;
- reloc.write_domain = I915_GEM_DOMAIN_RENDER;
- obj[2].handle = gem_create(fd, 4096);
- obj[2].relocs_ptr = to_user_pointer(&reloc);
- obj[2].relocation_count = 1;
-
- i = 0;
- batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0);
- if (gen >= 8) {
- batch[++i] = reloc.delta;
- batch[++i] = 0;
- } else if (gen >= 4) {
- batch[++i] = 0;
- batch[++i] = reloc.delta;
- reloc.offset += sizeof(uint32_t);
- } else {
- batch[i]--;
- batch[++i] = reloc.delta;
- }
- batch[++i] = eb.rsvd1;
- batch[++i] = MI_BATCH_BUFFER_END;
- gem_write(fd, obj[2].handle, 0, batch, sizeof(batch));
+ uint32_t context = ctx[n % MAX_CONTEXTS];
+ gem_context_set_priority(fd, context, MAX_PRIO - nreq + n);
- gem_context_set_priority(fd, eb.rsvd1, MAX_PRIO - nreq + n);
- for (int m = 0; m < XS; m++) {
- obj[1].handle = dep[m];
- reloc.target_handle = obj[1].handle;
- gem_execbuf(fd, &eb);
- }
- gem_close(fd, obj[2].handle);
+ for (int m = 0; m < XS; m++)
+ store_dword_plug(fd, context, ring, dep[m], 4*n, context, plug, I915_GEM_DOMAIN_INSTRUCTION);
}
igt_info("First deptree: %d requests [%.3fs]\n",
n * XS, 1e-9*igt_nsec_elapsed(&tv));
--
2.24.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/7] tests/gem_exec_reloc: Remove banned tests
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
` (5 preceding siblings ...)
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 7/7] tests/gem_exec_schedule: Use store_dword_plug again Daniel Vetter
@ 2021-06-08 11:22 ` Patchwork
2021-06-08 16:02 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
` (4 subsequent siblings)
11 siblings, 0 replies; 20+ messages in thread
From: Patchwork @ 2021-06-08 11:22 UTC (permalink / raw)
To: Daniel Vetter; +Cc: igt-dev
[-- Attachment #1.1: Type: text/plain, Size: 7582 bytes --]
== Series Details ==
Series: series starting with [i-g-t,1/7] tests/gem_exec_reloc: Remove banned tests
URL : https://patchwork.freedesktop.org/series/91160/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10189 -> IGTPW_5898
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/index.html
Known issues
------------
Here are the changes found in IGTPW_5898 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_cs_nop@sync-fork-compute0:
- fi-snb-2600: NOTRUN -> [SKIP][1] ([fdo#109271]) +17 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-snb-2600/igt@amdgpu/amd_cs_nop@sync-fork-compute0.html
* igt@i915_selftest@live@gt_heartbeat:
- fi-kbl-8809g: [PASS][2] -> [DMESG-FAIL][3] ([i915#2291] / [i915#541])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/fi-kbl-8809g/igt@i915_selftest@live@gt_heartbeat.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-kbl-8809g/igt@i915_selftest@live@gt_heartbeat.html
#### Possible fixes ####
* igt@i915_selftest@live@hangcheck:
- {fi-hsw-gt1}: [DMESG-WARN][4] ([i915#3303]) -> [PASS][5]
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/fi-hsw-gt1/igt@i915_selftest@live@hangcheck.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-hsw-gt1/igt@i915_selftest@live@hangcheck.html
- fi-snb-2600: [INCOMPLETE][6] ([i915#2782]) -> [PASS][7]
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
#### Warnings ####
* igt@i915_selftest@live@execlists:
- fi-bsw-nick: [INCOMPLETE][8] ([i915#2782] / [i915#2940] / [i915#3462]) -> [DMESG-FAIL][9] ([i915#3462])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/fi-bsw-nick/igt@i915_selftest@live@execlists.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-bsw-nick/igt@i915_selftest@live@execlists.html
- fi-icl-u2: [INCOMPLETE][10] ([i915#2782] / [i915#3462]) -> [DMESG-FAIL][11] ([i915#3462])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/fi-icl-u2/igt@i915_selftest@live@execlists.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-icl-u2/igt@i915_selftest@live@execlists.html
* igt@runner@aborted:
- fi-cfl-8700k: [FAIL][12] ([i915#2426] / [i915#3363]) -> [FAIL][13] ([i915#3363])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/fi-cfl-8700k/igt@runner@aborted.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-cfl-8700k/igt@runner@aborted.html
- fi-skl-6600u: [FAIL][14] ([i915#1436] / [i915#3363]) -> [FAIL][15] ([i915#1436] / [i915#2426] / [i915#3363])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/fi-skl-6600u/igt@runner@aborted.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-skl-6600u/igt@runner@aborted.html
- fi-icl-u2: [FAIL][16] ([i915#2782] / [i915#3363]) -> [FAIL][17] ([i915#2426] / [i915#2782] / [i915#3363])
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/fi-icl-u2/igt@runner@aborted.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-icl-u2/igt@runner@aborted.html
- fi-kbl-8809g: [FAIL][18] ([i915#1436] / [i915#3363]) -> [FAIL][19] ([i915#1436] / [i915#2426] / [i915#3363])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/fi-kbl-8809g/igt@runner@aborted.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-kbl-8809g/igt@runner@aborted.html
- fi-cml-u2: [FAIL][20] ([i915#3363] / [i915#3462]) -> [FAIL][21] ([i915#2082] / [i915#2426] / [i915#3363] / [i915#3462])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/fi-cml-u2/igt@runner@aborted.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-cml-u2/igt@runner@aborted.html
- fi-cml-s: [FAIL][22] ([i915#3363] / [i915#3462]) -> [FAIL][23] ([i915#2082] / [i915#2426] / [i915#3363] / [i915#3462])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/fi-cml-s/igt@runner@aborted.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/fi-cml-s/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#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
[i915#2082]: https://gitlab.freedesktop.org/drm/intel/issues/2082
[i915#2291]: https://gitlab.freedesktop.org/drm/intel/issues/2291
[i915#2426]: https://gitlab.freedesktop.org/drm/intel/issues/2426
[i915#2782]: https://gitlab.freedesktop.org/drm/intel/issues/2782
[i915#2940]: https://gitlab.freedesktop.org/drm/intel/issues/2940
[i915#3303]: https://gitlab.freedesktop.org/drm/intel/issues/3303
[i915#3363]: https://gitlab.freedesktop.org/drm/intel/issues/3363
[i915#3462]: https://gitlab.freedesktop.org/drm/intel/issues/3462
[i915#541]: https://gitlab.freedesktop.org/drm/intel/issues/541
Participating hosts (48 -> 41)
------------------------------
Missing (7): fi-ilk-m540 fi-hsw-4200u fi-bsw-n3050 fi-bsw-cyan fi-ctg-p8600 fi-dg1-1 fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_6100 -> IGTPW_5898
CI-20190529: 20190529
CI_DRM_10189: c3ceeb155161955dd02c901be572b45f2245f7f6 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_5898: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/index.html
IGT_6100: 801309d0c245cbf8115c03978d0819f484b477c0 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
== Testlist changes ==
-igt@gem_exec_reloc@basic-concurrent0
-igt@gem_exec_reloc@basic-concurrent16
-igt@gem_exec_reloc@basic-many-active
-igt@gem_exec_reloc@basic-parallel
-igt@gem_exec_reloc@basic-spin
-igt@gem_exec_reloc@basic-spin-others
-igt@gem_exec_reloc@basic-wide-active
-igt@gem_exec_reloc@cpu-gtt-hang
-igt@gem_exec_reloc@cpu-gtt-interruptible
-igt@gem_exec_reloc@cpu-hang
-igt@gem_exec_reloc@cpu-interruptible
-igt@gem_exec_reloc@cpu-read-hang
-igt@gem_exec_reloc@cpu-read-interruptible
-igt@gem_exec_reloc@cpu-wc-hang
-igt@gem_exec_reloc@cpu-wc-interruptible
-igt@gem_exec_reloc@gtt-cpu-hang
-igt@gem_exec_reloc@gtt-cpu-interruptible
-igt@gem_exec_reloc@gtt-hang
-igt@gem_exec_reloc@gtt-interruptible
-igt@gem_exec_reloc@gtt-read-hang
-igt@gem_exec_reloc@gtt-read-interruptible
-igt@gem_exec_reloc@gtt-wc-hang
-igt@gem_exec_reloc@gtt-wc-interruptible
-igt@gem_exec_reloc@range-hang
-igt@gem_exec_reloc@range-interruptible
-igt@gem_exec_reloc@wc-cpu-hang
-igt@gem_exec_reloc@wc-cpu-interruptible
-igt@gem_exec_reloc@wc-gtt-hang
-igt@gem_exec_reloc@wc-gtt-interruptible
-igt@gem_exec_reloc@wc-hang
-igt@gem_exec_reloc@wc-interruptible
-igt@gem_exec_reloc@wc-read-hang
-igt@gem_exec_reloc@wc-read-interruptible
-igt@gem_exec_reloc@write-cpu-hang
-igt@gem_exec_reloc@write-cpu-interruptible
-igt@gem_exec_reloc@write-gtt-hang
-igt@gem_exec_reloc@write-gtt-interruptible
-igt@gem_exec_reloc@write-read-hang
-igt@gem_exec_reloc@write-read-interruptible
-igt@gem_exec_reloc@write-wc-hang
-igt@gem_exec_reloc@write-wc-interruptible
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/index.html
[-- Attachment #1.2: Type: text/html, Size: 10677 bytes --]
[-- Attachment #2: Type: text/plain, Size: 154 bytes --]
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
* [igt-dev] ✗ Fi.CI.IGT: failure for series starting with [i-g-t,1/7] tests/gem_exec_reloc: Remove banned tests
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
` (6 preceding siblings ...)
2021-06-08 11:22 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/7] tests/gem_exec_reloc: Remove banned tests Patchwork
@ 2021-06-08 16:02 ` Patchwork
2021-06-08 19:25 ` [igt-dev] [PATCH i-g-t] " Daniel Vetter
` (3 subsequent siblings)
11 siblings, 0 replies; 20+ messages in thread
From: Patchwork @ 2021-06-08 16:02 UTC (permalink / raw)
To: Daniel Vetter; +Cc: igt-dev
[-- Attachment #1.1: Type: text/plain, Size: 30289 bytes --]
== Series Details ==
Series: series starting with [i-g-t,1/7] tests/gem_exec_reloc: Remove banned tests
URL : https://patchwork.freedesktop.org/series/91160/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_10189_full -> IGTPW_5898_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with IGTPW_5898_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_5898_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_5898_full:
### IGT changes ###
#### Possible regressions ####
* igt@gem_exec_reloc@basic-cpu-gtt-active:
- shard-iclb: [PASS][1] -> [TIMEOUT][2]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-iclb2/igt@gem_exec_reloc@basic-cpu-gtt-active.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb8/igt@gem_exec_reloc@basic-cpu-gtt-active.html
* igt@gem_exec_reloc@basic-cpu-read-active:
- shard-glk: [PASS][3] -> [TIMEOUT][4] +4 similar issues
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-glk1/igt@gem_exec_reloc@basic-cpu-read-active.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-glk1/igt@gem_exec_reloc@basic-cpu-read-active.html
* igt@gem_exec_reloc@basic-cpu-wc-active:
- shard-apl: [PASS][5] -> [FAIL][6]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-apl1/igt@gem_exec_reloc@basic-cpu-wc-active.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl6/igt@gem_exec_reloc@basic-cpu-wc-active.html
* igt@gem_exec_reloc@basic-gtt-active:
- shard-tglb: NOTRUN -> [FAIL][7]
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb5/igt@gem_exec_reloc@basic-gtt-active.html
- shard-apl: NOTRUN -> [TIMEOUT][8]
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl7/igt@gem_exec_reloc@basic-gtt-active.html
- shard-glk: NOTRUN -> [FAIL][9]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-glk7/igt@gem_exec_reloc@basic-gtt-active.html
- shard-iclb: NOTRUN -> [FAIL][10]
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb2/igt@gem_exec_reloc@basic-gtt-active.html
- shard-kbl: NOTRUN -> [FAIL][11]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-kbl4/igt@gem_exec_reloc@basic-gtt-active.html
* igt@gem_exec_reloc@basic-gtt-read-active:
- shard-iclb: [PASS][12] -> [FAIL][13] +13 similar issues
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-iclb1/igt@gem_exec_reloc@basic-gtt-read-active.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb1/igt@gem_exec_reloc@basic-gtt-read-active.html
* igt@gem_exec_reloc@basic-wc-active:
- shard-snb: [PASS][14] -> [FAIL][15] +2 similar issues
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-snb7/igt@gem_exec_reloc@basic-wc-active.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-snb2/igt@gem_exec_reloc@basic-wc-active.html
* igt@gem_exec_reloc@basic-wc-cpu-active:
- shard-apl: [PASS][16] -> [TIMEOUT][17] +7 similar issues
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-apl3/igt@gem_exec_reloc@basic-wc-cpu-active.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl1/igt@gem_exec_reloc@basic-wc-cpu-active.html
* igt@gem_exec_reloc@basic-wc-read-active:
- shard-tglb: [PASS][18] -> [FAIL][19] +14 similar issues
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-tglb6/igt@gem_exec_reloc@basic-wc-read-active.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb8/igt@gem_exec_reloc@basic-wc-read-active.html
- shard-kbl: [PASS][20] -> [FAIL][21] +9 similar issues
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-kbl1/igt@gem_exec_reloc@basic-wc-read-active.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-kbl4/igt@gem_exec_reloc@basic-wc-read-active.html
* igt@gem_exec_reloc@basic-write-gtt-active:
- shard-snb: NOTRUN -> [FAIL][22] +3 similar issues
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-snb6/igt@gem_exec_reloc@basic-write-gtt-active.html
* igt@gem_exec_reloc@basic-write-read-active:
- shard-glk: [PASS][23] -> [FAIL][24] +9 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-glk5/igt@gem_exec_reloc@basic-write-read-active.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-glk9/igt@gem_exec_reloc@basic-write-read-active.html
- shard-apl: NOTRUN -> [FAIL][25] +1 similar issue
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl7/igt@gem_exec_reloc@basic-write-read-active.html
* {igt@gem_exec_reloc@readonly-30} (NEW):
- shard-glk: NOTRUN -> [TIMEOUT][26] +1 similar issue
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-glk5/igt@gem_exec_reloc@readonly-30.html
- shard-snb: NOTRUN -> [TIMEOUT][27]
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-snb2/igt@gem_exec_reloc@readonly-30.html
* {igt@gem_exec_reloc@readonly-31} (NEW):
- shard-apl: NOTRUN -> [INCOMPLETE][28] +2 similar issues
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl1/igt@gem_exec_reloc@readonly-31.html
- shard-glk: NOTRUN -> [INCOMPLETE][29]
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-glk6/igt@gem_exec_reloc@readonly-31.html
- shard-iclb: NOTRUN -> [INCOMPLETE][30] +1 similar issue
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb2/igt@gem_exec_reloc@readonly-31.html
- shard-kbl: NOTRUN -> [INCOMPLETE][31]
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-kbl7/igt@gem_exec_reloc@readonly-31.html
- shard-snb: NOTRUN -> [INCOMPLETE][32]
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-snb2/igt@gem_exec_reloc@readonly-31.html
- shard-tglb: NOTRUN -> [TIMEOUT][33]
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb6/igt@gem_exec_reloc@readonly-31.html
* {igt@gem_exec_reloc@readonly-32} (NEW):
- shard-tglb: NOTRUN -> [INCOMPLETE][34] +1 similar issue
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb3/igt@gem_exec_reloc@readonly-32.html
#### Warnings ####
* igt@runner@aborted:
- shard-iclb: ([FAIL][35], [FAIL][36], [FAIL][37]) ([i915#2782] / [i915#3002]) -> ([FAIL][38], [FAIL][39], [FAIL][40], [FAIL][41], [FAIL][42]) ([i915#1814] / [i915#2782] / [i915#3002])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-iclb1/igt@runner@aborted.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-iclb7/igt@runner@aborted.html
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-iclb6/igt@runner@aborted.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb1/igt@runner@aborted.html
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb5/igt@runner@aborted.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb7/igt@runner@aborted.html
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb2/igt@runner@aborted.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb4/igt@runner@aborted.html
- shard-tglb: ([FAIL][43], [FAIL][44], [FAIL][45], [FAIL][46]) ([i915#1436] / [i915#2426] / [i915#2966] / [i915#3002] / [i915#409]) -> ([FAIL][47], [FAIL][48], [FAIL][49], [FAIL][50], [FAIL][51], [FAIL][52]) ([i915#1436] / [i915#2426] / [i915#2966] / [i915#3002])
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-tglb2/igt@runner@aborted.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-tglb8/igt@runner@aborted.html
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-tglb3/igt@runner@aborted.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-tglb6/igt@runner@aborted.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb6/igt@runner@aborted.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb3/igt@runner@aborted.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb5/igt@runner@aborted.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb3/igt@runner@aborted.html
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb3/igt@runner@aborted.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb6/igt@runner@aborted.html
New tests
---------
New tests have been introduced between CI_DRM_10189_full and IGTPW_5898_full:
### New IGT tests (107) ###
* igt@gem_exec_reloc@cpu-12:
- Statuses : 6 pass(s)
- Exec time: [0.09, 0.31] s
* igt@gem_exec_reloc@cpu-13:
- Statuses : 5 pass(s)
- Exec time: [0.06, 0.19] s
* igt@gem_exec_reloc@cpu-14:
- Statuses : 2 pass(s)
- Exec time: [0.07, 0.16] s
* igt@gem_exec_reloc@cpu-15:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@cpu-16:
- Statuses : 5 pass(s)
- Exec time: [0.30, 0.84] s
* igt@gem_exec_reloc@cpu-17:
- Statuses : 5 pass(s)
- Exec time: [0.09, 0.23] s
* igt@gem_exec_reloc@cpu-18:
- Statuses : 6 pass(s)
- Exec time: [0.07, 0.26] s
* igt@gem_exec_reloc@cpu-19:
- Statuses : 5 pass(s)
- Exec time: [0.12, 0.44] s
* igt@gem_exec_reloc@cpu-20:
- Statuses : 5 pass(s)
- Exec time: [0.07, 0.26] s
* igt@gem_exec_reloc@cpu-21:
- Statuses : 5 pass(s)
- Exec time: [0.29, 0.97] s
* igt@gem_exec_reloc@cpu-22:
- Statuses : 3 pass(s)
- Exec time: [0.09, 0.17] s
* igt@gem_exec_reloc@cpu-23:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@cpu-24:
- Statuses : 5 pass(s)
- Exec time: [0.34, 1.16] s
* igt@gem_exec_reloc@cpu-25:
- Statuses : 6 pass(s)
- Exec time: [0.18, 0.64] s
* igt@gem_exec_reloc@cpu-26:
- Statuses : 5 pass(s)
- Exec time: [0.27, 0.75] s
* igt@gem_exec_reloc@cpu-27:
- Statuses : 3 pass(s)
- Exec time: [0.80, 2.19] s
* igt@gem_exec_reloc@cpu-28:
- Statuses : 6 pass(s)
- Exec time: [0.89, 3.38] s
* igt@gem_exec_reloc@cpu-29:
- Statuses : 6 pass(s)
- Exec time: [1.77, 6.70] s
* igt@gem_exec_reloc@cpu-30:
- Statuses : 5 pass(s)
- Exec time: [3.41, 13.05] s
* igt@gem_exec_reloc@cpu-31:
- Statuses : 5 pass(s)
- Exec time: [6.70, 18.19] s
* igt@gem_exec_reloc@cpu-32:
- Statuses : 2 pass(s) 4 skip(s)
- Exec time: [0.36, 15.39] s
* igt@gem_exec_reloc@gpu:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@gtt-12:
- Statuses : 4 pass(s)
- Exec time: [0.13, 0.40] s
* igt@gem_exec_reloc@gtt-13:
- Statuses : 5 pass(s)
- Exec time: [0.27, 1.15] s
* igt@gem_exec_reloc@gtt-14:
- Statuses : 5 pass(s)
- Exec time: [0.07, 0.18] s
* igt@gem_exec_reloc@gtt-15:
- Statuses : 5 pass(s)
- Exec time: [0.06, 0.18] s
* igt@gem_exec_reloc@gtt-16:
- Statuses : 3 pass(s)
- Exec time: [0.06, 0.14] s
* igt@gem_exec_reloc@gtt-17:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@gtt-18:
- Statuses : 6 pass(s)
- Exec time: [0.09, 0.24] s
* igt@gem_exec_reloc@gtt-19:
- Statuses : 5 pass(s)
- Exec time: [0.07, 0.17] s
* igt@gem_exec_reloc@gtt-20:
- Statuses : 4 pass(s)
- Exec time: [0.10, 0.24] s
* igt@gem_exec_reloc@gtt-21:
- Statuses : 5 pass(s)
- Exec time: [0.14, 0.46] s
* igt@gem_exec_reloc@gtt-22:
- Statuses : 6 pass(s)
- Exec time: [0.41, 1.14] s
* igt@gem_exec_reloc@gtt-23:
- Statuses : 6 pass(s)
- Exec time: [0.60, 1.75] s
* igt@gem_exec_reloc@gtt-24:
- Statuses : 4 pass(s)
- Exec time: [0.66, 0.88] s
* igt@gem_exec_reloc@gtt-25:
- Statuses : 5 pass(s)
- Exec time: [0.45, 1.61] s
* igt@gem_exec_reloc@gtt-26:
- Statuses : 4 pass(s)
- Exec time: [0.96, 2.63] s
* igt@gem_exec_reloc@gtt-27:
- Statuses : 5 pass(s)
- Exec time: [1.85, 5.44] s
* igt@gem_exec_reloc@gtt-28:
- Statuses : 5 pass(s)
- Exec time: [3.19, 17.48] s
* igt@gem_exec_reloc@gtt-29:
- Statuses : 4 pass(s)
- Exec time: [6.44, 29.08] s
* igt@gem_exec_reloc@gtt-30:
- Statuses : 2 pass(s)
- Exec time: [23.55, 58.25] s
* igt@gem_exec_reloc@gtt-31:
- Statuses : 4 pass(s)
- Exec time: [34.70, 115.44] s
* igt@gem_exec_reloc@gtt-32:
- Statuses : 1 pass(s) 2 skip(s)
- Exec time: [0.15, 99.82] s
* igt@gem_exec_reloc@invalid-domains:
- Statuses : 1 pass(s)
- Exec time: [0.01] s
* igt@gem_exec_reloc@mmap-12:
- Statuses : 6 pass(s)
- Exec time: [0.31, 1.35] s
* igt@gem_exec_reloc@mmap-13:
- Statuses : 3 pass(s)
- Exec time: [0.12, 0.28] s
* igt@gem_exec_reloc@mmap-14:
- Statuses : 6 pass(s)
- Exec time: [0.31, 1.30] s
* igt@gem_exec_reloc@mmap-15:
- Statuses : 5 pass(s)
- Exec time: [0.07, 0.13] s
* igt@gem_exec_reloc@mmap-16:
- Statuses : 5 pass(s)
- Exec time: [0.08, 0.23] s
* igt@gem_exec_reloc@mmap-17:
- Statuses : 5 pass(s)
- Exec time: [0.42, 1.06] s
* igt@gem_exec_reloc@mmap-18:
- Statuses : 6 pass(s)
- Exec time: [0.14, 0.37] s
* igt@gem_exec_reloc@mmap-19:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@mmap-20:
- Statuses : 4 pass(s)
- Exec time: [0.06, 0.07] s
* igt@gem_exec_reloc@mmap-21:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@mmap-22:
- Statuses : 4 pass(s)
- Exec time: [0.08, 0.24] s
* igt@gem_exec_reloc@mmap-23:
- Statuses : 6 pass(s)
- Exec time: [0.08, 0.29] s
* igt@gem_exec_reloc@mmap-24:
- Statuses : 5 pass(s)
- Exec time: [0.10, 0.33] s
* igt@gem_exec_reloc@mmap-25:
- Statuses : 5 pass(s)
- Exec time: [0.15, 0.46] s
* igt@gem_exec_reloc@mmap-26:
- Statuses : 5 pass(s)
- Exec time: [0.34, 0.76] s
* igt@gem_exec_reloc@mmap-27:
- Statuses : 4 pass(s)
- Exec time: [0.24, 0.70] s
* igt@gem_exec_reloc@mmap-28:
- Statuses : 5 pass(s)
- Exec time: [0.34, 1.06] s
* igt@gem_exec_reloc@mmap-29:
- Statuses : 6 pass(s)
- Exec time: [0.49, 1.90] s
* igt@gem_exec_reloc@mmap-30:
- Statuses : 5 pass(s)
- Exec time: [0.93, 2.70] s
* igt@gem_exec_reloc@mmap-31:
- Statuses : 4 pass(s)
- Exec time: [1.96, 2.67] s
* igt@gem_exec_reloc@mmap-32:
- Statuses : 2 pass(s) 3 skip(s)
- Exec time: [0.41, 5.08] s
* igt@gem_exec_reloc@readonly-12:
- Statuses : 5 pass(s)
- Exec time: [0.32, 1.19] s
* igt@gem_exec_reloc@readonly-13:
- Statuses : 6 pass(s)
- Exec time: [0.29, 0.99] s
* igt@gem_exec_reloc@readonly-14:
- Statuses : 4 pass(s)
- Exec time: [0.32, 0.86] s
* igt@gem_exec_reloc@readonly-15:
- Statuses : 6 pass(s)
- Exec time: [0.07, 0.22] s
* igt@gem_exec_reloc@readonly-16:
- Statuses : 1 pass(s)
- Exec time: [0.08] s
* igt@gem_exec_reloc@readonly-17:
- Statuses : 5 pass(s)
- Exec time: [0.10, 0.25] s
* igt@gem_exec_reloc@readonly-18:
- Statuses : 2 pass(s)
- Exec time: [0.11, 0.27] s
* igt@gem_exec_reloc@readonly-19:
- Statuses : 3 pass(s)
- Exec time: [0.14, 0.29] s
* igt@gem_exec_reloc@readonly-20:
- Statuses : 5 pass(s)
- Exec time: [0.23, 0.77] s
* igt@gem_exec_reloc@readonly-21:
- Statuses : 6 pass(s)
- Exec time: [0.52, 1.38] s
* igt@gem_exec_reloc@readonly-22:
- Statuses : 6 pass(s)
- Exec time: [0.25, 2.18] s
* igt@gem_exec_reloc@readonly-23:
- Statuses : 5 pass(s)
- Exec time: [0.50, 3.27] s
* igt@gem_exec_reloc@readonly-24:
- Statuses : 5 pass(s)
- Exec time: [1.90, 7.45] s
* igt@gem_exec_reloc@readonly-25:
- Statuses : 6 pass(s)
- Exec time: [0.97, 11.60] s
* igt@gem_exec_reloc@readonly-26:
- Statuses : 5 pass(s)
- Exec time: [6.36, 17.73] s
* igt@gem_exec_reloc@readonly-27:
- Statuses : 5 pass(s)
- Exec time: [17.81, 70.74] s
* igt@gem_exec_reloc@readonly-28:
- Statuses : 5 pass(s)
- Exec time: [6.47, 67.41] s
* igt@gem_exec_reloc@readonly-29:
- Statuses : 1 incomplete(s) 4 pass(s) 1 timeout(s)
- Exec time: [0.0, 170.49] s
* igt@gem_exec_reloc@readonly-30:
- Statuses : 1 incomplete(s) 2 pass(s) 2 timeout(s)
- Exec time: [0.0, 247.28] s
* igt@gem_exec_reloc@readonly-31:
- Statuses : 5 incomplete(s) 1 timeout(s)
- Exec time: [0.0, 162.78] s
* igt@gem_exec_reloc@readonly-32:
- Statuses : 2 incomplete(s) 4 skip(s)
- Exec time: [0.0, 0.31] s
* igt@gem_exec_reloc@wc-12:
- Statuses : 5 pass(s)
- Exec time: [0.07, 0.17] s
* igt@gem_exec_reloc@wc-13:
- Statuses : 5 pass(s)
- Exec time: [0.07, 0.59] s
* igt@gem_exec_reloc@wc-14:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@wc-15:
- Statuses : 6 pass(s)
- Exec time: [0.09, 0.29] s
* igt@gem_exec_reloc@wc-16:
- Statuses : 6 pass(s)
- Exec time: [0.12, 0.37] s
* igt@gem_exec_reloc@wc-17:
- Statuses : 5 pass(s)
- Exec time: [0.09, 0.26] s
* igt@gem_exec_reloc@wc-18:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@wc-19:
- Statuses : 5 pass(s)
- Exec time: [0.09, 0.18] s
* igt@gem_exec_reloc@wc-20:
- Statuses : 5 pass(s)
- Exec time: [0.09, 0.16] s
* igt@gem_exec_reloc@wc-21:
- Statuses : 5 pass(s)
- Exec time: [0.09, 0.29] s
* igt@gem_exec_reloc@wc-22:
- Statuses : 4 pass(s)
- Exec time: [0.12, 0.29] s
* igt@gem_exec_reloc@wc-23:
- Statuses : 5 pass(s)
- Exec time: [0.15, 0.37] s
* igt@gem_exec_reloc@wc-24:
- Statuses : 3 pass(s)
- Exec time: [0.26, 0.50] s
* igt@gem_exec_reloc@wc-25:
- Statuses : 6 pass(s)
- Exec time: [0.39, 0.94] s
* igt@gem_exec_reloc@wc-26:
- Statuses : 4 pass(s)
- Exec time: [0.73, 1.51] s
* igt@gem_exec_reloc@wc-27:
- Statuses : 5 pass(s)
- Exec time: [1.38, 2.88] s
* igt@gem_exec_reloc@wc-28:
- Statuses : 5 pass(s)
- Exec time: [2.80, 5.39] s
* igt@gem_exec_reloc@wc-29:
- Statuses : 4 pass(s)
- Exec time: [5.40, 11.07] s
* igt@gem_exec_reloc@wc-30:
- Statuses : 2 pass(s)
- Exec time: [14.72, 19.59] s
* igt@gem_exec_reloc@wc-31:
- Statuses : 3 pass(s)
- Exec time: [21.05, 38.82] s
* igt@gem_exec_reloc@wc-32:
- Statuses : 1 incomplete(s) 1 pass(s) 3 skip(s)
- Exec time: [0.0, 42.72] s
Known issues
------------
Here are the changes found in IGTPW_5898_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@drm_import_export@flink:
- shard-tglb: [PASS][53] -> [INCOMPLETE][54] ([i915#750]) +1 similar issue
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-tglb3/igt@drm_import_export@flink.html
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb2/igt@drm_import_export@flink.html
* igt@gem_ctx_persistence@engines-queued:
- shard-snb: NOTRUN -> [SKIP][55] ([fdo#109271] / [i915#1099]) +2 similar issues
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-snb7/igt@gem_ctx_persistence@engines-queued.html
* igt@gem_eio@unwedge-stress:
- shard-snb: NOTRUN -> [FAIL][56] ([i915#3354])
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-snb2/igt@gem_eio@unwedge-stress.html
* igt@gem_exec_fair@basic-deadline:
- shard-apl: NOTRUN -> [FAIL][57] ([i915#2846])
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl3/igt@gem_exec_fair@basic-deadline.html
* igt@gem_exec_fair@basic-none@rcs0:
- shard-kbl: [PASS][58] -> [FAIL][59] ([i915#2842]) +2 similar issues
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-kbl4/igt@gem_exec_fair@basic-none@rcs0.html
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-kbl7/igt@gem_exec_fair@basic-none@rcs0.html
* igt@gem_exec_fair@basic-none@vcs0:
- shard-apl: NOTRUN -> [FAIL][60] ([i915#2842])
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl1/igt@gem_exec_fair@basic-none@vcs0.html
* igt@gem_exec_fair@basic-none@vcs1:
- shard-iclb: NOTRUN -> [FAIL][61] ([i915#2842])
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb1/igt@gem_exec_fair@basic-none@vcs1.html
* igt@gem_exec_fair@basic-pace-share@rcs0:
- shard-tglb: [PASS][62] -> [FAIL][63] ([i915#2842]) +1 similar issue
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-tglb3/igt@gem_exec_fair@basic-pace-share@rcs0.html
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb3/igt@gem_exec_fair@basic-pace-share@rcs0.html
* igt@gem_exec_fair@basic-pace-solo@rcs0:
- shard-glk: [PASS][64] -> [FAIL][65] ([i915#2842]) +2 similar issues
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-glk1/igt@gem_exec_fair@basic-pace-solo@rcs0.html
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-glk1/igt@gem_exec_fair@basic-pace-solo@rcs0.html
* {igt@gem_exec_reloc@readonly-32} (NEW):
- shard-glk: NOTRUN -> [SKIP][66] ([fdo#109271]) +53 similar issues
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-glk9/igt@gem_exec_reloc@readonly-32.html
* igt@gem_exec_whisper@basic-contexts-forked:
- shard-iclb: [PASS][67] -> [INCOMPLETE][68] ([i915#1895])
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-iclb7/igt@gem_exec_whisper@basic-contexts-forked.html
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb7/igt@gem_exec_whisper@basic-contexts-forked.html
* igt@gem_mmap_gtt@cpuset-big-copy-odd:
- shard-glk: [PASS][69] -> [FAIL][70] ([i915#307])
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-glk9/igt@gem_mmap_gtt@cpuset-big-copy-odd.html
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-glk6/igt@gem_mmap_gtt@cpuset-big-copy-odd.html
* igt@gem_mmap_gtt@cpuset-medium-copy:
- shard-iclb: [PASS][71] -> [FAIL][72] ([i915#307])
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-iclb8/igt@gem_mmap_gtt@cpuset-medium-copy.html
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb4/igt@gem_mmap_gtt@cpuset-medium-copy.html
* igt@gem_pread@exhaustion:
- shard-snb: NOTRUN -> [WARN][73] ([i915#2658])
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-snb6/igt@gem_pread@exhaustion.html
* igt@gem_render_copy@y-tiled-mc-ccs-to-vebox-yf-tiled:
- shard-iclb: NOTRUN -> [SKIP][74] ([i915#768])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb1/igt@gem_render_copy@y-tiled-mc-ccs-to-vebox-yf-tiled.html
* igt@gem_userptr_blits@input-checking:
- shard-snb: NOTRUN -> [DMESG-WARN][75] ([i915#3002])
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-snb2/igt@gem_userptr_blits@input-checking.html
* igt@gem_userptr_blits@unsync-overlap:
- shard-tglb: NOTRUN -> [SKIP][76] ([i915#3297])
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb3/igt@gem_userptr_blits@unsync-overlap.html
- shard-iclb: NOTRUN -> [SKIP][77] ([i915#3297])
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb8/igt@gem_userptr_blits@unsync-overlap.html
* igt@gem_userptr_blits@vma-merge:
- shard-snb: NOTRUN -> [FAIL][78] ([i915#2724])
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-snb7/igt@gem_userptr_blits@vma-merge.html
- shard-apl: NOTRUN -> [FAIL][79] ([i915#3318])
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl3/igt@gem_userptr_blits@vma-merge.html
- shard-iclb: NOTRUN -> [FAIL][80] ([i915#3318])
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb5/igt@gem_userptr_blits@vma-merge.html
- shard-glk: NOTRUN -> [FAIL][81] ([i915#3318])
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-glk9/igt@gem_userptr_blits@vma-merge.html
- shard-kbl: NOTRUN -> [FAIL][82] ([i915#3318])
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-kbl1/igt@gem_userptr_blits@vma-merge.html
- shard-tglb: NOTRUN -> [FAIL][83] ([i915#3318])
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb3/igt@gem_userptr_blits@vma-merge.html
* igt@gem_workarounds@suspend-resume-fd:
- shard-apl: NOTRUN -> [DMESG-WARN][84] ([i915#180])
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl2/igt@gem_workarounds@suspend-resume-fd.html
* igt@i915_hangman@engine-error@vecs0:
- shard-kbl: NOTRUN -> [SKIP][85] ([fdo#109271]) +162 similar issues
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-kbl3/igt@i915_hangman@engine-error@vecs0.html
* igt@i915_pm_dc@dc6-dpms:
- shard-kbl: NOTRUN -> [FAIL][86] ([i915#454])
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-kbl1/igt@i915_pm_dc@dc6-dpms.html
* igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-dp:
- shard-kbl: NOTRUN -> [SKIP][87] ([fdo#109271] / [i915#1937])
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-kbl2/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-dp.html
* igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a:
- shard-glk: NOTRUN -> [SKIP][88] ([fdo#109271] / [i915#1937])
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-glk8/igt@i915_pm_lpsp@kms-lpsp@kms-lpsp-hdmi-a.html
* igt@i915_pm_rpm@dpms-mode-unset-non-lpsp:
- shard-tglb: NOTRUN -> [SKIP][89] ([fdo#111644] / [i915#1397] / [i915#2411])
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb3/igt@i915_pm_rpm@dpms-mode-unset-non-lpsp.html
* igt@i915_selftest@live@execlists:
- shard-apl: NOTRUN -> [DMESG-FAIL][90] ([i915#3462])
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl8/igt@i915_selftest@live@execlists.html
* igt@kms_chamelium@dp-hpd-storm-disable:
- shard-apl: NOTRUN -> [SKIP][91] ([fdo#109271] / [fdo#111827]) +13 similar issues
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl8/igt@kms_chamelium@dp-hpd-storm-disable.html
* igt@kms_chamelium@hdmi-aspect-ratio:
- shard-glk: NOTRUN -> [SKIP][92] ([fdo#109271] / [fdo#111827]) +1 similar issue
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-glk7/igt@kms_chamelium@hdmi-aspect-ratio.html
- shard-tglb: NOTRUN -> [SKIP][93] ([fdo#109284] / [fdo#111827]) +3 similar issues
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb5/igt@kms_chamelium@hdmi-aspect-ratio.html
* igt@kms_chamelium@hdmi-hpd-enable-disable-mode:
- shard-snb: NOTRUN -> [SKIP][94] ([fdo#109271] / [fdo#111827]) +23 similar issues
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-snb7/igt@kms_chamelium@hdmi-hpd-enable-disable-mode.html
* igt@kms_color_chamelium@pipe-a-ctm-0-75:
- shard-kbl: NOTRUN -> [SKIP][95] ([fdo#109271] / [fdo#111827]) +7 similar issues
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-kbl2/igt@kms_color_chamelium@pipe-a-ctm-0-75.html
* igt@kms_color_chamelium@pipe-a-ctm-red-to-blue:
- shard-iclb: NOTRUN -> [SKIP][96] ([fdo#109284] / [fdo#111827]) +1 similar issue
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb5/igt@kms_color_chamelium@pipe-a-ctm-red-to-blue.html
* igt@kms_content_protection@atomic:
- shard-apl: NOTRUN -> [TIMEOUT][97] ([i915#1319])
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl2/igt@kms_content_protection@atomic.html
* igt@kms_content_protection@legacy:
- shard-kbl: NOTRUN -> [TIMEOUT][98] ([i915#1319])
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-kbl1/igt@kms_content_protection@legacy.html
* igt@kms_content_protection@uevent:
- shard-apl: NOTRUN -> [FAIL][99] ([i915#2105])
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-apl8/igt@kms_content_protection@uevent.html
* igt@kms_cursor_crc@pipe-a-cursor-suspend:
- shard-kbl: [PASS][100] -> [DMESG-WARN][101] ([i915#180]) +3 similar issues
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10189/shard-kbl6/igt@kms_cursor_crc@pipe-a-cursor-suspend.html
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-kbl1/igt@kms_cursor_crc@pipe-a-cursor-suspend.html
* igt@kms_cursor_crc@pipe-b-cursor-32x10-onscreen:
- shard-iclb: NOTRUN -> [SKIP][102] ([fdo#109278]) +11 similar issues
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb5/igt@kms_cursor_crc@pipe-b-cursor-32x10-onscreen.html
- shard-tglb: NOTRUN -> [SKIP][103] ([i915#3359]) +1 similar issue
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-tglb8/igt@kms_cursor_crc@pipe-b-cursor-32x10-onscreen.html
* igt@kms_cursor_crc@pipe-b-cursor-512x512-random:
- shard-iclb: NOTRUN -> [SKIP][104] ([fdo#109278] / [fdo#109279]) +1 similar issue
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/shard-iclb2/igt@kms_cursor_crc@pipe-b-cursor-512x512-random.html
* igt@kms_cursor_crc@pipe-b-cursor-suspend:
- shard-apl:
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5898/index.html
[-- Attachment #1.2: Type: text/html, Size: 37106 bytes --]
[-- Attachment #2: Type: text/plain, Size: 154 bytes --]
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
* [igt-dev] [PATCH i-g-t] tests/gem_exec_reloc: Remove banned tests
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
` (7 preceding siblings ...)
2021-06-08 16:02 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
@ 2021-06-08 19:25 ` Daniel Vetter
2021-06-09 17:53 ` Jason Ekstrand
2021-06-08 19:44 ` [igt-dev] ✗ GitLab.Pipeline: warning for series starting with [i-g-t] tests/gem_exec_reloc: Remove banned tests (rev2) Patchwork
` (2 subsequent siblings)
11 siblings, 1 reply; 20+ messages in thread
From: Daniel Vetter @ 2021-06-08 19:25 UTC (permalink / raw)
To: IGT development; +Cc: Petri Latvala, Daniel Vetter
These have been banned across all CI for over 2 years, realistically
that's not going to change and they're just in the way.
v2: Keep the !HANG parts (CI)
Cc: Jason Ekstrand <jason@jlekstrand.net>
Cc: Petri Latvala <petri.latvala@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
tests/i915/gem_exec_reloc.c | 33 +++++++--------------------------
tests/intel-ci/blacklist.txt | 1 -
2 files changed, 7 insertions(+), 27 deletions(-)
diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
index 397c335e4d3c..0e75a5e52cc4 100644
--- a/tests/i915/gem_exec_reloc.c
+++ b/tests/i915/gem_exec_reloc.c
@@ -614,8 +614,6 @@ static bool has_64b_reloc(int fd)
#define NORELOC 1
#define ACTIVE 2
-#define INTERRUPTIBLE 4
-#define HANG 8
static void basic_reloc(int fd, unsigned before, unsigned after, unsigned flags)
{
#define OBJSZ 8192
@@ -678,8 +676,7 @@ static void basic_reloc(int fd, unsigned before, unsigned after, unsigned flags)
spin = igt_spin_new(fd,
.engine = I915_EXEC_DEFAULT,
.dependency = obj.handle);
- if (!(flags & HANG))
- igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
+ igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
igt_assert(gem_bo_busy(fd, obj.handle));
}
@@ -746,8 +743,7 @@ static void basic_reloc(int fd, unsigned before, unsigned after, unsigned flags)
spin = igt_spin_new(fd,
.engine = I915_EXEC_DEFAULT,
.dependency = obj.handle);
- if (!(flags & HANG))
- igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
+ igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
igt_assert(gem_bo_busy(fd, obj.handle));
}
@@ -873,8 +869,7 @@ static void basic_range(int fd, unsigned flags)
if (flags & ACTIVE) {
spin = igt_spin_new(fd, .dependency = obj[n].handle);
- if (!(flags & HANG))
- igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
+ igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
igt_assert(gem_bo_busy(fd, obj[n].handle));
}
@@ -1468,8 +1463,6 @@ igt_main
{ "", 0 , true},
{ "-noreloc", NORELOC, true },
{ "-active", ACTIVE, true },
- { "-interruptible", ACTIVE | INTERRUPTIBLE },
- { "-hang", ACTIVE | HANG },
{ },
}, *f;
uint64_t size;
@@ -1483,14 +1476,7 @@ igt_main
}
for (f = flags; f->name; f++) {
- igt_hang_t hang;
-
igt_subtest_group {
- igt_fixture {
- if (f->flags & HANG)
- hang = igt_allow_hang(fd, 0, 0);
- }
-
for (m = modes; m->name; m++) {
igt_subtest_f("%s%s%s",
f->basic ? "basic-" : "",
@@ -1498,22 +1484,17 @@ igt_main
f->name) {
if ((m->before | m->after) & I915_GEM_DOMAIN_WC)
igt_require(gem_mmap__has_wc(fd));
- igt_while_interruptible(f->flags & INTERRUPTIBLE)
- basic_reloc(fd, m->before, m->after, f->flags);
+
+ basic_reloc(fd, m->before, m->after, f->flags);
}
}
if (!(f->flags & NORELOC)) {
igt_subtest_f("%srange%s",
f->basic ? "basic-" : "", f->name) {
- igt_while_interruptible(f->flags & INTERRUPTIBLE)
- basic_range(fd, f->flags);
- }
- }
- igt_fixture {
- if (f->flags & HANG)
- igt_disallow_hang(fd, hang);
+ basic_range(fd, f->flags);
+ }
}
}
}
diff --git a/tests/intel-ci/blacklist.txt b/tests/intel-ci/blacklist.txt
index 33f92e37f602..4d2555f8adb3 100644
--- a/tests/intel-ci/blacklist.txt
+++ b/tests/intel-ci/blacklist.txt
@@ -30,7 +30,6 @@ igt@gem_exec_flush@(?!.*basic).*
igt@gem_exec_latency(@.*)?
igt@gem_exec_lut_handle(@.*)?
igt@gem_exec_nop@(?!.*basic).*
-igt@gem_exec_reloc@(?!.*basic).*
igt@gem_exec_suspend@(?!.*basic).*
igt@gem_exec_whisper@(?!basic).*
igt@gem_fd_exhaustion(@.*)?
--
2.24.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [igt-dev] ✗ GitLab.Pipeline: warning for series starting with [i-g-t] tests/gem_exec_reloc: Remove banned tests (rev2)
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
` (8 preceding siblings ...)
2021-06-08 19:25 ` [igt-dev] [PATCH i-g-t] " Daniel Vetter
@ 2021-06-08 19:44 ` Patchwork
2021-06-08 20:08 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
2021-06-09 0:49 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
11 siblings, 0 replies; 20+ messages in thread
From: Patchwork @ 2021-06-08 19:44 UTC (permalink / raw)
To: Daniel Vetter; +Cc: igt-dev
== Series Details ==
Series: series starting with [i-g-t] tests/gem_exec_reloc: Remove banned tests (rev2)
URL : https://patchwork.freedesktop.org/series/91160/
State : warning
== Summary ==
Pipeline status: FAILED.
see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/334795 for the overview.
build-containers:build-debian has failed (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/10607484):
Running with gitlab-runner 13.12.0 (7a6612da)
on gst-htz-2 thys1kA2
section_start:1623180981:prepare_executor
Preparing the "docker" executor
Using Docker executor with image registry.freedesktop.org/wayland/ci-templates/buildah:2019-08-13.0 ...
Authenticating with credentials from job payload (GitLab Registry)
Pulling docker image registry.freedesktop.org/wayland/ci-templates/buildah:2019-08-13.0 ...
Using docker image sha256:594aa868d31ee3304dee8cae8a3433c89a6fcfcf6c7d420c04cce22f60147176 for registry.freedesktop.org/wayland/ci-templates/buildah:2019-08-13.0 with digest registry.freedesktop.org/wayland/ci-templates/buildah@sha256:7dbcf22cd2c1c7d49db0dc7b4ab207c3d6a4a09bd81cc3b71a688d3727d8749f ...
section_end:1623181067:prepare_executor
section_start:1623181067:prepare_script
Preparing environment
section_end:1623181187:prepare_script
ERROR: Failed to cleanup volumes
ERROR: Job failed (system failure): prepare environment: Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running? (docker.go:640:120s). Check https://docs.gitlab.com/runner/shells/index.html#shell-profile-loading for more information
== Logs ==
For more details see: https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/334795
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t] tests/gem_exec_reloc: Remove banned tests (rev2)
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
` (9 preceding siblings ...)
2021-06-08 19:44 ` [igt-dev] ✗ GitLab.Pipeline: warning for series starting with [i-g-t] tests/gem_exec_reloc: Remove banned tests (rev2) Patchwork
@ 2021-06-08 20:08 ` Patchwork
2021-06-09 0:49 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
11 siblings, 0 replies; 20+ messages in thread
From: Patchwork @ 2021-06-08 20:08 UTC (permalink / raw)
To: Daniel Vetter; +Cc: igt-dev
[-- Attachment #1.1: Type: text/plain, Size: 9063 bytes --]
== Series Details ==
Series: series starting with [i-g-t] tests/gem_exec_reloc: Remove banned tests (rev2)
URL : https://patchwork.freedesktop.org/series/91160/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10194 -> IGTPW_5901
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/index.html
Known issues
------------
Here are the changes found in IGTPW_5901 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@amdgpu/amd_cs_nop@sync-fork-compute0:
- fi-snb-2600: NOTRUN -> [SKIP][1] ([fdo#109271]) +17 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-snb-2600/igt@amdgpu/amd_cs_nop@sync-fork-compute0.html
* igt@gem_exec_suspend@basic-s3:
- fi-tgl-u2: [PASS][2] -> [FAIL][3] ([i915#1888])
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-tgl-u2/igt@gem_exec_suspend@basic-s3.html
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-tgl-u2/igt@gem_exec_suspend@basic-s3.html
* igt@i915_selftest@live@gt_heartbeat:
- fi-tgl-y: [PASS][4] -> [DMESG-FAIL][5] ([i915#541])
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-tgl-y/igt@i915_selftest@live@gt_heartbeat.html
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-tgl-y/igt@i915_selftest@live@gt_heartbeat.html
#### Possible fixes ####
* igt@i915_selftest@live@hangcheck:
- fi-snb-2600: [INCOMPLETE][6] ([i915#2782]) -> [PASS][7]
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
#### Warnings ####
* igt@i915_selftest@live@execlists:
- fi-icl-u2: [INCOMPLETE][8] ([i915#2782] / [i915#3462]) -> [DMESG-FAIL][9] ([i915#3462])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-icl-u2/igt@i915_selftest@live@execlists.html
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-icl-u2/igt@i915_selftest@live@execlists.html
- fi-tgl-u2: [INCOMPLETE][10] ([i915#3462]) -> [DMESG-FAIL][11] ([i915#3462])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-tgl-u2/igt@i915_selftest@live@execlists.html
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-tgl-u2/igt@i915_selftest@live@execlists.html
* igt@runner@aborted:
- fi-cfl-8700k: [FAIL][12] ([i915#3363]) -> [FAIL][13] ([i915#2426] / [i915#3363])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-cfl-8700k/igt@runner@aborted.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-cfl-8700k/igt@runner@aborted.html
- fi-icl-u2: [FAIL][14] ([i915#2782] / [i915#3363]) -> [FAIL][15] ([i915#2426] / [i915#2782] / [i915#3363])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-icl-u2/igt@runner@aborted.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-icl-u2/igt@runner@aborted.html
- fi-glk-dsi: [FAIL][16] ([i915#2426] / [i915#3363] / [k.org#202321]) -> [FAIL][17] ([i915#3363] / [k.org#202321])
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-glk-dsi/igt@runner@aborted.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-glk-dsi/igt@runner@aborted.html
- fi-kbl-8809g: [FAIL][18] ([i915#1436] / [i915#3363]) -> [FAIL][19] ([i915#1436] / [i915#2426] / [i915#3363])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-kbl-8809g/igt@runner@aborted.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-kbl-8809g/igt@runner@aborted.html
- fi-bdw-5557u: [FAIL][20] ([i915#2426] / [i915#3462]) -> [FAIL][21] ([i915#3462])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-bdw-5557u/igt@runner@aborted.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-bdw-5557u/igt@runner@aborted.html
- fi-kbl-soraka: [FAIL][22] ([i915#1436] / [i915#3363]) -> [FAIL][23] ([i915#1436] / [i915#2426] / [i915#3363])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-kbl-soraka/igt@runner@aborted.html
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-kbl-soraka/igt@runner@aborted.html
- fi-kbl-7500u: [FAIL][24] ([i915#1436] / [i915#2426] / [i915#3363]) -> [FAIL][25] ([i915#1436] / [i915#3363])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-kbl-7500u/igt@runner@aborted.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-kbl-7500u/igt@runner@aborted.html
- fi-cml-u2: [FAIL][26] ([i915#2082] / [i915#2426] / [i915#3363] / [i915#3462]) -> [FAIL][27] ([i915#3363] / [i915#3462])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-cml-u2/igt@runner@aborted.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-cml-u2/igt@runner@aborted.html
- fi-bxt-dsi: [FAIL][28] ([i915#2426] / [i915#3363]) -> [FAIL][29] ([i915#3363])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-bxt-dsi/igt@runner@aborted.html
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-bxt-dsi/igt@runner@aborted.html
- fi-cml-s: [FAIL][30] ([i915#3363] / [i915#3462]) -> [FAIL][31] ([i915#2082] / [i915#2426] / [i915#3363] / [i915#3462])
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-cml-s/igt@runner@aborted.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-cml-s/igt@runner@aborted.html
- fi-skl-guc: [FAIL][32] ([i915#1436] / [i915#2426] / [i915#3363]) -> [FAIL][33] ([i915#1436] / [i915#3363])
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/fi-skl-guc/igt@runner@aborted.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/fi-skl-guc/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#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
[i915#1888]: https://gitlab.freedesktop.org/drm/intel/issues/1888
[i915#2082]: https://gitlab.freedesktop.org/drm/intel/issues/2082
[i915#2426]: https://gitlab.freedesktop.org/drm/intel/issues/2426
[i915#2782]: https://gitlab.freedesktop.org/drm/intel/issues/2782
[i915#3303]: https://gitlab.freedesktop.org/drm/intel/issues/3303
[i915#3363]: https://gitlab.freedesktop.org/drm/intel/issues/3363
[i915#3462]: https://gitlab.freedesktop.org/drm/intel/issues/3462
[i915#541]: https://gitlab.freedesktop.org/drm/intel/issues/541
[k.org#202321]: https://bugzilla.kernel.org/show_bug.cgi?id=202321
Participating hosts (46 -> 41)
------------------------------
Missing (5): fi-ilk-m540 fi-hsw-4200u fi-bsw-cyan fi-dg1-1 fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* IGT: IGT_6102 -> IGTPW_5901
CI-20190529: 20190529
CI_DRM_10194: 08ac6453e0060df3adc8e95e0d6850414bc24fad @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_5901: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/index.html
IGT_6102: 6e67969bf93dda8f22773ccae362f757fce25c3d @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
== Testlist changes ==
-igt@gem_exec_reloc@basic-concurrent0
-igt@gem_exec_reloc@basic-concurrent16
-igt@gem_exec_reloc@basic-many-active
-igt@gem_exec_reloc@basic-parallel
-igt@gem_exec_reloc@basic-spin
-igt@gem_exec_reloc@basic-spin-others
-igt@gem_exec_reloc@basic-wide-active
-igt@gem_exec_reloc@cpu-gtt-hang
-igt@gem_exec_reloc@cpu-gtt-interruptible
-igt@gem_exec_reloc@cpu-hang
-igt@gem_exec_reloc@cpu-interruptible
-igt@gem_exec_reloc@cpu-read-hang
-igt@gem_exec_reloc@cpu-read-interruptible
-igt@gem_exec_reloc@cpu-wc-hang
-igt@gem_exec_reloc@cpu-wc-interruptible
-igt@gem_exec_reloc@gtt-cpu-hang
-igt@gem_exec_reloc@gtt-cpu-interruptible
-igt@gem_exec_reloc@gtt-hang
-igt@gem_exec_reloc@gtt-interruptible
-igt@gem_exec_reloc@gtt-read-hang
-igt@gem_exec_reloc@gtt-read-interruptible
-igt@gem_exec_reloc@gtt-wc-hang
-igt@gem_exec_reloc@gtt-wc-interruptible
-igt@gem_exec_reloc@range-hang
-igt@gem_exec_reloc@range-interruptible
-igt@gem_exec_reloc@wc-cpu-hang
-igt@gem_exec_reloc@wc-cpu-interruptible
-igt@gem_exec_reloc@wc-gtt-hang
-igt@gem_exec_reloc@wc-gtt-interruptible
-igt@gem_exec_reloc@wc-hang
-igt@gem_exec_reloc@wc-interruptible
-igt@gem_exec_reloc@wc-read-hang
-igt@gem_exec_reloc@wc-read-interruptible
-igt@gem_exec_reloc@write-cpu-hang
-igt@gem_exec_reloc@write-cpu-interruptible
-igt@gem_exec_reloc@write-gtt-hang
-igt@gem_exec_reloc@write-gtt-interruptible
-igt@gem_exec_reloc@write-read-hang
-igt@gem_exec_reloc@write-read-interruptible
-igt@gem_exec_reloc@write-wc-hang
-igt@gem_exec_reloc@write-wc-interruptible
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/index.html
[-- Attachment #1.2: Type: text/html, Size: 13385 bytes --]
[-- Attachment #2: Type: text/plain, Size: 154 bytes --]
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for series starting with [i-g-t] tests/gem_exec_reloc: Remove banned tests (rev2)
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
` (10 preceding siblings ...)
2021-06-08 20:08 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
@ 2021-06-09 0:49 ` Patchwork
11 siblings, 0 replies; 20+ messages in thread
From: Patchwork @ 2021-06-09 0:49 UTC (permalink / raw)
To: Daniel Vetter; +Cc: igt-dev
[-- Attachment #1.1: Type: text/plain, Size: 30295 bytes --]
== Series Details ==
Series: series starting with [i-g-t] tests/gem_exec_reloc: Remove banned tests (rev2)
URL : https://patchwork.freedesktop.org/series/91160/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_10194_full -> IGTPW_5901_full
====================================================
Summary
-------
**WARNING**
Minor unknown changes coming with IGTPW_5901_full need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_5901_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/index.html
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_5901_full:
### IGT changes ###
#### Possible regressions ####
* {igt@gem_exec_reloc@gpu} (NEW):
- shard-glk: NOTRUN -> [FAIL][1]
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk7/igt@gem_exec_reloc@gpu.html
- shard-kbl: NOTRUN -> [FAIL][2]
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl3/igt@gem_exec_reloc@gpu.html
* {igt@gem_exec_reloc@readonly-30} (NEW):
- shard-kbl: NOTRUN -> [TIMEOUT][3]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl1/igt@gem_exec_reloc@readonly-30.html
- shard-snb: NOTRUN -> [TIMEOUT][4]
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-snb2/igt@gem_exec_reloc@readonly-30.html
* {igt@gem_exec_reloc@readonly-31} (NEW):
- shard-apl: NOTRUN -> [INCOMPLETE][5] +1 similar issue
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl8/igt@gem_exec_reloc@readonly-31.html
- shard-glk: NOTRUN -> [INCOMPLETE][6] +1 similar issue
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk3/igt@gem_exec_reloc@readonly-31.html
- shard-iclb: NOTRUN -> [INCOMPLETE][7] +1 similar issue
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb4/igt@gem_exec_reloc@readonly-31.html
- shard-kbl: NOTRUN -> [INCOMPLETE][8]
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl2/igt@gem_exec_reloc@readonly-31.html
- shard-snb: NOTRUN -> [INCOMPLETE][9]
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-snb5/igt@gem_exec_reloc@readonly-31.html
- shard-tglb: NOTRUN -> [TIMEOUT][10]
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb6/igt@gem_exec_reloc@readonly-31.html
* {igt@gem_exec_reloc@readonly-32} (NEW):
- shard-tglb: NOTRUN -> [INCOMPLETE][11]
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb5/igt@gem_exec_reloc@readonly-32.html
#### Warnings ####
* igt@runner@aborted:
- shard-iclb: ([FAIL][12], [FAIL][13], [FAIL][14]) ([i915#2426] / [i915#2782] / [i915#3002]) -> ([FAIL][15], [FAIL][16], [FAIL][17], [FAIL][18], [FAIL][19]) ([i915#2782] / [i915#3002])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/shard-iclb1/igt@runner@aborted.html
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/shard-iclb8/igt@runner@aborted.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/shard-iclb1/igt@runner@aborted.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb3/igt@runner@aborted.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb3/igt@runner@aborted.html
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb7/igt@runner@aborted.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb4/igt@runner@aborted.html
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb1/igt@runner@aborted.html
New tests
---------
New tests have been introduced between CI_DRM_10194_full and IGTPW_5901_full:
### New IGT tests (107) ###
* igt@gem_exec_reloc@cpu-12:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@cpu-13:
- Statuses : 5 pass(s)
- Exec time: [0.29, 0.92] s
* igt@gem_exec_reloc@cpu-14:
- Statuses : 5 pass(s)
- Exec time: [0.19, 0.49] s
* igt@gem_exec_reloc@cpu-15:
- Statuses : 6 pass(s)
- Exec time: [0.11, 0.34] s
* igt@gem_exec_reloc@cpu-16:
- Statuses : 6 pass(s)
- Exec time: [0.06, 0.20] s
* igt@gem_exec_reloc@cpu-17:
- Statuses : 4 pass(s)
- Exec time: [0.33, 0.85] s
* igt@gem_exec_reloc@cpu-18:
- Statuses : 6 pass(s)
- Exec time: [0.06, 0.16] s
* igt@gem_exec_reloc@cpu-19:
- Statuses : 5 pass(s)
- Exec time: [0.36, 1.23] s
* igt@gem_exec_reloc@cpu-20:
- Statuses : 5 pass(s)
- Exec time: [0.07, 0.22] s
* igt@gem_exec_reloc@cpu-21:
- Statuses : 4 pass(s)
- Exec time: [0.07, 0.21] s
* igt@gem_exec_reloc@cpu-22:
- Statuses : 5 pass(s)
- Exec time: [0.09, 0.21] s
* igt@gem_exec_reloc@cpu-23:
- Statuses : 6 pass(s)
- Exec time: [0.09, 0.29] s
* igt@gem_exec_reloc@cpu-24:
- Statuses : 6 pass(s)
- Exec time: [0.12, 0.42] s
* igt@gem_exec_reloc@cpu-25:
- Statuses : 6 pass(s)
- Exec time: [0.21, 0.69] s
* igt@gem_exec_reloc@cpu-26:
- Statuses : 5 pass(s)
- Exec time: [0.53, 1.39] s
* igt@gem_exec_reloc@cpu-27:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@cpu-28:
- Statuses : 5 pass(s)
- Exec time: [0.93, 2.66] s
* igt@gem_exec_reloc@cpu-29:
- Statuses : 6 pass(s)
- Exec time: [1.69, 6.48] s
* igt@gem_exec_reloc@cpu-30:
- Statuses : 6 pass(s)
- Exec time: [3.44, 13.03] s
* igt@gem_exec_reloc@cpu-31:
- Statuses : 4 pass(s)
- Exec time: [6.68, 24.84] s
* igt@gem_exec_reloc@cpu-32:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@gpu:
- Statuses : 2 fail(s)
- Exec time: [0.08, 0.13] s
* igt@gem_exec_reloc@gtt-12:
- Statuses : 6 pass(s)
- Exec time: [0.07, 0.48] s
* igt@gem_exec_reloc@gtt-13:
- Statuses : 6 pass(s)
- Exec time: [0.18, 0.63] s
* igt@gem_exec_reloc@gtt-14:
- Statuses : 4 pass(s)
- Exec time: [0.09, 0.22] s
* igt@gem_exec_reloc@gtt-15:
- Statuses : 6 pass(s)
- Exec time: [0.28, 0.92] s
* igt@gem_exec_reloc@gtt-16:
- Statuses : 6 pass(s)
- Exec time: [0.06, 0.16] s
* igt@gem_exec_reloc@gtt-17:
- Statuses : 6 pass(s)
- Exec time: [0.07, 0.20] s
* igt@gem_exec_reloc@gtt-18:
- Statuses : 5 pass(s)
- Exec time: [0.08, 0.22] s
* igt@gem_exec_reloc@gtt-19:
- Statuses : 6 pass(s)
- Exec time: [0.33, 1.07] s
* igt@gem_exec_reloc@gtt-20:
- Statuses : 5 pass(s)
- Exec time: [0.11, 0.33] s
* igt@gem_exec_reloc@gtt-21:
- Statuses : 5 pass(s)
- Exec time: [0.11, 0.27] s
* igt@gem_exec_reloc@gtt-22:
- Statuses : 1 pass(s)
- Exec time: [0.23] s
* igt@gem_exec_reloc@gtt-23:
- Statuses : 6 pass(s)
- Exec time: [0.24, 0.70] s
* igt@gem_exec_reloc@gtt-24:
- Statuses : 6 pass(s)
- Exec time: [0.43, 1.42] s
* igt@gem_exec_reloc@gtt-25:
- Statuses : 5 pass(s)
- Exec time: [0.45, 1.62] s
* igt@gem_exec_reloc@gtt-26:
- Statuses : 5 pass(s)
- Exec time: [1.17, 3.56] s
* igt@gem_exec_reloc@gtt-27:
- Statuses : 5 pass(s)
- Exec time: [1.55, 5.38] s
* igt@gem_exec_reloc@gtt-28:
- Statuses : 5 pass(s)
- Exec time: [3.83, 17.30] s
* igt@gem_exec_reloc@gtt-29:
- Statuses : 6 pass(s)
- Exec time: [6.93, 30.12] s
* igt@gem_exec_reloc@gtt-30:
- Statuses : 4 pass(s)
- Exec time: [17.42, 57.39] s
* igt@gem_exec_reloc@gtt-31:
- Statuses : 6 pass(s)
- Exec time: [26.31, 118.00] s
* igt@gem_exec_reloc@gtt-32:
- Statuses : 2 pass(s) 2 skip(s)
- Exec time: [0.12, 100.09] s
* igt@gem_exec_reloc@invalid-domains:
- Statuses : 5 pass(s)
- Exec time: [0.01, 0.03] s
* igt@gem_exec_reloc@mmap-12:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@mmap-13:
- Statuses : 6 pass(s)
- Exec time: [0.09, 0.27] s
* igt@gem_exec_reloc@mmap-14:
- Statuses : 6 pass(s)
- Exec time: [0.08, 0.24] s
* igt@gem_exec_reloc@mmap-15:
- Statuses : 4 pass(s)
- Exec time: [0.11, 0.25] s
* igt@gem_exec_reloc@mmap-16:
- Statuses : 5 pass(s)
- Exec time: [0.10, 0.35] s
* igt@gem_exec_reloc@mmap-17:
- Statuses : 5 pass(s)
- Exec time: [0.08, 0.29] s
* igt@gem_exec_reloc@mmap-18:
- Statuses : 6 pass(s)
- Exec time: [0.32, 1.04] s
* igt@gem_exec_reloc@mmap-19:
- Statuses : 1 pass(s)
- Exec time: [0.22] s
* igt@gem_exec_reloc@mmap-20:
- Statuses : 5 pass(s)
- Exec time: [0.06, 0.20] s
* igt@gem_exec_reloc@mmap-21:
- Statuses : 2 pass(s)
- Exec time: [0.11, 0.27] s
* igt@gem_exec_reloc@mmap-22:
- Statuses : 6 pass(s)
- Exec time: [0.36, 1.20] s
* igt@gem_exec_reloc@mmap-23:
- Statuses : 5 pass(s)
- Exec time: [0.09, 0.23] s
* igt@gem_exec_reloc@mmap-24:
- Statuses : 6 pass(s)
- Exec time: [0.32, 1.03] s
* igt@gem_exec_reloc@mmap-25:
- Statuses : 6 pass(s)
- Exec time: [0.41, 1.67] s
* igt@gem_exec_reloc@mmap-26:
- Statuses : 6 pass(s)
- Exec time: [0.30, 0.92] s
* igt@gem_exec_reloc@mmap-27:
- Statuses : 6 pass(s)
- Exec time: [0.19, 0.94] s
* igt@gem_exec_reloc@mmap-28:
- Statuses : 5 pass(s)
- Exec time: [0.34, 0.81] s
* igt@gem_exec_reloc@mmap-29:
- Statuses : 5 pass(s)
- Exec time: [0.56, 1.47] s
* igt@gem_exec_reloc@mmap-30:
- Statuses : 5 pass(s)
- Exec time: [1.23, 3.68] s
* igt@gem_exec_reloc@mmap-31:
- Statuses : 6 pass(s)
- Exec time: [1.79, 7.21] s
* igt@gem_exec_reloc@mmap-32:
- Statuses : 2 pass(s) 3 skip(s)
- Exec time: [0.07, 4.85] s
* igt@gem_exec_reloc@readonly-12:
- Statuses : 5 pass(s)
- Exec time: [0.24, 0.71] s
* igt@gem_exec_reloc@readonly-13:
- Statuses : 6 pass(s)
- Exec time: [0.07, 0.21] s
* igt@gem_exec_reloc@readonly-14:
- Statuses : 6 pass(s)
- Exec time: [0.12, 0.34] s
* igt@gem_exec_reloc@readonly-15:
- Statuses : 5 pass(s)
- Exec time: [0.07, 0.21] s
* igt@gem_exec_reloc@readonly-16:
- Statuses : 5 pass(s)
- Exec time: [0.16, 0.30] s
* igt@gem_exec_reloc@readonly-17:
- Statuses : 6 pass(s)
- Exec time: [0.11, 0.38] s
* igt@gem_exec_reloc@readonly-18:
- Statuses : 5 pass(s)
- Exec time: [0.08, 0.24] s
* igt@gem_exec_reloc@readonly-19:
- Statuses : 6 pass(s)
- Exec time: [0.13, 0.52] s
* igt@gem_exec_reloc@readonly-20:
- Statuses : 5 pass(s)
- Exec time: [0.18, 0.48] s
* igt@gem_exec_reloc@readonly-21:
- Statuses : 5 pass(s)
- Exec time: [0.42, 1.19] s
* igt@gem_exec_reloc@readonly-22:
- Statuses : 4 pass(s)
- Exec time: [0.23, 1.43] s
* igt@gem_exec_reloc@readonly-23:
- Statuses : 6 pass(s)
- Exec time: [0.53, 2.49] s
* igt@gem_exec_reloc@readonly-24:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@readonly-25:
- Statuses : 4 pass(s)
- Exec time: [3.04, 7.93] s
* igt@gem_exec_reloc@readonly-26:
- Statuses : 6 pass(s)
- Exec time: [6.86, 27.54] s
* igt@gem_exec_reloc@readonly-27:
- Statuses : 6 pass(s)
- Exec time: [3.23, 69.91] s
* igt@gem_exec_reloc@readonly-28:
- Statuses : 5 pass(s)
- Exec time: [26.42, 104.69] s
* igt@gem_exec_reloc@readonly-29:
- Statuses : 5 pass(s)
- Exec time: [23.44, 97.25] s
* igt@gem_exec_reloc@readonly-30:
- Statuses : 2 incomplete(s) 2 pass(s) 2 timeout(s)
- Exec time: [0.0, 143.27] s
* igt@gem_exec_reloc@readonly-31:
- Statuses : 5 incomplete(s) 1 timeout(s)
- Exec time: [0.0, 241.27] s
* igt@gem_exec_reloc@readonly-32:
- Statuses : 2 incomplete(s) 2 skip(s)
- Exec time: [0.0, 0.26] s
* igt@gem_exec_reloc@wc-12:
- Statuses : 6 pass(s)
- Exec time: [0.06, 0.18] s
* igt@gem_exec_reloc@wc-13:
- Statuses : 5 pass(s)
- Exec time: [0.12, 0.30] s
* igt@gem_exec_reloc@wc-14:
- Statuses : 6 pass(s)
- Exec time: [0.06, 0.18] s
* igt@gem_exec_reloc@wc-15:
- Statuses : 6 pass(s)
- Exec time: [0.09, 0.49] s
* igt@gem_exec_reloc@wc-16:
- Statuses : 6 pass(s)
- Exec time: [0.09, 0.24] s
* igt@gem_exec_reloc@wc-17:
- Statuses : 6 pass(s)
- Exec time: [0.09, 0.31] s
* igt@gem_exec_reloc@wc-18:
- Statuses : 5 pass(s)
- Exec time: [0.12, 0.36] s
* igt@gem_exec_reloc@wc-19:
- Statuses : 5 pass(s)
- Exec time: [0.37, 0.96] s
* igt@gem_exec_reloc@wc-20:
- Statuses : 6 pass(s)
- Exec time: [0.07, 0.19] s
* igt@gem_exec_reloc@wc-21:
- Statuses : 6 pass(s)
- Exec time: [0.10, 0.26] s
* igt@gem_exec_reloc@wc-22:
- Statuses : 6 pass(s)
- Exec time: [0.15, 0.41] s
* igt@gem_exec_reloc@wc-23:
- Statuses : 4 pass(s)
- Exec time: [0.16, 0.34] s
* igt@gem_exec_reloc@wc-24:
- Statuses : 6 pass(s)
- Exec time: [0.23, 0.53] s
* igt@gem_exec_reloc@wc-25:
- Statuses : 1 pass(s)
- Exec time: [0.52] s
* igt@gem_exec_reloc@wc-26:
- Statuses : 6 pass(s)
- Exec time: [0.68, 1.48] s
* igt@gem_exec_reloc@wc-27:
- Statuses : 4 pass(s)
- Exec time: [1.32, 2.56] s
* igt@gem_exec_reloc@wc-28:
- Statuses :
- Exec time: [None] s
* igt@gem_exec_reloc@wc-29:
- Statuses : 5 pass(s)
- Exec time: [4.94, 11.04] s
* igt@gem_exec_reloc@wc-30:
- Statuses : 6 pass(s)
- Exec time: [10.15, 21.59] s
* igt@gem_exec_reloc@wc-31:
- Statuses : 5 pass(s)
- Exec time: [19.66, 43.73] s
* igt@gem_exec_reloc@wc-32:
- Statuses :
- Exec time: [None] s
Known issues
------------
Here are the changes found in IGTPW_5901_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_create@create-clear:
- shard-glk: [PASS][20] -> [FAIL][21] ([i915#3160])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/shard-glk6/igt@gem_create@create-clear.html
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk8/igt@gem_create@create-clear.html
* igt@gem_create@create-massive:
- shard-kbl: NOTRUN -> [DMESG-WARN][22] ([i915#3002])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl3/igt@gem_create@create-massive.html
* igt@gem_ctx_persistence@smoketest:
- shard-snb: NOTRUN -> [SKIP][23] ([fdo#109271] / [i915#1099]) +5 similar issues
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-snb7/igt@gem_ctx_persistence@smoketest.html
* igt@gem_eio@in-flight-contexts-10ms:
- shard-tglb: [PASS][24] -> [TIMEOUT][25] ([i915#3063])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/shard-tglb3/igt@gem_eio@in-flight-contexts-10ms.html
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb2/igt@gem_eio@in-flight-contexts-10ms.html
* igt@gem_exec_fair@basic-none-rrul@rcs0:
- shard-glk: [PASS][26] -> [FAIL][27] ([i915#2842])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/shard-glk5/igt@gem_exec_fair@basic-none-rrul@rcs0.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk1/igt@gem_exec_fair@basic-none-rrul@rcs0.html
* igt@gem_exec_fair@basic-pace-share@rcs0:
- shard-tglb: NOTRUN -> [FAIL][28] ([i915#2842])
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb8/igt@gem_exec_fair@basic-pace-share@rcs0.html
- shard-glk: NOTRUN -> [FAIL][29] ([i915#2842])
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk9/igt@gem_exec_fair@basic-pace-share@rcs0.html
* igt@gem_exec_fair@basic-pace@vecs0:
- shard-kbl: [PASS][30] -> [FAIL][31] ([i915#2842]) +1 similar issue
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/shard-kbl6/igt@gem_exec_fair@basic-pace@vecs0.html
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl2/igt@gem_exec_fair@basic-pace@vecs0.html
* igt@gem_exec_fair@basic-throttle@rcs0:
- shard-iclb: [PASS][32] -> [FAIL][33] ([i915#2849])
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/shard-iclb5/igt@gem_exec_fair@basic-throttle@rcs0.html
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb7/igt@gem_exec_fair@basic-throttle@rcs0.html
* igt@gem_huc_copy@huc-copy:
- shard-apl: NOTRUN -> [SKIP][34] ([fdo#109271] / [i915#2190])
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl8/igt@gem_huc_copy@huc-copy.html
* igt@gem_pwrite@basic-exhaustion:
- shard-apl: NOTRUN -> [WARN][35] ([i915#2658])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl3/igt@gem_pwrite@basic-exhaustion.html
* igt@gem_userptr_blits@dmabuf-sync:
- shard-kbl: NOTRUN -> [SKIP][36] ([fdo#109271] / [i915#3323])
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl7/igt@gem_userptr_blits@dmabuf-sync.html
- shard-iclb: NOTRUN -> [SKIP][37] ([i915#3323])
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb3/igt@gem_userptr_blits@dmabuf-sync.html
- shard-apl: NOTRUN -> [SKIP][38] ([fdo#109271] / [i915#3323])
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl6/igt@gem_userptr_blits@dmabuf-sync.html
- shard-glk: NOTRUN -> [SKIP][39] ([fdo#109271] / [i915#3323])
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk3/igt@gem_userptr_blits@dmabuf-sync.html
- shard-tglb: NOTRUN -> [SKIP][40] ([i915#3323])
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb8/igt@gem_userptr_blits@dmabuf-sync.html
* igt@gen7_exec_parse@oacontrol-tracking:
- shard-iclb: NOTRUN -> [SKIP][41] ([fdo#109289]) +1 similar issue
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb5/igt@gen7_exec_parse@oacontrol-tracking.html
- shard-tglb: NOTRUN -> [SKIP][42] ([fdo#109289]) +1 similar issue
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb3/igt@gen7_exec_parse@oacontrol-tracking.html
* igt@i915_pm_rpm@modeset-lpsp-stress:
- shard-apl: NOTRUN -> [SKIP][43] ([fdo#109271]) +194 similar issues
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl1/igt@i915_pm_rpm@modeset-lpsp-stress.html
* igt@i915_selftest@live@execlists:
- shard-apl: NOTRUN -> [DMESG-FAIL][44] ([i915#3462])
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl2/igt@i915_selftest@live@execlists.html
* igt@kms_ccs@pipe-c-random-ccs-data:
- shard-snb: NOTRUN -> [SKIP][45] ([fdo#109271]) +377 similar issues
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-snb6/igt@kms_ccs@pipe-c-random-ccs-data.html
* igt@kms_chamelium@hdmi-hpd-fast:
- shard-snb: NOTRUN -> [SKIP][46] ([fdo#109271] / [fdo#111827]) +20 similar issues
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-snb7/igt@kms_chamelium@hdmi-hpd-fast.html
* igt@kms_chamelium@vga-hpd:
- shard-apl: NOTRUN -> [SKIP][47] ([fdo#109271] / [fdo#111827]) +16 similar issues
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl1/igt@kms_chamelium@vga-hpd.html
* igt@kms_chamelium@vga-hpd-without-ddc:
- shard-kbl: NOTRUN -> [SKIP][48] ([fdo#109271] / [fdo#111827]) +11 similar issues
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl6/igt@kms_chamelium@vga-hpd-without-ddc.html
* igt@kms_color@pipe-d-ctm-0-25:
- shard-iclb: NOTRUN -> [SKIP][49] ([fdo#109278] / [i915#1149])
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb4/igt@kms_color@pipe-d-ctm-0-25.html
* igt@kms_color_chamelium@pipe-a-ctm-red-to-blue:
- shard-iclb: NOTRUN -> [SKIP][50] ([fdo#109284] / [fdo#111827]) +2 similar issues
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb2/igt@kms_color_chamelium@pipe-a-ctm-red-to-blue.html
* igt@kms_color_chamelium@pipe-b-ctm-max:
- shard-tglb: NOTRUN -> [SKIP][51] ([fdo#109284] / [fdo#111827]) +2 similar issues
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb5/igt@kms_color_chamelium@pipe-b-ctm-max.html
- shard-glk: NOTRUN -> [SKIP][52] ([fdo#109271] / [fdo#111827]) +2 similar issues
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk5/igt@kms_color_chamelium@pipe-b-ctm-max.html
* igt@kms_content_protection@atomic:
- shard-kbl: NOTRUN -> [TIMEOUT][53] ([i915#1319]) +1 similar issue
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl4/igt@kms_content_protection@atomic.html
- shard-apl: NOTRUN -> [TIMEOUT][54] ([i915#1319]) +1 similar issue
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl1/igt@kms_content_protection@atomic.html
* igt@kms_content_protection@srm:
- shard-tglb: NOTRUN -> [SKIP][55] ([fdo#111828])
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb6/igt@kms_content_protection@srm.html
- shard-iclb: NOTRUN -> [SKIP][56] ([fdo#109300] / [fdo#111066])
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb4/igt@kms_content_protection@srm.html
* igt@kms_content_protection@uevent:
- shard-kbl: NOTRUN -> [FAIL][57] ([i915#2105])
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl1/igt@kms_content_protection@uevent.html
* igt@kms_cursor_crc@pipe-a-cursor-max-size-rapid-movement:
- shard-tglb: NOTRUN -> [SKIP][58] ([i915#3359]) +2 similar issues
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb2/igt@kms_cursor_crc@pipe-a-cursor-max-size-rapid-movement.html
* igt@kms_cursor_crc@pipe-b-cursor-32x32-rapid-movement:
- shard-tglb: NOTRUN -> [SKIP][59] ([i915#3319]) +1 similar issue
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb5/igt@kms_cursor_crc@pipe-b-cursor-32x32-rapid-movement.html
* igt@kms_cursor_crc@pipe-b-cursor-512x170-offscreen:
- shard-iclb: NOTRUN -> [SKIP][60] ([fdo#109278] / [fdo#109279]) +2 similar issues
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb4/igt@kms_cursor_crc@pipe-b-cursor-512x170-offscreen.html
* igt@kms_cursor_crc@pipe-c-cursor-suspend:
- shard-kbl: [PASS][61] -> [DMESG-WARN][62] ([i915#180])
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/shard-kbl7/igt@kms_cursor_crc@pipe-c-cursor-suspend.html
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl2/igt@kms_cursor_crc@pipe-c-cursor-suspend.html
* igt@kms_cursor_crc@pipe-d-cursor-512x512-onscreen:
- shard-tglb: NOTRUN -> [SKIP][63] ([fdo#109279] / [i915#3359]) +3 similar issues
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb5/igt@kms_cursor_crc@pipe-d-cursor-512x512-onscreen.html
* igt@kms_cursor_edge_walk@pipe-d-128x128-top-edge:
- shard-iclb: NOTRUN -> [SKIP][64] ([fdo#109278]) +11 similar issues
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb8/igt@kms_cursor_edge_walk@pipe-d-128x128-top-edge.html
* igt@kms_cursor_legacy@cursorb-vs-flipb-atomic:
- shard-tglb: NOTRUN -> [SKIP][65] ([fdo#111825]) +15 similar issues
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb6/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic.html
- shard-iclb: NOTRUN -> [SKIP][66] ([fdo#109274] / [fdo#109278])
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb8/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic.html
* igt@kms_cursor_legacy@pipe-d-torture-bo:
- shard-kbl: NOTRUN -> [SKIP][67] ([fdo#109271] / [i915#533]) +2 similar issues
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl1/igt@kms_cursor_legacy@pipe-d-torture-bo.html
* igt@kms_cursor_legacy@short-flip-after-cursor-atomic-transitions-varying-size:
- shard-snb: [PASS][68] -> [SKIP][69] ([fdo#109271]) +2 similar issues
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/shard-snb6/igt@kms_cursor_legacy@short-flip-after-cursor-atomic-transitions-varying-size.html
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-snb2/igt@kms_cursor_legacy@short-flip-after-cursor-atomic-transitions-varying-size.html
* igt@kms_flip@2x-nonexisting-fb:
- shard-iclb: NOTRUN -> [SKIP][70] ([fdo#109274]) +2 similar issues
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb5/igt@kms_flip@2x-nonexisting-fb.html
* igt@kms_flip@2x-plain-flip-ts-check@ac-hdmi-a1-hdmi-a2:
- shard-glk: [PASS][71] -> [FAIL][72] ([i915#2122])
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_10194/shard-glk3/igt@kms_flip@2x-plain-flip-ts-check@ac-hdmi-a1-hdmi-a2.html
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk1/igt@kms_flip@2x-plain-flip-ts-check@ac-hdmi-a1-hdmi-a2.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs:
- shard-kbl: NOTRUN -> [SKIP][73] ([fdo#109271] / [i915#2672])
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl4/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytilegen12rcccs.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-64bpp-ytile:
- shard-apl: NOTRUN -> [SKIP][74] ([fdo#109271] / [i915#2642])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl1/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-64bpp-ytile.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-blt:
- shard-kbl: NOTRUN -> [SKIP][75] ([fdo#109271]) +141 similar issues
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl7/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-indfb-draw-mmap-wc:
- shard-glk: NOTRUN -> [SKIP][76] ([fdo#109271]) +37 similar issues
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk5/igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-indfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@psr-2p-primscrn-pri-shrfb-draw-mmap-wc:
- shard-iclb: NOTRUN -> [SKIP][77] ([fdo#109280]) +10 similar issues
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb2/igt@kms_frontbuffer_tracking@psr-2p-primscrn-pri-shrfb-draw-mmap-wc.html
* igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d:
- shard-glk: NOTRUN -> [SKIP][78] ([fdo#109271] / [i915#533])
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk2/igt@kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-d.html
* igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a:
- shard-kbl: NOTRUN -> [DMESG-WARN][79] ([i915#180]) +1 similar issue
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl2/igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a.html
* igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb:
- shard-apl: NOTRUN -> [FAIL][80] ([i915#265])
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl8/igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb.html
- shard-glk: NOTRUN -> [FAIL][81] ([i915#265])
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk3/igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb.html
- shard-kbl: NOTRUN -> [FAIL][82] ([i915#265])
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl2/igt@kms_plane_alpha_blend@pipe-a-alpha-transparent-fb.html
* igt@kms_plane_alpha_blend@pipe-b-alpha-7efc:
- shard-kbl: NOTRUN -> [FAIL][83] ([fdo#108145] / [i915#265]) +1 similar issue
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl1/igt@kms_plane_alpha_blend@pipe-b-alpha-7efc.html
* igt@kms_plane_alpha_blend@pipe-c-alpha-basic:
- shard-apl: NOTRUN -> [FAIL][84] ([fdo#108145] / [i915#265]) +1 similar issue
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl2/igt@kms_plane_alpha_blend@pipe-c-alpha-basic.html
* igt@kms_plane_lowres@pipe-c-tiling-yf:
- shard-iclb: NOTRUN -> [SKIP][85] ([i915#3536])
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb4/igt@kms_plane_lowres@pipe-c-tiling-yf.html
- shard-tglb: NOTRUN -> [SKIP][86] ([fdo#111615])
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb1/igt@kms_plane_lowres@pipe-c-tiling-yf.html
* igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5:
- shard-kbl: NOTRUN -> [SKIP][87] ([fdo#109271] / [i915#658]) +1 similar issue
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-kbl3/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5.html
- shard-apl: NOTRUN -> [SKIP][88] ([fdo#109271] / [i915#658]) +1 similar issue
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-apl3/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5.html
- shard-glk: NOTRUN -> [SKIP][89] ([fdo#109271] / [i915#658])
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-glk1/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5.html
- shard-iclb: NOTRUN -> [SKIP][90] ([i915#658])
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb1/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5.html
- shard-tglb: NOTRUN -> [SKIP][91] ([i915#2920])
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-tglb5/igt@kms_psr2_sf@primary-plane-update-sf-dmg-area-5.html
* igt@kms_psr@psr2_cursor_mmap_cpu:
- shard-iclb: NOTRUN -> [SKIP][92] ([fdo#109441])
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/shard-iclb3/igt@kms_psr@psr2_cursor_mmap_cpu.html
- shard-tglb: NOTRUN -> [FAIL][93] ([i915#132] / [i915#3467
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_5901/index.html
[-- Attachment #1.2: Type: text/html, Size: 36090 bytes --]
[-- Attachment #2: Type: text/plain, Size: 154 bytes --]
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [igt-dev] [PATCH i-g-t] tests/gem_exec_reloc: Remove banned tests
2021-06-08 19:25 ` [igt-dev] [PATCH i-g-t] " Daniel Vetter
@ 2021-06-09 17:53 ` Jason Ekstrand
0 siblings, 0 replies; 20+ messages in thread
From: Jason Ekstrand @ 2021-06-09 17:53 UTC (permalink / raw)
To: Daniel Vetter; +Cc: IGT development, Daniel Vetter, Petri Latvala
On Tue, Jun 8, 2021 at 2:25 PM Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
>
> These have been banned across all CI for over 2 years, realistically
> that's not going to change and they're just in the way.
>
> v2: Keep the !HANG parts (CI)
>
> Cc: Jason Ekstrand <jason@jlekstrand.net>
> Cc: Petri Latvala <petri.latvala@intel.com>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
> tests/i915/gem_exec_reloc.c | 33 +++++++--------------------------
> tests/intel-ci/blacklist.txt | 1 -
> 2 files changed, 7 insertions(+), 27 deletions(-)
>
> diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
> index 397c335e4d3c..0e75a5e52cc4 100644
> --- a/tests/i915/gem_exec_reloc.c
> +++ b/tests/i915/gem_exec_reloc.c
> @@ -614,8 +614,6 @@ static bool has_64b_reloc(int fd)
>
> #define NORELOC 1
> #define ACTIVE 2
> -#define INTERRUPTIBLE 4
> -#define HANG 8
> static void basic_reloc(int fd, unsigned before, unsigned after, unsigned flags)
> {
> #define OBJSZ 8192
> @@ -678,8 +676,7 @@ static void basic_reloc(int fd, unsigned before, unsigned after, unsigned flags)
> spin = igt_spin_new(fd,
> .engine = I915_EXEC_DEFAULT,
> .dependency = obj.handle);
> - if (!(flags & HANG))
> - igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
> + igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
> igt_assert(gem_bo_busy(fd, obj.handle));
> }
>
> @@ -746,8 +743,7 @@ static void basic_reloc(int fd, unsigned before, unsigned after, unsigned flags)
> spin = igt_spin_new(fd,
> .engine = I915_EXEC_DEFAULT,
> .dependency = obj.handle);
> - if (!(flags & HANG))
> - igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
> + igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
> igt_assert(gem_bo_busy(fd, obj.handle));
> }
>
> @@ -873,8 +869,7 @@ static void basic_range(int fd, unsigned flags)
>
> if (flags & ACTIVE) {
> spin = igt_spin_new(fd, .dependency = obj[n].handle);
> - if (!(flags & HANG))
> - igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
> + igt_spin_set_timeout(spin, NSEC_PER_SEC/100);
> igt_assert(gem_bo_busy(fd, obj[n].handle));
> }
>
> @@ -1468,8 +1463,6 @@ igt_main
> { "", 0 , true},
> { "-noreloc", NORELOC, true },
> { "-active", ACTIVE, true },
> - { "-interruptible", ACTIVE | INTERRUPTIBLE },
> - { "-hang", ACTIVE | HANG },
> { },
> }, *f;
> uint64_t size;
> @@ -1483,14 +1476,7 @@ igt_main
> }
>
> for (f = flags; f->name; f++) {
> - igt_hang_t hang;
> -
> igt_subtest_group {
> - igt_fixture {
> - if (f->flags & HANG)
> - hang = igt_allow_hang(fd, 0, 0);
> - }
> -
> for (m = modes; m->name; m++) {
> igt_subtest_f("%s%s%s",
> f->basic ? "basic-" : "",
> @@ -1498,22 +1484,17 @@ igt_main
> f->name) {
> if ((m->before | m->after) & I915_GEM_DOMAIN_WC)
> igt_require(gem_mmap__has_wc(fd));
> - igt_while_interruptible(f->flags & INTERRUPTIBLE)
> - basic_reloc(fd, m->before, m->after, f->flags);
> +
> + basic_reloc(fd, m->before, m->after, f->flags);
> }
> }
>
> if (!(f->flags & NORELOC)) {
> igt_subtest_f("%srange%s",
> f->basic ? "basic-" : "", f->name) {
> - igt_while_interruptible(f->flags & INTERRUPTIBLE)
> - basic_range(fd, f->flags);
Did you mean to drop all basic_range here? I think not. With that
restored without the igt_while_interruptable(),
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
> - }
> - }
>
> - igt_fixture {
> - if (f->flags & HANG)
> - igt_disallow_hang(fd, hang);
> + basic_range(fd, f->flags);
> + }
> }
> }
> }
> diff --git a/tests/intel-ci/blacklist.txt b/tests/intel-ci/blacklist.txt
> index 33f92e37f602..4d2555f8adb3 100644
> --- a/tests/intel-ci/blacklist.txt
> +++ b/tests/intel-ci/blacklist.txt
> @@ -30,7 +30,6 @@ igt@gem_exec_flush@(?!.*basic).*
> igt@gem_exec_latency(@.*)?
> igt@gem_exec_lut_handle(@.*)?
> igt@gem_exec_nop@(?!.*basic).*
> -igt@gem_exec_reloc@(?!.*basic).*
> igt@gem_exec_suspend@(?!.*basic).*
> igt@gem_exec_whisper@(?!basic).*
> igt@gem_fd_exhaustion(@.*)?
> --
> 2.24.1
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/7] Revert "test/i915/gem_exec_reloc: Restore interclient testings"
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 2/7] Revert "test/i915/gem_exec_reloc: Restore interclient testings" Daniel Vetter
@ 2021-06-09 17:56 ` Jason Ekstrand
2021-06-14 10:40 ` Arkadiusz Hiler
1 sibling, 0 replies; 20+ messages in thread
From: Jason Ekstrand @ 2021-06-09 17:56 UTC (permalink / raw)
To: Daniel Vetter; +Cc: Petri Latvala, IGT development, Daniel Vetter, Dave Airlie
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
On Tue, Jun 8, 2021 at 4:40 AM Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
>
> This reverts commit cf2f41b3d3dfabaf3a4837062f996f3491a350b1.
>
> And I mean revert with extreme prejudice because this was already
> thrown out in
>
> commit 39e9aa1032a4e60f776f34b3ccf4fb728abbfe5c
> Author: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Date: Mon Aug 10 11:00:34 2020 +0200
>
> tests/i915: Remove subtests that rely on async relocation behavior
>
> and Chris then decided to undo that without any acks from the people
> who've removed these tests. Or well anyone else. Commit fights in
> upstream repositories are not acceptable.
>
> Cc: Jason Ekstrand <jason@jlekstrand.net>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: Dave Airlie <airlied@redhat.com>
> Cc: Petri Latvala <petri.latvala@intel.com>
> Cc: Arkadiusz Hiler <arek@hiler.eu>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
> tests/i915/gem_exec_reloc.c | 219 ------------------------------------
> 1 file changed, 219 deletions(-)
>
> diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
> index cb4899fe6e1e..efe6e2e02c52 100644
> --- a/tests/i915/gem_exec_reloc.c
> +++ b/tests/i915/gem_exec_reloc.c
> @@ -426,76 +426,6 @@ static void many_active(int i915, unsigned engine)
> }
> }
>
> -static void __wide_active(int i915, unsigned engine, unsigned long count)
> -{
> - struct drm_i915_gem_relocation_entry *reloc =
> - calloc(count, sizeof(*reloc));
> - struct drm_i915_gem_exec_object2 *obj =
> - calloc(count + 1, sizeof(*obj));
> - struct drm_i915_gem_execbuffer2 execbuf = {
> - .buffers_ptr = to_user_pointer(obj),
> - .buffer_count = count + 1,
> - .flags = engine | I915_EXEC_HANDLE_LUT,
> - };
> - igt_spin_t *spin;
> -
> - for (unsigned long i = 0; i < count; i++) {
> - obj[i].handle = gem_create(i915, 4096);
> - obj[i].flags = EXEC_OBJECT_WRITE;
> - obj[i].flags |= EXEC_OBJECT_SUPPORTS_48B_ADDRESS;
> - }
> -
> - spin = __igt_spin_new(i915,
> - .engine = engine,
> - .flags = (IGT_SPIN_FENCE_OUT |
> - IGT_SPIN_NO_PREEMPTION));
> - obj[count] = spin->obj[1];
> - gem_execbuf(i915, &execbuf); /* mark all the objects as active */
> -
> - for (unsigned long i = 0; i < count; i++) {
> - reloc[i].target_handle = i;
> - reloc[i].presumed_offset = ~0ull;
> - obj[i].relocs_ptr = to_user_pointer(&reloc[i]);
> - obj[i].relocation_count = 1;
> - }
> - gem_execbuf(i915, &execbuf); /* relocation onto active objects */
> -
> - igt_assert_eq(sync_fence_status(spin->out_fence), 0);
> - igt_spin_free(i915, spin);
> -
> - for (unsigned long i = 0; i < count; i++) {
> - uint64_t addr;
> -
> - gem_read(i915, obj[i].handle, 0, &addr, sizeof(addr));
> - igt_assert_eq_u64(addr, obj[i].offset);
> -
> - gem_close(i915, obj[i].handle);
> - }
> - free(obj);
> - free(reloc);
> -}
> -
> -static void wide_active(int i915, unsigned engine)
> -{
> - const uint64_t max = gem_aperture_size(i915) / 4096 / 2;
> - unsigned long count = 256;
> -
> - igt_until_timeout(2) {
> - uint64_t required, total;
> -
> - if (!__intel_check_memory(count, 4096, CHECK_RAM,
> - &required, &total))
> - break;
> -
> - igt_debug("Testing count:%lu\n", count);
> - __wide_active(i915, engine, count);
> -
> - count <<= 1;
> - if (count >= max)
> - break;
> - }
> -}
> -
> static unsigned int offset_in_page(void *addr)
> {
> return (uintptr_t)addr & 4095;
> @@ -932,145 +862,6 @@ static void basic_softpin(int fd)
> gem_close(fd, obj[1].handle);
> }
>
> -static struct drm_i915_gem_relocation_entry *
> -parallel_relocs(int count, unsigned long *out)
> -{
> - struct drm_i915_gem_relocation_entry *reloc;
> - unsigned long sz;
> - int i;
> -
> - sz = count * sizeof(*reloc);
> - sz = ALIGN(sz, 4096);
> -
> - reloc = mmap(0, sz, PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0);
> - igt_assert(reloc != MAP_FAILED);
> - for (i = 0; i < count; i++) {
> - reloc[i].target_handle = 0;
> - reloc[i].presumed_offset = ~0ull;
> - reloc[i].offset = 8 * i;
> - reloc[i].delta = i;
> - reloc[i].read_domains = I915_GEM_DOMAIN_INSTRUCTION;
> - reloc[i].write_domain = 0;
> - }
> - mprotect(reloc, sz, PROT_READ);
> -
> - *out = sz;
> - return reloc;
> -}
> -
> -static int __execbuf(int i915, struct drm_i915_gem_execbuffer2 *execbuf)
> -{
> - int err;
> -
> - err = 0;
> - if (ioctl(i915, DRM_IOCTL_I915_GEM_EXECBUFFER2, execbuf)) {
> - err = -errno;
> - igt_assume(err);
> - }
> -
> - errno = 0;
> - return err;
> -}
> -
> -static int stop;
> -static void sighandler(int sig)
> -{
> - stop = 1;
> -}
> -
> -static void parallel_child(int i915,
> - const struct intel_execution_engine2 *engine,
> - struct drm_i915_gem_relocation_entry *reloc,
> - uint32_t common)
> -{
> - igt_spin_t *spin = __igt_spin_new(i915, .engine = engine->flags);
> - struct drm_i915_gem_exec_object2 reloc_target = {
> - .handle = gem_create(i915, 32 * 1024 * 8),
> - .relocation_count = 32 * 1024,
> - .relocs_ptr = to_user_pointer(reloc),
> - };
> - struct drm_i915_gem_exec_object2 obj[3] = {
> - reloc_target,
> - { .handle = common },
> - spin->obj[1],
> - };
> - struct drm_i915_gem_execbuffer2 execbuf = {
> - .buffers_ptr = to_user_pointer(obj),
> - .buffer_count = ARRAY_SIZE(obj),
> - .flags = engine->flags | I915_EXEC_HANDLE_LUT,
> - };
> - struct sigaction act = {
> - .sa_handler = sighandler,
> - };
> - unsigned long count = 0;
> -
> - sigaction(SIGINT, &act, NULL);
> - while (!READ_ONCE(stop)) {
> - int err = __execbuf(i915, &execbuf);
> - if (err == -EINTR)
> - break;
> -
> - igt_assert_eq(err, 0);
> - count++;
> - }
> -
> - igt_info("%s: count %lu\n", engine->name, count);
> - igt_spin_free(i915, spin);
> -}
> -
> -static void kill_children(int sig)
> -{
> - signal(sig, SIG_IGN);
> - kill(-getpgrp(), SIGINT);
> - signal(sig, SIG_DFL);
> -}
> -
> -static void parallel(int i915)
> -{
> - const struct intel_execution_engine2 *e;
> - struct drm_i915_gem_relocation_entry *reloc;
> - uint32_t common = gem_create(i915, 4096);
> - uint32_t batch = batch_create(i915);
> - unsigned long reloc_sz;
> -
> - reloc = parallel_relocs(32 * 1024, &reloc_sz);
> -
> - stop = 0;
> - __for_each_physical_engine(i915, e) {
> - igt_fork(child, 1)
> - parallel_child(i915, e, reloc, common);
> - }
> - sleep(2);
> -
> - if (gem_scheduler_has_preemption(i915)) {
> - uint32_t ctx = gem_context_clone_with_engines(i915, 0);
> -
> - __for_each_physical_engine(i915, e) {
> - struct drm_i915_gem_exec_object2 obj[2] = {
> - { .handle = common },
> - { .handle = batch },
> - };
> - struct drm_i915_gem_execbuffer2 execbuf = {
> - .buffers_ptr = to_user_pointer(obj),
> - .buffer_count = ARRAY_SIZE(obj),
> - .flags = e->flags,
> - .rsvd1 = ctx,
> - };
> - gem_execbuf(i915, &execbuf);
> - }
> -
> - gem_context_destroy(i915, ctx);
> - }
> - gem_sync(i915, batch);
> - gem_close(i915, batch);
> -
> - kill_children(SIGINT);
> - igt_waitchildren();
> -
> - gem_close(i915, common);
> - munmap(reloc, reloc_sz);
> -}
> -
> #define CONCURRENT 1024
>
> static uint64_t concurrent_relocs(int i915, int idx, int count)
> @@ -1552,16 +1343,6 @@ igt_main
> }
> }
>
> - igt_subtest_with_dynamic("basic-wide-active") {
> - __for_each_physical_engine(fd, e) {
> - igt_dynamic_f("%s", e->name)
> - wide_active(fd, e->flags);
> - }
> - }
> -
> - igt_subtest("basic-parallel")
> - parallel(fd);
> -
> igt_subtest("basic-concurrent0")
> concurrent(fd, 0);
> igt_subtest("basic-concurrent16")
> --
> 2.24.1
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 3/7] Revert "i915/gem_exec_reloc: Verify engine isolation"
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 3/7] Revert "i915/gem_exec_reloc: Verify engine isolation" Daniel Vetter
@ 2021-06-09 17:59 ` Jason Ekstrand
0 siblings, 0 replies; 20+ messages in thread
From: Jason Ekstrand @ 2021-06-09 17:59 UTC (permalink / raw)
To: Daniel Vetter; +Cc: IGT development, Dave Airlie, Daniel Vetter
On Tue, Jun 8, 2021 at 4:40 AM Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
>
> This reverts commit 9fe244cb751c9d3be0581a943bb9baa8651d8d29.
>
> This validates gpu relocations, which we're about to delete.
>
> Cc: Jason Ekstrand <jason@jlekstrand.net>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: Dave Airlie <airlied@redhat.com>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
> tests/i915/gem_exec_reloc.c | 73 -------------------------------------
> 1 file changed, 73 deletions(-)
>
> diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
> index efe6e2e02c52..3b200f557b2c 100644
> --- a/tests/i915/gem_exec_reloc.c
> +++ b/tests/i915/gem_exec_reloc.c
> @@ -471,72 +471,6 @@ static void active_spin(int fd, unsigned engine)
> igt_spin_free(fd, spin);
> }
>
> -static void others_spin(int i915, unsigned engine)
> -{
> - struct drm_i915_gem_relocation_entry reloc = {};
> - struct drm_i915_gem_exec_object2 obj = {
> - .relocs_ptr = to_user_pointer(&reloc),
> - .relocation_count = 1,
> - };
> - struct drm_i915_gem_execbuffer2 execbuf = {
> - .buffers_ptr = to_user_pointer(&obj),
> - .buffer_count = 1,
> - .flags = engine,
> - };
> - const struct intel_execution_engine2 *e;
> - igt_spin_t *spin = NULL;
> - uint64_t addr;
> - int fence;
> -
> - __for_each_physical_engine(i915, e) {
> - if (e->flags == engine)
> - continue;
> -
> - if (!spin) {
> - spin = igt_spin_new(i915,
> - .engine = e->flags,
> - .flags = IGT_SPIN_FENCE_OUT);
> - fence = dup(spin->out_fence);
> - } else {
> - int old_fence;
> -
> - spin->execbuf.flags &= ~I915_EXEC_RING_MASK;
> - spin->execbuf.flags |= e->flags;
> - gem_execbuf_wr(i915, &spin->execbuf);
> -
> - old_fence = fence;
> - fence = sync_fence_merge(old_fence,
> - spin->execbuf.rsvd2 >> 32);
> - close(spin->execbuf.rsvd2 >> 32);
> - close(old_fence);
> - }
> - }
> - igt_require(spin);
> -
> - /* All other engines are busy, let's relocate! */
> - obj.handle = batch_create(i915);
> - reloc.target_handle = obj.handle;
> - reloc.presumed_offset = -1;
> - reloc.offset = 64;
> - gem_execbuf(i915, &execbuf);
Does this really depend on async relocs? The spinners above ensure
that all the OTHER engines are busy and then we try to do something
with a relocation on a fresh BO here. That should be fine. We should
be able to place a relocation in a BO as long as that BO isn't busy.
Or am I missing something?
--Jason
> -
> - /* Verify the relocation took place */
> - gem_read(i915, obj.handle, 64, &addr, sizeof(addr));
> - igt_assert_eq_u64(addr, obj.offset);
> - gem_close(i915, obj.handle);
> -
> - /* Even if the spinner was harmed in the process */
> - igt_spin_end(spin);
> - igt_assert_eq(sync_fence_wait(fence, 200), 0);
> - igt_assert_neq(sync_fence_status(fence), 0);
> - if (sync_fence_status(fence) < 0)
> - igt_warn("Spinner was cancelled, %s\n",
> - strerror(-sync_fence_status(fence)));
> - close(fence);
> -
> - igt_spin_free(i915, spin);
> -}
> -
> static bool has_64b_reloc(int fd)
> {
> return intel_gen(intel_get_drm_devid(fd)) >= 8;
> @@ -1329,13 +1263,6 @@ igt_main
> }
> }
>
> - igt_subtest_with_dynamic("basic-spin-others") {
> - __for_each_physical_engine(fd, e) {
> - igt_dynamic_f("%s", e->name)
> - others_spin(fd, e->flags);
> - }
> - }
> -
> igt_subtest_with_dynamic("basic-many-active") {
> __for_each_physical_engine(fd, e) {
> igt_dynamic_f("%s", e->name)
> --
> 2.24.1
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 6/7] Revert "i915/gem_exec_reloc: Check that relocations do not block"
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 6/7] Revert "i915/gem_exec_reloc: Check that relocations do not block" Daniel Vetter
@ 2021-06-09 18:03 ` Jason Ekstrand
0 siblings, 0 replies; 20+ messages in thread
From: Jason Ekstrand @ 2021-06-09 18:03 UTC (permalink / raw)
To: Daniel Vetter; +Cc: IGT development, Dave Airlie, Daniel Vetter
Very much
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
On Tue, Jun 8, 2021 at 4:40 AM Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
>
> This reverts commit 5343ca6ad8fac39fe4d468f771af72c968404bea.
>
> This validates gpu relocations, which we're about to delete.
>
> Cc: Jason Ekstrand <jason@jlekstrand.net>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: Dave Airlie <airlied@redhat.com>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
> tests/i915/gem_exec_reloc.c | 52 -------------------------------------
> 1 file changed, 52 deletions(-)
>
> diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
> index cbd40962c610..5b15bcdc3a1f 100644
> --- a/tests/i915/gem_exec_reloc.c
> +++ b/tests/i915/gem_exec_reloc.c
> @@ -344,51 +344,6 @@ static void active(int fd, unsigned engine)
> gem_close(fd, obj[0].handle);
> }
>
> -static unsigned int offset_in_page(void *addr)
> -{
> - return (uintptr_t)addr & 4095;
> -}
> -
> -static void active_spin(int fd, unsigned engine)
> -{
> - const uint32_t bbe = MI_BATCH_BUFFER_END;
> - struct drm_i915_gem_relocation_entry reloc;
> - struct drm_i915_gem_exec_object2 obj[2];
> - struct drm_i915_gem_execbuffer2 execbuf;
> - igt_spin_t *spin;
> -
> - spin = igt_spin_new(fd,
> - .engine = engine,
> - .flags = IGT_SPIN_NO_PREEMPTION);
> -
> - memset(obj, 0, sizeof(obj));
> - obj[0] = spin->obj[IGT_SPIN_BATCH];
> - obj[0].relocs_ptr = to_user_pointer(&reloc);
> - obj[0].relocation_count = 1;
> - obj[1].handle = gem_create(fd, 4096);
> - gem_write(fd, obj[1].handle, 0, &bbe, sizeof(bbe));
> -
> - memset(&reloc, 0, sizeof(reloc));
> - reloc.presumed_offset = -1;
> - reloc.offset = offset_in_page(spin->condition);
> - reloc.target_handle = obj[0].handle;
> -
> - memset(&execbuf, 0, sizeof(execbuf));
> - execbuf.buffers_ptr = to_user_pointer(obj);
> - execbuf.buffer_count = 2;
> - execbuf.flags = engine;
> -
> - gem_execbuf(fd, &execbuf);
> - gem_close(fd, obj[1].handle);
> - igt_assert_eq(*spin->condition, spin->cmd_precondition);
> -
> - igt_spin_end(spin);
> - gem_sync(fd, spin->handle);
> -
> - igt_assert_eq(*spin->condition, obj[0].offset);
> - igt_spin_free(fd, spin);
> -}
> -
> static bool has_64b_reloc(int fd)
> {
> return intel_gen(intel_get_drm_devid(fd)) >= 8;
> @@ -964,13 +919,6 @@ igt_main
> }
> }
>
> - igt_subtest_with_dynamic("basic-spin") {
> - __for_each_physical_engine(fd, e) {
> - igt_dynamic_f("%s", e->name)
> - active_spin(fd, e->flags);
> - }
> - }
> -
> igt_subtest("invalid-domains")
> invalid_domains(fd);
>
> --
> 2.24.1
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 5/7] Revert "i915/gem_exec_reloc: Flood the ring with GPU relocs"
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 5/7] Revert "i915/gem_exec_reloc: Flood the ring with GPU relocs" Daniel Vetter
@ 2021-06-09 18:05 ` Jason Ekstrand
0 siblings, 0 replies; 20+ messages in thread
From: Jason Ekstrand @ 2021-06-09 18:05 UTC (permalink / raw)
To: Daniel Vetter; +Cc: IGT development, Dave Airlie, Daniel Vetter
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
On Tue, Jun 8, 2021 at 4:40 AM Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
>
> This reverts commit 57ee41f12b7a53283fd812c5f72fcb39e6ad2197.
>
> This validates gpu relocations, which we're about to delete.
>
> Cc: Jason Ekstrand <jason@jlekstrand.net>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: Dave Airlie <airlied@redhat.com>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
> tests/i915/gem_exec_reloc.c | 89 -------------------------------------
> 1 file changed, 89 deletions(-)
>
> diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
> index c3f42aff9c9a..cbd40962c610 100644
> --- a/tests/i915/gem_exec_reloc.c
> +++ b/tests/i915/gem_exec_reloc.c
> @@ -344,88 +344,6 @@ static void active(int fd, unsigned engine)
> gem_close(fd, obj[0].handle);
> }
>
> -static uint64_t many_relocs(unsigned long count, unsigned long *out)
> -{
> - struct drm_i915_gem_relocation_entry *reloc;
> - unsigned long sz;
> - int i;
> -
> - sz = count * sizeof(*reloc);
> - sz = ALIGN(sz, 4096);
> -
> - reloc = mmap(0, sz, PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0);
> - igt_assert(reloc != MAP_FAILED);
> - for (i = 0; i < count; i++) {
> - reloc[i].target_handle = 0;
> - reloc[i].presumed_offset = ~0ull;
> - reloc[i].offset = 8 * i;
> - reloc[i].delta = 8 * i;
> - }
> - mprotect(reloc, sz, PROT_READ);
> -
> - *out = sz;
> - return to_user_pointer(reloc);
> -}
> -
> -static void __many_active(int i915, unsigned engine, unsigned long count)
> -{
> - unsigned long reloc_sz;
> - struct drm_i915_gem_exec_object2 obj[2] = {{
> - .handle = gem_create(i915, count * sizeof(uint64_t)),
> - .relocs_ptr = many_relocs(count, &reloc_sz),
> - .relocation_count = count,
> - }};
> - struct drm_i915_gem_execbuffer2 execbuf = {
> - .buffers_ptr = to_user_pointer(obj),
> - .buffer_count = ARRAY_SIZE(obj),
> - .flags = engine | I915_EXEC_HANDLE_LUT,
> - };
> - igt_spin_t *spin;
> -
> - spin = __igt_spin_new(i915,
> - .engine = engine,
> - .dependency = obj[0].handle,
> - .flags = (IGT_SPIN_FENCE_OUT |
> - IGT_SPIN_NO_PREEMPTION));
> - obj[1] = spin->obj[1];
> - gem_execbuf(i915, &execbuf);
> - igt_assert_eq(sync_fence_status(spin->out_fence), 0);
> - igt_spin_free(i915, spin);
> -
> - for (unsigned long i = 0; i < count; i++) {
> - uint64_t addr;
> -
> - gem_read(i915, obj[0].handle, i * sizeof(addr),
> - &addr, sizeof(addr));
> -
> - igt_assert_eq_u64(addr, obj[0].offset + i * sizeof(addr));
> - }
> -
> - munmap(from_user_pointer(obj[0].relocs_ptr), reloc_sz);
> - gem_close(i915, obj[0].handle);
> -}
> -
> -static void many_active(int i915, unsigned engine)
> -{
> - const uint64_t max = 2048;
> - unsigned long count = 256;
> -
> - igt_until_timeout(2) {
> - uint64_t required, total;
> -
> - if (!__intel_check_memory(1, 8 * count, CHECK_RAM,
> - &required, &total))
> - break;
> -
> - igt_debug("Testing count:%lu\n", count);
> - __many_active(i915, engine, count);
> -
> - count <<= 1;
> - if (count >= max)
> - break;
> - }
> -}
> -
> static unsigned int offset_in_page(void *addr)
> {
> return (uintptr_t)addr & 4095;
> @@ -1053,13 +971,6 @@ igt_main
> }
> }
>
> - igt_subtest_with_dynamic("basic-many-active") {
> - __for_each_physical_engine(fd, e) {
> - igt_dynamic_f("%s", e->name)
> - many_active(fd, e->flags);
> - }
> - }
> -
> igt_subtest("invalid-domains")
> invalid_domains(fd);
>
> --
> 2.24.1
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 4/7] Revert "i915/gem_exec_reloc: Exercise concurrent relocations"
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 4/7] Revert "i915/gem_exec_reloc: Exercise concurrent relocations" Daniel Vetter
@ 2021-06-09 18:12 ` Jason Ekstrand
0 siblings, 0 replies; 20+ messages in thread
From: Jason Ekstrand @ 2021-06-09 18:12 UTC (permalink / raw)
To: Daniel Vetter; +Cc: IGT development, Dave Airlie, Daniel Vetter
On Tue, Jun 8, 2021 at 4:40 AM Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
>
> This reverts commit c1f30ee09ac2e7eb3e8e90245239731a169a6050.
>
> This validates gpu relocations, which we're about to delete.
>
> Cc: Jason Ekstrand <jason@jlekstrand.net>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: Dave Airlie <airlied@redhat.com>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
> tests/i915/gem_exec_reloc.c | 215 ------------------------------------
> 1 file changed, 215 deletions(-)
>
> diff --git a/tests/i915/gem_exec_reloc.c b/tests/i915/gem_exec_reloc.c
> index 3b200f557b2c..c3f42aff9c9a 100644
> --- a/tests/i915/gem_exec_reloc.c
> +++ b/tests/i915/gem_exec_reloc.c
> @@ -796,216 +796,6 @@ static void basic_softpin(int fd)
> gem_close(fd, obj[1].handle);
> }
>
> -#define CONCURRENT 1024
> -
> -static uint64_t concurrent_relocs(int i915, int idx, int count)
> -{
> - struct drm_i915_gem_relocation_entry *reloc;
> - const unsigned int gen = intel_gen(intel_get_drm_devid(i915));
> - unsigned long sz;
> - int offset;
> -
> - sz = count * sizeof(*reloc);
> - sz = ALIGN(sz, 4096);
> -
> - reloc = mmap(0, sz, PROT_WRITE, MAP_PRIVATE | MAP_ANON, -1, 0);
> - igt_assert(reloc != MAP_FAILED);
> -
> - offset = 1;
> - if (gen >= 4 && gen < 8)
> - offset += 1;
> -
> - for (int n = 0; n < count; n++) {
> - reloc[n].presumed_offset = ~0ull;
> - reloc[n].offset = (4 * n + offset) * sizeof(uint32_t);
> - reloc[n].delta = (count * idx + n) * sizeof(uint32_t);
> - }
> - mprotect(reloc, sz, PROT_READ);
> -
> - return to_user_pointer(reloc);
> -}
> -
> -static int flags_to_index(const struct intel_execution_engine2 *e)
> -{
> - return (e->flags & 63) | ((e->flags >> 13) & 3) << 4;
> -}
> -
> -static void xchg_u32(void *array, unsigned i, unsigned j)
> -{
> - uint32_t *u32 = array;
> - uint32_t tmp = u32[i];
> - u32[i] = u32[j];
> - u32[j] = tmp;
> -}
> -
> -static void concurrent_child(int i915,
> - const struct intel_execution_engine2 *e,
> - uint32_t *common, int num_common,
> - int in, int out)
> -{
> - const unsigned int gen = intel_gen(intel_get_drm_devid(i915));
> - int idx = flags_to_index(e);
> - uint64_t relocs = concurrent_relocs(i915, idx, CONCURRENT);
> - struct drm_i915_gem_exec_object2 obj[num_common + 2];
> - struct drm_i915_gem_execbuffer2 execbuf = {
> - .buffers_ptr = to_user_pointer(obj),
> - .buffer_count = ARRAY_SIZE(obj),
> - .flags = e->flags | I915_EXEC_HANDLE_LUT | (gen < 6 ? I915_EXEC_SECURE : 0),
> - };
> - uint32_t *batch = &obj[num_common + 1].handle;
> - unsigned long count = 0;
> - uint32_t *x;
> - int err = 0;
> -
> - memset(obj, 0, sizeof(obj));
> - obj[0].handle = gem_create(i915, 64 * CONCURRENT * 4);
> -
> - igt_permute_array(common, num_common, xchg_u32);
> - for (int n = 1; n <= num_common; n++) {
> - obj[n].handle = common[n - 1];
> - obj[n].relocation_count = CONCURRENT;
> - obj[n].relocs_ptr = relocs;
> - }
> -
> - obj[num_common + 1].relocation_count = CONCURRENT;
> - obj[num_common + 1].relocs_ptr = relocs;
> -
> - x = gem_mmap__device_coherent(i915, obj[0].handle,
> - 0, 64 * CONCURRENT * 4, PROT_READ);
> - x += idx * CONCURRENT;
> -
> - do {
> - read(in, batch, sizeof(*batch));
> - if (!*batch)
> - break;
> -
> - gem_execbuf(i915, &execbuf);
> - gem_sync(i915, *batch); /* write hazards lies */
> -
> - for (int n = 0; n < CONCURRENT; n++) {
> - if (x[n] != *batch) {
> - igt_warn("%s: Invalid store [bad reloc] found:%08x at index %d, expected %08x\n",
> - e->name, x[n], n, *batch);
> - err = -EINVAL;
> - break;
> - }
> - }
> -
> - write(out, &err, sizeof(err));
> - count++;
> - } while (err == 0);
> -
> - gem_close(i915, obj[0].handle);
> - igt_info("%s: completed %ld cycles\n", e->name, count);
> -}
> -
> -static uint32_t create_concurrent_batch(int i915, unsigned int count)
> -{
> - const unsigned int gen = intel_gen(intel_get_drm_devid(i915));
> - size_t sz = ALIGN(4 * (1 + 4 * count), 4096);
> - uint32_t handle = gem_create(i915, sz);
> - uint32_t *map, *cs;
> - uint32_t cmd;
> -
> - cmd = MI_STORE_DWORD_IMM;
> - if (gen < 6)
> - cmd |= 1 << 22;
> - if (gen < 4)
> - cmd--;
> -
> - cs = map = gem_mmap__device_coherent(i915, handle, 0, sz, PROT_WRITE);
> - for (int n = 0; n < count; n++) {
> - *cs++ = cmd;
> - *cs++ = 0;
> - if (gen >= 4) {
> - *cs++ = 0;
> - *cs++ = handle;
> - } else {
> - *cs++ = handle;
> - *cs++ = 0;
> - }
> - }
> - *cs++ = MI_BATCH_BUFFER_END;
> - munmap(map, sz);
> -
> - return handle;
> -}
> -
> -static void concurrent(int i915, int num_common)
> -{
> - const struct intel_execution_engine2 *e;
> - int in[2], out[2];
> - uint32_t common[16];
> - int result = -1;
> - uint32_t batch;
> - int nchild;
> -
> - /*
> - * Exercise a few clients all trying to submit the same batch
> - * buffer writing to different locations. This exercises that the
> - * relocation handling within the gem_execbuf() ioctl is atomic
> - * with respect to the batch -- that is this call to execbuf only
> - * uses the relocations as supplied with the ioctl and does not
> - * use any of the conflicting relocations from the concurrent
> - * submissions.
I'm less sure about this one. Is it really testing concurrent
relocations or just relocation atomicity? If the later, then that's
coverage we may want to keep. It's hard to tell, unfortunately. :-(
--Jason
> - */
> -
> - pipe(in);
> - pipe(out);
> -
> - for (int n = 0; n < num_common; n++)
> - common[n] = gem_create(i915, 4 * 4 * CONCURRENT);
> -
> - nchild = 0;
> - __for_each_physical_engine(i915, e) {
> - if (!gem_class_can_store_dword(i915, e->class))
> - continue;
> -
> - igt_fork(child, 1)
> - concurrent_child(i915, e,
> - common, num_common,
> - in[0], out[1]);
> -
> - if (++nchild == 64)
> - break;
> - }
> - close(in[0]);
> - close(out[1]);
> - igt_require(nchild > 1);
> -
> - igt_until_timeout(5) {
> - batch = create_concurrent_batch(i915, CONCURRENT);
> -
> - for (int n = 0; n < nchild; n++)
> - write(in[1], &batch, sizeof(batch));
> -
> - for (int n = 0; n < nchild; n++) {
> - result = -1;
> - read(out[0], &result, sizeof(result));
> - if (result < 0)
> - break;
> - }
> -
> - gem_close(i915, batch);
> - if (result < 0)
> - break;
> - }
> -
> - batch = 0;
> - for (int n = 0; n < nchild; n++)
> - write(in[1], &batch, sizeof(batch));
> -
> - close(in[1]);
> - close(out[0]);
> -
> - igt_waitchildren();
> -
> - for (int n = 0; n < num_common; n++)
> - gem_close(i915, common[n]);
> -
> - igt_assert_eq(result, 0);
> -}
> -
> static uint32_t
> pin_scanout(igt_display_t *dpy, igt_output_t *output, struct igt_fb *fb)
> {
> @@ -1270,11 +1060,6 @@ igt_main
> }
> }
>
> - igt_subtest("basic-concurrent0")
> - concurrent(fd, 0);
> - igt_subtest("basic-concurrent16")
> - concurrent(fd, 16);
> -
> igt_subtest("invalid-domains")
> invalid_domains(fd);
>
> --
> 2.24.1
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [igt-dev] [PATCH i-g-t 2/7] Revert "test/i915/gem_exec_reloc: Restore interclient testings"
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 2/7] Revert "test/i915/gem_exec_reloc: Restore interclient testings" Daniel Vetter
2021-06-09 17:56 ` Jason Ekstrand
@ 2021-06-14 10:40 ` Arkadiusz Hiler
1 sibling, 0 replies; 20+ messages in thread
From: Arkadiusz Hiler @ 2021-06-14 10:40 UTC (permalink / raw)
To: Daniel Vetter; +Cc: Petri Latvala, IGT development, Daniel Vetter, Dave Airlie
On Tue, Jun 08, 2021 at 11:40:15AM +0200, Daniel Vetter wrote:
> This reverts commit cf2f41b3d3dfabaf3a4837062f996f3491a350b1.
>
> And I mean revert with extreme prejudice because this was already
> thrown out in
>
> commit 39e9aa1032a4e60f776f34b3ccf4fb728abbfe5c
> Author: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Date: Mon Aug 10 11:00:34 2020 +0200
>
> tests/i915: Remove subtests that rely on async relocation behavior
>
> and Chris then decided to undo that without any acks from the people
> who've removed these tests. Or well anyone else. Commit fights in
> upstream repositories are not acceptable.
Acked-by: Arkadiusz Hiler <arek@hiler.eu>
on the whole series, but especially on this patch.
Thanks for catching it and calling it out.
The full review is up to people familiar with Intel's hardware.
--
Cheers,
Arek
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2021-06-14 11:19 UTC | newest]
Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-08 9:40 [igt-dev] [PATCH i-g-t 1/7] tests/gem_exec_reloc: Remove banned tests Daniel Vetter
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 2/7] Revert "test/i915/gem_exec_reloc: Restore interclient testings" Daniel Vetter
2021-06-09 17:56 ` Jason Ekstrand
2021-06-14 10:40 ` Arkadiusz Hiler
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 3/7] Revert "i915/gem_exec_reloc: Verify engine isolation" Daniel Vetter
2021-06-09 17:59 ` Jason Ekstrand
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 4/7] Revert "i915/gem_exec_reloc: Exercise concurrent relocations" Daniel Vetter
2021-06-09 18:12 ` Jason Ekstrand
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 5/7] Revert "i915/gem_exec_reloc: Flood the ring with GPU relocs" Daniel Vetter
2021-06-09 18:05 ` Jason Ekstrand
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 6/7] Revert "i915/gem_exec_reloc: Check that relocations do not block" Daniel Vetter
2021-06-09 18:03 ` Jason Ekstrand
2021-06-08 9:40 ` [igt-dev] [PATCH i-g-t 7/7] tests/gem_exec_schedule: Use store_dword_plug again Daniel Vetter
2021-06-08 11:22 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [i-g-t,1/7] tests/gem_exec_reloc: Remove banned tests Patchwork
2021-06-08 16:02 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
2021-06-08 19:25 ` [igt-dev] [PATCH i-g-t] " Daniel Vetter
2021-06-09 17:53 ` Jason Ekstrand
2021-06-08 19:44 ` [igt-dev] ✗ GitLab.Pipeline: warning for series starting with [i-g-t] tests/gem_exec_reloc: Remove banned tests (rev2) Patchwork
2021-06-08 20:08 ` [igt-dev] ✓ Fi.CI.BAT: success " Patchwork
2021-06-09 0:49 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
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.