diff --git a/src/imagination/vulkan/pvr_cmd_buffer.c b/src/imagination/vulkan/pvr_cmd_buffer.c index 4d6f529e649..daa36e95f4d 100644 --- a/src/imagination/vulkan/pvr_cmd_buffer.c +++ b/src/imagination/vulkan/pvr_cmd_buffer.c @@ -1168,6 +1168,7 @@ pvr_sub_cmd_gfx_align_ds_subtiles(struct pvr_cmd_buffer *const cmd_buffer, */ assert(list_last_entry(&cmd_buffer->sub_cmds, struct pvr_sub_cmd, link) == prev_sub_cmd); + assert(prev_sub_cmd == cmd_buffer->state.current_sub_cmd); if (!pvr_ds_attachment_requires_zls(ds)) return VK_SUCCESS; @@ -1222,6 +1223,8 @@ pvr_sub_cmd_gfx_align_ds_subtiles(struct pvr_cmd_buffer *const cmd_buffer, }; if (ds->load.d || ds->load.s) { + struct pvr_sub_cmd *new_sub_cmd; + cmd_buffer->state.current_sub_cmd = NULL; result = @@ -1229,6 +1232,8 @@ pvr_sub_cmd_gfx_align_ds_subtiles(struct pvr_cmd_buffer *const cmd_buffer, if (result != VK_SUCCESS) return result; + new_sub_cmd = cmd_buffer->state.current_sub_cmd; + result = pvr_copy_image_to_buffer_region_format(cmd_buffer, ds_image, buffer->dev_addr, @@ -1238,7 +1243,7 @@ pvr_sub_cmd_gfx_align_ds_subtiles(struct pvr_cmd_buffer *const cmd_buffer, if (result != VK_SUCCESS) return result; - cmd_buffer->state.current_sub_cmd->transfer.serialize_with_frag = true; + new_sub_cmd->transfer.serialize_with_frag = true; result = pvr_cmd_buffer_end_sub_cmd(cmd_buffer); if (result != VK_SUCCESS) @@ -1247,8 +1252,7 @@ pvr_sub_cmd_gfx_align_ds_subtiles(struct pvr_cmd_buffer *const cmd_buffer, /* Now we have to fiddle with cmd_buffer to place this transfer command * *before* the target gfx subcommand. */ - list_move_to(&cmd_buffer->state.current_sub_cmd->link, - &prev_sub_cmd->link); + list_move_to(&new_sub_cmd->link, &prev_sub_cmd->link); cmd_buffer->state.current_sub_cmd = prev_sub_cmd; }