[Mesa-dev] [PATCH 01/30] i965/miptree: Delete the layered rendering resolve

Jason Ekstrand jason at jlekstrand.net
Fri Jun 16 22:41:23 UTC 2017


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.
---
 src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
index 02e74ca..3dddfae 100644
--- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
+++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
@@ -2590,20 +2590,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
-- 
2.5.0.400.gff86faf



More information about the mesa-dev mailing list