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

Nanley Chery nanleychery at gmail.com
Tue Apr 18 23:41:47 UTC 2017


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

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



More information about the mesa-dev mailing list