From 1cdef8da579f4a021935a0b93967aa7a90bf8859 Mon Sep 17 00:00:00 2001
From: Simon Ser <contact@emersion.fr>
Date: Tue, 16 Mar 2021 11:29:56 +0100
Subject: render: drop wlr_renderer_blit_dmabuf

It can be replaced with wlr_renderer_bind_buffer. blit_dmabuf is
broken as-is (dies on an assertion).
---
 render/gles2/renderer.c | 82 -------------------------------------------------
 1 file changed, 82 deletions(-)

(limited to 'render/gles2/renderer.c')

diff --git a/render/gles2/renderer.c b/render/gles2/renderer.c
index eb93b3b4..1814ad07 100644
--- a/render/gles2/renderer.c
+++ b/render/gles2/renderer.c
@@ -530,87 +530,6 @@ static bool gles2_read_pixels(struct wlr_renderer *wlr_renderer,
 	return glGetError() == GL_NO_ERROR;
 }
 
-static bool gles2_blit_dmabuf(struct wlr_renderer *wlr_renderer,
-		struct wlr_dmabuf_attributes *dst_attr,
-		struct wlr_dmabuf_attributes *src_attr) {
-	struct wlr_gles2_renderer *renderer = gles2_get_renderer(wlr_renderer);
-	if (!renderer->procs.glEGLImageTargetRenderbufferStorageOES) {
-		return false;
-	}
-
-	struct wlr_egl_context old_context;
-	wlr_egl_save_context(&old_context);
-
-	bool r = false;
-	struct wlr_texture *src_tex =
-		wlr_texture_from_dmabuf(wlr_renderer, src_attr);
-	if (!src_tex) {
-		goto restore_context_out;
-	}
-
-	// TODO: get inverted_y right
-	// This is to take into account y-inversion on both buffers rather than
-	// just the source buffer.
-	bool src_inverted_y =
-		!!(src_attr->flags & WLR_DMABUF_ATTRIBUTES_FLAGS_Y_INVERT);
-	bool dst_inverted_y =
-		!!(dst_attr->flags & WLR_DMABUF_ATTRIBUTES_FLAGS_Y_INVERT);
-	struct wlr_gles2_texture *gles2_src_tex = gles2_get_texture(src_tex);
-	gles2_src_tex->inverted_y = src_inverted_y ^ dst_inverted_y;
-
-	if (!wlr_egl_make_current(renderer->egl)) {
-		goto texture_destroy_out;
-	}
-
-	// TODO: The imported buffer should be checked with
-	// eglQueryDmaBufModifiersEXT to see if it may be modified.
-	bool external_only = false;
-	EGLImageKHR image = wlr_egl_create_image_from_dmabuf(renderer->egl, dst_attr,
-			&external_only);
-	if (image == EGL_NO_IMAGE_KHR) {
-		goto texture_destroy_out;
-	}
-
-	GLuint rbo = 0;
-	glGenRenderbuffers(1, &rbo);
-	glBindRenderbuffer(GL_RENDERBUFFER, rbo);
-	renderer->procs.glEGLImageTargetRenderbufferStorageOES(GL_RENDERBUFFER,
-			image);
-	glBindRenderbuffer(GL_RENDERBUFFER, 0);
-
-	GLuint fbo = 0;
-	glGenFramebuffers(1, &fbo);
-	glBindFramebuffer(GL_FRAMEBUFFER, fbo);
-	glFramebufferRenderbuffer(GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0,
-			GL_RENDERBUFFER, rbo);
-
-	GLenum status = glCheckFramebufferStatus(GL_FRAMEBUFFER);
-	if (status != GL_FRAMEBUFFER_COMPLETE) {
-		goto out;
-	}
-
-	// TODO: use ANGLE_framebuffer_blit if available
-	float mat[9];
-	wlr_matrix_identity(mat);
-
-	wlr_renderer_begin(wlr_renderer, dst_attr->width, dst_attr->height);
-	wlr_renderer_clear(wlr_renderer, (float[]){ 0.0, 0.0, 0.0, 0.0 });
-	wlr_render_texture_with_matrix(wlr_renderer, src_tex, mat, 1.0f);
-	wlr_renderer_end(wlr_renderer);
-
-	r = true;
-out:
-	glBindFramebuffer(GL_FRAMEBUFFER, 0);
-	glDeleteFramebuffers(1, &fbo);
-	glDeleteRenderbuffers(1, &rbo);
-	wlr_egl_destroy_image(renderer->egl, image);
-texture_destroy_out:
-	wlr_texture_destroy(src_tex);
-restore_context_out:
-	wlr_egl_restore_context(&old_context);
-	return r;
-}
-
 static bool gles2_init_wl_display(struct wlr_renderer *wlr_renderer,
 		struct wl_display *wl_display) {
 	struct wlr_gles2_renderer *renderer =
@@ -707,7 +626,6 @@ static const struct wlr_renderer_impl renderer_impl = {
 	.texture_from_wl_drm = gles2_texture_from_wl_drm,
 	.texture_from_dmabuf = gles2_texture_from_dmabuf,
 	.init_wl_display = gles2_init_wl_display,
-	.blit_dmabuf = gles2_blit_dmabuf,
 	.get_drm_fd = gles2_get_drm_fd,
 };
 
-- 
cgit v1.2.3