Mesa (master): radv: Do a cache flush if needed before reading predicates.
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Mon Dec 31 20:02:03 UTC 2018
Module: Mesa
Branch: master
Commit: 8c93ef5de98a90a93434d351c769f3c06f206397
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=8c93ef5de98a90a93434d351c769f3c06f206397
Author: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Date: Sun Dec 30 23:37:11 2018 +0100
radv: Do a cache flush if needed before reading predicates.
This caused random failures for two conditional rendering tests:
dEQP-VK.conditional_rendering.draw_clear.draw.update_with_rendering_discard
dEQP-VK.conditional_rendering.draw_clear.draw.update_with_rendering_no_discard
These wrote the predicate with the vertex shader, did a barrier and then
started the conditional rendering. However the cache flushes for the barrier
only happen on first draw, so after the predicate has been read.
Fixes: e45ba51ea45 "radv: add support for VK_EXT_conditional_rendering"
Reviewed-by: Dave Airlie <airlied at redhat.com>
---
src/amd/vulkan/radv_cmd_buffer.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c
index 3361df1f34..1966098e08 100644
--- a/src/amd/vulkan/radv_cmd_buffer.c
+++ b/src/amd/vulkan/radv_cmd_buffer.c
@@ -4761,6 +4761,8 @@ void radv_CmdBeginConditionalRenderingEXT(
draw_visible = false;
}
+ si_emit_cache_flush(cmd_buffer);
+
/* Enable predication for this command buffer. */
si_emit_set_predication_state(cmd_buffer, draw_visible, va);
cmd_buffer->state.predicating = true;
More information about the mesa-commit
mailing list