drm/i915/xelpd: Enhanced pipe underrun reporting

XE_LPD brings enhanced underrun recovery:  the hardware can somewhat
mitigate underruns by using an interpolated replacement pixel (soft
underrun) or the previous pixel (hard underrun).  Furthermore, underruns
can now be caused downstream by the port, even if the pipe itself is
operating properly.  The interrupt register and PIPE_STATUS register
give us extra bits to recognize hard/soft underruns and determine
whether the underrun was caused by the port, so we'll use that
information to print some more descriptive errors when underruns occur.

v2:
 - Keep ICL's PIPE_STATUS defined separately from the old GMCH pipe
   status register.  (Ville)
 - Only read/clear the PIPE_STATUS register on platforms with
   display ver >= 11. (Lucas)
v3:
 - Actually enable+unmask all the new underrun interrupts, clear stale
   bits out from PIPE_STATUS before enabling the interrupts, report all
   FIFO underruns errors at once, rename a bunch of stuff to unconfuse
   vs. PIPESTAT. (Ville)

Bspec: 50335
Bspec: 50366
Cc: Lucas De Marchi <lucas.demarchi@intel.com>
Cc: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Reviewed-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210526000656.3060314-2-matthew.d.roper@intel.com

diff --git a/drivers/gpu/drm/i915/display/intel_fifo_underrun.c b/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
index 3315aa1d4d5aad0811a567aaad3e034b8f6873f4..eb841960840d30747c92682e4ec9ab655a6c8b07 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
+++ b/drivers/gpu/drm/i915/display/intel_fifo_underrun.c
@@ -185,15 +185,34 @@ static void ivb_set_fifo_underrun_reporting(struct drm_device *dev,
        }
 }
 
+static u32
+icl_pipe_status_underrun_mask(struct drm_i915_private *dev_priv)
+{
+       u32 mask = PIPE_STATUS_UNDERRUN;
+
+       if (DISPLAY_VER(dev_priv) >= 13)
+               mask |= PIPE_STATUS_SOFT_UNDERRUN_XELPD |
+                       PIPE_STATUS_HARD_UNDERRUN_XELPD |
+                       PIPE_STATUS_PORT_UNDERRUN_XELPD;
+
+       return mask;
+}
+
 static void bdw_set_fifo_underrun_reporting(struct drm_device *dev,
                                            enum pipe pipe, bool enable)
 {
        struct drm_i915_private *dev_priv = to_i915(dev);
+       u32 mask = gen8_de_pipe_underrun_mask(dev_priv);
 
-       if (enable)
-               bdw_enable_pipe_irq(dev_priv, pipe, GEN8_PIPE_FIFO_UNDERRUN);
-       else
-               bdw_disable_pipe_irq(dev_priv, pipe, GEN8_PIPE_FIFO_UNDERRUN);
+       if (enable) {
+               if (DISPLAY_VER(dev_priv) >= 11)
+                       intel_de_write(dev_priv, ICL_PIPESTATUS(pipe),
+                                      icl_pipe_status_underrun_mask(dev_priv));
+
+               bdw_enable_pipe_irq(dev_priv, pipe, mask);
+       } else {
+               bdw_disable_pipe_irq(dev_priv, pipe, mask);
+       }
 }
 
 static void ibx_set_fifo_underrun_reporting(struct drm_device *dev,
@@ -373,6 +392,7 @@ void intel_cpu_fifo_underrun_irq_handler(struct drm_i915_private *dev_priv,
                                         enum pipe pipe)
 {
        struct intel_crtc *crtc = intel_get_crtc_for_pipe(dev_priv, pipe);
+       u32 underruns = 0;
 
        /* We may be called too early in init, thanks BIOS! */
        if (crtc == NULL)
@@ -383,10 +403,35 @@ void intel_cpu_fifo_underrun_irq_handler(struct drm_i915_private *dev_priv,
            crtc->cpu_fifo_underrun_disabled)
                return;
 
+       /*
+        * Starting with display version 11, the PIPE_STAT register records
+        * whether an underrun has happened, and on XELPD+, it will also record
+        * whether the underrun was soft/hard and whether it was triggered by
+        * the downstream port logic.  We should clear these bits (which use
+        * write-1-to-clear logic) too.
+        *
+        * Note that although the IIR gives us the same underrun and soft/hard
+        * information, PIPE_STAT is the only place we can find out whether
+        * the underrun was caused by the downstream port.
+        */
+       if (DISPLAY_VER(dev_priv) >= 11) {
+               underruns = intel_de_read(dev_priv, ICL_PIPESTATUS(pipe)) &
+                       icl_pipe_status_underrun_mask(dev_priv);
+               intel_de_write(dev_priv, ICL_PIPESTATUS(pipe), underruns);
+       }
+
        if (intel_set_cpu_fifo_underrun_reporting(dev_priv, pipe, false)) {
                trace_intel_cpu_fifo_underrun(dev_priv, pipe);
-               drm_err(&dev_priv->drm, "CPU pipe %c FIFO underrun\n",
-                       pipe_name(pipe));
+
+               if (DISPLAY_VER(dev_priv) >= 11)
+                       drm_err(&dev_priv->drm, "CPU pipe %c FIFO underrun: %s%s%s%s\n",
+                               pipe_name(pipe),
+                               underruns & PIPE_STATUS_SOFT_UNDERRUN_XELPD ? "soft," : "",
+                               underruns & PIPE_STATUS_HARD_UNDERRUN_XELPD ? "hard," : "",
+                               underruns & PIPE_STATUS_PORT_UNDERRUN_XELPD ? "port," : "",
+                               underruns & PIPE_STATUS_UNDERRUN ? "transcoder," : "");
+               else
+                       drm_err(&dev_priv->drm, "CPU pipe %c FIFO underrun\n", pipe_name(pipe));
        }
 
        intel_fbc_handle_fifo_underrun_irq(dev_priv);

diff --git a/drivers/gpu/drm/i915/i915_irq.c b/drivers/gpu/drm/i915/i915_irq.c
index d4611c643446a46de9be75527e7c306e7913da79..957d401186d2e389764f5e85f56e73c32fbdac96 100644 (file)
--- a/drivers/gpu/drm/i915/i915_irq.c
+++ b/drivers/gpu/drm/i915/i915_irq.c
@@ -2425,6 +2425,17 @@ static u32 gen8_de_pipe_flip_done_mask(struct drm_i915_private *i915)
                return GEN8_PIPE_PRIMARY_FLIP_DONE;
 }
 
+u32 gen8_de_pipe_underrun_mask(struct drm_i915_private *dev_priv)
+{
+       u32 mask = GEN8_PIPE_FIFO_UNDERRUN;
+
+       if (DISPLAY_VER(dev_priv) >= 13)
+               mask |= XELPD_PIPE_SOFT_UNDERRUN |
+                       XELPD_PIPE_HARD_UNDERRUN;
+
+       return mask;
+}
+
 static irqreturn_t
 gen8_de_irq_handler(struct drm_i915_private *dev_priv, u32 master_ctl)
 {
@@ -2536,7 +2547,7 @@ gen8_de_irq_handler(struct drm_i915_private *dev_priv, u32 master_ctl)
                if (iir & GEN8_PIPE_CDCLK_CRC_DONE)
                        hsw_pipe_crc_irq_handler(dev_priv, pipe);
 
-               if (iir & GEN8_PIPE_FIFO_UNDERRUN)
+               if (iir & gen8_de_pipe_underrun_mask(dev_priv))
                        intel_cpu_fifo_underrun_irq_handler(dev_priv, pipe);
 
                fault_errors = iir & gen8_de_pipe_fault_mask(dev_priv);
@@ -3173,7 +3184,8 @@ void gen8_irq_power_well_post_enable(struct drm_i915_private *dev_priv,
                                     u8 pipe_mask)
 {
        struct intel_uncore *uncore = &dev_priv->uncore;
-       u32 extra_ier = GEN8_PIPE_VBLANK | GEN8_PIPE_FIFO_UNDERRUN |
+       u32 extra_ier = GEN8_PIPE_VBLANK |
+               gen8_de_pipe_underrun_mask(dev_priv) |
                gen8_de_pipe_flip_done_mask(dev_priv);
        enum pipe pipe;
 
@@ -3757,7 +3769,8 @@ static void gen8_de_irq_postinstall(struct drm_i915_private *dev_priv)
        }
 
        de_pipe_enables = de_pipe_masked |
-               GEN8_PIPE_VBLANK | GEN8_PIPE_FIFO_UNDERRUN |
+               GEN8_PIPE_VBLANK |
+               gen8_de_pipe_underrun_mask(dev_priv) |
                gen8_de_pipe_flip_done_mask(dev_priv);
 
        de_port_enables = de_port_masked;

diff --git a/drivers/gpu/drm/i915/i915_irq.h b/drivers/gpu/drm/i915/i915_irq.h
index 25f25cd9581813286fd0adf7da346e15573e89f3..db34d5dbe402bdc40819718f5e3ef3a49fc562f6 100644 (file)
--- a/drivers/gpu/drm/i915/i915_irq.h
+++ b/drivers/gpu/drm/i915/i915_irq.h
@@ -100,6 +100,7 @@ void gen8_irq_power_well_post_enable(struct drm_i915_private *dev_priv,
                                     u8 pipe_mask);
 void gen8_irq_power_well_pre_disable(struct drm_i915_private *dev_priv,
                                     u8 pipe_mask);
+u32 gen8_de_pipe_underrun_mask(struct drm_i915_private *dev_priv);
 
 bool intel_crtc_get_vblank_timestamp(struct drm_crtc *crtc, int *max_error,
                                     ktime_t *vblank_time, bool in_vblank_irq);

diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
index d4debb3ec4855c4ce3d149c70744611c8965d5d2..c775a6d25c6aaa40093f24be6e34af831b2f66b5 100644 (file)
--- a/drivers/gpu/drm/i915/i915_reg.h
+++ b/drivers/gpu/drm/i915/i915_reg.h
@@ -6168,6 +6168,13 @@ enum {
 #define   SKL_BOTTOM_COLOR_CSC_ENABLE  (1 << 30)
 #define SKL_BOTTOM_COLOR(pipe)         _MMIO_PIPE2(pipe, _SKL_BOTTOM_COLOR_A)
 
+#define _ICL_PIPE_A_STATUS                     0x70058
+#define ICL_PIPESTATUS(pipe)                   _MMIO_PIPE2(pipe, _ICL_PIPE_A_STATUS)
+#define   PIPE_STATUS_UNDERRUN                         REG_BIT(31)
+#define   PIPE_STATUS_SOFT_UNDERRUN_XELPD              REG_BIT(28)
+#define   PIPE_STATUS_HARD_UNDERRUN_XELPD              REG_BIT(27)
+#define   PIPE_STATUS_PORT_UNDERRUN_XELPD              REG_BIT(26)
+
 #define VLV_DPFLIPSTAT                         _MMIO(VLV_DISPLAY_BASE + 0x70028)
 #define   PIPEB_LINE_COMPARE_INT_EN            (1 << 29)
 #define   PIPEB_HLINE_INT_EN                   (1 << 28)
@@ -7849,6 +7856,8 @@ enum {
 #define  GEN8_PIPE_FIFO_UNDERRUN       (1 << 31)
 #define  GEN8_PIPE_CDCLK_CRC_ERROR     (1 << 29)
 #define  GEN8_PIPE_CDCLK_CRC_DONE      (1 << 28)
+#define  XELPD_PIPE_SOFT_UNDERRUN      (1 << 22)
+#define  XELPD_PIPE_HARD_UNDERRUN      (1 << 21)
 #define  GEN8_PIPE_CURSOR_FAULT                (1 << 10)
 #define  GEN8_PIPE_SPRITE_FAULT                (1 << 9)
 #define  GEN8_PIPE_PRIMARY_FAULT       (1 << 8)