[Mesa-dev] [PATCH] r600, compute: force tiling on 2D and 3D texture compute resources

[Michel Dänzer]

On 31.07.2015 17:39, Zoltan Gilian wrote:
> To circumvent a problem occuring when LINEAR_ALIGNED array mode is
> selected on a TEXTURE_2D RAT.
> This configuration causes MEM_RAT STORE_TYPED to write to incorrect
> locations.

[...]

> @@ -715,10 +716,15 @@ static unsigned r600_choose_tiling(struct r600_common_screen *rscreen,
>  	if (templ->flags & R600_RESOURCE_FLAG_TRANSFER)
>  		return RADEON_SURF_MODE_LINEAR_ALIGNED;
>  
> +	/* Force tiling on TEXTURE_2D and TEXTURE_3D compute resources. */
> +	if ((templ->bind & PIPE_BIND_COMPUTE_RESOURCE) &&
> +		(templ->target == PIPE_TEXTURE_2D ||
> +		 templ->target == PIPE_TEXTURE_3D))

The indentation of the second and third line of the if statement should
be better aligned to the first line. With that fixed,

Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer