radv/amdgpu: Add some debug flags.
[mesa.git] / src / amd / vulkan / radv_cmd_buffer.c
index 4ac5ad78c13fa256100948a33a30c0a4447b6e86..dec3e9fd21dfaf273cc7e6e38dce7a60e827dfe5 100644 (file)
 #include "vk_format.h"
 #include "radv_meta.h"
 
+#include "ac_debug.h"
+
 static void radv_handle_image_transition(struct radv_cmd_buffer *cmd_buffer,
                                         struct radv_image *image,
                                         VkImageLayout src_layout,
                                         VkImageLayout dst_layout,
-                                        VkImageSubresourceRange range,
+                                        uint32_t src_family,
+                                        uint32_t dst_family,
+                                        const VkImageSubresourceRange *range,
                                         VkImageAspectFlags pending_clears);
 
 const struct radv_dynamic_state default_dynamic_state = {
@@ -110,6 +114,25 @@ radv_dynamic_state_copy(struct radv_dynamic_state *dest,
                dest->stencil_reference = src->stencil_reference;
 }
 
+bool radv_cmd_buffer_uses_mec(struct radv_cmd_buffer *cmd_buffer)
+{
+       return cmd_buffer->queue_family_index == RADV_QUEUE_COMPUTE &&
+              cmd_buffer->device->physical_device->rad_info.chip_class >= CIK;
+}
+
+enum ring_type radv_queue_family_to_ring(int f) {
+       switch (f) {
+       case RADV_QUEUE_GENERAL:
+               return RING_GFX;
+       case RADV_QUEUE_COMPUTE:
+               return RING_COMPUTE;
+       case RADV_QUEUE_TRANSFER:
+               return RING_DMA;
+       default:
+               unreachable("Unknown queue family");
+       }
+}
+
 static VkResult radv_create_cmd_buffer(
        struct radv_device *                         device,
        struct radv_cmd_pool *                       pool,
@@ -118,7 +141,7 @@ static VkResult radv_create_cmd_buffer(
 {
        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);
        if (cmd_buffer == NULL)
@@ -132,14 +155,19 @@ static VkResult radv_create_cmd_buffer(
 
        if (pool) {
                list_addtail(&cmd_buffer->pool_link, &pool->cmd_buffers);
+               cmd_buffer->queue_family_index = pool->queue_family_index;
+
        } else {
                /* Init the pool_link so we can safefly call list_del when we destroy
                 * the command buffer
                 */
                list_inithead(&cmd_buffer->pool_link);
+               cmd_buffer->queue_family_index = RADV_QUEUE_GENERAL;
        }
 
-       cmd_buffer->cs = device->ws->cs_create(device->ws, RING_GFX);
+       ring = radv_queue_family_to_ring(cmd_buffer->queue_family_index);
+
+       cmd_buffer->cs = device->ws->cs_create(device->ws, ring);
        if (!cmd_buffer->cs) {
                result = VK_ERROR_OUT_OF_HOST_MEMORY;
                goto fail;
@@ -159,6 +187,51 @@ fail:
        return result;
 }
 
+static void
+radv_cmd_buffer_destroy(struct radv_cmd_buffer *cmd_buffer)
+{
+       list_del(&cmd_buffer->pool_link);
+
+       list_for_each_entry_safe(struct radv_cmd_buffer_upload, up,
+                                &cmd_buffer->upload.list, list) {
+               cmd_buffer->device->ws->buffer_destroy(up->upload_bo);
+               list_del(&up->list);
+               free(up);
+       }
+
+       if (cmd_buffer->upload.upload_bo)
+               cmd_buffer->device->ws->buffer_destroy(cmd_buffer->upload.upload_bo);
+       cmd_buffer->device->ws->cs_destroy(cmd_buffer->cs);
+       vk_free(&cmd_buffer->pool->alloc, cmd_buffer);
+}
+
+static void  radv_reset_cmd_buffer(struct radv_cmd_buffer *cmd_buffer)
+{
+
+       cmd_buffer->device->ws->cs_reset(cmd_buffer->cs);
+
+       list_for_each_entry_safe(struct radv_cmd_buffer_upload, up,
+                                &cmd_buffer->upload.list, list) {
+               cmd_buffer->device->ws->buffer_destroy(up->upload_bo);
+               list_del(&up->list);
+               free(up);
+       }
+
+       cmd_buffer->scratch_size_needed = 0;
+       cmd_buffer->compute_scratch_size_needed = 0;
+       cmd_buffer->esgs_ring_size_needed = 0;
+       cmd_buffer->gsvs_ring_size_needed = 0;
+
+       if (cmd_buffer->upload.upload_bo)
+               cmd_buffer->device->ws->cs_add_buffer(cmd_buffer->cs,
+                                                     cmd_buffer->upload.upload_bo, 8);
+       cmd_buffer->upload.offset = 0;
+
+       cmd_buffer->record_fail = false;
+
+       cmd_buffer->ring_offsets_idx = -1;
+}
+
 static bool
 radv_cmd_buffer_resize_upload_buf(struct radv_cmd_buffer *cmd_buffer,
                                  uint64_t min_needed)
@@ -246,6 +319,32 @@ radv_cmd_buffer_upload_data(struct radv_cmd_buffer *cmd_buffer,
        return true;
 }
 
+void radv_cmd_buffer_trace_emit(struct radv_cmd_buffer *cmd_buffer)
+{
+       struct radv_device *device = cmd_buffer->device;
+       struct radeon_winsys_cs *cs = cmd_buffer->cs;
+       uint64_t va;
+
+       if (!device->trace_bo)
+               return;
+
+       va = device->ws->buffer_get_va(device->trace_bo);
+
+       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);
+       radeon_emit(cs, PKT3(PKT3_NOP, 0, 0));
+       radeon_emit(cs, AC_ENCODE_TRACE_POINT(cmd_buffer->state.trace_id));
+}
+
 static void
 radv_emit_graphics_blend_state(struct radv_cmd_buffer *cmd_buffer,
                               struct radv_pipeline *pipeline)
@@ -276,6 +375,48 @@ static unsigned radv_pack_float_12p4(float x)
               x >= 4096 ? 0xffff : x * 16;
 }
 
+static uint32_t
+shader_stage_to_user_data_0(gl_shader_stage stage, bool has_gs)
+{
+       switch (stage) {
+       case MESA_SHADER_FRAGMENT:
+               return R_00B030_SPI_SHADER_USER_DATA_PS_0;
+       case MESA_SHADER_VERTEX:
+               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;
+       default:
+               unreachable("unknown shader");
+       }
+}
+
+static struct ac_userdata_info *
+radv_lookup_user_sgpr(struct radv_pipeline *pipeline,
+                     gl_shader_stage stage,
+                     int idx)
+{
+       return &pipeline->shaders[stage]->info.user_sgprs_locs.shader_data[idx];
+}
+
+static void
+radv_emit_userdata_address(struct radv_cmd_buffer *cmd_buffer,
+                          struct radv_pipeline *pipeline,
+                          gl_shader_stage stage,
+                          int idx, uint64_t va)
+{
+       struct ac_userdata_info *loc = radv_lookup_user_sgpr(pipeline, stage, idx);
+       uint32_t base_reg = shader_stage_to_user_data_0(stage, radv_pipeline_has_gs(pipeline));
+       if (loc->sgpr_idx == -1)
+               return;
+       assert(loc->num_sgprs == 2);
+       assert(!loc->indirect);
+       radeon_set_sh_reg_seq(cmd_buffer->cs, base_reg + loc->sgpr_idx * 4, 2);
+       radeon_emit(cmd_buffer->cs, va);
+       radeon_emit(cmd_buffer->cs, va >> 32);
+}
+
 static void
 radv_update_multisample_state(struct radv_cmd_buffer *cmd_buffer,
                              struct radv_pipeline *pipeline)
@@ -321,15 +462,16 @@ radv_update_multisample_state(struct radv_cmd_buffer *cmd_buffer,
        case 16:
                src = cmd_buffer->device->sample_locations_16x;
                break;
+       default:
+               unreachable("unknown number of samples");
        }
        memcpy(samples_ptr, src, num_samples * 4 * 2);
 
        uint64_t va = cmd_buffer->device->ws->buffer_get_va(cmd_buffer->upload.upload_bo);
        va += samples_offset;
 
-       radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B030_SPI_SHADER_USER_DATA_PS_0 + AC_USERDATA_PS_SAMPLE_POS * 4, 2);
-       radeon_emit(cmd_buffer->cs, va);
-       radeon_emit(cmd_buffer->cs, va >> 32);
+       radv_emit_userdata_address(cmd_buffer, pipeline, MESA_SHADER_FRAGMENT,
+                                  AC_UD_PS_SAMPLE_POS, va);
 }
 
 static void
@@ -345,7 +487,8 @@ radv_emit_graphics_raster_state(struct radv_cmd_buffer *cmd_buffer,
                               raster->spi_interp_control);
 
        radeon_set_context_reg_seq(cmd_buffer->cs, R_028A00_PA_SU_POINT_SIZE, 2);
-       radeon_emit(cmd_buffer->cs, 0);
+       unsigned tmp = (unsigned)(1.0 * 8.0);
+       radeon_emit(cmd_buffer->cs, S_028A00_HEIGHT(tmp) | S_028A00_WIDTH(tmp));
        radeon_emit(cmd_buffer->cs, S_028A04_MIN_SIZE(radv_pack_float_12p4(0)) |
                    S_028A04_MAX_SIZE(radv_pack_float_12p4(8192/2))); /* R_028A04_PA_SU_POINT_MINMAX */
 
@@ -357,47 +500,38 @@ radv_emit_graphics_raster_state(struct radv_cmd_buffer *cmd_buffer,
 }
 
 static void
-radv_emit_vertex_shader(struct radv_cmd_buffer *cmd_buffer,
-                       struct radv_pipeline *pipeline)
+radv_emit_hw_vs(struct radv_cmd_buffer *cmd_buffer,
+               struct radv_pipeline *pipeline,
+               struct radv_shader_variant *shader)
 {
        struct radeon_winsys *ws = cmd_buffer->device->ws;
-       struct radv_shader_variant *vs;
-       uint64_t va;
+       uint64_t va = ws->buffer_get_va(shader->bo);
        unsigned export_count;
-       unsigned clip_dist_mask, cull_dist_mask, total_mask;
-
-       assert (pipeline->shaders[MESA_SHADER_VERTEX]);
-
-       vs = pipeline->shaders[MESA_SHADER_VERTEX];
-       va = ws->buffer_get_va(vs->bo);
-       ws->cs_add_buffer(cmd_buffer->cs, vs->bo, 8);
 
-       clip_dist_mask = vs->info.vs.clip_dist_mask;
-       cull_dist_mask = vs->info.vs.cull_dist_mask;
-       total_mask = clip_dist_mask | cull_dist_mask;
-       radeon_set_context_reg(cmd_buffer->cs, R_028A40_VGT_GS_MODE, 0);
-       radeon_set_context_reg(cmd_buffer->cs, R_028A84_VGT_PRIMITIVEID_EN, 0);
+       ws->cs_add_buffer(cmd_buffer->cs, shader->bo, 8);
 
-       export_count = MAX2(1, vs->info.vs.param_exports);
+       export_count = MAX2(1, shader->info.vs.param_exports);
        radeon_set_context_reg(cmd_buffer->cs, R_0286C4_SPI_VS_OUT_CONFIG,
                               S_0286C4_VS_EXPORT_COUNT(export_count - 1));
+
        radeon_set_context_reg(cmd_buffer->cs, R_02870C_SPI_SHADER_POS_FORMAT,
                               S_02870C_POS0_EXPORT_FORMAT(V_02870C_SPI_SHADER_4COMP) |
-                              S_02870C_POS1_EXPORT_FORMAT(vs->info.vs.pos_exports > 1 ?
+                              S_02870C_POS1_EXPORT_FORMAT(shader->info.vs.pos_exports > 1 ?
                                                           V_02870C_SPI_SHADER_4COMP :
                                                           V_02870C_SPI_SHADER_NONE) |
-                              S_02870C_POS2_EXPORT_FORMAT(vs->info.vs.pos_exports > 2 ?
+                              S_02870C_POS2_EXPORT_FORMAT(shader->info.vs.pos_exports > 2 ?
                                                           V_02870C_SPI_SHADER_4COMP :
                                                           V_02870C_SPI_SHADER_NONE) |
-                              S_02870C_POS3_EXPORT_FORMAT(vs->info.vs.pos_exports > 3 ?
+                              S_02870C_POS3_EXPORT_FORMAT(shader->info.vs.pos_exports > 3 ?
                                                           V_02870C_SPI_SHADER_4COMP :
                                                           V_02870C_SPI_SHADER_NONE));
 
+
        radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B120_SPI_SHADER_PGM_LO_VS, 4);
        radeon_emit(cmd_buffer->cs, va >> 8);
        radeon_emit(cmd_buffer->cs, va >> 40);
-       radeon_emit(cmd_buffer->cs, vs->rsrc1);
-       radeon_emit(cmd_buffer->cs, vs->rsrc2);
+       radeon_emit(cmd_buffer->cs, shader->rsrc1);
+       radeon_emit(cmd_buffer->cs, shader->rsrc2);
 
        radeon_set_context_reg(cmd_buffer->cs, R_028818_PA_CL_VTE_CNTL,
                               S_028818_VTX_W0_FMT(1) |
@@ -405,18 +539,151 @@ radv_emit_vertex_shader(struct radv_cmd_buffer *cmd_buffer,
                               S_028818_VPORT_Y_SCALE_ENA(1) | S_028818_VPORT_Y_OFFSET_ENA(1) |
                               S_028818_VPORT_Z_SCALE_ENA(1) | S_028818_VPORT_Z_OFFSET_ENA(1));
 
+       unsigned clip_dist_mask, cull_dist_mask, total_mask;
+       clip_dist_mask = shader->info.vs.clip_dist_mask;
+       cull_dist_mask = shader->info.vs.cull_dist_mask;
+       total_mask = clip_dist_mask | cull_dist_mask;
+
        radeon_set_context_reg(cmd_buffer->cs, R_02881C_PA_CL_VS_OUT_CNTL,
-                              S_02881C_USE_VTX_POINT_SIZE(vs->info.vs.writes_pointsize) |
-                              S_02881C_VS_OUT_MISC_VEC_ENA(vs->info.vs.writes_pointsize) |
+                              S_02881C_USE_VTX_POINT_SIZE(shader->info.vs.writes_pointsize) |
+                              S_02881C_USE_VTX_RENDER_TARGET_INDX(shader->info.vs.writes_layer) |
+                              S_02881C_USE_VTX_VIEWPORT_INDX(shader->info.vs.writes_viewport_index) |
+                              S_02881C_VS_OUT_MISC_VEC_ENA(shader->info.vs.writes_pointsize ||
+                                                           shader->info.vs.writes_layer ||
+                                                           shader->info.vs.writes_viewport_index) |
                               S_02881C_VS_OUT_CCDIST0_VEC_ENA((total_mask & 0x0f) != 0) |
                               S_02881C_VS_OUT_CCDIST1_VEC_ENA((total_mask & 0xf0) != 0) |
                               pipeline->graphics.raster.pa_cl_vs_out_cntl |
                               cull_dist_mask << 8 |
                               clip_dist_mask);
 
+       radeon_set_context_reg(cmd_buffer->cs, R_028AB4_VGT_REUSE_OFF,
+                              S_028AB4_REUSE_OFF(shader->info.vs.writes_viewport_index));
 }
 
+static void
+radv_emit_hw_es(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);
+
+       ws->cs_add_buffer(cmd_buffer->cs, shader->bo, 8);
 
+       radeon_set_context_reg(cmd_buffer->cs, R_028AAC_VGT_ESGS_RING_ITEMSIZE,
+                              shader->info.vs.esgs_itemsize / 4);
+       radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B320_SPI_SHADER_PGM_LO_ES, 4);
+       radeon_emit(cmd_buffer->cs, va >> 8);
+       radeon_emit(cmd_buffer->cs, va >> 40);
+       radeon_emit(cmd_buffer->cs, shader->rsrc1);
+       radeon_emit(cmd_buffer->cs, shader->rsrc2);
+}
+
+static void
+radv_emit_vertex_shader(struct radv_cmd_buffer *cmd_buffer,
+                       struct radv_pipeline *pipeline)
+{
+       struct radv_shader_variant *vs;
+
+       assert (pipeline->shaders[MESA_SHADER_VERTEX]);
+
+       vs = pipeline->shaders[MESA_SHADER_VERTEX];
+
+       if (vs->info.vs.as_es)
+               radv_emit_hw_es(cmd_buffer, vs);
+       else
+               radv_emit_hw_vs(cmd_buffer, pipeline, vs);
+
+       radeon_set_context_reg(cmd_buffer->cs, R_028A84_VGT_PRIMITIVEID_EN, 0);
+}
+
+static uint32_t si_vgt_gs_mode(struct radv_shader_variant *gs)
+{
+       unsigned gs_max_vert_out = gs->info.gs.vertices_out;
+       unsigned cut_mode;
+
+       if (gs_max_vert_out <= 128) {
+               cut_mode = V_028A40_GS_CUT_128;
+       } else if (gs_max_vert_out <= 256) {
+               cut_mode = V_028A40_GS_CUT_256;
+       } else if (gs_max_vert_out <= 512) {
+               cut_mode = V_028A40_GS_CUT_512;
+       } else {
+               assert(gs_max_vert_out <= 1024);
+               cut_mode = V_028A40_GS_CUT_1024;
+       }
+
+       return S_028A40_MODE(V_028A40_GS_SCENARIO_G) |
+              S_028A40_CUT_MODE(cut_mode)|
+              S_028A40_ES_WRITE_OPTIMIZE(1) |
+              S_028A40_GS_WRITE_OPTIMIZE(1);
+}
+
+static void
+radv_emit_geometry_shader(struct radv_cmd_buffer *cmd_buffer,
+                         struct radv_pipeline *pipeline)
+{
+       struct radeon_winsys *ws = cmd_buffer->device->ws;
+       struct radv_shader_variant *gs;
+       uint64_t va;
+
+       gs = pipeline->shaders[MESA_SHADER_GEOMETRY];
+       if (!gs) {
+               radeon_set_context_reg(cmd_buffer->cs, R_028A40_VGT_GS_MODE, 0);
+               return;
+       }
+
+       radeon_set_context_reg(cmd_buffer->cs, R_028A40_VGT_GS_MODE, si_vgt_gs_mode(gs));
+
+       uint32_t gsvs_itemsize = gs->info.gs.max_gsvs_emit_size >> 2;
+
+       radeon_set_context_reg_seq(cmd_buffer->cs, R_028A60_VGT_GSVS_RING_OFFSET_1, 3);
+       radeon_emit(cmd_buffer->cs, gsvs_itemsize);
+       radeon_emit(cmd_buffer->cs, gsvs_itemsize);
+       radeon_emit(cmd_buffer->cs, gsvs_itemsize);
+
+       radeon_set_context_reg(cmd_buffer->cs, R_028AB0_VGT_GSVS_RING_ITEMSIZE, gsvs_itemsize);
+
+       radeon_set_context_reg(cmd_buffer->cs, R_028B38_VGT_GS_MAX_VERT_OUT, gs->info.gs.vertices_out);
+
+       uint32_t gs_vert_itemsize = gs->info.gs.gsvs_vertex_size;
+       radeon_set_context_reg_seq(cmd_buffer->cs, R_028B5C_VGT_GS_VERT_ITEMSIZE, 4);
+       radeon_emit(cmd_buffer->cs, gs_vert_itemsize >> 2);
+       radeon_emit(cmd_buffer->cs, 0);
+       radeon_emit(cmd_buffer->cs, 0);
+       radeon_emit(cmd_buffer->cs, 0);
+
+       uint32_t gs_num_invocations = gs->info.gs.invocations;
+       radeon_set_context_reg(cmd_buffer->cs, R_028B90_VGT_GS_INSTANCE_CNT,
+                              S_028B90_CNT(MIN2(gs_num_invocations, 127)) |
+                              S_028B90_ENABLE(gs_num_invocations > 0));
+
+       va = ws->buffer_get_va(gs->bo);
+       ws->cs_add_buffer(cmd_buffer->cs, gs->bo, 8);
+       radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B220_SPI_SHADER_PGM_LO_GS, 4);
+       radeon_emit(cmd_buffer->cs, va >> 8);
+       radeon_emit(cmd_buffer->cs, va >> 40);
+       radeon_emit(cmd_buffer->cs, gs->rsrc1);
+       radeon_emit(cmd_buffer->cs, gs->rsrc2);
+
+       radv_emit_hw_vs(cmd_buffer, pipeline, pipeline->gs_copy_shader);
+
+       struct ac_userdata_info *loc = radv_lookup_user_sgpr(cmd_buffer->state.pipeline, MESA_SHADER_GEOMETRY,
+                                                            AC_UD_GS_VS_RING_STRIDE_ENTRIES);
+       if (loc->sgpr_idx != -1) {
+               uint32_t stride = gs->info.gs.max_gsvs_emit_size;
+               uint32_t num_entries = 64;
+               bool is_vi = cmd_buffer->device->physical_device->rad_info.chip_class >= VI;
+
+               if (is_vi)
+                       num_entries *= stride;
+
+               stride = S_008F04_STRIDE(stride);
+               radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B230_SPI_SHADER_USER_DATA_GS_0 + loc->sgpr_idx * 4, 2);
+               radeon_emit(cmd_buffer->cs, stride);
+               radeon_emit(cmd_buffer->cs, num_entries);
+       }
+}
 
 static void
 radv_emit_fragment_shader(struct radv_cmd_buffer *cmd_buffer,
@@ -432,7 +699,7 @@ radv_emit_fragment_shader(struct radv_cmd_buffer *cmd_buffer,
        assert (pipeline->shaders[MESA_SHADER_FRAGMENT]);
 
        ps = pipeline->shaders[MESA_SHADER_FRAGMENT];
-       vs = pipeline->shaders[MESA_SHADER_VERTEX];
+       vs = radv_pipeline_has_gs(pipeline) ? pipeline->gs_copy_shader : pipeline->shaders[MESA_SHADER_VERTEX];
        va = ws->buffer_get_va(ps->bo);
        ws->cs_add_buffer(cmd_buffer->cs, ps->bo, 8);
 
@@ -452,6 +719,7 @@ radv_emit_fragment_shader(struct radv_cmd_buffer *cmd_buffer,
                               S_02880C_Z_EXPORT_ENABLE(ps->info.fs.writes_z) |
                               S_02880C_STENCIL_TEST_VAL_EXPORT_ENABLE(ps->info.fs.writes_stencil) |
                               S_02880C_KILL_ENABLE(!!ps->info.fs.can_discard) |
+                              S_02880C_MASK_EXPORT_ENABLE(ps->info.fs.writes_sample_mask) |
                               S_02880C_Z_ORDER(z_order) |
                               S_02880C_DEPTH_BEFORE_SHADER(ps->info.fs.early_fragment_test) |
                               S_02880C_EXEC_ON_HIER_FAIL(ps->info.fs.writes_memory) |
@@ -463,13 +731,16 @@ 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);
 
-       spi_baryc_cntl |= S_0286E0_POS_FLOAT_LOCATION(0);
+       if (ps->info.fs.force_persample)
+               spi_baryc_cntl |= S_0286E0_POS_FLOAT_LOCATION(2);
+
        radeon_set_context_reg(cmd_buffer->cs, R_0286D8_SPI_PS_IN_CONTROL,
                               S_0286D8_NUM_INTERP(ps->info.fs.num_interp));
 
        radeon_set_context_reg(cmd_buffer->cs, R_0286E0_SPI_BARYC_CNTL, spi_baryc_cntl);
 
        radeon_set_context_reg(cmd_buffer->cs, R_028710_SPI_SHADER_Z_FORMAT,
+                              ps->info.fs.writes_sample_mask ? V_028710_SPI_SHADER_32_ABGR :
                               ps->info.fs.writes_stencil ? V_028710_SPI_SHADER_32_GR :
                               ps->info.fs.writes_z ? V_028710_SPI_SHADER_32_R :
                               V_028710_SPI_SHADER_ZERO);
@@ -483,7 +754,27 @@ radv_emit_fragment_shader(struct radv_cmd_buffer *cmd_buffer,
                unsigned val;
                val = S_028644_PT_SPRITE_TEX(1) | S_028644_OFFSET(0x20);
                radeon_set_context_reg(cmd_buffer->cs, R_028644_SPI_PS_INPUT_CNTL_0 + 4 * ps_offset, val);
-               ps_offset = 1;
+               ps_offset++;
+       }
+
+       if (ps->info.fs.prim_id_input && (vs->info.vs.prim_id_output != 0xffffffff)) {
+               unsigned vs_offset, flat_shade;
+               unsigned val;
+               vs_offset = vs->info.vs.prim_id_output;
+               flat_shade = true;
+               val = S_028644_OFFSET(vs_offset) | S_028644_FLAT_SHADE(flat_shade);
+               radeon_set_context_reg(cmd_buffer->cs, R_028644_SPI_PS_INPUT_CNTL_0 + 4 * ps_offset, val);
+               ++ps_offset;
+       }
+
+       if (ps->info.fs.layer_input && (vs->info.vs.layer_output != 0xffffffff)) {
+               unsigned vs_offset, flat_shade;
+               unsigned val;
+               vs_offset = vs->info.vs.layer_output;
+               flat_shade = true;
+               val = S_028644_OFFSET(vs_offset) | S_028644_FLAT_SHADE(flat_shade);
+               radeon_set_context_reg(cmd_buffer->cs, R_028644_SPI_PS_INPUT_CNTL_0 + 4 * ps_offset, val);
+               ++ps_offset;
        }
 
        for (unsigned i = 0; i < 32 && (1u << i) <= ps->info.fs.input_mask; ++i) {
@@ -502,6 +793,14 @@ radv_emit_fragment_shader(struct radv_cmd_buffer *cmd_buffer,
                }
 
                vs_offset = util_bitcount(vs->info.vs.export_mask & ((1u << i) - 1));
+               if (vs->info.vs.prim_id_output != 0xffffffff) {
+                       if (vs_offset >= vs->info.vs.prim_id_output)
+                               vs_offset++;
+               }
+               if (vs->info.vs.layer_output != 0xffffffff) {
+                       if (vs_offset >= vs->info.vs.layer_output)
+                         vs_offset++;
+               }
                flat_shade = !!(ps->info.fs.flat_shaded_mask & (1u << ps_offset));
 
                val = S_028644_OFFSET(vs_offset) | S_028644_FLAT_SHADE(flat_shade);
@@ -522,11 +821,19 @@ radv_emit_graphics_pipeline(struct radv_cmd_buffer *cmd_buffer,
        radv_emit_graphics_raster_state(cmd_buffer, pipeline);
        radv_update_multisample_state(cmd_buffer, pipeline);
        radv_emit_vertex_shader(cmd_buffer, pipeline);
+       radv_emit_geometry_shader(cmd_buffer, pipeline);
        radv_emit_fragment_shader(cmd_buffer, pipeline);
 
        radeon_set_context_reg(cmd_buffer->cs, R_028A94_VGT_MULTI_PRIM_IB_RESET_EN,
                               pipeline->graphics.prim_restart_enable);
 
+       cmd_buffer->scratch_size_needed =
+                                 MAX2(cmd_buffer->scratch_size_needed,
+                                      pipeline->max_waves * pipeline->scratch_bytes_per_wave);
+
+       radeon_set_context_reg(cmd_buffer->cs, R_0286E8_SPI_TMPRING_SIZE,
+                              S_0286E8_WAVES(pipeline->max_waves) |
+                              S_0286E8_WAVESIZE(pipeline->scratch_bytes_per_wave >> 10));
        cmd_buffer->state.emitted_pipeline = pipeline;
 }
 
@@ -552,7 +859,7 @@ radv_emit_fb_color_state(struct radv_cmd_buffer *cmd_buffer,
                         int index,
                         struct radv_color_buffer_info *cb)
 {
-       bool is_vi = cmd_buffer->device->instance->physicalDevice.rad_info.chip_class >= VI;
+       bool is_vi = cmd_buffer->device->physical_device->rad_info.chip_class >= VI;
        radeon_set_context_reg_seq(cmd_buffer->cs, R_028C60_CB_COLOR0_BASE + index * 0x3c, 11);
        radeon_emit(cmd_buffer->cs, cb->cb_color_base);
        radeon_emit(cmd_buffer->cs, cb->cb_color_pitch);
@@ -818,13 +1125,13 @@ void radv_set_db_count_control(struct radv_cmd_buffer *cmd_buffer)
        uint32_t db_count_control;
 
        if(!cmd_buffer->state.active_occlusion_queries) {
-               if (cmd_buffer->device->instance->physicalDevice.rad_info.chip_class >= CIK) {
+               if (cmd_buffer->device->physical_device->rad_info.chip_class >= CIK) {
                        db_count_control = 0;
                } else {
                        db_count_control = S_028004_ZPASS_INCREMENT_DISABLE(1);
                }
        } else {
-               if (cmd_buffer->device->instance->physicalDevice.rad_info.chip_class >= CIK) {
+               if (cmd_buffer->device->physical_device->rad_info.chip_class >= CIK) {
                        db_count_control = S_028004_PERFECT_ZPASS_COUNTS(1) |
                                S_028004_SAMPLE_RATE(0) | /* TODO: set this to the number of samples of the current framebuffer */
                                S_028004_ZPASS_ENABLE(1) |
@@ -894,25 +1201,74 @@ radv_cmd_buffer_flush_dynamic_state(struct radv_cmd_buffer *cmd_buffer)
        cmd_buffer->state.dirty = 0;
 }
 
+static void
+emit_stage_descriptor_set_userdata(struct radv_cmd_buffer *cmd_buffer,
+                                  struct radv_pipeline *pipeline,
+                                  int idx,
+                                  uint64_t va,
+                                  gl_shader_stage stage)
+{
+       struct ac_userdata_info *desc_set_loc = &pipeline->shaders[stage]->info.user_sgprs_locs.descriptor_sets[idx];
+       uint32_t base_reg = shader_stage_to_user_data_0(stage, radv_pipeline_has_gs(pipeline));
+
+       if (desc_set_loc->sgpr_idx == -1)
+               return;
+
+       assert(!desc_set_loc->indirect);
+       assert(desc_set_loc->num_sgprs == 2);
+       radeon_set_sh_reg_seq(cmd_buffer->cs,
+                             base_reg + desc_set_loc->sgpr_idx * 4, 2);
+       radeon_emit(cmd_buffer->cs, va);
+       radeon_emit(cmd_buffer->cs, va >> 32);
+}
+
 static void
 radv_emit_descriptor_set_userdata(struct radv_cmd_buffer *cmd_buffer,
+                                 struct radv_pipeline *pipeline,
+                                 VkShaderStageFlags stages,
                                  struct radv_descriptor_set *set,
                                  unsigned idx)
 {
-       radeon_set_sh_reg_seq(cmd_buffer->cs,
-                               R_00B030_SPI_SHADER_USER_DATA_PS_0 + 8 * idx, 2);
-       radeon_emit(cmd_buffer->cs, set->va);
-       radeon_emit(cmd_buffer->cs, set->va >> 32);
+       if (stages & VK_SHADER_STAGE_FRAGMENT_BIT)
+               emit_stage_descriptor_set_userdata(cmd_buffer, pipeline,
+                                                  idx, set->va,
+                                                  MESA_SHADER_FRAGMENT);
 
-       radeon_set_sh_reg_seq(cmd_buffer->cs,
-                               R_00B130_SPI_SHADER_USER_DATA_VS_0 + 8 * idx, 2);
-       radeon_emit(cmd_buffer->cs, set->va);
-       radeon_emit(cmd_buffer->cs, set->va >> 32);
+       if (stages & VK_SHADER_STAGE_VERTEX_BIT)
+               emit_stage_descriptor_set_userdata(cmd_buffer, pipeline,
+                                                  idx, set->va,
+                                                  MESA_SHADER_VERTEX);
 
-       radeon_set_sh_reg_seq(cmd_buffer->cs,
-                               R_00B900_COMPUTE_USER_DATA_0 + 8 * idx, 2);
-       radeon_emit(cmd_buffer->cs, set->va);
-       radeon_emit(cmd_buffer->cs, set->va >> 32);
+       if ((stages & VK_SHADER_STAGE_GEOMETRY_BIT) && radv_pipeline_has_gs(pipeline))
+               emit_stage_descriptor_set_userdata(cmd_buffer, pipeline,
+                                                  idx, set->va,
+                                                  MESA_SHADER_GEOMETRY);
+
+       if (stages & VK_SHADER_STAGE_COMPUTE_BIT)
+               emit_stage_descriptor_set_userdata(cmd_buffer, pipeline,
+                                                  idx, set->va,
+                                                  MESA_SHADER_COMPUTE);
+}
+
+static void
+radv_flush_descriptors(struct radv_cmd_buffer *cmd_buffer,
+                      struct radv_pipeline *pipeline,
+                      VkShaderStageFlags stages)
+{
+       unsigned i;
+       if (!cmd_buffer->state.descriptors_dirty)
+               return;
+
+       for (i = 0; i < MAX_SETS; i++) {
+               if (!(cmd_buffer->state.descriptors_dirty & (1 << i)))
+                       continue;
+               struct radv_descriptor_set *set = cmd_buffer->state.descriptors[i];
+               if (!set)
+                       continue;
+
+               radv_emit_descriptor_set_userdata(cmd_buffer, pipeline, stages, set, i);
+       }
+       cmd_buffer->state.descriptors_dirty = 0;
 }
 
 static void
@@ -940,40 +1296,36 @@ radv_flush_constants(struct radv_cmd_buffer *cmd_buffer,
        va = cmd_buffer->device->ws->buffer_get_va(cmd_buffer->upload.upload_bo);
        va += offset;
 
-       if (stages & VK_SHADER_STAGE_VERTEX_BIT) {
-               radeon_set_sh_reg_seq(cmd_buffer->cs,
-                                     R_00B130_SPI_SHADER_USER_DATA_VS_0 + AC_USERDATA_PUSH_CONST_DYN * 4, 2);
-               radeon_emit(cmd_buffer->cs, va);
-               radeon_emit(cmd_buffer->cs, va >> 32);
-       }
+       if (stages & VK_SHADER_STAGE_VERTEX_BIT)
+               radv_emit_userdata_address(cmd_buffer, pipeline, MESA_SHADER_VERTEX,
+                                          AC_UD_PUSH_CONSTANTS, va);
 
-       if (stages & VK_SHADER_STAGE_FRAGMENT_BIT) {
-               radeon_set_sh_reg_seq(cmd_buffer->cs,
-                                     R_00B030_SPI_SHADER_USER_DATA_PS_0 + AC_USERDATA_PUSH_CONST_DYN * 4, 2);
-               radeon_emit(cmd_buffer->cs, va);
-               radeon_emit(cmd_buffer->cs, va >> 32);
-       }
+       if (stages & VK_SHADER_STAGE_FRAGMENT_BIT)
+               radv_emit_userdata_address(cmd_buffer, pipeline, MESA_SHADER_FRAGMENT,
+                                          AC_UD_PUSH_CONSTANTS, va);
 
-       if (stages & VK_SHADER_STAGE_COMPUTE_BIT) {
-               radeon_set_sh_reg_seq(cmd_buffer->cs,
-                                     R_00B900_COMPUTE_USER_DATA_0 + AC_USERDATA_PUSH_CONST_DYN * 4, 2);
-               radeon_emit(cmd_buffer->cs, va);
-               radeon_emit(cmd_buffer->cs, va >> 32);
-       }
+       if ((stages & VK_SHADER_STAGE_GEOMETRY_BIT) && radv_pipeline_has_gs(pipeline))
+               radv_emit_userdata_address(cmd_buffer, pipeline, MESA_SHADER_GEOMETRY,
+                                          AC_UD_PUSH_CONSTANTS, va);
+
+       if (stages & VK_SHADER_STAGE_COMPUTE_BIT)
+               radv_emit_userdata_address(cmd_buffer, pipeline, MESA_SHADER_COMPUTE,
+                                          AC_UD_PUSH_CONSTANTS, va);
 
        cmd_buffer->push_constant_stages &= ~stages;
 }
 
 static void
-radv_cmd_buffer_flush_state(struct radv_cmd_buffer *cmd_buffer)
+radv_cmd_buffer_flush_state(struct radv_cmd_buffer *cmd_buffer, bool instanced_or_indirect_draw,
+                           uint32_t draw_vertex_count)
 {
        struct radv_pipeline *pipeline = cmd_buffer->state.pipeline;
        struct radv_device *device = cmd_buffer->device;
        uint32_t ia_multi_vgt_param;
        uint32_t ls_hs_config = 0;
 
-       unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs,
-                                             4096);
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
+                                                          cmd_buffer->cs, 4096);
 
        if ((cmd_buffer->state.vertex_descriptors_dirty || cmd_buffer->state.vb_dirty) &&
            cmd_buffer->state.pipeline->num_vertex_attribs) {
@@ -1001,7 +1353,7 @@ radv_cmd_buffer_flush_state(struct radv_cmd_buffer *cmd_buffer)
                        va += offset + buffer->offset;
                        desc[0] = va;
                        desc[1] = S_008F04_BASE_ADDRESS_HI(va >> 32) | S_008F04_STRIDE(stride);
-                       if (cmd_buffer->device->instance->physicalDevice.rad_info.chip_class <= CIK && 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;
                        else
                                desc[2] = buffer->size - offset;
@@ -1010,11 +1362,9 @@ radv_cmd_buffer_flush_state(struct radv_cmd_buffer *cmd_buffer)
 
                va = device->ws->buffer_get_va(cmd_buffer->upload.upload_bo);
                va += vb_offset;
-               radeon_set_sh_reg_seq(cmd_buffer->cs,
-                                     R_00B130_SPI_SHADER_USER_DATA_VS_0 + AC_USERDATA_VS_VERTEX_BUFFERS * 4, 2);
-               radeon_emit(cmd_buffer->cs, va);
-               radeon_emit(cmd_buffer->cs, va >> 32);
 
+               radv_emit_userdata_address(cmd_buffer, pipeline, MESA_SHADER_VERTEX,
+                                          AC_UD_VS_VERTEX_BUFFERS, va);
        }
 
        cmd_buffer->state.vertex_descriptors_dirty = false;
@@ -1031,17 +1381,30 @@ radv_cmd_buffer_flush_state(struct radv_cmd_buffer *cmd_buffer)
        if (cmd_buffer->state.dirty & (RADV_CMD_DIRTY_DYNAMIC_SCISSOR))
                radv_emit_scissor(cmd_buffer);
 
+       ia_multi_vgt_param = si_get_ia_multi_vgt_param(cmd_buffer, instanced_or_indirect_draw, draw_vertex_count);
+       if (cmd_buffer->state.last_ia_multi_vgt_param != ia_multi_vgt_param) {
+               if (cmd_buffer->device->physical_device->rad_info.chip_class >= CIK)
+                       radeon_set_context_reg_idx(cmd_buffer->cs, R_028AA8_IA_MULTI_VGT_PARAM, 1, ia_multi_vgt_param);
+               else
+                       radeon_set_context_reg(cmd_buffer->cs, R_028AA8_IA_MULTI_VGT_PARAM, ia_multi_vgt_param);
+               cmd_buffer->state.last_ia_multi_vgt_param = ia_multi_vgt_param;
+       }
+
        if (cmd_buffer->state.dirty & RADV_CMD_DIRTY_PIPELINE) {
-               radeon_set_context_reg(cmd_buffer->cs, R_028B54_VGT_SHADER_STAGES_EN, 0);
-               ia_multi_vgt_param = si_get_ia_multi_vgt_param(cmd_buffer);
+               uint32_t stages = 0;
 
-               if (cmd_buffer->device->instance->physicalDevice.rad_info.chip_class >= CIK) {
-                       radeon_set_context_reg_idx(cmd_buffer->cs, R_028AA8_IA_MULTI_VGT_PARAM, 1, ia_multi_vgt_param);
+               if (radv_pipeline_has_gs(cmd_buffer->state.pipeline))
+                       stages |= S_028B54_ES_EN(V_028B54_ES_STAGE_REAL) |
+                               S_028B54_GS_EN(1) |
+                               S_028B54_VS_EN(V_028B54_VS_STAGE_COPY_SHADER);
+
+               radeon_set_context_reg(cmd_buffer->cs, R_028B54_VGT_SHADER_STAGES_EN, stages);
+
+               if (cmd_buffer->device->physical_device->rad_info.chip_class >= CIK) {
                        radeon_set_context_reg_idx(cmd_buffer->cs, R_028B58_VGT_LS_HS_CONFIG, 2, ls_hs_config);
                        radeon_set_uconfig_reg_idx(cmd_buffer->cs, R_030908_VGT_PRIMITIVE_TYPE, 1, cmd_buffer->state.pipeline->graphics.prim);
                } else {
                        radeon_set_config_reg(cmd_buffer->cs, R_008958_VGT_PRIMITIVE_TYPE, cmd_buffer->state.pipeline->graphics.prim);
-                       radeon_set_context_reg(cmd_buffer->cs, R_028AA8_IA_MULTI_VGT_PARAM, ia_multi_vgt_param);
                        radeon_set_context_reg(cmd_buffer->cs, R_028B58_VGT_LS_HS_CONFIG, ls_hs_config);
                }
                radeon_set_context_reg(cmd_buffer->cs, R_028A6C_VGT_GS_OUT_PRIM_TYPE, cmd_buffer->state.pipeline->graphics.gs_out);
@@ -1049,6 +1412,8 @@ radv_cmd_buffer_flush_state(struct radv_cmd_buffer *cmd_buffer)
 
        radv_cmd_buffer_flush_dynamic_state(cmd_buffer);
 
+       radv_flush_descriptors(cmd_buffer, cmd_buffer->state.pipeline,
+                              VK_SHADER_STAGE_ALL_GRAPHICS);
        radv_flush_constants(cmd_buffer, cmd_buffer->state.pipeline,
                             VK_SHADER_STAGE_ALL_GRAPHICS);
 
@@ -1087,11 +1452,67 @@ static void radv_stage_flush(struct radv_cmd_buffer *cmd_buffer,
        }
 }
 
+static void radv_src_access_flush(struct radv_cmd_buffer *cmd_buffer,
+                                 VkAccessFlags src_flags)
+{
+       enum radv_cmd_flush_bits flush_bits = 0;
+       uint32_t b;
+       for_each_bit(b, src_flags) {
+               switch ((VkAccessFlagBits)(1 << b)) {
+               case VK_ACCESS_SHADER_WRITE_BIT:
+                       flush_bits |= RADV_CMD_FLAG_INV_GLOBAL_L2;
+                       break;
+               case VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT:
+                       flush_bits |= RADV_CMD_FLAG_FLUSH_AND_INV_CB;
+                       break;
+               case VK_ACCESS_DEPTH_STENCIL_ATTACHMENT_WRITE_BIT:
+                       flush_bits |= RADV_CMD_FLAG_FLUSH_AND_INV_DB;
+                       break;
+               case VK_ACCESS_TRANSFER_WRITE_BIT:
+                       flush_bits |= RADV_CMD_FLAG_FLUSH_AND_INV_CB;
+                       break;
+               default:
+                       break;
+               }
+       }
+       cmd_buffer->state.flush_bits |= flush_bits;
+}
+
+static void radv_dst_access_flush(struct radv_cmd_buffer *cmd_buffer,
+                                 VkAccessFlags dst_flags)
+{
+       enum radv_cmd_flush_bits flush_bits = 0;
+       uint32_t b;
+       for_each_bit(b, dst_flags) {
+               switch ((VkAccessFlagBits)(1 << b)) {
+               case VK_ACCESS_INDIRECT_COMMAND_READ_BIT:
+               case VK_ACCESS_INDEX_READ_BIT:
+               case VK_ACCESS_VERTEX_ATTRIBUTE_READ_BIT:
+                       flush_bits |= RADV_CMD_FLAG_INV_VMEM_L1;
+                       break;
+               case VK_ACCESS_UNIFORM_READ_BIT:
+                       flush_bits |= RADV_CMD_FLAG_INV_VMEM_L1 | RADV_CMD_FLAG_INV_SMEM_L1;
+                       break;
+               case VK_ACCESS_SHADER_READ_BIT:
+                       flush_bits |= RADV_CMD_FLAG_INV_GLOBAL_L2;
+                       break;
+               case VK_ACCESS_COLOR_ATTACHMENT_READ_BIT:
+               case VK_ACCESS_TRANSFER_READ_BIT:
+               case VK_ACCESS_TRANSFER_WRITE_BIT:
+               case VK_ACCESS_INPUT_ATTACHMENT_READ_BIT:
+                       flush_bits |= RADV_CMD_FLUSH_AND_INV_FRAMEBUFFER | RADV_CMD_FLAG_INV_GLOBAL_L2;
+               default:
+                       break;
+               }
+       }
+       cmd_buffer->state.flush_bits |= flush_bits;
+}
+
 static void radv_subpass_barrier(struct radv_cmd_buffer *cmd_buffer, const struct radv_subpass_barrier *barrier)
 {
+       radv_src_access_flush(cmd_buffer, barrier->src_access_mask);
        radv_stage_flush(cmd_buffer, barrier->src_stage_mask);
-
-       /* TODO: actual cache flushes */
+       radv_dst_access_flush(cmd_buffer, barrier->dst_access_mask);
 }
 
 static void radv_handle_subpass_image_transition(struct radv_cmd_buffer *cmd_buffer,
@@ -1109,7 +1530,7 @@ static void radv_handle_subpass_image_transition(struct radv_cmd_buffer *cmd_buf
        radv_handle_image_transition(cmd_buffer,
                                     view->image,
                                     cmd_buffer->state.attachments[idx].current_layout,
-                                    att.layout, range,
+                                    att.layout, 0, 0, &range,
                                     cmd_buffer->state.attachments[idx].pending_clear_aspects);
 
        cmd_buffer->state.attachments[idx].current_layout = att.layout;
@@ -1209,9 +1630,26 @@ VkResult radv_AllocateCommandBuffers(
        VkResult result = VK_SUCCESS;
        uint32_t i;
 
+       memset(pCommandBuffers, 0,
+                       sizeof(*pCommandBuffers)*pAllocateInfo->commandBufferCount);
+
        for (i = 0; i < pAllocateInfo->commandBufferCount; i++) {
-               result = radv_create_cmd_buffer(device, pool, pAllocateInfo->level,
-                                               &pCommandBuffers[i]);
+
+               if (!list_empty(&pool->free_cmd_buffers)) {
+                       struct radv_cmd_buffer *cmd_buffer = list_first_entry(&pool->free_cmd_buffers, struct radv_cmd_buffer, pool_link);
+
+                       list_del(&cmd_buffer->pool_link);
+                       list_addtail(&cmd_buffer->pool_link, &pool->cmd_buffers);
+
+                       radv_reset_cmd_buffer(cmd_buffer);
+                       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]);
+               }
                if (result != VK_SUCCESS)
                        break;
        }
@@ -1223,24 +1661,6 @@ VkResult radv_AllocateCommandBuffers(
        return result;
 }
 
-static void
-radv_cmd_buffer_destroy(struct radv_cmd_buffer *cmd_buffer)
-{
-       list_del(&cmd_buffer->pool_link);
-
-       list_for_each_entry_safe(struct radv_cmd_buffer_upload, up,
-                                &cmd_buffer->upload.list, list) {
-               cmd_buffer->device->ws->buffer_destroy(up->upload_bo);
-               list_del(&up->list);
-               free(up);
-       }
-
-       if (cmd_buffer->upload.upload_bo)
-               cmd_buffer->device->ws->buffer_destroy(cmd_buffer->upload.upload_bo);
-       cmd_buffer->device->ws->cs_destroy(cmd_buffer->cs);
-       vk_free(&cmd_buffer->pool->alloc, cmd_buffer);
-}
-
 void radv_FreeCommandBuffers(
        VkDevice device,
        VkCommandPool commandPool,
@@ -1250,29 +1670,15 @@ void radv_FreeCommandBuffers(
        for (uint32_t i = 0; i < commandBufferCount; i++) {
                RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, pCommandBuffers[i]);
 
-               if (cmd_buffer)
-                       radv_cmd_buffer_destroy(cmd_buffer);
-       }
-}
-
-static void  radv_reset_cmd_buffer(struct radv_cmd_buffer *cmd_buffer)
-{
-
-       cmd_buffer->device->ws->cs_reset(cmd_buffer->cs);
+               if (cmd_buffer) {
+                       if (cmd_buffer->pool) {
+                               list_del(&cmd_buffer->pool_link);
+                               list_addtail(&cmd_buffer->pool_link, &cmd_buffer->pool->free_cmd_buffers);
+                       } else
+                               radv_cmd_buffer_destroy(cmd_buffer);
 
-       list_for_each_entry_safe(struct radv_cmd_buffer_upload, up,
-                                &cmd_buffer->upload.list, list) {
-               cmd_buffer->device->ws->buffer_destroy(up->upload_bo);
-               list_del(&up->list);
-               free(up);
+               }
        }
-
-       if (cmd_buffer->upload.upload_bo)
-               cmd_buffer->device->ws->cs_add_buffer(cmd_buffer->cs,
-                                                     cmd_buffer->upload.upload_bo, 8);
-       cmd_buffer->upload.offset = 0;
-
-       cmd_buffer->record_fail = false;
 }
 
 VkResult radv_ResetCommandBuffer(
@@ -1284,6 +1690,20 @@ VkResult radv_ResetCommandBuffer(
        return VK_SUCCESS;
 }
 
+static void emit_gfx_buffer_state(struct radv_cmd_buffer *cmd_buffer)
+{
+       struct radv_device *device = cmd_buffer->device;
+       if (device->gfx_init) {
+               uint64_t va = device->ws->buffer_get_va(device->gfx_init);
+               device->ws->cs_add_buffer(cmd_buffer->cs, device->gfx_init, 8);
+               radeon_emit(cmd_buffer->cs, PKT3(PKT3_INDIRECT_BUFFER_CIK, 2, 0));
+               radeon_emit(cmd_buffer->cs, va);
+               radeon_emit(cmd_buffer->cs, (va >> 32) & 0xffff);
+               radeon_emit(cmd_buffer->cs, device->gfx_init_size_dw & 0xffff);
+       } else
+               si_init_config(cmd_buffer);
+}
+
 VkResult radv_BeginCommandBuffer(
        VkCommandBuffer commandBuffer,
        const VkCommandBufferBeginInfo *pBeginInfo)
@@ -1295,17 +1715,18 @@ VkResult radv_BeginCommandBuffer(
 
        /* setup initial configuration into command buffer */
        if (cmd_buffer->level == VK_COMMAND_BUFFER_LEVEL_PRIMARY) {
-               /* Flush read caches at the beginning of CS not flushed by the kernel. */
-               cmd_buffer->state.flush_bits |= RADV_CMD_FLAG_INV_ICACHE |
-                       RADV_CMD_FLAG_PS_PARTIAL_FLUSH |
-                       RADV_CMD_FLAG_CS_PARTIAL_FLUSH |
-                       RADV_CMD_FLAG_INV_VMEM_L1 |
-                       RADV_CMD_FLAG_INV_SMEM_L1 |
-                       RADV_CMD_FLUSH_AND_INV_FRAMEBUFFER |
-                       RADV_CMD_FLAG_INV_GLOBAL_L2;
-               si_init_config(&cmd_buffer->device->instance->physicalDevice, cmd_buffer);
-               radv_set_db_count_control(cmd_buffer);
-               si_emit_cache_flush(cmd_buffer);
+               switch (cmd_buffer->queue_family_index) {
+               case RADV_QUEUE_GENERAL:
+                       emit_gfx_buffer_state(cmd_buffer);
+                       radv_set_db_count_control(cmd_buffer);
+                       break;
+               case RADV_QUEUE_COMPUTE:
+                       si_init_compute(cmd_buffer);
+                       break;
+               case RADV_QUEUE_TRANSFER:
+               default:
+                       break;
+               }
        }
 
        if (pBeginInfo->flags & VK_COMMAND_BUFFER_USAGE_RENDER_PASS_CONTINUE_BIT) {
@@ -1366,7 +1787,7 @@ void radv_bind_descriptor_set(struct radv_cmd_buffer *cmd_buffer,
        struct radeon_winsys *ws = cmd_buffer->device->ws;
 
        cmd_buffer->state.descriptors[idx] = set;
-
+       cmd_buffer->state.descriptors_dirty |= (1 << idx);
        if (!set)
                return;
 
@@ -1374,7 +1795,6 @@ void radv_bind_descriptor_set(struct radv_cmd_buffer *cmd_buffer,
                if (set->descriptors[j])
                        ws->cs_add_buffer(cmd_buffer->cs, set->descriptors[j], 7);
 
-       radv_emit_descriptor_set_userdata(cmd_buffer, set, idx);
        if(set->bo)
                ws->cs_add_buffer(cmd_buffer->cs, set->bo, 8);
 }
@@ -1393,8 +1813,8 @@ void radv_CmdBindDescriptorSets(
        RADV_FROM_HANDLE(radv_pipeline_layout, layout, _layout);
        unsigned dyn_idx = 0;
 
-       unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs,
-                                             MAX_SETS * 4 * 6);
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
+                                                          cmd_buffer->cs, MAX_SETS * 4 * 6);
 
        for (unsigned i = 0; i < descriptorSetCount; ++i) {
                unsigned idx = i + firstSet;
@@ -1442,7 +1862,9 @@ VkResult radv_EndCommandBuffer(
 {
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
 
-       si_emit_cache_flush(cmd_buffer);
+       if (cmd_buffer->queue_family_index != RADV_QUEUE_TRANSFER)
+               si_emit_cache_flush(cmd_buffer);
+
        if (!cmd_buffer->device->ws->cs_finalize(cmd_buffer->cs) ||
            cmd_buffer->record_fail)
                return VK_ERROR_OUT_OF_DEVICE_MEMORY;
@@ -1467,7 +1889,8 @@ radv_emit_compute_pipeline(struct radv_cmd_buffer *cmd_buffer)
 
        ws->cs_add_buffer(cmd_buffer->cs, compute_shader->bo, 8);
 
-       unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 16);
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
+                                                          cmd_buffer->cs, 16);
 
        radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B830_COMPUTE_PGM_LO, 2);
        radeon_emit(cmd_buffer->cs, va >> 8);
@@ -1477,9 +1900,15 @@ radv_emit_compute_pipeline(struct radv_cmd_buffer *cmd_buffer)
        radeon_emit(cmd_buffer->cs, compute_shader->rsrc1);
        radeon_emit(cmd_buffer->cs, compute_shader->rsrc2);
 
+
+       cmd_buffer->compute_scratch_size_needed =
+                                 MAX2(cmd_buffer->compute_scratch_size_needed,
+                                      pipeline->max_waves * pipeline->scratch_bytes_per_wave);
+
        /* change these once we have scratch support */
        radeon_set_sh_reg(cmd_buffer->cs, R_00B860_COMPUTE_TMPRING_SIZE,
-                         S_00B860_WAVES(32) | S_00B860_WAVESIZE(0));
+                         S_00B860_WAVES(pipeline->max_waves) |
+                         S_00B860_WAVESIZE(pipeline->scratch_bytes_per_wave >> 10));
 
        radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B81C_COMPUTE_NUM_THREAD_X, 3);
        radeon_emit(cmd_buffer->cs,
@@ -1501,6 +1930,11 @@ void radv_CmdBindPipeline(
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
        RADV_FROM_HANDLE(radv_pipeline, pipeline, _pipeline);
 
+       for (unsigned i = 0; i < MAX_SETS; i++) {
+               if (cmd_buffer->state.descriptors[i])
+                       cmd_buffer->state.descriptors_dirty |= (1 << i);
+       }
+
        switch (pipelineBindPoint) {
        case VK_PIPELINE_BIND_POINT_COMPUTE:
                cmd_buffer->state.compute_pipeline = pipeline;
@@ -1517,6 +1951,20 @@ void radv_CmdBindPipeline(
                radv_dynamic_state_copy(&cmd_buffer->state.dynamic,
                                        &pipeline->dynamic_state,
                                        pipeline->dynamic_state_mask);
+
+               if (pipeline->graphics.esgs_ring_size > cmd_buffer->esgs_ring_size_needed)
+                       cmd_buffer->esgs_ring_size_needed = pipeline->graphics.esgs_ring_size;
+               if (pipeline->graphics.gsvs_ring_size > cmd_buffer->gsvs_ring_size_needed)
+                       cmd_buffer->gsvs_ring_size_needed = pipeline->graphics.gsvs_ring_size;
+
+               if (radv_pipeline_has_gs(pipeline)) {
+                       struct ac_userdata_info *loc = radv_lookup_user_sgpr(cmd_buffer->state.pipeline, MESA_SHADER_GEOMETRY,
+                                                                            AC_UD_SCRATCH_RING_OFFSETS);
+                       if (cmd_buffer->ring_offsets_idx == -1)
+                               cmd_buffer->ring_offsets_idx = loc->sgpr_idx;
+                       else if (loc->sgpr_idx != -1)
+                               assert(loc->sgpr_idx == cmd_buffer->ring_offsets_idx);
+               }
                break;
        default:
                assert(!"invalid bind point");
@@ -1664,12 +2112,29 @@ void radv_CmdExecuteCommands(
        for (uint32_t i = 0; i < commandBufferCount; i++) {
                RADV_FROM_HANDLE(radv_cmd_buffer, secondary, pCmdBuffers[i]);
 
+               primary->scratch_size_needed = MAX2(primary->scratch_size_needed,
+                                                   secondary->scratch_size_needed);
+               primary->compute_scratch_size_needed = MAX2(primary->compute_scratch_size_needed,
+                                                           secondary->compute_scratch_size_needed);
+
+               if (secondary->esgs_ring_size_needed > primary->esgs_ring_size_needed)
+                       primary->esgs_ring_size_needed = secondary->esgs_ring_size_needed;
+               if (secondary->gsvs_ring_size_needed > primary->gsvs_ring_size_needed)
+                       primary->gsvs_ring_size_needed = secondary->gsvs_ring_size_needed;
+
+               if (secondary->ring_offsets_idx != -1) {
+                       if (primary->ring_offsets_idx == -1)
+                               primary->ring_offsets_idx = secondary->ring_offsets_idx;
+                       else
+                               assert(secondary->ring_offsets_idx == primary->ring_offsets_idx);
+               }
                primary->device->ws->cs_execute_secondary(primary->cs, secondary->cs);
        }
 
        /* if we execute secondary we need to re-emit out pipelines */
        if (commandBufferCount) {
                primary->state.emitted_pipeline = NULL;
+               primary->state.emitted_compute_pipeline = NULL;
                primary->state.dirty |= RADV_CMD_DIRTY_PIPELINE;
                primary->state.dirty |= RADV_CMD_DIRTY_DYNAMIC_ALL;
        }
@@ -1695,6 +2160,9 @@ VkResult radv_CreateCommandPool(
                pool->alloc = device->alloc;
 
        list_inithead(&pool->cmd_buffers);
+       list_inithead(&pool->free_cmd_buffers);
+
+       pool->queue_family_index = pCreateInfo->queueFamilyIndex;
 
        *pCmdPool = radv_cmd_pool_to_handle(pool);
 
@@ -1718,6 +2186,11 @@ void radv_DestroyCommandPool(
                radv_cmd_buffer_destroy(cmd_buffer);
        }
 
+       list_for_each_entry_safe(struct radv_cmd_buffer, cmd_buffer,
+                                &pool->free_cmd_buffers, pool_link) {
+               radv_cmd_buffer_destroy(cmd_buffer);
+       }
+
        vk_free2(&device->alloc, pAllocator, pool);
 }
 
@@ -1736,6 +2209,22 @@ VkResult radv_ResetCommandPool(
        return VK_SUCCESS;
 }
 
+void radv_TrimCommandPoolKHR(
+    VkDevice                                    device,
+    VkCommandPool                               commandPool,
+    VkCommandPoolTrimFlagsKHR                   flags)
+{
+       RADV_FROM_HANDLE(radv_cmd_pool, pool, commandPool);
+
+       if (!pool)
+               return;
+
+       list_for_each_entry_safe(struct radv_cmd_buffer, cmd_buffer,
+                                &pool->free_cmd_buffers, pool_link) {
+               radv_cmd_buffer_destroy(cmd_buffer);
+       }
+}
+
 void radv_CmdBeginRenderPass(
        VkCommandBuffer                             commandBuffer,
        const VkRenderPassBeginInfo*                pRenderPassBegin,
@@ -1745,8 +2234,8 @@ void radv_CmdBeginRenderPass(
        RADV_FROM_HANDLE(radv_render_pass, pass, pRenderPassBegin->renderPass);
        RADV_FROM_HANDLE(radv_framebuffer, framebuffer, pRenderPassBegin->framebuffer);
 
-       unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs,
-                                             2048);
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
+                                                          cmd_buffer->cs, 2048);
 
        cmd_buffer->state.framebuffer = framebuffer;
        cmd_buffer->state.pass = pass;
@@ -1785,13 +2274,20 @@ void radv_CmdDraw(
        uint32_t                                    firstInstance)
 {
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
-       radv_cmd_buffer_flush_state(cmd_buffer);
 
-       unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 9);
+       radv_cmd_buffer_flush_state(cmd_buffer, (instanceCount > 1), vertexCount);
+
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 10);
 
-       radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B130_SPI_SHADER_USER_DATA_VS_0 + AC_USERDATA_VS_BASE_VERTEX * 4, 2);
-       radeon_emit(cmd_buffer->cs, firstVertex);
-       radeon_emit(cmd_buffer->cs, firstInstance);
+       struct ac_userdata_info *loc = radv_lookup_user_sgpr(cmd_buffer->state.pipeline, MESA_SHADER_VERTEX,
+                                                            AC_UD_VS_BASE_VERTEX_START_INSTANCE);
+       if (loc->sgpr_idx != -1) {
+               uint32_t base_reg = shader_stage_to_user_data_0(MESA_SHADER_VERTEX, radv_pipeline_has_gs(cmd_buffer->state.pipeline));
+               radeon_set_sh_reg_seq(cmd_buffer->cs, base_reg + loc->sgpr_idx * 4, 3);
+               radeon_emit(cmd_buffer->cs, firstVertex);
+               radeon_emit(cmd_buffer->cs, firstInstance);
+               radeon_emit(cmd_buffer->cs, 0);
+       }
        radeon_emit(cmd_buffer->cs, PKT3(PKT3_NUM_INSTANCES, 0, 0));
        radeon_emit(cmd_buffer->cs, instanceCount);
 
@@ -1801,6 +2297,8 @@ void radv_CmdDraw(
                    S_0287F0_USE_OPAQUE(0));
 
        assert(cmd_buffer->cs->cdw <= cdw_max);
+
+       radv_cmd_buffer_trace_emit(cmd_buffer);
 }
 
 static void radv_emit_primitive_reset_index(struct radv_cmd_buffer *cmd_buffer)
@@ -1828,17 +2326,23 @@ void radv_CmdDrawIndexed(
        uint32_t index_max_size = (cmd_buffer->state.index_buffer->size - cmd_buffer->state.index_offset) / index_size;
        uint64_t index_va;
 
-       radv_cmd_buffer_flush_state(cmd_buffer);
+       radv_cmd_buffer_flush_state(cmd_buffer, (instanceCount > 1), indexCount);
        radv_emit_primitive_reset_index(cmd_buffer);
 
-       unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 14);
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 15);
 
        radeon_emit(cmd_buffer->cs, PKT3(PKT3_INDEX_TYPE, 0, 0));
        radeon_emit(cmd_buffer->cs, cmd_buffer->state.index_type);
 
-       radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B130_SPI_SHADER_USER_DATA_VS_0 + AC_USERDATA_VS_BASE_VERTEX * 4, 2);
-       radeon_emit(cmd_buffer->cs, vertexOffset);
-       radeon_emit(cmd_buffer->cs, firstInstance);
+       struct ac_userdata_info *loc = radv_lookup_user_sgpr(cmd_buffer->state.pipeline, MESA_SHADER_VERTEX,
+                                                            AC_UD_VS_BASE_VERTEX_START_INSTANCE);
+       if (loc->sgpr_idx != -1) {
+               uint32_t base_reg = shader_stage_to_user_data_0(MESA_SHADER_VERTEX, radv_pipeline_has_gs(cmd_buffer->state.pipeline));
+               radeon_set_sh_reg_seq(cmd_buffer->cs, base_reg + loc->sgpr_idx * 4, 3);
+               radeon_emit(cmd_buffer->cs, vertexOffset);
+               radeon_emit(cmd_buffer->cs, firstInstance);
+               radeon_emit(cmd_buffer->cs, 0);
+       }
        radeon_emit(cmd_buffer->cs, PKT3(PKT3_NUM_INSTANCES, 0, 0));
        radeon_emit(cmd_buffer->cs, instanceCount);
 
@@ -1852,6 +2356,7 @@ void radv_CmdDrawIndexed(
        radeon_emit(cmd_buffer->cs, V_0287F0_DI_SRC_SEL_DMA);
 
        assert(cmd_buffer->cs->cdw <= cdw_max);
+       radv_cmd_buffer_trace_emit(cmd_buffer);
 }
 
 static void
@@ -1883,6 +2388,10 @@ radv_emit_indirect_draw(struct radv_cmd_buffer *cmd_buffer,
 
        cmd_buffer->device->ws->cs_add_buffer(cs, buffer->bo, 8);
 
+       struct ac_userdata_info *loc = radv_lookup_user_sgpr(cmd_buffer->state.pipeline, MESA_SHADER_VERTEX,
+                                                            AC_UD_VS_BASE_VERTEX_START_INSTANCE);
+       uint32_t base_reg = shader_stage_to_user_data_0(MESA_SHADER_VERTEX, radv_pipeline_has_gs(cmd_buffer->state.pipeline));
+       assert(loc->sgpr_idx != -1);
        radeon_emit(cs, PKT3(PKT3_SET_BASE, 2, 0));
        radeon_emit(cs, 1);
        radeon_emit(cs, indirect_va);
@@ -1892,14 +2401,17 @@ radv_emit_indirect_draw(struct radv_cmd_buffer *cmd_buffer,
                                       PKT3_DRAW_INDIRECT_MULTI,
                             8, false));
        radeon_emit(cs, 0);
-       radeon_emit(cs, ((R_00B130_SPI_SHADER_USER_DATA_VS_0 + AC_USERDATA_VS_BASE_VERTEX * 4) - SI_SH_REG_OFFSET) >> 2);
-       radeon_emit(cs, ((R_00B130_SPI_SHADER_USER_DATA_VS_0 + AC_USERDATA_VS_START_INSTANCE * 4) - SI_SH_REG_OFFSET) >> 2);
-       radeon_emit(cs, S_2C3_COUNT_INDIRECT_ENABLE(!!count_va)); /* draw_index and count_indirect enable */
+       radeon_emit(cs, ((base_reg + loc->sgpr_idx * 4) - SI_SH_REG_OFFSET) >> 2);
+       radeon_emit(cs, ((base_reg + (loc->sgpr_idx + 1) * 4) - SI_SH_REG_OFFSET) >> 2);
+       radeon_emit(cs, (((base_reg + (loc->sgpr_idx + 2) * 4) - SI_SH_REG_OFFSET) >> 2) |
+                       S_2C3_DRAW_INDEX_ENABLE(1) |
+                       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);
 }
 
 static void
@@ -1912,9 +2424,10 @@ radv_cmd_draw_indirect_count(VkCommandBuffer                             command
                              uint32_t                                    stride)
 {
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
-       radv_cmd_buffer_flush_state(cmd_buffer);
+       radv_cmd_buffer_flush_state(cmd_buffer, true, 0);
 
-       unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 14);
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
+                                                          cmd_buffer->cs, 14);
 
        radv_emit_indirect_draw(cmd_buffer, buffer, offset,
                                countBuffer, countBufferOffset, maxDrawCount, stride, false);
@@ -1936,13 +2449,13 @@ radv_cmd_draw_indexed_indirect_count(
        int index_size = cmd_buffer->state.index_type ? 4 : 2;
        uint32_t index_max_size = (cmd_buffer->state.index_buffer->size - cmd_buffer->state.index_offset) / index_size;
        uint64_t index_va;
-       radv_cmd_buffer_flush_state(cmd_buffer);
+       radv_cmd_buffer_flush_state(cmd_buffer, true, 0);
        radv_emit_primitive_reset_index(cmd_buffer);
 
        index_va = cmd_buffer->device->ws->buffer_get_va(cmd_buffer->state.index_buffer->bo);
        index_va += cmd_buffer->state.index_buffer->offset + cmd_buffer->state.index_offset;
 
-       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, 21);
 
        radeon_emit(cmd_buffer->cs, PKT3(PKT3_INDEX_TYPE, 0, 0));
        radeon_emit(cmd_buffer->cs, cmd_buffer->state.index_type);
@@ -2014,6 +2527,8 @@ static void
 radv_flush_compute_state(struct radv_cmd_buffer *cmd_buffer)
 {
        radv_emit_compute_pipeline(cmd_buffer);
+       radv_flush_descriptors(cmd_buffer, cmd_buffer->state.compute_pipeline,
+                              VK_SHADER_STAGE_COMPUTE_BIT);
        radv_flush_constants(cmd_buffer, cmd_buffer->state.compute_pipeline,
                             VK_SHADER_STAGE_COMPUTE_BIT);
        si_emit_cache_flush(cmd_buffer);
@@ -2029,12 +2544,18 @@ void radv_CmdDispatch(
 
        radv_flush_compute_state(cmd_buffer);
 
-       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, 10);
 
-       radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B900_COMPUTE_USER_DATA_0 + AC_USERDATA_CS_GRID_SIZE * 4, 3);
-       radeon_emit(cmd_buffer->cs, x);
-       radeon_emit(cmd_buffer->cs, y);
-       radeon_emit(cmd_buffer->cs, z);
+       struct ac_userdata_info *loc = radv_lookup_user_sgpr(cmd_buffer->state.compute_pipeline,
+                                                            MESA_SHADER_COMPUTE, AC_UD_CS_GRID_SIZE);
+       if (loc->sgpr_idx != -1) {
+               assert(!loc->indirect);
+               assert(loc->num_sgprs == 3);
+               radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B900_COMPUTE_USER_DATA_0 + loc->sgpr_idx * 4, 3);
+               radeon_emit(cmd_buffer->cs, x);
+               radeon_emit(cmd_buffer->cs, y);
+               radeon_emit(cmd_buffer->cs, z);
+       }
 
        radeon_emit(cmd_buffer->cs, PKT3(PKT3_DISPATCH_DIRECT, 3, 0) |
                    PKT3_SHADER_TYPE_S(1));
@@ -2044,6 +2565,7 @@ void radv_CmdDispatch(
        radeon_emit(cmd_buffer->cs, 1);
 
        assert(cmd_buffer->cs->cdw <= cdw_max);
+       radv_cmd_buffer_trace_emit(cmd_buffer);
 }
 
 void radv_CmdDispatchIndirect(
@@ -2060,30 +2582,42 @@ void radv_CmdDispatchIndirect(
 
        radv_flush_compute_state(cmd_buffer);
 
-       unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 25);
-
-       for (unsigned i = 0; i < 3; ++i) {
-               radeon_emit(cmd_buffer->cs, PKT3(PKT3_COPY_DATA, 4, 0));
-               radeon_emit(cmd_buffer->cs, COPY_DATA_SRC_SEL(COPY_DATA_MEM) |
-                                           COPY_DATA_DST_SEL(COPY_DATA_REG));
-               radeon_emit(cmd_buffer->cs, (va +  4 * i));
-               radeon_emit(cmd_buffer->cs, (va + 4 * i) >> 32);
-               radeon_emit(cmd_buffer->cs, ((R_00B900_COMPUTE_USER_DATA_0 + AC_USERDATA_CS_GRID_SIZE * 4) >> 2) + i);
-               radeon_emit(cmd_buffer->cs, 0);
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cmd_buffer->cs, 25);
+       struct ac_userdata_info *loc = radv_lookup_user_sgpr(cmd_buffer->state.compute_pipeline,
+                                                            MESA_SHADER_COMPUTE, AC_UD_CS_GRID_SIZE);
+       if (loc->sgpr_idx != -1) {
+               for (unsigned i = 0; i < 3; ++i) {
+                       radeon_emit(cmd_buffer->cs, PKT3(PKT3_COPY_DATA, 4, 0));
+                       radeon_emit(cmd_buffer->cs, COPY_DATA_SRC_SEL(COPY_DATA_MEM) |
+                                   COPY_DATA_DST_SEL(COPY_DATA_REG));
+                       radeon_emit(cmd_buffer->cs, (va +  4 * i));
+                       radeon_emit(cmd_buffer->cs, (va + 4 * i) >> 32);
+                       radeon_emit(cmd_buffer->cs, ((R_00B900_COMPUTE_USER_DATA_0 + loc->sgpr_idx * 4) >> 2) + i);
+                       radeon_emit(cmd_buffer->cs, 0);
+               }
        }
 
-       radeon_emit(cmd_buffer->cs, PKT3(PKT3_SET_BASE, 2, 0) |
-                                   PKT3_SHADER_TYPE_S(1));
-       radeon_emit(cmd_buffer->cs, 1);
-       radeon_emit(cmd_buffer->cs, va);
-       radeon_emit(cmd_buffer->cs, va >> 32);
+       if (radv_cmd_buffer_uses_mec(cmd_buffer)) {
+               radeon_emit(cmd_buffer->cs, PKT3(PKT3_DISPATCH_INDIRECT, 2, 0) |
+                                       PKT3_SHADER_TYPE_S(1));
+               radeon_emit(cmd_buffer->cs, va);
+               radeon_emit(cmd_buffer->cs, va >> 32);
+               radeon_emit(cmd_buffer->cs, 1);
+       } else {
+               radeon_emit(cmd_buffer->cs, PKT3(PKT3_SET_BASE, 2, 0) |
+                                       PKT3_SHADER_TYPE_S(1));
+               radeon_emit(cmd_buffer->cs, 1);
+               radeon_emit(cmd_buffer->cs, va);
+               radeon_emit(cmd_buffer->cs, va >> 32);
 
-       radeon_emit(cmd_buffer->cs, PKT3(PKT3_DISPATCH_INDIRECT, 1, 0) |
-                                   PKT3_SHADER_TYPE_S(1));
-       radeon_emit(cmd_buffer->cs, 0);
-       radeon_emit(cmd_buffer->cs, 1);
+               radeon_emit(cmd_buffer->cs, PKT3(PKT3_DISPATCH_INDIRECT, 1, 0) |
+                                       PKT3_SHADER_TYPE_S(1));
+               radeon_emit(cmd_buffer->cs, 0);
+               radeon_emit(cmd_buffer->cs, 1);
+       }
 
        assert(cmd_buffer->cs->cdw <= cdw_max);
+       radv_cmd_buffer_trace_emit(cmd_buffer);
 }
 
 void radv_unaligned_dispatch(
@@ -2107,7 +2641,7 @@ void radv_unaligned_dispatch(
 
        radv_flush_compute_state(cmd_buffer);
 
-       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, 15);
 
        radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B81C_COMPUTE_NUM_THREAD_X, 3);
        radeon_emit(cmd_buffer->cs,
@@ -2120,11 +2654,14 @@ void radv_unaligned_dispatch(
                    S_00B81C_NUM_THREAD_FULL(compute_shader->info.cs.block_size[2]) |
                    S_00B81C_NUM_THREAD_PARTIAL(remainder[2]));
 
-       radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B900_COMPUTE_USER_DATA_0 + AC_USERDATA_CS_GRID_SIZE * 4, 3);
-       radeon_emit(cmd_buffer->cs, blocks[0]);
-       radeon_emit(cmd_buffer->cs, blocks[1]);
-       radeon_emit(cmd_buffer->cs, blocks[2]);
-
+       struct ac_userdata_info *loc = radv_lookup_user_sgpr(cmd_buffer->state.compute_pipeline,
+                                                            MESA_SHADER_COMPUTE, AC_UD_CS_GRID_SIZE);
+       if (loc->sgpr_idx != -1) {
+               radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B900_COMPUTE_USER_DATA_0 + loc->sgpr_idx * 4, 3);
+               radeon_emit(cmd_buffer->cs, blocks[0]);
+               radeon_emit(cmd_buffer->cs, blocks[1]);
+               radeon_emit(cmd_buffer->cs, blocks[2]);
+       }
        radeon_emit(cmd_buffer->cs, PKT3(PKT3_DISPATCH_DIRECT, 3, 0) |
                    PKT3_SHADER_TYPE_S(1));
        radeon_emit(cmd_buffer->cs, blocks[0]);
@@ -2134,6 +2671,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);
 }
 
 void radv_CmdEndRenderPass(
@@ -2181,7 +2719,7 @@ static void radv_handle_depth_image_transition(struct radv_cmd_buffer *cmd_buffe
                                               struct radv_image *image,
                                               VkImageLayout src_layout,
                                               VkImageLayout dst_layout,
-                                              VkImageSubresourceRange range,
+                                              const VkImageSubresourceRange *range,
                                               VkImageAspectFlags pending_clears)
 {
        if (dst_layout == VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL &&
@@ -2202,12 +2740,12 @@ static void radv_handle_depth_image_transition(struct radv_cmd_buffer *cmd_buffe
                    !radv_layout_has_htile(image, dst_layout)) ||
                   (radv_layout_is_htile_compressed(image, src_layout) &&
                    !radv_layout_is_htile_compressed(image, dst_layout))) {
+               VkImageSubresourceRange local_range = *range;
+               local_range.aspectMask = VK_IMAGE_ASPECT_DEPTH_BIT;
+               local_range.baseMipLevel = 0;
+               local_range.levelCount = 1;
 
-               range.aspectMask = VK_IMAGE_ASPECT_DEPTH_BIT;
-               range.baseMipLevel = 0;
-               range.levelCount = 1;
-
-               radv_decompress_depth_image_inplace(cmd_buffer, image, &range);
+               radv_decompress_depth_image_inplace(cmd_buffer, image, &local_range);
        }
 }
 
@@ -2230,7 +2768,9 @@ static void radv_handle_cmask_image_transition(struct radv_cmd_buffer *cmd_buffe
                                               struct radv_image *image,
                                               VkImageLayout src_layout,
                                               VkImageLayout dst_layout,
-                                              VkImageSubresourceRange range,
+                                              unsigned src_queue_mask,
+                                              unsigned dst_queue_mask,
+                                              const VkImageSubresourceRange *range,
                                               VkImageAspectFlags pending_clears)
 {
        if (src_layout == VK_IMAGE_LAYOUT_UNDEFINED) {
@@ -2238,9 +2778,9 @@ static void radv_handle_cmask_image_transition(struct radv_cmd_buffer *cmd_buffe
                        radv_initialise_cmask(cmd_buffer, image, 0xccccccccu);
                else
                        radv_initialise_cmask(cmd_buffer, image, 0xffffffffu);
-       } else if (radv_layout_has_cmask(image, src_layout) &&
-                  !radv_layout_has_cmask(image, dst_layout)) {
-               radv_fast_clear_flush_image_inplace(cmd_buffer, image);
+       } else if (radv_layout_can_fast_clear(image, src_layout, src_queue_mask) &&
+                  !radv_layout_can_fast_clear(image, dst_layout, dst_queue_mask)) {
+               radv_fast_clear_flush_image_inplace(cmd_buffer, image, range);
        }
 }
 
@@ -2265,14 +2805,16 @@ static void radv_handle_dcc_image_transition(struct radv_cmd_buffer *cmd_buffer,
                                             struct radv_image *image,
                                             VkImageLayout src_layout,
                                             VkImageLayout dst_layout,
-                                            VkImageSubresourceRange range,
+                                            unsigned src_queue_mask,
+                                            unsigned dst_queue_mask,
+                                            const VkImageSubresourceRange *range,
                                             VkImageAspectFlags pending_clears)
 {
        if (src_layout == VK_IMAGE_LAYOUT_UNDEFINED) {
                radv_initialize_dcc(cmd_buffer, image, 0x20202020u);
-       } else if(src_layout == VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL &&
-                 dst_layout != VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL) {
-               radv_fast_clear_flush_image_inplace(cmd_buffer, image);
+       } else if (radv_layout_can_fast_clear(image, src_layout, src_queue_mask) &&
+                  !radv_layout_can_fast_clear(image, dst_layout, dst_queue_mask)) {
+               radv_fast_clear_flush_image_inplace(cmd_buffer, image, range);
        }
 }
 
@@ -2280,20 +2822,46 @@ static void radv_handle_image_transition(struct radv_cmd_buffer *cmd_buffer,
                                         struct radv_image *image,
                                         VkImageLayout src_layout,
                                         VkImageLayout dst_layout,
-                                        VkImageSubresourceRange range,
+                                        uint32_t src_family,
+                                        uint32_t dst_family,
+                                        const VkImageSubresourceRange *range,
                                         VkImageAspectFlags pending_clears)
 {
+       if (image->exclusive && src_family != dst_family) {
+               /* This is an acquire or a release operation and there will be
+                * a corresponding release/acquire. Do the transition in the
+                * most flexible queue. */
+
+               assert(src_family == cmd_buffer->queue_family_index ||
+                      dst_family == cmd_buffer->queue_family_index);
+
+               if (cmd_buffer->queue_family_index == RADV_QUEUE_TRANSFER)
+                       return;
+
+               if (cmd_buffer->queue_family_index == RADV_QUEUE_COMPUTE &&
+                   (src_family == RADV_QUEUE_GENERAL ||
+                    dst_family == RADV_QUEUE_GENERAL))
+                       return;
+       }
+
+       unsigned src_queue_mask = radv_image_queue_family_mask(image, src_family, cmd_buffer->queue_family_index);
+       unsigned dst_queue_mask = radv_image_queue_family_mask(image, dst_family, cmd_buffer->queue_family_index);
+
        if (image->htile.size)
                radv_handle_depth_image_transition(cmd_buffer, image, src_layout,
                                                   dst_layout, range, pending_clears);
 
        if (image->cmask.size)
                radv_handle_cmask_image_transition(cmd_buffer, image, src_layout,
-                                                  dst_layout, range, pending_clears);
+                                                  dst_layout, src_queue_mask,
+                                                  dst_queue_mask, range,
+                                                  pending_clears);
 
        if (image->surface.dcc_size)
                radv_handle_dcc_image_transition(cmd_buffer, image, src_layout,
-                                                dst_layout, range, pending_clears);
+                                                dst_layout, src_queue_mask,
+                                                dst_queue_mask, range,
+                                                pending_clears);
 }
 
 void radv_CmdPipelineBarrier(
@@ -2311,7 +2879,7 @@ void radv_CmdPipelineBarrier(
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
        VkAccessFlags src_flags = 0;
        VkAccessFlags dst_flags = 0;
-       uint32_t b;
+
        for (uint32_t i = 0; i < memoryBarrierCount; i++) {
                src_flags |= pMemoryBarriers[i].srcAccessMask;
                dst_flags |= pMemoryBarriers[i].dstAccessMask;
@@ -2327,59 +2895,23 @@ void radv_CmdPipelineBarrier(
                dst_flags |= pImageMemoryBarriers[i].dstAccessMask;
        }
 
-       enum radv_cmd_flush_bits flush_bits = 0;
-       for_each_bit(b, src_flags) {
-               switch ((VkAccessFlagBits)(1 << b)) {
-               case VK_ACCESS_SHADER_WRITE_BIT:
-                       flush_bits |= RADV_CMD_FLAG_INV_GLOBAL_L2;
-                       break;
-               case VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT:
-                       flush_bits |= RADV_CMD_FLAG_FLUSH_AND_INV_CB;
-                       break;
-               case VK_ACCESS_DEPTH_STENCIL_ATTACHMENT_WRITE_BIT:
-                       flush_bits |= RADV_CMD_FLAG_FLUSH_AND_INV_DB;
-                       break;
-               case VK_ACCESS_TRANSFER_WRITE_BIT:
-                       flush_bits |= RADV_CMD_FLAG_FLUSH_AND_INV_CB;
-                       break;
-               default:
-                       break;
-               }
-       }
-       cmd_buffer->state.flush_bits |= flush_bits;
+       radv_src_access_flush(cmd_buffer, src_flags);
 
        for (uint32_t i = 0; i < imageMemoryBarrierCount; i++) {
                RADV_FROM_HANDLE(radv_image, image, pImageMemoryBarriers[i].image);
                radv_handle_image_transition(cmd_buffer, image,
                                             pImageMemoryBarriers[i].oldLayout,
                                             pImageMemoryBarriers[i].newLayout,
-                                            pImageMemoryBarriers[i].subresourceRange,
+                                            pImageMemoryBarriers[i].srcQueueFamilyIndex,
+                                            pImageMemoryBarriers[i].dstQueueFamilyIndex,
+                                            &pImageMemoryBarriers[i].subresourceRange,
                                             0);
        }
 
-       flush_bits = 0;
-
-       for_each_bit(b, dst_flags) {
-               switch ((VkAccessFlagBits)(1 << b)) {
-               case VK_ACCESS_INDIRECT_COMMAND_READ_BIT:
-               case VK_ACCESS_INDEX_READ_BIT:
-               case VK_ACCESS_VERTEX_ATTRIBUTE_READ_BIT:
-               case VK_ACCESS_UNIFORM_READ_BIT:
-                       flush_bits |= RADV_CMD_FLAG_INV_VMEM_L1;
-                       break;
-               case VK_ACCESS_SHADER_READ_BIT:
-                       flush_bits |= RADV_CMD_FLAG_INV_GLOBAL_L2;
-                       break;
-               case VK_ACCESS_COLOR_ATTACHMENT_READ_BIT:
-               case VK_ACCESS_TRANSFER_READ_BIT:
-               case VK_ACCESS_INPUT_ATTACHMENT_READ_BIT:
-                       flush_bits |= RADV_CMD_FLUSH_AND_INV_FRAMEBUFFER | RADV_CMD_FLAG_INV_GLOBAL_L2;
-               default:
-                       break;
-               }
-       }
+       radv_dst_access_flush(cmd_buffer, dst_flags);
 
-       flush_bits |= RADV_CMD_FLAG_CS_PARTIAL_FLUSH |
+       /* TODO reduce this */
+       enum radv_cmd_flush_bits flush_bits = RADV_CMD_FLAG_CS_PARTIAL_FLUSH |
                RADV_CMD_FLAG_PS_PARTIAL_FLUSH;
 
        cmd_buffer->state.flush_bits |= flush_bits;
@@ -2396,12 +2928,12 @@ static void write_event(struct radv_cmd_buffer *cmd_buffer,
 
        cmd_buffer->device->ws->cs_add_buffer(cs, event->bo, 8);
 
-       unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cs, 12);
+       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cs, 12);
 
        /* TODO: this is overkill. Probably should figure something out from
         * the stage mask. */
 
-       if (cmd_buffer->device->instance->physicalDevice.rad_info.chip_class == CIK) {
+       if (cmd_buffer->device->physical_device->rad_info.chip_class == CIK) {
                radeon_emit(cs, PKT3(PKT3_EVENT_WRITE_EOP, 4, 0));
                radeon_emit(cs, EVENT_TYPE(EVENT_TYPE_BOTTOM_OF_PIPE_TS) |
                                EVENT_INDEX(5));
@@ -2463,7 +2995,7 @@ void radv_CmdWaitEvents(VkCommandBuffer commandBuffer,
 
                cmd_buffer->device->ws->cs_add_buffer(cs, event->bo, 8);
 
-               unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cs, 7);
+               MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cs, 7);
 
                radeon_emit(cs, PKT3(PKT3_WAIT_REG_MEM, 5, 0));
                radeon_emit(cs, WAIT_REG_MEM_EQUAL | WAIT_REG_MEM_MEM_SPACE(1));
@@ -2483,7 +3015,9 @@ void radv_CmdWaitEvents(VkCommandBuffer commandBuffer,
                radv_handle_image_transition(cmd_buffer, image,
                                             pImageMemoryBarriers[i].oldLayout,
                                             pImageMemoryBarriers[i].newLayout,
-                                            pImageMemoryBarriers[i].subresourceRange,
+                                            pImageMemoryBarriers[i].srcQueueFamilyIndex,
+                                            pImageMemoryBarriers[i].dstQueueFamilyIndex,
+                                            &pImageMemoryBarriers[i].subresourceRange,
                                             0);
        }