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

Chris Wilson chris at chris-wilson.co.uk
Tue Jan 29 23:41:41 UTC 2019


Quoting Antonio Argenziano (2019-01-29 23:36:38)
> 
> 
> 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?

Not quite. There we set reset=INT_MAX (aka 2), here it's just reset=1.

Not much in it tbh; this allows us to check the pattern used by the
tests, but we explicitly require reset=1 in order to trigger the
wedging/EIO.
-Chris


More information about the igt-dev mailing list