Looks good to me :)
Reviewed-by: Plamena Manolova <plamena.manol...@intel.com>
On Tue, Mar 7, 2017 at 11:17 AM, Tapani Pälli <tapani.pa...@intel.com>
wrote:
> This is what comment above definition says and change fixes issue with
> 32bit build where BLOCK_POOL_MEMFD_SIZE is used as ftruncate parameter
> and constant currently gets converted from 4294967296 to 0.
>
> Signed-off-by: Tapani Pälli <tapani.pa...@intel.com>
> ---
> src/intel/vulkan/anv_private.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_
> private.h
> index c73196a..86b20c4 100644
> --- a/src/intel/vulkan/anv_private.h
> +++ b/src/intel/vulkan/anv_private.h
> @@ -386,7 +386,7 @@ struct anv_block_pool {
> };
>
> /* Block pools are backed by a fixed-size 2GB memfd */
> -#define BLOCK_POOL_MEMFD_SIZE (1ull << 32)
> +#define BLOCK_POOL_MEMFD_SIZE (1ul << 31)
>
> /* The center of the block pool is also the middle of the memfd. This may
> * change in the future if we decide differently for some reason.
> --
> 2.9.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
