From: Jason Ekstrand Date: Wed, 12 Jul 2017 02:04:38 +0000 (-0700) Subject: anv: Round u_vector element sizes to a power of two X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=8e3d9c5d09c48be9a88bdca7ae3124c12b6db4fe;p=mesa.git anv: Round u_vector element sizes to a power of two This fixes 32-bit builds of the driver. Commit 08413a81b93dc537fb0c3 changed things so that we now put struct anv_states in the u_vector for binding tables. On 64-bit builds, sizeof(struct anv_state) is a power of two but it isn't on 32-bit builds. Fixes: 08413a81b93dc537fb0c34327ad162f07e8c3427 Reviewed-by: Kenneth Graunke Cc: mesa-stable@lists.freedesktop.org --- diff --git a/src/intel/vulkan/anv_batch_chain.c b/src/intel/vulkan/anv_batch_chain.c index 47fee7398f7..ad76dc1fc0f 100644 --- a/src/intel/vulkan/anv_batch_chain.c +++ b/src/intel/vulkan/anv_batch_chain.c @@ -706,9 +706,10 @@ anv_cmd_buffer_init_batch_bo_chain(struct anv_cmd_buffer *cmd_buffer) *(struct anv_batch_bo **)u_vector_add(&cmd_buffer->seen_bbos) = batch_bo; + /* u_vector requires power-of-two size elements */ + unsigned pow2_state_size = util_next_power_of_two(sizeof(struct anv_state)); success = u_vector_init(&cmd_buffer->bt_block_states, - sizeof(struct anv_state), - 8 * sizeof(struct anv_state)); + pow2_state_size, 8 * pow2_state_size); if (!success) goto fail_seen_bbos;