diff options
author | Karolina Drobnik <karolina.drobnik@intel.com> | 2022-05-31 12:56:54 +0200 |
---|---|---|
committer | Petri Latvala <petri.latvala@intel.com> | 2022-06-01 12:02:06 +0300 |
commit | e4dc500add255ce27c4050dcc2384da71e2482bf (patch) | |
tree | f9f1a31966b36e637e174fe1a10c7ef697ce2b16 /tests/i915/gem_softpin.c | |
parent | 5857938d4d3539deb43ecb03604b7c664b75e57d (diff) |
tests/i915: Use correct type for gen parameter
intel_gen is designed to return a big unsigned number for an unknown device.
The value is interpreted to be the newest generation possible, and the matching
configuration is used. Some functions incorrectly pass the gen value as a signed
number, meaning it becomes negative, and the oldest configuration is used
instead of the newest one. This leads to GPU hangs in some cases.
Update the definition of submit functions in gem_exec_gttfill and gem_softpin,
and has_cs_timestamp in gem_ctx_engines to use a correct type for gen parameter.
Signed-off-by: Karolina Drobnik <karolina.drobnik@intel.com>
Cc: Zbigniew KempczyĆski <zbigniew.kempczynski@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'tests/i915/gem_softpin.c')
-rw-r--r-- | tests/i915/gem_softpin.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/i915/gem_softpin.c b/tests/i915/gem_softpin.c index 489d528f..b851c90e 100644 --- a/tests/i915/gem_softpin.c +++ b/tests/i915/gem_softpin.c @@ -898,7 +898,7 @@ static void xchg_batch(void *array, unsigned int i, unsigned int j) batches[j] = tmp; } -static void submit(int fd, int gen, +static void submit(int fd, unsigned int gen, struct drm_i915_gem_execbuffer2 *eb, struct batch *batches, unsigned int count, uint64_t ahnd) |