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

[Marek Olšák]

Please apply this behavior to r600g only. See rscreen->chip_class.

Marek

On Fri, Jul 31, 2015 at 12:06 AM, Zoltan Gilian <zoltan.gilian at gmail.com> 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.
> ---
>  src/gallium/drivers/radeon/r600_texture.c | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/src/gallium/drivers/radeon/r600_texture.c b/src/gallium/drivers/radeon/r600_texture.c
> index 8169e05..8979a98 100644
> --- a/src/gallium/drivers/radeon/r600_texture.c
> +++ b/src/gallium/drivers/radeon/r600_texture.c
> @@ -706,6 +706,7 @@ static unsigned r600_choose_tiling(struct r600_common_screen *rscreen,
>                                    const struct pipe_resource *templ)
>  {
>         const struct util_format_description *desc = util_format_description(templ->format);
> +       bool force_tiling = templ->flags & R600_RESOURCE_FLAG_FORCE_TILING;
>
>         /* MSAA resources must be 2D tiled. */
>         if (templ->nr_samples > 1)
> @@ -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->target == PIPE_TEXTURE_2D ||
> +                templ->target == PIPE_TEXTURE_3D) &&
> +           (templ->bind & PIPE_BIND_COMPUTE_RESOURCE))
> +               force_tiling = true;
> +
>         /* Handle common candidates for the linear mode.
>          * Compressed textures must always be tiled. */
> -       if (!(templ->flags & R600_RESOURCE_FLAG_FORCE_TILING) &&
> -           !util_format_is_compressed(templ->format)) {
> +       if (!force_tiling && !util_format_is_compressed(templ->format)) {
>                 /* Not everything can be linear, so we cannot enforce it
>                  * for all textures. */
>                 if ((rscreen->debug_flags & DBG_NO_TILING) &&
> --
> 2.4.6
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev