Thanks. I'm going to push this shortly.

Marek

On Wed, Mar 5, 2014 at 9:05 AM, Niels Ole Salscheider
<niels_...@salscheider-online.de> wrote:
> Using DMA for reads is much faster.
>
> Signed-off-by: Niels Ole Salscheider <niels_...@salscheider-online.de>
> ---
>  src/gallium/drivers/radeon/r600_buffer_common.c | 74 
> +++++++++++++++++++------
>  1 file changed, 56 insertions(+), 18 deletions(-)
>
> diff --git a/src/gallium/drivers/radeon/r600_buffer_common.c 
> b/src/gallium/drivers/radeon/r600_buffer_common.c
> index 340ebb2..90ca8cb 100644
> --- a/src/gallium/drivers/radeon/r600_buffer_common.c
> +++ b/src/gallium/drivers/radeon/r600_buffer_common.c
> @@ -260,6 +260,42 @@ static void *r600_buffer_transfer_map(struct 
> pipe_context *ctx,
>                 /* At this point, the buffer is always idle (we checked it 
> above). */
>                 usage |= PIPE_TRANSFER_UNSYNCHRONIZED;
>         }
> +       /* Using DMA for larger reads is much faster */
> +       else if ((usage & PIPE_TRANSFER_READ) &&
> +                !(usage & PIPE_TRANSFER_WRITE) &&
> +                (rbuffer->domains == RADEON_DOMAIN_VRAM)) {
> +               unsigned offset;
> +               struct r600_resource *staging = NULL;
> +
> +               u_upload_alloc(rctx->uploader, 0,
> +                              box->width + (box->x % 
> R600_MAP_BUFFER_ALIGNMENT),
> +                              &offset, (struct pipe_resource**)&staging, 
> (void**)&data);
> +
> +               if (staging) {
> +                       data += box->x % R600_MAP_BUFFER_ALIGNMENT;
> +
> +                       /* Copy the staging buffer into the original one. */
> +                       if (rctx->dma_copy(ctx, (struct 
> pipe_resource*)staging, 0,
> +                                                box->x % 
> R600_MAP_BUFFER_ALIGNMENT,
> +                                                0, 0, resource, level, box)) 
> {
> +                               rctx->rings.gfx.flush(rctx, 0);
> +                               if (rctx->rings.dma.cs)
> +                                       rctx->rings.dma.flush(rctx, 0);
> +
> +                               /* Wait for any offloaded CS flush to complete
> +                                * to avoid busy-waiting in the winsys. */
> +                               rctx->ws->cs_sync_flush(rctx->rings.gfx.cs);
> +                               if (rctx->rings.dma.cs)
> +                                       
> rctx->ws->cs_sync_flush(rctx->rings.dma.cs);
> +
> +                               rctx->ws->buffer_wait(staging->buf, 
> RADEON_USAGE_WRITE);
> +                               return r600_buffer_get_transfer(ctx, 
> resource, level, usage, box,
> +                                                               ptransfer, 
> data, staging, offset);
> +                       } else {
> +                               pipe_resource_reference((struct 
> pipe_resource**)&staging, NULL);
> +                       }
> +               }
> +       }
>
>         data = r600_buffer_map_sync_with_rings(rctx, rbuffer, usage);
>         if (!data) {
> @@ -279,24 +315,26 @@ static void r600_buffer_transfer_unmap(struct 
> pipe_context *ctx,
>         struct r600_resource *rbuffer = r600_resource(transfer->resource);
>
>         if (rtransfer->staging) {
> -               struct pipe_resource *dst, *src;
> -               unsigned soffset, doffset, size;
> -               struct pipe_box box;
> -
> -               dst = transfer->resource;
> -               src = &rtransfer->staging->b.b;
> -               size = transfer->box.width;
> -               doffset = transfer->box.x;
> -               soffset = rtransfer->offset + transfer->box.x % 
> R600_MAP_BUFFER_ALIGNMENT;
> -
> -               u_box_1d(soffset, size, &box);
> -
> -               /* Copy the staging buffer into the original one. */
> -               if (!(size % 4) && !(doffset % 4) && !(soffset % 4) &&
> -                   rctx->dma_copy(ctx, dst, 0, doffset, 0, 0, src, 0, &box)) 
> {
> -                       /* DONE. */
> -               } else {
> -                       ctx->resource_copy_region(ctx, dst, 0, doffset, 0, 0, 
> src, 0, &box);
> +               if (rtransfer->transfer.usage & PIPE_TRANSFER_WRITE) {
> +                       struct pipe_resource *dst, *src;
> +                       unsigned soffset, doffset, size;
> +                       struct pipe_box box;
> +
> +                       dst = transfer->resource;
> +                       src = &rtransfer->staging->b.b;
> +                       size = transfer->box.width;
> +                       doffset = transfer->box.x;
> +                       soffset = rtransfer->offset + transfer->box.x % 
> R600_MAP_BUFFER_ALIGNMENT;
> +
> +                       u_box_1d(soffset, size, &box);
> +
> +                       /* Copy the staging buffer into the original one. */
> +                       if (!(size % 4) && !(doffset % 4) && !(soffset % 4) &&
> +                           rctx->dma_copy(ctx, dst, 0, doffset, 0, 0, src, 
> 0, &box)) {
> +                               /* DONE. */
> +                       } else {
> +                               ctx->resource_copy_region(ctx, dst, 0, 
> doffset, 0, 0, src, 0, &box);
> +                       }
>                 }
>                 pipe_resource_reference((struct 
> pipe_resource**)&rtransfer->staging, NULL);
>         }
> --
> 1.9.0
>
> _______________________________________________
> 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

Reply via email to