v3dv: create the command buffer BO set before we init CLs
authorIago Toral Quiroga <itoral@igalia.com>
Tue, 17 Dec 2019 10:30:23 +0000 (11:30 +0100)
committerMarge Bot <eric+marge@anholt.net>
Tue, 13 Oct 2020 21:21:25 +0000 (21:21 +0000)
The init of the CLs doesn't create any BOs for now, but this looks
like a sensible decision to future-proof things.

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6766>

src/broadcom/vulkan/v3dv_cmd_buffer.c

index c444b22..c1d4473 100644 (file)
@@ -84,13 +84,12 @@ cmd_buffer_create(struct v3dv_device *device,
    cmd_buffer->level = level;
    cmd_buffer->usage_flags = 0;
 
+   cmd_buffer->bos =
+      _mesa_set_create(NULL, _mesa_hash_pointer, _mesa_key_pointer_equal);
    v3dv_cl_init(cmd_buffer, &cmd_buffer->bcl);
    v3dv_cl_init(cmd_buffer, &cmd_buffer->rcl);
    v3dv_cl_init(cmd_buffer, &cmd_buffer->indirect);
 
-   cmd_buffer->bos =
-      _mesa_set_create(NULL, _mesa_hash_pointer, _mesa_key_pointer_equal);
-
    cmd_buffer->status = V3DV_CMD_BUFFER_STATUS_NEW;
 
    assert(pool);