Thread (26 messages) 26 messages, 6 authors, 2021-06-10

Re: [igt-dev] [PATCH i-g-t 08/11] tests/kms_addfb_basic: Add invalid buffer object test for discrete

From: Petri Latvala <hidden>
Date: 2021-06-10 12:44:44

On Thu, Jun 10, 2021 at 03:30:46PM +0300, Ruhl, Michael J wrote:
quoted
-----Original Message-----
From: Patnana, Venkata Sai <redacted>
Sent: Wednesday, June 9, 2021 8:18 AM
To: igt-dev@lists.freedesktop.org
Cc: Patnana, Venkata Sai <redacted>; Mohammed
Khajapasha [off-list ref]; Latvala; Latvala, Petri
[off-list ref]; Ruhl, Michael J [off-list ref]
Subject: [PATCH i-g-t 08/11] tests/kms_addfb_basic: Add invalid buffer object
test for discrete

From: Mohammed Khajapasha <redacted>

Add invalid buffer object test case for discrete, to validate
IOCTL_MODE_ADDFB2 ioctl for LMEM, the framebuffer backing
gem objects should be from local memory for discrete.

Cc: Latvala, Petri <redacted>
Cc: Michael J. Ruhl <redacted>
Signed-off-by: Mohammed Khajapasha
[off-list ref]
---
tests/kms_addfb_basic.c | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/tests/kms_addfb_basic.c b/tests/kms_addfb_basic.c
index 1fb18d668f..aac74a7568 100644
--- a/tests/kms_addfb_basic.c
+++ b/tests/kms_addfb_basic.c
@@ -40,6 +40,7 @@
#include "igt_rand.h"
#include "igt_device.h"
+#include "i915/intel_memory_region.h"

uint32_t gem_bo;
uint32_t gem_bo_small;
@@ -146,6 +147,21 @@ static void invalid_tests(int fd)
		igt_assert(f.modifier[0] == 0);
	}

+	igt_subtest("invalid-smem-bo-on-discrete") {
+		int devid;
+		uint32_t handle, stride;
+		uint64_t size;
+
+		igt_require_intel(fd);
+		devid = intel_get_drm_devid(fd);
This will only work on if the discrete card is the only card on the system.

Is there a better way to:

1) identify discrete cards and/or
2) specifically select a discrete card for testing?
The test does not select the device to use. The admin _running_ the
test does that, with IGT_DEVICE.

With that done, mapping of results will be

non-intel: skip
intel integrated: skip
intel discrete: actually tested

which looks correct. We're trying to test what happens specifically
if the used device is intel discrete, skip otherwise.


-- 
Petri Latvala
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help