diff options
author | Eric Anholt <eric@anholt.net> | 2012-05-17 22:03:32 -0700 |
---|---|---|
committer | Eric Anholt <eric@anholt.net> | 2012-05-23 10:40:11 -0700 |
commit | 68216f35814ab8d292f37b8c0fa0a5f181b7f20d (patch) | |
tree | 67687b30ca5c4566da8b81d05af8ceb6fac0d8d3 /src/mesa/drivers/dri/i965/brw_clear.c | |
parent | 5b248e598293e7c21257d35904294da4c8f2da58 (diff) | |
download | external_mesa3d-68216f35814ab8d292f37b8c0fa0a5f181b7f20d.zip external_mesa3d-68216f35814ab8d292f37b8c0fa0a5f181b7f20d.tar.gz external_mesa3d-68216f35814ab8d292f37b8c0fa0a5f181b7f20d.tar.bz2 |
i965/gen6+: Add support for fast depth clears.
Improves citybench high-res performance 3.0% +- 0.4%, n=10. Improves
Lightsmark 1024x768 performance 0.74% +/- 0.20% (n=78). No
significant difference on openarena (n=5, didn't fast clear) or nexuiz
(n=3).
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_clear.c')
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_clear.c | 130 |
1 files changed, 130 insertions, 0 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_clear.c b/src/mesa/drivers/dri/i965/brw_clear.c index d171b7c..31c2e45 100644 --- a/src/mesa/drivers/dri/i965/brw_clear.c +++ b/src/mesa/drivers/dri/i965/brw_clear.c @@ -32,10 +32,12 @@ #include "swrast/swrast.h" #include "drivers/common/meta.h" +#include "intel_batchbuffer.h" #include "intel_context.h" #include "intel_blit.h" #include "intel_clear.h" #include "intel_fbo.h" +#include "intel_mipmap_tree.h" #include "intel_regions.h" #define FILE_DEBUG_FLAG DEBUG_BLIT @@ -75,6 +77,125 @@ debug_mask(const char *name, GLbitfield mask) } /** + * Implements fast depth clears on gen6+. + * + * Fast clears basically work by setting a flag in each of the subspans + * represented in the HiZ buffer that says "When you need the depth values for + * this subspan, it's the hardware's current clear value." Then later rendering + * can just use the static clear value instead of referencing memory. + * + * The tricky part of the implementation is that you have to have the clear + * value that was used on the depth buffer in place for all further rendering, + * at least until a resolve to the real depth buffer happens. + */ +static bool +brw_fast_clear_depth(struct gl_context *ctx) +{ + struct intel_context *intel = intel_context(ctx); + struct gl_framebuffer *fb = ctx->DrawBuffer; + struct intel_renderbuffer *depth_irb = + intel_get_renderbuffer(fb, BUFFER_DEPTH); + struct intel_mipmap_tree *mt = depth_irb->mt; + + if (intel->gen < 6) + return false; + + if (!mt->hiz_mt) + return false; + + /* We only handle full buffer clears -- otherwise you'd have to track whether + * a previous clear had happened at a different clear value and resolve it + * first. + */ + if (ctx->Scissor.Enabled) + return false; + + /* The rendered area has to be 8x4 samples, not resolved pixels, so we look + * at the miptree slice dimensions instead of renderbuffer size. + */ + if (mt->level[depth_irb->mt_level].width % 8 != 0 || + mt->level[depth_irb->mt_level].height % 4 != 0) { + return false; + } + + uint32_t depth_clear_value; + switch (mt->format) { + case MESA_FORMAT_Z32_FLOAT_X24S8: + case MESA_FORMAT_S8_Z24: + /* From the Sandy Bridge PRM, volume 2 part 1, page 314: + * + * "[DevSNB+]: Several cases exist where Depth Buffer Clear cannot be + * enabled (the legacy method of clearing must be performed): + * + * - If the depth buffer format is D32_FLOAT_S8X24_UINT or + * D24_UNORM_S8_UINT. + */ + return false; + + case MESA_FORMAT_Z32_FLOAT: + depth_clear_value = float_as_int(ctx->Depth.Clear); + break; + + case MESA_FORMAT_Z16: + /* From the Sandy Bridge PRM, volume 2 part 1, page 314: + * + * "[DevSNB+]: Several cases exist where Depth Buffer Clear cannot be + * enabled (the legacy method of clearing must be performed): + * + * - DevSNB{W/A}]: When depth buffer format is D16_UNORM and the + * width of the map (LOD0) is not multiple of 16, fast clear + * optimization must be disabled. + */ + if (intel->gen == 6 && (mt->level[depth_irb->mt_level].width % 16) != 0) + return false; + /* FALLTHROUGH */ + + default: + depth_clear_value = fb->_DepthMax * ctx->Depth.Clear; + break; + } + + /* If we're clearing to a new clear value, then we need to resolve any clear + * flags out of the HiZ buffer into the real depth buffer. + */ + if (mt->depth_clear_value != depth_clear_value) { + intel_miptree_all_slices_resolve_depth(intel, mt); + mt->depth_clear_value = depth_clear_value; + } + + /* From the Sandy Bridge PRM, volume 2 part 1, page 313: + * + * "If other rendering operations have preceded this clear, a + * PIPE_CONTROL with write cache flush enabled and Z-inhibit disabled + * must be issued before the rectangle primitive used for the depth + * buffer clear operation. + */ + intel_batchbuffer_emit_mi_flush(intel); + + intel_hiz_exec(intel, mt, depth_irb->mt_level, depth_irb->mt_layer, + GEN6_HIZ_OP_DEPTH_CLEAR); + + if (intel->gen == 6) { + /* From the Sandy Bridge PRM, volume 2 part 1, page 314: + * + * "DevSNB, DevSNB-B{W/A}]: Depth buffer clear pass must be followed + * by a PIPE_CONTROL command with DEPTH_STALL bit set and Then + * followed by Depth FLUSH' + */ + intel_batchbuffer_emit_mi_flush(intel); + } + + /* Now, the entire HiZ buffer contains data that needs to be resolved to the + * entire depth buffer (so any previous resolve records should get tossed + * out). + */ + intel_resolve_map_clear(&mt->hiz_map); + intel_renderbuffer_set_needs_depth_resolve(depth_irb); + + return true; +} + +/** * Called by ctx->Driver.Clear. */ static void @@ -89,6 +210,15 @@ brw_clear(struct gl_context *ctx, GLbitfield mask) intel->front_buffer_dirty = true; } + intel_prepare_render(intel); + + if (mask & BUFFER_BIT_DEPTH) { + if (brw_fast_clear_depth(ctx)) { + DBG("fast clear: depth\n"); + mask &= ~BUFFER_BIT_DEPTH; + } + } + GLbitfield tri_mask = mask & (BUFFER_BITS_COLOR | BUFFER_BIT_STENCIL | BUFFER_BIT_DEPTH); |