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

radeonsi/gfx12: fill missing dcc tiling info

Browse Source 
Display DCC support has been enabled in 0bb83a4060 but this TODO
was forgotten.
Now that the kernel is fixed, we can set the related fields.

Fixes: 0bb83a4060 ("ac/surface: finish display DCC for gfx12")
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/31683>
This commit is contained in:
Pierre-Eric Pelloux-Prayer
2024-10-14 13:08:36 +02:00
parent a5a5a50ae8
commit bb08596645
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/gallium/drivers/radeonsi/si_texture.c
View File

@@ -226,11 +226,11 @@ static int si_init_surface(struct si_screen *sscreen, struct radeon_surf *surfac
flags |= RADEON_SURF_NO_HTILE; flags |= RADEON_SURF_NO_HTILE;
} }
/* TODO: Set these for scanout after display DCC is enabled. The reason these are not set is /* The kernel code translating tiling flags into a modifier was wrong
* because they overlap DCC_OFFSET_256B and the kernel driver incorrectly reads DCC_OFFSET_256B * until .58, so don't set these attributes for older versions.
* on GFX12, which completely breaks the display code.
*/ */
if (!is_imported && !(ptex->bind & PIPE_BIND_SCANOUT)) { bool supports_display_dcc = sscreen->info.drm_minor >= 58;
if (!is_imported && (!(ptex->bind & PIPE_BIND_SCANOUT) || supports_display_dcc)) {
enum pipe_format format = util_format_get_depth_only(ptex->format); enum pipe_format format = util_format_get_depth_only(ptex->format);
/* These should be set for both color and Z/S. */ /* These should be set for both color and Z/S. */