From 2b7e85562ae4f700d60d9a7bbcfbf65d194f02d3 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Marek=20Ol=C5=A1=C3=A1k?= Date: Sat, 29 Jul 2017 03:15:27 +0200 Subject: [PATCH] ac/surface: enable tile swizzle for mipmapped textures MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The tile swizzle computation was done after the whole miptree was computed, but that was too late, because at that point AddrSurfInfoOut contained information about the smallest miplevel, which is never 2D-tiled. The correct way is to do the computation before the second level is computed. Reviewed-by: Dave Airlie Reviewed-by: Nicolai Hähnle --- src/amd/common/ac_surface.c | 80 +++++++++++++++++++++---------------- 1 file changed, 46 insertions(+), 34 deletions(-) diff --git a/src/amd/common/ac_surface.c b/src/amd/common/ac_surface.c index 1eff4e51dd6..87a89936e1c 100644 --- a/src/amd/common/ac_surface.c +++ b/src/amd/common/ac_surface.c @@ -407,12 +407,16 @@ static unsigned cik_get_macro_tile_index(struct radeon_surf *surf) } /** + * This must be called after the first level is computed. + * * Copy surface-global settings like pipe/bank config from level 0 surface - * computation. + * computation, and compute tile swizzle. */ -static void gfx6_surface_settings(const struct radeon_info* info, - ADDR_COMPUTE_SURFACE_INFO_OUTPUT* csio, - struct radeon_surf *surf) +static int gfx6_surface_settings(ADDR_HANDLE addrlib, + const struct radeon_info *info, + const struct ac_surf_config *config, + ADDR_COMPUTE_SURFACE_INFO_OUTPUT* csio, + struct radeon_surf *surf) { surf->surf_alignment = csio->baseAlign; surf->u.legacy.pipe_config = csio->pTileInfo->pipeConfig - 1; @@ -429,6 +433,34 @@ static void gfx6_surface_settings(const struct radeon_info* info, } else { surf->u.legacy.macro_tile_index = 0; } + + /* Compute tile swizzle. */ + if (config->info.surf_index && + surf->u.legacy.level[0].mode == RADEON_SURF_MODE_2D && + !(surf->flags & (RADEON_SURF_Z_OR_SBUFFER | RADEON_SURF_SHAREABLE)) && + (config->info.samples > 1 || !(surf->flags & RADEON_SURF_SCANOUT))) { + ADDR_COMPUTE_BASE_SWIZZLE_INPUT AddrBaseSwizzleIn = {0}; + ADDR_COMPUTE_BASE_SWIZZLE_OUTPUT AddrBaseSwizzleOut = {0}; + + AddrBaseSwizzleIn.size = sizeof(ADDR_COMPUTE_BASE_SWIZZLE_INPUT); + AddrBaseSwizzleOut.size = sizeof(ADDR_COMPUTE_BASE_SWIZZLE_OUTPUT); + + AddrBaseSwizzleIn.surfIndex = p_atomic_inc_return(config->info.surf_index) - 1; + AddrBaseSwizzleIn.tileIndex = csio->tileIndex; + AddrBaseSwizzleIn.macroModeIndex = csio->macroModeIndex; + AddrBaseSwizzleIn.pTileInfo = csio->pTileInfo; + AddrBaseSwizzleIn.tileMode = csio->tileMode; + + int r = AddrComputeBaseSwizzle(addrlib, &AddrBaseSwizzleIn, + &AddrBaseSwizzleOut); + if (r != ADDR_OK) + return r; + + assert(AddrBaseSwizzleOut.tileSwizzle <= + u_bit_consecutive(0, sizeof(surf->tile_swizzle) * 8)); + surf->tile_swizzle = AddrBaseSwizzleOut.tileSwizzle; + } + return 0; } /** @@ -644,7 +676,10 @@ static int gfx6_compute_surface(ADDR_HANDLE addrlib, if (level > 0) continue; - gfx6_surface_settings(info, &AddrSurfInfoOut, surf); + r = gfx6_surface_settings(addrlib, info, config, + &AddrSurfInfoOut, surf); + if (r) + return r; } } @@ -676,8 +711,12 @@ static int gfx6_compute_surface(ADDR_HANDLE addrlib, } if (level == 0) { - if (only_stencil) - gfx6_surface_settings(info, &AddrSurfInfoOut, surf); + if (only_stencil) { + r = gfx6_surface_settings(addrlib, info, config, + &AddrSurfInfoOut, surf); + if (r) + return r; + } /* For 2D modes only. */ if (AddrSurfInfoOut.tileMode >= ADDR_TM_2D_TILED_THIN1) { @@ -705,33 +744,6 @@ static int gfx6_compute_surface(ADDR_HANDLE addrlib, surf->htile_size *= 2; surf->is_linear = surf->u.legacy.level[0].mode == RADEON_SURF_MODE_LINEAR_ALIGNED; - - /* Work out tile swizzle. */ - if (config->info.surf_index && - surf->u.legacy.level[0].mode == RADEON_SURF_MODE_2D && - !(surf->flags & (RADEON_SURF_Z_OR_SBUFFER | RADEON_SURF_SHAREABLE)) && - (config->info.samples > 1 || !(surf->flags & RADEON_SURF_SCANOUT))) { - ADDR_COMPUTE_BASE_SWIZZLE_INPUT AddrBaseSwizzleIn = {0}; - ADDR_COMPUTE_BASE_SWIZZLE_OUTPUT AddrBaseSwizzleOut = {0}; - - AddrBaseSwizzleIn.size = sizeof(ADDR_COMPUTE_BASE_SWIZZLE_INPUT); - AddrBaseSwizzleOut.size = sizeof(ADDR_COMPUTE_BASE_SWIZZLE_OUTPUT); - - AddrBaseSwizzleIn.surfIndex = p_atomic_inc_return(config->info.surf_index) - 1; - AddrBaseSwizzleIn.tileIndex = AddrSurfInfoIn.tileIndex; - AddrBaseSwizzleIn.macroModeIndex = AddrSurfInfoOut.macroModeIndex; - AddrBaseSwizzleIn.pTileInfo = AddrSurfInfoOut.pTileInfo; - AddrBaseSwizzleIn.tileMode = AddrSurfInfoOut.tileMode; - - r = AddrComputeBaseSwizzle(addrlib, &AddrBaseSwizzleIn, - &AddrBaseSwizzleOut); - if (r != ADDR_OK) - return r; - - assert(AddrBaseSwizzleOut.tileSwizzle <= - u_bit_consecutive(0, sizeof(surf->tile_swizzle) * 8)); - surf->tile_swizzle = AddrBaseSwizzleOut.tileSwizzle; - } return 0; } -- 2.30.2