anv: Align fast clear color state buffer to a page.
authorRafael Antognolli <rafael.antognolli@intel.com>
Tue, 13 Aug 2019 21:47:27 +0000 (14:47 -0700)
committerRafael Antognolli <rafael.antognolli@intel.com>
Wed, 30 Oct 2019 19:41:29 +0000 (19:41 +0000)
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 <nanley.g.chery@intel.com>
src/intel/vulkan/anv_image.c

index c34cf209e75d1dbf4f2fc400ce411e805b38f417..998cfa75522d8a845bc429bd20e5585429a22696 100644 (file)
@@ -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;