drm/i915: Add hangcheck timer

We set a periodic timer to check on the GPU, resetting it every time a
batch is completed. If the timer elapses, we check acthd. If acthd
hasn't changed in two timer periods, we assume the chip is wedged.

This is implemented in such a way that it leaves the option open to
employ adaptive timer intervals in the future. One could wait until
several timer periods have elapsed before declaring the chip dead. If
the chip comes back after several periods but before the "dead"
threshold, the timer interval or dead threshold could be raised.

It is important to note that while checking for active requests, we need
to account for the fact that requests are removed from the list (i.e.
retired) in a deferred work queue handler. This means that merely
checking for an empty request_list is insufficient; the list could be
non-empty yet the GPU still idle, causing the hangcheck timer to
incorrectly mark the GPU as wedged (it took me a while to figure that
out---sigh...)

Signed-off-by: Ben Gamari <bgamari.foss@gmail.com>
Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>

1 files changed, 49 insertions, 0 deletions

diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index 6c89f2ff249..77e42e719d7 100644
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -601,6 +601,8 @@ irqreturn_t i915_driver_irq_handler(DRM_IRQ_ARGS)
                 if (iir & I915_USER_INTERRUPT) {
                         dev_priv->mm.irq_gem_seqno = i915_get_gem_seqno(dev);
                         DRM_WAKEUP(&dev_priv->irq_queue);
+                        dev_priv->hangcheck_count = 0;
+                        mod_timer(&dev_priv->hangcheck_timer, jiffies + DRM_I915_HANGCHECK_PERIOD);
                 }
 
                 if (pipea_stats & vblank_status) {
@@ -880,6 +882,53 @@ int i915_vblank_swap(struct drm_device *dev, void *data,
         return -EINVAL;
 }
 
+struct drm_i915_gem_request *i915_get_tail_request(struct drm_device *dev) {
+        drm_i915_private_t *dev_priv = dev->dev_private;
+        return list_entry(dev_priv->mm.request_list.prev, struct drm_i915_gem_request, list);
+}
+
+/**
+ * This is called when the chip hasn't reported back with completed
+ * batchbuffers in a long time. The first time this is called we simply record
+ * ACTHD. If ACTHD hasn't changed by the time the hangcheck timer elapses
+ * again, we assume the chip is wedged and try to fix it.
+ */
+void i915_hangcheck_elapsed(unsigned long data)
+{
+        struct drm_device *dev = (struct drm_device *)data;
+        drm_i915_private_t *dev_priv = dev->dev_private;
+        uint32_t acthd;
+       
+        if (!IS_I965G(dev))
+                acthd = I915_READ(ACTHD);
+        else
+                acthd = I915_READ(ACTHD_I965);
+
+        /* If all work is done then ACTHD clearly hasn't advanced. */
+        if (list_empty(&dev_priv->mm.request_list) ||
+                       i915_seqno_passed(i915_get_gem_seqno(dev), i915_get_tail_request(dev)->seqno)) {
+                dev_priv->hangcheck_count = 0;
+                return;
+        }
+
+        if (dev_priv->last_acthd == acthd && dev_priv->hangcheck_count > 0) {
+                DRM_ERROR("Hangcheck timer elapsed... GPU hung\n");
+                dev_priv->mm.wedged = true; /* Hopefully this is atomic */
+                i915_handle_error(dev);
+                return;
+        } 
+
+        /* Reset timer case chip hangs without another request being added */
+        mod_timer(&dev_priv->hangcheck_timer, jiffies + DRM_I915_HANGCHECK_PERIOD);
+
+        if (acthd != dev_priv->last_acthd)
+                dev_priv->hangcheck_count = 0;
+        else
+                dev_priv->hangcheck_count++;
+
+        dev_priv->last_acthd = acthd;
+}
+
 /* drm_dma.h hooks
 */
 static void igdng_irq_preinstall(struct drm_device *dev)