OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

anv: improve vma usage for descriptor buffer

Browse Source 
The dynamic visible memory type (or the prior descriptor buffer memory
type) doesn't need special aux-tt alignment or additional ccs space.

Signed-off-by: Yiwei Zhang <zzyiwei@chromium.org>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/30317>
This commit is contained in:
Yiwei Zhang
2024-07-22 23:09:08 -07:00
committed by Marge Bot
parent 8a5ef9413b
commit 6d0273f67a
1 changed files with 7 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/intel/vulkan/anv_device.c
View File

@@ -1414,25 +1414,22 @@ VkResult anv_AllocateMemory(
* alignment reported through vkGetImageMemoryRequirements() meet the
* AUX-TT requirement.
*
* TODO: when we enable EXT_descriptor_buffer, we'll be able to drop the
* AUX-TT alignment for that type of allocation.
* Allocations with the special dynamic_visible mem type are for things like
* descriptor buffers, so AUX-TT alignment is not needed here.
*/
if (device->info->has_aux_map)
if (device->info->has_aux_map && !mem_type->dynamic_visible)
alloc_flags |= ANV_BO_ALLOC_AUX_TT_ALIGNED;
/* If the allocation is not dedicated nor a host pointer, allocate
* additional CCS space.
*
* TODO: If we ever ship VK_EXT_descriptor_buffer (ahahah... :() we could
* drop this flag in the descriptor buffer case as we don't need any
* compression there.
*
* TODO: We could also create new memory types for allocations that don't
* need any compression.
* Allocations with the special dynamic_visible mem type are for things like
* descriptor buffers, which don't need any compression.
*/
if (device->physical->alloc_aux_tt_mem &&
dedicated_info == NULL &&
mem->vk.host_ptr == NULL)
mem->vk.host_ptr == NULL &&
!mem_type->dynamic_visible)
alloc_flags |= ANV_BO_ALLOC_AUX_CCS;
/* TODO: Android, ChromeOS and other applications may need another way to