[igt-dev] [PATCH i-g-t 1/2] lib/i915: Split gem_create.c from ioctl_wrappers.c

* [igt-dev] [PATCH i-g-t 1/2] lib/i915: Split gem_create.c from ioctl_wrappers.c
@ 2021-01-11 11:55 Chris Wilson
  2021-01-11 11:55 ` [igt-dev] [PATCH i-g-t 2/2] lib/i915: Replace size to pointer to size in __gem_create() Chris Wilson
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Chris Wilson @ 2021-01-11 11:55 UTC (permalink / raw)
  To: igt-dev; +Cc: Chris Wilson

From: Andrzej Turko <andrzej.turko@linux.intel.com>

In preparation for a variation on the exisiting GEM_CREATE API, split
the ioctl from out of the large ioctl_wrappers.c

Signed-off-by: Andrzej Turko <andrzej.turko@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
 benchmarks/gem_blt.c                  |  1 +
 benchmarks/gem_busy.c                 |  1 +
 benchmarks/gem_create.c               |  3 +-
 benchmarks/gem_exec_ctx.c             |  3 +-
 benchmarks/gem_exec_fault.c           |  5 +--
 benchmarks/gem_exec_nop.c             |  5 +--
 benchmarks/gem_exec_reloc.c           |  8 +++--
 benchmarks/gem_exec_trace.c           |  5 +--
 benchmarks/gem_latency.c              |  3 +-
 benchmarks/gem_prw.c                  |  3 +-
 benchmarks/gem_set_domain.c           |  3 +-
 benchmarks/gem_syslatency.c           |  1 +
 benchmarks/gem_wsim.c                 |  7 ++--
 benchmarks/prime_lookup.c             |  5 +--
 lib/Makefile.sources                  |  1 +
 lib/i915/gem.h                        |  5 +++
 lib/i915/gem_create.c                 | 48 +++++++++++++++++++++++++++
 lib/i915/gem_mman.c                   |  1 +
 lib/i915/gem_mman.h                   |  4 ---
 lib/igt_draw.c                        |  1 +
 lib/igt_fb.c                          |  3 +-
 lib/intel_batchbuffer.c               |  1 +
 lib/intel_bufops.c                    |  2 ++
 lib/ioctl_wrappers.c                  | 38 +--------------------
 lib/ioctl_wrappers.h                  |  2 --
 lib/meson.build                       |  1 +
 tests/i915/gem_basic.c                |  2 ++
 tests/i915/gem_exec_basic.c           |  2 ++
 tests/i915/gem_fd_exhaustion.c        |  2 ++
 tests/i915/gem_fence_thrash.c         |  1 +
 tests/i915/gem_fence_upload.c         |  2 ++
 tests/i915/gem_flink_race.c           |  1 +
 tests/i915/gem_gtt_cpu_tlb.c          |  2 ++
 tests/i915/gem_gtt_speed.c            |  2 ++
 tests/i915/gem_lut_handle.c           |  2 ++
 tests/i915/gem_madvise.c              |  1 +
 tests/i915/gem_mmap.c                 |  2 ++
 tests/i915/gem_mmap_wc.c              |  2 ++
 tests/i915/gem_pread.c                |  3 +-
 tests/i915/gem_pwrite.c               |  2 +-
 tests/i915/gem_readwrite.c            |  2 ++
 tests/i915/gem_set_tiling_vs_gtt.c    |  2 ++
 tests/i915/gem_set_tiling_vs_pwrite.c |  2 ++
 tests/i915/gem_tiled_pread_basic.c    |  3 +-
 tests/i915/gem_tiled_pread_pwrite.c   |  4 +--
 tests/i915/gem_tiled_swapping.c       |  4 +--
 tests/i915/gem_tiled_wb.c             |  2 ++
 tests/i915/gem_tiled_wc.c             |  2 ++
 tests/i915/gem_tiling_max_stride.c    |  2 ++
 tests/i915/i915_module_load.c         | 13 ++++----
 tests/kms_available_modes_crc.c       |  1 +
 tests/kms_big_fb.c                    |  2 ++
 tests/kms_ccs.c                       |  2 ++
 tests/kms_flip.c                      |  1 +
 tests/kms_getfb.c                     |  3 +-
 tests/prime_mmap.c                    |  5 +--
 tests/prime_mmap_kms.c                |  2 ++
 tests/prime_self_import.c             |  1 +
 tools/intel_reg.c                     |  1 +
 59 files changed, 161 insertions(+), 79 deletions(-)
 create mode 100644 lib/i915/gem_create.c

diff --git a/benchmarks/gem_blt.c b/benchmarks/gem_blt.c
index 5c6d28d36..34b82a87b 100644
--- a/benchmarks/gem_blt.c
+++ b/benchmarks/gem_blt.c
@@ -41,6 +41,7 @@
 #include <time.h>
 
 #include "drm.h"
+#include "i915/gem.h"
 
 #define COPY_BLT_CMD		(2<<29|0x53<<22|0x6)
 #define BLT_WRITE_ALPHA		(1<<21)
diff --git a/benchmarks/gem_busy.c b/benchmarks/gem_busy.c
index 50410150f..69be8c178 100644
--- a/benchmarks/gem_busy.c
+++ b/benchmarks/gem_busy.c
@@ -45,6 +45,7 @@
 #include "intel_chipset.h"
 #include "intel_reg.h"
 #include "igt_stats.h"
+#include "i915/gem.h"
 #include "i915/gem_mman.h"
 
 #define ENGINE_FLAGS  (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
diff --git a/benchmarks/gem_create.c b/benchmarks/gem_create.c
index 5dd996c9b..d4e297ed4 100644
--- a/benchmarks/gem_create.c
+++ b/benchmarks/gem_create.c
@@ -39,11 +39,12 @@
 #include <time.h>
 
 #include "drm.h"
-#include "ioctl_wrappers.h"
 #include "drmtest.h"
+#include "i915/gem.h"
 #include "igt_aux.h"
 #include "igt_stats.h"
 #include "intel_reg.h"
+#include "ioctl_wrappers.h"
 
 #define OBJECT_SIZE (1<<23)
 
diff --git a/benchmarks/gem_exec_ctx.c b/benchmarks/gem_exec_ctx.c
index a2e8d93e9..b6f403fbd 100644
--- a/benchmarks/gem_exec_ctx.c
+++ b/benchmarks/gem_exec_ctx.c
@@ -39,11 +39,12 @@
 #include <time.h>
 
 #include "drm.h"
-#include "ioctl_wrappers.h"
 #include "drmtest.h"
+#include "i915/gem.h"
 #include "intel_io.h"
 #include "intel_reg.h"
 #include "igt_stats.h"
+#include "ioctl_wrappers.h"
 
 enum mode { NOP, CREATE, SWITCH, DEFAULT };
 #define SYNC 0x1
diff --git a/benchmarks/gem_exec_fault.c b/benchmarks/gem_exec_fault.c
index 4bd053726..a3a0992cd 100644
--- a/benchmarks/gem_exec_fault.c
+++ b/benchmarks/gem_exec_fault.c
@@ -41,11 +41,12 @@
 #include <time.h>
 
 #include "drm.h"
-#include "ioctl_wrappers.h"
 #include "drmtest.h"
+#include "i915/gem.h"
+#include "igt_stats.h"
 #include "intel_io.h"
 #include "intel_reg.h"
-#include "igt_stats.h"
+#include "ioctl_wrappers.h"
 
 #define ENGINE_FLAGS  (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
 
diff --git a/benchmarks/gem_exec_nop.c b/benchmarks/gem_exec_nop.c
index 0f17d2327..c44d34543 100644
--- a/benchmarks/gem_exec_nop.c
+++ b/benchmarks/gem_exec_nop.c
@@ -39,11 +39,12 @@
 #include <time.h>
 
 #include "drm.h"
-#include "ioctl_wrappers.h"
 #include "drmtest.h"
+#include "i915/gem.h"
+#include "igt_stats.h"
 #include "intel_io.h"
 #include "intel_reg.h"
-#include "igt_stats.h"
+#include "ioctl_wrappers.h"
 
 #define ENGINE_FLAGS  (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK)
 
diff --git a/benchmarks/gem_exec_reloc.c b/benchmarks/gem_exec_reloc.c
index a0736b517..abaf93452 100644
--- a/benchmarks/gem_exec_reloc.c
+++ b/benchmarks/gem_exec_reloc.c
@@ -35,12 +35,14 @@
 #include <errno.h>
 #include <sys/stat.h>
 #include <sys/time.h>
+
 #include "drm.h"
-#include "intel_reg.h"
-#include "ioctl_wrappers.h"
-#include "igt_debugfs.h"
 #include "drmtest.h"
+#include "i915/gem.h"
 #include "i915/gem_mman.h"
+#include "igt_debugfs.h"
+#include "intel_reg.h"
+#include "ioctl_wrappers.h"
 
 #define SKIP_RELOC 0x1
 #define NO_RELOC 0x2
diff --git a/benchmarks/gem_exec_trace.c b/benchmarks/gem_exec_trace.c
index 2724ee920..e37f6fb7d 100644
--- a/benchmarks/gem_exec_trace.c
+++ b/benchmarks/gem_exec_trace.c
@@ -40,10 +40,11 @@
 #include <assert.h>
 
 #include "drm.h"
-#include "ioctl_wrappers.h"
 #include "drmtest.h"
-#include "intel_io.h"
+#include "i915/gem.h"
 #include "igt_stats.h"
+#include "intel_io.h"
+#include "ioctl_wrappers.h"
 
 enum {
 	ADD_BO = 0,
diff --git a/benchmarks/gem_latency.c b/benchmarks/gem_latency.c
index 774a33d31..9b3d22b76 100644
--- a/benchmarks/gem_latency.c
+++ b/benchmarks/gem_latency.c
@@ -41,8 +41,9 @@
 #include <sys/time.h>
 #include <sys/poll.h>
 #include <sys/resource.h>
-#include "drm.h"
 
+#include "drm.h"
+#include "i915/gem.h"
 #include "igt.h"
 #include "igt_device.h"
 
diff --git a/benchmarks/gem_prw.c b/benchmarks/gem_prw.c
index c2846040e..5d3fda9ae 100644
--- a/benchmarks/gem_prw.c
+++ b/benchmarks/gem_prw.c
@@ -39,10 +39,11 @@
 #include <time.h>
 
 #include "drm.h"
-#include "ioctl_wrappers.h"
 #include "drmtest.h"
+#include "i915/gem.h"
 #include "igt_aux.h"
 #include "igt_stats.h"
+#include "ioctl_wrappers.h"
 
 #define OBJECT_SIZE (1<<23)
 
diff --git a/benchmarks/gem_set_domain.c b/benchmarks/gem_set_domain.c
index afb410dc8..1c9d797c7 100644
--- a/benchmarks/gem_set_domain.c
+++ b/benchmarks/gem_set_domain.c
@@ -39,9 +39,10 @@
 #include <time.h>
 
 #include "drm.h"
-#include "ioctl_wrappers.h"
 #include "drmtest.h"
+#include "i915/gem.h"
 #include "igt_aux.h"
+#include "ioctl_wrappers.h"
 
 static double elapsed(const struct timespec *start,
 		      const struct timespec *end)
diff --git a/benchmarks/gem_syslatency.c b/benchmarks/gem_syslatency.c
index 9e57df3af..c10d4f504 100644
--- a/benchmarks/gem_syslatency.c
+++ b/benchmarks/gem_syslatency.c
@@ -44,6 +44,7 @@
 
 #include <linux/unistd.h>
 
+#include "i915/gem.h"
 #include "i915/gem_ring.h"
 
 #define sigev_notify_thread_id _sigev_un._tid
diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
index 1c0c591ed..1b66e63eb 100644
--- a/benchmarks/gem_wsim.c
+++ b/benchmarks/gem_wsim.c
@@ -43,21 +43,22 @@
 #include <pthread.h>
 #include <math.h>
 
+#include "drm.h"
+#include "drmtest.h"
 #include "igt_device_scan.h"
 #include "intel_chipset.h"
 #include "intel_reg.h"
-#include "drm.h"
 #include "ioctl_wrappers.h"
-#include "drmtest.h"
 
 #include "intel_io.h"
 #include "igt_aux.h"
 #include "igt_rand.h"
 #include "igt_perf.h"
 #include "sw_sync.h"
-#include "i915/gem_mman.h"
 
+#include "i915/gem.h"
 #include "i915/gem_engine_topology.h"
+#include "i915/gem_mman.h"
 
 enum intel_engine_id {
 	DEFAULT,
diff --git a/benchmarks/prime_lookup.c b/benchmarks/prime_lookup.c
index d6c397299..b1891756d 100644
--- a/benchmarks/prime_lookup.c
+++ b/benchmarks/prime_lookup.c
@@ -37,10 +37,11 @@
 #include <time.h>
 
 #include "drm.h"
-#include "ioctl_wrappers.h"
 #include "drmtest.h"
-#include "intel_io.h"
+#include "i915/gem.h"
 #include "igt_rand.h"
+#include "intel_io.h"
+#include "ioctl_wrappers.h"
 
 #define CLOSE_DEVICE 0x1
 
diff --git a/lib/Makefile.sources b/lib/Makefile.sources
index 7102f95e7..4f6389f8a 100644
--- a/lib/Makefile.sources
+++ b/lib/Makefile.sources
@@ -7,6 +7,7 @@ lib_source_list =	 	\
 	i915/gem.h		\
 	i915/gem_context.c	\
 	i915/gem_context.h	\
+	i915/gem_create.c	\
 	i915/gem_engine_topology.c	\
 	i915/gem_engine_topology.h	\
 	i915/gem_scheduler.c	\
diff --git a/lib/i915/gem.h b/lib/i915/gem.h
index e5403f09c..9178b773c 100644
--- a/lib/i915/gem.h
+++ b/lib/i915/gem.h
@@ -25,9 +25,14 @@
 #ifndef I915_GEM_H
 #define I915_GEM_H
 
+#include <stdint.h>
+
 void igt_require_gem(int i915);
 void gem_quiescent_gpu(int i915);
 
 int gem_reopen_driver(int i915);
 
+int __gem_create(int fd, uint64_t size, uint32_t *handle);
+uint32_t gem_create(int fd, uint64_t size);
+
 #endif /* I915_GEM_H */
diff --git a/lib/i915/gem_create.c b/lib/i915/gem_create.c
new file mode 100644
index 000000000..fa2ca1e37
--- /dev/null
+++ b/lib/i915/gem_create.c
@@ -0,0 +1,48 @@
+// SPDX-License-Identifier: MIT
+/*
+ * Copyright © 2021 Intel Corporation
+ */
+
+#include <errno.h>
+
+#include "gem.h"
+#include "i915_drm.h"
+#include "igt_core.h"
+#include "ioctl_wrappers.h"
+
+int __gem_create(int fd, uint64_t size, uint32_t *handle)
+{
+	struct drm_i915_gem_create create = {
+		.size = size,
+	};
+	int err = 0;
+
+	if (igt_ioctl(fd, DRM_IOCTL_I915_GEM_CREATE, &create) == 0) {
+		*handle = create.handle;
+	} else {
+		err = -errno;
+		igt_assume(err != 0);
+	}
+
+	errno = 0;
+	return err;
+}
+
+/**
+ * gem_create:
+ * @fd: open i915 drm file descriptor
+ * @size: desired size of the buffer
+ *
+ * This wraps the GEM_CREATE ioctl, which allocates a new gem buffer object of
+ * @size.
+ *
+ * Returns: The file-private handle of the created buffer object
+ */
+uint32_t gem_create(int fd, uint64_t size)
+{
+	uint32_t handle;
+
+	igt_assert_eq(__gem_create(fd, size, &handle), 0);
+
+	return handle;
+}
diff --git a/lib/i915/gem_mman.c b/lib/i915/gem_mman.c
index 728f6e963..ab04cbecb 100644
--- a/lib/i915/gem_mman.c
+++ b/lib/i915/gem_mman.c
@@ -31,6 +31,7 @@
 #include "ioctl_wrappers.h"
 #include "intel_chipset.h"
 
+#include "gem.h"
 #include "gem_mman.h"
 
 #ifdef HAVE_VALGRIND
diff --git a/lib/i915/gem_mman.h b/lib/i915/gem_mman.h
index ec2899ffe..4a69b2595 100644
--- a/lib/i915/gem_mman.h
+++ b/lib/i915/gem_mman.h
@@ -42,10 +42,6 @@ void *gem_mmap__device_coherent(int fd, uint32_t handle, uint64_t offset,
 void *gem_mmap__cpu_coherent(int fd, uint32_t handle, uint64_t offset,
 			     uint64_t size, unsigned prot);
 
-#ifndef I915_GEM_DOMAIN_WC
-#define I915_GEM_DOMAIN_WC 0x80
-#endif
-
 bool gem_has_mappable_ggtt(int i915);
 void gem_require_mappable_ggtt(int i915);
 bool gem_has_mmap_offset(int fd);
diff --git a/lib/igt_draw.c b/lib/igt_draw.c
index 353467a2d..949c0d77b 100644
--- a/lib/igt_draw.c
+++ b/lib/igt_draw.c
@@ -34,6 +34,7 @@
 #include "igt_fb.h"
 #include "ioctl_wrappers.h"
 #include "i830_reg.h"
+#include "i915/gem.h"
 #include "i915/gem_mman.h"
 
 #ifndef PAGE_ALIGN
diff --git a/lib/igt_fb.c b/lib/igt_fb.c
index 422a9e061..4b9be47eb 100644
--- a/lib/igt_fb.c
+++ b/lib/igt_fb.c
@@ -33,6 +33,8 @@
 #include <pixman.h>
 
 #include "drmtest.h"
+#include "i915/gem.h"
+#include "i915/gem_mman.h"
 #include "igt_aux.h"
 #include "igt_color_encoding.h"
 #include "igt_fb.h"
@@ -45,7 +47,6 @@
 #include "ioctl_wrappers.h"
 #include "intel_batchbuffer.h"
 #include "intel_chipset.h"
-#include "i915/gem_mman.h"
 #include "intel_bufops.h"
 
 /**
diff --git a/lib/intel_batchbuffer.c b/lib/intel_batchbuffer.c
index ead890d17..8118dc945 100644
--- a/lib/intel_batchbuffer.c
+++ b/lib/intel_batchbuffer.c
@@ -35,6 +35,7 @@
 
 #include "drm.h"
 #include "drmtest.h"
+#include "i915/gem.h"
 #include "intel_batchbuffer.h"
 #include "intel_bufmgr.h"
 #include "intel_bufops.h"
diff --git a/lib/intel_bufops.c b/lib/intel_bufops.c
index a1e9ba556..a50035e40 100644
--- a/lib/intel_bufops.c
+++ b/lib/intel_bufops.c
@@ -24,6 +24,8 @@
 
 #include <sys/ioctl.h>
 #include <cairo.h>
+
+#include "i915/gem.h"
 #include "igt.h"
 #include "igt_x86.h"
 #include "intel_bufops.h"
diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
index a928f894f..45415621b 100644
--- a/lib/ioctl_wrappers.c
+++ b/lib/ioctl_wrappers.c
@@ -49,6 +49,7 @@
 
 #include "drmtest.h"
 #include "i915_drm.h"
+#include "i915/gem.h"
 #include "intel_batchbuffer.h"
 #include "intel_chipset.h"
 #include "intel_io.h"
@@ -469,43 +470,6 @@ void gem_sync(int fd, uint32_t handle)
 	errno = 0;
 }
 
-int __gem_create(int fd, uint64_t size, uint32_t *handle)
-{
-	struct drm_i915_gem_create create = {
-		.size = size,
-	};
-	int err = 0;
-
-	if (igt_ioctl(fd, DRM_IOCTL_I915_GEM_CREATE, &create) == 0) {
-		*handle = create.handle;
-	} else {
-		err = -errno;
-		igt_assume(err != 0);
-	}
-
-	errno = 0;
-	return err;
-}
-
-/**
- * gem_create:
- * @fd: open i915 drm file descriptor
- * @size: desired size of the buffer
- *
- * This wraps the GEM_CREATE ioctl, which allocates a new gem buffer object of
- * @size.
- *
- * Returns: The file-private handle of the created buffer object
- */
-uint32_t gem_create(int fd, uint64_t size)
-{
-	uint32_t handle;
-
-	igt_assert_eq(__gem_create(fd, size, &handle), 0);
-
-	return handle;
-}
-
 /**
  * __gem_execbuf:
  * @fd: open i915 drm file descriptor
diff --git a/lib/ioctl_wrappers.h b/lib/ioctl_wrappers.h
index 07879ae96..69e198419 100644
--- a/lib/ioctl_wrappers.h
+++ b/lib/ioctl_wrappers.h
@@ -75,8 +75,6 @@ int __gem_set_domain(int fd, uint32_t handle, uint32_t read, uint32_t write);
 void gem_set_domain(int fd, uint32_t handle, uint32_t read, uint32_t write);
 int gem_wait(int fd, uint32_t handle, int64_t *timeout_ns);
 void gem_sync(int fd, uint32_t handle);
-int __gem_create(int fd, uint64_t size, uint32_t *handle);
-uint32_t gem_create(int fd, uint64_t size);
 void gem_execbuf_wr(int fd, struct drm_i915_gem_execbuffer2 *execbuf);
 int __gem_execbuf_wr(int fd, struct drm_i915_gem_execbuffer2 *execbuf);
 void gem_execbuf(int fd, struct drm_i915_gem_execbuffer2 *execbuf);
diff --git a/lib/meson.build b/lib/meson.build
index 3abc42cb3..02ecef53e 100644
--- a/lib/meson.build
+++ b/lib/meson.build
@@ -3,6 +3,7 @@ lib_sources = [
 	'huc_copy.c',
 	'i915/gem.c',
 	'i915/gem_context.c',
+	'i915/gem_create.c',
 	'i915/gem_engine_topology.c',
 	'i915/gem_scheduler.c',
 	'i915/gem_submission.c',
diff --git a/tests/i915/gem_basic.c b/tests/i915/gem_basic.c
index 9f7412f2a..984bdf603 100644
--- a/tests/i915/gem_basic.c
+++ b/tests/i915/gem_basic.c
@@ -35,7 +35,9 @@
 #include <errno.h>
 #include <sys/stat.h>
 #include <sys/ioctl.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 static void
 test_bad_close(int fd)
diff --git a/tests/i915/gem_exec_basic.c b/tests/i915/gem_exec_basic.c
index 0d05819ce..31f6a2346 100644
--- a/tests/i915/gem_exec_basic.c
+++ b/tests/i915/gem_exec_basic.c
@@ -23,6 +23,8 @@
 
 #include "igt.h"
 
+#include "i915/gem.h"
+
 IGT_TEST_DESCRIPTION("Basic sanity check of execbuf-ioctl rings.");
 
 static uint32_t batch_create(int fd)
diff --git a/tests/i915/gem_fd_exhaustion.c b/tests/i915/gem_fd_exhaustion.c
index 23602b980..37c84e6c0 100644
--- a/tests/i915/gem_fd_exhaustion.c
+++ b/tests/i915/gem_fd_exhaustion.c
@@ -33,6 +33,8 @@
 #include <fcntl.h>
 #include <limits.h>
 
+#include "i915/gem.h"
+
 igt_simple_main
 {
 	int fd;
diff --git a/tests/i915/gem_fence_thrash.c b/tests/i915/gem_fence_thrash.c
index 97902890a..872dc554d 100644
--- a/tests/i915/gem_fence_thrash.c
+++ b/tests/i915/gem_fence_thrash.c
@@ -40,6 +40,7 @@
 #include <pthread.h>
 #include "drm.h"
 
+#include "i915/gem.h"
 #include "igt.h"
 #include "igt_x86.h"
 
diff --git a/tests/i915/gem_fence_upload.c b/tests/i915/gem_fence_upload.c
index 17580aff8..87ed1edff 100644
--- a/tests/i915/gem_fence_upload.c
+++ b/tests/i915/gem_fence_upload.c
@@ -37,8 +37,10 @@
 #include <errno.h>
 #include <sys/time.h>
 #include <pthread.h>
+
 #include "drm.h"
 #include "i915_drm.h"
+#include "i915/gem.h"
 
 #define OBJECT_SIZE (1024*1024) /* restricted to 1MiB alignment on i915 fences */
 
diff --git a/tests/i915/gem_flink_race.c b/tests/i915/gem_flink_race.c
index 00363c043..58431b36e 100644
--- a/tests/i915/gem_flink_race.c
+++ b/tests/i915/gem_flink_race.c
@@ -32,6 +32,7 @@
 #include <pthread.h>
 #include <errno.h>
 
+#include "i915/gem.h"
 #include "igt.h"
 #include "igt_stats.h"
 
diff --git a/tests/i915/gem_gtt_cpu_tlb.c b/tests/i915/gem_gtt_cpu_tlb.c
index 0af6a1c42..07f8f1a3c 100644
--- a/tests/i915/gem_gtt_cpu_tlb.c
+++ b/tests/i915/gem_gtt_cpu_tlb.c
@@ -40,7 +40,9 @@
 #include <sys/stat.h>
 #include <sys/time.h>
 #include <sys/ioctl.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 IGT_TEST_DESCRIPTION("Check whether gtt tlbs for cpu access are correctly"
 		     " invalidated.");
diff --git a/tests/i915/gem_gtt_speed.c b/tests/i915/gem_gtt_speed.c
index f0c1954fa..06fe70f84 100644
--- a/tests/i915/gem_gtt_speed.c
+++ b/tests/i915/gem_gtt_speed.c
@@ -39,7 +39,9 @@
 #include <sys/stat.h>
 #include <sys/ioctl.h>
 #include <sys/time.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 #define OBJECT_SIZE 16384
 
diff --git a/tests/i915/gem_lut_handle.c b/tests/i915/gem_lut_handle.c
index d0c58b8b7..f14a9bdaa 100644
--- a/tests/i915/gem_lut_handle.c
+++ b/tests/i915/gem_lut_handle.c
@@ -36,7 +36,9 @@
 #include <errno.h>
 #include <sys/stat.h>
 #include <sys/time.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 IGT_TEST_DESCRIPTION("Exercises the basic execbuffer using the handle LUT"
 		     " interface.");
diff --git a/tests/i915/gem_madvise.c b/tests/i915/gem_madvise.c
index 54c9befff..623c8b091 100644
--- a/tests/i915/gem_madvise.c
+++ b/tests/i915/gem_madvise.c
@@ -37,6 +37,7 @@
 #include <signal.h>
 
 #include "drm.h"
+#include "i915/gem.h"
 
 IGT_TEST_DESCRIPTION("Checks that the kernel reports EFAULT when trying to use"
 		     " purged bo.");
diff --git a/tests/i915/gem_mmap.c b/tests/i915/gem_mmap.c
index 60a64c134..7c36571c9 100644
--- a/tests/i915/gem_mmap.c
+++ b/tests/i915/gem_mmap.c
@@ -35,7 +35,9 @@
 #include <errno.h>
 #include <sys/stat.h>
 #include <sys/ioctl.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 #define OBJECT_SIZE 16384
 #define PAGE_SIZE 4096
diff --git a/tests/i915/gem_mmap_wc.c b/tests/i915/gem_mmap_wc.c
index cd2c8f83b..4a2192b30 100644
--- a/tests/i915/gem_mmap_wc.c
+++ b/tests/i915/gem_mmap_wc.c
@@ -36,7 +36,9 @@
 #include <errno.h>
 #include <sys/stat.h>
 #include <sys/ioctl.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 struct local_i915_gem_mmap_v2 {
 	uint32_t handle;
diff --git a/tests/i915/gem_pread.c b/tests/i915/gem_pread.c
index d4d022f19..0c4ec0d2f 100644
--- a/tests/i915/gem_pread.c
+++ b/tests/i915/gem_pread.c
@@ -41,8 +41,9 @@
 #include <sys/stat.h>
 #include <sys/time.h>
 #include <pthread.h>
-#include "drm.h"
 
+#include "drm.h"
+#include "i915/gem.h"
 #include "igt_vgem.h"
 
 #define MiB(x) ((x) * 1024 * 1024)
diff --git a/tests/i915/gem_pwrite.c b/tests/i915/gem_pwrite.c
index f76d2bc70..98bec5582 100644
--- a/tests/i915/gem_pwrite.c
+++ b/tests/i915/gem_pwrite.c
@@ -42,7 +42,7 @@
 #include <sys/time.h>
 
 #include "drm.h"
-
+#include "i915/gem.h"
 #include "igt.h"
 #include "igt_rand.h"
 #include "igt_vgem.h"
diff --git a/tests/i915/gem_readwrite.c b/tests/i915/gem_readwrite.c
index 6b2977c1c..d675810ef 100644
--- a/tests/i915/gem_readwrite.c
+++ b/tests/i915/gem_readwrite.c
@@ -35,7 +35,9 @@
 #include <errno.h>
 #include <sys/stat.h>
 #include <sys/ioctl.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 #define OBJECT_SIZE 16384
 
diff --git a/tests/i915/gem_set_tiling_vs_gtt.c b/tests/i915/gem_set_tiling_vs_gtt.c
index cee9c02ca..f09022c68 100644
--- a/tests/i915/gem_set_tiling_vs_gtt.c
+++ b/tests/i915/gem_set_tiling_vs_gtt.c
@@ -35,7 +35,9 @@
 #include <errno.h>
 #include <sys/stat.h>
 #include <sys/ioctl.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 IGT_TEST_DESCRIPTION("Check set_tiling vs gtt mmap coherency.");
 
diff --git a/tests/i915/gem_set_tiling_vs_pwrite.c b/tests/i915/gem_set_tiling_vs_pwrite.c
index 302ea24b6..771dd2e13 100644
--- a/tests/i915/gem_set_tiling_vs_pwrite.c
+++ b/tests/i915/gem_set_tiling_vs_pwrite.c
@@ -35,7 +35,9 @@
 #include <errno.h>
 #include <sys/stat.h>
 #include <sys/ioctl.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 IGT_TEST_DESCRIPTION("Check set_tiling vs pwrite coherency.");
 
diff --git a/tests/i915/gem_tiled_pread_basic.c b/tests/i915/gem_tiled_pread_basic.c
index 7cb644104..862714140 100644
--- a/tests/i915/gem_tiled_pread_basic.c
+++ b/tests/i915/gem_tiled_pread_basic.c
@@ -45,8 +45,9 @@
 #include <sys/stat.h>
 #include <sys/time.h>
 #include <sys/ioctl.h>
-#include "drm.h"
 
+#include "drm.h"
+#include "i915/gem.h"
 
 IGT_TEST_DESCRIPTION("Test pread behavior on tiled objects with respect to the"
 		     " reported swizzling value.");
diff --git a/tests/i915/gem_tiled_pread_pwrite.c b/tests/i915/gem_tiled_pread_pwrite.c
index f58048faa..b73fa1262 100644
--- a/tests/i915/gem_tiled_pread_pwrite.c
+++ b/tests/i915/gem_tiled_pread_pwrite.c
@@ -56,8 +56,8 @@
 #include <sys/time.h>
 #include <sys/ioctl.h>
 
-#include <drm.h>
-
+#include "drm.h"
+#include "i915/gem.h"
 
 IGT_TEST_DESCRIPTION("Test swizzling by testing pwrite does the inverse of"
 		     " pread.");
diff --git a/tests/i915/gem_tiled_swapping.c b/tests/i915/gem_tiled_swapping.c
index 9e0540ba9..1bcf2fa48 100644
--- a/tests/i915/gem_tiled_swapping.c
+++ b/tests/i915/gem_tiled_swapping.c
@@ -56,8 +56,8 @@
 #include <sys/ioctl.h>
 #include <pthread.h>
 
-#include <drm.h>
-
+#include "drm.h"
+#include "i915/gem.h"
 
 IGT_TEST_DESCRIPTION("Exercise swizzle code for swapping.");
 
diff --git a/tests/i915/gem_tiled_wb.c b/tests/i915/gem_tiled_wb.c
index aedec4d4b..2ba424e8d 100644
--- a/tests/i915/gem_tiled_wb.c
+++ b/tests/i915/gem_tiled_wb.c
@@ -42,7 +42,9 @@
 #include <sys/stat.h>
 #include <sys/time.h>
 #include <sys/ioctl.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 IGT_TEST_DESCRIPTION("This is a test of write-combining mmap's behavior on"
 		     " tiled objects with respect to the reported swizzling"
diff --git a/tests/i915/gem_tiled_wc.c b/tests/i915/gem_tiled_wc.c
index 3213b5d57..898aad40e 100644
--- a/tests/i915/gem_tiled_wc.c
+++ b/tests/i915/gem_tiled_wc.c
@@ -42,7 +42,9 @@
 #include <sys/stat.h>
 #include <sys/time.h>
 #include <sys/ioctl.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 #define WIDTH 512
 #define HEIGHT 512
diff --git a/tests/i915/gem_tiling_max_stride.c b/tests/i915/gem_tiling_max_stride.c
index 77f7223ba..4a273441b 100644
--- a/tests/i915/gem_tiling_max_stride.c
+++ b/tests/i915/gem_tiling_max_stride.c
@@ -36,7 +36,9 @@
 #include <limits.h>
 #include <sys/stat.h>
 #include <sys/ioctl.h>
+
 #include "drm.h"
+#include "i915/gem.h"
 
 IGT_TEST_DESCRIPTION("Check that max fence stride works.");
 
diff --git a/tests/i915/i915_module_load.c b/tests/i915/i915_module_load.c
index 189fd48b2..06522ba61 100644
--- a/tests/i915/i915_module_load.c
+++ b/tests/i915/i915_module_load.c
@@ -21,12 +21,6 @@
  * IN THE SOFTWARE.
  */
 #include "igt.h"
-#include "igt_debugfs.h"
-#include "igt_aux.h"
-#include "igt_kmod.h"
-#include "igt_sysfs.h"
-#include "igt_core.h"
-
 #include <dirent.h>
 #include <sys/utsname.h>
 #include <linux/limits.h>
@@ -36,6 +30,13 @@
 #include <sys/ioctl.h>
 #include <fcntl.h>
 
+#include "i915/gem.h"
+#include "igt_debugfs.h"
+#include "igt_aux.h"
+#include "igt_kmod.h"
+#include "igt_sysfs.h"
+#include "igt_core.h"
+
 static void store_all(int fd)
 {
 	const unsigned int gen = intel_gen(intel_get_drm_devid(fd));
diff --git a/tests/kms_available_modes_crc.c b/tests/kms_available_modes_crc.c
index 1c400bbe8..17e0f2379 100644
--- a/tests/kms_available_modes_crc.c
+++ b/tests/kms_available_modes_crc.c
@@ -24,6 +24,7 @@
 
 #include "drm_mode.h"
 #include "drm_fourcc.h"
+#include "i915/gem.h"
 #include "igt.h"
 
 IGT_TEST_DESCRIPTION("CRC test all different plane modes which kernel advertises.");
diff --git a/tests/kms_big_fb.c b/tests/kms_big_fb.c
index 8794ace08..5260176e1 100644
--- a/tests/kms_big_fb.c
+++ b/tests/kms_big_fb.c
@@ -27,6 +27,8 @@
 #include <stdio.h>
 #include <string.h>
 
+#include "i915/gem.h"
+
 IGT_TEST_DESCRIPTION("Test big framebuffers");
 
 typedef struct {
diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
index e89bddaaa..c0b024744 100644
--- a/tests/kms_ccs.c
+++ b/tests/kms_ccs.c
@@ -24,6 +24,8 @@
 
 #include "igt.h"
 
+#include "i915/gem.h"
+
 #define SDR_PLANE_BASE	3
 
 IGT_TEST_DESCRIPTION("Test render compression (RC), in which the main surface "
diff --git a/tests/kms_flip.c b/tests/kms_flip.c
index 0f0565cf6..e0d009d22 100755
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -42,6 +42,7 @@
 #include <time.h>
 #include <pthread.h>
 
+#include "i915/gem.h"
 #include "igt_stats.h"
 
 #define TEST_DPMS		(1 << 0)
diff --git a/tests/kms_getfb.c b/tests/kms_getfb.c
index c3d3c9307..8c59b8a9e 100644
--- a/tests/kms_getfb.c
+++ b/tests/kms_getfb.c
@@ -37,9 +37,10 @@
 #include <errno.h>
 #include <sys/stat.h>
 #include <sys/ioctl.h>
+
 #include "drm.h"
 #include "drm_fourcc.h"
-
+#include "i915/gem.h"
 #include "igt_device.h"
 
 IGT_TEST_DESCRIPTION("Tests GETFB and GETFB2 ioctls.");
diff --git a/tests/prime_mmap.c b/tests/prime_mmap.c
index 7c43ced85..cdf2d5149 100644
--- a/tests/prime_mmap.c
+++ b/tests/prime_mmap.c
@@ -41,11 +41,12 @@
 #include <pthread.h>
 
 #include "drm.h"
-#include "i915_drm.h"
 #include "drmtest.h"
+#include "i915_drm.h"
+#include "i915/gem.h"
+#include "i915/gem_mman.h"
 #include "igt_debugfs.h"
 #include "ioctl_wrappers.h"
-#include "i915/gem_mman.h"
 
 #define BO_SIZE (16*1024)
 
diff --git a/tests/prime_mmap_kms.c b/tests/prime_mmap_kms.c
index c06d4ab27..6cc1db87e 100644
--- a/tests/prime_mmap_kms.c
+++ b/tests/prime_mmap_kms.c
@@ -32,6 +32,8 @@
 
 #include "igt.h"
 
+#include "i915/gem.h"
+
 IGT_TEST_DESCRIPTION(
    "Efficiently sharing CPU and GPU buffers");
 
diff --git a/tests/prime_self_import.c b/tests/prime_self_import.c
index 45b70e811..4a0a98035 100644
--- a/tests/prime_self_import.c
+++ b/tests/prime_self_import.c
@@ -44,6 +44,7 @@
 #include <pthread.h>
 
 #include "drm.h"
+#include "i915/gem.h"
 
 IGT_TEST_DESCRIPTION("Check whether prime import/export works on the same"
 		     " device... but with different fds.");
diff --git a/tools/intel_reg.c b/tools/intel_reg.c
index 5f1beba4a..da1832be2 100644
--- a/tools/intel_reg.c
+++ b/tools/intel_reg.c
@@ -32,6 +32,7 @@
 #include <sys/types.h>
 #include <unistd.h>
 
+#include "i915/gem.h"
 #include "igt.h"
 #include "igt_gt.h"
 #include "intel_io.h"
-- 
2.30.0

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

^ permalink raw reply related	[flat|nested] 4+ messages in thread