anv: remove assert from GetImageMemoryRequirements[2]
authorTapani Pälli <tapani.palli@intel.com>
Fri, 24 Apr 2020 12:28:41 +0000 (15:28 +0300)
committerMarge Bot <eric+marge@anholt.net>
Wed, 29 Apr 2020 08:30:42 +0000 (08:30 +0000)
commit1a33358b274631e0b8b493b0d885091d839f9d13
treeff9bdf60c6f7dbb68c44ca00f04e09485c504ead
parent2f6648dc3cb989c16fea9d3de968388e0496339b
anv: remove assert from GetImageMemoryRequirements[2]

This assert is actually correct but due to how android hardware buffer
support is implemented we should remove it, otherwise debug build of
mesa hits the assert with Android CTS tests.

Test creates VkImage with non-external format and sets up
VkExternalMemoryImageCreateInfo to indicate that image *may* be used
with Android hardwarebuffer handle. Then test attempts to get image
memory requirements. Problem with this is that we setup all android
supporting images as having external format and thus hit the assert as
the size has not been set yet. This is not a problem in practice since
android will bind ahw memory with the image later on.

Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/2807
Signed-off-by: Tapani Pälli <tapani.palli@intel.com>
Acked-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4762>
src/intel/vulkan/anv_device.c