Curro: Ping? :)
On 26/09/16 20:02, Jason Ekstrand wrote:
Looks good to me. Curro, do you see anything wrong with this?
--Jason
On Sep 26, 2016 7:31 AM, "Lionel Landwerlin" <llandwer...@gmail.com
<mailto:llandwer...@gmail.com>> wrote:
Anv programs the hardware to use L3 data cache if we use either
SSBOs or
images in the shaders, we can program i965 the same way.
gl_shader_program has a bit of a confusing named field with
'NumAtomicBuffers'. It doesn't tell how many buffers are accessed
by the
shader in an atomic way but instead the number of atomic counters
manipulated by the shader.
Signed-off-by: Lionel Landwerlin <lionel.g.landwer...@intel.com
<mailto:lionel.g.landwer...@intel.com>>
Cc: Jason Ekstrand <ja...@jlekstrand.net
<mailto:ja...@jlekstrand.net>>
---
src/mesa/drivers/dri/i965/gen7_l3_state.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/gen7_l3_state.c
b/src/mesa/drivers/dri/i965/gen7_l3_state.c
index 92e8788..fdaea81 100644
--- a/src/mesa/drivers/dri/i965/gen7_l3_state.c
+++ b/src/mesa/drivers/dri/i965/gen7_l3_state.c
@@ -55,7 +55,8 @@ get_pipeline_state_l3_weights(const struct
brw_context *brw)
prog ? prog->_LinkedShaders[stage_states[i]->stage] : NULL;
const struct brw_stage_prog_data *prog_data =
stage_states[i]->prog_data;
- needs_dc |= (prog && prog->NumAtomicBuffers) ||
+ needs_dc |= (prog && (prog->NumAtomicBuffers ||
+ prog->NumShaderStorageBlocks)) ||
(shader && shader->NumImages) ||
(prog_data && prog_data->total_scratch);
needs_slm |= prog_data && prog_data->total_shared;
--
2.9.3
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
