anv/device: Add some asserts to MapMemory
Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
This commit is contained in:
parent
843775bab7
commit
73ef9c8f04
|
@ -1283,6 +1283,16 @@ VkResult anv_MapMemory(
|
||||||
if (size == VK_WHOLE_SIZE)
|
if (size == VK_WHOLE_SIZE)
|
||||||
size = mem->bo.size - offset;
|
size = mem->bo.size - offset;
|
||||||
|
|
||||||
|
/* From the Vulkan spec version 1.0.32 docs for MapMemory:
|
||||||
|
*
|
||||||
|
* * If size is not equal to VK_WHOLE_SIZE, size must be greater than 0
|
||||||
|
* assert(size != 0);
|
||||||
|
* * If size is not equal to VK_WHOLE_SIZE, size must be less than or
|
||||||
|
* equal to the size of the memory minus offset
|
||||||
|
*/
|
||||||
|
assert(size > 0);
|
||||||
|
assert(offset + size <= mem->bo.size);
|
||||||
|
|
||||||
/* FIXME: Is this supposed to be thread safe? Since vkUnmapMemory() only
|
/* FIXME: Is this supposed to be thread safe? Since vkUnmapMemory() only
|
||||||
* takes a VkDeviceMemory pointer, it seems like only one map of the memory
|
* takes a VkDeviceMemory pointer, it seems like only one map of the memory
|
||||||
* at a time is valid. We could just mmap up front and return an offset
|
* at a time is valid. We could just mmap up front and return an offset
|
||||||
|
|
Loading…
Reference in New Issue