Module: Mesa Branch: main Commit: 94abccf3ce50383a1ca0feceb6e95a065b89c19f URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=94abccf3ce50383a1ca0feceb6e95a065b89c19f
Author: Rhys Perry <[email protected]> Date: Mon Feb 20 14:37:33 2023 +0000 aco: fix pathological case in LdsDirectVALUHazard Similar to bfd4ac4581bae0d36b2c58524131f91c7c0d27ce. No fossil-db changes. Signed-off-by: Rhys Perry <[email protected]> Reviewed-by: Timur Kristóf <[email protected]> Fixes: 296b4d95a3e ("aco/gfx11: workaround LdsDirectVALUHazard") Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/21423> --- src/amd/compiler/aco_insert_NOPs.cpp | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/src/amd/compiler/aco_insert_NOPs.cpp b/src/amd/compiler/aco_insert_NOPs.cpp index de9fd0105ff..e9a0c328f2d 100644 --- a/src/amd/compiler/aco_insert_NOPs.cpp +++ b/src/amd/compiler/aco_insert_NOPs.cpp @@ -1044,6 +1044,9 @@ struct LdsDirectVALUHazardGlobalState { struct LdsDirectVALUHazardBlockState { unsigned num_valu = 0; bool has_trans = false; + + unsigned num_instrs = 0; + unsigned num_blocks = 0; }; bool @@ -1074,6 +1077,14 @@ handle_lds_direct_valu_hazard_instr(LdsDirectVALUHazardGlobalState& global_state if (parse_vdst_wait(instr) == 0) return true; + block_state.num_instrs++; + if (block_state.num_instrs > 256 || block_state.num_blocks > 32) { + /* Exit to limit compile times and set wait_vdst to be safe. */ + global_state.wait_vdst = + MIN2(global_state.wait_vdst, block_state.has_trans ? 0 : block_state.num_valu); + return true; + } + return block_state.num_valu >= global_state.wait_vdst; } @@ -1087,6 +1098,8 @@ handle_lds_direct_valu_hazard_block(LdsDirectVALUHazardGlobalState& global_state global_state.loop_headers_visited.insert(block->index); } + block_state.num_blocks++; + return true; }
