On 26 Jan. 2018 01:10, "Matthew Nicholls" <mnicho...@feralinteractive.com>
wrote:

Otherwise cache flushes could get conditionally disabled while still
clearing
the flush_bits, and thus flushes due to application pipeline barriers may
never
get executed.


I wonder would we better not predicating flushes. I added that as an extra
opt, but it might be the wrong move.

Dave.


Cc: mesa-sta...@lists.freedesktop.org
---
 src/amd/vulkan/radv_meta_fast_clear.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/amd/vulkan/radv_meta_fast_clear.c
b/src/amd/vulkan/radv_meta_fast_clear.c
index fdeeaeedbf..f4353fd889 100644
--- a/src/amd/vulkan/radv_meta_fast_clear.c
+++ b/src/amd/vulkan/radv_meta_fast_clear.c
@@ -602,6 +602,8 @@ radv_emit_color_decompress(struct radv_cmd_buffer
*cmd_buffer,
        }

        if (!decompress_dcc && image->surface.dcc_size) {
+               si_emit_cache_flush(cmd_buffer);
+
                radv_emit_set_predication_state_from_image(cmd_buffer,
image, true);
                cmd_buffer->state.predicating = true;
        }
--
2.13.6
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to