diff options
| author | Robert Bragg <robert@sixbynine.org> | 2016-11-08 14:02:40 +0000 |
|---|---|---|
| committer | Robert Bragg <robert@sixbynine.org> | 2016-11-17 13:06:06 +0000 |
| commit | 263c46ba1d718523d110a3dbf58bb7dffb3d7f66 (patch) | |
| tree | 00c431fe8cfc3647eb3a66cf6bef398dfb042cd6 | |
| parent | 2547fad7d5d029987483672e1dfc23ee316318a3 (diff) | |
igt/gem_exec_parse: update bitmasks test for v >=8
With v8 of the command parser (where we won't get an EINVAL for an
access violation) this updates the bitmasks test to explicitly confirm
that the command became a NOOP by reading back from where the QW_WRITE
would have otherwise landed.
Signed-off-by: Robert Bragg <robert@sixbynine.org>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
| -rw-r--r-- | tests/gem_exec_parse.c | 18 |
1 files changed, 14 insertions, 4 deletions
diff --git a/tests/gem_exec_parse.c b/tests/gem_exec_parse.c index 03e4d0e9..d5769e42 100644 --- a/tests/gem_exec_parse.c +++ b/tests/gem_exec_parse.c @@ -504,10 +504,20 @@ igt_main 0, MI_BATCH_BUFFER_END, }; - exec_batch(fd, handle, - pc, sizeof(pc), - I915_EXEC_RENDER, - -EINVAL); + if (parser_version >= 8) { + /* Expect to read back zero since the command should be + * squashed to a NOOP + */ + exec_batch_patched(fd, handle, + pc, sizeof(pc), + 8, /* patch offset, */ + 0x0); + } else { + exec_batch(fd, handle, + pc, sizeof(pc), + I915_EXEC_RENDER, + -EINVAL); + } } igt_subtest("batch-without-end") { |