diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2020-10-13 15:16:57 +0100 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2020-10-15 12:30:17 +0100 |
commit | 148b00789bd4fce4476f9360187c1cdb18a4e90c (patch) | |
tree | 166bc3ca836e2311b67e4a16b066034dcab5a9a7 /tests/i915/gen9_exec_parse.c | |
parent | 0c3b29498a624ad42033a219d031cb9dd475405b (diff) |
i915/gen9_exec_parse: Check oversized batch with length==0
Include the implicit eb.batch_len=0 into the mix of various offsets and
lengths.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Matthew Auld <matthew.auld@intel.com>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Diffstat (limited to 'tests/i915/gen9_exec_parse.c')
-rw-r--r-- | tests/i915/gen9_exec_parse.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/tests/i915/gen9_exec_parse.c b/tests/i915/gen9_exec_parse.c index 7ddb5bf2..087d6f35 100644 --- a/tests/i915/gen9_exec_parse.c +++ b/tests/i915/gen9_exec_parse.c @@ -628,6 +628,8 @@ static void test_bb_oversize(int i915) gem_write(i915, obj.handle, (4ull << 30) - sizeof(bbe), &bbe, sizeof(bbe)); + igt_assert_eq(__checked_execbuf(i915, &execbuf), 0); + for (int i = 13; i <= 32; i++) { igt_debug("Checking length %#llx\n", 1ull << i); @@ -638,6 +640,9 @@ static void test_bb_oversize(int i915) igt_assert_eq(__checked_execbuf(i915, &execbuf), 0); } + execbuf.batch_len = 0; + igt_assert_eq(__checked_execbuf(i915, &execbuf), 0); + gem_close(i915, obj.handle); } |