i965/miptree: Drop an if case from retile_as_linear
authorNanley Chery <nanley.g.chery@intel.com>
Wed, 23 May 2018 22:50:14 +0000 (15:50 -0700)
committerNanley Chery <nanley.g.chery@intel.com>
Fri, 13 Jul 2018 02:16:30 +0000 (19:16 -0700)
Drop an if statement whose predicate never evaluates to true. row_pitch
belongs to a surface with non-linear tiling. According to
isl_calc_tiled_min_row_pitch, the pitch is a multiple of the tile width.
By looking at isl_tiling_get_info, we see that non-linear tilings have
widths greater than or equal to 128B.

Cc: <mesa-stable@lists.freedesktop.org>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
src/mesa/drivers/dri/i965/intel_mipmap_tree.c

index 4a61cee08b5ec521141939943089832b21d1cbbb..0552e092e8e72943ff6b84c4df1568526aa20483 100644 (file)
@@ -519,10 +519,6 @@ need_to_retile_as_linear(struct brw_context *brw, unsigned row_pitch,
    if (tiling == ISL_TILING_LINEAR)
       return false;
 
-    /* If the width is much smaller than a tile, don't bother tiling. */
-   if (row_pitch < 64)
-      return true;
-
    if (ALIGN(row_pitch, 512) >= 32768) {
       perf_debug("row pitch %u too large to blit, falling back to untiled",
                  row_pitch);