winsys/amdgpu: cosmetic touchups

Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27060>
This commit is contained in:
Marek Olšák 2024-01-07 14:58:40 -05:00 committed by Marge Bot
parent 1e06cdad0d
commit 2da16e9635
2 changed files with 13 additions and 11 deletions

View file

@ -251,16 +251,15 @@ struct pb_slab *amdgpu_bo_slab_alloc(void *priv, unsigned heap, unsigned entry_s
void amdgpu_bo_slab_free(struct amdgpu_winsys *ws, struct pb_slab *slab);
uint64_t amdgpu_bo_get_va(struct pb_buffer_lean *buf);
static inline
struct amdgpu_winsys_bo *amdgpu_winsys_bo(struct pb_buffer_lean *bo)
static inline struct amdgpu_winsys_bo *
amdgpu_winsys_bo(struct pb_buffer_lean *bo)
{
return (struct amdgpu_winsys_bo *)bo;
}
static inline
void amdgpu_winsys_bo_reference(struct amdgpu_winsys *ws,
struct amdgpu_winsys_bo **dst,
struct amdgpu_winsys_bo *src)
static inline void
amdgpu_winsys_bo_reference(struct amdgpu_winsys *ws, struct amdgpu_winsys_bo **dst,
struct amdgpu_winsys_bo *src)
{
radeon_bo_reference(&ws->dummy_ws.base,
(struct pb_buffer_lean**)dst, (struct pb_buffer_lean*)src);

View file

@ -1205,10 +1205,11 @@ static void amdgpu_add_bo_fences_to_dependencies(struct amdgpu_cs *acs,
{
struct amdgpu_winsys *ws = acs->ws;
unsigned queue_index = acs->queue_index;
struct amdgpu_cs_buffer *buffers = list->buffers;
unsigned num_buffers = list->num_buffers;
for (unsigned i = 0; i < num_buffers; i++) {
struct amdgpu_cs_buffer *buffer = &list->buffers[i];
struct amdgpu_cs_buffer *buffer = &buffers[i];
struct amdgpu_winsys_bo *bo = buffer->bo;
/* Add BO fences from queues other than 'queue_index' to dependencies. */
@ -1243,10 +1244,11 @@ static void amdgpu_cs_add_syncobj_signal(struct radeon_cmdbuf *rws,
*/
static bool amdgpu_add_sparse_backing_buffers(struct amdgpu_cs_context *cs)
{
struct amdgpu_cs_buffer *buffers = cs->buffer_lists[AMDGPU_BO_SPARSE].buffers;
unsigned num_sparse_buffers = cs->buffer_lists[AMDGPU_BO_SPARSE].num_buffers;
for (unsigned i = 0; i < num_sparse_buffers; ++i) {
struct amdgpu_cs_buffer *buffer = &cs->buffer_lists[AMDGPU_BO_SPARSE].buffers[i];
struct amdgpu_cs_buffer *buffer = &buffers[i];
struct amdgpu_bo_sparse *bo = get_sparse_bo(buffer->bo);
simple_mtx_lock(&bo->commit_lock);
@ -1277,7 +1279,7 @@ static void amdgpu_cs_submit_ib(void *job, void *gdata, int thread_index)
struct amdgpu_cs *acs = (struct amdgpu_cs*)job;
struct amdgpu_winsys *ws = acs->ws;
struct amdgpu_cs_context *cs = acs->cst;
int i, r;
int r;
uint64_t seq_no = 0;
bool has_user_fence = amdgpu_cs_has_user_fence(acs);
@ -1423,11 +1425,12 @@ static void amdgpu_cs_submit_ib(void *job, void *gdata, int thread_index)
goto cleanup;
}
struct amdgpu_cs_buffer *real_buffers = cs->buffer_lists[AMDGPU_BO_REAL].buffers;
unsigned num_real_buffers = cs->buffer_lists[AMDGPU_BO_REAL].num_buffers;
bo_list = alloca((num_real_buffers + 2) * sizeof(struct drm_amdgpu_bo_list_entry));
for (i = 0; i < num_real_buffers; ++i) {
struct amdgpu_cs_buffer *buffer = &cs->buffer_lists[AMDGPU_BO_REAL].buffers[i];
for (unsigned i = 0; i < num_real_buffers; ++i) {
struct amdgpu_cs_buffer *buffer = &real_buffers[i];
bo_list[num_bo_handles].bo_handle = get_real_bo(buffer->bo)->kms_handle;
bo_list[num_bo_handles].bo_priority =