drm/i915: Update the breadcrumb interrupt counter before enabling
authorChris Wilson <chris@chris-wilson.co.uk>
Sun, 24 Jul 2016 09:10:21 +0000 (10:10 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Sun, 24 Jul 2016 09:57:20 +0000 (10:57 +0100)
In order to close a race with a long running hangcheck comparing a stale
interrupt counter with a just started waiter, we need to first bump the
counter as we start the fresh wait.

References: https://bugs.freedesktop.org/show_bug.cgi?id=96974
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1469351421-13493-2-git-send-email-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/intel_breadcrumbs.c

index f0b56e3f4abe0bf73972108056b6403d1b02a7a1..d893ccdd62accb5b98871d584b5ec675a50498cf 100644 (file)
@@ -51,6 +51,13 @@ static void irq_enable(struct intel_engine_cs *engine)
         */
        engine->breadcrumbs.irq_posted = true;
 
+       /* Make sure the current hangcheck doesn't falsely accuse a just
+        * started irq handler from missing an interrupt (because the
+        * interrupt count still matches the stale value from when
+        * the irq handler was disabled, many hangchecks ago).
+        */
+       engine->breadcrumbs.irq_wakeups++;
+
        spin_lock_irq(&engine->i915->irq_lock);
        engine->irq_enable(engine);
        spin_unlock_irq(&engine->i915->irq_lock);