All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dominik Grzegorzek <dominik.grzegorzek@intel.com>
To: igt-dev@lists.freedesktop.org
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Subject: [igt-dev] [PATCH i-g-t v4 08/25] i915/gem_threaded_access_tiled.c: Remove libdrm dependency
Date: Tue, 22 Sep 2020 13:52:12 +0200	[thread overview]
Message-ID: <20200922115229.28371-9-dominik.grzegorzek@intel.com> (raw)
In-Reply-To: <20200922115229.28371-1-dominik.grzegorzek@intel.com>

Use intel_bb / intel_buf to remove libdrm dependency.

Signed-off-by: Dominik Grzegorzek <dominik.grzegorzek@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
---
 tests/i915/gem_threaded_access_tiled.c | 38 ++++++++++++--------------
 1 file changed, 18 insertions(+), 20 deletions(-)

diff --git a/tests/i915/gem_threaded_access_tiled.c b/tests/i915/gem_threaded_access_tiled.c
index 6442d589..47b2962e 100644
--- a/tests/i915/gem_threaded_access_tiled.c
+++ b/tests/i915/gem_threaded_access_tiled.c
@@ -31,8 +31,6 @@
 #include <unistd.h>
 #include <pthread.h>
 
-#include "intel_bufmgr.h"
-
 IGT_TEST_DESCRIPTION("Check parallel access to tiled memory.");
 
 /* Testcase: check parallel access to tiled memory
@@ -45,10 +43,10 @@ IGT_TEST_DESCRIPTION("Check parallel access to tiled memory.");
 #define HEIGHT 4096
 
 struct thread_ctx {
-	drm_intel_bo *bo;
+	struct intel_buf *buf;
 };
 
-static drm_intel_bufmgr *bufmgr;
+static struct buf_ops *bops;
 static struct thread_ctx tctx[NUM_THREADS];
 
 static void *copy_fn(void *p)
@@ -60,13 +58,13 @@ static void *copy_fn(void *p)
 	if (buf == NULL)
 		return (void *)1;
 
-	memcpy(buf, c->bo->virtual, WIDTH * HEIGHT);
+	memcpy(buf, c->buf->ptr, WIDTH * HEIGHT);
 
 	free(buf);
 	return (void *)0;
 }
 
-static int copy_tile_threaded(drm_intel_bo *bo)
+static int copy_tile_threaded(struct intel_buf *buf)
 {
 	int i;
 	int r;
@@ -74,7 +72,7 @@ static int copy_tile_threaded(drm_intel_bo *bo)
 	void *status;
 
 	for (i = 0; i < NUM_THREADS; i++) {
-		tctx[i].bo = bo;
+		tctx[i].buf = buf;
 		r = pthread_create(&thr[i], NULL, copy_fn, (void *)&tctx[i]);
 		igt_assert_eq(r, 0);
 	}
@@ -90,9 +88,8 @@ static int copy_tile_threaded(drm_intel_bo *bo)
 igt_simple_main
 {
 	int fd;
-	drm_intel_bo *bo;
+	struct intel_buf *buf;
 	uint32_t tiling_mode = I915_TILING_Y;
-	unsigned long pitch = 0;
 	int r;
 
 	fd = drm_open_driver(DRIVER_INTEL);
@@ -100,24 +97,25 @@ igt_simple_main
 
 	igt_require(gem_available_fences(fd) > 0);
 
-	bufmgr = drm_intel_bufmgr_gem_init(fd, 4096);
-	igt_assert(bufmgr);
+	bops = buf_ops_create(fd);
 
-	bo = drm_intel_bo_alloc_tiled(bufmgr, "mmap bo", WIDTH, HEIGHT, 1,
-				      &tiling_mode, &pitch, 0);
-	igt_assert(bo);
+	buf = intel_buf_create(bops, WIDTH, HEIGHT, 8, 0, tiling_mode,
+			       I915_COMPRESSION_NONE);
+	igt_assert(buf);
 
-	r = drm_intel_gem_bo_map_gtt(bo);
-	igt_assert(!r);
+	buf->ptr = __gem_mmap__gtt(fd, buf->handle, buf->surface[0].size,
+				   PROT_WRITE | PROT_READ);
+	igt_assert(buf->ptr);
 
-	r = copy_tile_threaded(bo);
+	r = copy_tile_threaded(buf);
 	igt_assert(!r);
 
-	r = drm_intel_gem_bo_unmap_gtt(bo);
+	r = gem_munmap(buf->ptr, buf->surface[0].size);
+	buf->ptr = NULL;
 	igt_assert(!r);
 
-	drm_intel_bo_unreference(bo);
-	drm_intel_bufmgr_destroy(bufmgr);
+	intel_buf_destroy(buf);
+	buf_ops_destroy(bops);
 
 	close(fd);
 }
-- 
2.20.1

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

  parent reply	other threads:[~2020-09-22 11:53 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-22 11:52 [igt-dev] [PATCH i-g-t v4 00/25] tests/benchmarks: Libdrm removal Dominik Grzegorzek
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 01/25] i915/gem_mmap: Modified offset in subtest "bad-size" Dominik Grzegorzek
2020-09-22 11:56   ` Grzegorzek, Dominik
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 02/25] lib/intel_batchbuffer: Add control over fencing in intel_bb Dominik Grzegorzek
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 03/25] lib/intel_batchbuffer: add intel_bb_blit_copy wrapper Dominik Grzegorzek
2020-09-22 13:44   ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 04/25] Remove unused intel_bufmgr.h headers Dominik Grzegorzek
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 05/25] i915/gem_pwrite_snooped: Remove libdrm dependency Dominik Grzegorzek
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 06/25] i915/gem_pipe_control_store_loop.c: " Dominik Grzegorzek
2020-09-22 13:51   ` Zbigniew Kempczyński
2020-09-23  7:19     ` Grzegorzek, Dominik
2020-09-23  8:24       ` Zbigniew Kempczyński
2020-09-23 13:09       ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 07/25] i915/gem_pread_after_blit.c: " Dominik Grzegorzek
2020-09-22 14:01   ` Zbigniew Kempczyński
2020-09-22 11:52 ` Dominik Grzegorzek [this message]
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 09/25] i915/gem_tiled_blits: " Dominik Grzegorzek
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 10/25] i915/gem_unfence_active_buffers.c: Remove librdm dependency Dominik Grzegorzek
2020-09-23  8:39   ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 11/25] i915/gem_unref_active_buffers.c: Remove libdrm dependency Dominik Grzegorzek
2020-09-23 10:35   ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 12/25] i915/gem_tiled_partial_pwrite_pread: " Dominik Grzegorzek
2020-09-23 11:36   ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 13/25] i915/gem_set_tiling_vs_blit.c: " Dominik Grzegorzek
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 14/25] tests/kms_fence_pin_leak.c: " Dominik Grzegorzek
2020-09-23 12:59   ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 15/25] tests/kms_flip.c: " Dominik Grzegorzek
2020-09-23 13:07   ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 16/25] tests/kms_psr2_su.c: Get rid of unused bufmgr Dominik Grzegorzek
2020-09-24  5:47   ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 17/25] benchmarks: Remove libdrm dependency Dominik Grzegorzek
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 18/25] tests/prime_mmap_coherency.c: " Dominik Grzegorzek
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 19/25] tools/intel_perf_counters: " Dominik Grzegorzek
2020-09-28  8:03   ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 20/25] tests/prime_udl: " Dominik Grzegorzek
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 21/25] tests/prime_nv_pcopy.c: " Dominik Grzegorzek
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 22/25] tests/prime_nv_api.c: " Dominik Grzegorzek
2020-09-28  7:40   ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 23/25] tests/prime_nv_test.c: " Dominik Grzegorzek
2020-09-28  7:49   ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 24/25] tests/i915/gem_ppgtt: make copying more readable Dominik Grzegorzek
2020-09-28  7:53   ` Zbigniew Kempczyński
2020-09-22 11:52 ` [igt-dev] [PATCH i-g-t v4 25/25] HAX: run changed tests in BAT only Dominik Grzegorzek
2020-09-22 12:35 ` [igt-dev] ✗ Fi.CI.BAT: failure for tests/benchmarks: Libdrm removal (rev4) Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200922115229.28371-9-dominik.grzegorzek@intel.com \
    --to=dominik.grzegorzek@intel.com \
    --cc=chris@chris-wilson.co.uk \
    --cc=igt-dev@lists.freedesktop.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.