From 0556b87de4302195402ade43f400e859d9bfad0e Mon Sep 17 00:00:00 2001 From: Francisco Jerez Date: Sun, 3 Jan 2016 10:06:52 -0800 Subject: [PATCH] i965/gen7.5+: Disable resource streamer during GPGPU workloads. The RS and hardware binding tables are only supported on the 3D pipeline and can lead to corruption if left enabled during a GPGPU workload. Disable it when switching to the GPGPU (or media) pipeline and re-enable it when switching back to the 3D pipeline. Reviewed-by: Matt Turner Reviewed-by: Abdiel Janulgue --- .../drivers/dri/i965/brw_binding_tables.c | 2 +- src/mesa/drivers/dri/i965/brw_misc_state.c | 40 +++++++++++++++++++ src/mesa/drivers/dri/i965/brw_state.h | 1 + 3 files changed, 42 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_binding_tables.c b/src/mesa/drivers/dri/i965/brw_binding_tables.c index 7fa5d602b96..f3a0310861c 100644 --- a/src/mesa/drivers/dri/i965/brw_binding_tables.c +++ b/src/mesa/drivers/dri/i965/brw_binding_tables.c @@ -365,7 +365,7 @@ gen7_disable_hw_binding_tables(struct brw_context *brw) /** * Enable hardware binding tables and set up the binding table pool. */ -static void +void gen7_enable_hw_binding_tables(struct brw_context *brw) { if (!brw->use_resource_streamer) diff --git a/src/mesa/drivers/dri/i965/brw_misc_state.c b/src/mesa/drivers/dri/i965/brw_misc_state.c index 3686cdf8ff4..319c2a5669f 100644 --- a/src/mesa/drivers/dri/i965/brw_misc_state.c +++ b/src/mesa/drivers/dri/i965/brw_misc_state.c @@ -868,6 +868,26 @@ brw_emit_select_pipeline(struct brw_context *brw, enum brw_pipeline pipeline) const uint32_t _3DSTATE_PIPELINE_SELECT = is_965 ? CMD_PIPELINE_SELECT_965 : CMD_PIPELINE_SELECT_GM45; + if (brw->use_resource_streamer && pipeline != BRW_RENDER_PIPELINE) { + /* From "BXML » GT » MI » vol1a GPU Overview » [Instruction] + * PIPELINE_SELECT [DevBWR+]": + * + * Project: HSW, BDW, CHV, SKL, BXT + * + * Hardware Binding Tables are only supported for 3D + * workloads. Resource streamer must be enabled only for 3D + * workloads. Resource streamer must be disabled for Media and GPGPU + * workloads. + */ + BEGIN_BATCH(1); + OUT_BATCH(MI_RS_CONTROL | 0); + ADVANCE_BATCH(); + + gen7_disable_hw_binding_tables(brw); + + /* XXX - Disable gather constant pool too when we start using it. */ + } + if (brw->gen >= 8 && brw->gen < 10) { /* From the Broadwell PRM, Volume 2a: Instructions, PIPELINE_SELECT: * @@ -968,6 +988,26 @@ brw_emit_select_pipeline(struct brw_context *brw, enum brw_pipeline pipeline) OUT_BATCH(0); ADVANCE_BATCH(); } + + if (brw->use_resource_streamer && pipeline == BRW_RENDER_PIPELINE) { + /* From "BXML » GT » MI » vol1a GPU Overview » [Instruction] + * PIPELINE_SELECT [DevBWR+]": + * + * Project: HSW, BDW, CHV, SKL, BXT + * + * Hardware Binding Tables are only supported for 3D + * workloads. Resource streamer must be enabled only for 3D + * workloads. Resource streamer must be disabled for Media and GPGPU + * workloads. + */ + BEGIN_BATCH(1); + OUT_BATCH(MI_RS_CONTROL | 1); + ADVANCE_BATCH(); + + gen7_enable_hw_binding_tables(brw); + + /* XXX - Re-enable gather constant pool here. */ + } } /** diff --git a/src/mesa/drivers/dri/i965/brw_state.h b/src/mesa/drivers/dri/i965/brw_state.h index d29b997b963..7d61b7c4ab6 100644 --- a/src/mesa/drivers/dri/i965/brw_state.h +++ b/src/mesa/drivers/dri/i965/brw_state.h @@ -396,6 +396,7 @@ void gen7_update_binding_table_from_array(struct brw_context *brw, gl_shader_stage stage, const uint32_t* binding_table, int num_surfaces); +void gen7_enable_hw_binding_tables(struct brw_context *brw); void gen7_disable_hw_binding_tables(struct brw_context *brw); void gen7_reset_hw_bt_pool_offsets(struct brw_context *brw); -- 2.30.2