ac/gpu_info: don't align IBs to the GL2 cache line size
authorMarek Olšák <marek.olsak@amd.com>
Mon, 7 Aug 2023 05:04:04 +0000 (01:04 -0400)
committerMarge Bot <emma+marge@anholt.net>
Sat, 30 Sep 2023 23:08:46 +0000 (23:08 +0000)
PAL doesn't do it. If drivers want IBs not to share cache lines with other buffers,
they should align the size manually.

Reviewed-by: Timur Kristóf <timur.kristof@gmail.com>
Reviewed-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25492>

src/amd/common/ac_gpu_info.c

index e8ec9ce..f75e0c2 100644 (file)
@@ -1331,10 +1331,6 @@ bool ac_query_gpu_info(int fd, void *dev_p, struct radeon_info *info,
    if (info->gfx_level == GFX6)
       info->gfx_ib_pad_with_type2 = true;
 
-   /* GFX10 and maybe GFX9 need this alignment for cache coherency. */
-   if (info->gfx_level >= GFX9)
-      info->ib_alignment = MAX2(info->ib_alignment, info->tcc_cache_line_size);
-
    if (info->gfx_level >= GFX11) {
       /* With num_cu = 4 in gfx11 measured power for idle, video playback and observed
        * power savings, hence enable dcc with retile for gfx11 with num_cu >= 4.