summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2020-09-28 21:51:02 +0100
committerChris Wilson <chris@chris-wilson.co.uk>2020-09-29 14:25:47 +0100
commitcbaf441899f3b4f36cca5996aa6a69e7399b2dbd (patch)
tree56f42b2b3c5810cf8844a2ceca9777ea2b003dd7
parentd089599869755e80accd16d3018b5f9ace139588 (diff)
i915/gen9_exec_parse: Check parsing of large objects
Simply check that we support parsing of batches as large as the uAPI allows. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
-rw-r--r--tests/i915/gen9_exec_parse.c80
1 files changed, 80 insertions, 0 deletions
diff --git a/tests/i915/gen9_exec_parse.c b/tests/i915/gen9_exec_parse.c
index 8cd82f568..7ddb5bf2b 100644
--- a/tests/i915/gen9_exec_parse.c
+++ b/tests/i915/gen9_exec_parse.c
@@ -566,6 +566,81 @@ static void test_bb_start(const int i915, const uint32_t handle, int test)
gem_close(i915, target_bo);
}
+static void test_bb_large(int i915)
+{
+ const uint32_t bbe = MI_BATCH_BUFFER_END;
+ static const uint32_t sizes[] = {
+ (1ull << 30) - 4096,
+ (1ull << 30) + 4096,
+ (2ull << 30) - 4096,
+ (2ull << 30) + 4096,
+ (3ull << 30) - 4096,
+ (3ull << 30) + 4096,
+ (4ull << 30) - 4096 /* upper bound of execbuf2 uAPI */
+ };
+ struct drm_i915_gem_exec_object2 obj = {};
+ struct drm_i915_gem_execbuffer2 execbuf = {
+ .buffers_ptr = to_user_pointer(&obj),
+ .buffer_count = 1,
+ .flags = I915_EXEC_BLT,
+ };
+ uint64_t required, total;
+ int i;
+
+ for (i = 0; i < ARRAY_SIZE(sizes); i++) {
+ if (!__intel_check_memory(2, sizes[i], CHECK_RAM,
+ &required, &total))
+ break;
+
+ igt_debug("Using object size %#x\n", sizes[i]);
+ obj.handle = gem_create(i915, sizes[i]),
+ gem_write(i915, obj.handle, sizes[i] - 64, &bbe, sizeof(bbe));
+
+ execbuf.batch_start_offset = 0;
+ igt_assert_eq(__checked_execbuf(i915, &execbuf), 0);
+
+ execbuf.batch_start_offset = sizes[i] - 64;
+ igt_assert_eq(__checked_execbuf(i915, &execbuf), 0);
+
+ gem_close(i915, obj.handle);
+ }
+
+ igt_require_f(i > 0 && sizes[i - 1] > 1ull << 31,
+ "Insufficient free memory, require at least %'"PRIu64"MiB but only have %'"PRIu64"MiB available\n",
+ required >> 20, total >> 20);
+}
+
+static void test_bb_oversize(int i915)
+{
+ const uint32_t bbe = MI_BATCH_BUFFER_END;
+ struct drm_i915_gem_exec_object2 obj = {
+ .handle = gem_create(i915, 8ull << 30),
+ .flags = EXEC_OBJECT_SUPPORTS_48B_ADDRESS
+ };
+ struct drm_i915_gem_execbuffer2 execbuf = {
+ .buffers_ptr = to_user_pointer(&obj),
+ .buffer_count = 1,
+ .batch_start_offset = (4ull << 30) - 4096,
+ .flags = I915_EXEC_BLT,
+ };
+
+ intel_require_memory(2, 8ull << 30, CHECK_RAM);
+ gem_write(i915, obj.handle, (4ull << 30) - sizeof(bbe),
+ &bbe, sizeof(bbe));
+
+ for (int i = 13; i <= 32; i++) {
+ igt_debug("Checking length %#llx\n", 1ull << i);
+
+ execbuf.batch_len = (1ull << i) - 4096;
+ igt_assert_eq(__checked_execbuf(i915, &execbuf), 0);
+
+ execbuf.batch_len = (1ull << i) + 4096; /* will wrap */
+ igt_assert_eq(__checked_execbuf(i915, &execbuf), 0);
+ }
+
+ gem_close(i915, obj.handle);
+}
+
static void test_bb_chained(const int i915, const uint32_t handle)
{
const uint32_t batch[] = {
@@ -1053,6 +1128,11 @@ igt_main
igt_subtest("bb-start-far")
test_bb_start(i915, handle, BB_START_FAR);
+ igt_subtest("bb-large")
+ test_bb_large(i915);
+ igt_subtest("bb-oversize")
+ test_bb_oversize(i915);
+
igt_fixture {
igt_stop_hang_detector();
gem_close(i915, handle);