From: Marek Olšák Date: Tue, 19 May 2020 23:30:54 +0000 (-0400) Subject: radeonsi/gfx8: enable TC-compatible HTILE from the beginning as before X-Git-Url: https://git.libre-soc.org/?p=mesa.git;a=commitdiff_plain;h=9375e72d8d1a7aedefeb1d6fbce6384d640a8d2e radeonsi/gfx8: enable TC-compatible HTILE from the beginning as before Fixes: 0d83e7f4b98 - radeonsi: enable TC-compatible HTILE on demand for best Z/S performance Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/2921 Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/2967 Reviewed-by: Pierre-Eric Pelloux-Prayer Part-of: --- diff --git a/src/gallium/drivers/radeonsi/si_texture.c b/src/gallium/drivers/radeonsi/si_texture.c index f6f108f3ced..63a3571b0a5 100644 --- a/src/gallium/drivers/radeonsi/si_texture.c +++ b/src/gallium/drivers/radeonsi/si_texture.c @@ -995,7 +995,15 @@ static struct si_texture *si_texture_create_object(struct pipe_screen *screen, /* don't include stencil-only formats which we don't support for rendering */ tex->is_depth = util_format_has_depth(util_format_description(tex->buffer.b.b.format)); tex->surface = *surface; - tex->tc_compatible_htile = false; /* This will be enabled on demand. */ + + /* On GFX8, HTILE uses different tiling depending on the TC_COMPATIBLE_HTILE + * setting, so we have to enable it if we enabled it at allocation. + * + * GFX9 and later use the same tiling for both, so TC-compatible HTILE can be + * enabled on demand. + */ + tex->tc_compatible_htile = sscreen->info.chip_class == GFX8 && + tex->surface.flags & RADEON_SURF_TC_COMPATIBLE_HTILE; /* TC-compatible HTILE: * - GFX8 only supports Z32_FLOAT.