[Mesa-dev] [PATCH 3/3] llvmpipe: don't store number of layers per level

[Brian Paul]

On 07/31/2014 12:26 PM, sroland at vmware.com wrote:
> From: Roland Scheidegger <sroland at vmware.com>
>
> This could be recalculated, though it turns out the only use of it after
> resource allocation is for calculating whole resource size (for scene size
> accounting though that isn't quite ideal neither). Thus, instead just store
> the whole resource size and drop it (saving a couple bytes of storage per
> resource). It makes things simpler too. Note that for the accounting winsys
> resources always come back with size 0 but this is unchanged (we don't actually
> know the size in any case).
> Also reformat llvmpipe_texture_layout (drop unneded indentation).
> ---
>   src/gallium/drivers/llvmpipe/lp_texture.c | 128 ++++++++++++------------------
>   src/gallium/drivers/llvmpipe/lp_texture.h |   4 +-
>   2 files changed, 51 insertions(+), 81 deletions(-)

The series looks good to me.

Reviewed-by: Brian Paul <brianp at vmware.com>