[igt-dev] [PATCH i-g-t] igt/gem_tiled_fence_blits: Remove libdrm_intel dependence
Tvrtko Ursulin
tvrtko.ursulin at linux.intel.com
Thu Oct 18 14:45:12 UTC 2018
On 23/07/2018 13:14, Chris Wilson wrote:
> Modernise the test to use igt's ioctl library as opposed to the
> antiquated libdrm_intel.
>
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> ---
> tests/gem_tiled_fence_blits.c | 188 ++++++++++++++++++++--------------
> 1 file changed, 110 insertions(+), 78 deletions(-)
>
> diff --git a/tests/gem_tiled_fence_blits.c b/tests/gem_tiled_fence_blits.c
> index 693e96cec..5c1e1a68a 100644
> --- a/tests/gem_tiled_fence_blits.c
> +++ b/tests/gem_tiled_fence_blits.c
> @@ -42,54 +42,38 @@
> */
>
> #include "igt.h"
> -#include <stdlib.h>
> -#include <stdio.h>
> -#include <string.h>
> -#include <fcntl.h>
> -#include <inttypes.h>
> -#include <errno.h>
> -#include <sys/stat.h>
> -#include <sys/time.h>
> -
> -#include <drm.h>
> -
> -#include "intel_bufmgr.h"
> -
> -static drm_intel_bufmgr *bufmgr;
> -struct intel_batchbuffer *batch;
> -enum {width=512, height=512};
> -static const int bo_size = width * height * 4;
> +#include "igt_x86.h"
> +
> +enum { width = 512, height = 512 };
> static uint32_t linear[width * height];
> +static const int bo_size = sizeof(linear);
>
> -static drm_intel_bo *
> -create_bo(int fd, uint32_t start_val)
> +static uint32_t create_bo(int fd, uint32_t start_val)
> {
> - drm_intel_bo *bo;
> - uint32_t tiling = I915_TILING_X;
> - int ret, i;
> + uint32_t handle;
> + uint32_t *ptr;
>
> - bo = drm_intel_bo_alloc(bufmgr, "tiled bo", bo_size, 4096);
> - ret = drm_intel_bo_set_tiling(bo, &tiling, width * 4);
> - igt_assert_eq(ret, 0);
> - igt_assert(tiling == I915_TILING_X);
> + handle = gem_create(fd, bo_size);
> + gem_set_tiling(fd, handle, I915_TILING_X, width * 4);
>
> /* Fill the BO with dwords starting at start_val */
> - for (i = 0; i < width * height; i++)
> - linear[i] = start_val++;
> -
> - gem_write(fd, bo->handle, 0, linear, sizeof(linear));
> + ptr = gem_mmap__gtt(fd, handle, bo_size, PROT_WRITE);
> + for (int i = 0; i < width * height; i++)
> + ptr[i] = start_val++;
> + munmap(ptr, bo_size);
>
> - return bo;
> + return handle;
> }
>
> -static void
> -check_bo(int fd, drm_intel_bo *bo, uint32_t start_val)
> +static void check_bo(int fd, uint32_t handle, uint32_t start_val)
> {
> - int i;
> + uint32_t *ptr;
>
> - gem_read(fd, bo->handle, 0, linear, sizeof(linear));
> + ptr = gem_mmap__gtt(fd, handle, bo_size, PROT_READ);
> + igt_memcpy_from_wc(linear, ptr, bo_size);
> + munmap(ptr, bo_size);
>
> - for (i = 0; i < width * height; i++) {
> + for (int i = 0; i < width * height; i++) {
> igt_assert_f(linear[i] == start_val,
> "Expected 0x%08x, found 0x%08x "
> "at offset 0x%08x\n",
> @@ -98,73 +82,122 @@ check_bo(int fd, drm_intel_bo *bo, uint32_t start_val)
> }
> }
>
> +static uint32_t
> +create_batch(int fd, struct drm_i915_gem_relocation_entry *reloc)
> +{
> + const int gen = intel_gen(intel_get_drm_devid(fd));
> + const bool has_64b_reloc = gen >= 8;
> + uint32_t *batch;
> + uint32_t handle;
> + uint32_t pitch;
> + int i = 0;
> +
> + handle = gem_create(fd, 4096);
> + batch = gem_mmap__cpu(fd, handle, 0, 4096, PROT_WRITE);
> +
> + batch[i] = (XY_SRC_COPY_BLT_CMD |
> + XY_SRC_COPY_BLT_WRITE_ALPHA |
> + XY_SRC_COPY_BLT_WRITE_RGB);
> + if (gen >= 4) {
> + batch[i] |= (XY_SRC_COPY_BLT_SRC_TILED |
> + XY_SRC_COPY_BLT_DST_TILED);
> + pitch = width;
> + } else {
> + pitch = 4 * width;
> + }
> + batch[i++] |= 6 + 2 * has_64b_reloc;
> +
> + batch[i++] = 3 << 24 | 0xcc << 16 | pitch;
> + batch[i++] = 0; /* dst (x1, y1) */
> + batch[i++] = height << 16 | width; /* dst (x2 y2) */
> + reloc[0].offset = sizeof(*batch) * i;
> + reloc[0].read_domains = I915_GEM_DOMAIN_RENDER;
> + reloc[0].write_domain = I915_GEM_DOMAIN_RENDER;
> + batch[i++] = 0;
> + if (has_64b_reloc)
> + batch[i++] = 0;
> +
> + batch[i++] = 0; /* src (x1, y1) */
> + batch[i++] = pitch;
> + reloc[1].offset = sizeof(*batch) * i;
> + reloc[1].read_domains = I915_GEM_DOMAIN_RENDER;
> + batch[i++] = 0;
> + if (has_64b_reloc)
> + batch[i++] = 0;
> +
> + batch[i++] = MI_BATCH_BUFFER_END;
> + munmap(batch, 4096);
> +
> + return handle;
> +}
> +
> static void run_test(int fd, int count)
> {
> - drm_intel_bo **bo;
> - uint32_t *bo_start_val;
> + struct drm_i915_gem_relocation_entry reloc[2];
> + struct drm_i915_gem_exec_object2 obj[3];
> + struct drm_i915_gem_execbuffer2 eb;
> + uint32_t *bo, *bo_start_val;
> uint32_t start = 0;
> - int i;
> +
> + memset(reloc, 0, sizeof(reloc));
> + memset(obj, 0, sizeof(obj));
> + obj[2].handle = create_batch(fd, reloc);
> + obj[2].relocs_ptr = to_user_pointer(reloc);
> + obj[2].relocation_count = ARRAY_SIZE(reloc);
> +
> + memset(&eb, 0, sizeof(eb));
> + eb.buffers_ptr = to_user_pointer(obj);
> + eb.buffer_count = ARRAY_SIZE(obj);
> + if (intel_gen(intel_get_drm_devid(fd)) >= 6)
> + eb.flags = I915_EXEC_BLT;
>
> count |= 1;
> igt_info("Using %d 1MiB buffers\n", count);
>
> - bo = malloc(count * sizeof(*bo));
> - bo_start_val = malloc(count * sizeof(*bo_start_val));
> - igt_assert(bo && bo_start_val);
> -
> - bufmgr = drm_intel_bufmgr_gem_init(fd, 4096);
> - drm_intel_bufmgr_gem_enable_reuse(bufmgr);
> - batch = intel_batchbuffer_alloc(bufmgr, intel_get_drm_devid(fd));
> + bo = malloc(count * (sizeof(*bo) + sizeof(*bo_start_val)));
> + igt_assert(bo);
> + bo_start_val = bo + count;
>
> - for (i = 0; i < count; i++) {
> + for (int i = 0; i < count; i++) {
> bo[i] = create_bo(fd, start);
> bo_start_val[i] = start;
> -
> - /*
> - igt_info("Creating bo %d\n", i);
> - check_bo(bo[i], bo_start_val[i]);
> - */
> -
> start += width * height;
> }
>
> - for (i = 0; i < count; i++) {
> - int src = count - i - 1;
> - intel_copy_bo(batch, bo[i], bo[src], bo_size);
> - bo_start_val[i] = bo_start_val[src];
> + for (int dst = 0; dst < count; dst++) {
> + int src = count - dst - 1;
> +
> + if (src == dst)
> + continue;
> +
> + reloc[0].target_handle = obj[0].handle = bo[dst];
> + reloc[1].target_handle = obj[1].handle = bo[src];
> +
> + gem_execbuf(fd, &eb);
> + bo_start_val[dst] = bo_start_val[src];
> }
>
> - for (i = 0; i < count * 4; i++) {
> + for (int i = 0; i < count * 4; i++) {
> int src = random() % count;
> int dst = random() % count;
>
> if (src == dst)
> continue;
>
> - intel_copy_bo(batch, bo[dst], bo[src], bo_size);
> - bo_start_val[dst] = bo_start_val[src];
> + reloc[0].target_handle = obj[0].handle = bo[dst];
> + reloc[1].target_handle = obj[1].handle = bo[src];
>
> - /*
> - check_bo(bo[dst], bo_start_val[dst]);
> - igt_info("%d: copy bo %d to %d\n", i, src, dst);
> - */
> + gem_execbuf(fd, &eb);
> + bo_start_val[dst] = bo_start_val[src];
> }
>
> - for (i = 0; i < count; i++) {
> - /*
> - igt_info("check %d\n", i);
> - */
> + for (int i = 0; i < count; i++) {
> check_bo(fd, bo[i], bo_start_val[i]);
> -
> - drm_intel_bo_unreference(bo[i]);
> - bo[i] = NULL;
> + gem_close(fd, bo[i]);
> }
> -
> - intel_batchbuffer_free(batch);
> - drm_intel_bufmgr_destroy(bufmgr);
> -
> - free(bo_start_val);
> free(bo);
> +
> + gem_close(fd, obj[2].handle);
> }
>
> #define MAX_32b ((1ull << 32) - 4096)
> @@ -178,9 +211,8 @@ igt_main
> igt_require_gem(fd);
> }
>
> - igt_subtest("basic") {
> + igt_subtest("basic")
> run_test (fd, 2);
> - }
>
> /* the rest of the tests are too long for simulation */
> igt_skip_on_simulation();
>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
Regards,
Tvrtko
More information about the igt-dev
mailing list