[Mesa-dev] [PATCH 3/3] llvmpipe: don't store number of layers per level
brianp at vmware.com
Thu Jul 31 12:39:16 PDT 2014
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>
More information about the mesa-dev