[Mesa-dev] [PATCH] anv/image: Remove incorrect assertion in anv_BindImage

Jason Ekstrand jason at jlekstrand.net
Wed Apr 19 02:52:28 UTC 2017


On Tue, Apr 18, 2017 at 4:41 PM, Nanley Chery <nanleychery at gmail.com> 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 <nanley.g.chery at intel.com>
> ---
>  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
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170418/3d790593/attachment.html>


More information about the mesa-dev mailing list