From patchwork Mon Jul 23 20:07:36 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chris Wilson X-Patchwork-Id: 10540829 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id CECD8112B for ; Mon, 23 Jul 2018 20:07:52 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id B486E1FF66 for ; Mon, 23 Jul 2018 20:07:52 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id A5CB81FF81; Mon, 23 Jul 2018 20:07:52 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 20DC51FF66 for ; Mon, 23 Jul 2018 20:07:52 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 779DF6E514; Mon, 23 Jul 2018 20:07:51 +0000 (UTC) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from fireflyinternet.com (mail.fireflyinternet.com [109.228.58.192]) by gabe.freedesktop.org (Postfix) with ESMTPS id 80D346E518; Mon, 23 Jul 2018 20:07:50 +0000 (UTC) X-Default-Received-SPF: pass (skip=forwardok (res=PASS)) x-ip-name=78.156.65.138; Received: from haswell.alporthouse.com (unverified [78.156.65.138]) by fireflyinternet.com (Firefly Internet (M1)) with ESMTP id 12439490-1500050 for multiple; Mon, 23 Jul 2018 21:07:38 +0100 From: Chris Wilson To: intel-gfx@lists.freedesktop.org Date: Mon, 23 Jul 2018 21:07:36 +0100 Message-Id: <20180723200736.29508-4-chris@chris-wilson.co.uk> X-Mailer: git-send-email 2.18.0 In-Reply-To: <20180723200736.29508-1-chris@chris-wilson.co.uk> References: <20180723200736.29508-1-chris@chris-wilson.co.uk> Subject: [Intel-gfx] [PATCH i-g-t 4/4] igt/gem_exec_capture: Capture many, many objects X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: igt-dev@lists.freedesktop.org MIME-Version: 1.0 Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Virus-Scanned: ClamAV using ClamSMTP Exercise O(N^2) behaviour in reading the error state, and push it to the extreme. Reported-by: Jason Ekstrand Signed-off-by: Chris Wilson Reviewed-by: Katarzyna Dec --- tests/gem_exec_capture.c | 156 ++++++++++++++++++++++++++++++++++++++- 1 file changed, 153 insertions(+), 3 deletions(-) diff --git a/tests/gem_exec_capture.c b/tests/gem_exec_capture.c index 2dc06ce43..6cc175551 100644 --- a/tests/gem_exec_capture.c +++ b/tests/gem_exec_capture.c @@ -23,6 +23,7 @@ #include "igt.h" #include "igt_device.h" +#include "igt_rand.h" #include "igt_sysfs.h" #define LOCAL_OBJECT_CAPTURE (1 << 7) @@ -57,7 +58,7 @@ static void check_error_state(int dir, struct drm_i915_gem_exec_object2 *obj) igt_assert(found); } -static void __capture(int fd, int dir, unsigned ring, uint32_t target) +static void __capture1(int fd, int dir, unsigned ring, uint32_t target) { const int gen = intel_gen(intel_get_drm_devid(fd)); struct drm_i915_gem_exec_object2 obj[4]; @@ -167,10 +168,149 @@ static void capture(int fd, int dir, unsigned ring) uint32_t handle; handle = gem_create(fd, 4096); - __capture(fd, dir, ring, handle); + __capture1(fd, dir, ring, handle); gem_close(fd, handle); } +static void __captureN(int fd, int dir, unsigned ring, + unsigned int size, int count, unsigned int flags) +#define RANDOM 0x1 +{ + const int gen = intel_gen(intel_get_drm_devid(fd)); + struct drm_i915_gem_exec_object2 *obj; + struct drm_i915_gem_relocation_entry reloc[2]; + struct drm_i915_gem_execbuffer2 execbuf; + uint32_t *batch, *seqno; + int i; + + obj = calloc(count + 2, sizeof(*obj)); + igt_assert(obj); + + obj[0].handle = gem_create(fd, 4096); + for (i = 0; i < count; i++) { + obj[i + 1].handle = gem_create(fd, size); + obj[i + 1].flags = + LOCAL_OBJECT_CAPTURE | EXEC_OBJECT_SUPPORTS_48B_ADDRESS; + if (flags & RANDOM) { + uint32_t *ptr; + + ptr = gem_mmap__cpu(fd, obj[i + 1].handle, + 0, size, PROT_WRITE); + for (unsigned int n = 0; n < size / sizeof(*ptr); n++) + ptr[n] = hars_petruska_f54_1_random_unsafe(); + munmap(ptr, size); + } + } + + obj[count + 1].handle = gem_create(fd, 4096); + obj[count + 1].relocs_ptr = (uintptr_t)reloc; + obj[count + 1].relocation_count = ARRAY_SIZE(reloc); + + memset(reloc, 0, sizeof(reloc)); + reloc[0].target_handle = obj[count + 1].handle; /* recurse */ + reloc[0].presumed_offset = 0; + reloc[0].offset = 5*sizeof(uint32_t); + reloc[0].delta = 0; + reloc[0].read_domains = I915_GEM_DOMAIN_COMMAND; + reloc[0].write_domain = 0; + + reloc[1].target_handle = obj[0].handle; /* breadcrumb */ + reloc[1].presumed_offset = 0; + reloc[1].offset = sizeof(uint32_t); + reloc[1].delta = 0; + reloc[1].read_domains = I915_GEM_DOMAIN_RENDER; + reloc[1].write_domain = I915_GEM_DOMAIN_RENDER; + + seqno = gem_mmap__wc(fd, obj[0].handle, 0, 4096, PROT_READ); + gem_set_domain(fd, obj[0].handle, + I915_GEM_DOMAIN_GTT, I915_GEM_DOMAIN_GTT); + + batch = gem_mmap__cpu(fd, obj[count + 1].handle, 0, 4096, PROT_WRITE); + gem_set_domain(fd, obj[count + 1].handle, + I915_GEM_DOMAIN_CPU, I915_GEM_DOMAIN_CPU); + + i = 0; + batch[i] = MI_STORE_DWORD_IMM | (gen < 6 ? 1 << 22 : 0); + if (gen >= 8) { + batch[++i] = 0; + batch[++i] = 0; + } else if (gen >= 4) { + batch[++i] = 0; + batch[++i] = 0; + reloc[1].offset += sizeof(uint32_t); + } else { + batch[i]--; + batch[++i] = 0; + } + batch[++i] = 0xc0ffee; + if (gen < 3) + batch[++i] = MI_NOOP; + + batch[++i] = MI_BATCH_BUFFER_START; /* not crashed? try again! */ + if (gen >= 8) { + batch[i] |= 1 << 8 | 1; + batch[++i] = 0; + batch[++i] = 0; + } else if (gen >= 6) { + batch[i] |= 1 << 8; + batch[++i] = 0; + } else { + batch[i] |= 2 << 6; + batch[++i] = 0; + if (gen < 4) { + batch[i] |= 1; + reloc[0].delta = 1; + } + } + munmap(batch, 4096); + + memset(&execbuf, 0, sizeof(execbuf)); + execbuf.buffers_ptr = (uintptr_t)obj; + execbuf.buffer_count = count + 2; + execbuf.flags = ring; + if (gen > 3 && gen < 6) + execbuf.flags |= I915_EXEC_SECURE; + gem_execbuf(fd, &execbuf); + + /* Wait for the request to start */ + while (*(volatile uint32_t *)seqno != 0xc0ffee) + igt_assert(gem_bo_busy(fd, obj[0].handle)); + munmap(seqno, 4096); + + igt_force_gpu_reset(fd); + + gem_sync(fd, obj[count + 1].handle); + gem_close(fd, obj[count + 1].handle); + for (i = 0; i < count; i++) + gem_close(fd, obj[i + 1].handle); + gem_close(fd, obj[0].handle); +} + +static void many(int fd, int dir, unsigned int flags) +{ + uint64_t ram, gtt; + unsigned long count; + char *error; + + gtt = (gem_aperture_size(fd) >> 20) / 4; + ram = intel_get_avail_ram_mb() / 4; + igt_debug("Available objects in GTT:%"PRIu64", RAM:%"PRIu64"\n", + gtt, ram); + + count = min(gtt, ram); + igt_require(count > 1); + + intel_require_memory(count, 2 << 20, CHECK_RAM); + + __captureN(fd, dir, 0, 2 << 20, count, flags); + + error = igt_sysfs_get(dir, "error"); + igt_sysfs_set(dir, "error", "Begone!"); + + igt_assert(error); + igt_debug("%s\n", error); +} + static void userptr(int fd, int dir) { uint32_t handle; @@ -179,7 +319,7 @@ static void userptr(int fd, int dir) igt_assert(posix_memalign(&ptr, 4096, 4096) == 0); igt_require(__gem_userptr(fd, ptr, 4096, 0, 0, &handle) == 0); - __capture(fd, dir, 0, handle); + __capture1(fd, dir, 0, handle); gem_close(fd, handle); free(ptr); @@ -236,6 +376,16 @@ igt_main } } + igt_subtest_f("many-zero") { + igt_require(gem_can_store_dword(fd, 0)); + many(fd, dir, 0); + } + + igt_subtest_f("many-random") { + igt_require(gem_can_store_dword(fd, 0)); + many(fd, dir, RANDOM); + } + /* And check we can read from different types of objects */ igt_subtest_f("userptr") {