From ec4851253bbf7fd7d11c5570f19f9733a885e471 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Marek=20Ol=C5=A1=C3=A1k?= Date: Sat, 24 Dec 2011 11:43:10 +0100 Subject: [PATCH] r300g: mapping buffers for read should be unsynchronized The GPU never uses them for write. --- src/gallium/drivers/r300/r300_screen_buffer.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/r300/r300_screen_buffer.c b/src/gallium/drivers/r300/r300_screen_buffer.c index a8392d2..2e9e7d7 100644 --- a/src/gallium/drivers/r300/r300_screen_buffer.c +++ b/src/gallium/drivers/r300/r300_screen_buffer.c @@ -107,13 +107,21 @@ r300_buffer_transfer_map( struct pipe_context *pipe, struct radeon_winsys *rws = r300screen->rws; struct r300_resource *rbuf = r300_resource(transfer->resource); uint8_t *map; + enum pipe_transfer_usage usage; if (rbuf->b.user_ptr) return (uint8_t *) rbuf->b.user_ptr + transfer->box.x; if (rbuf->constant_buffer) return (uint8_t *) rbuf->constant_buffer + transfer->box.x; - map = rws->buffer_map(rbuf->buf, r300->cs, transfer->usage); + /* Buffers are never used for write, therefore mapping for read can be + * unsynchronized. */ + usage = transfer->usage; + if (!(usage & PIPE_TRANSFER_WRITE)) { + usage |= PIPE_TRANSFER_UNSYNCHRONIZED; + } + + map = rws->buffer_map(rbuf->buf, r300->cs, usage); if (map == NULL) return NULL; -- 2.7.4