From e51722a7c79e64057a68ae16eee4735b9b190c7d Mon Sep 17 00:00:00 2001 From: Rafael Antognolli Date: Tue, 13 Aug 2019 14:47:27 -0700 Subject: [PATCH] anv: Align fast clear color state buffer to a page. On gen11 and older, compressed images are tiled and aligned to 4K. On gen12 this 4K alignment restriction was removed. However, only aligning the fast clear color buffer to 64B (a cacheline, as it's on the documentation) is causing some bugs where the fast clear color is not converted during the fast clear operation. Aligning things to 4K seems to fix it. v2: Assert that image->planes[plane].offset is 4K aligned (Nanley) Reviewed-by: Nanley Chery --- src/intel/vulkan/anv_image.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c index c34cf209e75..998cfa75522 100644 --- a/src/intel/vulkan/anv_image.c +++ b/src/intel/vulkan/anv_image.c @@ -285,6 +285,15 @@ add_aux_state_tracking_buffer(struct anv_image *image, } } + /* Add some padding to make sure the fast clear color state buffer starts at + * a 4K alignment. We believe that 256B might be enough, but due to lack of + * testing we will leave this as 4K for now. + */ + image->planes[plane].size = ALIGN(image->planes[plane].size, 4096); + image->size = ALIGN(image->size, 4096); + + assert(image->planes[plane].offset % 4096 == 0); + image->planes[plane].fast_clear_state_offset = image->planes[plane].offset + image->planes[plane].size; -- 2.30.2