radv: fix a potential crash if attachments allocation failed
[mesa.git] / src / amd / vulkan / radv_cmd_buffer.c
index a2578126c5259db853cfc20798308b651386a800..5f22733a7d9e5cf7ceb4ee5649844ac82eba51c6 100644 (file)
 
 #include "radv_private.h"
 #include "radv_radeon_winsys.h"
+#include "radv_shader.h"
 #include "radv_cs.h"
 #include "sid.h"
 #include "gfx9d.h"
 #include "vk_format.h"
+#include "radv_debug.h"
 #include "radv_meta.h"
 
 #include "ac_debug.h"
@@ -141,7 +143,6 @@ static VkResult radv_create_cmd_buffer(
        VkCommandBuffer*                            pCommandBuffer)
 {
        struct radv_cmd_buffer *cmd_buffer;
-       VkResult result;
        unsigned ring;
        cmd_buffer = vk_alloc(&pool->alloc, sizeof(*cmd_buffer), 8,
                                VK_SYSTEM_ALLOCATION_SCOPE_OBJECT);
@@ -170,8 +171,8 @@ static VkResult radv_create_cmd_buffer(
 
        cmd_buffer->cs = device->ws->cs_create(device->ws, ring);
        if (!cmd_buffer->cs) {
-               result = VK_ERROR_OUT_OF_HOST_MEMORY;
-               goto fail;
+               vk_free(&cmd_buffer->pool->alloc, cmd_buffer);
+               return vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
        }
 
        *pCommandBuffer = radv_cmd_buffer_to_handle(cmd_buffer);
@@ -181,11 +182,6 @@ static VkResult radv_create_cmd_buffer(
        list_inithead(&cmd_buffer->upload.list);
 
        return VK_SUCCESS;
-
-fail:
-       vk_free(&cmd_buffer->pool->alloc, cmd_buffer);
-
-       return result;
 }
 
 static void
@@ -207,7 +203,8 @@ radv_cmd_buffer_destroy(struct radv_cmd_buffer *cmd_buffer)
        vk_free(&cmd_buffer->pool->alloc, cmd_buffer);
 }
 
-static void  radv_reset_cmd_buffer(struct radv_cmd_buffer *cmd_buffer)
+static VkResult
+radv_reset_cmd_buffer(struct radv_cmd_buffer *cmd_buffer)
 {
 
        cmd_buffer->device->ws->cs_reset(cmd_buffer->cs);
@@ -219,6 +216,7 @@ static void  radv_reset_cmd_buffer(struct radv_cmd_buffer *cmd_buffer)
                free(up);
        }
 
+       cmd_buffer->push_constant_stages = 0;
        cmd_buffer->scratch_size_needed = 0;
        cmd_buffer->compute_scratch_size_needed = 0;
        cmd_buffer->esgs_ring_size_needed = 0;
@@ -231,7 +229,7 @@ static void  radv_reset_cmd_buffer(struct radv_cmd_buffer *cmd_buffer)
                                                      cmd_buffer->upload.upload_bo, 8);
        cmd_buffer->upload.offset = 0;
 
-       cmd_buffer->record_fail = false;
+       cmd_buffer->record_result = VK_SUCCESS;
 
        cmd_buffer->ring_offsets_idx = -1;
 
@@ -242,6 +240,8 @@ static void  radv_reset_cmd_buffer(struct radv_cmd_buffer *cmd_buffer)
                                             &fence_ptr);
                cmd_buffer->gfx9_fence_bo = cmd_buffer->upload.upload_bo;
        }
+
+       return cmd_buffer->record_result;
 }
 
 static bool
@@ -262,7 +262,7 @@ radv_cmd_buffer_resize_upload_buf(struct radv_cmd_buffer *cmd_buffer,
                                       RADEON_FLAG_CPU_ACCESS);
 
        if (!bo) {
-               cmd_buffer->record_fail = true;
+               cmd_buffer->record_result = VK_ERROR_OUT_OF_DEVICE_MEMORY;
                return false;
        }
 
@@ -271,7 +271,7 @@ radv_cmd_buffer_resize_upload_buf(struct radv_cmd_buffer *cmd_buffer,
                upload = malloc(sizeof(*upload));
 
                if (!upload) {
-                       cmd_buffer->record_fail = true;
+                       cmd_buffer->record_result = VK_ERROR_OUT_OF_HOST_MEMORY;
                        device->ws->buffer_destroy(bo);
                        return false;
                }
@@ -286,7 +286,7 @@ radv_cmd_buffer_resize_upload_buf(struct radv_cmd_buffer *cmd_buffer,
        cmd_buffer->upload.map = device->ws->buffer_map(cmd_buffer->upload.upload_bo);
 
        if (!cmd_buffer->upload.map) {
-               cmd_buffer->record_fail = true;
+               cmd_buffer->record_result = VK_ERROR_OUT_OF_DEVICE_MEMORY;
                return false;
        }
 
@@ -331,6 +331,19 @@ radv_cmd_buffer_upload_data(struct radv_cmd_buffer *cmd_buffer,
        return true;
 }
 
+static void
+radv_emit_write_data_packet(struct radeon_winsys_cs *cs, uint64_t va,
+                           unsigned count, const uint32_t *data)
+{
+       radeon_emit(cs, PKT3(PKT3_WRITE_DATA, 2 + count, 0));
+       radeon_emit(cs, S_370_DST_SEL(V_370_MEM_ASYNC) |
+                   S_370_WR_CONFIRM(1) |
+                   S_370_ENGINE_SEL(V_370_ME));
+       radeon_emit(cs, va);
+       radeon_emit(cs, va >> 32);
+       radeon_emit_array(cs, data, count);
+}
+
 void radv_cmd_buffer_trace_emit(struct radv_cmd_buffer *cmd_buffer)
 {
        struct radv_device *device = cmd_buffer->device;
@@ -341,22 +354,102 @@ void radv_cmd_buffer_trace_emit(struct radv_cmd_buffer *cmd_buffer)
                return;
 
        va = device->ws->buffer_get_va(device->trace_bo);
+       if (cmd_buffer->level == VK_COMMAND_BUFFER_LEVEL_SECONDARY)
+               va += 4;
 
        MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 7);
 
        ++cmd_buffer->state.trace_id;
        device->ws->cs_add_buffer(cs, device->trace_bo, 8);
-       radeon_emit(cs, PKT3(PKT3_WRITE_DATA, 3, 0));
-       radeon_emit(cs, S_370_DST_SEL(V_370_MEM_ASYNC) |
-                   S_370_WR_CONFIRM(1) |
-                   S_370_ENGINE_SEL(V_370_ME));
-       radeon_emit(cs, va);
-       radeon_emit(cs, va >> 32);
-       radeon_emit(cs, cmd_buffer->state.trace_id);
+       radv_emit_write_data_packet(cs, va, 1, &cmd_buffer->state.trace_id);
        radeon_emit(cs, PKT3(PKT3_NOP, 0, 0));
        radeon_emit(cs, AC_ENCODE_TRACE_POINT(cmd_buffer->state.trace_id));
 }
 
+static void
+radv_cmd_buffer_after_draw(struct radv_cmd_buffer *cmd_buffer)
+{
+       if (cmd_buffer->device->debug_flags & RADV_DEBUG_SYNC_SHADERS) {
+               enum radv_cmd_flush_bits flags;
+
+               /* Force wait for graphics/compute engines to be idle. */
+               flags = RADV_CMD_FLAG_PS_PARTIAL_FLUSH |
+                       RADV_CMD_FLAG_CS_PARTIAL_FLUSH;
+
+               si_cs_emit_cache_flush(cmd_buffer->cs, false,
+                                      cmd_buffer->device->physical_device->rad_info.chip_class,
+                                      NULL, 0,
+                                      radv_cmd_buffer_uses_mec(cmd_buffer),
+                                      flags);
+       }
+
+       radv_cmd_buffer_trace_emit(cmd_buffer);
+}
+
+static void
+radv_save_pipeline(struct radv_cmd_buffer *cmd_buffer,
+                  struct radv_pipeline *pipeline, enum ring_type ring)
+{
+       struct radv_device *device = cmd_buffer->device;
+       struct radeon_winsys_cs *cs = cmd_buffer->cs;
+       uint32_t data[2];
+       uint64_t va;
+
+       if (!device->trace_bo)
+               return;
+
+       va = device->ws->buffer_get_va(device->trace_bo);
+
+       switch (ring) {
+       case RING_GFX:
+               va += 8;
+               break;
+       case RING_COMPUTE:
+               va += 16;
+               break;
+       default:
+               assert(!"invalid ring type");
+       }
+
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(device->ws,
+                                                          cmd_buffer->cs, 6);
+
+       data[0] = (uintptr_t)pipeline;
+       data[1] = (uintptr_t)pipeline >> 32;
+
+       device->ws->cs_add_buffer(cs, device->trace_bo, 8);
+       radv_emit_write_data_packet(cs, va, 2, data);
+}
+
+static void
+radv_save_descriptors(struct radv_cmd_buffer *cmd_buffer)
+{
+       struct radv_device *device = cmd_buffer->device;
+       struct radeon_winsys_cs *cs = cmd_buffer->cs;
+       uint32_t data[MAX_SETS * 2] = {};
+       uint64_t va;
+
+       if (!device->trace_bo)
+               return;
+
+       va = device->ws->buffer_get_va(device->trace_bo) + 24;
+
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(device->ws,
+                                                          cmd_buffer->cs, 4 + MAX_SETS * 2);
+
+       for (int i = 0; i < MAX_SETS; i++) {
+               struct radv_descriptor_set *set = cmd_buffer->state.descriptors[i];
+               if (!set)
+                       continue;
+
+               data[i * 2] = (uintptr_t)set;
+               data[i * 2 + 1] = (uintptr_t)set >> 32;
+       }
+
+       device->ws->cs_add_buffer(cs, device->trace_bo, 8);
+       radv_emit_write_data_packet(cs, va, MAX_SETS * 2, data);
+}
+
 static void
 radv_emit_graphics_blend_state(struct radv_cmd_buffer *cmd_buffer,
                               struct radv_pipeline *pipeline)
@@ -368,6 +461,10 @@ radv_emit_graphics_blend_state(struct radv_cmd_buffer *cmd_buffer,
        radeon_set_context_reg(cmd_buffer->cs, R_028B70_DB_ALPHA_TO_MASK, pipeline->graphics.blend.db_alpha_to_mask);
 
        if (cmd_buffer->device->physical_device->has_rbplus) {
+
+               radeon_set_context_reg_seq(cmd_buffer->cs, R_028760_SX_MRT0_BLEND_OPT, 8);
+               radeon_emit_array(cmd_buffer->cs, pipeline->graphics.blend.sx_mrt_blend_opt, 8);
+
                radeon_set_context_reg_seq(cmd_buffer->cs, R_028754_SX_PS_DOWNCONVERT, 3);
                radeon_emit(cmd_buffer->cs, 0); /* R_028754_SX_PS_DOWNCONVERT */
                radeon_emit(cmd_buffer->cs, 0); /* R_028758_SX_BLEND_OPT_EPSILON */
@@ -394,33 +491,6 @@ static unsigned radv_pack_float_12p4(float x)
               x >= 4096 ? 0xffff : x * 16;
 }
 
-uint32_t
-radv_shader_stage_to_user_data_0(gl_shader_stage stage, bool has_gs, bool has_tess)
-{
-       switch (stage) {
-       case MESA_SHADER_FRAGMENT:
-               return R_00B030_SPI_SHADER_USER_DATA_PS_0;
-       case MESA_SHADER_VERTEX:
-               if (has_tess)
-                       return R_00B530_SPI_SHADER_USER_DATA_LS_0;
-               else
-                       return has_gs ? R_00B330_SPI_SHADER_USER_DATA_ES_0 : R_00B130_SPI_SHADER_USER_DATA_VS_0;
-       case MESA_SHADER_GEOMETRY:
-               return R_00B230_SPI_SHADER_USER_DATA_GS_0;
-       case MESA_SHADER_COMPUTE:
-               return R_00B900_COMPUTE_USER_DATA_0;
-       case MESA_SHADER_TESS_CTRL:
-               return R_00B430_SPI_SHADER_USER_DATA_HS_0;
-       case MESA_SHADER_TESS_EVAL:
-               if (has_gs)
-                       return R_00B330_SPI_SHADER_USER_DATA_ES_0;
-               else
-                       return R_00B130_SPI_SHADER_USER_DATA_VS_0;
-       default:
-               unreachable("unknown shader");
-       }
-}
-
 struct ac_userdata_info *
 radv_lookup_user_sgpr(struct radv_pipeline *pipeline,
                      gl_shader_stage stage,
@@ -546,7 +616,7 @@ radv_emit_hw_vs(struct radv_cmd_buffer *cmd_buffer,
                struct ac_vs_output_info *outinfo)
 {
        struct radeon_winsys *ws = cmd_buffer->device->ws;
-       uint64_t va = ws->buffer_get_va(shader->bo);
+       uint64_t va = ws->buffer_get_va(shader->bo) + shader->bo_offset;
        unsigned export_count;
 
        ws->cs_add_buffer(cmd_buffer->cs, shader->bo, 8);
@@ -596,7 +666,7 @@ radv_emit_hw_es(struct radv_cmd_buffer *cmd_buffer,
                struct ac_es_output_info *outinfo)
 {
        struct radeon_winsys *ws = cmd_buffer->device->ws;
-       uint64_t va = ws->buffer_get_va(shader->bo);
+       uint64_t va = ws->buffer_get_va(shader->bo) + shader->bo_offset;
 
        ws->cs_add_buffer(cmd_buffer->cs, shader->bo, 8);
        radv_emit_prefetch(cmd_buffer, va, shader->code_size);
@@ -615,7 +685,7 @@ radv_emit_hw_ls(struct radv_cmd_buffer *cmd_buffer,
                struct radv_shader_variant *shader)
 {
        struct radeon_winsys *ws = cmd_buffer->device->ws;
-       uint64_t va = ws->buffer_get_va(shader->bo);
+       uint64_t va = ws->buffer_get_va(shader->bo) + shader->bo_offset;
        uint32_t rsrc2 = shader->rsrc2;
 
        ws->cs_add_buffer(cmd_buffer->cs, shader->bo, 8);
@@ -640,7 +710,7 @@ radv_emit_hw_hs(struct radv_cmd_buffer *cmd_buffer,
                struct radv_shader_variant *shader)
 {
        struct radeon_winsys *ws = cmd_buffer->device->ws;
-       uint64_t va = ws->buffer_get_va(shader->bo);
+       uint64_t va = ws->buffer_get_va(shader->bo) + shader->bo_offset;
 
        ws->cs_add_buffer(cmd_buffer->cs, shader->bo, 8);
        radv_emit_prefetch(cmd_buffer, va, shader->code_size);
@@ -775,7 +845,7 @@ radv_emit_geometry_shader(struct radv_cmd_buffer *cmd_buffer,
                               S_028B90_CNT(MIN2(gs_num_invocations, 127)) |
                               S_028B90_ENABLE(gs_num_invocations > 0));
 
-       va = ws->buffer_get_va(gs->bo);
+       va = ws->buffer_get_va(gs->bo) + gs->bo_offset;
        ws->cs_add_buffer(cmd_buffer->cs, gs->bo, 8);
        radv_emit_prefetch(cmd_buffer, va, gs->code_size);
 
@@ -816,8 +886,7 @@ radv_emit_fragment_shader(struct radv_cmd_buffer *cmd_buffer,
        assert (pipeline->shaders[MESA_SHADER_FRAGMENT]);
 
        ps = pipeline->shaders[MESA_SHADER_FRAGMENT];
-
-       va = ws->buffer_get_va(ps->bo);
+       va = ws->buffer_get_va(ps->bo) + ps->bo_offset;
        ws->cs_add_buffer(cmd_buffer->cs, ps->bo, 8);
        radv_emit_prefetch(cmd_buffer, va, ps->code_size);
 
@@ -836,7 +905,7 @@ radv_emit_fragment_shader(struct radv_cmd_buffer *cmd_buffer,
        radeon_set_context_reg(cmd_buffer->cs, R_0286D0_SPI_PS_INPUT_ADDR,
                               ps->config.spi_ps_input_addr);
 
-       if (ps->info.fs.force_persample)
+       if (ps->info.info.ps.force_persample)
                spi_baryc_cntl |= S_0286E0_POS_FLOAT_LOCATION(2);
 
        radeon_set_context_reg(cmd_buffer->cs, R_0286D8_SPI_PS_IN_CONTROL,
@@ -882,9 +951,10 @@ static void polaris_set_vgt_vertex_reuse(struct radv_cmd_buffer *cmd_buffer,
 }
 
 static void
-radv_emit_graphics_pipeline(struct radv_cmd_buffer *cmd_buffer,
-                           struct radv_pipeline *pipeline)
+radv_emit_graphics_pipeline(struct radv_cmd_buffer *cmd_buffer)
 {
+       struct radv_pipeline *pipeline = cmd_buffer->state.pipeline;
+
        if (!pipeline || cmd_buffer->state.emitted_pipeline == pipeline)
                return;
 
@@ -920,6 +990,8 @@ radv_emit_graphics_pipeline(struct radv_cmd_buffer *cmd_buffer,
        }
        radeon_set_context_reg(cmd_buffer->cs, R_028A6C_VGT_GS_OUT_PRIM_TYPE, pipeline->graphics.gs_out);
 
+       radv_save_pipeline(cmd_buffer, pipeline, RING_GFX);
+
        cmd_buffer->state.emitted_pipeline = pipeline;
 }
 
@@ -942,6 +1014,73 @@ radv_emit_scissor(struct radv_cmd_buffer *cmd_buffer)
                               cmd_buffer->state.pipeline->graphics.ms.pa_sc_mode_cntl_0 | S_028A48_VPORT_SCISSOR_ENABLE(count ? 1 : 0));
 }
 
+static void
+radv_emit_line_width(struct radv_cmd_buffer *cmd_buffer)
+{
+       unsigned width = cmd_buffer->state.dynamic.line_width * 8;
+
+       radeon_set_context_reg(cmd_buffer->cs, R_028A08_PA_SU_LINE_CNTL,
+                              S_028A08_WIDTH(CLAMP(width, 0, 0xFFF)));
+}
+
+static void
+radv_emit_blend_constants(struct radv_cmd_buffer *cmd_buffer)
+{
+       struct radv_dynamic_state *d = &cmd_buffer->state.dynamic;
+
+       radeon_set_context_reg_seq(cmd_buffer->cs, R_028414_CB_BLEND_RED, 4);
+       radeon_emit_array(cmd_buffer->cs, (uint32_t *)d->blend_constants, 4);
+}
+
+static void
+radv_emit_stencil(struct radv_cmd_buffer *cmd_buffer)
+{
+       struct radv_dynamic_state *d = &cmd_buffer->state.dynamic;
+
+       radeon_set_context_reg_seq(cmd_buffer->cs,
+                                  R_028430_DB_STENCILREFMASK, 2);
+       radeon_emit(cmd_buffer->cs,
+                   S_028430_STENCILTESTVAL(d->stencil_reference.front) |
+                   S_028430_STENCILMASK(d->stencil_compare_mask.front) |
+                   S_028430_STENCILWRITEMASK(d->stencil_write_mask.front) |
+                   S_028430_STENCILOPVAL(1));
+       radeon_emit(cmd_buffer->cs,
+                   S_028434_STENCILTESTVAL_BF(d->stencil_reference.back) |
+                   S_028434_STENCILMASK_BF(d->stencil_compare_mask.back) |
+                   S_028434_STENCILWRITEMASK_BF(d->stencil_write_mask.back) |
+                   S_028434_STENCILOPVAL_BF(1));
+}
+
+static void
+radv_emit_depth_bounds(struct radv_cmd_buffer *cmd_buffer)
+{
+       struct radv_dynamic_state *d = &cmd_buffer->state.dynamic;
+
+       radeon_set_context_reg(cmd_buffer->cs, R_028020_DB_DEPTH_BOUNDS_MIN,
+                              fui(d->depth_bounds.min));
+       radeon_set_context_reg(cmd_buffer->cs, R_028024_DB_DEPTH_BOUNDS_MAX,
+                              fui(d->depth_bounds.max));
+}
+
+static void
+radv_emit_depth_biais(struct radv_cmd_buffer *cmd_buffer)
+{
+       struct radv_raster_state *raster = &cmd_buffer->state.pipeline->graphics.raster;
+       struct radv_dynamic_state *d = &cmd_buffer->state.dynamic;
+       unsigned slope = fui(d->depth_bias.slope * 16.0f);
+       unsigned bias = fui(d->depth_bias.bias * cmd_buffer->state.offset_scale);
+
+       if (G_028814_POLY_OFFSET_FRONT_ENABLE(raster->pa_su_sc_mode_cntl)) {
+               radeon_set_context_reg_seq(cmd_buffer->cs,
+                                          R_028B7C_PA_SU_POLY_OFFSET_CLAMP, 5);
+               radeon_emit(cmd_buffer->cs, fui(d->depth_bias.clamp)); /* CLAMP */
+               radeon_emit(cmd_buffer->cs, slope); /* FRONT SCALE */
+               radeon_emit(cmd_buffer->cs, bias); /* FRONT OFFSET */
+               radeon_emit(cmd_buffer->cs, slope); /* BACK SCALE */
+               radeon_emit(cmd_buffer->cs, bias); /* BACK OFFSET */
+       }
+}
+
 static void
 radv_emit_fb_color_state(struct radv_cmd_buffer *cmd_buffer,
                         int index,
@@ -1007,6 +1146,8 @@ radv_emit_fb_ds_state(struct radv_cmd_buffer *cmd_buffer,
        }
 
        radeon_set_context_reg(cmd_buffer->cs, R_028008_DB_DEPTH_VIEW, ds->db_depth_view);
+       radeon_set_context_reg(cmd_buffer->cs, R_028ABC_DB_HTILE_SURFACE, ds->db_htile_surface);
+
 
        if (cmd_buffer->device->physical_device->rad_info.chip_class >= GFX9) {
                radeon_set_context_reg_seq(cmd_buffer->cs, R_028014_DB_HTILE_DATA_BASE, 3);
@@ -1043,7 +1184,6 @@ radv_emit_fb_ds_state(struct radv_cmd_buffer *cmd_buffer,
                radeon_emit(cmd_buffer->cs, ds->db_depth_size); /* R_028058_DB_DEPTH_SIZE */
                radeon_emit(cmd_buffer->cs, ds->db_depth_slice);        /* R_02805C_DB_DEPTH_SLICE */
 
-               radeon_set_context_reg(cmd_buffer->cs, R_028ABC_DB_HTILE_SURFACE, ds->db_htile_surface);
        }
 
        radeon_set_context_reg(cmd_buffer->cs, R_028B78_PA_SU_POLY_OFFSET_DB_FMT_CNTL,
@@ -1117,6 +1257,35 @@ radv_load_depth_clear_regs(struct radv_cmd_buffer *cmd_buffer,
        radeon_emit(cmd_buffer->cs, 0);
 }
 
+/*
+ *with DCC some colors don't require CMASK elimiation before being
+ * used as a texture. This sets a predicate value to determine if the
+ * cmask eliminate is required.
+ */
+void
+radv_set_dcc_need_cmask_elim_pred(struct radv_cmd_buffer *cmd_buffer,
+                                 struct radv_image *image,
+                                 bool value)
+{
+       uint64_t pred_val = value;
+       uint64_t va = cmd_buffer->device->ws->buffer_get_va(image->bo);
+       va += image->offset + image->dcc_pred_offset;
+
+       if (!image->surface.dcc_size)
+               return;
+
+       cmd_buffer->device->ws->cs_add_buffer(cmd_buffer->cs, image->bo, 8);
+
+       radeon_emit(cmd_buffer->cs, PKT3(PKT3_WRITE_DATA, 4, 0));
+       radeon_emit(cmd_buffer->cs, S_370_DST_SEL(V_370_MEM_ASYNC) |
+                                   S_370_WR_CONFIRM(1) |
+                                   S_370_ENGINE_SEL(V_370_PFP));
+       radeon_emit(cmd_buffer->cs, va);
+       radeon_emit(cmd_buffer->cs, va >> 32);
+       radeon_emit(cmd_buffer->cs, pred_val);
+       radeon_emit(cmd_buffer->cs, pred_val >> 32);
+}
+
 void
 radv_set_color_clear_regs(struct radv_cmd_buffer *cmd_buffer,
                          struct radv_image *image,
@@ -1179,7 +1348,17 @@ radv_emit_framebuffer_state(struct radv_cmd_buffer *cmd_buffer)
        struct radv_framebuffer *framebuffer = cmd_buffer->state.framebuffer;
        const struct radv_subpass *subpass = cmd_buffer->state.subpass;
 
-       for (i = 0; i < subpass->color_count; ++i) {
+       /* this may happen for inherited secondary recording */
+       if (!framebuffer)
+               return;
+
+       for (i = 0; i < 8; ++i) {
+               if (i >= subpass->color_count || subpass->color_attachments[i].attachment == VK_ATTACHMENT_UNUSED) {
+                       radeon_set_context_reg(cmd_buffer->cs, R_028C70_CB_COLOR0_INFO + i * 0x3C,
+                                      S_028C70_FORMAT(V_028C70_COLOR_INVALID));
+                       continue;
+               }
+
                int idx = subpass->color_attachments[i].attachment;
                struct radv_attachment_info *att = &framebuffer->attachments[idx];
 
@@ -1191,19 +1370,15 @@ radv_emit_framebuffer_state(struct radv_cmd_buffer *cmd_buffer)
                radv_load_color_clear_regs(cmd_buffer, att->attachment->image, i);
        }
 
-       for (i = subpass->color_count; i < 8; i++)
-               radeon_set_context_reg(cmd_buffer->cs, R_028C70_CB_COLOR0_INFO + i * 0x3C,
-                                      S_028C70_FORMAT(V_028C70_COLOR_INVALID));
-
        if(subpass->depth_stencil_attachment.attachment != VK_ATTACHMENT_UNUSED) {
                int idx = subpass->depth_stencil_attachment.attachment;
                VkImageLayout layout = subpass->depth_stencil_attachment.layout;
                struct radv_attachment_info *att = &framebuffer->attachments[idx];
                struct radv_image *image = att->attachment->image;
                cmd_buffer->device->ws->cs_add_buffer(cmd_buffer->cs, att->attachment->bo, 8);
-               uint32_t queue_mask = radv_image_queue_family_mask(image,
-                                                                  cmd_buffer->queue_family_index,
-                                                                  cmd_buffer->queue_family_index);
+               MAYBE_UNUSED uint32_t queue_mask = radv_image_queue_family_mask(image,
+                                                                               cmd_buffer->queue_family_index,
+                                                                               cmd_buffer->queue_family_index);
                /* We currently don't support writing decompressed HTILE */
                assert(radv_layout_has_htile(image, layout, queue_mask) ==
                       radv_layout_is_htile_compressed(image, layout, queue_mask));
@@ -1216,9 +1391,13 @@ radv_emit_framebuffer_state(struct radv_cmd_buffer *cmd_buffer)
                }
                radv_load_depth_clear_regs(cmd_buffer, image);
        } else {
-               radeon_set_context_reg_seq(cmd_buffer->cs, R_028040_DB_Z_INFO, 2);
-               radeon_emit(cmd_buffer->cs, S_028040_FORMAT(V_028040_Z_INVALID)); /* R_028040_DB_Z_INFO */
-               radeon_emit(cmd_buffer->cs, S_028044_FORMAT(V_028044_STENCIL_INVALID)); /* R_028044_DB_STENCIL_INFO */
+               if (cmd_buffer->device->physical_device->rad_info.chip_class >= GFX9)
+                       radeon_set_context_reg_seq(cmd_buffer->cs, R_028038_DB_Z_INFO, 2);
+               else
+                       radeon_set_context_reg_seq(cmd_buffer->cs, R_028040_DB_Z_INFO, 2);
+
+               radeon_emit(cmd_buffer->cs, S_028040_FORMAT(V_028040_Z_INVALID)); /* DB_Z_INFO */
+               radeon_emit(cmd_buffer->cs, S_028044_FORMAT(V_028044_STENCIL_INVALID)); /* DB_STENCIL_INFO */
        }
        radeon_set_context_reg(cmd_buffer->cs, R_028208_PA_SC_WINDOW_SCISSOR_BR,
                               S_028208_BR_X(framebuffer->width) |
@@ -1259,8 +1438,6 @@ void radv_set_db_count_control(struct radv_cmd_buffer *cmd_buffer)
 static void
 radv_cmd_buffer_flush_dynamic_state(struct radv_cmd_buffer *cmd_buffer)
 {
-       struct radv_dynamic_state *d = &cmd_buffer->state.dynamic;
-
        if (G_028810_DX_RASTERIZATION_KILL(cmd_buffer->state.pipeline->graphics.raster.pa_cl_clip_cntl))
                return;
 
@@ -1270,52 +1447,24 @@ radv_cmd_buffer_flush_dynamic_state(struct radv_cmd_buffer *cmd_buffer)
        if (cmd_buffer->state.dirty & (RADV_CMD_DIRTY_DYNAMIC_SCISSOR | RADV_CMD_DIRTY_DYNAMIC_VIEWPORT))
                radv_emit_scissor(cmd_buffer);
 
-       if (cmd_buffer->state.dirty & RADV_CMD_DIRTY_DYNAMIC_LINE_WIDTH) {
-               unsigned width = cmd_buffer->state.dynamic.line_width * 8;
-               radeon_set_context_reg(cmd_buffer->cs, R_028A08_PA_SU_LINE_CNTL,
-                                      S_028A08_WIDTH(CLAMP(width, 0, 0xFFF)));
-       }
+       if (cmd_buffer->state.dirty & RADV_CMD_DIRTY_DYNAMIC_LINE_WIDTH)
+               radv_emit_line_width(cmd_buffer);
 
-       if (cmd_buffer->state.dirty & RADV_CMD_DIRTY_DYNAMIC_BLEND_CONSTANTS) {
-               radeon_set_context_reg_seq(cmd_buffer->cs, R_028414_CB_BLEND_RED, 4);
-               radeon_emit_array(cmd_buffer->cs, (uint32_t*)d->blend_constants, 4);
-       }
+       if (cmd_buffer->state.dirty & RADV_CMD_DIRTY_DYNAMIC_BLEND_CONSTANTS)
+               radv_emit_blend_constants(cmd_buffer);
 
        if (cmd_buffer->state.dirty & (RADV_CMD_DIRTY_DYNAMIC_STENCIL_REFERENCE |
                                       RADV_CMD_DIRTY_DYNAMIC_STENCIL_WRITE_MASK |
-                                      RADV_CMD_DIRTY_DYNAMIC_STENCIL_COMPARE_MASK)) {
-               radeon_set_context_reg_seq(cmd_buffer->cs, R_028430_DB_STENCILREFMASK, 2);
-               radeon_emit(cmd_buffer->cs, S_028430_STENCILTESTVAL(d->stencil_reference.front) |
-                           S_028430_STENCILMASK(d->stencil_compare_mask.front) |
-                           S_028430_STENCILWRITEMASK(d->stencil_write_mask.front) |
-                           S_028430_STENCILOPVAL(1));
-               radeon_emit(cmd_buffer->cs, S_028434_STENCILTESTVAL_BF(d->stencil_reference.back) |
-                           S_028434_STENCILMASK_BF(d->stencil_compare_mask.back) |
-                           S_028434_STENCILWRITEMASK_BF(d->stencil_write_mask.back) |
-                           S_028434_STENCILOPVAL_BF(1));
-       }
+                                      RADV_CMD_DIRTY_DYNAMIC_STENCIL_COMPARE_MASK))
+               radv_emit_stencil(cmd_buffer);
 
        if (cmd_buffer->state.dirty & (RADV_CMD_DIRTY_PIPELINE |
-                                      RADV_CMD_DIRTY_DYNAMIC_DEPTH_BOUNDS)) {
-               radeon_set_context_reg(cmd_buffer->cs, R_028020_DB_DEPTH_BOUNDS_MIN, fui(d->depth_bounds.min));
-               radeon_set_context_reg(cmd_buffer->cs, R_028024_DB_DEPTH_BOUNDS_MAX, fui(d->depth_bounds.max));
-       }
+                                      RADV_CMD_DIRTY_DYNAMIC_DEPTH_BOUNDS))
+               radv_emit_depth_bounds(cmd_buffer);
 
        if (cmd_buffer->state.dirty & (RADV_CMD_DIRTY_PIPELINE |
-                                      RADV_CMD_DIRTY_DYNAMIC_DEPTH_BIAS)) {
-               struct radv_raster_state *raster = &cmd_buffer->state.pipeline->graphics.raster;
-               unsigned slope = fui(d->depth_bias.slope * 16.0f);
-               unsigned bias = fui(d->depth_bias.bias * cmd_buffer->state.offset_scale);
-
-               if (G_028814_POLY_OFFSET_FRONT_ENABLE(raster->pa_su_sc_mode_cntl)) {
-                       radeon_set_context_reg_seq(cmd_buffer->cs, R_028B7C_PA_SU_POLY_OFFSET_CLAMP, 5);
-                       radeon_emit(cmd_buffer->cs, fui(d->depth_bias.clamp)); /* CLAMP */
-                       radeon_emit(cmd_buffer->cs, slope); /* FRONT SCALE */
-                       radeon_emit(cmd_buffer->cs, bias); /* FRONT OFFSET */
-                       radeon_emit(cmd_buffer->cs, slope); /* BACK SCALE */
-                       radeon_emit(cmd_buffer->cs, bias); /* BACK OFFSET */
-               }
-       }
+                                      RADV_CMD_DIRTY_DYNAMIC_DEPTH_BIAS))
+               radv_emit_depth_biais(cmd_buffer);
 
        cmd_buffer->state.dirty = 0;
 }
@@ -1366,18 +1515,15 @@ static void
 radv_flush_push_descriptors(struct radv_cmd_buffer *cmd_buffer)
 {
        struct radv_descriptor_set *set = &cmd_buffer->push_descriptors.set;
-       uint32_t *ptr = NULL;
        unsigned bo_offset;
 
-       if (!radv_cmd_buffer_upload_alloc(cmd_buffer, set->size, 32,
-                                         &bo_offset,
-                                         (void**) &ptr))
+       if (!radv_cmd_buffer_upload_data(cmd_buffer, set->size, 32,
+                                        set->mapped_ptr,
+                                        &bo_offset))
                return;
 
        set->va = cmd_buffer->device->ws->buffer_get_va(cmd_buffer->upload.upload_bo);
        set->va += bo_offset;
-
-       memcpy(ptr, set->mapped_ptr, set->size);
 }
 
 static void
@@ -1463,6 +1609,9 @@ radv_flush_descriptors(struct radv_cmd_buffer *cmd_buffer,
        }
        cmd_buffer->state.descriptors_dirty = 0;
        cmd_buffer->state.push_descriptors_dirty = false;
+
+       radv_save_descriptors(cmd_buffer);
+
        assert(cmd_buffer->cs->cdw <= cdw_max);
 }
 
@@ -1533,43 +1682,45 @@ static void radv_emit_primitive_reset_state(struct radv_cmd_buffer *cmd_buffer,
        }
 }
 
-static void
+static bool
 radv_cmd_buffer_update_vertex_descriptors(struct radv_cmd_buffer *cmd_buffer)
 {
        struct radv_device *device = cmd_buffer->device;
 
        if ((cmd_buffer->state.pipeline != cmd_buffer->state.emitted_pipeline || cmd_buffer->state.vb_dirty) &&
-           cmd_buffer->state.pipeline->num_vertex_attribs &&
+           cmd_buffer->state.pipeline->vertex_elements.count &&
            cmd_buffer->state.pipeline->shaders[MESA_SHADER_VERTEX]->info.info.vs.has_vertex_buffers) {
+               struct radv_vertex_elements_info *velems = &cmd_buffer->state.pipeline->vertex_elements;
                unsigned vb_offset;
                void *vb_ptr;
                uint32_t i = 0;
-               uint32_t num_attribs = cmd_buffer->state.pipeline->num_vertex_attribs;
+               uint32_t count = velems->count;
                uint64_t va;
 
                /* allocate some descriptor state for vertex buffers */
-               radv_cmd_buffer_upload_alloc(cmd_buffer, num_attribs * 16, 256,
-                                            &vb_offset, &vb_ptr);
+               if (!radv_cmd_buffer_upload_alloc(cmd_buffer, count * 16, 256,
+                                                 &vb_offset, &vb_ptr))
+                       return false;
 
-               for (i = 0; i < num_attribs; i++) {
+               for (i = 0; i < count; i++) {
                        uint32_t *desc = &((uint32_t *)vb_ptr)[i * 4];
                        uint32_t offset;
-                       int vb = cmd_buffer->state.pipeline->va_binding[i];
+                       int vb = velems->binding[i];
                        struct radv_buffer *buffer = cmd_buffer->state.vertex_bindings[vb].buffer;
                        uint32_t stride = cmd_buffer->state.pipeline->binding_stride[vb];
 
                        device->ws->cs_add_buffer(cmd_buffer->cs, buffer->bo, 8);
                        va = device->ws->buffer_get_va(buffer->bo);
 
-                       offset = cmd_buffer->state.vertex_bindings[vb].offset + cmd_buffer->state.pipeline->va_offset[i];
+                       offset = cmd_buffer->state.vertex_bindings[vb].offset + velems->offset[i];
                        va += offset + buffer->offset;
                        desc[0] = va;
                        desc[1] = S_008F04_BASE_ADDRESS_HI(va >> 32) | S_008F04_STRIDE(stride);
                        if (cmd_buffer->device->physical_device->rad_info.chip_class <= CIK && stride)
-                               desc[2] = (buffer->size - offset - cmd_buffer->state.pipeline->va_format_size[i]) / stride + 1;
+                               desc[2] = (buffer->size - offset - velems->format_size[i]) / stride + 1;
                        else
                                desc[2] = buffer->size - offset;
-                       desc[3] = cmd_buffer->state.pipeline->va_rsrc_word3[i];
+                       desc[3] = velems->rsrc_word3[i];
                }
 
                va = device->ws->buffer_get_va(cmd_buffer->upload.upload_bo);
@@ -1578,7 +1729,9 @@ radv_cmd_buffer_update_vertex_descriptors(struct radv_cmd_buffer *cmd_buffer)
                radv_emit_userdata_address(cmd_buffer, cmd_buffer->state.pipeline, MESA_SHADER_VERTEX,
                                           AC_UD_VS_VERTEX_BUFFERS, va);
        }
-       cmd_buffer->state.vb_dirty = 0;
+       cmd_buffer->state.vb_dirty = false;
+
+       return true;
 }
 
 static void
@@ -1587,16 +1740,16 @@ radv_cmd_buffer_flush_state(struct radv_cmd_buffer *cmd_buffer,
                            bool indirect_draw,
                            uint32_t draw_vertex_count)
 {
-       struct radv_pipeline *pipeline = cmd_buffer->state.pipeline;
        uint32_t ia_multi_vgt_param;
 
        MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
                                                           cmd_buffer->cs, 4096);
 
-       radv_cmd_buffer_update_vertex_descriptors(cmd_buffer);
+       if (!radv_cmd_buffer_update_vertex_descriptors(cmd_buffer))
+               return;
 
        if (cmd_buffer->state.dirty & RADV_CMD_DIRTY_PIPELINE)
-               radv_emit_graphics_pipeline(cmd_buffer, pipeline);
+               radv_emit_graphics_pipeline(cmd_buffer);
 
        if (cmd_buffer->state.dirty & RADV_CMD_DIRTY_RENDER_TARGETS)
                radv_emit_framebuffer_state(cmd_buffer);
@@ -1768,8 +1921,9 @@ radv_cmd_buffer_set_subpass(struct radv_cmd_buffer *cmd_buffer,
                radv_subpass_barrier(cmd_buffer, &subpass->start_barrier);
 
                for (unsigned i = 0; i < subpass->color_count; ++i) {
-                       radv_handle_subpass_image_transition(cmd_buffer,
-                                                       subpass->color_attachments[i]);
+                       if (subpass->color_attachments[i].attachment != VK_ATTACHMENT_UNUSED)
+                               radv_handle_subpass_image_transition(cmd_buffer,
+                                                                    subpass->color_attachments[i]);
                }
 
                for (unsigned i = 0; i < subpass->input_count; ++i) {
@@ -1788,7 +1942,7 @@ radv_cmd_buffer_set_subpass(struct radv_cmd_buffer *cmd_buffer,
        cmd_buffer->state.dirty |= RADV_CMD_DIRTY_RENDER_TARGETS;
 }
 
-static void
+static VkResult
 radv_cmd_state_setup_attachments(struct radv_cmd_buffer *cmd_buffer,
                                 struct radv_render_pass *pass,
                                 const VkRenderPassBeginInfo *info)
@@ -1797,7 +1951,7 @@ radv_cmd_state_setup_attachments(struct radv_cmd_buffer *cmd_buffer,
 
        if (pass->attachment_count == 0) {
                state->attachments = NULL;
-               return;
+               return VK_SUCCESS;
        }
 
        state->attachments = vk_alloc(&cmd_buffer->pool->alloc,
@@ -1805,8 +1959,8 @@ radv_cmd_state_setup_attachments(struct radv_cmd_buffer *cmd_buffer,
                                        sizeof(state->attachments[0]),
                                        8, VK_SYSTEM_ALLOCATION_SCOPE_OBJECT);
        if (state->attachments == NULL) {
-               /* FIXME: Propagate VK_ERROR_OUT_OF_HOST_MEMORY to vkEndCommandBuffer */
-               abort();
+               cmd_buffer->record_result = VK_ERROR_OUT_OF_HOST_MEMORY;
+               return cmd_buffer->record_result;
        }
 
        for (uint32_t i = 0; i < pass->attachment_count; ++i) {
@@ -1824,6 +1978,9 @@ radv_cmd_state_setup_attachments(struct radv_cmd_buffer *cmd_buffer,
                        if ((att_aspects & VK_IMAGE_ASPECT_DEPTH_BIT) &&
                            att->load_op == VK_ATTACHMENT_LOAD_OP_CLEAR) {
                                clear_aspects |= VK_IMAGE_ASPECT_DEPTH_BIT;
+                               if ((att_aspects & VK_IMAGE_ASPECT_STENCIL_BIT) &&
+                                   att->stencil_load_op == VK_ATTACHMENT_LOAD_OP_DONT_CARE)
+                                       clear_aspects |= VK_IMAGE_ASPECT_STENCIL_BIT;
                        }
                        if ((att_aspects & VK_IMAGE_ASPECT_STENCIL_BIT) &&
                            att->stencil_load_op == VK_ATTACHMENT_LOAD_OP_CLEAR) {
@@ -1832,6 +1989,7 @@ radv_cmd_state_setup_attachments(struct radv_cmd_buffer *cmd_buffer,
                }
 
                state->attachments[i].pending_clear_aspects = clear_aspects;
+               state->attachments[i].cleared_views = 0;
                if (clear_aspects && info) {
                        assert(info->clearValueCount > i);
                        state->attachments[i].clear_value = info->pClearValues[i];
@@ -1839,6 +1997,8 @@ radv_cmd_state_setup_attachments(struct radv_cmd_buffer *cmd_buffer,
 
                state->attachments[i].current_layout = att->initial_layout;
        }
+
+       return VK_SUCCESS;
 }
 
 VkResult radv_AllocateCommandBuffers(
@@ -1863,12 +2023,11 @@ VkResult radv_AllocateCommandBuffers(
                        list_del(&cmd_buffer->pool_link);
                        list_addtail(&cmd_buffer->pool_link, &pool->cmd_buffers);
 
-                       radv_reset_cmd_buffer(cmd_buffer);
+                       result = radv_reset_cmd_buffer(cmd_buffer);
                        cmd_buffer->_loader_data.loaderMagic = ICD_LOADER_MAGIC;
                        cmd_buffer->level = pAllocateInfo->level;
 
                        pCommandBuffers[i] = radv_cmd_buffer_to_handle(cmd_buffer);
-                       result = VK_SUCCESS;
                } else {
                        result = radv_create_cmd_buffer(device, pool, pAllocateInfo->level,
                                                        &pCommandBuffers[i]);
@@ -1909,8 +2068,7 @@ VkResult radv_ResetCommandBuffer(
        VkCommandBufferResetFlags flags)
 {
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
-       radv_reset_cmd_buffer(cmd_buffer);
-       return VK_SUCCESS;
+       return radv_reset_cmd_buffer(cmd_buffer);
 }
 
 static void emit_gfx_buffer_state(struct radv_cmd_buffer *cmd_buffer)
@@ -1932,10 +2090,15 @@ VkResult radv_BeginCommandBuffer(
        const VkCommandBufferBeginInfo *pBeginInfo)
 {
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
-       radv_reset_cmd_buffer(cmd_buffer);
+       VkResult result;
+
+       result = radv_reset_cmd_buffer(cmd_buffer);
+       if (result != VK_SUCCESS)
+               return result;
 
        memset(&cmd_buffer->state, 0, sizeof(cmd_buffer->state));
        cmd_buffer->state.last_primitive_reset_en = -1;
+       cmd_buffer->usage_flags = pBeginInfo->flags;
 
        /* setup initial configuration into command buffer */
        if (cmd_buffer->level == VK_COMMAND_BUFFER_LEVEL_PRIMARY) {
@@ -1960,12 +2123,15 @@ VkResult radv_BeginCommandBuffer(
                struct radv_subpass *subpass =
                        &cmd_buffer->state.pass->subpasses[pBeginInfo->pInheritanceInfo->subpass];
 
-               radv_cmd_state_setup_attachments(cmd_buffer, cmd_buffer->state.pass, NULL);
+               result = radv_cmd_state_setup_attachments(cmd_buffer, cmd_buffer->state.pass, NULL);
+               if (result != VK_SUCCESS)
+                       return result;
+
                radv_cmd_buffer_set_subpass(cmd_buffer, subpass, false);
        }
 
        radv_cmd_buffer_trace_emit(cmd_buffer);
-       return VK_SUCCESS;
+       return result;
 }
 
 void radv_CmdBindVertexBuffers(
@@ -1981,12 +2147,13 @@ void radv_CmdBindVertexBuffers(
        /* We have to defer setting up vertex buffer since we need the buffer
         * stride from the pipeline. */
 
-       assert(firstBinding + bindingCount < MAX_VBS);
+       assert(firstBinding + bindingCount <= MAX_VBS);
        for (uint32_t i = 0; i < bindingCount; i++) {
                vb[firstBinding + i].buffer = radv_buffer_from_handle(pBuffers[i]);
                vb[firstBinding + i].offset = pOffsets[i];
-               cmd_buffer->state.vb_dirty |= 1 << (firstBinding + i);
        }
+
+       cmd_buffer->state.vb_dirty = true;
 }
 
 void radv_CmdBindIndexBuffer(
@@ -2088,7 +2255,7 @@ static bool radv_init_push_descriptor_set(struct radv_cmd_buffer *cmd_buffer,
 
                if (!set->mapped_ptr) {
                        cmd_buffer->push_descriptors.capacity = 0;
-                       cmd_buffer->record_fail = true;
+                       cmd_buffer->record_result = VK_ERROR_OUT_OF_HOST_MEMORY;
                        return false;
                }
 
@@ -2110,6 +2277,7 @@ void radv_meta_push_descriptor_set(
        struct radv_descriptor_set *push_set = &cmd_buffer->meta_push_descriptors;
        unsigned bo_offset;
 
+       assert(set == 0);
        assert(layout->set[set].layout->flags & VK_DESCRIPTOR_SET_LAYOUT_CREATE_PUSH_DESCRIPTOR_BIT_KHR);
 
        push_set->size = layout->set[set].layout->size;
@@ -2198,13 +2366,16 @@ VkResult radv_EndCommandBuffer(
 {
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
 
-       if (cmd_buffer->queue_family_index != RADV_QUEUE_TRANSFER)
+       if (cmd_buffer->queue_family_index != RADV_QUEUE_TRANSFER) {
+               if (cmd_buffer->device->physical_device->rad_info.chip_class == SI)
+                       cmd_buffer->state.flush_bits |= RADV_CMD_FLAG_CS_PARTIAL_FLUSH | RADV_CMD_FLAG_PS_PARTIAL_FLUSH | RADV_CMD_FLAG_WRITEBACK_GLOBAL_L2;
                si_emit_cache_flush(cmd_buffer);
+       }
 
-       if (!cmd_buffer->device->ws->cs_finalize(cmd_buffer->cs) ||
-           cmd_buffer->record_fail)
+       if (!cmd_buffer->device->ws->cs_finalize(cmd_buffer->cs))
                return VK_ERROR_OUT_OF_DEVICE_MEMORY;
-       return VK_SUCCESS;
+
+       return cmd_buffer->record_result;
 }
 
 static void
@@ -2221,7 +2392,7 @@ radv_emit_compute_pipeline(struct radv_cmd_buffer *cmd_buffer)
        cmd_buffer->state.emitted_compute_pipeline = pipeline;
 
        compute_shader = pipeline->shaders[MESA_SHADER_COMPUTE];
-       va = ws->buffer_get_va(compute_shader->bo);
+       va = ws->buffer_get_va(compute_shader->bo) + compute_shader->bo_offset;
 
        ws->cs_add_buffer(cmd_buffer->cs, compute_shader->bo, 8);
        radv_emit_prefetch(cmd_buffer, va, compute_shader->code_size);
@@ -2256,6 +2427,7 @@ radv_emit_compute_pipeline(struct radv_cmd_buffer *cmd_buffer)
                    S_00B81C_NUM_THREAD_FULL(compute_shader->info.cs.block_size[2]));
 
        assert(cmd_buffer->cs->cdw <= cdw_max);
+       radv_save_pipeline(cmd_buffer, pipeline, RING_COMPUTE);
 }
 
 static void radv_mark_descriptor_sets_dirty(struct radv_cmd_buffer *cmd_buffer)
@@ -2555,10 +2727,13 @@ VkResult radv_ResetCommandPool(
        VkCommandPoolResetFlags                     flags)
 {
        RADV_FROM_HANDLE(radv_cmd_pool, pool, commandPool);
+       VkResult result;
 
        list_for_each_entry(struct radv_cmd_buffer, cmd_buffer,
                            &pool->cmd_buffers, pool_link) {
-               radv_reset_cmd_buffer(cmd_buffer);
+               result = radv_reset_cmd_buffer(cmd_buffer);
+               if (result != VK_SUCCESS)
+                       return result;
        }
 
        return VK_SUCCESS;
@@ -2591,11 +2766,15 @@ void radv_CmdBeginRenderPass(
 
        MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
                                                           cmd_buffer->cs, 2048);
+       MAYBE_UNUSED VkResult result;
 
        cmd_buffer->state.framebuffer = framebuffer;
        cmd_buffer->state.pass = pass;
        cmd_buffer->state.render_area = pRenderPassBegin->renderArea;
-       radv_cmd_state_setup_attachments(cmd_buffer, pass, pRenderPassBegin);
+
+       result = radv_cmd_state_setup_attachments(cmd_buffer, pass, pRenderPassBegin);
+       if (result != VK_SUCCESS)
+               return;
 
        radv_cmd_buffer_set_subpass(cmd_buffer, pass->subpasses, true);
        assert(cmd_buffer->cs->cdw <= cdw_max);
@@ -2618,6 +2797,38 @@ void radv_CmdNextSubpass(
        radv_cmd_buffer_clear_subpass(cmd_buffer);
 }
 
+static void radv_emit_view_index(struct radv_cmd_buffer *cmd_buffer, unsigned index)
+{
+       struct radv_pipeline *pipeline = cmd_buffer->state.pipeline;
+       for (unsigned stage = 0; stage < MESA_SHADER_STAGES; ++stage) {
+               if (!pipeline->shaders[stage])
+                       continue;
+               struct ac_userdata_info *loc = radv_lookup_user_sgpr(pipeline, stage, AC_UD_VIEW_INDEX);
+               if (loc->sgpr_idx == -1)
+                       continue;
+               uint32_t base_reg = radv_shader_stage_to_user_data_0(stage, radv_pipeline_has_gs(pipeline), radv_pipeline_has_tess(pipeline));
+               radeon_set_sh_reg(cmd_buffer->cs, base_reg + loc->sgpr_idx * 4, index);
+
+       }
+       if (pipeline->gs_copy_shader) {
+               struct ac_userdata_info *loc = &pipeline->gs_copy_shader->info.user_sgprs_locs.shader_data[AC_UD_VIEW_INDEX];
+               if (loc->sgpr_idx != -1) {
+                       uint32_t base_reg = R_00B130_SPI_SHADER_USER_DATA_VS_0;
+                       radeon_set_sh_reg(cmd_buffer->cs, base_reg + loc->sgpr_idx * 4, index);
+               }
+       }
+}
+
+static void
+radv_cs_emit_draw_packet(struct radv_cmd_buffer *cmd_buffer,
+                         uint32_t vertex_count)
+{
+       radeon_emit(cmd_buffer->cs, PKT3(PKT3_DRAW_INDEX_AUTO, 1, cmd_buffer->state.predicating));
+       radeon_emit(cmd_buffer->cs, vertex_count);
+       radeon_emit(cmd_buffer->cs, V_0287F0_DI_SRC_SEL_AUTO_INDEX |
+                                   S_0287F0_USE_OPAQUE(0));
+}
+
 void radv_CmdDraw(
        VkCommandBuffer                             commandBuffer,
        uint32_t                                    vertexCount,
@@ -2629,7 +2840,7 @@ void radv_CmdDraw(
 
        radv_cmd_buffer_flush_state(cmd_buffer, false, (instanceCount > 1), false, vertexCount);
 
-       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 10);
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 20 * MAX_VIEWS);
 
        assert(cmd_buffer->state.pipeline->graphics.vtx_base_sgpr);
        radeon_set_sh_reg_seq(cmd_buffer->cs, cmd_buffer->state.pipeline->graphics.vtx_base_sgpr,
@@ -2642,14 +2853,34 @@ void radv_CmdDraw(
        radeon_emit(cmd_buffer->cs, PKT3(PKT3_NUM_INSTANCES, 0, cmd_buffer->state.predicating));
        radeon_emit(cmd_buffer->cs, instanceCount);
 
-       radeon_emit(cmd_buffer->cs, PKT3(PKT3_DRAW_INDEX_AUTO, 1, cmd_buffer->state.predicating));
-       radeon_emit(cmd_buffer->cs, vertexCount);
-       radeon_emit(cmd_buffer->cs, V_0287F0_DI_SRC_SEL_AUTO_INDEX |
-                   S_0287F0_USE_OPAQUE(0));
+       if (!cmd_buffer->state.subpass->view_mask) {
+               radv_cs_emit_draw_packet(cmd_buffer, vertexCount);
+       } else {
+               unsigned i;
+               for_each_bit(i, cmd_buffer->state.subpass->view_mask) {
+                       radv_emit_view_index(cmd_buffer, i);
+
+                       radv_cs_emit_draw_packet(cmd_buffer, vertexCount);
+               }
+       }
 
        assert(cmd_buffer->cs->cdw <= cdw_max);
 
-       radv_cmd_buffer_trace_emit(cmd_buffer);
+       radv_cmd_buffer_after_draw(cmd_buffer);
+}
+
+
+static void
+radv_cs_emit_draw_indexed_packet(struct radv_cmd_buffer *cmd_buffer,
+                                 uint64_t index_va,
+                                 uint32_t index_count)
+{
+       radeon_emit(cmd_buffer->cs, PKT3(PKT3_DRAW_INDEX_2, 4, false));
+       radeon_emit(cmd_buffer->cs, cmd_buffer->state.max_index_count);
+       radeon_emit(cmd_buffer->cs, index_va);
+       radeon_emit(cmd_buffer->cs, (index_va >> 32UL) & 0xFF);
+       radeon_emit(cmd_buffer->cs, index_count);
+       radeon_emit(cmd_buffer->cs, V_0287F0_DI_SRC_SEL_DMA);
 }
 
 void radv_CmdDrawIndexed(
@@ -2666,7 +2897,7 @@ void radv_CmdDrawIndexed(
 
        radv_cmd_buffer_flush_state(cmd_buffer, true, (instanceCount > 1), false, indexCount);
 
-       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 15);
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 26 * MAX_VIEWS);
 
        if (cmd_buffer->device->physical_device->rad_info.chip_class >= GFX9) {
                radeon_set_uconfig_reg_idx(cmd_buffer->cs, R_03090C_VGT_INDEX_TYPE,
@@ -2689,15 +2920,58 @@ void radv_CmdDrawIndexed(
 
        index_va = cmd_buffer->state.index_va;
        index_va += firstIndex * index_size;
-       radeon_emit(cmd_buffer->cs, PKT3(PKT3_DRAW_INDEX_2, 4, false));
-       radeon_emit(cmd_buffer->cs, cmd_buffer->state.max_index_count);
-       radeon_emit(cmd_buffer->cs, index_va);
-       radeon_emit(cmd_buffer->cs, (index_va >> 32UL) & 0xFF);
-       radeon_emit(cmd_buffer->cs, indexCount);
-       radeon_emit(cmd_buffer->cs, V_0287F0_DI_SRC_SEL_DMA);
+       if (!cmd_buffer->state.subpass->view_mask) {
+               radv_cs_emit_draw_indexed_packet(cmd_buffer, index_va, indexCount);
+       } else {
+               unsigned i;
+               for_each_bit(i, cmd_buffer->state.subpass->view_mask) {
+                       radv_emit_view_index(cmd_buffer, i);
+
+                       radv_cs_emit_draw_indexed_packet(cmd_buffer, index_va, indexCount);
+               }
+       }
 
        assert(cmd_buffer->cs->cdw <= cdw_max);
-       radv_cmd_buffer_trace_emit(cmd_buffer);
+       radv_cmd_buffer_after_draw(cmd_buffer);
+}
+
+static void
+radv_cs_emit_indirect_draw_packet(struct radv_cmd_buffer *cmd_buffer,
+                                  bool indexed,
+                                  uint32_t draw_count,
+                                  uint64_t count_va,
+                                  uint32_t stride)
+{
+       struct radeon_winsys_cs *cs = cmd_buffer->cs;
+       unsigned di_src_sel = indexed ? V_0287F0_DI_SRC_SEL_DMA
+                                     : V_0287F0_DI_SRC_SEL_AUTO_INDEX;
+       bool draw_id_enable = cmd_buffer->state.pipeline->shaders[MESA_SHADER_VERTEX]->info.info.vs.needs_draw_id;
+       uint32_t base_reg = cmd_buffer->state.pipeline->graphics.vtx_base_sgpr;
+       assert(base_reg);
+
+       if (draw_count == 1 && !count_va && !draw_id_enable) {
+               radeon_emit(cs, PKT3(indexed ? PKT3_DRAW_INDEX_INDIRECT :
+                                    PKT3_DRAW_INDIRECT, 3, false));
+               radeon_emit(cs, 0);
+               radeon_emit(cs, (base_reg - SI_SH_REG_OFFSET) >> 2);
+               radeon_emit(cs, ((base_reg + 4) - SI_SH_REG_OFFSET) >> 2);
+               radeon_emit(cs, di_src_sel);
+       } else {
+               radeon_emit(cs, PKT3(indexed ? PKT3_DRAW_INDEX_INDIRECT_MULTI :
+                                    PKT3_DRAW_INDIRECT_MULTI,
+                                    8, false));
+               radeon_emit(cs, 0);
+               radeon_emit(cs, (base_reg - SI_SH_REG_OFFSET) >> 2);
+               radeon_emit(cs, ((base_reg + 4) - SI_SH_REG_OFFSET) >> 2);
+               radeon_emit(cs, (((base_reg + 8) - SI_SH_REG_OFFSET) >> 2) |
+                           S_2C3_DRAW_INDEX_ENABLE(draw_id_enable) |
+                           S_2C3_COUNT_INDIRECT_ENABLE(!!count_va));
+               radeon_emit(cs, draw_count); /* count */
+               radeon_emit(cs, count_va); /* count_addr */
+               radeon_emit(cs, count_va >> 32);
+               radeon_emit(cs, stride); /* stride */
+               radeon_emit(cs, di_src_sel);
+       }
 }
 
 static void
@@ -2713,8 +2987,7 @@ radv_emit_indirect_draw(struct radv_cmd_buffer *cmd_buffer,
        RADV_FROM_HANDLE(radv_buffer, buffer, _buffer);
        RADV_FROM_HANDLE(radv_buffer, count_buffer, _count_buffer);
        struct radeon_winsys_cs *cs = cmd_buffer->cs;
-       unsigned di_src_sel = indexed ? V_0287F0_DI_SRC_SEL_DMA
-                                           : V_0287F0_DI_SRC_SEL_AUTO_INDEX;
+
        uint64_t indirect_va = cmd_buffer->device->ws->buffer_get_va(buffer->bo);
        indirect_va += offset + buffer->offset;
        uint64_t count_va = 0;
@@ -2728,30 +3001,23 @@ radv_emit_indirect_draw(struct radv_cmd_buffer *cmd_buffer,
                return;
 
        cmd_buffer->device->ws->cs_add_buffer(cs, buffer->bo, 8);
-       bool draw_id_enable = cmd_buffer->state.pipeline->shaders[MESA_SHADER_VERTEX]->info.info.vs.needs_draw_id;
-       uint32_t base_reg = cmd_buffer->state.pipeline->graphics.vtx_base_sgpr;
-       assert(base_reg);
 
        radeon_emit(cs, PKT3(PKT3_SET_BASE, 2, 0));
        radeon_emit(cs, 1);
        radeon_emit(cs, indirect_va);
        radeon_emit(cs, indirect_va >> 32);
 
-       radeon_emit(cs, PKT3(indexed ? PKT3_DRAW_INDEX_INDIRECT_MULTI :
-                                      PKT3_DRAW_INDIRECT_MULTI,
-                            8, false));
-       radeon_emit(cs, 0);
-       radeon_emit(cs, (base_reg - SI_SH_REG_OFFSET) >> 2);
-       radeon_emit(cs, ((base_reg + 4) - SI_SH_REG_OFFSET) >> 2);
-       radeon_emit(cs, (((base_reg + 8) - SI_SH_REG_OFFSET) >> 2) |
-                       S_2C3_DRAW_INDEX_ENABLE(draw_id_enable) |
-                       S_2C3_COUNT_INDIRECT_ENABLE(!!count_va));
-       radeon_emit(cs, draw_count); /* count */
-       radeon_emit(cs, count_va); /* count_addr */
-       radeon_emit(cs, count_va >> 32);
-       radeon_emit(cs, stride); /* stride */
-       radeon_emit(cs, di_src_sel);
-       radv_cmd_buffer_trace_emit(cmd_buffer);
+       if (!cmd_buffer->state.subpass->view_mask) {
+               radv_cs_emit_indirect_draw_packet(cmd_buffer, indexed, draw_count, count_va, stride);
+       } else {
+               unsigned i;
+               for_each_bit(i, cmd_buffer->state.subpass->view_mask) {
+                       radv_emit_view_index(cmd_buffer, i);
+
+                       radv_cs_emit_indirect_draw_packet(cmd_buffer, indexed, draw_count, count_va, stride);
+               }
+       }
+       radv_cmd_buffer_after_draw(cmd_buffer);
 }
 
 static void
@@ -2767,7 +3033,7 @@ radv_cmd_draw_indirect_count(VkCommandBuffer                             command
        radv_cmd_buffer_flush_state(cmd_buffer, false, false, true, 0);
 
        MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
-                                                          cmd_buffer->cs, 14);
+                                                          cmd_buffer->cs, 24 * MAX_VIEWS);
 
        radv_emit_indirect_draw(cmd_buffer, buffer, offset,
                                countBuffer, countBufferOffset, maxDrawCount, stride, false);
@@ -2791,7 +3057,7 @@ radv_cmd_draw_indexed_indirect_count(
 
        index_va = cmd_buffer->state.index_va;
 
-       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 21);
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 31 * MAX_VIEWS);
 
        radeon_emit(cmd_buffer->cs, PKT3(PKT3_INDEX_TYPE, 0, 0));
        radeon_emit(cmd_buffer->cs, cmd_buffer->state.index_type);
@@ -2903,7 +3169,7 @@ void radv_CmdDispatch(
        radeon_emit(cmd_buffer->cs, 1);
 
        assert(cmd_buffer->cs->cdw <= cdw_max);
-       radv_cmd_buffer_trace_emit(cmd_buffer);
+       radv_cmd_buffer_after_draw(cmd_buffer);
 }
 
 void radv_CmdDispatchIndirect(
@@ -2956,7 +3222,7 @@ void radv_CmdDispatchIndirect(
        }
 
        assert(cmd_buffer->cs->cdw <= cdw_max);
-       radv_cmd_buffer_trace_emit(cmd_buffer);
+       radv_cmd_buffer_after_draw(cmd_buffer);
 }
 
 void radv_unaligned_dispatch(
@@ -3013,7 +3279,7 @@ void radv_unaligned_dispatch(
                                    S_00B800_PARTIAL_TG_EN(1));
 
        assert(cmd_buffer->cs->cdw <= cdw_max);
-       radv_cmd_buffer_trace_emit(cmd_buffer);
+       radv_cmd_buffer_after_draw(cmd_buffer);
 }
 
 void radv_CmdEndRenderPass(