From baa4cf9b8ed25630d795926ff2e1dfae9ae955b2 Mon Sep 17 00:00:00 2001 From: Jason Ekstrand Date: Thu, 25 Oct 2018 17:53:23 -0500 Subject: [PATCH] iris: Implement set_global_binding All this has to do is track which globals are bound and make sure the batch references them every time. We use A64 messages to access them so there are no binding table entries to manage. Reviewed-by: Kenneth Graunke Part-of: --- src/gallium/drivers/iris/iris_context.h | 4 +++ src/gallium/drivers/iris/iris_state.c | 35 +++++++++++++++++++++++++ 2 files changed, 39 insertions(+) diff --git a/src/gallium/drivers/iris/iris_context.h b/src/gallium/drivers/iris/iris_context.h index 361c63cc2ad..e34e3e93475 100644 --- a/src/gallium/drivers/iris/iris_context.h +++ b/src/gallium/drivers/iris/iris_context.h @@ -49,6 +49,7 @@ struct blorp_params; #define IRIS_MAX_SSBOS 16 #define IRIS_MAX_VIEWPORTS 16 #define IRIS_MAX_CLIP_PLANES 8 +#define IRIS_MAX_GLOBAL_BINDINGS 32 enum iris_param_domain { BRW_PARAM_DOMAIN_BUILTIN = 0, @@ -697,6 +698,9 @@ struct iris_context { /** Do any samplers need border color? One bit per shader stage. */ uint8_t need_border_colors; + /** Global resource bindings */ + struct pipe_resource *global_bindings[IRIS_MAX_GLOBAL_BINDINGS]; + struct pipe_stream_output_target *so_target[PIPE_MAX_SO_BUFFERS]; bool streamout_active; diff --git a/src/gallium/drivers/iris/iris_state.c b/src/gallium/drivers/iris/iris_state.c index 83fb9d8c39f..5837d219356 100644 --- a/src/gallium/drivers/iris/iris_state.c +++ b/src/gallium/drivers/iris/iris_state.c @@ -2853,6 +2853,31 @@ iris_set_compute_resources(struct pipe_context *ctx, assert(count == 0); } +static void +iris_set_global_binding(struct pipe_context *ctx, + unsigned start_slot, unsigned count, + struct pipe_resource **resources, + uint32_t **handles) +{ + struct iris_context *ice = (struct iris_context *) ctx; + + assert(start_slot + count <= IRIS_MAX_GLOBAL_BINDINGS); + for (unsigned i = 0; i < count; i++) { + if (resources && resources[i]) { + pipe_resource_reference(&ice->state.global_bindings[start_slot + i], + resources[i]); + struct iris_resource *res = (void *) resources[i]; + uint64_t addr = res->bo->gtt_offset; + memcpy(handles[i], &addr, sizeof(addr)); + } else { + pipe_resource_reference(&ice->state.global_bindings[start_slot + i], + NULL); + } + } + + ice->state.stage_dirty |= IRIS_STAGE_DIRTY_BINDINGS_CS; +} + /** * The pipe->set_tess_state() driver hook. */ @@ -6688,6 +6713,15 @@ iris_upload_gpgpu_walker(struct iris_context *ice, } } + for (unsigned i = 0; i < IRIS_MAX_GLOBAL_BINDINGS; i++) { + struct pipe_resource *res = ice->state.global_bindings[i]; + if (!res) + continue; + + iris_use_pinned_bo(batch, iris_resource_bo(res), + true, IRIS_DOMAIN_NONE); + } + if (stage_dirty & (IRIS_STAGE_DIRTY_SAMPLER_STATES_CS | IRIS_STAGE_DIRTY_BINDINGS_CS | IRIS_STAGE_DIRTY_CONSTANTS_CS | @@ -7719,6 +7753,7 @@ genX(init_state)(struct iris_context *ice) ctx->set_shader_images = iris_set_shader_images; ctx->set_sampler_views = iris_set_sampler_views; ctx->set_compute_resources = iris_set_compute_resources; + ctx->set_global_binding = iris_set_global_binding; ctx->set_tess_state = iris_set_tess_state; ctx->set_framebuffer_state = iris_set_framebuffer_state; ctx->set_polygon_stipple = iris_set_polygon_stipple; -- 2.30.2