Mesa (master): i965/gen7: Program stencil buffers on Ivybridge.

Kenneth Graunke kwg at kemper.freedesktop.org
Wed Jun 8 21:52:33 UTC 2011


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

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Mon May 23 14:01:16 2011 -0700

i965/gen7: Program stencil buffers on Ivybridge.

Thanks to Chad's hard work implementing separate stencil and HiZ
support, this is entirely straightforward.

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Eric Anholt <eric at anholt.net>

---

 src/mesa/drivers/dri/i965/gen7_misc_state.c |   61 ++++++++++++++++++--------
 1 files changed, 42 insertions(+), 19 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/gen7_misc_state.c b/src/mesa/drivers/dri/i965/gen7_misc_state.c
index 0240211..dd0ccd2 100644
--- a/src/mesa/drivers/dri/i965/gen7_misc_state.c
+++ b/src/mesa/drivers/dri/i965/gen7_misc_state.c
@@ -75,41 +75,55 @@ static void emit_depthbuffer(struct brw_context *brw)
    struct intel_context *intel = &brw->intel;
    struct gl_context *ctx = &intel->ctx;
    struct gl_framebuffer *fb = ctx->DrawBuffer;
+
+   /* _NEW_BUFFERS */
    struct intel_renderbuffer *drb = intel_get_renderbuffer(fb, BUFFER_DEPTH);
    struct intel_renderbuffer *srb = intel_get_renderbuffer(fb, BUFFER_STENCIL);
-   struct intel_region *region = NULL;
 
-   /* _NEW_BUFFERS */
-   if (drb)
-      region = drb->region;
-   else if (srb)
-      region = srb->region;
+   /* Gen7 doesn't support packed depth/stencil */
+   assert(srb == NULL || srb != drb);
+
+   if (drb == NULL) {
+      uint32_t dw1 = BRW_DEPTHFORMAT_D32_FLOAT << 18;
+      uint32_t dw3 = 0;
+
+      if (srb == NULL) {
+	 dw1 |= (BRW_SURFACE_NULL << 29);
+      } else {
+	 struct intel_region *region = srb->region;
+
+	 /* _NEW_STENCIL: enable stencil buffer writes */
+	 dw1 |= ((ctx->Stencil.WriteMask != 0) << 27);
+
+	 /* 3DSTATE_STENCIL_BUFFER inherits surface type and dimensions. */
+	 dw1 |= (BRW_SURFACE_2D << 29);
+	 dw3 = ((region->width - 1) << 4) | ((2 * region->height - 1) << 18);
+      }
 
-   if (region == NULL) {
       BEGIN_BATCH(7);
       OUT_BATCH(GEN7_3DSTATE_DEPTH_BUFFER << 16 | (7 - 2));
-      OUT_BATCH((BRW_DEPTHFORMAT_D32_FLOAT << 18) |
-		(BRW_SURFACE_NULL << 29));
-      OUT_BATCH(0);
+      OUT_BATCH(dw1);
       OUT_BATCH(0);
+      OUT_BATCH(dw3);
       OUT_BATCH(0);
       OUT_BATCH(0);
       OUT_BATCH(0);
       ADVANCE_BATCH();
    } else {
+      struct intel_region *region = drb->region;
       uint32_t tile_x, tile_y, offset;
 
       offset = intel_region_tile_offsets(region, &tile_x, &tile_y);
 
       assert(region->tiling == I915_TILING_Y);
 
-      /* _NEW_DEPTH */
+      /* _NEW_DEPTH, _NEW_STENCIL */
       BEGIN_BATCH(7);
       OUT_BATCH(GEN7_3DSTATE_DEPTH_BUFFER << 16 | (7 - 2));
       OUT_BATCH(((region->pitch * region->cpp) - 1) |
 		(gen7_depth_format(brw) << 18) |
 		(0 << 22) /* no HiZ buffer */ |
-		(0 << 27) /* no stencil write */ |
+		((srb != NULL && ctx->Stencil.WriteMask != 0) << 27) |
 		((ctx->Depth.Mask != 0) << 28) |
 		(BRW_SURFACE_2D << 29));
       OUT_RELOC(region->buffer,
@@ -129,12 +143,21 @@ static void emit_depthbuffer(struct brw_context *brw)
    OUT_BATCH(0);
    ADVANCE_BATCH();
 
-   BEGIN_BATCH(4);
-   OUT_BATCH(GEN7_3DSTATE_STENCIL_BUFFER << 16 | (4 - 2));
-   OUT_BATCH(0);
-   OUT_BATCH(0);
-   OUT_BATCH(0);
-   ADVANCE_BATCH();
+   if (srb == NULL) {
+      BEGIN_BATCH(3);
+      OUT_BATCH(GEN7_3DSTATE_STENCIL_BUFFER << 16 | (3 - 2));
+      OUT_BATCH(0);
+      OUT_BATCH(0);
+      ADVANCE_BATCH();
+   } else {
+      BEGIN_BATCH(3);
+      OUT_BATCH(GEN7_3DSTATE_STENCIL_BUFFER << 16 | (3 - 2));
+      OUT_BATCH(srb->region->pitch * srb->region->cpp - 1);
+      OUT_RELOC(srb->region->buffer,
+	        I915_GEM_DOMAIN_RENDER, I915_GEM_DOMAIN_RENDER,
+		0);
+      ADVANCE_BATCH();
+   }
 
    BEGIN_BATCH(3);
    OUT_BATCH(GEN7_3DSTATE_CLEAR_PARAMS << 16 | (3 - 2));
@@ -148,7 +171,7 @@ static void emit_depthbuffer(struct brw_context *brw)
  */
 const struct brw_tracked_state gen7_depthbuffer = {
    .dirty = {
-      .mesa = (_NEW_BUFFERS | _NEW_DEPTH),
+      .mesa = (_NEW_BUFFERS | _NEW_DEPTH | _NEW_STENCIL),
       .brw = BRW_NEW_BATCH,
       .cache = 0,
    },




More information about the mesa-commit mailing list