[igt-dev] [Intel-gfx] [PATCH i-g-t] i915/perf_pmu: Check for hangs allowed

Tvrtko Ursulin tvrtko.ursulin at linux.intel.com
Mon Jan 28 11:00:43 UTC 2019


On 28/01/2019 10:18, Chris Wilson wrote:
> Check we can reset the GPU before running the reset test.
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> ---
> Different meaning of flags, it's not the ring id!
> ---
>   tests/perf_pmu.c | 7 ++++++-
>   1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/perf_pmu.c b/tests/perf_pmu.c
> index 21292bf3a..c9881e59f 100644
> --- a/tests/perf_pmu.c
> +++ b/tests/perf_pmu.c
> @@ -1812,8 +1812,13 @@ igt_main
>   					accuracy(fd, e, pct[i], 10);
>   			}
>   
> -			igt_subtest_f("busy-hang-%s", e->name)
> +			igt_subtest_f("busy-hang-%s", e->name) {
> +				igt_hang_t hang = igt_allow_hang(fd, 0, 0);
> +
>   				single(fd, e, TEST_BUSY | FLAG_HANG);
> +
> +				igt_disallow_hang(fd, hang);
> +			}
>   		}
>   
>   		/**
> 

So all IGTs which trigger hangs/resets should call this?

Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>

Regards,

Tvrtko


More information about the igt-dev mailing list