Mesa (master): tu: Use tu_cs_add_entries() with non-render-pass secondaries
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Fri Apr 17 14:25:16 UTC 2020
Module: Mesa
Branch: master
Commit: 64e3b8d66b9d4103ff815606ff20b39246418122
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=64e3b8d66b9d4103ff815606ff20b39246418122
Author: Connor Abbott <cwabbott0 at gmail.com>
Date: Fri Apr 17 15:28:10 2020 +0200
tu: Use tu_cs_add_entries() with non-render-pass secondaries
Even though vkCmdRenderPassBegin() isn't allowed inside a secondary
command buffer, vkCmdDispatch() is, and we emit an IB with compute
dispatches, which means that if the secondary command buffer records a
vkCmdDispatch() then we'll have an IB inside an IB, which is illegal.
Fixes hangs in e.g.
dEQP-VK.api.command_buffers.record_simul_use_secondary_one_primary.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4605>
---
src/freedreno/vulkan/tu_cmd_buffer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/freedreno/vulkan/tu_cmd_buffer.c b/src/freedreno/vulkan/tu_cmd_buffer.c
index d89a553fd9f..7df98b34e05 100644
--- a/src/freedreno/vulkan/tu_cmd_buffer.c
+++ b/src/freedreno/vulkan/tu_cmd_buffer.c
@@ -2189,7 +2189,7 @@ tu_CmdExecuteCommands(VkCommandBuffer commandBuffer,
MSM_SUBMIT_BO_READ | MSM_SUBMIT_BO_DUMP);
}
- tu_cs_emit_call(&cmd->cs, &secondary->cs);
+ tu_cs_add_entries(&cmd->cs, &secondary->cs);
}
}
cmd->state.dirty = ~0u; /* TODO: set dirty only what needs to be */
More information about the mesa-commit
mailing list