[Intel-gfx] [PATCH] drm/i915/selftests: Skip over live context testing when wedged
Chris Wilson
chris at chris-wilson.co.uk
Fri Jul 6 06:42:00 UTC 2018
Quoting Rodrigo Vivi (2018-07-05 21:52:10)
> On Thu, Jul 05, 2018 at 03:58:45PM +0100, Chris Wilson wrote:
> > If the GPU is terminally wedged we cannot submit any requests into a
> > context, completely unfulfilling our purpose of doing so. As this
> > expectedly fails, skip over the test.
> >
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > ---
> > drivers/gpu/drm/i915/selftests/i915_gem_context.c | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_context.c b/drivers/gpu/drm/i915/selftests/i915_gem_context.c
> > index cc848ceeb3c3..0b36265a0f96 100644
> > --- a/drivers/gpu/drm/i915/selftests/i915_gem_context.c
> > +++ b/drivers/gpu/drm/i915/selftests/i915_gem_context.c
> > @@ -599,6 +599,9 @@ int i915_gem_context_live_selftests(struct drm_i915_private *dev_priv)
> > bool fake_alias = false;
> > int err;
> >
> > + if (i915_terminally_wedged(&dev_priv->gpu_error))
> > + return 0;
> > +
>
> I wonder if this could mask a real failure under the skips?
The *test* can't be run, so what failure relevant to this *test* can be
shown?
As you notice, when we get to the reset test, we do proclaim failure as
we've already demonstrated reset is bust.
-Chris
More information about the Intel-gfx
mailing list