From 1ffe77e7bb2486ea74cda077ed2a9622b758395c Mon Sep 17 00:00:00 2001 From: Marek Olšák Date: Sat, 16 Jul 2016 21:19:48 +0200 Subject: gallium: split transfer_inline_write into buffer and texture callbacks MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit to reduce the call indirections with u_resource_vtbl. The worst call tree you could get was: - u_transfer_inline_write_vtbl - u_default_transfer_inline_write - u_transfer_map_vtbl - driver_transfer_map - u_transfer_unmap_vtbl - driver_transfer_unmap That's 6 indirect calls. Some drivers only had 5. The goal is to have 1 indirect call for drivers that care. The resource type can be determined statically at most call sites. The new interface is: pipe_context::buffer_subdata(ctx, resource, usage, offset, size, data) pipe_context::texture_subdata(ctx, resource, level, usage, box, data, stride, layer_stride) v2: fix whitespace, correct ilo's behavior Reviewed-by: Nicolai Hähnle Acked-by: Roland Scheidegger --- src/gallium/tests/graw/gs-test.c | 45 +++++++++++++--------------------------- 1 file changed, 14 insertions(+), 31 deletions(-) (limited to 'src/gallium/tests/graw/gs-test.c') diff --git a/src/gallium/tests/graw/gs-test.c b/src/gallium/tests/graw/gs-test.c index c680b62eaaa..00fb59161a8 100644 --- a/src/gallium/tests/graw/gs-test.c +++ b/src/gallium/tests/graw/gs-test.c @@ -149,7 +149,6 @@ static float constants2[] = static void init_fs_constbuf( void ) { struct pipe_resource templat; - struct pipe_box box; templat.target = PIPE_BUFFER; templat.format = PIPE_FORMAT_R8_UNORM; @@ -169,34 +168,18 @@ static void init_fs_constbuf( void ) exit(4); { - u_box_2d(0,0,sizeof(constants1),1, &box); - - ctx->transfer_inline_write(ctx, - constbuf1, - 0, - PIPE_TRANSFER_WRITE, - &box, - constants1, - sizeof constants1, - sizeof constants1); - + ctx->buffer_subdata(ctx, constbuf1, + PIPE_TRANSFER_WRITE, + 0, sizeof(constants1), constants1); pipe_set_constant_buffer(ctx, PIPE_SHADER_GEOMETRY, 0, constbuf1); } { - u_box_2d(0,0,sizeof(constants2),1, &box); - - ctx->transfer_inline_write(ctx, - constbuf2, - 0, - PIPE_TRANSFER_WRITE, - &box, - constants2, - sizeof constants2, - sizeof constants2); - + ctx->buffer_subdata(ctx, constbuf2, + PIPE_TRANSFER_WRITE, + 0, sizeof(constants2), constants2); pipe_set_constant_buffer(ctx, PIPE_SHADER_GEOMETRY, 1, @@ -418,14 +401,14 @@ static void init_tex( void ) u_box_2d(0,0,SIZE,SIZE, &box); - ctx->transfer_inline_write(ctx, - samptex, - 0, - PIPE_TRANSFER_WRITE, - &box, - tex2d, - sizeof tex2d[0], - sizeof tex2d); + ctx->texture_subdata(ctx, + samptex, + 0, + PIPE_TRANSFER_WRITE, + &box, + tex2d, + sizeof tex2d[0], + sizeof tex2d); /* Possibly read back & compare against original data: */ -- cgit v1.2.3