[i-g-t] i915/gem_busy: Drop i915_hangcheck_info
diff mbox series

Message ID 20200406110026.1641189-1-chris@chris-wilson.co.uk
State New
Headers show
Series
  • [i-g-t] i915/gem_busy: Drop i915_hangcheck_info
Related show

Commit Message

Chris Wilson April 6, 2020, 11 a.m. UTC
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(-)

Comments

Andi Shyti April 6, 2020, 11:16 a.m. UTC | #1
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

Patch
diff mbox series

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",