diff options
| author | John Harrison <John.C.Harrison@Intel.com> | 2021-11-03 10:04:42 -0700 |
|---|---|---|
| committer | Ashutosh Dixit <ashutosh.dixit@intel.com> | 2021-11-03 18:50:25 -0700 |
| commit | 3c0addebebd2ffb0bef5eece92a0a4dcd32c0b9e (patch) | |
| tree | 990e0ee3bb706ac66445689ed9e6fd4b8c620b41 /tests/i915/gem_exec_capture.c | |
| parent | d31f96a54900b7375a903f49813dc969dd168280 (diff) | |
tests/i915/gem_exec_capture: Remove pointless assert
The 'many' test ended with an 'assert(count)', presumably meaning to
ensure that some objects were actually captured. However, 'count' is
the number of objects created not how many were captured. Plus, there
is already a 'require(count > 1)' at the start and count is invarient
so the final assert is basically pointless.
General concensus appears to be that the test should not fail
irrespective of how many blobs are captured as low memory situations
could cause the capture to be abbreviated. So just remove the
pointless assert completely.
Signed-off-by: John Harrison <John.C.Harrison@Intel.com>
Reviewed-by: Matthew Brost <matthew.brost@intel.com>
Diffstat (limited to 'tests/i915/gem_exec_capture.c')
| -rw-r--r-- | tests/i915/gem_exec_capture.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/tests/i915/gem_exec_capture.c b/tests/i915/gem_exec_capture.c index 7e0a8b8a..53649cdb 100644 --- a/tests/i915/gem_exec_capture.c +++ b/tests/i915/gem_exec_capture.c @@ -524,7 +524,6 @@ static void many(int fd, int dir, uint64_t size, unsigned int flags) } igt_info("Captured %lu %"PRId64"-blobs out of a total of %lu\n", blobs, size >> 12, count); - igt_assert(count); free(error); free(offsets); |