[virglrenderer-devel] [PATCH 5/7] renderer: add image support. (v2)
Dave Airlie
airlied at gmail.com
Fri Jul 20 04:50:40 UTC 2018
From: Dave Airlie <airlied at redhat.com>
This adds support for tracking images, and binding them
to the GL context.
includes:
vrend_renderer: create texture when TBO is used as image
vrend_renderer: specify correct access to glBindImageTexture
v2:
vrend_renderer: invert glBindImageTexture layered logic
Co-authors: Gurchetan Singh <gurchetansingh at chromium.org>
Signed-off-by: Dave Airlie <airlied at redhat.com>
---
src/virgl_hw.h | 2 +
src/virgl_protocol.h | 12 ++++
src/vrend_decode.c | 34 ++++++++++++
src/vrend_renderer.c | 154 +++++++++++++++++++++++++++++++++++++++++++++++++++
src/vrend_renderer.h | 6 ++
5 files changed, 208 insertions(+)
diff --git a/src/virgl_hw.h b/src/virgl_hw.h
index 4add191..6db90f5 100644
--- a/src/virgl_hw.h
+++ b/src/virgl_hw.h
@@ -315,6 +315,8 @@ struct virgl_caps_v2 {
uint32_t max_vertex_attrib_stride;
uint32_t max_shader_buffer_frag_compute;
uint32_t max_shader_buffer_other_stages;
+ uint32_t max_shader_image_frag_compute;
+ uint32_t max_shader_image_other_stages;
};
union virgl_caps {
diff --git a/src/virgl_protocol.h b/src/virgl_protocol.h
index 6c73dfb..dc1557c 100644
--- a/src/virgl_protocol.h
+++ b/src/virgl_protocol.h
@@ -86,6 +86,7 @@ enum virgl_context_cmd {
VIRGL_CCMD_SET_TESS_STATE,
VIRGL_CCMD_SET_MIN_SAMPLES,
VIRGL_CCMD_SET_SHADER_BUFFERS,
+ VIRGL_CCMD_SET_SHADER_IMAGES,
};
/*
@@ -500,4 +501,15 @@ enum virgl_context_cmd {
#define VIRGL_SET_SHADER_BUFFER_LENGTH(x) ((x) * VIRGL_SET_SHADER_BUFFER_ELEMENT_SIZE + 3)
#define VIRGL_SET_SHADER_BUFFER_RES_HANDLE(x) ((x) * VIRGL_SET_SHADER_BUFFER_ELEMENT_SIZE + 4)
+/* set shader images */
+#define VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE 5
+#define VIRGL_SET_SHADER_IMAGE_SIZE(x) (VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE * (x)) + 2
+#define VIRGL_SET_SHADER_IMAGE_SHADER_TYPE 1
+#define VIRGL_SET_SHADER_IMAGE_START_SLOT 2
+#define VIRGL_SET_SHADER_IMAGE_FORMAT(x) ((x) * VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE + 2)
+#define VIRGL_SET_SHADER_IMAGE_ACCESS(x) ((x) * VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE + 3)
+#define VIRGL_SET_SHADER_IMAGE_LAYER_OFFSET(x) ((x) * VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE + 4)
+#define VIRGL_SET_SHADER_IMAGE_LEVEL_SIZE(x) ((x) * VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE + 5)
+#define VIRGL_SET_SHADER_IMAGE_RES_HANDLE(x) ((x) * VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE + 6)
+
#endif
diff --git a/src/vrend_decode.c b/src/vrend_decode.c
index 11fe149..09f2ef9 100644
--- a/src/vrend_decode.c
+++ b/src/vrend_decode.c
@@ -1107,6 +1107,37 @@ static int vrend_decode_set_shader_buffers(struct vrend_decode_ctx *ctx, uint16_
return 0;
}
+static int vrend_decode_set_shader_images(struct vrend_decode_ctx *ctx, uint16_t length)
+{
+ int num_images;
+ uint32_t shader_type, start_slot;
+ if (length < 2)
+ return EINVAL;
+
+ num_images = (length - 2) / VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE;
+ shader_type = get_buf_entry(ctx, VIRGL_SET_SHADER_IMAGE_SHADER_TYPE);
+ start_slot = get_buf_entry(ctx, VIRGL_SET_SHADER_IMAGE_START_SLOT);
+ if (shader_type >= PIPE_SHADER_TYPES)
+ return EINVAL;
+
+ if (num_images < 1) {
+ return 0;
+ }
+ if (start_slot + num_images > PIPE_MAX_SHADER_IMAGES)
+ return EINVAL;
+
+ for (int i = 0; i < num_images; i++) {
+ uint32_t format = get_buf_entry(ctx, i * VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE + 3);
+ uint32_t access = get_buf_entry(ctx, i * VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE + 4);
+ uint32_t layer_offset = get_buf_entry(ctx, i * VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE + 5);
+ uint32_t level_size = get_buf_entry(ctx, i * VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE + 6);
+ uint32_t handle = get_buf_entry(ctx, i * VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE + 7);
+ vrend_set_single_image_view(ctx->grctx, shader_type, start_slot + i, format, access,
+ layer_offset, level_size, handle);
+ }
+ return 0;
+}
+
static int vrend_decode_set_streamout_targets(struct vrend_decode_ctx *ctx,
uint16_t length)
{
@@ -1338,6 +1369,9 @@ int vrend_decode_block(uint32_t ctx_id, uint32_t *block, int ndw)
case VIRGL_CCMD_SET_SHADER_BUFFERS:
ret = vrend_decode_set_shader_buffers(gdctx, len);
break;
+ case VIRGL_CCMD_SET_SHADER_IMAGES:
+ ret = vrend_decode_set_shader_images(gdctx, len);
+ break;
default:
ret = EINVAL;
}
diff --git a/src/vrend_renderer.c b/src/vrend_renderer.c
index 4086abd..8939a92 100644
--- a/src/vrend_renderer.c
+++ b/src/vrend_renderer.c
@@ -121,6 +121,7 @@ struct global_renderer_state {
bool have_texture_view;
bool have_copy_image;
bool have_ssbo;
+ bool have_images;
/* these appeared broken on at least one driver */
bool use_explicit_locations;
@@ -169,6 +170,9 @@ struct vrend_linked_shader_program {
GLuint clip_locs[8];
+ uint32_t images_used_mask[PIPE_SHADER_TYPES];
+ GLuint *img_locs[PIPE_SHADER_TYPES];
+
uint32_t ssbo_used_mask[PIPE_SHADER_TYPES];
GLuint *ssbo_locs[PIPE_SHADER_TYPES];
};
@@ -248,6 +252,24 @@ struct vrend_sampler_view {
struct vrend_resource *texture;
};
+struct vrend_image_view {
+ GLuint id;
+ GLenum access;
+ GLenum format;
+ union {
+ struct {
+ unsigned first_layer:16; /**< first layer to use for array textures */
+ unsigned last_layer:16; /**< last layer to use for array textures */
+ unsigned level:8; /**< mipmap level to use */
+ } tex;
+ struct {
+ unsigned offset; /**< offset in bytes */
+ unsigned size; /**< size of the accessible sub-range in bytes */
+ } buf;
+ } u;
+ struct vrend_resource *texture;
+};
+
struct vrend_ssbo {
struct vrend_resource *res;
unsigned buffer_size;
@@ -326,6 +348,7 @@ struct vrend_sub_context {
bool shader_dirty;
bool sampler_state_dirty;
bool stencil_state_dirty;
+ bool image_state_dirty;
uint32_t long_shader_in_progress_handle[PIPE_SHADER_TYPES];
struct vrend_shader_selector *shaders[PIPE_SHADER_TYPES];
@@ -393,6 +416,9 @@ struct vrend_sub_context {
uint32_t cond_render_q_id;
GLenum cond_render_gl_mode;
+ struct vrend_image_view image_views[PIPE_SHADER_TYPES][PIPE_MAX_SHADER_IMAGES];
+ uint32_t images_used_mask[PIPE_SHADER_TYPES];
+
struct vrend_ssbo ssbo[PIPE_SHADER_TYPES][PIPE_MAX_SHADER_BUFFERS];
uint32_t ssbo_used_mask[PIPE_SHADER_TYPES];
};
@@ -985,6 +1011,32 @@ static void bind_ssbo_locs(struct vrend_linked_shader_program *sprog,
sprog->ssbo_used_mask[id] = sprog->ss[id]->sel->sinfo.ssbo_used_mask;
}
+static void bind_image_locs(struct vrend_linked_shader_program *sprog,
+ int id)
+{
+ int i;
+ char name[32];
+ if (sprog->ss[id]->sel->sinfo.images_used_mask) {
+ uint32_t mask = sprog->ss[id]->sel->sinfo.images_used_mask;
+ int nsamp = util_bitcount(sprog->ss[id]->sel->sinfo.images_used_mask);
+ int index;
+ sprog->img_locs[id] = calloc(nsamp, sizeof(uint32_t));
+ if (sprog->img_locs[id]) {
+ const char *prefix = pipe_shader_to_prefix(id);
+ index = 0;
+ while(mask) {
+ i = u_bit_scan(&mask);
+ snprintf(name, 32, "%simg%d", prefix, i);
+ sprog->img_locs[id][index] = glGetUniformLocation(sprog->id, name);
+ index++;
+ }
+ }
+ } else {
+ sprog->img_locs[id] = NULL;
+ }
+ sprog->images_used_mask[id] = sprog->ss[id]->sel->sinfo.images_used_mask;
+}
+
static struct vrend_linked_shader_program *add_shader_program(struct vrend_context *ctx,
struct vrend_shader *vs,
struct vrend_shader *fs,
@@ -1130,6 +1182,7 @@ static struct vrend_linked_shader_program *add_shader_program(struct vrend_conte
bind_sampler_locs(sprog, id);
bind_const_locs(sprog, id);
bind_ubo_locs(sprog, id);
+ bind_image_locs(sprog, id);
bind_ssbo_locs(sprog, id);
}
@@ -1199,6 +1252,7 @@ static void vrend_destroy_program(struct vrend_linked_shader_program *ent)
free(ent->shadow_samp_add_locs[i]);
free(ent->samp_locs[i]);
free(ent->ssbo_locs[i]);
+ free(ent->img_locs[i]);
free(ent->const_locs[i]);
free(ent->ubo_locs[i]);
}
@@ -2286,6 +2340,34 @@ void vrend_set_num_sampler_views(struct vrend_context *ctx,
ctx->sub->views[shader_type].num_views = last_slot;
}
+void vrend_set_single_image_view(struct vrend_context *ctx,
+ uint32_t shader_type,
+ int index,
+ uint32_t format, uint32_t access,
+ uint32_t layer_offset, uint32_t level_size,
+ uint32_t handle)
+{
+ struct vrend_image_view *iview = &ctx->sub->image_views[shader_type][index];
+ struct vrend_resource *res;
+ if (handle) {
+ res = vrend_renderer_ctx_res_lookup(ctx, handle);
+ if (!res) {
+ report_context_error(ctx, VIRGL_ERROR_CTX_ILLEGAL_RESOURCE, handle);
+ return;
+ }
+ iview->texture = res;
+ iview->format = tex_conv_table[format].internalformat;
+ iview->access = access;
+ iview->u.buf.offset = layer_offset;
+ iview->u.buf.size = level_size;
+ ctx->sub->images_used_mask[shader_type] |= (1 << index);
+ } else {
+ iview->texture = NULL;
+ iview->format = 0;
+ ctx->sub->images_used_mask[shader_type] &= ~(1 << index);
+ }
+}
+
void vrend_set_single_ssbo(struct vrend_context *ctx,
uint32_t shader_type,
int index,
@@ -3231,6 +3313,63 @@ static void vrend_draw_bind_ssbo_shader(struct vrend_context *ctx, int shader_ty
}
}
+static void vrend_draw_bind_images_shader(struct vrend_context *ctx, int shader_type)
+{
+ struct vrend_image_view *iview;
+ uint32_t mask;
+ uint32_t tex_id;
+ GLenum access;
+
+ if (!ctx->sub->images_used_mask[shader_type])
+ return;
+
+ if (!ctx->sub->prog->img_locs[shader_type])
+ return;
+
+ mask = ctx->sub->images_used_mask[shader_type];
+ while (mask) {
+ unsigned i = u_bit_scan(&mask);
+
+ iview = &ctx->sub->image_views[shader_type][i];
+ tex_id = iview->texture->id;
+ if (iview->texture->is_buffer) {
+ if (!iview->texture->tbo_tex_id)
+ glGenTextures(1, &iview->texture->tbo_tex_id);
+
+ glBindBufferARB(GL_TEXTURE_BUFFER, iview->texture->id);
+ glBindTexture(GL_TEXTURE_BUFFER, iview->texture->tbo_tex_id);
+ glTexBuffer(GL_TEXTURE_BUFFER, iview->format, iview->texture->id);
+ tex_id = iview->texture->tbo_tex_id;
+ }
+
+ glUniform1i(ctx->sub->prog->img_locs[shader_type][i], i);
+ GLboolean layered = !((iview->texture->base.array_size > 1 ||
+ iview->texture->base.depth0 > 1) && (iview->u.tex.first_layer == iview->u.tex.last_layer));
+
+ switch (iview->access) {
+ case PIPE_IMAGE_ACCESS_READ:
+ access = GL_READ_ONLY;
+ break;
+ case PIPE_IMAGE_ACCESS_WRITE:
+ access = GL_WRITE_ONLY;
+ break;
+ case PIPE_IMAGE_ACCESS_READ_WRITE:
+ access = GL_READ_WRITE;
+ break;
+ default:
+ fprintf(stderr, "Invalid access specified\n");
+ return;
+ }
+
+ glBindImageTexture(i, tex_id,
+ iview->u.tex.level,
+ layered,
+ iview->u.tex.first_layer,
+ access,
+ iview->format);
+ }
+}
+
static void vrend_draw_bind_objects(struct vrend_context *ctx, bool new_program)
{
int ubo_id = 0, sampler_id = 0;
@@ -3238,6 +3377,7 @@ static void vrend_draw_bind_objects(struct vrend_context *ctx, bool new_program)
vrend_draw_bind_ubo_shader(ctx, shader_type, &ubo_id);
vrend_draw_bind_const_shader(ctx, shader_type, new_program);
vrend_draw_bind_samplers_shader(ctx, shader_type, &sampler_id);
+ vrend_draw_bind_images_shader(ctx, shader_type);
vrend_draw_bind_ssbo_shader(ctx, shader_type);
}
@@ -4640,6 +4780,9 @@ int vrend_renderer_init(struct vrend_if_cbs *cbs, uint32_t flags)
if (gl_ver >= 43 || epoxy_has_gl_extension("GL_ARB_shader_storage_buffer_object"))
vrend_state.have_ssbo = true;
+ if (gl_ver >= 42 || epoxy_has_gl_extension("GL_ARB_shader_image_load_store"))
+ vrend_state.have_images = true;
+
if (gl_ver >= 46 || epoxy_has_gl_extension("GL_ARB_polygon_offset_clamp"))
vrend_state.have_polygon_offset_clamp = true;
@@ -7732,6 +7875,17 @@ void vrend_renderer_fill_caps(uint32_t set, uint32_t version,
caps->v2.max_shader_buffer_frag_compute = max;
}
+ if (gl_ver >= 42 || epoxy_has_gl_extension("GL_ARB_shader_image_load_store")) {
+ glGetIntegerv(GL_MAX_VERTEX_IMAGE_UNIFORMS, &max);
+ if (max > PIPE_MAX_SHADER_IMAGES)
+ max = PIPE_MAX_SHADER_IMAGES;
+ caps->v2.max_shader_image_other_stages = max;
+ glGetIntegerv(GL_MAX_FRAGMENT_IMAGE_UNIFORMS, &max);
+ if (max > PIPE_MAX_SHADER_IMAGES)
+ max = PIPE_MAX_SHADER_IMAGES;
+ caps->v2.max_shader_image_frag_compute = max;
+ }
+
caps->v1.max_samples = vrend_renderer_query_multisample_caps(max, &caps->v2);
caps->v2.capability_bits |= VIRGL_CAP_TGSI_INVARIANT | VIRGL_CAP_SET_MIN_SAMPLES;
diff --git a/src/vrend_renderer.h b/src/vrend_renderer.h
index bb76b44..4a6f0c5 100644
--- a/src/vrend_renderer.h
+++ b/src/vrend_renderer.h
@@ -247,6 +247,12 @@ void vrend_set_index_buffer(struct vrend_context *ctx,
uint32_t res_handle,
uint32_t index_size,
uint32_t offset);
+void vrend_set_single_image_view(struct vrend_context *ctx,
+ uint32_t shader_type,
+ int index,
+ uint32_t format, uint32_t access,
+ uint32_t layer_offset, uint32_t level_size,
+ uint32_t handle);
void vrend_set_single_ssbo(struct vrend_context *ctx,
uint32_t shader_type,
int index,
--
2.14.3
More information about the virglrenderer-devel
mailing list