* [igt-dev] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests
@ 2018-12-07 8:59 Tvrtko Ursulin
2018-12-07 8:59 ` [Intel-gfx] [RFT i-g-t 2/2] intel-ci: Unblack list the new tests?? Tvrtko Ursulin
` (6 more replies)
0 siblings, 7 replies; 13+ messages in thread
From: Tvrtko Ursulin @ 2018-12-07 8:59 UTC (permalink / raw)
To: igt-dev; +Cc: Intel-gfx, Tvrtko Ursulin
From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
...
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
---
tests/i915/gem_shrink.c | 213 ++++++++++++++++++++++++++++++++++++++++
1 file changed, 213 insertions(+)
diff --git a/tests/i915/gem_shrink.c b/tests/i915/gem_shrink.c
index c8e05814ee70..acc12efed15e 100644
--- a/tests/i915/gem_shrink.c
+++ b/tests/i915/gem_shrink.c
@@ -26,6 +26,9 @@
*
* Exercise the shrinker by overallocating GEM objects
*/
+#include <sys/types.h>
+#include <sys/stat.h>
+#include <fcntl.h>
#include "igt.h"
#include "igt_gt.h"
@@ -366,6 +369,210 @@ static void reclaim(unsigned engine, int timeout)
close(fd);
}
+static unsigned long get_meminfo(const char *info, const char *tag)
+{
+ const char *str;
+ unsigned long val;
+
+ str = strstr(info, tag);
+ if (str && sscanf(str + strlen(tag), " %lu", &val) == 1)
+ return val >> 10;
+
+ igt_warn("Unrecognised /proc/meminfo field: '%s'\n", tag);
+ return 0;
+}
+
+static unsigned long get_avail_ram_mb(void)
+{
+ int fd;
+ int ret;
+ char buf[4096];
+ unsigned long ram;
+
+ fd = open("/proc/meminfo", O_RDONLY);
+ igt_assert_fd(fd);
+
+ ret = read(fd, buf, sizeof(buf));
+ igt_assert(ret >= 0);
+
+ close(fd);
+
+ ram = get_meminfo(buf, "MemAvailable:");
+ ram += get_meminfo(buf, "Buffers:");
+ ram += get_meminfo(buf, "Cached:");
+ ram += get_meminfo(buf, "SwapCached:");
+
+ return ram;
+}
+
+struct test {
+#define TEST_BO (1)
+#define TEST_USERPTR (2)
+ unsigned int flags;
+ int fd;
+};
+
+static uint32_t __get_pages(int fd, unsigned long alloc)
+{
+ uint32_t handle = gem_create(fd, alloc);
+
+ gem_set_domain(fd, handle, I915_GEM_DOMAIN_GTT, 0);
+ gem_madvise(fd, handle, I915_MADV_DONTNEED);
+
+ return handle;
+}
+
+struct test_obj {
+ void *ptr;
+ uint32_t handle;
+};
+
+static void
+__get_userptr(int fd, struct test_obj *obj, unsigned long sz)
+{
+ struct local_i915_gem_userptr userptr = { };
+ void *ptr;
+
+ igt_assert_eq(sz & 4095, 0);
+
+ ptr = mmap(NULL, sz, PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, -1, 0);
+ assert(ptr != MAP_FAILED);
+
+ userptr.user_size = sz;
+ userptr.user_ptr = to_user_pointer(ptr);
+ do_ioctl(fd, LOCAL_IOCTL_I915_GEM_USERPTR, &userptr);
+
+ gem_set_domain(fd, userptr.handle, I915_GEM_DOMAIN_GTT, 0);
+
+ obj->ptr = ptr;
+ obj->handle = userptr.handle;
+}
+
+#define PAGE_SIZE 4096
+static void *mempressure(void *arg)
+{
+ struct test_obj *list = NULL;
+ struct test *test = arg;
+ const unsigned int sz_mb = 2;
+ const unsigned int sz = sz_mb << 20;
+ unsigned int n = 0, max = 0;
+ unsigned int blocks;
+
+ while (true) {
+ unsigned long ram_mb = get_avail_ram_mb();
+
+ if (!list) {
+ blocks = ram_mb / sz_mb;
+ list = calloc(blocks, sizeof(*list));
+ igt_assert(list);
+ } else if (ram_mb < 256) {
+ blocks = max + 1;
+ }
+
+ if (list[n].ptr || list[n].handle) {
+ if (test->flags & TEST_BO)
+ gem_close(test->fd, list[n].handle);
+ else
+ munmap(list[n].ptr, sz);
+ }
+
+ if (test->flags & TEST_BO) {
+ list[n].handle = __get_pages(test->fd, sz);
+ } else if (test->flags & TEST_USERPTR) {
+ __get_userptr(test->fd, &list[n], sz);
+ } else {
+ list[n].ptr = mmap(NULL, sz, PROT_WRITE,
+ MAP_PRIVATE | MAP_ANONYMOUS, -1, 0);
+ assert(list[n].ptr != MAP_FAILED);
+
+ madvise(list[n].ptr, sz, MADV_HUGEPAGE);
+
+ for (size_t page = 0; page < sz; page += PAGE_SIZE)
+ *(volatile uint32_t *)((unsigned char *)list[n].ptr + page) = 0;
+ }
+
+ if (n > max)
+ max = n;
+
+ n++;
+
+ if (n >= blocks)
+ n = 0;
+ }
+
+ return NULL;
+}
+
+static void oom_adjust(const char *score)
+{
+ int fd;
+
+ fd = open("/proc/self/oom_score_adj", O_WRONLY);
+ igt_assert_fd(fd);
+ igt_assert(write(fd, score, sizeof(score)) == sizeof(score));
+ close(fd);
+}
+
+static void trigger_oom(void)
+{
+ const char *cmd = "f";
+ int fd;
+
+ fd = open("/proc/sysrq-trigger", O_WRONLY);
+ igt_assert_fd(fd);
+ igt_assert(write(fd, cmd, sizeof(cmd)) == sizeof(cmd));
+ close(fd);
+}
+
+static void reclaim_oom(unsigned int flags)
+{
+ unsigned int count = 0;
+
+ igt_assert_eq(__builtin_popcount(flags), 1);
+
+ oom_adjust("-1000");
+
+ do {
+ struct igt_helper_process gem_child = { };
+ struct igt_helper_process mem_child = { };
+
+ igt_debug("Iteration %u...\n", ++count);
+
+ igt_fork_helper(&mem_child) {
+ struct test mem = { };
+
+ oom_adjust("500");
+ mempressure(&mem);
+ }
+
+ igt_fork_helper(&gem_child) {
+ struct test gem = { .flags = flags };
+
+ oom_adjust("500");
+
+ gem.fd = drm_open_driver(DRIVER_INTEL);
+ igt_require_gem(gem.fd);
+
+ mempressure(&gem);
+
+ close(gem.fd);
+ }
+
+ for (unsigned long ram_mb = 0;
+ (ram_mb = get_avail_ram_mb()) > 512;) {
+ igt_debug("[%u] %lu free mb\n", count, ram_mb);
+ sleep(1);
+ }
+
+ trigger_oom();
+
+ sleep(1);
+
+ igt_stop_helper(&mem_child);
+ igt_stop_helper(&gem_child);
+ } while (count < 5);
+}
+
igt_main
{
const struct test {
@@ -432,6 +639,12 @@ igt_main
igt_subtest("reclaim")
reclaim(I915_EXEC_DEFAULT, 2);
+ igt_subtest("two-reclaims-and-oom")
+ reclaim_oom(TEST_BO);
+
+ igt_subtest("two-reclaims-and-oom-userptr")
+ reclaim_oom(TEST_USERPTR);
+
for(const struct test *t = tests; t->name; t++) {
for(const struct mode *m = modes; m->suffix; m++) {
igt_subtest_f("%s%s", t->name, m->suffix)
--
2.19.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 13+ messages in thread* [Intel-gfx] [RFT i-g-t 2/2] intel-ci: Unblack list the new tests?? 2018-12-07 8:59 [igt-dev] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests Tvrtko Ursulin @ 2018-12-07 8:59 ` Tvrtko Ursulin 2018-12-07 9:24 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests Patchwork ` (5 subsequent siblings) 6 siblings, 0 replies; 13+ messages in thread From: Tvrtko Ursulin @ 2018-12-07 8:59 UTC (permalink / raw) To: igt-dev; +Cc: Intel-gfx From: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> --- tests/intel-ci/blacklist.txt | 1 + tests/intel-ci/fast-feedback.testlist | 2 ++ 2 files changed, 3 insertions(+) diff --git a/tests/intel-ci/blacklist.txt b/tests/intel-ci/blacklist.txt index 73d127603d28..8083efc407d4 100644 --- a/tests/intel-ci/blacklist.txt +++ b/tests/intel-ci/blacklist.txt @@ -60,6 +60,7 @@ igt@gem_ring_sync_copy(@.*)? igt@gem_ring_sync_loop(@.*)? igt@gem_seqno_wrap(@.*)? igt@gem_shrink@(?!reclaim$).* +igt@gem_shrink@(?!two-reclaims-and-oom).* igt@gem_softpin@.*(hang|S4).* igt@gem_spin_batch(@.*)? igt@gem_stolen@.*hibernate.* diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist index 6d42792c67f7..77b5ae15a64a 100644 --- a/tests/intel-ci/fast-feedback.testlist +++ b/tests/intel-ci/fast-feedback.testlist @@ -124,6 +124,8 @@ igt@gem_ringfill@basic-default igt@gem_ringfill@basic-default-interruptible igt@gem_ringfill@basic-default-forked igt@gem_ringfill@basic-default-fd +igt@gem_shrink@two-reclaims-and-oom +igt@gem_shrink@two-reclaims-and-oom-userptr igt@gem_sync@basic-all igt@gem_sync@basic-each igt@gem_sync@basic-many-each -- 2.19.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests 2018-12-07 8:59 [igt-dev] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests Tvrtko Ursulin 2018-12-07 8:59 ` [Intel-gfx] [RFT i-g-t 2/2] intel-ci: Unblack list the new tests?? Tvrtko Ursulin @ 2018-12-07 9:24 ` Patchwork 2018-12-07 10:14 ` [Intel-gfx] [RFT i-g-t 1/2] " Tvrtko Ursulin ` (4 subsequent siblings) 6 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2018-12-07 9:24 UTC (permalink / raw) To: Tvrtko Ursulin; +Cc: igt-dev == Series Details == Series: series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests URL : https://patchwork.freedesktop.org/series/53726/ State : failure == Summary == CI Bug Log - changes from CI_DRM_5280 -> IGTPW_2131 ==================================================== Summary ------- **FAILURE** Serious unknown changes coming with IGTPW_2131 absolutely need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in IGTPW_2131, please notify your bug team to allow them to document this new failure mode, which will reduce false positives in CI. External URL: https://patchwork.freedesktop.org/api/1.0/series/53726/revisions/1/mbox/ Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_2131: ### IGT changes ### #### Possible regressions #### * {igt@gem_shrink@two-reclaims-and-oom}: - fi-skl-guc: NOTRUN -> INCOMPLETE - fi-gdg-551: NOTRUN -> INCOMPLETE - fi-bwr-2160: NOTRUN -> INCOMPLETE - fi-blb-e6850: NOTRUN -> INCOMPLETE - fi-bsw-kefka: NOTRUN -> INCOMPLETE - fi-bsw-n3050: NOTRUN -> INCOMPLETE - fi-hsw-peppy: NOTRUN -> INCOMPLETE * {igt@gem_shrink@two-reclaims-and-oom-userptr}: - fi-glk-j4005: NOTRUN -> TIMEOUT - fi-bwr-2160: NOTRUN -> FAIL * igt@i915_selftest@live_sanitycheck: - fi-bxt-dsi: PASS -> DMESG-WARN * {igt@runner@aborted}: - fi-bxt-dsi: NOTRUN -> FAIL Known issues ------------ Here are the changes found in IGTPW_2131 that come from known issues: ### IGT changes ### #### Issues hit #### * {igt@gem_shrink@two-reclaims-and-oom}: - fi-byt-n2820: NOTRUN -> INCOMPLETE [fdo#102657] - fi-bxt-dsi: NOTRUN -> INCOMPLETE [fdo#103927] - fi-glk-j4005: NOTRUN -> INCOMPLETE [fdo#103359] / [k.org#198133] - fi-bdw-gvtdvm: NOTRUN -> INCOMPLETE [fdo#105600] - fi-skl-gvtdvm: NOTRUN -> INCOMPLETE [fdo#105600] - fi-elk-e7500: NOTRUN -> INCOMPLETE [fdo#103989] * igt@kms_flip@basic-flip-vs-wf_vblank: - fi-bsw-n3050: PASS -> FAIL [fdo#100368] #### Possible fixes #### * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-b: - fi-blb-e6850: INCOMPLETE [fdo#107718] -> PASS {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#100368]: https://bugs.freedesktop.org/show_bug.cgi?id=100368 [fdo#102657]: https://bugs.freedesktop.org/show_bug.cgi?id=102657 [fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#103989]: https://bugs.freedesktop.org/show_bug.cgi?id=103989 [fdo#105600]: https://bugs.freedesktop.org/show_bug.cgi?id=105600 [fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718 [k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133 Participating hosts (45 -> 43) ------------------------------ Additional (2): fi-skl-guc fi-skl-6600u Missing (4): fi-ilk-m540 fi-byt-squawks fi-bsw-cyan fi-hsw-4200u Build changes ------------- * IGT: IGT_4743 -> IGTPW_2131 CI_DRM_5280: 6047933c2fafdfd42353d735b213e74826d5a939 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_2131: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2131/ IGT_4743: edb2db2cf2b6665d7ba3fa9117263302f6307a4f @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools == Testlist changes == +igt@gem_shrink@two-reclaims-and-oom +igt@gem_shrink@two-reclaims-and-oom-userptr == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2131/ _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [Intel-gfx] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests 2018-12-07 8:59 [igt-dev] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests Tvrtko Ursulin 2018-12-07 8:59 ` [Intel-gfx] [RFT i-g-t 2/2] intel-ci: Unblack list the new tests?? Tvrtko Ursulin 2018-12-07 9:24 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests Patchwork @ 2018-12-07 10:14 ` Tvrtko Ursulin 2018-12-07 10:17 ` [igt-dev] " Chris Wilson 2018-12-07 14:04 ` Tvrtko Ursulin 2018-12-07 10:41 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev2) Patchwork ` (3 subsequent siblings) 6 siblings, 2 replies; 13+ messages in thread From: Tvrtko Ursulin @ 2018-12-07 10:14 UTC (permalink / raw) To: igt-dev; +Cc: Intel-gfx From: Tvrtko Ursulin <tvrtko.ursulin@intel.com> ... Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> --- lib/igt_core.c | 18 ++++ lib/igt_core.h | 1 + tests/i915/gem_shrink.c | 213 ++++++++++++++++++++++++++++++++++++++++ 3 files changed, 232 insertions(+) diff --git a/lib/igt_core.c b/lib/igt_core.c index 64883d6402af..d8fa0c83e279 100644 --- a/lib/igt_core.c +++ b/lib/igt_core.c @@ -1680,6 +1680,24 @@ void igt_stop_helper(struct igt_helper_process *proc) assert(helper_was_alive(proc, status)); } +/** + * igt_try_stop_helper: + * @proc: #igt_helper_process structure + * + * Terminates a helper process if it is still running. + */ +void igt_try_stop_helper(struct igt_helper_process *proc) +{ + int status; + + /* failure here means the pid is already dead and so waiting is safe */ + kill(proc->pid, proc->use_SIGKILL ? SIGKILL : SIGTERM); + + status = igt_wait_helper(proc); + if (!helper_was_alive(proc, status)) + igt_debug("Helper died too early with status=%d\n", status); +} + static void children_exit_handler(int sig) { int status; diff --git a/lib/igt_core.h b/lib/igt_core.h index 6f8c3852a686..beec34667524 100644 --- a/lib/igt_core.h +++ b/lib/igt_core.h @@ -795,6 +795,7 @@ bool __igt_fork_helper(struct igt_helper_process *proc); for (; __igt_fork_helper(proc); exit(0)) int igt_wait_helper(struct igt_helper_process *proc); void igt_stop_helper(struct igt_helper_process *proc); +void igt_try_stop_helper(struct igt_helper_process *proc); /* exit handler code */ diff --git a/tests/i915/gem_shrink.c b/tests/i915/gem_shrink.c index c8e05814ee70..0071c1ae21ff 100644 --- a/tests/i915/gem_shrink.c +++ b/tests/i915/gem_shrink.c @@ -26,6 +26,9 @@ * * Exercise the shrinker by overallocating GEM objects */ +#include <sys/types.h> +#include <sys/stat.h> +#include <fcntl.h> #include "igt.h" #include "igt_gt.h" @@ -366,6 +369,210 @@ static void reclaim(unsigned engine, int timeout) close(fd); } +static unsigned long get_meminfo(const char *info, const char *tag) +{ + const char *str; + unsigned long val; + + str = strstr(info, tag); + if (str && sscanf(str + strlen(tag), " %lu", &val) == 1) + return val >> 10; + + igt_warn("Unrecognised /proc/meminfo field: '%s'\n", tag); + return 0; +} + +static unsigned long get_avail_ram_mb(void) +{ + int fd; + int ret; + char buf[4096]; + unsigned long ram; + + fd = open("/proc/meminfo", O_RDONLY); + igt_assert_fd(fd); + + ret = read(fd, buf, sizeof(buf)); + igt_assert(ret >= 0); + + close(fd); + + ram = get_meminfo(buf, "MemAvailable:"); + ram += get_meminfo(buf, "Buffers:"); + ram += get_meminfo(buf, "Cached:"); + ram += get_meminfo(buf, "SwapCached:"); + + return ram; +} + +struct test { +#define TEST_BO (1) +#define TEST_USERPTR (2) + unsigned int flags; + int fd; +}; + +static uint32_t __get_pages(int fd, unsigned long alloc) +{ + uint32_t handle = gem_create(fd, alloc); + + gem_set_domain(fd, handle, I915_GEM_DOMAIN_GTT, 0); + gem_madvise(fd, handle, I915_MADV_DONTNEED); + + return handle; +} + +struct test_obj { + void *ptr; + uint32_t handle; +}; + +static void +__get_userptr(int fd, struct test_obj *obj, unsigned long sz) +{ + struct local_i915_gem_userptr userptr = { }; + void *ptr; + + igt_assert_eq(sz & 4095, 0); + + ptr = mmap(NULL, sz, PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); + assert(ptr != MAP_FAILED); + + userptr.user_size = sz; + userptr.user_ptr = to_user_pointer(ptr); + do_ioctl(fd, LOCAL_IOCTL_I915_GEM_USERPTR, &userptr); + + gem_set_domain(fd, userptr.handle, I915_GEM_DOMAIN_GTT, 0); + + obj->ptr = ptr; + obj->handle = userptr.handle; +} + +#define PAGE_SIZE 4096 +static void *mempressure(void *arg) +{ + struct test_obj *list = NULL; + struct test *test = arg; + const unsigned int sz_mb = 2; + const unsigned int sz = sz_mb << 20; + unsigned int n = 0, max = 0; + unsigned int blocks; + + while (true) { + unsigned long ram_mb = get_avail_ram_mb(); + + if (!list) { + blocks = ram_mb / sz_mb; + list = calloc(blocks, sizeof(*list)); + igt_assert(list); + } else if (ram_mb < 256) { + blocks = max + 1; + } + + if (list[n].ptr || list[n].handle) { + if (test->flags & TEST_BO) + gem_close(test->fd, list[n].handle); + else + munmap(list[n].ptr, sz); + } + + if (test->flags & TEST_BO) { + list[n].handle = __get_pages(test->fd, sz); + } else if (test->flags & TEST_USERPTR) { + __get_userptr(test->fd, &list[n], sz); + } else { + list[n].ptr = mmap(NULL, sz, PROT_WRITE, + MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); + assert(list[n].ptr != MAP_FAILED); + + madvise(list[n].ptr, sz, MADV_HUGEPAGE); + + for (size_t page = 0; page < sz; page += PAGE_SIZE) + *(volatile uint32_t *)((unsigned char *)list[n].ptr + page) = 0; + } + + if (n > max) + max = n; + + n++; + + if (n >= blocks) + n = 0; + } + + return NULL; +} + +static void oom_adjust(const char *score) +{ + int fd; + + fd = open("/proc/self/oom_score_adj", O_WRONLY); + igt_assert_fd(fd); + igt_assert(write(fd, score, sizeof(score)) == sizeof(score)); + close(fd); +} + +static void trigger_oom(void) +{ + const char *cmd = "f"; + int fd; + + fd = open("/proc/sysrq-trigger", O_WRONLY); + igt_assert_fd(fd); + igt_assert(write(fd, cmd, sizeof(cmd)) == sizeof(cmd)); + close(fd); +} + +static void reclaim_oom(unsigned int flags) +{ + unsigned int count = 0; + + igt_assert_eq(__builtin_popcount(flags), 1); + + oom_adjust("-1000"); + + do { + struct igt_helper_process gem_child = { .use_SIGKILL = true }; + struct igt_helper_process mem_child = { .use_SIGKILL = true }; + + igt_debug("Iteration %u...\n", ++count); + + igt_fork_helper(&mem_child) { + struct test mem = { }; + + oom_adjust("500"); + mempressure(&mem); + } + + igt_fork_helper(&gem_child) { + struct test gem = { .flags = flags }; + + oom_adjust("500"); + + gem.fd = drm_open_driver(DRIVER_INTEL); + igt_require_gem(gem.fd); + + mempressure(&gem); + + close(gem.fd); + } + + for (unsigned long ram_mb = 0; + (ram_mb = get_avail_ram_mb()) > 512;) { + igt_debug("[%u] %lu free mb\n", count, ram_mb); + sleep(1); + } + + trigger_oom(); + + sleep(1); + + igt_try_stop_helper(&mem_child); + igt_try_stop_helper(&gem_child); + } while (count < 5); +} + igt_main { const struct test { @@ -432,6 +639,12 @@ igt_main igt_subtest("reclaim") reclaim(I915_EXEC_DEFAULT, 2); + igt_subtest("two-reclaims-and-oom") + reclaim_oom(TEST_BO); + + igt_subtest("two-reclaims-and-oom-userptr") + reclaim_oom(TEST_USERPTR); + for(const struct test *t = tests; t->name; t++) { for(const struct mode *m = modes; m->suffix; m++) { igt_subtest_f("%s%s", t->name, m->suffix) -- 2.19.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [igt-dev] [Intel-gfx] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests 2018-12-07 10:14 ` [Intel-gfx] [RFT i-g-t 1/2] " Tvrtko Ursulin @ 2018-12-07 10:17 ` Chris Wilson 2018-12-07 10:28 ` Tvrtko Ursulin 2018-12-07 14:04 ` Tvrtko Ursulin 1 sibling, 1 reply; 13+ messages in thread From: Chris Wilson @ 2018-12-07 10:17 UTC (permalink / raw) To: Tvrtko Ursulin, igt-dev; +Cc: Intel-gfx Quoting Tvrtko Ursulin (2018-12-07 10:14:53) > +static void trigger_oom(void) > +{ > + const char *cmd = "f"; > + int fd; > + > + fd = open("/proc/sysrq-trigger", O_WRONLY); > + igt_assert_fd(fd); /proc/sysrq-trigger may not exist. subgroup fixture igt_require? -Chris _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [igt-dev] [Intel-gfx] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests 2018-12-07 10:17 ` [igt-dev] " Chris Wilson @ 2018-12-07 10:28 ` Tvrtko Ursulin 0 siblings, 0 replies; 13+ messages in thread From: Tvrtko Ursulin @ 2018-12-07 10:28 UTC (permalink / raw) To: Chris Wilson, igt-dev; +Cc: Intel-gfx On 07/12/2018 10:17, Chris Wilson wrote: > Quoting Tvrtko Ursulin (2018-12-07 10:14:53) >> +static void trigger_oom(void) >> +{ >> + const char *cmd = "f"; >> + int fd; >> + >> + fd = open("/proc/sysrq-trigger", O_WRONLY); >> + igt_assert_fd(fd); > > /proc/sysrq-trigger may not exist. subgroup fixture igt_require? I guess so. Marking for TODO. First I need to re-visit whether my test is actually exercising all three reclaim path (or better say shrink_lock paths), since afterwards I figured out there is some "misleadingness" there. Regards, Tvrtko _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [Intel-gfx] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests 2018-12-07 10:14 ` [Intel-gfx] [RFT i-g-t 1/2] " Tvrtko Ursulin 2018-12-07 10:17 ` [igt-dev] " Chris Wilson @ 2018-12-07 14:04 ` Tvrtko Ursulin 2018-12-07 14:06 ` [igt-dev] " Chris Wilson 1 sibling, 1 reply; 13+ messages in thread From: Tvrtko Ursulin @ 2018-12-07 14:04 UTC (permalink / raw) To: igt-dev; +Cc: Intel-gfx From: Tvrtko Ursulin <tvrtko.ursulin@intel.com> ... Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> --- lib/igt_core.c | 18 +++ lib/igt_core.h | 1 + tests/i915/gem_shrink.c | 299 ++++++++++++++++++++++++++++++++++++++++ 3 files changed, 318 insertions(+) diff --git a/lib/igt_core.c b/lib/igt_core.c index 64883d6402af..d8fa0c83e279 100644 --- a/lib/igt_core.c +++ b/lib/igt_core.c @@ -1680,6 +1680,24 @@ void igt_stop_helper(struct igt_helper_process *proc) assert(helper_was_alive(proc, status)); } +/** + * igt_try_stop_helper: + * @proc: #igt_helper_process structure + * + * Terminates a helper process if it is still running. + */ +void igt_try_stop_helper(struct igt_helper_process *proc) +{ + int status; + + /* failure here means the pid is already dead and so waiting is safe */ + kill(proc->pid, proc->use_SIGKILL ? SIGKILL : SIGTERM); + + status = igt_wait_helper(proc); + if (!helper_was_alive(proc, status)) + igt_debug("Helper died too early with status=%d\n", status); +} + static void children_exit_handler(int sig) { int status; diff --git a/lib/igt_core.h b/lib/igt_core.h index 6f8c3852a686..beec34667524 100644 --- a/lib/igt_core.h +++ b/lib/igt_core.h @@ -795,6 +795,7 @@ bool __igt_fork_helper(struct igt_helper_process *proc); for (; __igt_fork_helper(proc); exit(0)) int igt_wait_helper(struct igt_helper_process *proc); void igt_stop_helper(struct igt_helper_process *proc); +void igt_try_stop_helper(struct igt_helper_process *proc); /* exit handler code */ diff --git a/tests/i915/gem_shrink.c b/tests/i915/gem_shrink.c index c8e05814ee70..2c8e8f9453d2 100644 --- a/tests/i915/gem_shrink.c +++ b/tests/i915/gem_shrink.c @@ -26,6 +26,9 @@ * * Exercise the shrinker by overallocating GEM objects */ +#include <sys/types.h> +#include <sys/stat.h> +#include <fcntl.h> #include "igt.h" #include "igt_gt.h" @@ -366,6 +369,287 @@ static void reclaim(unsigned engine, int timeout) close(fd); } +static unsigned long get_meminfo(const char *info, const char *tag) +{ + const char *str; + unsigned long val; + + str = strstr(info, tag); + if (str && sscanf(str + strlen(tag), " %lu", &val) == 1) + return val >> 10; + + igt_warn("Unrecognised /proc/meminfo field: '%s'\n", tag); + return 0; +} + +static unsigned long get_avail_ram_mb(void) +{ + int fd; + int ret; + char buf[4096]; + unsigned long ram; + + fd = open("/proc/meminfo", O_RDONLY); + igt_assert_fd(fd); + + ret = read(fd, buf, sizeof(buf)); + igt_assert(ret >= 0); + + close(fd); + + ram = get_meminfo(buf, "MemAvailable:"); + ram += get_meminfo(buf, "Buffers:"); + ram += get_meminfo(buf, "Cached:"); + ram += get_meminfo(buf, "SwapCached:"); + + return ram; +} + +struct test { +#define TEST_BO (1) +#define TEST_USERPTR (2) + unsigned int flags; + int fd; +}; + +static uint32_t __get_pages(int fd, unsigned long alloc) +{ + uint32_t handle = gem_create(fd, alloc); + + gem_set_domain(fd, handle, I915_GEM_DOMAIN_GTT, 0); + gem_madvise(fd, handle, I915_MADV_DONTNEED); + + return handle; +} + +struct test_obj { + void *ptr; + uint32_t handle; +}; + +static void +__get_userptr(int fd, struct test_obj *obj, unsigned long sz) +{ + struct local_i915_gem_userptr userptr = { }; + void *ptr; + + igt_assert_eq(sz & 4095, 0); + + ptr = mmap(NULL, sz, PROT_WRITE, MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); + assert(ptr != MAP_FAILED); + + userptr.user_size = sz; + userptr.user_ptr = to_user_pointer(ptr); + do_ioctl(fd, LOCAL_IOCTL_I915_GEM_USERPTR, &userptr); + + gem_set_domain(fd, userptr.handle, I915_GEM_DOMAIN_GTT, 0); + gem_madvise(fd, userptr.handle, I915_MADV_DONTNEED); + + obj->ptr = ptr; + obj->handle = userptr.handle; +} + +#define PAGE_SIZE 4096 +static void *mempressure(void *arg) +{ + struct test_obj *list = NULL; + struct test *test = arg; + const unsigned int sz_mb = 2; + const unsigned int sz = sz_mb << 20; + unsigned int n = 0, max = 0; + unsigned int blocks; + + while (true) { + unsigned long ram_mb = get_avail_ram_mb(); + + if (!list) { + blocks = ram_mb / sz_mb; + list = calloc(blocks, sizeof(*list)); + igt_assert(list); + } else if (ram_mb < 256) { + blocks = max + 1; + } + + if (list[n].ptr || list[n].handle) { + if (test->flags & TEST_USERPTR) { + munmap(list[n].ptr, sz); + gem_close(test->fd, list[n].handle); + } else if (test->flags & TEST_BO) { + gem_close(test->fd, list[n].handle); + } else { + munmap(list[n].ptr, sz); + } + } + + if (test->flags & TEST_BO) { + list[n].handle = __get_pages(test->fd, sz); + } else if (test->flags & TEST_USERPTR) { + __get_userptr(test->fd, &list[n], sz); + } else { + list[n].ptr = mmap(NULL, sz, PROT_WRITE, + MAP_PRIVATE | MAP_ANONYMOUS, -1, 0); + assert(list[n].ptr != MAP_FAILED); + + madvise(list[n].ptr, sz, MADV_HUGEPAGE); + + for (size_t page = 0; page < sz; page += PAGE_SIZE) + *(volatile uint32_t *)((unsigned char *)list[n].ptr + page) = 0; + } + + if (n > max) + max = n; + + n++; + + if (n >= blocks) + n = 0; + } + + return NULL; +} + +static void oom_adjust(const char *score) +{ + int fd; + + fd = open("/proc/self/oom_score_adj", O_WRONLY); + igt_assert_fd(fd); + igt_assert(write(fd, score, sizeof(score)) == sizeof(score)); + close(fd); +} + +static void trigger_oom(void) +{ + const char *cmd = "f"; + int fd; + + fd = open("/proc/sysrq-trigger", O_WRONLY); + igt_assert_fd(fd); + igt_assert(write(fd, cmd, sizeof(cmd)) == sizeof(cmd)); + close(fd); +} + +static bool has_sysrq_trigger(void) +{ + int fd; + + fd = open("/proc/sysrq-trigger", O_WRONLY); + close(fd); + + return fd >= 0; +} + +static void reclaim_oom(unsigned int flags) +{ + unsigned int count = 0; + + igt_assert_eq(__builtin_popcount(flags), 1); + + oom_adjust("-1000"); + + do { + struct igt_helper_process gem_child = { .use_SIGKILL = true }; + struct igt_helper_process mem_child = { .use_SIGKILL = true }; + struct igt_helper_process eb_child = { .use_SIGKILL = true }; + struct igt_helper_process drop_child = { .use_SIGKILL = true }; + + igt_debug("Iteration %u...\n", ++count); + + igt_fork_helper(&mem_child) { + struct test test = { }; + + oom_adjust("500"); + mempressure(&test); + } + + igt_fork_helper(&gem_child) { + struct test test = { .flags = flags }; + + oom_adjust("500"); + + test.fd = drm_open_driver_render(DRIVER_INTEL); + igt_require_gem(test.fd); + + mempressure(&test); + + close(test.fd); + } + + igt_fork_helper(&eb_child) { + struct test test = { .flags = flags }; + const uint32_t bbe = MI_BATCH_BUFFER_END; + struct drm_i915_gem_exec_object2 obj = { }; + struct drm_i915_gem_execbuffer2 execbuf = { }; + + oom_adjust("500"); + + execbuf.buffers_ptr = to_user_pointer(&obj); + execbuf.buffer_count = 1; + + test.fd = drm_open_driver_render(DRIVER_INTEL); + igt_require_gem(test.fd); + + for (;;) { + unsigned long eb = 0; + struct timespec ts = { }; + unsigned long sec, last = 0; + + igt_nsec_elapsed(&ts); + + for (;;) { + obj.handle = gem_create(test.fd, 4096); + gem_write(test.fd, obj.handle, 0, &bbe, + sizeof(bbe)); + gem_execbuf(test.fd, &execbuf); + eb++; + sec = igt_nsec_elapsed(&ts) / 1e9; + if (sec > last) + gem_sync(test.fd, obj.handle); + gem_close(test.fd, obj.handle); + if ((last - sec) > 1) + break; + last = sec; + } + + igt_debug("%lu eb\n", eb); + usleep(500e3); + } + + close(test.fd); + } + + igt_fork_helper(&drop_child) { + int fd; + + fd = drm_open_driver(DRIVER_INTEL); + igt_require_gem(fd); + + for (;;) { + usleep(334e3); + igt_drop_caches_set(fd, DROP_ACTIVE); + } + + close(fd); + } + + for (unsigned long ram_mb = 0; + (ram_mb = get_avail_ram_mb()) > 512;) { + igt_debug("[%u] %lu free mb\n", count, ram_mb); + sleep(1); + } + + igt_debug("Triggering OOM\n"); + trigger_oom(); + + sleep(1); + + igt_try_stop_helper(&mem_child); + igt_try_stop_helper(&gem_child); + igt_try_stop_helper(&eb_child); + igt_try_stop_helper(&drop_child); + } while (count < 3); +} + igt_main { const struct test { @@ -432,6 +716,21 @@ igt_main igt_subtest("reclaim") reclaim(I915_EXEC_DEFAULT, 2); + igt_subtest_group { + igt_fixture { + igt_require(has_sysrq_trigger()); + } + + igt_subtest("two-reclaims-and-oom") + reclaim_oom(TEST_BO); + + igt_subtest("two-reclaims-and-oom-userptr") { + igt_require(has_userptr()); + + reclaim_oom(TEST_USERPTR); + } + } + for(const struct test *t = tests; t->name; t++) { for(const struct mode *m = modes; m->suffix; m++) { igt_subtest_f("%s%s", t->name, m->suffix) -- 2.19.1 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [igt-dev] [Intel-gfx] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests 2018-12-07 14:04 ` Tvrtko Ursulin @ 2018-12-07 14:06 ` Chris Wilson 2018-12-07 14:13 ` Tvrtko Ursulin 0 siblings, 1 reply; 13+ messages in thread From: Chris Wilson @ 2018-12-07 14:06 UTC (permalink / raw) To: Tvrtko Ursulin, igt-dev; +Cc: Intel-gfx Quoting Tvrtko Ursulin (2018-12-07 14:04:05) > From: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > > ... > > Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > --- > lib/igt_core.c | 18 +++ > lib/igt_core.h | 1 + > tests/i915/gem_shrink.c | 299 ++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 318 insertions(+) > > diff --git a/lib/igt_core.c b/lib/igt_core.c > index 64883d6402af..d8fa0c83e279 100644 > --- a/lib/igt_core.c > +++ b/lib/igt_core.c > @@ -1680,6 +1680,24 @@ void igt_stop_helper(struct igt_helper_process *proc) > assert(helper_was_alive(proc, status)); > } > > +/** > + * igt_try_stop_helper: > + * @proc: #igt_helper_process structure > + * > + * Terminates a helper process if it is still running. > + */ > +void igt_try_stop_helper(struct igt_helper_process *proc) General thoughtless comment about try_func is that usually report a bool. -Chris _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [Intel-gfx] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests 2018-12-07 14:06 ` [igt-dev] " Chris Wilson @ 2018-12-07 14:13 ` Tvrtko Ursulin 0 siblings, 0 replies; 13+ messages in thread From: Tvrtko Ursulin @ 2018-12-07 14:13 UTC (permalink / raw) To: Chris Wilson, igt-dev; +Cc: Intel-gfx On 07/12/2018 14:06, Chris Wilson wrote: > Quoting Tvrtko Ursulin (2018-12-07 14:04:05) >> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com> >> >> ... >> >> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> >> --- >> lib/igt_core.c | 18 +++ >> lib/igt_core.h | 1 + >> tests/i915/gem_shrink.c | 299 ++++++++++++++++++++++++++++++++++++++++ >> 3 files changed, 318 insertions(+) >> >> diff --git a/lib/igt_core.c b/lib/igt_core.c >> index 64883d6402af..d8fa0c83e279 100644 >> --- a/lib/igt_core.c >> +++ b/lib/igt_core.c >> @@ -1680,6 +1680,24 @@ void igt_stop_helper(struct igt_helper_process *proc) >> assert(helper_was_alive(proc, status)); >> } >> >> +/** >> + * igt_try_stop_helper: >> + * @proc: #igt_helper_process structure >> + * >> + * Terminates a helper process if it is still running. >> + */ >> +void igt_try_stop_helper(struct igt_helper_process *proc) > > General thoughtless comment about try_func is that usually report a > bool. Okay, another TODO item. First I wanted to call them __igt_stop_helper which would have avoided it. :) But in general no need to pay too much attention for now, I am only using it for CI access. Hitting the nested lock path is proving to be tricky, locally it only manages a handful of times per run. And since it is nested we cannot count on lockdep to stop things. Userptr half also regressed in so it's not triggering much shrinking any longer. :I So perhaps strength in (CI) numbers shows something new. Or not.. Regards, Tvrtko _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev2) 2018-12-07 8:59 [igt-dev] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests Tvrtko Ursulin ` (2 preceding siblings ...) 2018-12-07 10:14 ` [Intel-gfx] [RFT i-g-t 1/2] " Tvrtko Ursulin @ 2018-12-07 10:41 ` Patchwork 2018-12-07 15:25 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev3) Patchwork ` (2 subsequent siblings) 6 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2018-12-07 10:41 UTC (permalink / raw) To: Tvrtko Ursulin; +Cc: igt-dev == Series Details == Series: series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev2) URL : https://patchwork.freedesktop.org/series/53726/ State : success == Summary == CI Bug Log - changes from CI_DRM_5281 -> IGTPW_2134 ==================================================== Summary ------- **SUCCESS** No regressions found. External URL: https://patchwork.freedesktop.org/api/1.0/series/53726/revisions/2/mbox/ Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_2134: ### IGT changes ### #### Possible regressions #### * {igt@gem_shrink@two-reclaims-and-oom-userptr}: - fi-glk-j4005: NOTRUN -> TIMEOUT - fi-byt-n2820: NOTRUN -> TIMEOUT - fi-bwr-2160: NOTRUN -> FAIL Known issues ------------ Here are the changes found in IGTPW_2134 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_ctx_create@basic-files: - fi-bsw-n3050: PASS -> FAIL [fdo#108656] - fi-bsw-kefka: PASS -> DMESG-FAIL [fdo#108656] * igt@gem_exec_suspend@basic-s3: - fi-blb-e6850: PASS -> INCOMPLETE [fdo#107718] * igt@gem_exec_suspend@basic-s4-devices: - fi-ivb-3520m: PASS -> FAIL [fdo#108880] * {igt@runner@aborted}: - fi-bsw-kefka: NOTRUN -> FAIL [fdo#108656] #### Possible fixes #### * igt@i915_selftest@live_execlists: - fi-apl-guc: INCOMPLETE [fdo#103927] -> PASS * igt@i915_selftest@live_hangcheck: - fi-bwr-2160: DMESG-FAIL [fdo#108735] -> PASS - fi-skl-gvtdvm: INCOMPLETE [fdo#105600] / [fdo#108744] -> PASS {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#105600]: https://bugs.freedesktop.org/show_bug.cgi?id=105600 [fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718 [fdo#108656]: https://bugs.freedesktop.org/show_bug.cgi?id=108656 [fdo#108735]: https://bugs.freedesktop.org/show_bug.cgi?id=108735 [fdo#108744]: https://bugs.freedesktop.org/show_bug.cgi?id=108744 [fdo#108880]: https://bugs.freedesktop.org/show_bug.cgi?id=108880 Participating hosts (50 -> 41) ------------------------------ Missing (9): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-gdg-551 fi-pnv-d510 fi-icl-y Build changes ------------- * IGT: IGT_4743 -> IGTPW_2134 CI_DRM_5281: 678f4e6d807098fc02d94627490658d17be1080d @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_2134: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2134/ IGT_4743: edb2db2cf2b6665d7ba3fa9117263302f6307a4f @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools == Testlist changes == +igt@gem_shrink@two-reclaims-and-oom +igt@gem_shrink@two-reclaims-and-oom-userptr == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2134/ _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev3) 2018-12-07 8:59 [igt-dev] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests Tvrtko Ursulin ` (3 preceding siblings ...) 2018-12-07 10:41 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev2) Patchwork @ 2018-12-07 15:25 ` Patchwork 2018-12-07 15:56 ` [igt-dev] ✓ Fi.CI.IGT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev2) Patchwork 2018-12-07 22:19 ` [igt-dev] ✓ Fi.CI.IGT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev3) Patchwork 6 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2018-12-07 15:25 UTC (permalink / raw) To: Tvrtko Ursulin; +Cc: igt-dev == Series Details == Series: series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev3) URL : https://patchwork.freedesktop.org/series/53726/ State : success == Summary == CI Bug Log - changes from CI_DRM_5286 -> IGTPW_2137 ==================================================== Summary ------- **WARNING** Minor unknown changes coming with IGTPW_2137 need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in IGTPW_2137, please notify your bug team to allow them to document this new failure mode, which will reduce false positives in CI. External URL: https://patchwork.freedesktop.org/api/1.0/series/53726/revisions/3/mbox/ Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_2137: ### IGT changes ### #### Possible regressions #### * igt@kms_flip@basic-flip-vs-wf_vblank: - {fi-icl-u3}: NOTRUN -> INCOMPLETE #### Warnings #### * igt@pm_rps@basic-api: - fi-apl-guc: SKIP -> PASS +14 Known issues ------------ Here are the changes found in IGTPW_2137 that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_exec_suspend@basic-s4-devices: - fi-ivb-3520m: PASS -> FAIL [fdo#108880] * igt@i915_module_load@reload: - fi-blb-e6850: PASS -> INCOMPLETE [fdo#107718] * igt@kms_pipe_crc_basic@suspend-read-crc-pipe-a: - fi-snb-2600: PASS -> DMESG-WARN [fdo#102365] #### Possible fixes #### * igt@gem_exec_suspend@basic-s3: - {fi-icl-u3}: INCOMPLETE [fdo#107713] -> PASS * igt@i915_selftest@live_execlists: - fi-apl-guc: INCOMPLETE [fdo#103927] -> PASS * igt@vgem_basic@dmabuf-export: - fi-apl-guc: DMESG-WARN [fdo#108968] -> PASS +10 {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#102365]: https://bugs.freedesktop.org/show_bug.cgi?id=102365 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713 [fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718 [fdo#108880]: https://bugs.freedesktop.org/show_bug.cgi?id=108880 [fdo#108968]: https://bugs.freedesktop.org/show_bug.cgi?id=108968 Participating hosts (50 -> 46) ------------------------------ Additional (2): fi-glk-j4005 fi-pnv-d510 Missing (6): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 Build changes ------------- * IGT: IGT_4743 -> IGTPW_2137 CI_DRM_5286: 47717a5a4d159f980da051acd168f2c40dbc6736 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_2137: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2137/ IGT_4743: edb2db2cf2b6665d7ba3fa9117263302f6307a4f @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools == Testlist changes == +igt@gem_shrink@two-reclaims-and-oom +igt@gem_shrink@two-reclaims-and-oom-userptr == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2137/ _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev2) 2018-12-07 8:59 [igt-dev] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests Tvrtko Ursulin ` (4 preceding siblings ...) 2018-12-07 15:25 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev3) Patchwork @ 2018-12-07 15:56 ` Patchwork 2018-12-07 22:19 ` [igt-dev] ✓ Fi.CI.IGT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev3) Patchwork 6 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2018-12-07 15:56 UTC (permalink / raw) To: Tvrtko Ursulin; +Cc: igt-dev == Series Details == Series: series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev2) URL : https://patchwork.freedesktop.org/series/53726/ State : success == Summary == CI Bug Log - changes from CI_DRM_5281_full -> IGTPW_2134_full ==================================================== Summary ------- **WARNING** Minor unknown changes coming with IGTPW_2134_full need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in IGTPW_2134_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://patchwork.freedesktop.org/api/1.0/series/53726/revisions/2/mbox/ Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_2134_full: ### IGT changes ### #### Warnings #### * igt@pm_rc6_residency@rc6-accuracy: - shard-snb: SKIP -> PASS Known issues ------------ Here are the changes found in IGTPW_2134_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@gem_exec_fence@basic-await-default: - shard-hsw: PASS -> FAIL [fdo#108888] * igt@gem_exec_schedule@pi-ringfull-render: - shard-glk: NOTRUN -> FAIL [fdo#103158] * igt@kms_cursor_crc@cursor-128x128-random: - shard-apl: PASS -> FAIL [fdo#103232] +4 * igt@kms_cursor_crc@cursor-256x256-dpms: - shard-glk: PASS -> FAIL [fdo#103232] +7 * igt@kms_cursor_crc@cursor-256x256-suspend: - shard-kbl: PASS -> FAIL [fdo#103191] / [fdo#103232] - shard-glk: NOTRUN -> FAIL [fdo#103232] +1 - shard-apl: PASS -> FAIL [fdo#103191] / [fdo#103232] * igt@kms_cursor_crc@cursor-64x64-dpms: - shard-kbl: PASS -> FAIL [fdo#103232] +2 * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-pwrite: - shard-apl: PASS -> FAIL [fdo#103167] +2 * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-move: - shard-kbl: PASS -> FAIL [fdo#103167] +2 * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-onoff: - shard-glk: PASS -> FAIL [fdo#103167] +9 * igt@kms_plane@plane-position-covered-pipe-c-planes: - shard-apl: PASS -> FAIL [fdo#103166] +2 * igt@kms_plane_multiple@atomic-pipe-b-tiling-y: - shard-glk: PASS -> FAIL [fdo#103166] +5 * igt@pm_rpm@modeset-stress-extra-wait: - shard-apl: PASS -> INCOMPLETE [fdo#103927] #### Possible fixes #### * igt@kms_cursor_crc@cursor-128x128-random: - shard-glk: FAIL [fdo#103232] -> PASS * igt@kms_cursor_crc@cursor-128x42-sliding: - shard-kbl: FAIL [fdo#103232] -> PASS - shard-apl: FAIL [fdo#103232] -> PASS +2 * igt@kms_cursor_crc@cursor-64x64-suspend: - shard-apl: FAIL [fdo#103191] / [fdo#103232] -> PASS * igt@kms_flip@2x-flip-vs-expired-vblank-interruptible: - shard-glk: FAIL [fdo#105363] -> PASS * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-gtt: - shard-apl: FAIL [fdo#103167] -> PASS - shard-kbl: FAIL [fdo#103167] -> PASS * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-fullscreen: - shard-glk: FAIL [fdo#103167] -> PASS +1 * igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb: - shard-apl: FAIL [fdo#108145] -> PASS * igt@kms_plane_alpha_blend@pipe-c-constant-alpha-max: - shard-glk: FAIL [fdo#108145] -> PASS +4 * igt@kms_plane_multiple@atomic-pipe-b-tiling-x: - shard-glk: FAIL [fdo#103166] -> PASS +1 - shard-kbl: FAIL [fdo#103166] -> PASS * igt@kms_plane_multiple@atomic-pipe-c-tiling-yf: - shard-apl: FAIL [fdo#103166] -> PASS +1 * {igt@kms_rotation_crc@multiplane-rotation-cropping-top}: - shard-kbl: DMESG-FAIL [fdo#108950] -> PASS - shard-glk: DMESG-FAIL [fdo#105763] / [fdo#106538] -> PASS * igt@kms_setmode@basic: - shard-apl: FAIL [fdo#99912] -> PASS - shard-hsw: FAIL [fdo#99912] -> PASS * igt@kms_vblank@pipe-c-ts-continuation-suspend: - shard-kbl: INCOMPLETE [fdo#103665] -> PASS {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#103158]: https://bugs.freedesktop.org/show_bug.cgi?id=103158 [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166 [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167 [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191 [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232 [fdo#103665]: https://bugs.freedesktop.org/show_bug.cgi?id=103665 [fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927 [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363 [fdo#105763]: https://bugs.freedesktop.org/show_bug.cgi?id=105763 [fdo#106538]: https://bugs.freedesktop.org/show_bug.cgi?id=106538 [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145 [fdo#108888]: https://bugs.freedesktop.org/show_bug.cgi?id=108888 [fdo#108950]: https://bugs.freedesktop.org/show_bug.cgi?id=108950 [fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912 Participating hosts (7 -> 5) ------------------------------ Missing (2): shard-skl shard-iclb Build changes ------------- * IGT: IGT_4743 -> IGTPW_2134 * Piglit: piglit_4509 -> None CI_DRM_5281: 678f4e6d807098fc02d94627490658d17be1080d @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_2134: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2134/ IGT_4743: edb2db2cf2b6665d7ba3fa9117263302f6307a4f @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2134/ _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev3) 2018-12-07 8:59 [igt-dev] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests Tvrtko Ursulin ` (5 preceding siblings ...) 2018-12-07 15:56 ` [igt-dev] ✓ Fi.CI.IGT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev2) Patchwork @ 2018-12-07 22:19 ` Patchwork 6 siblings, 0 replies; 13+ messages in thread From: Patchwork @ 2018-12-07 22:19 UTC (permalink / raw) To: Tvrtko Ursulin; +Cc: igt-dev == Series Details == Series: series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev3) URL : https://patchwork.freedesktop.org/series/53726/ State : success == Summary == CI Bug Log - changes from CI_DRM_5286_full -> IGTPW_2137_full ==================================================== Summary ------- **WARNING** Minor unknown changes coming with IGTPW_2137_full need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in IGTPW_2137_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://patchwork.freedesktop.org/api/1.0/series/53726/revisions/3/mbox/ Possible new issues ------------------- Here are the unknown changes that may have been introduced in IGTPW_2137_full: ### IGT changes ### #### Warnings #### * igt@pm_rc6_residency@rc6-accuracy: - shard-kbl: PASS -> SKIP +1 - shard-snb: SKIP -> PASS Known issues ------------ Here are the changes found in IGTPW_2137_full that come from known issues: ### IGT changes ### #### Issues hit #### * igt@kms_cursor_crc@cursor-128x128-onscreen: - shard-apl: PASS -> FAIL [fdo#103232] +3 * igt@kms_cursor_crc@cursor-256x256-random: - shard-glk: PASS -> FAIL [fdo#103232] +5 * igt@kms_cursor_crc@cursor-64x64-suspend: - shard-kbl: PASS -> FAIL [fdo#103191] / [fdo#103232] * igt@kms_flip@2x-flip-vs-expired-vblank-interruptible: - shard-glk: PASS -> FAIL [fdo#105363] * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-pwrite: - shard-apl: PASS -> FAIL [fdo#103167] +5 * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-move: - shard-kbl: PASS -> FAIL [fdo#103167] +2 * igt@kms_frontbuffer_tracking@fbc-shrfb-scaledprimary: - shard-glk: PASS -> FAIL [fdo#103167] +10 * igt@kms_plane@pixel-format-pipe-b-planes: - shard-kbl: PASS -> FAIL [fdo#103166] * {igt@kms_plane@pixel-format-pipe-c-planes-source-clamping}: - shard-glk: PASS -> FAIL [fdo#108948] * igt@kms_plane@plane-position-covered-pipe-c-planes: - shard-apl: PASS -> FAIL [fdo#103166] +2 * igt@kms_plane_alpha_blend@pipe-a-alpha-opaque-fb: - shard-apl: PASS -> FAIL [fdo#108145] * igt@kms_plane_alpha_blend@pipe-b-constant-alpha-max: - shard-glk: PASS -> FAIL [fdo#108145] * igt@kms_plane_multiple@atomic-pipe-b-tiling-y: - shard-glk: PASS -> FAIL [fdo#103166] +8 #### Possible fixes #### * igt@kms_cursor_crc@cursor-128x42-sliding: - shard-kbl: FAIL [fdo#103232] -> PASS - shard-apl: FAIL [fdo#103232] -> PASS +2 - shard-glk: FAIL [fdo#103232] -> PASS * igt@kms_flip@blocking-wf_vblank: - shard-kbl: DMESG-WARN [fdo#103558] / [fdo#105602] -> PASS +8 * igt@kms_frontbuffer_tracking@fbc-1p-primscrn-spr-indfb-draw-mmap-gtt: - shard-apl: FAIL [fdo#103167] -> PASS +1 - shard-kbl: FAIL [fdo#103167] -> PASS * igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-pwrite: - shard-glk: FAIL [fdo#103167] -> PASS +1 * {igt@kms_plane@pixel-format-pipe-b-planes-source-clamping}: - shard-apl: FAIL [fdo#108948] -> PASS +1 * igt@kms_plane_alpha_blend@pipe-c-constant-alpha-max: - shard-glk: FAIL [fdo#108145] -> PASS +1 * igt@kms_plane_multiple@atomic-pipe-b-tiling-x: - shard-apl: FAIL [fdo#103166] -> PASS +2 - shard-kbl: FAIL [fdo#103166] -> PASS {name}: This element is suppressed. This means it is ignored when computing the status of the difference (SUCCESS, WARNING, or FAILURE). [fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166 [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167 [fdo#103191]: https://bugs.freedesktop.org/show_bug.cgi?id=103191 [fdo#103232]: https://bugs.freedesktop.org/show_bug.cgi?id=103232 [fdo#103558]: https://bugs.freedesktop.org/show_bug.cgi?id=103558 [fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363 [fdo#105602]: https://bugs.freedesktop.org/show_bug.cgi?id=105602 [fdo#108145]: https://bugs.freedesktop.org/show_bug.cgi?id=108145 [fdo#108948]: https://bugs.freedesktop.org/show_bug.cgi?id=108948 Participating hosts (7 -> 5) ------------------------------ Missing (2): shard-skl shard-iclb Build changes ------------- * IGT: IGT_4743 -> IGTPW_2137 * Piglit: piglit_4509 -> None CI_DRM_5286: 47717a5a4d159f980da051acd168f2c40dbc6736 @ git://anongit.freedesktop.org/gfx-ci/linux IGTPW_2137: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2137/ IGT_4743: edb2db2cf2b6665d7ba3fa9117263302f6307a4f @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2137/ _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2018-12-07 22:19 UTC | newest] Thread overview: 13+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-12-07 8:59 [igt-dev] [RFT i-g-t 1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests Tvrtko Ursulin 2018-12-07 8:59 ` [Intel-gfx] [RFT i-g-t 2/2] intel-ci: Unblack list the new tests?? Tvrtko Ursulin 2018-12-07 9:24 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests Patchwork 2018-12-07 10:14 ` [Intel-gfx] [RFT i-g-t 1/2] " Tvrtko Ursulin 2018-12-07 10:17 ` [igt-dev] " Chris Wilson 2018-12-07 10:28 ` Tvrtko Ursulin 2018-12-07 14:04 ` Tvrtko Ursulin 2018-12-07 14:06 ` [igt-dev] " Chris Wilson 2018-12-07 14:13 ` Tvrtko Ursulin 2018-12-07 10:41 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev2) Patchwork 2018-12-07 15:25 ` [igt-dev] ✓ Fi.CI.BAT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev3) Patchwork 2018-12-07 15:56 ` [igt-dev] ✓ Fi.CI.IGT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev2) Patchwork 2018-12-07 22:19 ` [igt-dev] ✓ Fi.CI.IGT: success for series starting with [RFT,i-g-t,1/2] tests/gem_shrink: Background, direct and OOM shrinker plus userptr tests (rev3) Patchwork
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox