[Mesa-dev] [PATCH] ac/surface: remove the overallocation workaround for Vega12

Samuel Pitoiset samuel.pitoiset at gmail.com
Thu Nov 8 07:48:50 UTC 2018


Reviewed-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>

On 11/8/18 3:13 AM, Marek Olšák wrote:
> From: Marek Olšák <marek.olsak at amd.com>
> 
> not needed anymore (probably since the tile_swizzle fix)
> ---
>   src/amd/common/ac_surface.c | 4 ----
>   1 file changed, 4 deletions(-)
> 
> diff --git a/src/amd/common/ac_surface.c b/src/amd/common/ac_surface.c
> index 1f7e2344625..edd710a968c 100644
> --- a/src/amd/common/ac_surface.c
> +++ b/src/amd/common/ac_surface.c
> @@ -1587,24 +1587,20 @@ static int gfx9_compute_surface(ADDR_HANDLE addrlib,
>   		case ADDR_SW_4KB_Z_X:
>   		case ADDR_SW_64KB_Z_X:
>   		case ADDR_SW_VAR_Z_X:
>   			surf->micro_tile_mode = RADEON_MICRO_MODE_DEPTH;
>   			break;
>   
>   		default:
>   			assert(0);
>   	}
>   
> -	/* Temporary workaround to prevent VM faults and hangs. */
> -	if (info->family == CHIP_VEGA12)
> -		surf->fmask_size *= 8;
> -
>   	return 0;
>   }
>   
>   int ac_compute_surface(ADDR_HANDLE addrlib, const struct radeon_info *info,
>   		       const struct ac_surf_config *config,
>   		       enum radeon_surf_mode mode,
>   		       struct radeon_surf *surf)
>   {
>   	int r;
>   
> 


More information about the mesa-dev mailing list