[Mesa-dev] [PATCH 26/30] i965: Delete most of the old resolve interface

Jason Ekstrand jason at jlekstrand.net
Fri May 26 23:30:30 UTC 2017


---
 src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 91 +++------------------------
 src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 49 ---------------
 2 files changed, 10 insertions(+), 130 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
index c17aba9..cae8358 100644
--- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
+++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
@@ -1940,43 +1940,6 @@ intel_miptree_level_has_hiz(struct intel_mipmap_tree *mt, uint32_t level)
    return mt->level[level].has_hiz;
 }
 
-void
-intel_miptree_slice_set_needs_hiz_resolve(struct intel_mipmap_tree *mt,
-					  uint32_t level,
-					  uint32_t layer)
-{
-   if (!intel_miptree_level_has_hiz(mt, level))
-      return;
-
-   intel_resolve_map_set(&mt->hiz_map,
-			 level, layer, BLORP_HIZ_OP_HIZ_RESOLVE);
-}
-
-
-void
-intel_miptree_slice_set_needs_depth_resolve(struct intel_mipmap_tree *mt,
-                                            uint32_t level,
-                                            uint32_t layer)
-{
-   if (!intel_miptree_level_has_hiz(mt, level))
-      return;
-
-   intel_resolve_map_set(&mt->hiz_map,
-			 level, layer, BLORP_HIZ_OP_DEPTH_RESOLVE);
-}
-
-void
-intel_miptree_set_all_slices_need_depth_resolve(struct intel_mipmap_tree *mt,
-                                                uint32_t level)
-{
-   uint32_t layer;
-   uint32_t end_layer = mt->level[level].depth;
-
-   for (layer = 0; layer < end_layer; layer++) {
-      intel_miptree_slice_set_needs_depth_resolve(mt, level, layer);
-   }
-}
-
 static bool
 intel_miptree_depth_hiz_resolve(struct brw_context *brw,
                                 struct intel_mipmap_tree *mt,
@@ -2005,34 +1968,6 @@ intel_miptree_depth_hiz_resolve(struct brw_context *brw,
 }
 
 bool
-intel_miptree_slice_resolve_hiz(struct brw_context *brw,
-				struct intel_mipmap_tree *mt,
-				uint32_t level,
-				uint32_t layer)
-{
-   return intel_miptree_depth_hiz_resolve(brw, mt, level, 1, layer, 1,
-                                          BLORP_HIZ_OP_HIZ_RESOLVE);
-}
-
-bool
-intel_miptree_slice_resolve_depth(struct brw_context *brw,
-				  struct intel_mipmap_tree *mt,
-				  uint32_t level,
-				  uint32_t layer)
-{
-   return intel_miptree_depth_hiz_resolve(brw, mt, level, 1, layer, 1,
-                                          BLORP_HIZ_OP_DEPTH_RESOLVE);
-}
-
-bool
-intel_miptree_all_slices_resolve_hiz(struct brw_context *brw,
-				     struct intel_mipmap_tree *mt)
-{
-   return intel_miptree_depth_hiz_resolve(brw, mt, 0, UINT32_MAX, 0, UINT32_MAX,
-                                          BLORP_HIZ_OP_HIZ_RESOLVE);
-}
-
-bool
 intel_miptree_all_slices_resolve_depth(struct brw_context *brw,
 				       struct intel_mipmap_tree *mt)
 {
@@ -2161,7 +2096,7 @@ intel_miptree_needs_color_resolve(const struct brw_context *brw,
    return true;
 }
 
-bool
+static bool
 intel_miptree_resolve_color(struct brw_context *brw,
                             struct intel_mipmap_tree *mt,
                             uint32_t start_level, uint32_t num_levels,
@@ -2211,15 +2146,6 @@ intel_miptree_resolve_color(struct brw_context *brw,
 }
 
 void
-intel_miptree_all_slices_resolve_color(struct brw_context *brw,
-                                       struct intel_mipmap_tree *mt,
-                                       int flags)
-{
-
-   intel_miptree_resolve_color(brw, mt, 0, UINT32_MAX, 0, UINT32_MAX, flags);
-}
-
-void
 intel_miptree_prepare_access(struct brw_context *brw,
                              struct intel_mipmap_tree *mt,
                              uint32_t start_level, uint32_t num_levels,
@@ -2282,17 +2208,20 @@ intel_miptree_finish_write(struct brw_context *brw,
    } else if (mt->format == MESA_FORMAT_S_UINT8) {
       /* Nothing to do for stencil */
    } else {
+      if (!intel_miptree_level_has_hiz(mt, level))
+         return;
+
       if (written_with_aux) {
          for (unsigned a = 0; a < num_layers; a++) {
             intel_miptree_check_level_layer(mt, level, start_layer);
-            intel_miptree_slice_set_needs_depth_resolve(mt, level,
-                                                        start_layer + a);
+            intel_resolve_map_set(&mt->hiz_map, level, start_layer + a,
+                                  BLORP_HIZ_OP_DEPTH_RESOLVE);
          }
       } else {
          for (unsigned a = 0; a < num_layers; a++) {
             intel_miptree_check_level_layer(mt, level, start_layer);
-            intel_miptree_slice_set_needs_hiz_resolve(mt, level,
-                                                      start_layer + a);
+            intel_resolve_map_set(&mt->hiz_map, level, start_layer + a,
+                                  BLORP_HIZ_OP_HIZ_RESOLVE);
          }
       }
    }
@@ -2363,8 +2292,8 @@ intel_miptree_set_aux_state(struct brw_context *brw,
    } else {
       for (unsigned a = 0; a < num_layers; a++) {
          intel_miptree_check_level_layer(mt, level, start_layer);
-         intel_miptree_slice_set_needs_depth_resolve(mt, level,
-                                                     start_layer + a);
+         intel_resolve_map_set(&mt->hiz_map, level, start_layer + a,
+                               BLORP_HIZ_OP_DEPTH_RESOLVE);
       }
    }
 }
diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.h b/src/mesa/drivers/dri/i965/intel_mipmap_tree.h
index 7ddd943..3f3a78d 100644
--- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.h
+++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.h
@@ -818,44 +818,6 @@ intel_miptree_alloc_hiz(struct brw_context *brw,
 bool
 intel_miptree_level_has_hiz(struct intel_mipmap_tree *mt, uint32_t level);
 
-void
-intel_miptree_slice_set_needs_hiz_resolve(struct intel_mipmap_tree *mt,
-                                          uint32_t level,
-					  uint32_t depth);
-void
-intel_miptree_slice_set_needs_depth_resolve(struct intel_mipmap_tree *mt,
-                                            uint32_t level,
-					    uint32_t depth);
-
-void
-intel_miptree_set_all_slices_need_depth_resolve(struct intel_mipmap_tree *mt,
-                                                uint32_t level);
-
-/**
- * \return false if no resolve was needed
- */
-bool
-intel_miptree_slice_resolve_hiz(struct brw_context *brw,
-				struct intel_mipmap_tree *mt,
-				unsigned int level,
-				unsigned int depth);
-
-/**
- * \return false if no resolve was needed
- */
-bool
-intel_miptree_slice_resolve_depth(struct brw_context *brw,
-				  struct intel_mipmap_tree *mt,
-				  unsigned int level,
-				  unsigned int depth);
-
-/**
- * \return false if no resolve was needed
- */
-bool
-intel_miptree_all_slices_resolve_hiz(struct brw_context *brw,
-				     struct intel_mipmap_tree *mt);
-
 /**
  * \return false if no resolve was needed
  */
@@ -900,17 +862,6 @@ intel_miptree_used_for_rendering(const struct brw_context *brw,
  */
 #define INTEL_MIPTREE_IGNORE_CCS_E (1 << 0)
 
-bool
-intel_miptree_resolve_color(struct brw_context *brw,
-                            struct intel_mipmap_tree *mt,
-                            uint32_t start_level, uint32_t num_levels,
-                            uint32_t start_layer, uint32_t num_layers,
-                            int flags);
-
-void
-intel_miptree_all_slices_resolve_color(struct brw_context *brw,
-                                       struct intel_mipmap_tree *mt,
-                                       int flags);
 
 #define INTEL_REMAINING_LAYERS UINT32_MAX
 #define INTEL_REMAINING_LEVELS UINT32_MAX
-- 
2.5.0.400.gff86faf



More information about the mesa-dev mailing list