fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-05 11:48:06 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

anv: Remove alignment to aux ratio on size of main surface

Browse source 
The existing 4KB alignment is sufficient for both 1MB and
64KB alignment requirements of CCS buffer.

Signed-off-by: Zhang, Jianxun <jianxun.zhang@intel.com>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23002>
This commit is contained in:
Zhang, Jianxun 2023-05-16 14:31:05 -07:00 • committed by Marge Bot
parent d2b6f16145
commit ec9efee907
1 changed files with 5 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/intel/vulkan/anv_allocator.c
View file

@ -1452,22 +1452,17 @@ anv_device_alloc_bo(struct anv_device *device,
anv_bo_alloc_flags_to_bo_flags(device, alloc_flags);
assert(bo_flags == (bo_flags & ANV_BO_CACHE_SUPPORTED_FLAGS));
/* The kernel is going to give us whole pages anyway */
/* The kernel is going to give us whole pages anyway. And we
* also need 4KB alignment for 1MB AUX buffer that follows
* the main region. The 4KB also covers 64KB AUX granularity
* that has 256B AUX mapping to the main.
*/
size = align64(size, 4096);
uint64_t ccs_size = 0;
if (device->info->has_aux_map && (alloc_flags & ANV_BO_ALLOC_IMPLICIT_CCS)) {
uint64_t aux_ratio =
intel_aux_get_main_to_aux_ratio(device->aux_map_ctx);
/* Aligning main the size up to the next multiple of main AUX CCS
* alignment requirement is wasteful, especially on MTL where the
* alignment is 1Mb. Instead align to the ratio of compression (1/256)
* which is also the requirement for the L1 aux-data address in the
* AUX-TT tables.
*/
size = align64(size, aux_ratio);
/* See anv_bo::_ccs_size */
ccs_size = align64(DIV_ROUND_UP(size, aux_ratio), 4096);
}