vrend: If available use glCopyImageSubData to execute memcopy like blits

When the host is  gles >= 3.2, gl >= 4.3, or when the extension
GL_(ARB|EXT|OES)_copy_image is available, memcopy like blitting and region
copying can be done for many color format combinations by using
glCopyImageSubData.

This fixes a number of tests from the subset
  dEQP-GLES31.functional.copy_image.non_compressed.viewclass_*

v2: - Clean list of canonical formats (Gurchetan Singh)
    - Use size of canonical formats to decide whether they can be copied via
      gCopyImageSubData
    - Also honour the render state when deciding whether glCopyImageSubData
      will do, or whether we need to do a blit.

v3: - replace format size check by compatibility check (Gurchetan Singh) but
      keep the check seperate because we need to add logic for compressed
      texture later

Reviewed-by: Gurchetan Singh <gurchetansingh at chromium.org>
Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Signed-off-by: Jakob Bornecrantz <jakob@collabora.com>
macos/master
Gert Wollny 6 years ago committed by Jakob Bornecrantz
parent be3b1078eb
commit 2846dcf565
  1. 17
      src/vrend_formats.c
  2. 53
      src/vrend_renderer.c
  3. 2
      src/vrend_renderer.h

@ -487,3 +487,20 @@ unsigned vrend_renderer_query_multisample_caps(unsigned max_samples, struct virg
glDeleteFramebuffers(1, &fbo); glDeleteFramebuffers(1, &fbo);
return max_samples_confirmed; return max_samples_confirmed;
} }
boolean format_is_copy_compatible(enum pipe_format src, enum pipe_format dst)
{
int r;
if (src == dst)
return true;
if (util_format_is_plain(src) && util_format_is_plain(dst)) {
const struct util_format_description *src_desc = util_format_description(src);
const struct util_format_description *dst_desc = util_format_description(dst);
return util_is_format_compatible(src_desc, dst_desc);
}
/* compressed formats will be supported later */
return false;
}

@ -119,6 +119,7 @@ struct global_renderer_state {
bool have_texture_storage; bool have_texture_storage;
bool have_tessellation; bool have_tessellation;
bool have_texture_view; bool have_texture_view;
bool have_copy_image;
/* these appeared broken on at least one driver */ /* these appeared broken on at least one driver */
bool use_explicit_locations; bool use_explicit_locations;
@ -4551,6 +4552,12 @@ int vrend_renderer_init(struct vrend_if_cbs *cbs, uint32_t flags)
if (gl_ver >= 46 || epoxy_has_gl_extension("GL_ARB_polygon_offset_clamp")) if (gl_ver >= 46 || epoxy_has_gl_extension("GL_ARB_polygon_offset_clamp"))
vrend_state.have_polygon_offset_clamp = true; vrend_state.have_polygon_offset_clamp = true;
if (gl_ver >= 43 || (gles && gl_ver >= 32) ||
epoxy_has_gl_extension("GL_ARB_copy_image") ||
epoxy_has_gl_extension("GL_EXT_copy_image") ||
epoxy_has_gl_extension("GL_OES_copy_image"))
vrend_state.have_copy_image = true;
/* callbacks for when we are cleaning up the object table */ /* callbacks for when we are cleaning up the object table */
vrend_resource_set_destroy_callback(vrend_destroy_resource_object); vrend_resource_set_destroy_callback(vrend_destroy_resource_object);
vrend_object_set_destroy_callback(VIRGL_OBJECT_QUERY, vrend_destroy_query_object); vrend_object_set_destroy_callback(VIRGL_OBJECT_QUERY, vrend_destroy_query_object);
@ -6365,6 +6372,22 @@ static void vrend_resource_copy_fallback(struct vrend_resource *src_res,
free(tptr); free(tptr);
} }
static inline void
vrend_copy_sub_image(struct vrend_resource* src_res, struct vrend_resource * dst_res,
uint32_t src_level, const struct pipe_box *src_box,
uint32_t dst_level, uint32_t dstx, uint32_t dsty, uint32_t dstz)
{
glCopyImageSubData(src_res->id,
tgsitargettogltarget(src_res->base.target, src_res->base.nr_samples),
src_level, src_box->x, src_box->y, src_box->z,
dst_res->id,
tgsitargettogltarget(dst_res->base.target, dst_res->base.nr_samples),
dst_level, dstx, dsty, dstz,
src_box->width, src_box->height,src_box->depth);
}
void vrend_renderer_resource_copy_region(struct vrend_context *ctx, void vrend_renderer_resource_copy_region(struct vrend_context *ctx,
uint32_t dst_handle, uint32_t dst_level, uint32_t dst_handle, uint32_t dst_level,
uint32_t dstx, uint32_t dsty, uint32_t dstz, uint32_t dstx, uint32_t dsty, uint32_t dstz,
@ -6397,11 +6420,19 @@ void vrend_renderer_resource_copy_region(struct vrend_context *ctx,
return; return;
} }
if (vrend_state.have_copy_image) {
if (format_is_copy_compatible(src_res->base.format,dst_res->base.format) &&
src_res->base.nr_samples == dst_res->base.nr_samples) {
vrend_copy_sub_image(src_res, dst_res, src_level, src_box,
dst_level, dstx, dsty, dstz);
return;
}
}
if (!vrend_format_can_render(src_res->base.format) || if (!vrend_format_can_render(src_res->base.format) ||
!vrend_format_can_render(dst_res->base.format)) { !vrend_format_can_render(dst_res->base.format)) {
vrend_resource_copy_fallback(src_res, dst_res, dst_level, dstx, vrend_resource_copy_fallback(src_res, dst_res, dst_level, dstx,
dsty, dstz, src_level, src_box); dsty, dstz, src_level, src_box);
return; return;
} }
@ -6673,7 +6704,25 @@ void vrend_renderer_blit(struct vrend_context *ctx,
if (info->render_condition_enable == false) if (info->render_condition_enable == false)
vrend_pause_render_condition(ctx, true); vrend_pause_render_condition(ctx, true);
vrend_renderer_blit_int(ctx, src_res, dst_res, info); /* The gallium blit function can be called for a general blit that may
* scale, convert the data, and apply some rander states or if is called via
* glCopyImageSubData. For the latter case forward the call to the
* glCopyImageSubData function, otherwise use a framebuffer blit.
*/
if (vrend_state.have_copy_image && !info->render_condition_enable &&
format_is_copy_compatible(info->src.format,info->dst.format) &&
!info->scissor_enable && (info->filter == PIPE_TEX_FILTER_NEAREST) &&
!info->alpha_blend && (info->mask == PIPE_MASK_RGBA) &&
(src_res->base.nr_samples == dst_res->base.nr_samples) &&
info->src.box.width == info->dst.box.width &&
info->src.box.height == info->dst.box.height &&
info->src.box.depth == info->dst.box.depth) {
vrend_copy_sub_image(src_res, dst_res, info->src.level, &info->src.box,
info->dst.level, info->dst.box.x, info->dst.box.y,
info->dst.box.z);
} else {
vrend_renderer_blit_int(ctx, src_res, dst_res, info);
}
if (info->render_condition_enable == false) if (info->render_condition_enable == false)
vrend_pause_render_condition(ctx, false); vrend_pause_render_condition(ctx, false);

@ -381,6 +381,8 @@ void vrend_fb_bind_texture(struct vrend_resource *res,
uint32_t level, uint32_t layer); uint32_t level, uint32_t layer);
bool vrend_is_ds_format(enum virgl_formats format); bool vrend_is_ds_format(enum virgl_formats format);
bool vrend_format_is_emulated_alpha(enum virgl_formats format); bool vrend_format_is_emulated_alpha(enum virgl_formats format);
boolean format_is_copy_compatible(enum pipe_format src, enum pipe_format dst);
/* blitter interface */ /* blitter interface */
void vrend_renderer_blit_gl(struct vrend_context *ctx, void vrend_renderer_blit_gl(struct vrend_context *ctx,
struct vrend_resource *src_res, struct vrend_resource *src_res,

Loading…
Cancel
Save