[Mesa-dev] [PATCH 06/16] i965/gen7.5: Enable hardware-generated binding tables on render path.
Abdiel Janulgue
abdiel.janulgue at linux.intel.com
Thu Oct 10 14:17:12 PDT 2013
This patch implements the binding table enable command which is also
used to allocate a binding table pool where where hardware-generated
binding table entries are flushed into.
Each binding table offset in the binding table pool is unique per
each shader stage that are enabled within a batch.
In addition, this change inserts the required brw_tracked_state objects
to enable hw-generated binding tables in normal render path.
v3: Update binding table pool offsets on new geometry shader constant
buffers, geometry shader program cache, and fragment shader program
cache. Previous implementation failed to catch this flags.
Signed-off-by: Abdiel Janulgue <abdiel.janulgue at linux.intel.com>
---
src/mesa/drivers/dri/i965/brw_binding_tables.c | 87 ++++++++++++++++++++++++
src/mesa/drivers/dri/i965/brw_context.c | 1 +
src/mesa/drivers/dri/i965/brw_context.h | 1 +
src/mesa/drivers/dri/i965/brw_defines.h | 3 +
src/mesa/drivers/dri/i965/brw_state.h | 5 ++
src/mesa/drivers/dri/i965/brw_state_upload.c | 3 +
6 files changed, 100 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_binding_tables.c b/src/mesa/drivers/dri/i965/brw_binding_tables.c
index 9d15bac..534fbb8 100644
--- a/src/mesa/drivers/dri/i965/brw_binding_tables.c
+++ b/src/mesa/drivers/dri/i965/brw_binding_tables.c
@@ -164,6 +164,93 @@ const struct brw_tracked_state brw_gs_binding_table = {
.emit = brw_gs_upload_binding_table,
};
+/**
+ * Hardware-generated binding tables for the resource streamer
+ */
+void
+gen7_enable_hw_binding_tables(struct brw_context *brw)
+{
+ if (!brw->has_resource_streamer)
+ return;
+
+ if (!brw->hw_bt_pool)
+ gen7_update_hw_bt(brw);
+
+ BEGIN_BATCH(3);
+ OUT_BATCH(_3DSTATE_BINDING_TABLE_POOL_ALLOC << 16 | (3 - 2));
+ OUT_RELOC(brw->hw_bt_pool, I915_GEM_DOMAIN_SAMPLER, 0,
+ HSW_BINDING_TABLE_ALLOC_OFFSET);
+ OUT_RELOC(brw->hw_bt_pool, I915_GEM_DOMAIN_SAMPLER, 0,
+ brw->hw_bt_pool->size);
+ ADVANCE_BATCH();
+
+ /* Pipe control workaround */
+ BEGIN_BATCH(4);
+ OUT_BATCH(_3DSTATE_PIPE_CONTROL | (4 - 2));
+ OUT_BATCH(PIPE_CONTROL_STATE_CACHE_INVALIDATE);
+ OUT_BATCH(0); /* address */
+ OUT_BATCH(0); /* write data */
+ ADVANCE_BATCH();
+
+ /* Do a block clear for existing on-chip binding table entries
+ that might have stuck from the old batch*/
+ BEGIN_BATCH(3);
+ OUT_BATCH(_3DSTATE_BINDING_TABLE_EDIT_VS << 16 | (3 - 2));
+ OUT_BATCH(0xffff << 16 | 0x3 );
+ OUT_BATCH(0);
+ ADVANCE_BATCH();
+
+ BEGIN_BATCH(3);
+ OUT_BATCH(_3DSTATE_BINDING_TABLE_EDIT_PS << 16 | (3 - 2));
+ OUT_BATCH(0xffff << 16 | 0x3 );
+ OUT_BATCH(0);
+ ADVANCE_BATCH();
+}
+
+void
+gen7_update_hw_bt(struct brw_context *brw)
+{
+ static const int bt_size = 256 * sizeof(uint16_t);
+
+ if (!brw->has_resource_streamer)
+ return;
+
+ if (brw->hw_bt_pool &&
+ (brw->wm.base.bind_bo_offset + bt_size) < brw->hw_bt_pool->size) {
+ brw->vs.base.bind_bo_offset = brw->wm.base.bind_bo_offset + bt_size;
+ brw->wm.base.bind_bo_offset = brw->vs.base.bind_bo_offset + bt_size;
+ } else {
+ brw->vs.base.bind_bo_offset = 64;
+ brw->wm.base.bind_bo_offset = brw->vs.base.bind_bo_offset + bt_size;
+ drm_intel_bo_unreference(brw->hw_bt_pool);
+ brw->hw_bt_pool = drm_intel_bo_alloc(brw->bufmgr, "hw_bt",
+ 65536, 4096);
+ }
+}
+
+const struct brw_tracked_state gen7_hw_binding_tables = {
+ .dirty = {
+ .mesa = 0,
+ .brw = BRW_NEW_BATCH,
+ .cache = 0
+ },
+ .emit = gen7_enable_hw_binding_tables
+};
+
+const struct brw_tracked_state gen7_hw_bt_update = {
+ .dirty = {
+ .mesa = 0,
+ .brw = (BRW_NEW_BATCH |
+ BRW_NEW_VS_CONSTBUF |
+ BRW_NEW_GS_CONSTBUF |
+ BRW_NEW_SURFACES),
+ .cache = (CACHE_NEW_VS_PROG |
+ CACHE_NEW_GS_PROG |
+ CACHE_NEW_WM_PROG)
+ },
+ .emit = gen7_update_hw_bt
+};
+
/** @} */
/**
diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c
index 776d221..ad70517 100644
--- a/src/mesa/drivers/dri/i965/brw_context.c
+++ b/src/mesa/drivers/dri/i965/brw_context.c
@@ -497,6 +497,7 @@ brwCreateContext(int api,
brw_fs_alloc_reg_sets(brw);
brw_vec4_alloc_reg_set(brw);
+ brw->hw_bt_pool = 0;
if (INTEL_DEBUG & DEBUG_SHADER_TIME)
brw_init_shader_time(brw);
diff --git a/src/mesa/drivers/dri/i965/brw_context.h b/src/mesa/drivers/dri/i965/brw_context.h
index 1151163..f4c7824 100644
--- a/src/mesa/drivers/dri/i965/brw_context.h
+++ b/src/mesa/drivers/dri/i965/brw_context.h
@@ -1320,6 +1320,7 @@ struct brw_context
} reg_sets[2];
} wm;
+ drm_intel_bo *hw_bt_pool;
struct {
uint32_t state_offset;
diff --git a/src/mesa/drivers/dri/i965/brw_defines.h b/src/mesa/drivers/dri/i965/brw_defines.h
index 7dfb2b9..3530bbf 100644
--- a/src/mesa/drivers/dri/i965/brw_defines.h
+++ b/src/mesa/drivers/dri/i965/brw_defines.h
@@ -1244,6 +1244,9 @@ enum brw_message_target {
#define _3DSTATE_BINDING_TABLE_POINTERS_GS 0x7829 /* GEN7+ */
#define _3DSTATE_BINDING_TABLE_POINTERS_PS 0x782A /* GEN7+ */
+#define _3DSTATE_BINDING_TABLE_POOL_ALLOC 0x7919 /* GEN7.5+ */
+# define HSW_BINDING_TABLE_ALLOC_OFFSET 0x860 /* GEN7.5+ */
+
#define _3DSTATE_SAMPLER_STATE_POINTERS 0x7802 /* GEN6+ */
# define PS_SAMPLER_STATE_CHANGE (1 << 12)
# define GS_SAMPLER_STATE_CHANGE (1 << 9)
diff --git a/src/mesa/drivers/dri/i965/brw_state.h b/src/mesa/drivers/dri/i965/brw_state.h
index 144d370..0e08e03 100644
--- a/src/mesa/drivers/dri/i965/brw_state.h
+++ b/src/mesa/drivers/dri/i965/brw_state.h
@@ -127,6 +127,8 @@ extern const struct brw_tracked_state gen7_sol_state;
extern const struct brw_tracked_state gen7_urb;
extern const struct brw_tracked_state gen7_vs_state;
extern const struct brw_tracked_state gen7_wm_state;
+extern const struct brw_tracked_state gen7_hw_binding_tables;
+extern const struct brw_tracked_state gen7_hw_bt_update;
extern const struct brw_tracked_state haswell_cut_index;
@@ -255,6 +257,9 @@ gen7_upload_constant_state(struct brw_context *brw,
/* gen7_misc_state.c */
void gen7_rs_control(struct brw_context *brw, int enable);
+void gen7_update_hw_bt(struct brw_context *brw);
+void gen7_enable_hw_binding_tables(struct brw_context *brw);
+
#ifdef __cplusplus
}
#endif
diff --git a/src/mesa/drivers/dri/i965/brw_state_upload.c b/src/mesa/drivers/dri/i965/brw_state_upload.c
index d7fe319..6c3e07a 100644
--- a/src/mesa/drivers/dri/i965/brw_state_upload.c
+++ b/src/mesa/drivers/dri/i965/brw_state_upload.c
@@ -199,6 +199,8 @@ static const struct brw_tracked_state *gen7_atoms[] =
&gen7_gs_push_constants, /* Before gs_state */
&gen6_wm_push_constants, /* Before wm_surfaces and constant_buffer */
+ &gen7_hw_binding_tables, /* Enable hw-generated binding tables for Haswell */
+
/* Surface state setup. Must come before the VS/WM unit. The binding
* table upload must be last.
*/
@@ -220,6 +222,7 @@ static const struct brw_tracked_state *gen7_atoms[] =
&gen6_multisample_state,
&gen7_disable_stages,
+ &gen7_hw_bt_update, /* Update offsets to binding table pool*/
&gen7_vs_state,
&gen7_gs_state,
&gen7_sol_state,
--
1.7.9.5
More information about the mesa-dev
mailing list