[Mesa-dev] [PATCH] nvc0: tune PREFER_BLIT_BASED_TEXTURE_TRANSFER capability

[Ilia Mirkin]

Amended locally to put this back into the "supported caps" section
(but at the bottom), rather than the non-boolean caps section. Will
push this out shortly.

On Wed, Jul 1, 2015 at 10:36 PM, Alexandre Courbot <acourbot at nvidia.com> wrote:
> Prefer blit-based texture transfers only if the chip has dedicated VRAM
> since it would translate to a copy into the same memory on shared-memory
> chips.
>
> Signed-off-by: Alexandre Courbot <acourbot at nvidia.com>
> Reported-by: Ilia Mirkin <imirkin at alum.mit.edu>
> ---
>  src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
> index 95e246b98669..1a8ce3cd68da 100644
> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
> +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
> @@ -120,6 +120,8 @@ nvc0_screen_get_param(struct pipe_screen *pscreen, enum pipe_cap param)
>        return PIPE_QUIRK_TEXTURE_BORDER_COLOR_SWIZZLE_NV50;
>     case PIPE_CAP_ENDIANNESS:
>        return PIPE_ENDIAN_LITTLE;
> +   case PIPE_CAP_PREFER_BLIT_BASED_TEXTURE_TRANSFER:
> +      return nouveau_screen(pscreen)->vram_domain & NOUVEAU_BO_VRAM ? 1 : 0;
>
>     /* supported caps */
>     case PIPE_CAP_TEXTURE_MIRROR_CLAMP:
> @@ -163,7 +165,6 @@ nvc0_screen_get_param(struct pipe_screen *pscreen, enum pipe_cap param)
>     case PIPE_CAP_USER_CONSTANT_BUFFERS:
>     case PIPE_CAP_USER_INDEX_BUFFERS:
>     case PIPE_CAP_USER_VERTEX_BUFFERS:
> -   case PIPE_CAP_PREFER_BLIT_BASED_TEXTURE_TRANSFER:
>     case PIPE_CAP_TEXTURE_QUERY_LOD:
>     case PIPE_CAP_SAMPLE_SHADING:
>     case PIPE_CAP_TEXTURE_GATHER_OFFSETS:
> --
> 2.4.4
>