diff --git a/src/imagination/vulkan/pvr_cmd_buffer.c b/src/imagination/vulkan/pvr_cmd_buffer.c index daa36e95f4d..c9acf4dc5c6 100644 --- a/src/imagination/vulkan/pvr_cmd_buffer.c +++ b/src/imagination/vulkan/pvr_cmd_buffer.c @@ -1251,8 +1251,11 @@ 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. + * + * Note the doc for list_move_to() is subtly wrong - item is placed + * directly *after* loc in the list, not "in front of". */ - list_move_to(&new_sub_cmd->link, &prev_sub_cmd->link); + list_move_to(&new_sub_cmd->link, prev_sub_cmd->link.prev); cmd_buffer->state.current_sub_cmd = prev_sub_cmd; }