| Message ID | 1368451933-32571-3-git-send-email-mika.kuoppala@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Mon, May 13, 2013 at 04:32:11PM +0300, Mika Kuoppala wrote: > In preparation to track per ring progress in hangcheck, > add i915_hangcheck_ring_hung. > > v2: omit dev parameter (Ben Widawsky) > > Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com> With the updated patch 2, patches 1-3 are: Reviewed-by: Ben Widawsky <ben@bwidawsk.net> I'm on vacation, so I won't get to 4 & 5 for a few days. > --- > drivers/gpu/drm/i915/i915_irq.c | 29 +++++++++++++++++------------ > 1 file changed, 17 insertions(+), 12 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c > index 004ad34..6efe3b0 100644 > --- a/drivers/gpu/drm/i915/i915_irq.c > +++ b/drivers/gpu/drm/i915/i915_irq.c > @@ -2345,28 +2345,33 @@ static bool kick_ring(struct intel_ring_buffer *ring) > return false; > } > > +static bool i915_hangcheck_ring_hung(struct intel_ring_buffer *ring) > +{ > + if (IS_GEN2(ring->dev)) > + return false; > + > + /* Is the chip hanging on a WAIT_FOR_EVENT? > + * If so we can simply poke the RB_WAIT bit > + * and break the hang. This should work on > + * all but the second generation chipsets. > + */ > + return !kick_ring(ring); > +} > + > static bool i915_hangcheck_hung(struct drm_device *dev) > { > drm_i915_private_t *dev_priv = dev->dev_private; > > if (dev_priv->gpu_error.hangcheck_count++ > 1) { > bool hung = true; > + struct intel_ring_buffer *ring; > + int i; > > DRM_ERROR("Hangcheck timer elapsed... GPU hung\n"); > i915_handle_error(dev, true); > > - if (!IS_GEN2(dev)) { > - struct intel_ring_buffer *ring; > - int i; > - > - /* Is the chip hanging on a WAIT_FOR_EVENT? > - * If so we can simply poke the RB_WAIT bit > - * and break the hang. This should work on > - * all but the second generation chipsets. > - */ > - for_each_ring(ring, dev_priv, i) > - hung &= !kick_ring(ring); > - } > + for_each_ring(ring, dev_priv, i) > + hung &= i915_hangcheck_ring_hung(ring); > > return hung; > } > -- > 1.7.9.5 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Fri, May 24, 2013 at 11:26:42AM -0700, Ben Widawsky wrote: > On Mon, May 13, 2013 at 04:32:11PM +0300, Mika Kuoppala wrote: > > In preparation to track per ring progress in hangcheck, > > add i915_hangcheck_ring_hung. > > > > v2: omit dev parameter (Ben Widawsky) > > > > Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com> > > With the updated patch 2, patches 1-3 are: > Reviewed-by: Ben Widawsky <ben@bwidawsk.net> Merged the same to dinq, thanks. -Daniel
diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c index 004ad34..6efe3b0 100644 --- a/drivers/gpu/drm/i915/i915_irq.c +++ b/drivers/gpu/drm/i915/i915_irq.c @@ -2345,28 +2345,33 @@ static bool kick_ring(struct intel_ring_buffer *ring) return false; } +static bool i915_hangcheck_ring_hung(struct intel_ring_buffer *ring) +{ + if (IS_GEN2(ring->dev)) + return false; + + /* Is the chip hanging on a WAIT_FOR_EVENT? + * If so we can simply poke the RB_WAIT bit + * and break the hang. This should work on + * all but the second generation chipsets. + */ + return !kick_ring(ring); +} + static bool i915_hangcheck_hung(struct drm_device *dev) { drm_i915_private_t *dev_priv = dev->dev_private; if (dev_priv->gpu_error.hangcheck_count++ > 1) { bool hung = true; + struct intel_ring_buffer *ring; + int i; DRM_ERROR("Hangcheck timer elapsed... GPU hung\n"); i915_handle_error(dev, true); - if (!IS_GEN2(dev)) { - struct intel_ring_buffer *ring; - int i; - - /* Is the chip hanging on a WAIT_FOR_EVENT? - * If so we can simply poke the RB_WAIT bit - * and break the hang. This should work on - * all but the second generation chipsets. - */ - for_each_ring(ring, dev_priv, i) - hung &= !kick_ring(ring); - } + for_each_ring(ring, dev_priv, i) + hung &= i915_hangcheck_ring_hung(ring); return hung; }
In preparation to track per ring progress in hangcheck, add i915_hangcheck_ring_hung. v2: omit dev parameter (Ben Widawsky) Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com> --- drivers/gpu/drm/i915/i915_irq.c | 29 +++++++++++++++++------------ 1 file changed, 17 insertions(+), 12 deletions(-)