On Tue, Apr 18, 2017 at 4:41 PM, Nanley Chery <[email protected]> wrote:

> According to the Linux kernel sources, the ioctl in anv_gem_mmap() will
> not fail if the size isn't a multiple of 4KB.
>

I was fairly sure that wasn't the case... I'll have to do some looking.


> Signed-off-by: Nanley Chery <[email protected]>
> ---
>  src/intel/vulkan/anv_image.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c
> index cf34dbe3b0..f374ab0b9a 100644
> --- a/src/intel/vulkan/anv_image.c
> +++ b/src/intel/vulkan/anv_image.c
> @@ -346,11 +346,10 @@ VkResult anv_BindImageMemory(
>
>     if (image->aux_surface.isl.size > 0) {
>
> -      /* The offset and size must be a multiple of 4K or else the
> -       * anv_gem_mmap call below will return NULL.
> +      /* The offset must be a multiple of 4K or else the anv_gem_mmap call
> +       * below will return NULL.
>         */
>        assert((image->offset + image->aux_surface.offset) % 4096 == 0);
> -      assert(image->aux_surface.isl.size % 4096 == 0);
>
>        /* Auxiliary surfaces need to have their memory cleared to 0 before
> they
>         * can be used.  For CCS surfaces, this puts them in the "resolved"
> --
> 2.12.2
>
> _______________________________________________
> mesa-dev mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to