drm/i915: drop DPF code for gen8+
authorDaniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Wed, 9 Jan 2019 21:31:47 +0000 (13:31 -0800)
committerChris Wilson <chris@chris-wilson.co.uk>
Thu, 10 Jan 2019 09:43:33 +0000 (09:43 +0000)
The only gen8+ platform that has the feature is BDW, but we don't define
the feature flag on any BDW platform and we only have partial support in
the gen8 path (irq enabling code, but no handler).
The only thing we could do in the irq handler is report the error
to userspace, but no one asked/cared about that since BDW was
released so it is relatively safe to assume that even if we added the
message no one would look at it. Just drop the dead code from the
driver instead.

Signed-off-by: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20190109213147.16851-1-daniele.ceraolospurio@intel.com
drivers/gpu/drm/i915/i915_irq.c
drivers/gpu/drm/i915/intel_lrc.c

index 3fcb3aad2e543fc40893d6a700b4847a3085f96f..288b0662f7b7b3f63a0f6f6d582d7e5d43c4b36f 100644 (file)
@@ -4172,9 +4172,6 @@ static void gen8_gt_irq_postinstall(struct drm_i915_private *dev_priv)
                        GT_CONTEXT_SWITCH_INTERRUPT << GEN8_VECS_IRQ_SHIFT
                };
 
-       if (HAS_L3_DPF(dev_priv))
-               gt_interrupts[0] |= GT_RENDER_L3_PARITY_ERROR_INTERRUPT;
-
        dev_priv->pm_ier = 0x0;
        dev_priv->pm_imr = ~dev_priv->pm_ier;
        GEN8_IRQ_INIT_NDX(GT, 0, ~gt_interrupts[0], gt_interrupts[0]);
index 59115d3944c955c73fbc2513596f333826d92349..72ab89151ab987c6b1c32f5d70aceeb09ad88bf6 100644 (file)
@@ -2268,14 +2268,10 @@ static int logical_ring_init(struct intel_engine_cs *engine)
 
 int logical_render_ring_init(struct intel_engine_cs *engine)
 {
-       struct drm_i915_private *dev_priv = engine->i915;
        int ret;
 
        logical_ring_setup(engine);
 
-       if (HAS_L3_DPF(dev_priv))
-               engine->irq_keep_mask |= GT_RENDER_L3_PARITY_ERROR_INTERRUPT;
-
        /* Override some for render ring. */
        engine->init_context = gen8_init_rcs_context;
        engine->emit_flush = gen8_emit_flush_render;