Re: [PATCH 3/3] igt/gem_stolen: Check for available stolen memory size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 03/06/16 09:51, ankitprasad.r.sharma@xxxxxxxxx wrote:
From: Ankitprasad Sharma <ankitprasad.r.sharma@xxxxxxxxx>

Check for available stolen memory size before attempting to run
the stolen memory tests. This way we make sure that we do not
create objects from stolen memory without knowing the available size.

This checks if the kernel supports creation of stolen backed objects
before doing any operation on stolen backed objects.

Also correcting the CREATE_VERSION ioctl number in getparam ioctl,
due to kernel changes added in between.

Signed-off-by: Ankitprasad Sharma <ankitprasad.r.sharma@xxxxxxxxx>
---
  lib/ioctl_wrappers.c | 48 +++++++++++++++++++++++++++++++++++++++++++++++-
  lib/ioctl_wrappers.h |  7 +++++--
  tests/gem_create.c   |  2 +-
  tests/gem_pread.c    |  3 +++
  tests/gem_pwrite.c   |  2 ++
  tests/gem_stolen.c   | 16 ++++++++--------
  6 files changed, 66 insertions(+), 12 deletions(-)

diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c
index f224091..e6120bb 100644
--- a/lib/ioctl_wrappers.c
+++ b/lib/ioctl_wrappers.c
@@ -455,7 +455,7 @@ bool gem_create__has_stolen_support(int fd)

  	if (has_stolen_support < 0) {
  		memset(&gp, 0, sizeof(gp));
-		gp.param = 36; /* CREATE_VERSION */
+		gp.param = 38; /* CREATE_VERSION */
  		gp.value = &val;

  		/* Do we have the extended gem_create_ioctl? */
@@ -1230,6 +1230,52 @@ bool gem_has_bsd2(int fd)
  		has_bsd2 = has_param(fd, LOCAL_I915_PARAM_HAS_BSD2);
  	return has_bsd2;
  }
+
+struct local_i915_gem_get_aperture {
+	__u64 aper_size;
+	__u64 aper_available_size;
+	__u64 version;
+	__u64 map_total_size;
+	__u64 stolen_total_size;
+};
+#define DRM_I915_GEM_GET_APERTURE	0x23
+#define LOCAL_IOCTL_I915_GEM_GET_APERTURE DRM_IOR  (DRM_COMMAND_BASE + DRM_I915_GEM_GET_APERTURE, struct local_i915_gem_get_aperture)
+/**
+ * gem_total_mappable_size:
+ * @fd: open i915 drm file descriptor
+ *
+ * Feature test macro to query the kernel for the total mappable size.

Minor: it is not a feature test nor a macro.

+ *
+ * Returns: Total mappable address space size.
+ */
+uint64_t gem_total_mappable_size(int fd)
+{
+	struct local_i915_gem_get_aperture aperture;
+
+	memset(&aperture, 0, sizeof(aperture));
+	do_ioctl(fd, LOCAL_IOCTL_I915_GEM_GET_APERTURE, &aperture);
+
+	return aperture.map_total_size;
+}
+
+/**
+ * gem_total_stolen_size:
+ * @fd: open i915 drm file descriptor
+ *
+ * Feature test macro to query the kernel for the total stolen size.

Same here.

+ *
+ * Returns: Total stolen memory.
+ */
+uint64_t gem_total_stolen_size(int fd)
+{
+	struct local_i915_gem_get_aperture aperture;
+
+	memset(&aperture, 0, sizeof(aperture));
+	do_ioctl(fd, LOCAL_IOCTL_I915_GEM_GET_APERTURE, &aperture);
+
+	return aperture.stolen_total_size;
+}
+
  /**
   * gem_available_aperture_size:
   * @fd: open i915 drm file descriptor
diff --git a/lib/ioctl_wrappers.h b/lib/ioctl_wrappers.h
index f3bd23f..ae04b35 100644
--- a/lib/ioctl_wrappers.h
+++ b/lib/ioctl_wrappers.h
@@ -93,8 +93,9 @@ void *__gem_mmap__wc(int fd, uint32_t handle, uint64_t offset, uint64_t size, un
   * Test macro to query whether support for allocating objects from stolen
   * memory is available. Automatically skips through igt_require() if not.
   */
-#define gem_require_stolen_support(fd) \
-			igt_require(gem_create__has_stolen_support(fd))
+#define gem_require_stolen_support(fd, size) \
+			igt_require(gem_create__has_stolen_support(fd) && \
+				    (gem_total_stolen_size(fd) > size))

Looks like a needless complication to pass in a size, this should only be about is it supported or not. How the test uses it is even stranger, it is not likely (or even possible) some platform will have a PAGE_SIZE of stolen but not OBJECT_SIZE or something. I suggest you just drop the size param.


  /**
   * gem_require_mmap_wc:
@@ -153,6 +154,8 @@ int gem_gtt_type(int fd);
  bool gem_uses_ppgtt(int fd);
  bool gem_uses_full_ppgtt(int fd);
  int gem_available_fences(int fd);
+uint64_t gem_total_mappable_size(int fd);
+uint64_t gem_total_stolen_size(int fd);
  uint64_t gem_available_aperture_size(int fd);
  uint64_t gem_aperture_size(int fd);
  uint64_t gem_global_aperture_size(int fd);
diff --git a/tests/gem_create.c b/tests/gem_create.c
index 25f75d4..b251216 100644
--- a/tests/gem_create.c
+++ b/tests/gem_create.c
@@ -78,7 +78,7 @@ static void invalid_flag_test(int fd)
  {
  	int ret;

-	gem_require_stolen_support(fd);
+	gem_require_stolen_support(fd, PAGE_SIZE);

  	create.handle = 0;
  	create.size = PAGE_SIZE;
diff --git a/tests/gem_pread.c b/tests/gem_pread.c
index afa072d..3da8bed 100644
--- a/tests/gem_pread.c
+++ b/tests/gem_pread.c
@@ -152,6 +152,7 @@ int main(int argc, char **argv)
  	}

  	igt_subtest("stolen-normal") {
+		gem_require_stolen_support(fd, OBJECT_SIZE);
  		for (count = 1; count <= 1<<17; count <<= 1) {
  			struct timeval start, end;

@@ -167,6 +168,7 @@ int main(int argc, char **argv)
  	}
  	for (c = cache; c->level != -1; c++) {
  		igt_subtest_f("stolen-%s", c->name) {
+			gem_require_stolen_support(fd, OBJECT_SIZE);
  			gem_set_caching(fd, src_stolen, c->level);

  			for (count = 1; count <= 1<<17; count <<= 1) {
@@ -190,6 +192,7 @@ int main(int argc, char **argv)
  	 * user space buffer
  	 */
  	igt_subtest("pagefault-pread") {
+		gem_require_stolen_support(fd, LARGE_OBJECT_SIZE);
  		large_stolen = gem_create_stolen(fd, LARGE_OBJECT_SIZE);
  		stolen_nopf_user = (uint32_t *) mmap(NULL, LARGE_OBJECT_SIZE,
  						PROT_WRITE,
diff --git a/tests/gem_pwrite.c b/tests/gem_pwrite.c
index a322f91..de720c5 100644
--- a/tests/gem_pwrite.c
+++ b/tests/gem_pwrite.c
@@ -280,6 +280,7 @@ int main(int argc, char **argv)
  	}

  	igt_subtest("stolen-normal") {
+		gem_require_stolen_support(fd, OBJECT_SIZE);
  		for (count = 1; count <= 1<<17; count <<= 1) {
  			struct timeval start, end;

@@ -297,6 +298,7 @@ int main(int argc, char **argv)

  	for (c = cache; c->level != -1; c++) {
  		igt_subtest_f("stolen-%s", c->name) {
+			gem_require_stolen_support(fd, OBJECT_SIZE);
  			gem_set_caching(fd, dst, c->level);
  			for (count = 1; count <= 1<<17; count <<= 1) {
  				struct timeval start, end;
diff --git a/tests/gem_stolen.c b/tests/gem_stolen.c
index 7d329dd..541585b 100644
--- a/tests/gem_stolen.c
+++ b/tests/gem_stolen.c
@@ -105,7 +105,7 @@ static void stolen_pwrite(int fd)
  	for (i = 0; i < SIZE/DWORD_SIZE; i++)
  		buf[i] = DATA;

-	gem_require_stolen_support(fd);
+	gem_require_stolen_support(fd, SIZE);

  	handle = gem_create_stolen(fd, SIZE);

@@ -135,7 +135,7 @@ static void stolen_pread(int fd)

  	CLEAR(buf);

-	gem_require_stolen_support(fd);
+	gem_require_stolen_support(fd, SIZE);

  	handle = gem_create_stolen(fd, SIZE);

@@ -165,7 +165,7 @@ static void copy_test(int fd)
  	drm_intel_bo *src, *dest;
  	uint32_t src_handle = 0, dest_handle = 0;

-	gem_require_stolen_support(fd);
+	gem_require_stolen_support(fd, SIZE);

  	src_handle = gem_create_stolen(fd, SIZE);
  	dest_handle = gem_create_stolen(fd, SIZE);
@@ -191,7 +191,7 @@ static void verify_object_clear(int fd)
  	uint32_t *virt;
  	int i, ret;

-	gem_require_stolen_support(fd);
+	gem_require_stolen_support(fd, SIZE);

  	handle = gem_create_stolen(fd, SIZE);

@@ -215,7 +215,7 @@ static void stolen_large_obj_alloc(int fd)
  {
  	uint32_t handle = 0;

-	gem_require_stolen_support(fd);
+	gem_require_stolen_support(fd, SIZE);
  	handle = __gem_create_stolen(fd, (unsigned long long) LARGE_SIZE + 4096);
  	igt_assert(!handle);
  }
@@ -230,7 +230,7 @@ static void stolen_fill_purge_test(int fd)
  	uint32_t *virt;
  	int retained;

-	gem_require_stolen_support(fd);
+	gem_require_stolen_support(fd, SIZE);

  	/* Exhaust Stolen space */
  	do {
@@ -299,7 +299,7 @@ static void stolen_hibernate(int fd)
  	uint32_t handle[MAX_OBJECTS], src_handle;
  	uint32_t *virt;

-	gem_require_stolen_support(fd);
+	gem_require_stolen_support(fd, SIZE);

  	src_handle = gem_create(fd, SIZE);
  	src = gem_handle_to_libdrm_bo(bufmgr, fd,
@@ -388,7 +388,7 @@ stolen_no_mmap(int fd)
  	void *addr;
  	uint32_t handle = 0;

-	gem_require_stolen_support(fd);
+	gem_require_stolen_support(fd, SIZE);

  	handle = gem_create_stolen(fd, SIZE);



Regards,

Tvrtko
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux