[Mesa-dev] [PATCH v03 18/38] i965: Port Gen6+ DEPTH_STENCIL state to genxml.
Rafael Antognolli
rafael.antognolli at intel.com
Tue May 2 01:43:06 UTC 2017
From: Kenneth Graunke <kenneth at whitecape.org>
This emits 3DSTATE_WM_DEPTH_STENCIL on Gen8+ or DEPTH_STENCIL_STATE
(and the relevant pointer packets) on Gen6-7.5 from a single function.
v3:
- Watch for BRW_NEW_BATCH too on gen8+ (Ken)
Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Signed-off-by: Rafael Antognolli <rafael.antognolli at intel.com>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
---
src/mesa/drivers/dri/i965/Makefile.sources | 2 +-
src/mesa/drivers/dri/i965/brw_state.h | 2 +-
src/mesa/drivers/dri/i965/gen6_depthstencil.c | 114 +--------------
src/mesa/drivers/dri/i965/gen8_wm_depth_stencil.c | 118 +---------------
src/mesa/drivers/dri/i965/genX_state_upload.c | 103 ++++++++++++-
5 files changed, 101 insertions(+), 238 deletions(-)
delete mode 100644 src/mesa/drivers/dri/i965/gen6_depthstencil.c
delete mode 100644 src/mesa/drivers/dri/i965/gen8_wm_depth_stencil.c
diff --git a/src/mesa/drivers/dri/i965/Makefile.sources b/src/mesa/drivers/dri/i965/Makefile.sources
index 41f4d83..b085251 100644
--- a/src/mesa/drivers/dri/i965/Makefile.sources
+++ b/src/mesa/drivers/dri/i965/Makefile.sources
@@ -80,7 +80,6 @@ i965_FILES = \
gen6_clip_state.c \
gen6_constant_state.c \
gen6_depth_state.c \
- gen6_depthstencil.c \
gen6_gs_state.c \
gen6_multisample_state.c \
gen6_queryobj.c \
@@ -119,7 +118,6 @@ i965_FILES = \
gen8_surface_state.c \
gen8_viewport_state.c \
gen8_vs_state.c \
- gen8_wm_depth_stencil.c \
hsw_queryobj.c \
hsw_sol.c \
intel_batchbuffer.c \
diff --git a/src/mesa/drivers/dri/i965/brw_state.h b/src/mesa/drivers/dri/i965/brw_state.h
index 6403570..7b6d718 100644
--- a/src/mesa/drivers/dri/i965/brw_state.h
+++ b/src/mesa/drivers/dri/i965/brw_state.h
@@ -112,7 +112,6 @@ extern const struct brw_tracked_state gen6_blend_state;
extern const struct brw_tracked_state gen6_clip_state;
extern const struct brw_tracked_state gen6_sf_and_clip_viewports;
extern const struct brw_tracked_state gen6_color_calc_state;
-extern const struct brw_tracked_state gen6_depth_stencil_state;
extern const struct brw_tracked_state gen6_gs_state;
extern const struct brw_tracked_state gen6_gs_push_constants;
extern const struct brw_tracked_state gen6_gs_binding_table;
@@ -157,7 +156,6 @@ extern const struct brw_tracked_state gen8_pma_fix;
extern const struct brw_tracked_state gen8_ps_blend;
extern const struct brw_tracked_state gen8_ps_extra;
extern const struct brw_tracked_state gen8_ps_state;
-extern const struct brw_tracked_state gen8_wm_depth_stencil;
extern const struct brw_tracked_state gen8_wm_state;
extern const struct brw_tracked_state gen8_raster_state;
extern const struct brw_tracked_state gen8_sbe_state;
diff --git a/src/mesa/drivers/dri/i965/gen6_depthstencil.c b/src/mesa/drivers/dri/i965/gen6_depthstencil.c
deleted file mode 100644
index 0f9626c..0000000
--- a/src/mesa/drivers/dri/i965/gen6_depthstencil.c
+++ /dev/null
@@ -1,114 +0,0 @@
-/*
- * Copyright © 2009 Intel Corporation
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice (including the next
- * paragraph) shall be included in all copies or substantial portions of the
- * Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
- * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
- * IN THE SOFTWARE.
- *
- * Authors:
- * Eric Anholt <eric at anholt.net>
- *
- */
-
-#include "intel_batchbuffer.h"
-#include "intel_fbo.h"
-#include "brw_context.h"
-#include "brw_defines.h"
-#include "brw_state.h"
-
-static void
-gen6_upload_depth_stencil_state(struct brw_context *brw)
-{
- struct gl_context *ctx = &brw->ctx;
- struct gen6_depth_stencil_state *ds;
- struct intel_renderbuffer *depth_irb;
-
- /* _NEW_BUFFERS */
- depth_irb = intel_get_renderbuffer(ctx->DrawBuffer, BUFFER_DEPTH);
-
- ds = brw_state_batch(brw, sizeof(*ds), 64,
- &brw->cc.depth_stencil_state_offset);
- memset(ds, 0, sizeof(*ds));
-
- /* _NEW_STENCIL | _NEW_BUFFERS */
- if (ctx->Stencil._Enabled) {
- int back = ctx->Stencil._BackFace;
-
- ds->ds0.stencil_enable = 1;
- ds->ds0.stencil_func =
- intel_translate_compare_func(ctx->Stencil.Function[0]);
- ds->ds0.stencil_fail_op =
- intel_translate_stencil_op(ctx->Stencil.FailFunc[0]);
- ds->ds0.stencil_pass_depth_fail_op =
- intel_translate_stencil_op(ctx->Stencil.ZFailFunc[0]);
- ds->ds0.stencil_pass_depth_pass_op =
- intel_translate_stencil_op(ctx->Stencil.ZPassFunc[0]);
- ds->ds1.stencil_write_mask = ctx->Stencil.WriteMask[0];
- ds->ds1.stencil_test_mask = ctx->Stencil.ValueMask[0];
-
- if (ctx->Stencil._TestTwoSide) {
- ds->ds0.bf_stencil_enable = 1;
- ds->ds0.bf_stencil_func =
- intel_translate_compare_func(ctx->Stencil.Function[back]);
- ds->ds0.bf_stencil_fail_op =
- intel_translate_stencil_op(ctx->Stencil.FailFunc[back]);
- ds->ds0.bf_stencil_pass_depth_fail_op =
- intel_translate_stencil_op(ctx->Stencil.ZFailFunc[back]);
- ds->ds0.bf_stencil_pass_depth_pass_op =
- intel_translate_stencil_op(ctx->Stencil.ZPassFunc[back]);
- ds->ds1.bf_stencil_write_mask = ctx->Stencil.WriteMask[back];
- ds->ds1.bf_stencil_test_mask = ctx->Stencil.ValueMask[back];
- }
-
- ds->ds0.stencil_write_enable = ctx->Stencil._WriteEnabled;
- }
-
- /* _NEW_DEPTH */
- if (ctx->Depth.Test && depth_irb) {
- ds->ds2.depth_test_enable = ctx->Depth.Test;
- ds->ds2.depth_test_func = intel_translate_compare_func(ctx->Depth.Func);
- ds->ds2.depth_write_enable = brw_depth_writes_enabled(brw);
- }
-
- /* Point the GPU at the new indirect state. */
- if (brw->gen == 6) {
- BEGIN_BATCH(4);
- OUT_BATCH(_3DSTATE_CC_STATE_POINTERS << 16 | (4 - 2));
- OUT_BATCH(0);
- OUT_BATCH(brw->cc.depth_stencil_state_offset | 1);
- OUT_BATCH(0);
- ADVANCE_BATCH();
- } else {
- BEGIN_BATCH(2);
- OUT_BATCH(_3DSTATE_DEPTH_STENCIL_STATE_POINTERS << 16 | (2 - 2));
- OUT_BATCH(brw->cc.depth_stencil_state_offset | 1);
- ADVANCE_BATCH();
- }
-}
-
-const struct brw_tracked_state gen6_depth_stencil_state = {
- .dirty = {
- .mesa = _NEW_BUFFERS |
- _NEW_DEPTH |
- _NEW_STENCIL,
- .brw = BRW_NEW_BATCH |
- BRW_NEW_BLORP |
- BRW_NEW_STATE_BASE_ADDRESS,
- },
- .emit = gen6_upload_depth_stencil_state,
-};
diff --git a/src/mesa/drivers/dri/i965/gen8_wm_depth_stencil.c b/src/mesa/drivers/dri/i965/gen8_wm_depth_stencil.c
deleted file mode 100644
index 9a6c9e0..0000000
--- a/src/mesa/drivers/dri/i965/gen8_wm_depth_stencil.c
+++ /dev/null
@@ -1,118 +0,0 @@
-/*
- * Copyright © 2012 Intel Corporation
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice (including the next
- * paragraph) shall be included in all copies or substantial portions of the
- * Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
- * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
- * IN THE SOFTWARE.
- */
-
-#include "intel_batchbuffer.h"
-#include "intel_fbo.h"
-#include "brw_context.h"
-#include "brw_defines.h"
-#include "brw_state.h"
-#include "main/stencil.h"
-
-static void
-gen8_upload_wm_depth_stencil(struct brw_context *brw)
-{
- struct gl_context *ctx = &brw->ctx;
- uint32_t dw1 = 0, dw2 = 0, dw3 = 0;
-
- /* _NEW_BUFFERS */
- struct intel_renderbuffer *depth_irb =
- intel_get_renderbuffer(ctx->DrawBuffer, BUFFER_DEPTH);
-
- struct gl_stencil_attrib *stencil = &ctx->Stencil;
-
- /* _NEW_STENCIL | _NEW_BUFFERS */
- if (stencil->_Enabled) {
- #define FUNC intel_translate_compare_func
- #define OP intel_translate_stencil_op
-
- dw1 |=
- GEN8_WM_DS_STENCIL_TEST_ENABLE |
- FUNC(stencil->Function[0]) << GEN8_WM_DS_STENCIL_FUNC_SHIFT |
- OP(stencil->FailFunc[0]) << GEN8_WM_DS_STENCIL_FAIL_OP_SHIFT |
- OP(stencil->ZFailFunc[0]) << GEN8_WM_DS_Z_FAIL_OP_SHIFT |
- OP(stencil->ZPassFunc[0]) << GEN8_WM_DS_Z_PASS_OP_SHIFT;
-
- if (stencil->_WriteEnabled)
- dw1 |= GEN8_WM_DS_STENCIL_BUFFER_WRITE_ENABLE;
-
- dw2 |=
- SET_FIELD(stencil->WriteMask[0] & 0xff, GEN8_WM_DS_STENCIL_WRITE_MASK) |
- SET_FIELD(stencil->ValueMask[0] & 0xff, GEN8_WM_DS_STENCIL_TEST_MASK);
-
- if (stencil->_TestTwoSide) {
- const int b = stencil->_BackFace;
-
- dw1 |=
- GEN8_WM_DS_DOUBLE_SIDED_STENCIL_ENABLE |
- FUNC(stencil->Function[b]) << GEN8_WM_DS_BF_STENCIL_FUNC_SHIFT |
- OP(stencil->FailFunc[b]) << GEN8_WM_DS_BF_STENCIL_FAIL_OP_SHIFT |
- OP(stencil->ZFailFunc[b]) << GEN8_WM_DS_BF_Z_FAIL_OP_SHIFT |
- OP(stencil->ZPassFunc[b]) << GEN8_WM_DS_BF_Z_PASS_OP_SHIFT;
-
- dw2 |= SET_FIELD(stencil->WriteMask[b] & 0xff,
- GEN8_WM_DS_BF_STENCIL_WRITE_MASK) |
- SET_FIELD(stencil->ValueMask[b] & 0xff,
- GEN8_WM_DS_BF_STENCIL_TEST_MASK);
- }
-
- if (brw->gen >= 9) {
- int stencil_ref = _mesa_get_stencil_ref(ctx, 0);
- int backface_ref = _mesa_get_stencil_ref(ctx, ctx->Stencil._BackFace);
-
- dw3 = SET_FIELD(stencil_ref, GEN9_WM_DS_STENCIL_REF) |
- SET_FIELD(backface_ref, GEN9_WM_DS_BF_STENCIL_REF);
- }
- }
-
- /* _NEW_DEPTH */
- if (ctx->Depth.Test && depth_irb) {
- dw1 |=
- GEN8_WM_DS_DEPTH_TEST_ENABLE |
- FUNC(ctx->Depth.Func) << GEN8_WM_DS_DEPTH_FUNC_SHIFT;
-
- if (brw_depth_writes_enabled(brw))
- dw1 |= GEN8_WM_DS_DEPTH_BUFFER_WRITE_ENABLE;
- }
-
- int pkt_len = brw->gen >= 9 ? 4 : 3;
-
- BEGIN_BATCH(pkt_len);
- OUT_BATCH(_3DSTATE_WM_DEPTH_STENCIL << 16 | (pkt_len - 2));
- OUT_BATCH(dw1);
- OUT_BATCH(dw2);
- if (pkt_len > 3) {
- OUT_BATCH(dw3);
- }
- ADVANCE_BATCH();
-}
-
-const struct brw_tracked_state gen8_wm_depth_stencil = {
- .dirty = {
- .mesa = _NEW_BUFFERS |
- _NEW_DEPTH |
- _NEW_STENCIL,
- .brw = BRW_NEW_BLORP |
- BRW_NEW_CONTEXT,
- },
- .emit = gen8_upload_wm_depth_stencil,
-};
diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c b/src/mesa/drivers/dri/i965/genX_state_upload.c
index b0487f9..49caf63 100644
--- a/src/mesa/drivers/dri/i965/genX_state_upload.c
+++ b/src/mesa/drivers/dri/i965/genX_state_upload.c
@@ -30,6 +30,9 @@
#include "brw_state.h"
#include "intel_batchbuffer.h"
+#include "intel_fbo.h"
+
+#include "main/stencil.h"
UNUSED static void *
emit_dwords(struct brw_context *brw, unsigned n)
@@ -108,9 +111,105 @@ __gen_combine_address(struct brw_context *brw, void *location,
/* ---------------------------------------------------------------------- */
+#if GEN_GEN >= 6
/* ---------------------------------------------------------------------- */
+static void
+genX(upload_depth_stencil_state)(struct brw_context *brw)
+{
+ struct gl_context *ctx = &brw->ctx;
+
+ /* _NEW_BUFFERS */
+ struct intel_renderbuffer *depth_irb =
+ intel_get_renderbuffer(ctx->DrawBuffer, BUFFER_DEPTH);
+
+ /* _NEW_DEPTH */
+ struct gl_depthbuffer_attrib *depth = &ctx->Depth;
+
+ /* _NEW_STENCIL */
+ struct gl_stencil_attrib *stencil = &ctx->Stencil;
+ const int b = stencil->_BackFace;
+
+#if GEN_GEN >= 8
+ brw_batch_emit(brw, GENX(3DSTATE_WM_DEPTH_STENCIL), wmds) {
+#else
+ uint32_t ds_offset;
+ brw_state_emit(brw, GENX(DEPTH_STENCIL_STATE), 64, &ds_offset, wmds) {
+#endif
+ if (depth->Test && depth_irb) {
+ wmds.DepthTestEnable = true;
+ wmds.DepthBufferWriteEnable = brw_depth_writes_enabled(brw);
+ wmds.DepthTestFunction = intel_translate_compare_func(depth->Func);
+ }
+
+ if (stencil->_Enabled) {
+ wmds.StencilTestEnable = true;
+ wmds.StencilWriteMask = stencil->WriteMask[0] & 0xff;
+ wmds.StencilTestMask = stencil->ValueMask[0] & 0xff;
+
+ wmds.StencilTestFunction =
+ intel_translate_compare_func(stencil->Function[0]);
+ wmds.StencilFailOp =
+ intel_translate_stencil_op(stencil->FailFunc[0]);
+ wmds.StencilPassDepthPassOp =
+ intel_translate_stencil_op(stencil->ZPassFunc[0]);
+ wmds.StencilPassDepthFailOp =
+ intel_translate_stencil_op(stencil->ZFailFunc[0]);
+
+ wmds.StencilBufferWriteEnable = stencil->_WriteEnabled;
+
+ if (stencil->_TestTwoSide) {
+ wmds.DoubleSidedStencilEnable = true;
+ wmds.BackfaceStencilWriteMask = stencil->WriteMask[b] & 0xff;
+ wmds.BackfaceStencilTestMask = stencil->ValueMask[b] & 0xff;
+
+ wmds.BackfaceStencilTestFunction =
+ intel_translate_compare_func(stencil->Function[b]);
+ wmds.BackfaceStencilFailOp =
+ intel_translate_stencil_op(stencil->FailFunc[b]);
+ wmds.BackfaceStencilPassDepthPassOp =
+ intel_translate_stencil_op(stencil->ZPassFunc[b]);
+ wmds.BackfaceStencilPassDepthFailOp =
+ intel_translate_stencil_op(stencil->ZFailFunc[b]);
+ }
+
+#if GEN_GEN >= 9
+ wmds.StencilReferenceValue = _mesa_get_stencil_ref(ctx, 0);
+ wmds.BackfaceStencilReferenceValue = _mesa_get_stencil_ref(ctx, b);
+#endif
+ }
+ }
+
+#if GEN_GEN == 6
+ brw_batch_emit(brw, GENX(3DSTATE_CC_STATE_POINTERS), ptr) {
+ ptr.PointertoDEPTH_STENCIL_STATE = ds_offset;
+ ptr.DEPTH_STENCIL_STATEChange = true;
+ }
+#elif GEN_GEN == 7
+ brw_batch_emit(brw, GENX(3DSTATE_DEPTH_STENCIL_STATE_POINTERS), ptr) {
+ ptr.PointertoDEPTH_STENCIL_STATE = ds_offset;
+ }
+#endif
+}
+
+static const struct brw_tracked_state genX(depth_stencil_state) = {
+ .dirty = {
+ .mesa = _NEW_BUFFERS |
+ _NEW_DEPTH |
+ _NEW_STENCIL,
+ .brw = BRW_NEW_BLORP |
+ (GEN_GEN >= 8 ? BRW_NEW_CONTEXT
+ : BRW_NEW_BATCH |
+ BRW_NEW_STATE_BASE_ADDRESS),
+ },
+ .emit = genX(upload_depth_stencil_state),
+};
+
+/* ---------------------------------------------------------------------- */
+
+#endif
+
void
genX(init_atoms)(struct brw_context *brw)
{
@@ -250,7 +349,7 @@ genX(init_atoms)(struct brw_context *brw)
&gen7_urb,
&gen6_blend_state, /* must do before cc unit */
&gen6_color_calc_state, /* must do before cc unit */
- &gen6_depth_stencil_state, /* must do before cc unit */
+ &genX(depth_stencil_state), /* must do before cc unit */
&brw_vs_image_surfaces, /* Before vs push/pull constants and binding table */
&brw_tcs_image_surfaces, /* Before tcs push/pull constants and binding table */
@@ -398,7 +497,7 @@ genX(init_atoms)(struct brw_context *brw)
&gen8_ps_blend,
&gen8_ps_extra,
&gen8_ps_state,
- &gen8_wm_depth_stencil,
+ &genX(depth_stencil_state),
&gen8_wm_state,
&gen6_scissor_state,
--
git-series 0.9.1
More information about the mesa-dev
mailing list