Mesa (main): dzn: Don't call dzn_cmd_buffer_queue_transition_barriers() prematurely

[GitLab Mirror]

Module: Mesa
Branch: main
Commit: 9eb70b5dcf0f21d97f500f5860562040534dc173
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=9eb70b5dcf0f21d97f500f5860562040534dc173

Author: Boris Brezillon <boris.brezillon at collabora.com>
Date:   Wed Jun 29 17:51:44 2022 +0200

dzn: Don't call dzn_cmd_buffer_queue_transition_barriers() prematurely

We want to pack ResourceBarriers() call as much as we can, so let's
not dzn_cmd_buffer_queue_transition_barriers() when we could still queue
new barriers.

Reviewed-by: Jesse Natalie <jenatali at microsoft.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17250>

---

 src/microsoft/vulkan/dzn_cmd_buffer.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/microsoft/vulkan/dzn_cmd_buffer.c b/src/microsoft/vulkan/dzn_cmd_buffer.c
index 799f00bcd3f..f0a9f71fd81 100644
--- a/src/microsoft/vulkan/dzn_cmd_buffer.c
+++ b/src/microsoft/vulkan/dzn_cmd_buffer.c
@@ -178,8 +178,10 @@ dzn_cmd_buffer_queue_image_subres_range_transition(struct dzn_cmd_buffer *cmdbuf
          if (!barrier_count) {
             first_barrier = subres;
             barrier_count = level_count;
+            continue;
          } else if (first_barrier + barrier_count == subres) {
             barrier_count += level_count;
+            continue;
          }
 
          ret = dzn_cmd_buffer_queue_transition_barriers(cmdbuf, image->res,