radv/amdgpu: Add some debug flags.
[mesa.git] / src / amd / vulkan / radv_cmd_buffer.c
index 38867b282ef2c5a6d19d3ca949ea9397028db292..dec3e9fd21dfaf273cc7e6e38dce7a60e827dfe5 100644 (file)
@@ -38,9 +38,9 @@ static void radv_handle_image_transition(struct radv_cmd_buffer *cmd_buffer,
                                         struct radv_image *image,
                                         VkImageLayout src_layout,
                                         VkImageLayout dst_layout,
-                                        int src_family,
-                                        int dst_family,
-                                        VkImageSubresourceRange range,
+                                        uint32_t src_family,
+                                        uint32_t dst_family,
+                                        const VkImageSubresourceRange *range,
                                         VkImageAspectFlags pending_clears);
 
 const struct radv_dynamic_state default_dynamic_state = {
@@ -187,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)
@@ -331,13 +376,15 @@ static unsigned radv_pack_float_12p4(float x)
 }
 
 static uint32_t
-shader_stage_to_user_data_0(gl_shader_stage stage)
+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 R_00B130_SPI_SHADER_USER_DATA_VS_0;
+               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:
@@ -360,7 +407,7 @@ radv_emit_userdata_address(struct radv_cmd_buffer *cmd_buffer,
                           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);
+       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);
@@ -514,6 +561,24 @@ radv_emit_hw_vs(struct radv_cmd_buffer *cmd_buffer,
                               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)
@@ -524,12 +589,101 @@ radv_emit_vertex_shader(struct radv_cmd_buffer *cmd_buffer,
 
        vs = pipeline->shaders[MESA_SHADER_VERTEX];
 
-       radv_emit_hw_vs(cmd_buffer, pipeline, vs);
+       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,
@@ -545,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);
 
@@ -565,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) |
@@ -576,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);
@@ -596,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) {
@@ -615,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);
@@ -635,6 +821,7 @@ 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,
@@ -1022,7 +1209,7 @@ emit_stage_descriptor_set_userdata(struct radv_cmd_buffer *cmd_buffer,
                                   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);
+       uint32_t base_reg = shader_stage_to_user_data_0(stage, radv_pipeline_has_gs(pipeline));
 
        if (desc_set_loc->sgpr_idx == -1)
                return;
@@ -1052,6 +1239,11 @@ radv_emit_descriptor_set_userdata(struct radv_cmd_buffer *cmd_buffer,
                                                   idx, set->va,
                                                   MESA_SHADER_VERTEX);
 
+       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,
@@ -1112,6 +1304,10 @@ radv_flush_constants(struct radv_cmd_buffer *cmd_buffer,
                radv_emit_userdata_address(cmd_buffer, pipeline, MESA_SHADER_FRAGMENT,
                                           AC_UD_PUSH_CONSTANTS, va);
 
+       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);
@@ -1120,7 +1316,8 @@ radv_flush_constants(struct radv_cmd_buffer *cmd_buffer,
 }
 
 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;
@@ -1184,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 (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_028AA8_IA_MULTI_VGT_PARAM, 1, ia_multi_vgt_param);
                        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);
@@ -1242,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,
@@ -1264,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, 0, 0, range,
+                                    att.layout, 0, 0, &range,
                                     cmd_buffer->state.attachments[idx].pending_clear_aspects);
 
        cmd_buffer->state.attachments[idx].current_layout = att.layout;
@@ -1368,8 +1634,22 @@ VkResult radv_AllocateCommandBuffers(
                        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;
        }
@@ -1381,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,
@@ -1408,31 +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);
-       }
-}
+               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);
 
-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;
-       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(
@@ -1444,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)
@@ -1457,26 +1717,11 @@ VkResult radv_BeginCommandBuffer(
        if (cmd_buffer->level == VK_COMMAND_BUFFER_LEVEL_PRIMARY) {
                switch (cmd_buffer->queue_family_index) {
                case RADV_QUEUE_GENERAL:
-                       /* 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->physical_device, cmd_buffer);
+                       emit_gfx_buffer_state(cmd_buffer);
                        radv_set_db_count_control(cmd_buffer);
-                       si_emit_cache_flush(cmd_buffer);
                        break;
                case RADV_QUEUE_COMPUTE:
-                       cmd_buffer->state.flush_bits = RADV_CMD_FLAG_INV_ICACHE |
-                               RADV_CMD_FLAG_CS_PARTIAL_FLUSH |
-                               RADV_CMD_FLAG_INV_VMEM_L1 |
-                               RADV_CMD_FLAG_INV_SMEM_L1 |
-                               RADV_CMD_FLAG_INV_GLOBAL_L2;
-                       si_init_compute(cmd_buffer->device->physical_device, cmd_buffer);
-                       si_emit_cache_flush(cmd_buffer);
+                       si_init_compute(cmd_buffer);
                        break;
                case RADV_QUEUE_TRANSFER:
                default:
@@ -1619,6 +1864,7 @@ VkResult radv_EndCommandBuffer(
 
        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;
@@ -1705,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");
@@ -1857,12 +2117,24 @@ void radv_CmdExecuteCommands(
                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;
        }
@@ -1888,6 +2160,7 @@ VkResult radv_CreateCommandPool(
                pool->alloc = device->alloc;
 
        list_inithead(&pool->cmd_buffers);
+       list_inithead(&pool->free_cmd_buffers);
 
        pool->queue_family_index = pCreateInfo->queueFamilyIndex;
 
@@ -1913,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);
 }
 
@@ -1936,6 +2214,15 @@ void radv_TrimCommandPoolKHR(
     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(
@@ -1987,16 +2274,19 @@ void radv_CmdDraw(
        uint32_t                                    firstInstance)
 {
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
-       radv_cmd_buffer_flush_state(cmd_buffer);
 
-       MAYBE_UNUSED 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);
 
        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) {
-               radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B130_SPI_SHADER_USER_DATA_VS_0 + loc->sgpr_idx * 4, 2);
+               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);
@@ -2036,10 +2326,10 @@ 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);
 
-       MAYBE_UNUSED 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);
@@ -2047,9 +2337,11 @@ void radv_CmdDrawIndexed(
        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) {
-               radeon_set_sh_reg_seq(cmd_buffer->cs, R_00B130_SPI_SHADER_USER_DATA_VS_0 + loc->sgpr_idx * 4, 2);
+               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);
@@ -2098,6 +2390,7 @@ radv_emit_indirect_draw(struct radv_cmd_buffer *cmd_buffer,
 
        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);
@@ -2108,9 +2401,11 @@ 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 + loc->sgpr_idx * 4) - SI_SH_REG_OFFSET) >> 2);
-       radeon_emit(cs, ((R_00B130_SPI_SHADER_USER_DATA_VS_0 + (loc->sgpr_idx + 1) * 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);
@@ -2129,7 +2424,7 @@ 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);
 
        MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
                                                           cmd_buffer->cs, 14);
@@ -2154,7 +2449,7 @@ 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);
@@ -2424,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 &&
@@ -2445,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);
        }
 }
 
@@ -2475,7 +2770,7 @@ static void radv_handle_cmask_image_transition(struct radv_cmd_buffer *cmd_buffe
                                               VkImageLayout dst_layout,
                                               unsigned src_queue_mask,
                                               unsigned dst_queue_mask,
-                                              VkImageSubresourceRange range,
+                                              const VkImageSubresourceRange *range,
                                               VkImageAspectFlags pending_clears)
 {
        if (src_layout == VK_IMAGE_LAYOUT_UNDEFINED) {
@@ -2485,7 +2780,7 @@ static void radv_handle_cmask_image_transition(struct radv_cmd_buffer *cmd_buffe
                        radv_initialise_cmask(cmd_buffer, image, 0xffffffffu);
        } 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);
+               radv_fast_clear_flush_image_inplace(cmd_buffer, image, range);
        }
 }
 
@@ -2512,14 +2807,14 @@ static void radv_handle_dcc_image_transition(struct radv_cmd_buffer *cmd_buffer,
                                             VkImageLayout dst_layout,
                                             unsigned src_queue_mask,
                                             unsigned dst_queue_mask,
-                                            VkImageSubresourceRange range,
+                                            const VkImageSubresourceRange *range,
                                             VkImageAspectFlags pending_clears)
 {
        if (src_layout == VK_IMAGE_LAYOUT_UNDEFINED) {
                radv_initialize_dcc(cmd_buffer, image, 0x20202020u);
        } 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);
+               radv_fast_clear_flush_image_inplace(cmd_buffer, image, range);
        }
 }
 
@@ -2527,9 +2822,9 @@ static void radv_handle_image_transition(struct radv_cmd_buffer *cmd_buffer,
                                         struct radv_image *image,
                                         VkImageLayout src_layout,
                                         VkImageLayout dst_layout,
-                                        int src_family,
-                                        int dst_family,
-                                        VkImageSubresourceRange range,
+                                        uint32_t src_family,
+                                        uint32_t dst_family,
+                                        const VkImageSubresourceRange *range,
                                         VkImageAspectFlags pending_clears)
 {
        if (image->exclusive && src_family != dst_family) {
@@ -2549,8 +2844,8 @@ static void radv_handle_image_transition(struct radv_cmd_buffer *cmd_buffer,
                        return;
        }
 
-       unsigned src_queue_mask = radv_image_queue_family_mask(image, src_family);
-       unsigned dst_queue_mask = radv_image_queue_family_mask(image, dst_family);
+       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,
@@ -2584,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;
@@ -2600,26 +2895,7 @@ 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);
@@ -2628,36 +2904,14 @@ void radv_CmdPipelineBarrier(
                                             pImageMemoryBarriers[i].newLayout,
                                             pImageMemoryBarriers[i].srcQueueFamilyIndex,
                                             pImageMemoryBarriers[i].dstQueueFamilyIndex,
-                                            pImageMemoryBarriers[i].subresourceRange,
+                                            &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:
-                       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;
-               }
-       }
+       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;
@@ -2763,7 +3017,7 @@ void radv_CmdWaitEvents(VkCommandBuffer commandBuffer,
                                             pImageMemoryBarriers[i].newLayout,
                                             pImageMemoryBarriers[i].srcQueueFamilyIndex,
                                             pImageMemoryBarriers[i].dstQueueFamilyIndex,
-                                            pImageMemoryBarriers[i].subresourceRange,
+                                            &pImageMemoryBarriers[i].subresourceRange,
                                             0);
        }