drm/i915: Implement fbc_status "Compressing" info for all platforms
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 6 Jun 2017 12:43:18 +0000 (15:43 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 6 Jun 2017 16:01:46 +0000 (19:01 +0300)
The number of compressed segments has been available ever since
FBC2 was introduced in g4x, it just moved from the STATUS register
into STATUS2 on IVB.

For FBC1 if we really wanted the number of compressed segments we'd
have to trawl through the tags, but in this case since the code just
uses the number of compressed segments as an indicator whether
compression has occurred we can just check the state of the
COMPRESSING and COMPRESSED bits. IIRC the hardware will try to
periodically recompress all uncompressed lines even if they haven't
changed and the COMPRESSED bit will be cleared while the compressor
is running, so just checking the COMPRESSED bit might not give us
the right answer. Hence it seems better to check for both
COMPRESSED and COMPRESSING as that should tell us that the
compressor is at least trying to do something.

While at it move the IVB+ register define to the right place, unify
the naming convention of the compressed segment count masks, and
fix up the mask for g4x.

v2: s/ILK_DPFC_STATUS2/IVB_FBC_STATUS2/ (Paulo)

Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
Cc: Gabriel Krisman Bertazi <krisman@collabora.co.uk>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Tested-by: Gabriel Krisman Bertazi <krisman@collabora.co.uk> # SNB
Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com> # ilk+
Acked-by: Paulo Zanoni <paulo.r.zanoni@intel.com> # pre-ilk
Link: http://patchwork.freedesktop.org/patch/msgid/20170606124318.31755-1-ville.syrjala@linux.intel.com
drivers/gpu/drm/i915/i915_debugfs.c
drivers/gpu/drm/i915/i915_reg.h

index 3b088685a553a4ecc0cb603391f805bd263f0608..8fdb911344b31079f263f63a0e0e13088b646687 100644 (file)
@@ -1670,12 +1670,22 @@ static int i915_fbc_status(struct seq_file *m, void *unused)
                seq_printf(m, "FBC disabled: %s\n",
                           dev_priv->fbc.no_fbc_reason);
 
-       if (intel_fbc_is_active(dev_priv) && INTEL_GEN(dev_priv) >= 7) {
-               uint32_t mask = INTEL_GEN(dev_priv) >= 8 ?
-                               BDW_FBC_COMPRESSION_MASK :
-                               IVB_FBC_COMPRESSION_MASK;
-               seq_printf(m, "Compressing: %s\n",
-                          yesno(I915_READ(FBC_STATUS2) & mask));
+       if (intel_fbc_is_active(dev_priv)) {
+               u32 mask;
+
+               if (INTEL_GEN(dev_priv) >= 8)
+                       mask = I915_READ(IVB_FBC_STATUS2) & BDW_FBC_COMP_SEG_MASK;
+               else if (INTEL_GEN(dev_priv) >= 7)
+                       mask = I915_READ(IVB_FBC_STATUS2) & IVB_FBC_COMP_SEG_MASK;
+               else if (INTEL_GEN(dev_priv) >= 5)
+                       mask = I915_READ(ILK_DPFC_STATUS) & ILK_DPFC_COMP_SEG_MASK;
+               else if (IS_G4X(dev_priv))
+                       mask = I915_READ(DPFC_STATUS) & DPFC_COMP_SEG_MASK;
+               else
+                       mask = I915_READ(FBC_STATUS) & (FBC_STAT_COMPRESSING |
+                                                       FBC_STAT_COMPRESSED);
+
+               seq_printf(m, "Compressing: %s\n", yesno(mask));
        }
 
        mutex_unlock(&dev_priv->fbc.lock);
index 1329420f4a1e984970c93f906c02b951f4177810..ac0bf2364efa1b177e998480f3bd2548a8011238 100644 (file)
@@ -2507,10 +2507,6 @@ enum skl_disp_power_wells {
 #define FBC_FENCE_OFF          _MMIO(0x3218) /* BSpec typo has 321Bh */
 #define FBC_TAG(i)             _MMIO(0x3300 + (i) * 4)
 
-#define FBC_STATUS2                    _MMIO(0x43214)
-#define  IVB_FBC_COMPRESSION_MASK      0x7ff
-#define  BDW_FBC_COMPRESSION_MASK      0xfff
-
 #define FBC_LL_SIZE            (1536)
 
 #define FBC_LLC_READ_CTRL      _MMIO(0x9044)
@@ -2539,7 +2535,7 @@ enum skl_disp_power_wells {
 #define   DPFC_INVAL_SEG_SHIFT  (16)
 #define   DPFC_INVAL_SEG_MASK  (0x07ff0000)
 #define   DPFC_COMP_SEG_SHIFT  (0)
-#define   DPFC_COMP_SEG_MASK   (0x000003ff)
+#define   DPFC_COMP_SEG_MASK   (0x000007ff)
 #define DPFC_STATUS2           _MMIO(0x3214)
 #define DPFC_FENCE_YOFF                _MMIO(0x3218)
 #define DPFC_CHICKEN           _MMIO(0x3224)
@@ -2553,6 +2549,10 @@ enum skl_disp_power_wells {
 #define   DPFC_RESERVED                (0x1FFFFF00)
 #define ILK_DPFC_RECOMP_CTL    _MMIO(0x4320c)
 #define ILK_DPFC_STATUS                _MMIO(0x43210)
+#define  ILK_DPFC_COMP_SEG_MASK        0x7ff
+#define IVB_FBC_STATUS2                _MMIO(0x43214)
+#define  IVB_FBC_COMP_SEG_MASK 0x7ff
+#define  BDW_FBC_COMP_SEG_MASK 0xfff
 #define ILK_DPFC_FENCE_YOFF    _MMIO(0x43218)
 #define ILK_DPFC_CHICKEN       _MMIO(0x43224)
 #define   ILK_DPFC_DISABLE_DUMMY0 (1<<8)