| Message ID | 20200406110026.1641189-1-chris@chris-wilson.co.uk (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [i-g-t] i915/gem_busy: Drop i915_hangcheck_info | expand |
Hi Chris, On Mon, Apr 06, 2020 at 12:00:26PM +0100, Chris Wilson wrote: > All the information for hangcheck is pulled into i915_engine_info so the > separate dump is redundant. > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > Cc: Andi Shyti <andi.shyti@intel.com> Reviewed-by: Andi Shyti <andi.shyti@intel.com> now we can get rid of i915_hangcheck_info. Thanks, Andi
diff --git a/tests/i915/gem_busy.c b/tests/i915/gem_busy.c index 64e6fe6bd..5e4e2d939 100644 --- a/tests/i915/gem_busy.c +++ b/tests/i915/gem_busy.c @@ -452,7 +452,6 @@ static void basic(int fd, const struct intel_execution_engine2 *e, unsigned flag while (gem_bo_busy(fd, spin->handle)) { if (igt_seconds_elapsed(&tv) > timeout) { igt_debugfs_dump(fd, "i915_engine_info"); - igt_debugfs_dump(fd, "i915_hangcheck_info"); igt_assert_f(igt_seconds_elapsed(&tv) < timeout, "%s batch did not complete within %ds\n", flags & HANG ? "Hanging" : "Normal",
All the information for hangcheck is pulled into i915_engine_info so the separate dump is redundant. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Andi Shyti <andi.shyti@intel.com> --- tests/i915/gem_busy.c | 1 - 1 file changed, 1 deletion(-)