On Wed, Apr 2, 2014 at 9:09 AM, Leo Liu <leo....@amd.com> wrote:
> From: Leo Liu <leo....@amd.com>
>
> This reverts commit 96e8b916a7a39a9ba58e92d1ad77b5501de63ac7.
> In the case of VCE encoding with raw YUV file, CPU load directly
> to VRAM is faster than combination of CPU writing to GTT and
> then blit to VRAM with GPU.
Why was the original patch applied? I just want to make sure it
wasn't a bug fix for avoiding a sigbus or something similar if we ran
out of cpu visible vram.
Alex
> ---
> src/gallium/drivers/radeon/r600_texture.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/gallium/drivers/radeon/r600_texture.c
> b/src/gallium/drivers/radeon/r600_texture.c
> index 45a9508..3dfddca 100644
> --- a/src/gallium/drivers/radeon/r600_texture.c
> +++ b/src/gallium/drivers/radeon/r600_texture.c
> @@ -928,8 +928,8 @@ static void *r600_texture_transfer_map(struct
> pipe_context *ctx,
> if (rtex->surface.level[level].mode >= RADEON_SURF_MODE_1D)
> use_staging_texture = TRUE;
>
> - /* Untiled buffers in VRAM, which is slow for CPU reads and writes */
> - if (!(usage & PIPE_TRANSFER_MAP_DIRECTLY) &&
> + /* Untiled buffers in VRAM, which is slow for CPU reads */
> + if ((usage & PIPE_TRANSFER_READ) && !(usage &
> PIPE_TRANSFER_MAP_DIRECTLY) &&
> (rtex->resource.domains == RADEON_DOMAIN_VRAM)) {
> use_staging_texture = TRUE;
> }
> --
> 1.8.1.2
>
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
