[Mesa-dev] [PATCH 13/18] anv/allocator: Rework a comment

Juan A. Suarez Romero jasuarez at igalia.com
Thu Apr 27 16:30:34 UTC 2017


Reviewed-by: Juan A. Suarez Romero <jasuarez at igalia.com>


On Wed, 2017-04-26 at 07:35 -0700, Jason Ekstrand wrote:
> This commit just fixes up the English a bit and re-flows the comment.
> ---
>  src/intel/vulkan/anv_allocator.c | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> diff --git a/src/intel/vulkan/anv_allocator.c b/src/intel/vulkan/anv_allocator.c
> index 05666ec..fa7101b 100644
> --- a/src/intel/vulkan/anv_allocator.c
> +++ b/src/intel/vulkan/anv_allocator.c
> @@ -553,10 +553,11 @@ anv_block_pool_alloc_new(struct anv_block_pool *pool,
>           assert(pool->map);
>           return state.next;
>        } else if (state.next == state.end) {
> -         /* We allocated the first block outside the pool, we have to grow it.
> -          * pool_state->next acts a mutex: threads who try to allocate now will
> -          * get block indexes above the current limit and hit futex_wait
> -          * below. */
> +         /* We allocated the first block outside the pool so we have to grow
> +          * the pool.  pool_state->next acts a mutex: threads who try to
> +          * allocate now will get block indexes above the current limit and
> +          * hit futex_wait below.
> +          */
>           new.next = state.next + block_size;
>           new.end = anv_block_pool_grow(pool, pool_state, block_size);
>           old.u64 = __sync_lock_test_and_set(&pool_state->u64, new.u64);


More information about the mesa-dev mailing list