We should always do this. So far we have been getting away with this because we overallocate at v3dv_job_start_frame, but that won't do for secondary command buffers for example, it is also unreliable if CLs grow past that initial allocation.
In the future, we might want to fix our emit macros so they do the allocation check implicitly, which would simplify the code and would make this process a lot less error prone.
814dd98df51 v3dv: ensure BCL space is available before emitting packets
src/broadcom/vulkan/v3dv_cmd_buffer.c | 88 +++++++++++++++++++++++++++++++++++
1 file changed, 88 insertions(+)