virgl: add virgl_shader_binding_state

virgl_shader_binding_state will be used to manage all per-stage
shader bindings.  For now, it manages only sampler views.

This replaces virgl_textures_info and fixes some issues

 - start_slot is now honored
 - views outside of [start_slot, slart_slot+count) are unmodified
 - views are released when the context is destroyed

Signed-off-by: Chia-I Wu <olvaffe@gmail.com>
Reviewed-by: Alexandros Frantzis <alexandros.frantzis@collabora.com>
This commit is contained in:
Chia-I Wu 2019-05-16 13:32:18 -07:00
parent 30314270d4
commit 2e21d66d7a
2 changed files with 44 additions and 43 deletions

View file

@ -91,17 +91,16 @@ static void virgl_attach_res_sampler_views(struct virgl_context *vctx,
enum pipe_shader_type shader_type)
{
struct virgl_winsys *vws = virgl_screen(vctx->base.screen)->vws;
struct virgl_textures_info *tinfo = &vctx->samplers[shader_type];
const struct virgl_shader_binding_state *binding =
&vctx->shader_bindings[shader_type];
uint32_t remaining_mask = binding->view_enabled_mask;
struct virgl_resource *res;
uint32_t remaining_mask = tinfo->enabled_mask;
unsigned i;
while (remaining_mask) {
i = u_bit_scan(&remaining_mask);
assert(tinfo->views[i]);
res = virgl_resource(tinfo->views[i]->base.texture);
if (res)
vws->emit_res(vws, vctx->cbuf, res->hw_res, FALSE);
while (remaining_mask) {
int i = u_bit_scan(&remaining_mask);
assert(binding->views[i] && binding->views[i]->texture);
res = virgl_resource(binding->views[i]->texture);
vws->emit_res(vws, vctx->cbuf, res->hw_res, FALSE);
}
}
@ -809,39 +808,22 @@ static void virgl_set_sampler_views(struct pipe_context *ctx,
struct pipe_sampler_view **views)
{
struct virgl_context *vctx = virgl_context(ctx);
int i;
uint32_t disable_mask = ~((1ull << num_views) - 1);
struct virgl_textures_info *tinfo = &vctx->samplers[shader_type];
uint32_t new_mask = 0;
uint32_t remaining_mask;
struct virgl_shader_binding_state *binding =
&vctx->shader_bindings[shader_type];
remaining_mask = tinfo->enabled_mask & disable_mask;
while (remaining_mask) {
i = u_bit_scan(&remaining_mask);
assert(tinfo->views[i]);
pipe_sampler_view_reference((struct pipe_sampler_view **)&tinfo->views[i], NULL);
}
for (i = 0; i < num_views; i++) {
struct virgl_sampler_view *grview = virgl_sampler_view(views[i]);
if (views[i] == (struct pipe_sampler_view *)tinfo->views[i])
continue;
if (grview) {
new_mask |= 1 << i;
pipe_sampler_view_reference((struct pipe_sampler_view **)&tinfo->views[i], views[i]);
binding->view_enabled_mask &= ~u_bit_consecutive(start_slot, num_views);
for (unsigned i = 0; i < num_views; i++) {
unsigned idx = start_slot + i;
if (views && views[i]) {
pipe_sampler_view_reference(&binding->views[idx], views[i]);
binding->view_enabled_mask |= 1 << idx;
} else {
pipe_sampler_view_reference((struct pipe_sampler_view **)&tinfo->views[i], NULL);
disable_mask |= 1 << i;
pipe_sampler_view_reference(&binding->views[idx], NULL);
}
}
tinfo->enabled_mask &= ~disable_mask;
tinfo->enabled_mask |= new_mask;
virgl_encode_set_sampler_views(vctx, shader_type, start_slot, num_views, tinfo->views);
virgl_encode_set_sampler_views(vctx, shader_type,
start_slot, num_views, (struct virgl_sampler_view **)binding->views);
virgl_attach_res_sampler_views(vctx, shader_type);
}
@ -1162,17 +1144,35 @@ static void virgl_launch_grid(struct pipe_context *ctx,
virgl_encode_launch_grid(vctx, info);
}
static void
virgl_release_shader_binding(struct virgl_context *vctx,
enum pipe_shader_type shader_type)
{
struct virgl_shader_binding_state *binding =
&vctx->shader_bindings[shader_type];
while (binding->view_enabled_mask) {
int i = u_bit_scan(&binding->view_enabled_mask);
pipe_sampler_view_reference(
(struct pipe_sampler_view **)&binding->views[i], NULL);
}
}
static void
virgl_context_destroy( struct pipe_context *ctx )
{
struct virgl_context *vctx = virgl_context(ctx);
struct virgl_screen *rs = virgl_screen(ctx->screen);
enum pipe_shader_type shader_type;
vctx->framebuffer.zsbuf = NULL;
vctx->framebuffer.nr_cbufs = 0;
virgl_encoder_destroy_sub_ctx(vctx, vctx->hw_sub_ctx_id);
virgl_flush_eq(vctx, vctx, NULL);
for (shader_type = 0; shader_type < PIPE_SHADER_TYPES; shader_type++)
virgl_release_shader_binding(vctx, shader_type);
rs->vws->cmd_buf_destroy(vctx->cbuf);
if (vctx->uploader)
u_upload_destroy(vctx->uploader);

View file

@ -46,22 +46,23 @@ struct virgl_so_target {
uint32_t handle;
};
struct virgl_textures_info {
struct virgl_sampler_view *views[16];
uint32_t enabled_mask;
};
struct virgl_rasterizer_state {
struct pipe_rasterizer_state rs;
uint32_t handle;
};
struct virgl_shader_binding_state {
struct pipe_sampler_view *views[16];
uint32_t view_enabled_mask;
};
struct virgl_context {
struct pipe_context base;
struct virgl_cmd_buf *cbuf;
unsigned cbuf_initial_cdw;
struct virgl_textures_info samplers[PIPE_SHADER_TYPES];
struct virgl_shader_binding_state shader_bindings[PIPE_SHADER_TYPES];
struct virgl_vertex_elements_state *vertex_elements;
struct pipe_framebuffer_state framebuffer;