i965/miptree: Delete the layered rendering resolve
authorJason Ekstrand <jason.ekstrand@intel.com>
Fri, 16 Jun 2017 05:17:13 +0000 (22:17 -0700)
committerJason Ekstrand <jason.ekstrand@intel.com>
Fri, 23 Jun 2017 19:30:24 +0000 (12:30 -0700)
We never fast-clear more than the base slice (LOD 0, layer 0) anyway, so
layered rendering without a resolve is always perfectly safe.  Should
this ever change in the future, we'll have to put some sort of resolve
back in but we can cross that bridge when we come to it.

Reviewed-by: Chad Versace <chadversary@chromium.org>
src/mesa/drivers/dri/i965/intel_mipmap_tree.c

index 7996ca7..366e45c 100644 (file)
@@ -2536,20 +2536,6 @@ intel_miptree_prepare_render(struct brw_context *brw,
       intel_miptree_prepare_access(brw, mt, level, 1, start_layer, layer_count,
                                    false, false);
    }
-
-   /* For layered rendering non-compressed fast cleared buffers need to be
-    * resolved. Surface state can carry only one fast color clear value
-    * while each layer may have its own fast clear color value. For
-    * compressed buffers color value is available in the color buffer.
-    */
-   if (layer_count > 1 &&
-       !(mt->aux_disable & INTEL_AUX_DISABLE_CCS) &&
-       !intel_miptree_is_lossless_compressed(brw, mt)) {
-      assert(brw->gen >= 8);
-
-      intel_miptree_prepare_access(brw, mt, level, 1, start_layer, layer_count,
-                                   false, false);
-   }
 }
 
 void