[Mesa-dev] [PATCH 1/3] i965: Don't double the depth width for separate-stencil-only rendering.

Eric Anholt eric at anholt.net
Mon Nov 28 10:48:19 PST 2011


This looks to have been confused with pitch setup, which does have to
be doubled.  The width is inherited by separate stencil, and it should
naturally look like the setup in the block below involving a valid
depthbuffer.
---
 src/mesa/drivers/dri/i965/brw_misc_state.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_misc_state.c b/src/mesa/drivers/dri/i965/brw_misc_state.c
index cb1405c..f6a5ad6 100644
--- a/src/mesa/drivers/dri/i965/brw_misc_state.c
+++ b/src/mesa/drivers/dri/i965/brw_misc_state.c
@@ -298,7 +298,7 @@ static void emit_depthbuffer(struct brw_context *brw)
 	        (BRW_SURFACE_2D << 29));
       OUT_BATCH(0);
       OUT_BATCH(((region->width - 1) << 6) |
-	         (2 * region->height - 1) << 19);
+	         (region->height - 1) << 19);
       OUT_BATCH(0);
       OUT_BATCH(0);
 
-- 
1.7.7.3



More information about the mesa-dev mailing list