isl: Don't force linear for 1d surfaces in gen7_filter_tiling()
authorChad Versace <chad.versace@intel.com>
Tue, 5 Jan 2016 19:35:44 +0000 (11:35 -0800)
committerChad Versace <chad.versace@intel.com>
Tue, 5 Jan 2016 19:37:32 +0000 (11:37 -0800)
gen7_filter_tiling() should filter out only tiling flags that are
incompatible with the surface. It shouldn't make performance decisions,
such as forcing linear for 1D; that's the role of the caller.

src/isl/isl_gen7.c

index 4484abb8191179616c52d74db2b2257f33c124b4..7064e852e654d71b0889da24f3692e2043007fa6 100644 (file)
@@ -265,14 +265,6 @@ gen7_filter_tiling(const struct isl_device *dev,
       *flags &= (ISL_TILING_ANY_Y_MASK | ISL_TILING_W_BIT);
    }
 
-   /* For 1D surfaces, use linear when possible. 1D surfaces (array and
-    * non-array) do not benefit from tiling. In fact, it leads to less
-    * efficient use of memory due to tile alignment.
-    */
-   if (info->dim == ISL_SURF_DIM_1D && (*flags & ISL_TILING_LINEAR_BIT)) {
-      *flags = ISL_TILING_LINEAR_BIT;
-   }
-
    /* workaround */
    if (ISL_DEV_GEN(dev) == 7 &&
        gen7_format_needs_valign2(dev, info->format) &&