Mesa (master): i965/miptree: Delete the layered rendering resolve

Jason Ekstrand jekstrand at kemper.freedesktop.org
Fri Jun 23 19:30:55 UTC 2017


Module: Mesa
Branch: master
Commit: a7059a764e571fed6276f6376e48c6726889f10a
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=a7059a764e571fed6276f6376e48c6726889f10a

Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date:   Thu Jun 15 22:17:13 2017 -0700

i965/miptree: Delete the layered rendering resolve

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 at chromium.org>

---

 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 7996ca7cc9..366e45c518 100644
--- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
+++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
@@ -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




More information about the mesa-commit mailing list