From 53630a0a6086739187970ff4dcea351530474cf2 Mon Sep 17 00:00:00 2001 From: YaoBing Xiao Date: Mon, 23 Mar 2026 21:04:32 +0800 Subject: [PATCH] render: unify render pass accessors Introduce helper functions to retrieve backend-specific render passes from struct wlr_render_pass and replace ad-hoc internal accessors. --- include/render/gles2.h | 14 +++++--- include/render/pixman.h | 2 ++ include/wlr/render/interface.h | 2 ++ render/gles2/pass.c | 65 +++++++++++++++++----------------- render/gles2/renderer.c | 44 +++++++++++------------ render/gles2/texture.c | 28 +++++++-------- render/pixman/pass.c | 11 +++--- render/vulkan/pass.c | 11 +++--- 8 files changed, 94 insertions(+), 83 deletions(-) diff --git a/include/render/gles2.h b/include/render/gles2.h index efdb6599d..a2479e82e 100644 --- a/include/render/gles2.h +++ b/include/render/gles2.h @@ -139,7 +139,7 @@ void get_gles2_shm_formats(const struct wlr_gles2_renderer *renderer, GLuint gles2_buffer_get_fbo(struct wlr_gles2_buffer *buffer); -struct wlr_gles2_renderer *gles2_get_renderer( +struct wlr_gles2_renderer *wlr_gles2_renderer_from_renderer( struct wlr_renderer *wlr_renderer); struct wlr_gles2_render_timer *gles2_get_render_timer( struct wlr_render_timer *timer); @@ -152,10 +152,10 @@ struct wlr_texture *gles2_texture_from_buffer(struct wlr_renderer *wlr_renderer, struct wlr_buffer *buffer); void gles2_texture_destroy(struct wlr_gles2_texture *texture); -void push_gles2_debug_(struct wlr_gles2_renderer *renderer, +void wlr_gles2_push_debug_(struct wlr_gles2_renderer *renderer, const char *file, const char *func); -#define push_gles2_debug(renderer) push_gles2_debug_(renderer, _WLR_FILENAME, __func__) -void pop_gles2_debug(struct wlr_gles2_renderer *renderer); +#define wlr_gles2_push_debug(renderer) wlr_gles2_push_debug_(renderer, _WLR_FILENAME, __func__) +void wlr_gles2_pop_debug(struct wlr_gles2_renderer *renderer); struct wlr_gles2_render_pass *begin_gles2_buffer_pass(struct wlr_gles2_buffer *buffer, struct wlr_egl_context *prev_ctx, struct wlr_gles2_render_timer *timer, @@ -198,7 +198,11 @@ bool wlr_render_submit_pass_is_gles2(const struct wlr_render_submit_pass *submit struct wlr_gles2_render_submit_pass *wlr_gles2_render_submit_pass_from_pass( struct wlr_render_submit_pass *submit_pass); -GLuint gles2_link_program(struct wlr_gles2_renderer *renderer, +GLuint wlr_gles2_link_program(struct wlr_gles2_renderer *renderer, const GLchar *vert_src, const GLchar *frag_src); +struct wlr_gles2_render_pass *wlr_gles2_render_pass_from_render_pass( + struct wlr_render_pass *wlr_pass); +void wlr_gles_set_proj_matrix(GLint loc, float proj[9], const struct wlr_box *box); + #endif diff --git a/include/render/pixman.h b/include/render/pixman.h index 3fd9ceb3b..7e60b5f4a 100644 --- a/include/render/pixman.h +++ b/include/render/pixman.h @@ -84,5 +84,7 @@ struct wlr_pixman_render_submit_pass { bool wlr_render_submit_pass_is_pixman(const struct wlr_render_submit_pass *submit_pass); struct wlr_pixman_render_submit_pass *wlr_pixman_render_submit_pass_from_pass( struct wlr_render_submit_pass *submit_pass); +struct wlr_pixman_render_pass *wlr_pixman_render_pass_from_render_pass( + struct wlr_render_pass *wlr_pass); #endif diff --git a/include/wlr/render/interface.h b/include/wlr/render/interface.h index 13b34a70e..89ce62171 100644 --- a/include/wlr/render/interface.h +++ b/include/wlr/render/interface.h @@ -165,5 +165,7 @@ struct wlr_render_submit_pass *wlr_pixman_render_submit_pass_create(void); struct wlr_render_submit_pass *wlr_gles2_render_submit_pass_create(void); struct wlr_render_submit_pass *wlr_vk_render_submit_pass_create( struct wlr_renderer *wlr_renderer); +struct wlr_vk_render_pass *wlr_vk_render_pass_from_render_pass( + struct wlr_render_pass *wlr_pass); #endif diff --git a/render/gles2/pass.c b/render/gles2/pass.c index a33d9a73b..b832ad11f 100644 --- a/render/gles2/pass.c +++ b/render/gles2/pass.c @@ -19,19 +19,20 @@ static const struct wlr_render_pass_impl render_pass_impl; -static struct wlr_gles2_render_pass *get_render_pass(struct wlr_render_pass *wlr_pass) { +struct wlr_gles2_render_pass *wlr_gles2_render_pass_from_render_pass( + struct wlr_render_pass *wlr_pass) { assert(wlr_pass->impl == &render_pass_impl); struct wlr_gles2_render_pass *pass = wl_container_of(wlr_pass, pass, base); return pass; } static bool render_pass_submit(struct wlr_render_pass *wlr_pass) { - struct wlr_gles2_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_gles2_render_pass *pass = wlr_gles2_render_pass_from_render_pass(wlr_pass); struct wlr_gles2_renderer *renderer = pass->buffer->renderer; struct wlr_gles2_render_timer *timer = pass->timer; bool ok = false; - push_gles2_debug(renderer); + wlr_gles2_push_debug(renderer); if (timer) { // clear disjoint flag @@ -71,7 +72,7 @@ static bool render_pass_submit(struct wlr_render_pass *wlr_pass) { out: glBindFramebuffer(GL_FRAMEBUFFER, 0); - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); wlr_egl_restore_context(&pass->prev_ctx); wlr_drm_syncobj_timeline_unref(pass->signal_timeline); @@ -130,7 +131,7 @@ static void render(const struct wlr_box *box, const pixman_region32_t *clip, GLi pixman_region32_fini(®ion); } -static void set_proj_matrix(GLint loc, float proj[9], const struct wlr_box *box) { +void wlr_gles_set_proj_matrix(GLint loc, float proj[9], const struct wlr_box *box) { float gl_matrix[9]; wlr_matrix_identity(gl_matrix); wlr_matrix_translate(gl_matrix, box->x, box->y); @@ -172,7 +173,7 @@ static void setup_blending(enum wlr_render_blend_mode mode) { static void render_pass_add_texture(struct wlr_render_pass *wlr_pass, const struct wlr_render_texture_options *options) { - struct wlr_gles2_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_gles2_render_pass *pass = wlr_gles2_render_pass_from_render_pass(wlr_pass); struct wlr_gles2_renderer *renderer = pass->buffer->renderer; struct wlr_gles2_texture *texture = gles2_get_texture(options->texture); struct wlr_gles2_render_texture_pass *texture_pass = @@ -208,7 +209,7 @@ static void render_pass_add_texture(struct wlr_render_pass *wlr_pass, src_fbox.width /= options->texture->width; src_fbox.height /= options->texture->height; - push_gles2_debug(renderer); + wlr_gles2_push_debug(renderer); if (options->wait_timeline != NULL) { int sync_file_fd = @@ -251,18 +252,18 @@ static void render_pass_add_texture(struct wlr_render_pass *wlr_pass, glUniform1i(shader->tex, 0); glUniform1f(shader->alpha, alpha); - set_proj_matrix(shader->proj, pass->projection_matrix, &dst_box); + wlr_gles_set_proj_matrix(shader->proj, pass->projection_matrix, &dst_box); set_tex_matrix(shader->tex_proj, options->transform, &src_fbox); render(&dst_box, options->clip, shader->pos_attrib); glBindTexture(texture->target, 0); - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); } static void render_pass_add_rect(struct wlr_render_pass *wlr_pass, const struct wlr_render_rect_options *options) { - struct wlr_gles2_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_gles2_render_pass *pass = wlr_gles2_render_pass_from_render_pass(wlr_pass); struct wlr_gles2_renderer *renderer = pass->buffer->renderer; struct wlr_gles2_render_rect_pass *rect_pass = wlr_gles2_render_rect_pass_from_pass(renderer->wlr_renderer.rect_pass); @@ -272,7 +273,7 @@ static void render_pass_add_rect(struct wlr_render_pass *wlr_pass, struct wlr_buffer *wlr_buffer = pass->buffer->buffer; wlr_render_rect_options_get_box(options, wlr_buffer, &box); - push_gles2_debug(renderer); + wlr_gles2_push_debug(renderer); enum wlr_render_blend_mode blend_mode = color->a == 1.0 ? WLR_RENDER_BLEND_MODE_NONE : options->blend_mode; if (blend_mode == WLR_RENDER_BLEND_MODE_NONE && @@ -285,12 +286,12 @@ static void render_pass_add_rect(struct wlr_render_pass *wlr_pass, } else { setup_blending(blend_mode); glUseProgram(rect_pass->shader.program); - set_proj_matrix(rect_pass->shader.proj, pass->projection_matrix, &box); + wlr_gles_set_proj_matrix(rect_pass->shader.proj, pass->projection_matrix, &box); glUniform4f(rect_pass->shader.color, color->r, color->g, color->b, color->a); render(&box, options->clip, rect_pass->shader.pos_attrib); } - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); } static void render_pass_destory(struct wlr_render_pass *wlr_pass) { @@ -298,7 +299,7 @@ static void render_pass_destory(struct wlr_render_pass *wlr_pass) { } static struct wlr_renderer *render_pass_get_renderer(struct wlr_render_pass *wlr_pass) { - struct wlr_gles2_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_gles2_render_pass *pass = wlr_gles2_render_pass_from_render_pass(wlr_pass); struct wlr_gles2_renderer *renderer = pass->buffer->renderer; return &renderer->wlr_renderer; @@ -360,13 +361,13 @@ struct wlr_gles2_render_pass *begin_gles2_buffer_pass(struct wlr_gles2_buffer *b matrix_projection(pass->projection_matrix, wlr_buffer->width, wlr_buffer->height, WL_OUTPUT_TRANSFORM_FLIPPED_180); - push_gles2_debug(renderer); + wlr_gles2_push_debug(renderer); glBindFramebuffer(GL_FRAMEBUFFER, fbo); glViewport(0, 0, wlr_buffer->width, wlr_buffer->height); glBlendFunc(GL_ONE, GL_ONE_MINUS_SRC_ALPHA); glDisable(GL_SCISSOR_TEST); - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); return pass; } @@ -374,9 +375,9 @@ struct wlr_gles2_render_pass *begin_gles2_buffer_pass(struct wlr_gles2_buffer *b static void render_rect_pass_destroy(struct wlr_render_rect_pass *pass) { struct wlr_gles2_render_rect_pass *gles2_pass = wlr_gles2_render_rect_pass_from_pass(pass); - push_gles2_debug(gles2_pass->renderer); + wlr_gles2_push_debug(gles2_pass->renderer); glDeleteProgram(gles2_pass->shader.program); - pop_gles2_debug(gles2_pass->renderer); + wlr_gles2_pop_debug(gles2_pass->renderer); free(gles2_pass); } @@ -393,17 +394,17 @@ struct wlr_render_rect_pass *wlr_gles2_render_rect_pass_create( return NULL; } - struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer); + struct wlr_gles2_renderer *renderer = wlr_gles2_renderer_from_renderer(wlr_renderer); if (!wlr_egl_make_current(renderer->egl, NULL)) { free(pass); return NULL; } wlr_render_rect_pass_init(&pass->base, &render_rect_pass_impl); - push_gles2_debug(renderer); + wlr_gles2_push_debug(renderer); GLuint prog; pass->shader.program = prog = - gles2_link_program(renderer, common_vert_src, quad_frag_src); + wlr_gles2_link_program(renderer, common_vert_src, quad_frag_src); if (!pass->shader.program) { goto error; } @@ -411,14 +412,14 @@ struct wlr_render_rect_pass *wlr_gles2_render_rect_pass_create( pass->shader.proj = glGetUniformLocation(pass->shader.program, "proj"); pass->shader.color = glGetUniformLocation(pass->shader.program, "color"); pass->shader.pos_attrib = glGetAttribLocation(pass->shader.program, "pos"); - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); wlr_egl_unset_current(renderer->egl); pass->renderer = renderer; return &pass->base; error: render_rect_pass_destroy(&pass->base); - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); wlr_egl_unset_current(renderer->egl); return NULL; @@ -443,11 +444,11 @@ struct wlr_gles2_render_rect_pass *wlr_gles2_render_rect_pass_from_pass( static void render_texture_pass_destroy(struct wlr_render_texture_pass *pass) { struct wlr_gles2_render_texture_pass *gles2_pass = wlr_gles2_render_texture_pass_from_pass(pass); - push_gles2_debug(gles2_pass->renderer); + wlr_gles2_push_debug(gles2_pass->renderer); glDeleteProgram(gles2_pass->shaders.tex_rgba.program); glDeleteProgram(gles2_pass->shaders.tex_rgbx.program); glDeleteProgram(gles2_pass->shaders.tex_ext.program); - pop_gles2_debug(gles2_pass->renderer); + wlr_gles2_pop_debug(gles2_pass->renderer); free(gles2_pass); } @@ -464,15 +465,15 @@ struct wlr_render_texture_pass *wlr_gles2_render_texture_pass_create( return NULL; } wlr_render_texture_pass_init(&pass->base, &render_texture_pass_impl); - struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer); + struct wlr_gles2_renderer *renderer = wlr_gles2_renderer_from_renderer(wlr_renderer); if (!wlr_egl_make_current(renderer->egl, NULL)) { free(pass); return NULL; } - push_gles2_debug(renderer); + wlr_gles2_push_debug(renderer); GLuint prog; pass->shaders.tex_rgba.program = prog = - gles2_link_program(renderer, common_vert_src, tex_rgba_frag_src); + wlr_gles2_link_program(renderer, common_vert_src, tex_rgba_frag_src); if (!pass->shaders.tex_rgba.program) { goto error; } @@ -483,7 +484,7 @@ struct wlr_render_texture_pass *wlr_gles2_render_texture_pass_create( pass->shaders.tex_rgba.pos_attrib = glGetAttribLocation(prog, "pos"); pass->shaders.tex_rgbx.program = prog = - gles2_link_program(renderer, common_vert_src, tex_rgbx_frag_src); + wlr_gles2_link_program(renderer, common_vert_src, tex_rgbx_frag_src); if (!pass->shaders.tex_rgbx.program) { goto error; } @@ -495,7 +496,7 @@ struct wlr_render_texture_pass *wlr_gles2_render_texture_pass_create( if (renderer->exts.OES_egl_image_external) { pass->shaders.tex_ext.program = prog = - gles2_link_program(renderer, common_vert_src, tex_external_frag_src); + wlr_gles2_link_program(renderer, common_vert_src, tex_external_frag_src); if (!pass->shaders.tex_ext.program) { goto error; } @@ -506,7 +507,7 @@ struct wlr_render_texture_pass *wlr_gles2_render_texture_pass_create( pass->shaders.tex_ext.pos_attrib = glGetAttribLocation(prog, "pos"); } - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); wlr_egl_unset_current(renderer->egl); pass->renderer = renderer; @@ -514,7 +515,7 @@ struct wlr_render_texture_pass *wlr_gles2_render_texture_pass_create( error: render_texture_pass_destroy(&pass->base); - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); wlr_egl_unset_current(renderer->egl); return NULL; diff --git a/render/gles2/renderer.c b/render/gles2/renderer.c index 010623354..9c8a4955c 100644 --- a/render/gles2/renderer.c +++ b/render/gles2/renderer.c @@ -27,7 +27,7 @@ bool wlr_renderer_is_gles2(const struct wlr_renderer *wlr_renderer) { return wlr_renderer->impl == &renderer_impl; } -struct wlr_gles2_renderer *gles2_get_renderer( +struct wlr_gles2_renderer *wlr_gles2_renderer_from_renderer( struct wlr_renderer *wlr_renderer) { assert(wlr_renderer_is_gles2(wlr_renderer)); struct wlr_gles2_renderer *renderer = wl_container_of(wlr_renderer, renderer, wlr_renderer); @@ -51,13 +51,13 @@ static void destroy_buffer(struct wlr_gles2_buffer *buffer) { struct wlr_egl_context prev_ctx; wlr_egl_make_current(buffer->renderer->egl, &prev_ctx); - push_gles2_debug(buffer->renderer); + wlr_gles2_push_debug(buffer->renderer); glDeleteFramebuffers(1, &buffer->fbo); glDeleteRenderbuffers(1, &buffer->rbo); glDeleteTextures(1, &buffer->tex); - pop_gles2_debug(buffer->renderer); + wlr_gles2_pop_debug(buffer->renderer); wlr_egl_destroy_image(buffer->renderer->egl, buffer->image); @@ -87,7 +87,7 @@ GLuint gles2_buffer_get_fbo(struct wlr_gles2_buffer *buffer) { return buffer->fbo; } - push_gles2_debug(buffer->renderer); + wlr_gles2_push_debug(buffer->renderer); if (!buffer->rbo) { glGenRenderbuffers(1, &buffer->rbo); @@ -110,7 +110,7 @@ GLuint gles2_buffer_get_fbo(struct wlr_gles2_buffer *buffer) { buffer->fbo = 0; } - pop_gles2_debug(buffer->renderer); + wlr_gles2_pop_debug(buffer->renderer); return buffer->fbo; } @@ -160,7 +160,7 @@ error_buffer: static const struct wlr_drm_format_set *gles2_get_texture_formats( struct wlr_renderer *wlr_renderer, uint32_t buffer_caps) { - struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer); + struct wlr_gles2_renderer *renderer = wlr_gles2_renderer_from_renderer(wlr_renderer); if (buffer_caps & WLR_BUFFER_CAP_DMABUF) { return wlr_egl_get_dmabuf_texture_formats(renderer->egl); } else if (buffer_caps & WLR_BUFFER_CAP_DATA_PTR) { @@ -172,13 +172,13 @@ static const struct wlr_drm_format_set *gles2_get_texture_formats( static const struct wlr_drm_format_set *gles2_get_render_formats( struct wlr_renderer *wlr_renderer) { - struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer); + struct wlr_gles2_renderer *renderer = wlr_gles2_renderer_from_renderer(wlr_renderer); return wlr_egl_get_dmabuf_render_formats(renderer->egl); } static int gles2_get_drm_fd(struct wlr_renderer *wlr_renderer) { struct wlr_gles2_renderer *renderer = - gles2_get_renderer(wlr_renderer); + wlr_gles2_renderer_from_renderer(wlr_renderer); if (renderer->drm_fd < 0) { renderer->drm_fd = wlr_egl_dup_drm_fd(renderer->egl); @@ -189,12 +189,12 @@ static int gles2_get_drm_fd(struct wlr_renderer *wlr_renderer) { struct wlr_egl *wlr_gles2_renderer_get_egl(struct wlr_renderer *wlr_renderer) { struct wlr_gles2_renderer *renderer = - gles2_get_renderer(wlr_renderer); + wlr_gles2_renderer_from_renderer(wlr_renderer); return renderer->egl; } static void gles2_destroy(struct wlr_renderer *wlr_renderer) { - struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer); + struct wlr_gles2_renderer *renderer = wlr_gles2_renderer_from_renderer(wlr_renderer); wlr_egl_make_current(renderer->egl, NULL); @@ -227,7 +227,7 @@ static void gles2_destroy(struct wlr_renderer *wlr_renderer) { static struct wlr_render_pass *gles2_begin_buffer_pass(struct wlr_renderer *wlr_renderer, struct wlr_buffer *wlr_buffer, const struct wlr_buffer_pass_options *options) { - struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer); + struct wlr_gles2_renderer *renderer = wlr_gles2_renderer_from_renderer(wlr_renderer); struct wlr_egl_context prev_ctx = {0}; if (!wlr_egl_make_current(renderer->egl, &prev_ctx)) { @@ -255,7 +255,7 @@ static struct wlr_render_pass *gles2_begin_buffer_pass(struct wlr_renderer *wlr_ GLuint wlr_gles2_renderer_get_buffer_fbo(struct wlr_renderer *wlr_renderer, struct wlr_buffer *wlr_buffer) { - struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer); + struct wlr_gles2_renderer *renderer = wlr_gles2_renderer_from_renderer(wlr_renderer); GLuint fbo = 0; struct wlr_egl_context prev_ctx = {0}; @@ -273,7 +273,7 @@ GLuint wlr_gles2_renderer_get_buffer_fbo(struct wlr_renderer *wlr_renderer, } static struct wlr_render_timer *gles2_render_timer_create(struct wlr_renderer *wlr_renderer) { - struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer); + struct wlr_gles2_renderer *renderer = wlr_gles2_renderer_from_renderer(wlr_renderer); if (!renderer->exts.EXT_disjoint_timer_query) { wlr_log(WLR_ERROR, "can't create timer, EXT_disjoint_timer_query not available"); return NULL; @@ -354,7 +354,7 @@ static const struct wlr_render_timer_impl render_timer_impl = { .destroy = gles2_render_timer_destroy, }; -void push_gles2_debug_(struct wlr_gles2_renderer *renderer, +void wlr_gles2_push_debug_(struct wlr_gles2_renderer *renderer, const char *file, const char *func) { if (!renderer->procs.glPushDebugGroupKHR) { return; @@ -366,7 +366,7 @@ void push_gles2_debug_(struct wlr_gles2_renderer *renderer, renderer->procs.glPushDebugGroupKHR(GL_DEBUG_SOURCE_APPLICATION_KHR, 1, -1, str); } -void pop_gles2_debug(struct wlr_gles2_renderer *renderer) { +void wlr_gles2_pop_debug(struct wlr_gles2_renderer *renderer) { if (renderer->procs.glPopDebugGroupKHR) { renderer->procs.glPopDebugGroupKHR(); } @@ -394,7 +394,7 @@ static void gles2_log(GLenum src, GLenum type, GLuint id, GLenum severity, static GLuint compile_shader(struct wlr_gles2_renderer *renderer, GLenum type, const GLchar *src) { - push_gles2_debug(renderer); + wlr_gles2_push_debug(renderer); GLuint shader = glCreateShader(type); glShaderSource(shader, 1, &src, NULL); @@ -408,13 +408,13 @@ static GLuint compile_shader(struct wlr_gles2_renderer *renderer, shader = 0; } - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); return shader; } -GLuint gles2_link_program(struct wlr_gles2_renderer *renderer, +GLuint wlr_gles2_link_program(struct wlr_gles2_renderer *renderer, const GLchar *vert_src, const GLchar *frag_src) { - push_gles2_debug(renderer); + wlr_gles2_push_debug(renderer); GLuint vert = compile_shader(renderer, GL_VERTEX_SHADER, vert_src); if (!vert) { @@ -445,11 +445,11 @@ GLuint gles2_link_program(struct wlr_gles2_renderer *renderer, goto error; } - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); return prog; error: - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); return 0; } @@ -631,6 +631,6 @@ struct wlr_renderer *wlr_gles2_renderer_create(struct wlr_egl *egl) { bool wlr_gles2_renderer_check_ext(struct wlr_renderer *wlr_renderer, const char *ext) { - struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer); + struct wlr_gles2_renderer *renderer = wlr_gles2_renderer_from_renderer(wlr_renderer); return check_gl_ext(renderer->exts_str, ext); } diff --git a/render/gles2/texture.c b/render/gles2/texture.c index 7460755cd..780978206 100644 --- a/render/gles2/texture.c +++ b/render/gles2/texture.c @@ -69,7 +69,7 @@ static bool gles2_texture_update_from_buffer(struct wlr_texture *wlr_texture, struct wlr_egl_context prev_ctx; wlr_egl_make_current(texture->renderer->egl, &prev_ctx); - push_gles2_debug(texture->renderer); + wlr_gles2_push_debug(texture->renderer); glBindTexture(GL_TEXTURE_2D, texture->tex); @@ -95,7 +95,7 @@ static bool gles2_texture_update_from_buffer(struct wlr_texture *wlr_texture, glBindTexture(GL_TEXTURE_2D, 0); - pop_gles2_debug(texture->renderer); + wlr_gles2_pop_debug(texture->renderer); wlr_egl_restore_context(&prev_ctx); @@ -112,12 +112,12 @@ void gles2_texture_destroy(struct wlr_gles2_texture *texture) { struct wlr_egl_context prev_ctx; wlr_egl_make_current(texture->renderer->egl, &prev_ctx); - push_gles2_debug(texture->renderer); + wlr_gles2_push_debug(texture->renderer); glDeleteTextures(1, &texture->tex); glDeleteFramebuffers(1, &texture->fbo); - pop_gles2_debug(texture->renderer); + wlr_gles2_pop_debug(texture->renderer); wlr_egl_restore_context(&prev_ctx); } @@ -191,7 +191,7 @@ static bool gles2_texture_read_pixels(struct wlr_texture *wlr_texture, return false; } - push_gles2_debug(texture->renderer); + wlr_gles2_push_debug(texture->renderer); struct wlr_egl_context prev_ctx; if (!wlr_egl_make_current(texture->renderer->egl, &prev_ctx)) { return false; @@ -226,7 +226,7 @@ static bool gles2_texture_read_pixels(struct wlr_texture *wlr_texture, } wlr_egl_restore_context(&prev_ctx); - pop_gles2_debug(texture->renderer); + wlr_gles2_pop_debug(texture->renderer); return glGetError() == GL_NO_ERROR; } @@ -234,7 +234,7 @@ static bool gles2_texture_read_pixels(struct wlr_texture *wlr_texture, static uint32_t gles2_texture_preferred_read_format(struct wlr_texture *wlr_texture) { struct wlr_gles2_texture *texture = gles2_get_texture(wlr_texture); - push_gles2_debug(texture->renderer); + wlr_gles2_push_debug(texture->renderer); uint32_t fmt = DRM_FORMAT_INVALID; @@ -253,7 +253,7 @@ static uint32_t gles2_texture_preferred_read_format(struct wlr_texture *wlr_text glGetIntegerv(GL_ALPHA_BITS, &alpha_size); glBindFramebuffer(GL_FRAMEBUFFER, 0); - pop_gles2_debug(texture->renderer); + wlr_gles2_pop_debug(texture->renderer); const struct wlr_gles2_pixel_format *pix_fmt = get_gles2_format_from_gl(gl_format, gl_type, alpha_size > 0); @@ -297,7 +297,7 @@ static struct wlr_texture *gles2_texture_from_pixels( struct wlr_renderer *wlr_renderer, uint32_t drm_format, uint32_t stride, uint32_t width, uint32_t height, const void *data) { - struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer); + struct wlr_gles2_renderer *renderer = wlr_gles2_renderer_from_renderer(wlr_renderer); const struct wlr_gles2_pixel_format *fmt = get_gles2_format_from_drm(drm_format); @@ -335,7 +335,7 @@ static struct wlr_texture *gles2_texture_from_pixels( struct wlr_egl_context prev_ctx; wlr_egl_make_current(renderer->egl, &prev_ctx); - push_gles2_debug(renderer); + wlr_gles2_push_debug(renderer); glGenTextures(1, &texture->tex); glBindTexture(GL_TEXTURE_2D, texture->tex); @@ -349,7 +349,7 @@ static struct wlr_texture *gles2_texture_from_pixels( glBindTexture(GL_TEXTURE_2D, 0); - pop_gles2_debug(renderer); + wlr_gles2_pop_debug(renderer); wlr_egl_restore_context(&prev_ctx); @@ -381,7 +381,7 @@ static struct wlr_texture *gles2_texture_from_dmabuf( struct wlr_egl_context prev_ctx; wlr_egl_make_current(renderer->egl, &prev_ctx); - push_gles2_debug(texture->renderer); + wlr_gles2_push_debug(texture->renderer); bool invalid; if (!buffer->tex) { @@ -400,7 +400,7 @@ static struct wlr_texture *gles2_texture_from_dmabuf( glBindTexture(texture->target, 0); } - pop_gles2_debug(texture->renderer); + wlr_gles2_pop_debug(texture->renderer); wlr_egl_restore_context(&prev_ctx); texture->tex = buffer->tex; @@ -410,7 +410,7 @@ static struct wlr_texture *gles2_texture_from_dmabuf( struct wlr_texture *gles2_texture_from_buffer(struct wlr_renderer *wlr_renderer, struct wlr_buffer *buffer) { - struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer); + struct wlr_gles2_renderer *renderer = wlr_gles2_renderer_from_renderer(wlr_renderer); void *data; uint32_t format; diff --git a/render/pixman/pass.c b/render/pixman/pass.c index 736e6d553..80660f412 100644 --- a/render/pixman/pass.c +++ b/render/pixman/pass.c @@ -7,7 +7,8 @@ static const struct wlr_render_pass_impl render_pass_impl; -static struct wlr_pixman_render_pass *get_render_pass(struct wlr_render_pass *wlr_pass) { +struct wlr_pixman_render_pass *wlr_pixman_render_pass_from_render_pass( + struct wlr_render_pass *wlr_pass) { assert(wlr_pass->impl == &render_pass_impl); struct wlr_pixman_render_pass *pass = wl_container_of(wlr_pass, pass, base); return pass; @@ -20,7 +21,7 @@ static struct wlr_pixman_texture *get_texture(struct wlr_texture *wlr_texture) { } static bool render_pass_submit(struct wlr_render_pass *wlr_pass) { - struct wlr_pixman_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_pixman_render_pass *pass = wlr_pixman_render_pass_from_render_pass(wlr_pass); wlr_buffer_end_data_ptr_access(pass->buffer->buffer); wlr_buffer_unlock(pass->buffer->buffer); @@ -41,7 +42,7 @@ static pixman_op_t get_pixman_blending(enum wlr_render_blend_mode mode) { static void render_pass_add_texture(struct wlr_render_pass *wlr_pass, const struct wlr_render_texture_options *options) { - struct wlr_pixman_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_pixman_render_pass *pass = wlr_pixman_render_pass_from_render_pass(wlr_pass); struct wlr_pixman_texture *texture = get_texture(options->texture); struct wlr_pixman_buffer *buffer = pass->buffer; @@ -203,7 +204,7 @@ static void render_pass_add_texture(struct wlr_render_pass *wlr_pass, static void render_pass_add_rect(struct wlr_render_pass *wlr_pass, const struct wlr_render_rect_options *options) { - struct wlr_pixman_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_pixman_render_pass *pass = wlr_pixman_render_pass_from_render_pass(wlr_pass); struct wlr_pixman_buffer *buffer = pass->buffer; struct wlr_box box; wlr_render_rect_options_get_box(options, pass->buffer->buffer, &box); @@ -233,7 +234,7 @@ static void render_pass_destory(struct wlr_render_pass *wlr_pass) { } static struct wlr_renderer *render_pass_get_renderer(struct wlr_render_pass *wlr_pass) { - struct wlr_pixman_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_pixman_render_pass *pass = wlr_pixman_render_pass_from_render_pass(wlr_pass); struct wlr_pixman_renderer *renderer = pass->buffer->renderer; return &renderer->wlr_renderer; diff --git a/render/vulkan/pass.c b/render/vulkan/pass.c index 7373e935f..ceab5aae8 100644 --- a/render/vulkan/pass.c +++ b/render/vulkan/pass.c @@ -32,7 +32,8 @@ static bool create_shader_module(VkDevice dev, const uint32_t *code, return true; } -static struct wlr_vk_render_pass *get_render_pass(struct wlr_render_pass *wlr_pass) { +struct wlr_vk_render_pass *wlr_vk_render_pass_from_render_pass( + struct wlr_render_pass *wlr_pass) { assert(wlr_pass->impl == &render_pass_impl); struct wlr_vk_render_pass *pass = wl_container_of(wlr_pass, pass, base); return pass; @@ -240,7 +241,7 @@ static bool unwrap_color_transform(struct wlr_color_transform *transform, } static bool render_pass_submit(struct wlr_render_pass *wlr_pass) { - struct wlr_vk_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_vk_render_pass *pass = wlr_vk_render_pass_from_render_pass(wlr_pass); struct wlr_vk_renderer *renderer = pass->renderer; struct wlr_vk_render_submit_pass *submit_pass = wlr_vk_render_submit_pass_from_pass(renderer->wlr_renderer.submit_pass); @@ -705,7 +706,7 @@ static void render_pass_mark_box_updated(struct wlr_vk_render_pass *pass, static void render_pass_add_rect(struct wlr_render_pass *wlr_pass, const struct wlr_render_rect_options *options) { - struct wlr_vk_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_vk_render_pass *pass = wlr_vk_render_pass_from_render_pass(wlr_pass); struct wlr_vk_render_rect_pass *rect_pass = wlr_vk_render_rect_pass_from_pass(pass->renderer->wlr_renderer.rect_pass); assert(rect_pass != NULL); @@ -812,7 +813,7 @@ static void render_pass_add_rect(struct wlr_render_pass *wlr_pass, static void render_pass_add_texture(struct wlr_render_pass *wlr_pass, const struct wlr_render_texture_options *options) { - struct wlr_vk_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_vk_render_pass *pass = wlr_vk_render_pass_from_render_pass(wlr_pass); struct wlr_vk_renderer *renderer = pass->renderer; struct wlr_vk_render_texture_pass *texture_pass = wlr_vk_render_texture_pass_from_pass(renderer->wlr_renderer.texture_pass); @@ -1018,7 +1019,7 @@ static void render_pass_destory(struct wlr_render_pass *wlr_pass) { } static struct wlr_renderer *render_pass_get_renderer(struct wlr_render_pass *wlr_pass) { - struct wlr_vk_render_pass *pass = get_render_pass(wlr_pass); + struct wlr_vk_render_pass *pass = wlr_vk_render_pass_from_render_pass(wlr_pass); struct wlr_vk_renderer *renderer = pass->renderer; return &renderer->wlr_renderer;