radv: fix getting the index type size for uint8_t
[mesa.git] / src / amd / vulkan / radv_cmd_buffer.c
index 3f4940e9c668682d9de0ca7152959d575a50bb4c..952f5e7d54c19354c7b79d97254f3e6a220b2e99 100644 (file)
@@ -54,7 +54,9 @@ enum {
 static void radv_handle_image_transition(struct radv_cmd_buffer *cmd_buffer,
                                         struct radv_image *image,
                                         VkImageLayout src_layout,
+                                        bool src_render_loop,
                                         VkImageLayout dst_layout,
+                                        bool dst_render_loop,
                                         uint32_t src_family,
                                         uint32_t dst_family,
                                         const VkImageSubresourceRange *range,
@@ -990,7 +992,6 @@ radv_emit_rbplus_state(struct radv_cmd_buffer *cmd_buffer)
                return;
 
        struct radv_pipeline *pipeline = cmd_buffer->state.pipeline;
-       struct radv_framebuffer *framebuffer = cmd_buffer->state.framebuffer;
        const struct radv_subpass *subpass = cmd_buffer->state.subpass;
 
        unsigned sx_ps_downconvert = 0;
@@ -1005,7 +1006,7 @@ radv_emit_rbplus_state(struct radv_cmd_buffer *cmd_buffer)
                }
 
                int idx = subpass->color_attachments[i].attachment;
-               struct radv_color_buffer_info *cb = &framebuffer->attachments[idx].cb;
+               struct radv_color_buffer_info *cb = &cmd_buffer->state.attachments[idx].cb;
 
                unsigned format = G_028C70_FORMAT(cb->cb_color_info);
                unsigned swap = G_028C70_COMP_SWAP(cb->cb_color_info);
@@ -1284,16 +1285,16 @@ radv_emit_depth_bias(struct radv_cmd_buffer *cmd_buffer)
 static void
 radv_emit_fb_color_state(struct radv_cmd_buffer *cmd_buffer,
                         int index,
-                        struct radv_attachment_info *att,
+                        struct radv_color_buffer_info *cb,
                         struct radv_image_view *iview,
-                        VkImageLayout layout)
+                        VkImageLayout layout,
+                        bool in_render_loop)
 {
        bool is_vi = cmd_buffer->device->physical_device->rad_info.chip_class >= GFX8;
-       struct radv_color_buffer_info *cb = &att->cb;
        uint32_t cb_color_info = cb->cb_color_info;
        struct radv_image *image = iview->image;
 
-       if (!radv_layout_dcc_compressed(image, layout,
+       if (!radv_layout_dcc_compressed(cmd_buffer->device, image, layout, in_render_loop,
                                        radv_image_queue_family_mask(image,
                                                                     cmd_buffer->queue_family_index,
                                                                     cmd_buffer->queue_family_index))) {
@@ -1395,7 +1396,7 @@ static void
 radv_update_zrange_precision(struct radv_cmd_buffer *cmd_buffer,
                             struct radv_ds_buffer_info *ds,
                             struct radv_image *image, VkImageLayout layout,
-                            bool requires_cond_exec)
+                            bool in_render_loop, bool requires_cond_exec)
 {
        uint32_t db_z_info = ds->db_z_info;
        uint32_t db_z_info_reg;
@@ -1404,7 +1405,7 @@ radv_update_zrange_precision(struct radv_cmd_buffer *cmd_buffer,
            !radv_image_is_tc_compat_htile(image))
                return;
 
-       if (!radv_layout_has_htile(image, layout,
+       if (!radv_layout_has_htile(image, layout, in_render_loop,
                                   radv_image_queue_family_mask(image,
                                                                cmd_buffer->queue_family_index,
                                                                cmd_buffer->queue_family_index))) {
@@ -1441,12 +1442,13 @@ static void
 radv_emit_fb_ds_state(struct radv_cmd_buffer *cmd_buffer,
                      struct radv_ds_buffer_info *ds,
                      struct radv_image *image,
-                     VkImageLayout layout)
+                     VkImageLayout layout,
+                     bool in_render_loop)
 {
        uint32_t db_z_info = ds->db_z_info;
        uint32_t db_stencil_info = ds->db_stencil_info;
 
-       if (!radv_layout_has_htile(image, layout,
+       if (!radv_layout_has_htile(image, layout, in_render_loop,
                                   radv_image_queue_family_mask(image,
                                                                cmd_buffer->queue_family_index,
                                                                cmd_buffer->queue_family_index))) {
@@ -1514,7 +1516,7 @@ radv_emit_fb_ds_state(struct radv_cmd_buffer *cmd_buffer,
        }
 
        /* Update the ZRANGE_PRECISION value for the TC-compat bug. */
-       radv_update_zrange_precision(cmd_buffer, ds, image, layout, true);
+       radv_update_zrange_precision(cmd_buffer, ds, image, layout, in_render_loop, true);
 
        radeon_set_context_reg(cmd_buffer->cs, R_028B78_PA_SU_POLY_OFFSET_DB_FMT_CNTL,
                               ds->pa_su_poly_offset_db_fmt_cntl);
@@ -1530,21 +1532,18 @@ radv_update_bound_fast_clear_ds(struct radv_cmd_buffer *cmd_buffer,
                                VkClearDepthStencilValue ds_clear_value,
                                VkImageAspectFlags aspects)
 {
-       struct radv_framebuffer *framebuffer = cmd_buffer->state.framebuffer;
        const struct radv_subpass *subpass = cmd_buffer->state.subpass;
        struct radeon_cmdbuf *cs = cmd_buffer->cs;
-       struct radv_attachment_info *att;
        uint32_t att_idx;
 
-       if (!framebuffer || !subpass)
+       if (!cmd_buffer->state.attachments || !subpass)
                return;
 
        if (!subpass->depth_stencil_attachment)
                return;
 
        att_idx = subpass->depth_stencil_attachment->attachment;
-       att = &framebuffer->attachments[att_idx];
-       if (att->attachment->image != image)
+       if (cmd_buffer->state.attachments[att_idx].iview->image != image)
                return;
 
        radeon_set_context_reg_seq(cs, R_028028_DB_STENCIL_CLEAR, 2);
@@ -1557,9 +1556,10 @@ radv_update_bound_fast_clear_ds(struct radv_cmd_buffer *cmd_buffer,
        if ((aspects & VK_IMAGE_ASPECT_DEPTH_BIT) &&
            ds_clear_value.depth == 0.0) {
                VkImageLayout layout = subpass->depth_stencil_attachment->layout;
+               bool in_render_loop = subpass->depth_stencil_attachment->in_render_loop;
 
-               radv_update_zrange_precision(cmd_buffer, &att->ds, image,
-                                            layout, false);
+               radv_update_zrange_precision(cmd_buffer, &cmd_buffer->state.attachments[att_idx].ds, image,
+                                            layout, in_render_loop, false);
        }
 
        cmd_buffer->state.context_roll_without_scissor_emitted = true;
@@ -1780,21 +1780,18 @@ radv_update_bound_fast_clear_color(struct radv_cmd_buffer *cmd_buffer,
                                   int cb_idx,
                                   uint32_t color_values[2])
 {
-       struct radv_framebuffer *framebuffer = cmd_buffer->state.framebuffer;
        const struct radv_subpass *subpass = cmd_buffer->state.subpass;
        struct radeon_cmdbuf *cs = cmd_buffer->cs;
-       struct radv_attachment_info *att;
        uint32_t att_idx;
 
-       if (!framebuffer || !subpass)
+       if (!cmd_buffer->state.attachments || !subpass)
                return;
 
        att_idx = subpass->color_attachments[cb_idx].attachment;
        if (att_idx == VK_ATTACHMENT_UNUSED)
                return;
 
-       att = &framebuffer->attachments[att_idx];
-       if (att->attachment->image != image)
+       if (cmd_buffer->state.attachments[att_idx].iview->image != image)
                return;
 
        radeon_set_context_reg_seq(cs, R_028C8C_CB_COLOR0_CLEAR_WORD0 + cb_idx * 0x3c, 2);
@@ -1919,15 +1916,15 @@ radv_emit_framebuffer_state(struct radv_cmd_buffer *cmd_buffer)
                }
 
                int idx = subpass->color_attachments[i].attachment;
-               struct radv_attachment_info *att = &framebuffer->attachments[idx];
-               struct radv_image_view *iview = att->attachment;
+               struct radv_image_view *iview = cmd_buffer->state.attachments[idx].iview;
                VkImageLayout layout = subpass->color_attachments[i].layout;
+               bool in_render_loop = subpass->color_attachments[i].in_render_loop;
 
-               radv_cs_add_buffer(cmd_buffer->device->ws, cmd_buffer->cs, att->attachment->bo);
+               radv_cs_add_buffer(cmd_buffer->device->ws, cmd_buffer->cs, iview->bo);
 
-               assert(att->attachment->aspect_mask & (VK_IMAGE_ASPECT_COLOR_BIT | VK_IMAGE_ASPECT_PLANE_0_BIT |
+               assert(iview->aspect_mask & (VK_IMAGE_ASPECT_COLOR_BIT | VK_IMAGE_ASPECT_PLANE_0_BIT |
                                                       VK_IMAGE_ASPECT_PLANE_1_BIT | VK_IMAGE_ASPECT_PLANE_2_BIT));
-               radv_emit_fb_color_state(cmd_buffer, i, att, iview, layout);
+               radv_emit_fb_color_state(cmd_buffer, i, &cmd_buffer->state.attachments[idx].cb, iview, layout, in_render_loop);
 
                radv_load_color_clear_metadata(cmd_buffer, iview, i);
        }
@@ -1935,21 +1932,21 @@ radv_emit_framebuffer_state(struct radv_cmd_buffer *cmd_buffer)
        if (subpass->depth_stencil_attachment) {
                int idx = subpass->depth_stencil_attachment->attachment;
                VkImageLayout layout = subpass->depth_stencil_attachment->layout;
-               struct radv_attachment_info *att = &framebuffer->attachments[idx];
-               struct radv_image *image = att->attachment->image;
-               radv_cs_add_buffer(cmd_buffer->device->ws, cmd_buffer->cs, att->attachment->bo);
-               MAYBE_UNUSED uint32_t queue_mask = radv_image_queue_family_mask(image,
+               bool in_render_loop = subpass->depth_stencil_attachment->in_render_loop;
+               struct radv_image *image = cmd_buffer->state.attachments[idx].iview->image;
+               radv_cs_add_buffer(cmd_buffer->device->ws, cmd_buffer->cs, cmd_buffer->state.attachments[idx].iview->bo);
+               ASSERTED uint32_t queue_mask = radv_image_queue_family_mask(image,
                                                                                cmd_buffer->queue_family_index,
                                                                                cmd_buffer->queue_family_index);
                /* We currently don't support writing decompressed HTILE */
-               assert(radv_layout_has_htile(image, layout, queue_mask) ==
-                      radv_layout_is_htile_compressed(image, layout, queue_mask));
+               assert(radv_layout_has_htile(image, layout, in_render_loop, queue_mask) ==
+                      radv_layout_is_htile_compressed(image, layout, in_render_loop, queue_mask));
 
-               radv_emit_fb_ds_state(cmd_buffer, &att->ds, image, layout);
+               radv_emit_fb_ds_state(cmd_buffer, &cmd_buffer->state.attachments[idx].ds, image, layout, in_render_loop);
 
-               if (att->ds.offset_scale != cmd_buffer->state.offset_scale) {
+               if (cmd_buffer->state.attachments[idx].ds.offset_scale != cmd_buffer->state.offset_scale) {
                        cmd_buffer->state.dirty |= RADV_CMD_DIRTY_DYNAMIC_DEPTH_BIAS;
-                       cmd_buffer->state.offset_scale = att->ds.offset_scale;
+                       cmd_buffer->state.offset_scale = cmd_buffer->state.attachments[idx].ds.offset_scale;
                }
                radv_load_ds_clear_metadata(cmd_buffer, image);
        } else {
@@ -2214,7 +2211,7 @@ radv_flush_descriptors(struct radv_cmd_buffer *cmd_buffer,
        if (flush_indirect_descriptors)
                radv_flush_indirect_descriptor_sets(cmd_buffer, bind_point);
 
-       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
+       ASSERTED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
                                                           cmd_buffer->cs,
                                                           MAX_SETS * MESA_SHADER_STAGES * 4);
 
@@ -2300,7 +2297,7 @@ radv_flush_constants(struct radv_cmd_buffer *cmd_buffer,
                va = radv_buffer_get_va(cmd_buffer->upload.upload_bo);
                va += offset;
 
-               MAYBE_UNUSED unsigned cdw_max =
+               ASSERTED unsigned cdw_max =
                        radeon_check_space(cmd_buffer->device->ws,
                                           cmd_buffer->cs, MESA_SHADER_STAGES * 4);
 
@@ -2461,8 +2458,15 @@ radv_flush_streamout_descriptors(struct radv_cmd_buffer *cmd_buffer)
                        desc[3] = S_008F0C_DST_SEL_X(V_008F0C_SQ_SEL_X) |
                                  S_008F0C_DST_SEL_Y(V_008F0C_SQ_SEL_Y) |
                                  S_008F0C_DST_SEL_Z(V_008F0C_SQ_SEL_Z) |
-                                 S_008F0C_DST_SEL_W(V_008F0C_SQ_SEL_W) |
-                                 S_008F0C_DATA_FORMAT(V_008F0C_BUF_DATA_FORMAT_32);
+                                 S_008F0C_DST_SEL_W(V_008F0C_SQ_SEL_W);
+
+                       if (cmd_buffer->device->physical_device->rad_info.chip_class >= GFX10) {
+                               desc[3] |= S_008F0C_FORMAT(V_008F0C_IMG_FORMAT_32_FLOAT) |
+                                          S_008F0C_OOB_SELECT(3) |
+                                          S_008F0C_RESOURCE_LEVEL(1);
+                       } else {
+                               desc[3] |= S_008F0C_DATA_FORMAT(V_008F0C_BUF_DATA_FORMAT_32);
+                       }
                }
 
                va = radv_buffer_get_va(cmd_buffer->upload.upload_bo);
@@ -2534,6 +2538,21 @@ struct radv_draw_info {
        uint64_t strmout_buffer_offset;
 };
 
+static uint32_t
+radv_get_primitive_reset_index(struct radv_cmd_buffer *cmd_buffer)
+{
+       switch (cmd_buffer->state.index_type) {
+       case V_028A7C_VGT_INDEX_8:
+               return 0xffu;
+       case V_028A7C_VGT_INDEX_16:
+               return 0xffffu;
+       case V_028A7C_VGT_INDEX_32:
+               return 0xffffffffu;
+       default:
+               unreachable("invalid index type");
+       }
+}
+
 static void
 si_emit_ia_multi_vgt_param(struct radv_cmd_buffer *cmd_buffer,
                           bool instanced_draw, bool indirect_draw,
@@ -2605,7 +2624,7 @@ radv_emit_draw_registers(struct radv_cmd_buffer *cmd_buffer,
 
        if (primitive_reset_en) {
                uint32_t primitive_reset_index =
-                       state->index_type ? 0xffffffffu : 0xffffu;
+                       radv_get_primitive_reset_index(cmd_buffer);
 
                if (primitive_reset_index != state->last_primitive_reset_index) {
                        radeon_set_context_reg(cs,
@@ -2825,7 +2844,7 @@ radv_get_attachment_sample_locations(struct radv_cmd_buffer *cmd_buffer,
 {
        struct radv_cmd_state *state = &cmd_buffer->state;
        uint32_t subpass_id = radv_get_subpass_id(cmd_buffer);
-       struct radv_image_view *view = state->framebuffer->attachments[att_idx].attachment;
+       struct radv_image_view *view = state->attachments[att_idx].iview;
 
        if (view->image->info.samples == 1)
                return NULL;
@@ -2864,7 +2883,7 @@ static void radv_handle_subpass_image_transition(struct radv_cmd_buffer *cmd_buf
                                                 bool begin_subpass)
 {
        unsigned idx = att.attachment;
-       struct radv_image_view *view = cmd_buffer->state.framebuffer->attachments[idx].attachment;
+       struct radv_image_view *view = cmd_buffer->state.attachments[idx].iview;
        struct radv_sample_locations_state *sample_locs;
        VkImageSubresourceRange range;
        range.aspectMask = 0;
@@ -2893,9 +2912,12 @@ 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, sample_locs);
+                                    cmd_buffer->state.attachments[idx].current_in_render_loop,
+                                    att.layout, att.in_render_loop,
+                                    0, 0, &range, sample_locs);
 
        cmd_buffer->state.attachments[idx].current_layout = att.layout;
+       cmd_buffer->state.attachments[idx].current_in_render_loop = att.in_render_loop;
 
 
 }
@@ -2918,7 +2940,6 @@ radv_cmd_state_setup_sample_locations(struct radv_cmd_buffer *cmd_buffer,
                vk_find_struct_const(info->pNext,
                                     RENDER_PASS_SAMPLE_LOCATIONS_BEGIN_INFO_EXT);
        struct radv_cmd_state *state = &cmd_buffer->state;
-       struct radv_framebuffer *framebuffer = state->framebuffer;
 
        if (!sample_locs) {
                state->subpass_sample_locs = NULL;
@@ -2929,8 +2950,7 @@ radv_cmd_state_setup_sample_locations(struct radv_cmd_buffer *cmd_buffer,
                const VkAttachmentSampleLocationsEXT *att_sample_locs =
                        &sample_locs->pAttachmentInitialSampleLocations[i];
                uint32_t att_idx = att_sample_locs->attachmentIndex;
-               struct radv_attachment_info *att = &framebuffer->attachments[att_idx];
-               struct radv_image *image = att->attachment->image;
+               struct radv_image *image = cmd_buffer->state.attachments[att_idx].iview->image;
 
                assert(vk_format_is_depth_or_stencil(image->vk_format));
 
@@ -2998,6 +3018,13 @@ radv_cmd_state_setup_attachments(struct radv_cmd_buffer *cmd_buffer,
                                 const VkRenderPassBeginInfo *info)
 {
        struct radv_cmd_state *state = &cmd_buffer->state;
+       const struct VkRenderPassAttachmentBeginInfoKHR *attachment_info = NULL;
+
+       if (info) {
+               attachment_info = vk_find_struct_const(info->pNext,
+                                                      RENDER_PASS_ATTACHMENT_BEGIN_INFO_KHR);
+       }
+
 
        if (pass->attachment_count == 0) {
                state->attachments = NULL;
@@ -3047,6 +3074,20 @@ radv_cmd_state_setup_attachments(struct radv_cmd_buffer *cmd_buffer,
 
                state->attachments[i].current_layout = att->initial_layout;
                state->attachments[i].sample_location.count = 0;
+
+               struct radv_image_view *iview;
+               if (attachment_info && attachment_info->attachmentCount > i) {
+                       iview = radv_image_view_from_handle(attachment_info->pAttachments[i]);
+               } else {
+                       iview = state->framebuffer->attachments[i];
+               }
+
+               state->attachments[i].iview = iview;
+               if (iview->aspect_mask & (VK_IMAGE_ASPECT_DEPTH_BIT | VK_IMAGE_ASPECT_STENCIL_BIT)) {
+                       radv_initialise_ds_surface(cmd_buffer->device, &state->attachments[i].ds, iview);
+               } else {
+                       radv_initialise_color_surface(cmd_buffer->device, &state->attachments[i].cb, iview);
+               }
        }
 
        return VK_SUCCESS;
@@ -3166,9 +3207,11 @@ VkResult radv_BeginCommandBuffer(
                struct radv_subpass *subpass =
                        &cmd_buffer->state.pass->subpasses[pBeginInfo->pInheritanceInfo->subpass];
 
-               result = radv_cmd_state_setup_attachments(cmd_buffer, cmd_buffer->state.pass, NULL);
-               if (result != VK_SUCCESS)
-                       return result;
+               if (cmd_buffer->state.framebuffer) {
+                       result = radv_cmd_state_setup_attachments(cmd_buffer, cmd_buffer->state.pass, NULL);
+                       if (result != VK_SUCCESS)
+                               return result;
+               }
 
                radv_cmd_buffer_set_subpass(cmd_buffer, subpass);
        }
@@ -3226,6 +3269,36 @@ void radv_CmdBindVertexBuffers(
        cmd_buffer->state.dirty |= RADV_CMD_DIRTY_VERTEX_BUFFER;
 }
 
+static uint32_t
+vk_to_index_type(VkIndexType type)
+{
+       switch (type) {
+       case VK_INDEX_TYPE_UINT8_EXT:
+               return V_028A7C_VGT_INDEX_8;
+       case VK_INDEX_TYPE_UINT16:
+               return V_028A7C_VGT_INDEX_16;
+       case VK_INDEX_TYPE_UINT32:
+               return V_028A7C_VGT_INDEX_32;
+       default:
+               unreachable("invalid index type");
+       }
+}
+
+static uint32_t
+radv_get_vgt_index_size(uint32_t type)
+{
+       switch (type) {
+       case V_028A7C_VGT_INDEX_8:
+               return 1;
+       case V_028A7C_VGT_INDEX_16:
+               return 2;
+       case V_028A7C_VGT_INDEX_32:
+               return 4;
+       default:
+               unreachable("invalid index type");
+       }
+}
+
 void radv_CmdBindIndexBuffer(
        VkCommandBuffer                             commandBuffer,
        VkBuffer buffer,
@@ -3244,12 +3317,12 @@ void radv_CmdBindIndexBuffer(
 
        cmd_buffer->state.index_buffer = index_buffer;
        cmd_buffer->state.index_offset = offset;
-       cmd_buffer->state.index_type = indexType; /* vk matches hw */
+       cmd_buffer->state.index_type = vk_to_index_type(indexType);
        cmd_buffer->state.index_va = radv_buffer_get_va(index_buffer->bo);
        cmd_buffer->state.index_va += index_buffer->offset + offset;
 
-       int index_size_shift = cmd_buffer->state.index_type ? 2 : 1;
-       cmd_buffer->state.max_index_count = (index_buffer->size - offset) >> index_size_shift;
+       int index_size = radv_get_vgt_index_size(vk_to_index_type(indexType));
+       cmd_buffer->state.max_index_count = (index_buffer->size - offset) / index_size;
        cmd_buffer->state.dirty |= RADV_CMD_DIRTY_INDEX_BUFFER;
        radv_cs_add_buffer(cmd_buffer->device->ws, cmd_buffer->cs, index_buffer->bo);
 }
@@ -3298,7 +3371,13 @@ void radv_CmdBindDescriptorSets(
        for (unsigned i = 0; i < descriptorSetCount; ++i) {
                unsigned idx = i + firstSet;
                RADV_FROM_HANDLE(radv_descriptor_set, set, pDescriptorSets[i]);
-               radv_bind_descriptor_set(cmd_buffer, pipelineBindPoint, set, idx);
+
+               /* If the set is already bound we only need to update the
+                * (potentially changed) dynamic offsets. */
+               if (descriptors_state->sets[idx] != set ||
+                   !(descriptors_state->valid & (1u << idx))) {
+                       radv_bind_descriptor_set(cmd_buffer, pipelineBindPoint, set, idx);
+               }
 
                for(unsigned j = 0; j < set->layout->dynamic_offset_count; ++j, ++dyn_idx) {
                        unsigned idx = j + layout->set[i + firstSet].dynamic_offset_start;
@@ -3418,7 +3497,7 @@ void radv_CmdPushDescriptorSetKHR(
         * because it is invalid, according to Vulkan spec.
         */
        for (int i = 0; i < descriptorWriteCount; i++) {
-               MAYBE_UNUSED const VkWriteDescriptorSet *writeset = &pDescriptorWrites[i];
+               ASSERTED const VkWriteDescriptorSet *writeset = &pDescriptorWrites[i];
                assert(writeset->descriptorType != VK_DESCRIPTOR_TYPE_INLINE_UNIFORM_BLOCK_EXT);
        }
 
@@ -3574,6 +3653,20 @@ void radv_CmdBindPipeline(
                /* Prefetch all pipeline shaders at first draw time. */
                cmd_buffer->state.prefetch_L2_mask |= RADV_PREFETCH_SHADERS;
 
+               if ((cmd_buffer->device->physical_device->rad_info.family == CHIP_NAVI10 ||
+                    cmd_buffer->device->physical_device->rad_info.family == CHIP_NAVI12 ||
+                    cmd_buffer->device->physical_device->rad_info.family == CHIP_NAVI14) &&
+                   cmd_buffer->state.emitted_pipeline &&
+                   radv_pipeline_has_ngg(cmd_buffer->state.emitted_pipeline) &&
+                   !radv_pipeline_has_ngg(cmd_buffer->state.pipeline)) {
+                       /* Transitioning from NGG to legacy GS requires
+                        * VGT_FLUSH on Navi10-14.  VGT_FLUSH is also emitted
+                        * at the beginning of IBs when legacy GS ring pointers
+                        * are set.
+                        */
+                       cmd_buffer->state.flush_bits |= RADV_CMD_FLAG_VGT_FLUSH;
+               }
+
                radv_bind_dynamic_state(cmd_buffer, &pipeline->dynamic_state);
                radv_bind_streamout_state(cmd_buffer, pipeline);
 
@@ -3599,7 +3692,7 @@ void radv_CmdSetViewport(
 {
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
        struct radv_cmd_state *state = &cmd_buffer->state;
-       MAYBE_UNUSED const uint32_t total_count = firstViewport + viewportCount;
+       ASSERTED const uint32_t total_count = firstViewport + viewportCount;
 
        assert(firstViewport < MAX_VIEWPORTS);
        assert(total_count >= 1 && total_count <= MAX_VIEWPORTS);
@@ -3623,7 +3716,7 @@ void radv_CmdSetScissor(
 {
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
        struct radv_cmd_state *state = &cmd_buffer->state;
-       MAYBE_UNUSED const uint32_t total_count = firstScissor + scissorCount;
+       ASSERTED const uint32_t total_count = firstScissor + scissorCount;
 
        assert(firstScissor < MAX_SCISSORS);
        assert(total_count >= 1 && total_count <= MAX_SCISSORS);
@@ -3785,7 +3878,7 @@ void radv_CmdSetDiscardRectangleEXT(
 {
        RADV_FROM_HANDLE(radv_cmd_buffer, cmd_buffer, commandBuffer);
        struct radv_cmd_state *state = &cmd_buffer->state;
-       MAYBE_UNUSED const uint32_t total_count = firstDiscardRectangle + discardRectangleCount;
+       ASSERTED const uint32_t total_count = firstDiscardRectangle + discardRectangleCount;
 
        assert(firstDiscardRectangle < MAX_DISCARD_RECTANGLES);
        assert(total_count >= 1 && total_count <= MAX_DISCARD_RECTANGLES);
@@ -4008,7 +4101,7 @@ radv_cmd_buffer_begin_subpass(struct radv_cmd_buffer *cmd_buffer,
        struct radv_cmd_state *state = &cmd_buffer->state;
        struct radv_subpass *subpass = &state->pass->subpasses[subpass_id];
 
-       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
+       ASSERTED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws,
                                                           cmd_buffer->cs, 4096);
 
        radv_subpass_barrier(cmd_buffer, &subpass->start_barrier);
@@ -4268,9 +4361,15 @@ radv_emit_draw_packets(struct radv_cmd_buffer *cmd_buffer,
                }
 
                if (info->indexed) {
-                       int index_size = state->index_type ? 4 : 2;
+                       int index_size = radv_get_vgt_index_size(state->index_type);
                        uint64_t index_va;
 
+                       /* Skip draw calls with 0-sized index buffers. They
+                        * cause a hang on some chips, like Navi10-14.
+                        */
+                       if (!cmd_buffer->state.max_index_count)
+                               return;
+
                        index_va = state->index_va;
                        index_va += info->first_index * index_size;
 
@@ -4347,8 +4446,11 @@ static bool radv_need_late_scissor_emission(struct radv_cmd_buffer *cmd_buffer,
        if (cmd_buffer->state.dirty & used_states)
                return true;
 
+       uint32_t primitive_reset_index =
+               radv_get_primitive_reset_index(cmd_buffer);
+
        if (info->indexed && state->pipeline->graphics.prim_restart_enable &&
-           (state->index_type ? 0xffffffffu : 0xffffu) != state->last_primitive_reset_index)
+           primitive_reset_index != state->last_primitive_reset_index)
                return true;
 
        return false;
@@ -4410,7 +4512,7 @@ radv_draw(struct radv_cmd_buffer *cmd_buffer,
                (cmd_buffer->state.dirty & RADV_CMD_DIRTY_PIPELINE) &&
                cmd_buffer->state.pipeline != cmd_buffer->state.emitted_pipeline;
 
-       MAYBE_UNUSED unsigned cdw_max =
+       ASSERTED unsigned cdw_max =
                radeon_check_space(cmd_buffer->device->ws,
                                   cmd_buffer->cs, 4096);
 
@@ -4665,7 +4767,7 @@ radv_emit_dispatch_packets(struct radv_cmd_buffer *cmd_buffer,
        loc = radv_lookup_user_sgpr(pipeline, MESA_SHADER_COMPUTE,
                                    AC_UD_CS_GRID_SIZE);
 
-       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(ws, cs, 25);
+       ASSERTED unsigned cdw_max = radeon_check_space(ws, cs, 25);
 
        if (info->indirect) {
                uint64_t va = radv_buffer_get_va(info->indirect->bo);
@@ -4976,7 +5078,9 @@ static void radv_initialize_htile(struct radv_cmd_buffer *cmd_buffer,
 static void radv_handle_depth_image_transition(struct radv_cmd_buffer *cmd_buffer,
                                               struct radv_image *image,
                                               VkImageLayout src_layout,
+                                              bool src_render_loop,
                                               VkImageLayout dst_layout,
+                                              bool dst_render_loop,
                                               unsigned src_queue_mask,
                                               unsigned dst_queue_mask,
                                               const VkImageSubresourceRange *range,
@@ -4988,18 +5092,18 @@ static void radv_handle_depth_image_transition(struct radv_cmd_buffer *cmd_buffe
        if (src_layout == VK_IMAGE_LAYOUT_UNDEFINED) {
                uint32_t clear_value = vk_format_is_stencil(image->vk_format) ? 0xfffff30f : 0xfffc000f;
 
-               if (radv_layout_is_htile_compressed(image, dst_layout,
+               if (radv_layout_is_htile_compressed(image, dst_layout, dst_render_loop,
                                                    dst_queue_mask)) {
                        clear_value = 0;
                }
 
                radv_initialize_htile(cmd_buffer, image, range, clear_value);
-       } else if (!radv_layout_is_htile_compressed(image, src_layout, src_queue_mask) &&
-                  radv_layout_is_htile_compressed(image, dst_layout, dst_queue_mask)) {
+       } else if (!radv_layout_is_htile_compressed(image, src_layout, src_render_loop, src_queue_mask) &&
+                  radv_layout_is_htile_compressed(image, dst_layout, dst_render_loop, dst_queue_mask)) {
                uint32_t clear_value = vk_format_is_stencil(image->vk_format) ? 0xfffff30f : 0xfffc000f;
                radv_initialize_htile(cmd_buffer, image, range, clear_value);
-       } else if (radv_layout_is_htile_compressed(image, src_layout, src_queue_mask) &&
-                  !radv_layout_is_htile_compressed(image, dst_layout, dst_queue_mask)) {
+       } else if (radv_layout_is_htile_compressed(image, src_layout, src_render_loop, src_queue_mask) &&
+                  !radv_layout_is_htile_compressed(image, dst_layout, dst_render_loop, dst_queue_mask)) {
                VkImageSubresourceRange local_range = *range;
                local_range.aspectMask = VK_IMAGE_ASPECT_DEPTH_BIT;
                local_range.baseMipLevel = 0;
@@ -5103,7 +5207,9 @@ void radv_initialize_dcc(struct radv_cmd_buffer *cmd_buffer,
 static void radv_init_color_image_metadata(struct radv_cmd_buffer *cmd_buffer,
                                           struct radv_image *image,
                                           VkImageLayout src_layout,
+                                          bool src_render_loop,
                                           VkImageLayout dst_layout,
+                                          bool dst_render_loop,
                                           unsigned src_queue_mask,
                                           unsigned dst_queue_mask,
                                           const VkImageSubresourceRange *range)
@@ -5127,7 +5233,8 @@ static void radv_init_color_image_metadata(struct radv_cmd_buffer *cmd_buffer,
                uint32_t value = 0xffffffffu; /* Fully expanded mode. */
                bool need_decompress_pass = false;
 
-               if (radv_layout_dcc_compressed(image, dst_layout,
+               if (radv_layout_dcc_compressed(cmd_buffer->device, image, dst_layout,
+                                              dst_render_loop,
                                               dst_queue_mask)) {
                        value = 0x20202020u;
                        need_decompress_pass = true;
@@ -5153,14 +5260,17 @@ static void radv_init_color_image_metadata(struct radv_cmd_buffer *cmd_buffer,
 static void radv_handle_color_image_transition(struct radv_cmd_buffer *cmd_buffer,
                                               struct radv_image *image,
                                               VkImageLayout src_layout,
+                                              bool src_render_loop,
                                               VkImageLayout dst_layout,
+                                              bool dst_render_loop,
                                               unsigned src_queue_mask,
                                               unsigned dst_queue_mask,
                                               const VkImageSubresourceRange *range)
 {
        if (src_layout == VK_IMAGE_LAYOUT_UNDEFINED) {
                radv_init_color_image_metadata(cmd_buffer, image,
-                                              src_layout, dst_layout,
+                                              src_layout, src_render_loop,
+                                              dst_layout, dst_render_loop,
                                               src_queue_mask, dst_queue_mask,
                                               range);
                return;
@@ -5169,18 +5279,18 @@ static void radv_handle_color_image_transition(struct radv_cmd_buffer *cmd_buffe
        if (radv_dcc_enabled(image, range->baseMipLevel)) {
                if (src_layout == VK_IMAGE_LAYOUT_PREINITIALIZED) {
                        radv_initialize_dcc(cmd_buffer, image, range, 0xffffffffu);
-               } else if (radv_layout_dcc_compressed(image, src_layout, src_queue_mask) &&
-                          !radv_layout_dcc_compressed(image, dst_layout, dst_queue_mask)) {
+               } else if (radv_layout_dcc_compressed(cmd_buffer->device, image, src_layout, src_render_loop, src_queue_mask) &&
+                          !radv_layout_dcc_compressed(cmd_buffer->device, image, dst_layout, dst_render_loop, dst_queue_mask)) {
                        radv_decompress_dcc(cmd_buffer, image, range);
-               } else if (radv_layout_can_fast_clear(image, src_layout, src_queue_mask) &&
-                          !radv_layout_can_fast_clear(image, dst_layout, dst_queue_mask)) {
+               } else if (radv_layout_can_fast_clear(image, src_layout, src_render_loop, src_queue_mask) &&
+                          !radv_layout_can_fast_clear(image, dst_layout, dst_render_loop, dst_queue_mask)) {
                        radv_fast_clear_flush_image_inplace(cmd_buffer, image, range);
                }
        } else if (radv_image_has_cmask(image) || radv_image_has_fmask(image)) {
                bool fce_eliminate = false, fmask_expand = false;
 
-               if (radv_layout_can_fast_clear(image, src_layout, src_queue_mask) &&
-                   !radv_layout_can_fast_clear(image, dst_layout, dst_queue_mask)) {
+               if (radv_layout_can_fast_clear(image, src_layout, src_render_loop, src_queue_mask) &&
+                   !radv_layout_can_fast_clear(image, dst_layout, dst_render_loop, dst_queue_mask)) {
                        fce_eliminate = true;
                }
 
@@ -5205,7 +5315,9 @@ static void radv_handle_color_image_transition(struct radv_cmd_buffer *cmd_buffe
 static void radv_handle_image_transition(struct radv_cmd_buffer *cmd_buffer,
                                         struct radv_image *image,
                                         VkImageLayout src_layout,
+                                        bool src_render_loop,
                                         VkImageLayout dst_layout,
+                                        bool dst_render_loop,
                                         uint32_t src_family,
                                         uint32_t dst_family,
                                         const VkImageSubresourceRange *range,
@@ -5244,12 +5356,14 @@ static void radv_handle_image_transition(struct radv_cmd_buffer *cmd_buffer,
 
        if (vk_format_is_depth(image->vk_format)) {
                radv_handle_depth_image_transition(cmd_buffer, image,
-                                                  src_layout, dst_layout,
+                                                  src_layout, src_render_loop,
+                                                  dst_layout, dst_render_loop,
                                                   src_queue_mask, dst_queue_mask,
                                                   range, sample_locs);
        } else {
                radv_handle_color_image_transition(cmd_buffer, image,
-                                                  src_layout, dst_layout,
+                                                  src_layout, src_render_loop,
+                                                  dst_layout, dst_render_loop,
                                                   src_queue_mask, dst_queue_mask,
                                                   range);
        }
@@ -5282,7 +5396,7 @@ radv_barrier(struct radv_cmd_buffer *cmd_buffer,
 
                radv_cs_add_buffer(cmd_buffer->device->ws, cs, event->bo);
 
-               MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cs, 7);
+               ASSERTED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cs, 7);
 
                radv_cp_wait_mem(cs, WAIT_REG_MEM_EQUAL, va, 1, 0xffffffff);
                assert(cmd_buffer->cs->cdw <= cdw_max);
@@ -5346,7 +5460,9 @@ radv_barrier(struct radv_cmd_buffer *cmd_buffer,
 
                radv_handle_image_transition(cmd_buffer, image,
                                             pImageMemoryBarriers[i].oldLayout,
+                                            false, /* Outside of a renderpass we are never in a renderloop */
                                             pImageMemoryBarriers[i].newLayout,
+                                            false, /* Outside of a renderpass we are never in a renderloop */
                                             pImageMemoryBarriers[i].srcQueueFamilyIndex,
                                             pImageMemoryBarriers[i].dstQueueFamilyIndex,
                                             &pImageMemoryBarriers[i].subresourceRange,
@@ -5401,7 +5517,7 @@ static void write_event(struct radv_cmd_buffer *cmd_buffer,
 
        radv_cs_add_buffer(cmd_buffer->device->ws, cs, event->bo);
 
-       MAYBE_UNUSED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cs, 21);
+       ASSERTED unsigned cdw_max = radeon_check_space(cmd_buffer->device->ws, cs, 21);
 
        /* Flags that only require a top-of-pipe event. */
        VkPipelineStageFlags top_of_pipe_flags =