radeonsi/gfx10: implement si_set_{constant,shader}_buffer
[mesa.git] / src / gallium / drivers / radeonsi / si_descriptors.c
index 0f22c55723c4400198948812c372c0eb5d785087..b6be087b8718f19b8979be2c1a7b3da25bd9a3cc 100644 (file)
@@ -55,7 +55,6 @@
 
 #include "si_pipe.h"
 #include "sid.h"
-#include "gfx9d.h"
 
 #include "util/hash_table.h"
 #include "util/u_idalloc.h"
@@ -220,13 +219,6 @@ si_get_sampler_view_priority(struct si_resource *res)
        return RADEON_PRIO_SAMPLER_TEXTURE;
 }
 
-static unsigned
-si_sampler_and_image_descriptors_idx(unsigned shader)
-{
-       return SI_DESCS_FIRST_SHADER + shader * SI_NUM_SHADER_DESCS +
-              SI_SHADER_DESCS_SAMPLERS_AND_IMAGES;
-}
-
 static struct si_descriptors *
 si_sampler_and_image_descriptors(struct si_context *sctx, unsigned shader)
 {
@@ -347,15 +339,14 @@ void si_set_mutable_tex_desc_fields(struct si_screen *sscreen,
            base_level_info->mode == RADEON_SURF_MODE_2D)
                state[0] |= tex->surface.tile_swizzle;
 
-       if (sscreen->info.chip_class >= VI) {
+       if (sscreen->info.chip_class >= GFX8) {
                state[6] &= C_008F28_COMPRESSION_EN;
-               state[7] = 0;
 
                if (vi_dcc_enabled(tex, first_level)) {
                        meta_va = (!tex->dcc_separate_buffer ? tex->buffer.gpu_address : 0) +
                                  tex->dcc_offset;
 
-                       if (sscreen->info.chip_class == VI) {
+                       if (sscreen->info.chip_class == GFX8) {
                                meta_va += base_level_info->dcc_offset;
                                assert(base_level_info->mode == RADEON_SURF_MODE_2D);
                        }
@@ -365,22 +356,48 @@ void si_set_mutable_tex_desc_fields(struct si_screen *sscreen,
                        meta_va = tex->buffer.gpu_address + tex->htile_offset;
                }
 
-               if (meta_va) {
+               if (meta_va)
                        state[6] |= S_008F28_COMPRESSION_EN(1);
-                       state[7] = meta_va >> 8;
-               }
        }
 
-       if (sscreen->info.chip_class >= GFX9) {
+       if (sscreen->info.chip_class >= GFX8 && sscreen->info.chip_class <= GFX9)
+               state[7] = meta_va >> 8;
+
+       if (sscreen->info.chip_class >= GFX10) {
+               state[3] &= C_00A00C_SW_MODE;
+
+               if (is_stencil) {
+                       state[3] |= S_00A00C_SW_MODE(tex->surface.u.gfx9.stencil.swizzle_mode);
+               } else {
+                       state[3] |= S_00A00C_SW_MODE(tex->surface.u.gfx9.surf.swizzle_mode);
+               }
+
+               state[6] &= C_00A018_META_DATA_ADDRESS_LO &
+                           C_00A018_META_PIPE_ALIGNED;
+
+               if (meta_va) {
+                       struct gfx9_surf_meta_flags meta;
+
+                       if (tex->dcc_offset)
+                               meta = tex->surface.u.gfx9.dcc;
+                       else
+                               meta = tex->surface.u.gfx9.htile;
+
+                       state[6] |= S_00A018_META_PIPE_ALIGNED(meta.pipe_aligned) |
+                                   S_00A018_META_DATA_ADDRESS_LO(meta_va >> 8);
+               }
+
+               state[7] = meta_va >> 16;
+       } else if (sscreen->info.chip_class >= GFX9) {
                state[3] &= C_008F1C_SW_MODE;
-               state[4] &= C_008F20_PITCH_GFX9;
+               state[4] &= C_008F20_PITCH;
 
                if (is_stencil) {
                        state[3] |= S_008F1C_SW_MODE(tex->surface.u.gfx9.stencil.swizzle_mode);
-                       state[4] |= S_008F20_PITCH_GFX9(tex->surface.u.gfx9.stencil.epitch);
+                       state[4] |= S_008F20_PITCH(tex->surface.u.gfx9.stencil.epitch);
                } else {
                        state[3] |= S_008F1C_SW_MODE(tex->surface.u.gfx9.surf.swizzle_mode);
-                       state[4] |= S_008F20_PITCH_GFX9(tex->surface.u.gfx9.surf.epitch);
+                       state[4] |= S_008F20_PITCH(tex->surface.u.gfx9.surf.epitch);
                }
 
                state[5] &= C_008F24_META_DATA_ADDRESS &
@@ -399,14 +416,14 @@ void si_set_mutable_tex_desc_fields(struct si_screen *sscreen,
                                    S_008F24_META_RB_ALIGNED(meta.rb_aligned);
                }
        } else {
-               /* SI-CI-VI */
+               /* GFX6-GFX8 */
                unsigned pitch = base_level_info->nblk_x * block_width;
                unsigned index = si_tile_mode_index(tex, base_level, is_stencil);
 
                state[3] &= C_008F1C_TILING_INDEX;
                state[3] |= S_008F1C_TILING_INDEX(index);
-               state[4] &= C_008F20_PITCH_GFX6;
-               state[4] |= S_008F20_PITCH_GFX6(pitch - 1);
+               state[4] &= C_008F20_PITCH;
+               state[4] |= S_008F20_PITCH(pitch - 1);
        }
 }
 
@@ -669,7 +686,8 @@ si_mark_image_range_valid(const struct pipe_image_view *view)
 {
        struct si_resource *res = si_resource(view->resource);
 
-       assert(res && res->b.b.target == PIPE_BUFFER);
+       if (res->b.b.target != PIPE_BUFFER)
+               return;
 
        util_range_add(&res->valid_buffer_range,
                       view->u.buf.offset,
@@ -686,7 +704,8 @@ static void si_set_shader_image_desc(struct si_context *ctx,
 
        res = si_resource(view->resource);
 
-       if (res->b.b.target == PIPE_BUFFER) {
+       if (res->b.b.target == PIPE_BUFFER ||
+           view->shader_access & SI_IMAGE_ACCESS_AS_BUFFER) {
                if (view->access & PIPE_IMAGE_ACCESS_WRITE)
                        si_mark_image_range_valid(view);
 
@@ -748,7 +767,7 @@ static void si_set_shader_image_desc(struct si_context *ctx,
                        hw_level = 0;
                }
 
-               si_make_texture_descriptor(screen, tex,
+               screen->make_texture_descriptor(screen, tex,
                                           false, res->b.b.target,
                                           view->format, swizzle,
                                           hw_level, hw_level,
@@ -787,7 +806,8 @@ static void si_set_shader_image(struct si_context *ctx,
 
        si_set_shader_image_desc(ctx, view, skip_decompress, desc, NULL);
 
-       if (res->b.b.target == PIPE_BUFFER) {
+       if (res->b.b.target == PIPE_BUFFER ||
+           view->shader_access & SI_IMAGE_ACCESS_AS_BUFFER) {
                images->needs_color_decompress_mask &= ~(1 << slot);
                res->bind_history |= PIPE_BIND_SHADER_IMAGE;
        } else {
@@ -946,7 +966,7 @@ static void si_bind_sampler_states(struct pipe_context *ctx,
        struct si_sampler_state **sstates = (struct si_sampler_state**)states;
        int i;
 
-       if (!count || shader >= SI_NUM_SHADERS)
+       if (!count || shader >= SI_NUM_SHADERS || !sstates)
                return;
 
        for (i = 0; i < count; i++) {
@@ -957,7 +977,7 @@ static void si_bind_sampler_states(struct pipe_context *ctx,
                    sstates[i] == samplers->sampler_states[slot])
                        continue;
 
-#ifdef DEBUG
+#ifndef NDEBUG
                assert(sstates[i]->magic == SI_SAMPLER_STATE_MAGIC);
 #endif
                samplers->sampler_states[slot] = sstates[i];
@@ -990,16 +1010,13 @@ static void si_init_buffer_resources(struct si_buffer_resources *buffers,
                                     struct si_descriptors *descs,
                                     unsigned num_buffers,
                                     short shader_userdata_rel_index,
-                                    enum radeon_bo_usage shader_usage,
-                                    enum radeon_bo_usage shader_usage_constbuf,
                                     enum radeon_bo_priority priority,
                                     enum radeon_bo_priority priority_constbuf)
 {
-       buffers->shader_usage = shader_usage;
-       buffers->shader_usage_constbuf = shader_usage_constbuf;
        buffers->priority = priority;
        buffers->priority_constbuf = priority_constbuf;
        buffers->buffers = CALLOC(num_buffers, sizeof(struct pipe_resource*));
+       buffers->offsets = CALLOC(num_buffers, sizeof(buffers->offsets[0]));
 
        si_init_descriptors(descs, shader_userdata_rel_index, 4, num_buffers);
 }
@@ -1014,6 +1031,7 @@ static void si_release_buffer_resources(struct si_buffer_resources *buffers,
        }
 
        FREE(buffers->buffers);
+       FREE(buffers->offsets);
 }
 
 static void si_buffer_resources_begin_new_cs(struct si_context *sctx,
@@ -1027,8 +1045,8 @@ static void si_buffer_resources_begin_new_cs(struct si_context *sctx,
 
                radeon_add_to_buffer_list(sctx, sctx->gfx_cs,
                        si_resource(buffers->buffers[i]),
-                       i < SI_NUM_SHADER_BUFFERS ? buffers->shader_usage :
-                                                   buffers->shader_usage_constbuf,
+                       buffers->writable_mask & (1u << i) ? RADEON_USAGE_READWRITE :
+                                                            RADEON_USAGE_READ,
                        i < SI_NUM_SHADER_BUFFERS ? buffers->priority :
                                                    buffers->priority_constbuf);
        }
@@ -1142,7 +1160,7 @@ bool si_upload_vertex_buffer_descriptors(struct si_context *sctx)
                uint64_t va = buf->gpu_address + offset;
 
                int64_t num_records = (int64_t)buf->b.b.width0 - offset;
-               if (sctx->chip_class != VI && vb->stride) {
+               if (sctx->chip_class != GFX8 && vb->stride) {
                        /* Round up by rounding down and adding 1 */
                        num_records = (num_records - velems->format_size[i]) /
                                      vb->stride + 1;
@@ -1176,13 +1194,6 @@ bool si_upload_vertex_buffer_descriptors(struct si_context *sctx)
 
 /* CONSTANT BUFFERS */
 
-static unsigned
-si_const_and_shader_buffer_descriptors_idx(unsigned shader)
-{
-       return SI_DESCS_FIRST_SHADER + shader * SI_NUM_SHADER_DESCS +
-              SI_SHADER_DESCS_CONST_AND_SHADER_BUFFERS;
-}
-
 static struct si_descriptors *
 si_const_and_shader_buffer_descriptors(struct si_context *sctx, unsigned shader)
 {
@@ -1211,20 +1222,19 @@ static void si_set_constant_buffer(struct si_context *sctx,
        assert(slot < descs->num_elements);
        pipe_resource_reference(&buffers->buffers[slot], NULL);
 
-       /* CIK cannot unbind a constant buffer (S_BUFFER_LOAD is buggy
+       /* GFX7 cannot unbind a constant buffer (S_BUFFER_LOAD is buggy
         * with a NULL buffer). We need to use a dummy buffer instead. */
-       if (sctx->chip_class == CIK &&
+       if (sctx->chip_class == GFX7 &&
            (!input || (!input->buffer && !input->user_buffer)))
                input = &sctx->null_const_buf;
 
        if (input && (input->buffer || input->user_buffer)) {
                struct pipe_resource *buffer = NULL;
                uint64_t va;
+               unsigned buffer_offset;
 
                /* Upload the user buffer if needed. */
                if (input->user_buffer) {
-                       unsigned buffer_offset;
-
                        si_upload_const_buffer(sctx,
                                               (struct si_resource**)&buffer, input->user_buffer,
                                               input->buffer_size, &buffer_offset);
@@ -1233,12 +1243,13 @@ static void si_set_constant_buffer(struct si_context *sctx,
                                si_set_constant_buffer(sctx, buffers, descriptors_idx, slot, NULL);
                                return;
                        }
-                       va = si_resource(buffer)->gpu_address + buffer_offset;
                } else {
                        pipe_resource_reference(&buffer, input->buffer);
-                       va = si_resource(buffer)->gpu_address + input->buffer_offset;
+                       buffer_offset = input->buffer_offset;
                }
 
+               va = si_resource(buffer)->gpu_address + buffer_offset;
+
                /* Set the descriptor. */
                uint32_t *desc = descs->list + slot*4;
                desc[0] = va;
@@ -1248,14 +1259,22 @@ static void si_set_constant_buffer(struct si_context *sctx,
                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_NUM_FORMAT(V_008F0C_BUF_NUM_FORMAT_FLOAT) |
-                         S_008F0C_DATA_FORMAT(V_008F0C_BUF_DATA_FORMAT_32);
+                         S_008F0C_DST_SEL_W(V_008F0C_SQ_SEL_W);
+
+               if (sctx->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_NUM_FORMAT(V_008F0C_BUF_NUM_FORMAT_FLOAT) |
+                                  S_008F0C_DATA_FORMAT(V_008F0C_BUF_DATA_FORMAT_32);
+               }
 
                buffers->buffers[slot] = buffer;
+               buffers->offsets[slot] = buffer_offset;
                radeon_add_to_gfx_buffer_list_check_mem(sctx,
                                                        si_resource(buffer),
-                                                       buffers->shader_usage_constbuf,
+                                                       RADEON_USAGE_READ,
                                                        buffers->priority_constbuf, true);
                buffers->enabled_mask |= 1u << slot;
        } else {
@@ -1308,7 +1327,7 @@ static void si_set_shader_buffer(struct si_context *sctx,
                                 struct si_buffer_resources *buffers,
                                 unsigned descriptors_idx,
                                 uint slot, const struct pipe_shader_buffer *sbuffer,
-                                enum radeon_bo_priority priority)
+                                bool writable, enum radeon_bo_priority priority)
 {
        struct si_descriptors *descs = &sctx->descriptors[descriptors_idx];
        uint32_t *desc = descs->list + slot * 4;
@@ -1317,6 +1336,7 @@ static void si_set_shader_buffer(struct si_context *sctx,
                pipe_resource_reference(&buffers->buffers[slot], NULL);
                memset(desc, 0, sizeof(uint32_t) * 4);
                buffers->enabled_mask &= ~(1u << slot);
+               buffers->writable_mask &= ~(1u << slot);
                sctx->descriptors_dirty |= 1u << descriptors_idx;
                return;
        }
@@ -1331,14 +1351,27 @@ static void si_set_shader_buffer(struct si_context *sctx,
        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_NUM_FORMAT(V_008F0C_BUF_NUM_FORMAT_FLOAT) |
-                 S_008F0C_DATA_FORMAT(V_008F0C_BUF_DATA_FORMAT_32);
+                 S_008F0C_DST_SEL_W(V_008F0C_SQ_SEL_W);
+
+       if (sctx->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_NUM_FORMAT(V_008F0C_BUF_NUM_FORMAT_FLOAT) |
+                          S_008F0C_DATA_FORMAT(V_008F0C_BUF_DATA_FORMAT_32);
+       }
 
        pipe_resource_reference(&buffers->buffers[slot], &buf->b.b);
+       buffers->offsets[slot] = sbuffer->buffer_offset;
        radeon_add_to_gfx_buffer_list_check_mem(sctx, buf,
-                                               buffers->shader_usage,
+                                               writable ? RADEON_USAGE_READWRITE :
+                                                          RADEON_USAGE_READ,
                                                priority, true);
+       if (writable)
+               buffers->writable_mask |= 1u << slot;
+       else
+               buffers->writable_mask &= ~(1u << slot);
 
        buffers->enabled_mask |= 1u << slot;
        sctx->descriptors_dirty |= 1u << descriptors_idx;
@@ -1350,7 +1383,8 @@ static void si_set_shader_buffer(struct si_context *sctx,
 static void si_set_shader_buffers(struct pipe_context *ctx,
                                  enum pipe_shader_type shader,
                                  unsigned start_slot, unsigned count,
-                                 const struct pipe_shader_buffer *sbuffers)
+                                 const struct pipe_shader_buffer *sbuffers,
+                                 unsigned writable_bitmask)
 {
        struct si_context *sctx = (struct si_context *)ctx;
        struct si_buffer_resources *buffers = &sctx->const_and_shader_buffers[shader];
@@ -1367,6 +1401,7 @@ static void si_set_shader_buffers(struct pipe_context *ctx,
                        si_resource(sbuffer->buffer)->bind_history |= PIPE_BIND_SHADER_BUFFER;
 
                si_set_shader_buffer(sctx, buffers, descriptors_idx, slot, sbuffer,
+                                    !!(writable_bitmask & (1u << i)),
                                     buffers->priority);
        }
 }
@@ -1401,7 +1436,7 @@ void si_set_rw_shader_buffer(struct si_context *sctx, uint slot,
                             const struct pipe_shader_buffer *sbuffer)
 {
        si_set_shader_buffer(sctx, &sctx->rw_buffers, SI_DESCS_RW_BUFFERS,
-                            slot, sbuffer, RADEON_PRIO_SHADER_RW_BUFFER);
+                            slot, sbuffer, true, RADEON_PRIO_SHADER_RW_BUFFER);
 }
 
 void si_set_ring_buffer(struct si_context *sctx, uint slot,
@@ -1460,7 +1495,7 @@ void si_set_ring_buffer(struct si_context *sctx, uint slot,
                        break;
                }
 
-               if (sctx->chip_class >= VI && stride)
+               if (sctx->chip_class >= GFX8 && stride)
                        num_records *= stride;
 
                /* Set the descriptor. */
@@ -1487,7 +1522,7 @@ void si_set_ring_buffer(struct si_context *sctx, uint slot,
                pipe_resource_reference(&buffers->buffers[slot], buffer);
                radeon_add_to_buffer_list(sctx, sctx->gfx_cs,
                                      si_resource(buffer),
-                                     buffers->shader_usage, buffers->priority);
+                                     RADEON_USAGE_READWRITE, buffers->priority);
                buffers->enabled_mask |= 1u << slot;
        } else {
                /* Clear the descriptor. */
@@ -1498,20 +1533,6 @@ void si_set_ring_buffer(struct si_context *sctx, uint slot,
        sctx->descriptors_dirty |= 1u << SI_DESCS_RW_BUFFERS;
 }
 
-static void si_desc_reset_buffer_offset(uint32_t *desc, uint64_t old_buf_va,
-                                       struct pipe_resource *new_buf)
-{
-       /* Retrieve the buffer offset from the descriptor. */
-       uint64_t old_desc_va = si_desc_extract_buffer_address(desc);
-
-       assert(old_buf_va <= old_desc_va);
-       uint64_t offset_within_buffer = old_desc_va - old_buf_va;
-
-       /* Update the descriptor. */
-       si_set_buf_desc_address(si_resource(new_buf), offset_within_buffer,
-                               desc);
-}
-
 /* INTERNAL CONST BUFFERS */
 
 static void si_set_polygon_stipple(struct pipe_context *ctx,
@@ -1590,14 +1611,14 @@ void si_update_needs_color_decompress_masks(struct si_context *sctx)
 
 /* BUFFER DISCARD/INVALIDATION */
 
-/** Reset descriptors of buffer resources after \p buf has been invalidated. */
+/* Reset descriptors of buffer resources after \p buf has been invalidated.
+ * If buf == NULL, reset all descriptors.
+ */
 static void si_reset_buffer_resources(struct si_context *sctx,
                                      struct si_buffer_resources *buffers,
                                      unsigned descriptors_idx,
                                      unsigned slot_mask,
                                      struct pipe_resource *buf,
-                                     uint64_t old_va,
-                                     enum radeon_bo_usage usage,
                                      enum radeon_bo_priority priority)
 {
        struct si_descriptors *descs = &sctx->descriptors[descriptors_idx];
@@ -1605,23 +1626,30 @@ static void si_reset_buffer_resources(struct si_context *sctx,
 
        while (mask) {
                unsigned i = u_bit_scan(&mask);
-               if (buffers->buffers[i] == buf) {
-                       si_desc_reset_buffer_offset(descs->list + i*4,
-                                                   old_va, buf);
+               struct pipe_resource *buffer = buffers->buffers[i];
+
+               if (buffer && (!buf || buffer == buf)) {
+                       si_set_buf_desc_address(si_resource(buffer), buffers->offsets[i],
+                                               descs->list + i*4);
                        sctx->descriptors_dirty |= 1u << descriptors_idx;
 
                        radeon_add_to_gfx_buffer_list_check_mem(sctx,
-                                                               si_resource(buf),
-                                                               usage, priority, true);
+                                                               si_resource(buffer),
+                                                               buffers->writable_mask & (1u << i) ?
+                                                                       RADEON_USAGE_READWRITE :
+                                                                       RADEON_USAGE_READ,
+                                                               priority, true);
                }
        }
 }
 
-/* Update all resource bindings where the buffer is bound, including
+/* Update all buffer bindings where the buffer is bound, including
  * all resource descriptors. This is invalidate_buffer without
- * the invalidation. */
-void si_rebind_buffer(struct si_context *sctx, struct pipe_resource *buf,
-                     uint64_t old_va)
+ * the invalidation.
+ *
+ * If buf == NULL, update all buffer bindings.
+ */
+void si_rebind_buffer(struct si_context *sctx, struct pipe_resource *buf)
 {
        struct si_resource *buffer = si_resource(buf);
        unsigned i, shader;
@@ -1635,7 +1663,10 @@ void si_rebind_buffer(struct si_context *sctx, struct pipe_resource *buf,
         */
 
        /* Vertex buffers. */
-       if (buffer->bind_history & PIPE_BIND_VERTEX_BUFFER) {
+       if (!buffer) {
+               if (num_elems)
+                       sctx->vertex_buffers_dirty = true;
+       } else if (buffer->bind_history & PIPE_BIND_VERTEX_BUFFER) {
                for (i = 0; i < num_elems; i++) {
                        int vb = sctx->vertex_elements->vertex_buffer_index[i];
 
@@ -1652,21 +1683,23 @@ void si_rebind_buffer(struct si_context *sctx, struct pipe_resource *buf,
        }
 
        /* Streamout buffers. (other internal buffers can't be invalidated) */
-       if (buffer->bind_history & PIPE_BIND_STREAM_OUTPUT) {
+       if (!buffer || buffer->bind_history & PIPE_BIND_STREAM_OUTPUT) {
                for (i = SI_VS_STREAMOUT_BUF0; i <= SI_VS_STREAMOUT_BUF3; i++) {
                        struct si_buffer_resources *buffers = &sctx->rw_buffers;
                        struct si_descriptors *descs =
                                &sctx->descriptors[SI_DESCS_RW_BUFFERS];
+                       struct pipe_resource *buffer = buffers->buffers[i];
 
-                       if (buffers->buffers[i] != buf)
+                       if (!buffer || (buf && buffer != buf))
                                continue;
 
-                       si_desc_reset_buffer_offset(descs->list + i*4,
-                                                   old_va, buf);
+                       si_set_buf_desc_address(si_resource(buffer), buffers->offsets[i],
+                                               descs->list + i*4);
                        sctx->descriptors_dirty |= 1u << SI_DESCS_RW_BUFFERS;
 
                        radeon_add_to_gfx_buffer_list_check_mem(sctx,
-                                                               buffer, buffers->shader_usage,
+                                                               si_resource(buffer),
+                                                               RADEON_USAGE_WRITE,
                                                                RADEON_PRIO_SHADER_RW_BUFFER,
                                                                true);
 
@@ -1680,27 +1713,25 @@ void si_rebind_buffer(struct si_context *sctx, struct pipe_resource *buf,
        }
 
        /* Constant and shader buffers. */
-       if (buffer->bind_history & PIPE_BIND_CONSTANT_BUFFER) {
+       if (!buffer || buffer->bind_history & PIPE_BIND_CONSTANT_BUFFER) {
                for (shader = 0; shader < SI_NUM_SHADERS; shader++)
                        si_reset_buffer_resources(sctx, &sctx->const_and_shader_buffers[shader],
                                                  si_const_and_shader_buffer_descriptors_idx(shader),
                                                  u_bit_consecutive(SI_NUM_SHADER_BUFFERS, SI_NUM_CONST_BUFFERS),
-                                                 buf, old_va,
-                                                 sctx->const_and_shader_buffers[shader].shader_usage_constbuf,
+                                                 buf,
                                                  sctx->const_and_shader_buffers[shader].priority_constbuf);
        }
 
-       if (buffer->bind_history & PIPE_BIND_SHADER_BUFFER) {
+       if (!buffer || buffer->bind_history & PIPE_BIND_SHADER_BUFFER) {
                for (shader = 0; shader < SI_NUM_SHADERS; shader++)
                        si_reset_buffer_resources(sctx, &sctx->const_and_shader_buffers[shader],
                                                  si_const_and_shader_buffer_descriptors_idx(shader),
                                                  u_bit_consecutive(0, SI_NUM_SHADER_BUFFERS),
-                                                 buf, old_va,
-                                                 sctx->const_and_shader_buffers[shader].shader_usage,
+                                                 buf,
                                                  sctx->const_and_shader_buffers[shader].priority);
        }
 
-       if (buffer->bind_history & PIPE_BIND_SAMPLER_VIEW) {
+       if (!buffer || buffer->bind_history & PIPE_BIND_SAMPLER_VIEW) {
                /* Texture buffers - update bindings. */
                for (shader = 0; shader < SI_NUM_SHADERS; shader++) {
                        struct si_samplers *samplers = &sctx->samplers[shader];
@@ -1710,26 +1741,29 @@ void si_rebind_buffer(struct si_context *sctx, struct pipe_resource *buf,
 
                        while (mask) {
                                unsigned i = u_bit_scan(&mask);
-                               if (samplers->views[i]->texture == buf) {
+                               struct pipe_resource *buffer = samplers->views[i]->texture;
+
+                               if (buffer && buffer->target == PIPE_BUFFER &&
+                                   (!buf || buffer == buf)) {
                                        unsigned desc_slot = si_get_sampler_slot(i);
 
-                                       si_desc_reset_buffer_offset(descs->list +
-                                                                   desc_slot * 16 + 4,
-                                                                   old_va, buf);
+                                       si_set_buf_desc_address(si_resource(buffer),
+                                                               samplers->views[i]->u.buf.offset,
+                                                               descs->list + desc_slot * 16 + 4);
                                        sctx->descriptors_dirty |=
                                                1u << si_sampler_and_image_descriptors_idx(shader);
 
-                                       radeon_add_to_gfx_buffer_list_check_mem(sctx,
-                                                                           buffer, RADEON_USAGE_READ,
-                                                                           RADEON_PRIO_SAMPLER_BUFFER,
-                                                                           true);
+                                       radeon_add_to_gfx_buffer_list_check_mem(
+                                               sctx, si_resource(buffer),
+                                               RADEON_USAGE_READ,
+                                               RADEON_PRIO_SAMPLER_BUFFER, true);
                                }
                        }
                }
        }
 
        /* Shader images */
-       if (buffer->bind_history & PIPE_BIND_SHADER_IMAGE) {
+       if (!buffer || buffer->bind_history & PIPE_BIND_SHADER_IMAGE) {
                for (shader = 0; shader < SI_NUM_SHADERS; ++shader) {
                        struct si_images *images = &sctx->images[shader];
                        struct si_descriptors *descs =
@@ -1738,21 +1772,23 @@ void si_rebind_buffer(struct si_context *sctx, struct pipe_resource *buf,
 
                        while (mask) {
                                unsigned i = u_bit_scan(&mask);
+                               struct pipe_resource *buffer = images->views[i].resource;
 
-                               if (images->views[i].resource == buf) {
+                               if (buffer && buffer->target == PIPE_BUFFER &&
+                                   (!buf || buffer == buf)) {
                                        unsigned desc_slot = si_get_image_slot(i);
 
                                        if (images->views[i].access & PIPE_IMAGE_ACCESS_WRITE)
                                                si_mark_image_range_valid(&images->views[i]);
 
-                                       si_desc_reset_buffer_offset(
-                                               descs->list + desc_slot * 8 + 4,
-                                               old_va, buf);
+                                       si_set_buf_desc_address(si_resource(buffer),
+                                                               images->views[i].u.buf.offset,
+                                                               descs->list + desc_slot * 8 + 4);
                                        sctx->descriptors_dirty |=
                                                1u << si_sampler_and_image_descriptors_idx(shader);
 
                                        radeon_add_to_gfx_buffer_list_check_mem(
-                                               sctx, buffer,
+                                               sctx, si_resource(buffer),
                                                RADEON_USAGE_READWRITE,
                                                RADEON_PRIO_SAMPLER_BUFFER, true);
                                }
@@ -1761,16 +1797,18 @@ void si_rebind_buffer(struct si_context *sctx, struct pipe_resource *buf,
        }
 
        /* Bindless texture handles */
-       if (buffer->texture_handle_allocated) {
+       if (!buffer || buffer->texture_handle_allocated) {
                struct si_descriptors *descs = &sctx->bindless_descriptors;
 
                util_dynarray_foreach(&sctx->resident_tex_handles,
                                      struct si_texture_handle *, tex_handle) {
                        struct pipe_sampler_view *view = (*tex_handle)->view;
                        unsigned desc_slot = (*tex_handle)->desc_slot;
+                       struct pipe_resource *buffer = view->texture;
 
-                       if (view->texture == buf) {
-                               si_set_buf_desc_address(buffer,
+                       if (buffer && buffer->target == PIPE_BUFFER &&
+                           (!buf || buffer == buf)) {
+                               si_set_buf_desc_address(si_resource(buffer),
                                                        view->u.buf.offset,
                                                        descs->list +
                                                        desc_slot * 16 + 4);
@@ -1779,7 +1817,7 @@ void si_rebind_buffer(struct si_context *sctx, struct pipe_resource *buf,
                                sctx->bindless_descriptors_dirty = true;
 
                                radeon_add_to_gfx_buffer_list_check_mem(
-                                       sctx, buffer,
+                                       sctx, si_resource(buffer),
                                        RADEON_USAGE_READ,
                                        RADEON_PRIO_SAMPLER_BUFFER, true);
                        }
@@ -1787,19 +1825,21 @@ void si_rebind_buffer(struct si_context *sctx, struct pipe_resource *buf,
        }
 
        /* Bindless image handles */
-       if (buffer->image_handle_allocated) {
+       if (!buffer || buffer->image_handle_allocated) {
                struct si_descriptors *descs = &sctx->bindless_descriptors;
 
                util_dynarray_foreach(&sctx->resident_img_handles,
                                      struct si_image_handle *, img_handle) {
                        struct pipe_image_view *view = &(*img_handle)->view;
                        unsigned desc_slot = (*img_handle)->desc_slot;
+                       struct pipe_resource *buffer = view->resource;
 
-                       if (view->resource == buf) {
+                       if (buffer && buffer->target == PIPE_BUFFER &&
+                           (!buf || buffer == buf)) {
                                if (view->access & PIPE_IMAGE_ACCESS_WRITE)
                                        si_mark_image_range_valid(view);
 
-                               si_set_buf_desc_address(buffer,
+                               si_set_buf_desc_address(si_resource(buffer),
                                                        view->u.buf.offset,
                                                        descs->list +
                                                        desc_slot * 16 + 4);
@@ -1808,12 +1848,25 @@ void si_rebind_buffer(struct si_context *sctx, struct pipe_resource *buf,
                                sctx->bindless_descriptors_dirty = true;
 
                                radeon_add_to_gfx_buffer_list_check_mem(
-                                       sctx, buffer,
+                                       sctx, si_resource(buffer),
                                        RADEON_USAGE_READWRITE,
                                        RADEON_PRIO_SAMPLER_BUFFER, true);
                        }
                }
        }
+
+       if (buffer) {
+               /* Do the same for other contexts. They will invoke this function
+                * with buffer == NULL.
+                */
+               unsigned new_counter = p_atomic_inc_return(&sctx->screen->dirty_buf_counter);
+
+               /* Skip the update for the current context, because we have already updated
+                * the buffer bindings.
+                */
+               if (new_counter == sctx->last_dirty_buf_counter + 1)
+                       sctx->last_dirty_buf_counter = new_counter;
+       }
 }
 
 static void si_upload_bindless_descriptor(struct si_context *sctx,
@@ -1867,7 +1920,7 @@ static void si_upload_bindless_descriptors(struct si_context *sctx)
        }
 
        /* Invalidate L1 because it doesn't know that L2 changed. */
-       sctx->flags |= SI_CONTEXT_INV_SMEM_L1;
+       sctx->flags |= SI_CONTEXT_INV_SCACHE;
        si_emit_cache_flush(sctx);
 
        sctx->bindless_descriptors_dirty = false;
@@ -2607,8 +2660,7 @@ static void si_make_image_handle_resident(struct pipe_context *ctx,
        }
 }
 
-
-void si_all_resident_buffers_begin_new_cs(struct si_context *sctx)
+static void si_resident_buffers_add_all_to_bo_list(struct si_context *sctx)
 {
        unsigned num_resident_tex_handles, num_resident_img_handles;
 
@@ -2640,6 +2692,8 @@ void si_all_resident_buffers_begin_new_cs(struct si_context *sctx)
 
        sctx->num_resident_handles += num_resident_tex_handles +
                                        num_resident_img_handles;
+       assert(sctx->bo_list_add_all_resident_resources);
+       sctx->bo_list_add_all_resident_resources = false;
 }
 
 /* INIT/DEINIT/UPLOAD */
@@ -2673,8 +2727,6 @@ void si_init_all_descriptors(struct si_context *sctx)
                desc = si_const_and_shader_buffer_descriptors(sctx, i);
                si_init_buffer_resources(&sctx->const_and_shader_buffers[i], desc,
                                         num_buffer_slots, rel_dw_offset,
-                                        RADEON_USAGE_READWRITE,
-                                        RADEON_USAGE_READ,
                                         RADEON_PRIO_SHADER_RW_BUFFER,
                                         RADEON_PRIO_CONST_BUFFER);
                desc->slot_index_to_bind_directly = si_get_constbuf_slot(0);
@@ -2704,9 +2756,8 @@ void si_init_all_descriptors(struct si_context *sctx)
        si_init_buffer_resources(&sctx->rw_buffers,
                                 &sctx->descriptors[SI_DESCS_RW_BUFFERS],
                                 SI_NUM_RW_BUFFERS, SI_SGPR_RW_BUFFERS,
-                                /* The second set of usage/priority is used by
+                                /* The second priority is used by
                                  * const buffers in RW buffer slots. */
-                                RADEON_USAGE_READWRITE, RADEON_USAGE_READ,
                                 RADEON_PRIO_SHADER_RINGS, RADEON_PRIO_CONST_BUFFER);
        sctx->descriptors[SI_DESCS_RW_BUFFERS].num_active_slots = SI_NUM_RW_BUFFERS;
 
@@ -2818,11 +2869,9 @@ void si_release_all_descriptors(struct si_context *sctx)
        si_release_bindless_descriptors(sctx);
 }
 
-void si_all_descriptors_begin_new_cs(struct si_context *sctx)
+void si_gfx_resources_add_all_to_bo_list(struct si_context *sctx)
 {
-       int i;
-
-       for (i = 0; i < SI_NUM_SHADERS; i++) {
+       for (unsigned i = 0; i < SI_NUM_GRAPHICS_SHADERS; i++) {
                si_buffer_resources_begin_new_cs(sctx, &sctx->const_and_shader_buffers[i]);
                si_sampler_views_begin_new_cs(sctx, &sctx->samplers[i]);
                si_image_views_begin_new_cs(sctx, &sctx->images[i]);
@@ -2830,11 +2879,40 @@ void si_all_descriptors_begin_new_cs(struct si_context *sctx)
        si_buffer_resources_begin_new_cs(sctx, &sctx->rw_buffers);
        si_vertex_buffers_begin_new_cs(sctx);
 
-       for (i = 0; i < SI_NUM_DESCS; ++i)
+       if (sctx->bo_list_add_all_resident_resources)
+               si_resident_buffers_add_all_to_bo_list(sctx);
+
+       assert(sctx->bo_list_add_all_gfx_resources);
+       sctx->bo_list_add_all_gfx_resources = false;
+}
+
+void si_compute_resources_add_all_to_bo_list(struct si_context *sctx)
+{
+       unsigned sh = PIPE_SHADER_COMPUTE;
+
+       si_buffer_resources_begin_new_cs(sctx, &sctx->const_and_shader_buffers[sh]);
+       si_sampler_views_begin_new_cs(sctx, &sctx->samplers[sh]);
+       si_image_views_begin_new_cs(sctx, &sctx->images[sh]);
+       si_buffer_resources_begin_new_cs(sctx, &sctx->rw_buffers);
+
+       if (sctx->bo_list_add_all_resident_resources)
+               si_resident_buffers_add_all_to_bo_list(sctx);
+
+       assert(sctx->bo_list_add_all_compute_resources);
+       sctx->bo_list_add_all_compute_resources = false;
+}
+
+void si_all_descriptors_begin_new_cs(struct si_context *sctx)
+{
+       for (unsigned i = 0; i < SI_NUM_DESCS; ++i)
                si_descriptors_begin_new_cs(sctx, &sctx->descriptors[i]);
        si_descriptors_begin_new_cs(sctx, &sctx->bindless_descriptors);
 
        si_shader_pointers_begin_new_cs(sctx);
+
+       sctx->bo_list_add_all_resident_resources = true;
+       sctx->bo_list_add_all_gfx_resources = true;
+       sctx->bo_list_add_all_compute_resources = true;
 }
 
 void si_set_active_descriptors(struct si_context *sctx, unsigned desc_idx,