[Intel-gfx] [igt-dev] [PATCH i-g-t] i915/gem_eio: Check for allow-hang prior to issuing a reset

Antonio Argenziano antonio.argenziano at intel.com
Tue Jan 29 23:36:38 UTC 2019



On 27/01/19 04:49, Chris Wilson wrote:
> Check that we are allowed to hang/reset the GPU before we actually do so
> for the first time.
> 
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> ---
>   tests/i915/gem_eio.c | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/i915/gem_eio.c b/tests/i915/gem_eio.c
> index 5250a414c..09059c311 100644
> --- a/tests/i915/gem_eio.c
> +++ b/tests/i915/gem_eio.c
> @@ -745,14 +745,14 @@ igt_main
>   		fd = drm_open_driver(DRIVER_INTEL);
>   		igt_device_drop_master(fd);
>   
> -		igt_require(i915_reset_control(true));
> -		igt_force_gpu_reset(fd);
> -		igt_install_exit_handler(exit_handler);
> -
>   		gem_submission_print_method(fd);
>   		igt_require_gem(fd);
>   
>   		igt_allow_hang(fd, 0, 0);
> +
> +		igt_require(i915_reset_control(true));

Don't we do this already in allow_hang?

Antnoio

> +		igt_force_gpu_reset(fd);
> +		igt_install_exit_handler(exit_handler);
>   	}
>   
>   	igt_subtest("throttle")
> 


More information about the Intel-gfx mailing list