radv: do not add the image BO in radv_set_depth_clear_regs()
authorSamuel Pitoiset <samuel.pitoiset@gmail.com>
Wed, 15 Nov 2017 14:44:00 +0000 (15:44 +0100)
committerSamuel Pitoiset <samuel.pitoiset@gmail.com>
Mon, 20 Nov 2017 09:45:23 +0000 (10:45 +0100)
For the fast path, radv_fill_buffer() ensures that the BO is
already in the list. For the slow path, the depth surface is
part of the framebuffer which means the BO is added to the list
when the framebuffer is emitted.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
src/amd/vulkan/radv_cmd_buffer.c

index 04b8147df8ac9f538f0e5c9604aed5e0c34f4648..ae522d2088fdd6c4eee1fa51c7f95cd3d2037ce6 100644 (file)
@@ -1309,8 +1309,6 @@ radv_set_depth_clear_regs(struct radv_cmd_buffer *cmd_buffer,
        if (aspects & VK_IMAGE_ASPECT_DEPTH_BIT)
                ++reg_count;
 
-       radv_cs_add_buffer(cmd_buffer->device->ws, cmd_buffer->cs, image->bo, 8);
-
        radeon_emit(cmd_buffer->cs, PKT3(PKT3_WRITE_DATA, 2 + reg_count, 0));
        radeon_emit(cmd_buffer->cs, S_370_DST_SEL(V_370_MEM_ASYNC) |
                                    S_370_WR_CONFIRM(1) |