Re: [Intel-gfx] [PATCH v2] drm/i915: Beware temporary wedging when determining -EIO

2019-02-19 Thread Chris Wilson
Quoting Mika Kuoppala (2019-02-19 13:47:33) > Chris Wilson writes: > > > Quoting Chris Wilson (2019-02-19 13:38:55) > >> diff --git a/drivers/gpu/drm/i915/i915_reset.c > >> b/drivers/gpu/drm/i915/i915_reset.c > >> index 4df4c674223d..3c74b828f5be 100644 > >> --- a/drivers/gpu/drm/i915/i915_reset

Re: [Intel-gfx] [PATCH v2] drm/i915: Beware temporary wedging when determining -EIO

2019-02-19 Thread Mika Kuoppala
Chris Wilson writes: > Quoting Chris Wilson (2019-02-19 13:38:55) >> diff --git a/drivers/gpu/drm/i915/i915_reset.c >> b/drivers/gpu/drm/i915/i915_reset.c >> index 4df4c674223d..3c74b828f5be 100644 >> --- a/drivers/gpu/drm/i915/i915_reset.c >> +++ b/drivers/gpu/drm/i915/i915_reset.c >> @@ -1334,

Re: [Intel-gfx] [PATCH v2] drm/i915: Beware temporary wedging when determining -EIO

2019-02-19 Thread Chris Wilson
Quoting Chris Wilson (2019-02-19 13:38:55) > diff --git a/drivers/gpu/drm/i915/i915_reset.c > b/drivers/gpu/drm/i915/i915_reset.c > index 4df4c674223d..3c74b828f5be 100644 > --- a/drivers/gpu/drm/i915/i915_reset.c > +++ b/drivers/gpu/drm/i915/i915_reset.c > @@ -1334,6 +1334,21 @@ __releases(&i915-

[Intel-gfx] [PATCH v2] drm/i915: Beware temporary wedging when determining -EIO

2019-02-19 Thread Chris Wilson
At a few points in our uABI, we check to see if the driver is wedged and report -EIO back to the user in that case. However, as we perform the check and reset asynchronously, we may instead see the temporary wedging used to cancel inflight rendering to avoid a deadlock during reset. If we suspect t