[Mesa-dev] [PATCH 6/9] intel/blorp: Make blorp_ccs_ambiguate just an internal helper

Jason Ekstrand jason at jlekstrand.net
Tue May 15 22:28:09 UTC 2018


Now that anv uses blorp_ccs_op for everything, we no longer need to
expose the ccs_ambiguate function directly.  It's much better tucked
away as an implementation detail.
---
 src/intel/blorp/blorp.h       |  5 -----
 src/intel/blorp/blorp_clear.c | 21 ++++++++++-----------
 2 files changed, 10 insertions(+), 16 deletions(-)

diff --git a/src/intel/blorp/blorp.h b/src/intel/blorp/blorp.h
index 8c775bf..e27ea7e 100644
--- a/src/intel/blorp/blorp.h
+++ b/src/intel/blorp/blorp.h
@@ -208,11 +208,6 @@ blorp_ccs_op(struct blorp_batch *batch,
              enum isl_aux_op ccs_op);
 
 void
-blorp_ccs_ambiguate(struct blorp_batch *batch,
-                    struct blorp_surf *surf,
-                    uint32_t level, uint32_t layer);
-
-void
 blorp_mcs_partial_resolve(struct blorp_batch *batch,
                           struct blorp_surf *surf,
                           enum isl_format format,
diff --git a/src/intel/blorp/blorp_clear.c b/src/intel/blorp/blorp_clear.c
index 6f5549f..39bc0c6 100644
--- a/src/intel/blorp/blorp_clear.c
+++ b/src/intel/blorp/blorp_clear.c
@@ -814,6 +814,11 @@ blorp_clear_attachments(struct blorp_batch *batch,
    batch->blorp->exec(batch, &params);
 }
 
+static void
+blorp_legacy_ccs_ambiguate(struct blorp_batch *batch,
+                           struct blorp_surf *surf,
+                           uint32_t level, uint32_t layer);
+
 void
 blorp_ccs_op(struct blorp_batch *batch,
              struct blorp_surf *surf, uint32_t level,
@@ -835,7 +840,7 @@ blorp_ccs_op(struct blorp_batch *batch,
        * mess to another function.
        */
       for (uint32_t a = 0; a < num_layers; a++)
-         blorp_ccs_ambiguate(batch, surf, level, start_layer + a);
+         blorp_legacy_ccs_ambiguate(batch, surf, level, start_layer + a);
       return;
    }
 
@@ -1022,17 +1027,11 @@ blorp_mcs_partial_resolve(struct blorp_batch *batch,
  * for a given layer/level of a surface to 0x0 which is the "uncompressed"
  * state which tells the sampler to go look at the main surface.
  */
-void
-blorp_ccs_ambiguate(struct blorp_batch *batch,
-                    struct blorp_surf *surf,
-                    uint32_t level, uint32_t layer)
+static void
+blorp_legacy_ccs_ambiguate(struct blorp_batch *batch,
+                           struct blorp_surf *surf,
+                           uint32_t level, uint32_t layer)
 {
-   if (ISL_DEV_GEN(batch->blorp->isl_dev) >= 10) {
-      /* On gen10 and above, we have a hardware resolve op for this */
-      return blorp_ccs_op(batch, surf, level, layer, 1,
-                          surf->surf->format, ISL_AUX_OP_AMBIGUATE);
-   }
-
    struct blorp_params params;
    blorp_params_init(&params);
 
-- 
2.5.0.400.gff86faf



More information about the mesa-dev mailing list