This includes metadata as well. On GFX10, we have to invalidate
the L2 metadata cache when shaders read DCC.

Note that we still have to implement GFX10 coherency by
introducing INV_L2_METATADA but for now just flush L2.

This fixes a corruption with DCC and Talos.

Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com>
---
 src/amd/vulkan/radv_cmd_buffer.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c
index 86b5c812405..2bbb6a52096 100644
--- a/src/amd/vulkan/radv_cmd_buffer.c
+++ b/src/amd/vulkan/radv_cmd_buffer.c
@@ -2690,7 +2690,9 @@ radv_dst_access_flush(struct radv_cmd_buffer *cmd_buffer,
                if (!radv_image_has_htile(image))
                        flush_DB_meta = false;
 
-               if (cmd_buffer->device->physical_device->rad_info.chip_class >= 
GFX9) {
+               /* TODO: implement shader coherent for GFX10 */
+
+               if (cmd_buffer->device->physical_device->rad_info.chip_class == 
GFX9) {
                        if (image->info.samples == 1 &&
                            (image->usage & 
(VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT |
                                             
VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT)) &&
-- 
2.22.0

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to