diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2020-03-09 23:14:47 +0000 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2020-03-09 23:14:47 +0000 |
commit | d6788bf0404f76b66170e18eb26c85004b5ccb25 (patch) | |
tree | 84a6679433086b6206f577a99637b97e78f09a6e /tests/i915/gem_busy.c | |
parent | b4e6a0731957934d0c20e9612dd0c863434eb6be (diff) |
i915/gem_busy: Drop redundant assert(gem_bo_busy)
The spin->handle is already validated by igt_spin_new(), and in
particular we need to be careful when using INVALID_CS that it the hang
may be detect and the request reset + completed before the gem_bo_busy
is called.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Diffstat (limited to 'tests/i915/gem_busy.c')
-rw-r--r-- | tests/i915/gem_busy.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/tests/i915/gem_busy.c b/tests/i915/gem_busy.c index 2f1b04e3..ddaca12f 100644 --- a/tests/i915/gem_busy.c +++ b/tests/i915/gem_busy.c @@ -440,9 +440,6 @@ static void basic(int fd, const struct intel_execution_engine2 *e, unsigned flag (flags & HANG ? IGT_SPIN_INVALID_CS : 0)); struct timespec tv; int timeout; - bool busy; - - busy = gem_bo_busy(fd, spin->handle); timeout = 120; if ((flags & HANG) == 0) { @@ -450,7 +447,6 @@ static void basic(int fd, const struct intel_execution_engine2 *e, unsigned flag timeout = 1; } - igt_assert(busy); memset(&tv, 0, sizeof(tv)); while (gem_bo_busy(fd, spin->handle)) { if (igt_seconds_elapsed(&tv) > timeout) { |