diff options
| author | John Harrison <John.C.Harrison@Intel.com> | 2022-01-14 10:17:05 -0800 |
|---|---|---|
| committer | Ashutosh Dixit <ashutosh.dixit@intel.com> | 2022-01-18 12:43:38 -0800 |
| commit | 64aacf9ad3a1987fb4353c5fae96519ffc70d9b1 (patch) | |
| tree | 45b3be9c3c61e05c7d958d7ecc2c40aad455db30 /tests/i915/i915_hangman.c | |
| parent | 0af1bfc9cd9c8e61ceed2e9529c4fc4f3bcf4544 (diff) | |
lib/store: Refactor common store code into helper function
A lot of tests use almost identical code for creating a batch buffer
which does a single write to memory and another is about to be added.
Instead, move the most generic version into a common helper function.
Unfortunately, the other instances are all subtly different enough to
make it not so trivial to try to use the helper. It could be done but
it is unclear if it is worth the effort at this point. This patch
proves the concept, if people like it enough then it can be extended.
v2: Fix up object address vs store offset confusion (with help from
Zbigniew K).
v3: Cope with >32bit store_offset (review feedback from Matthew Brost).
Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
Reviewed-by: Matthew Brost <matthew.brost@intel.com>
Diffstat (limited to 'tests/i915/i915_hangman.c')
| -rw-r--r-- | tests/i915/i915_hangman.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/i915/i915_hangman.c b/tests/i915/i915_hangman.c index 6656b3fc..5a0c9497 100644 --- a/tests/i915/i915_hangman.c +++ b/tests/i915/i915_hangman.c @@ -36,6 +36,7 @@ #include "i915/gem.h" #include "i915/gem_create.h" #include "igt.h" +#include "igt_store.h" #include "igt_sysfs.h" #include "igt_debugfs.h" #include "sw_sync.h" |