Mesa (master): radv: Only flush for the needed stages, and before the flushes.

Bas Nieuwenhuizen bnieuwenhuizen at kemper.freedesktop.org
Thu Mar 9 01:36:54 UTC 2017


Module: Mesa
Branch: master
Commit: 9251f8b35ef081db65cb8c2018d7f8f4bc3c1be3
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=9251f8b35ef081db65cb8c2018d7f8f4bc3c1be3

Author: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>
Date:   Wed Mar  8 23:51:01 2017 +0100

radv: Only flush for the needed stages, and before the flushes.

Signed-off-by: Bas Nieuwenhuizen <basni at google.com>
Reviewed-by: Dave Airlie <airlied at redhat.com>

---

 src/amd/vulkan/radv_cmd_buffer.c | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c
index 5d4236c..274495f 100644
--- a/src/amd/vulkan/radv_cmd_buffer.c
+++ b/src/amd/vulkan/radv_cmd_buffer.c
@@ -2928,6 +2928,7 @@ void radv_CmdPipelineBarrier(
 		                                        image);
 	}
 
+	radv_stage_flush(cmd_buffer, srcStageMask);
 	cmd_buffer->state.flush_bits |= src_flush_bits;
 
 	for (uint32_t i = 0; i < imageMemoryBarrierCount; i++) {
@@ -2942,12 +2943,6 @@ void radv_CmdPipelineBarrier(
 	}
 
 	cmd_buffer->state.flush_bits |= dst_flush_bits;
-
-	/* TODO reduce this */
-	enum radv_cmd_flush_bits flush_bits = RADV_CMD_FLAG_CS_PARTIAL_FLUSH |
-		RADV_CMD_FLAG_PS_PARTIAL_FLUSH;
-
-	cmd_buffer->state.flush_bits |= flush_bits;
 }
 
 




More information about the mesa-commit mailing list