diff options
author | Matt Roper <matthew.d.roper@intel.com> | 2024-03-28 10:41:39 -0700 |
---|---|---|
committer | Matt Roper <matthew.d.roper@intel.com> | 2024-03-29 08:29:58 -0700 |
commit | 7d7be3a56e299d9ad14ac2c6535a7e14b4cfd4df (patch) | |
tree | e9afa89cc576a3610e45fb8628d2949def78e1a4 /lib/igt_dummyload.c | |
parent | 5ec1ff6da3535cf80fd4e1844867d75c481ef86a (diff) |
tests/intel: Use MI_ARB_CHECK definition more consistently
Even though we have MI_ARB_CHECK defined in our GPU instruction header,
several of our tests and test libraries seem to be using alternate
definitions and/or magic numbers to emit this instruction, which makes
grep'ing the code for it challenging (and makes the behavior of the
tests themselves less obvious in some cases). Try to use the standard
definition more consistently everywhere.
Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
Link: https://lore.kernel.org/r/20240328174139.1533658-1-matthew.d.roper@intel.com
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Diffstat (limited to 'lib/igt_dummyload.c')
-rw-r--r-- | lib/igt_dummyload.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/lib/igt_dummyload.c b/lib/igt_dummyload.c index d3cee9154..2e842929b 100644 --- a/lib/igt_dummyload.c +++ b/lib/igt_dummyload.c @@ -64,8 +64,6 @@ #define ENGINE_MASK (I915_EXEC_RING_MASK | I915_EXEC_BSD_MASK) -#define MI_ARB_CHK (0x5 << 23) - static const int BATCH_SIZE = 4096; static const int LOOP_START_OFFSET = 64; @@ -286,7 +284,7 @@ emit_recursive_batch(igt_spin_t *spin, /* Allow ourselves to be preempted */ if (!(opts->flags & IGT_SPIN_NO_PREEMPTION)) - *cs++ = MI_ARB_CHK; + *cs++ = MI_ARB_CHECK; if (opts->flags & IGT_SPIN_INVALID_CS) { igt_assert(opts->ctx); if (!gem_engine_has_cmdparser(fd, &opts->ctx->cfg, opts->engine)) |