[Intel-gfx] [PATCH i-g-t] lib/igt_gt.c : allow changes to stop_rings mode bits
tim.gore at intel.com
tim.gore at intel.com
Fri Jul 10 06:06:06 PDT 2015
From: Tim Gore <tim.gore at intel.com>
In function igt_set_stop_rings, the test
igt_assert_f(flags == 0 || current == 0, ..
will fail if we are trying to force a hang but the
STOP_RINGS_ALLOW_BAN or STOP_RINGS_ALLOW_ERROR bit is set.
With the introduction of per ring resets in the driver
(in android) these bits do not get cleared to zero when
an individual ring is reset. This causes subsequent
attempt to cause a ring hang via this function to fail,
leading to several igt tests failing (ie gem_reset_stats
subtest ban-xxx etc).
So, modify this test to look only at the bits that are
used to hang the gpu rings.
Signed-off-by: Tim Gore <tim.gore at intel.com>
---
lib/igt_gt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/igt_gt.c b/lib/igt_gt.c
index 8e5e076..12c56fa 100644
--- a/lib/igt_gt.c
+++ b/lib/igt_gt.c
@@ -345,8 +345,8 @@ void igt_set_stop_rings(enum stop_ring_flags flags)
STOP_RING_ALLOW_ERRORS)) == 0);
current = igt_get_stop_rings();
- igt_assert_f(flags == 0 || current == 0,
- "previous i915_ring_stop is still 0x%x\n", current);
+ igt_assert_f( (flags & STOP_RING_ALL) == 0 || (current & STOP_RING_ALL) == 0,
+ "previous i915_ring_stop is still 0x%x\n", current);
stop_rings_write(flags);
current = igt_get_stop_rings();
--
1.9.1
More information about the Intel-gfx
mailing list