Commit 64486ae7 authored by Chris Wilson's avatar Chris Wilson

drm/i915: Flush idle work when changing missed-irq fault injection

In order for the missed-irq update to take effect, the device must be
idle. So when the user updates the fault injection via debugfs, idle the
device.

v2: Idle is explicitly required for setting test_irq, and good behaviour
for clearing the missed_irq.
v3: Use matching types; expanding to more than ulong rings is left as an
exercise to the reader.

Testcase: igt/drv_missed_irq
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarTvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170307155908.14576-1-chris@chris-wilson.co.uk
parent 7fba8306
...@@ -4150,6 +4150,41 @@ DEFINE_SIMPLE_ATTRIBUTE(i915_wedged_fops, ...@@ -4150,6 +4150,41 @@ DEFINE_SIMPLE_ATTRIBUTE(i915_wedged_fops,
i915_wedged_get, i915_wedged_set, i915_wedged_get, i915_wedged_set,
"%llu\n"); "%llu\n");
static int
fault_irq_set(struct drm_i915_private *i915,
unsigned long *irq,
unsigned long val)
{
int err;
err = mutex_lock_interruptible(&i915->drm.struct_mutex);
if (err)
return err;
err = i915_gem_wait_for_idle(i915,
I915_WAIT_LOCKED |
I915_WAIT_INTERRUPTIBLE);
if (err)
goto err_unlock;
/* Retire to kick idle work */
i915_gem_retire_requests(i915);
GEM_BUG_ON(i915->gt.active_requests);
*irq = val;
mutex_unlock(&i915->drm.struct_mutex);
/* Flush idle worker to disarm irq */
while (flush_delayed_work(&i915->gt.idle_work))
;
return 0;
err_unlock:
mutex_unlock(&i915->drm.struct_mutex);
return err;
}
static int static int
i915_ring_missed_irq_get(void *data, u64 *val) i915_ring_missed_irq_get(void *data, u64 *val)
{ {
...@@ -4162,18 +4197,9 @@ i915_ring_missed_irq_get(void *data, u64 *val) ...@@ -4162,18 +4197,9 @@ i915_ring_missed_irq_get(void *data, u64 *val)
static int static int
i915_ring_missed_irq_set(void *data, u64 val) i915_ring_missed_irq_set(void *data, u64 val)
{ {
struct drm_i915_private *dev_priv = data; struct drm_i915_private *i915 = data;
struct drm_device *dev = &dev_priv->drm;
int ret;
/* Lock against concurrent debugfs callers */ return fault_irq_set(i915, &i915->gpu_error.missed_irq_rings, val);
ret = mutex_lock_interruptible(&dev->struct_mutex);
if (ret)
return ret;
dev_priv->gpu_error.missed_irq_rings = val;
mutex_unlock(&dev->struct_mutex);
return 0;
} }
DEFINE_SIMPLE_ATTRIBUTE(i915_ring_missed_irq_fops, DEFINE_SIMPLE_ATTRIBUTE(i915_ring_missed_irq_fops,
...@@ -4193,13 +4219,12 @@ i915_ring_test_irq_get(void *data, u64 *val) ...@@ -4193,13 +4219,12 @@ i915_ring_test_irq_get(void *data, u64 *val)
static int static int
i915_ring_test_irq_set(void *data, u64 val) i915_ring_test_irq_set(void *data, u64 val)
{ {
struct drm_i915_private *dev_priv = data; struct drm_i915_private *i915 = data;
val &= INTEL_INFO(dev_priv)->ring_mask; val &= INTEL_INFO(i915)->ring_mask;
DRM_DEBUG_DRIVER("Masking interrupts on rings 0x%08llx\n", val); DRM_DEBUG_DRIVER("Masking interrupts on rings 0x%08llx\n", val);
dev_priv->gpu_error.test_irq_rings = val;
return 0; return fault_irq_set(i915, &i915->gpu_error.test_irq_rings, val);
} }
DEFINE_SIMPLE_ATTRIBUTE(i915_ring_test_irq_fops, DEFINE_SIMPLE_ATTRIBUTE(i915_ring_test_irq_fops,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment