From 98702342945987c822d3c071aa3ef87350b8a67b Mon Sep 17 00:00:00 2001 From: Benjamin Otte Date: Wed, 2 Oct 2024 12:58:20 +0200 Subject: [PATCH] dmabuf: Change a debug message Instead of only printing the advertised dmabuf formats, print all supported ones. This is now useful information because we will try to use them when downloading. Fixes me looking like I don't know what I'm talking about on IRC when claiming that llvmpipe can't do dmabufs, because it only supports linear ones and we didn't print those. --- gdk/gdkdmabufegl.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/gdk/gdkdmabufegl.c b/gdk/gdkdmabufegl.c index 452746da13..64566a7615 100644 --- a/gdk/gdkdmabufegl.c +++ b/gdk/gdkdmabufegl.c @@ -97,15 +97,15 @@ gdk_dmabuf_egl_downloader_collect_formats (GdkDisplay *display, /* All linear formats we support are already added my the mmap downloader. * We don't add external formats, unless we can use them (via GLES) */ + GDK_DISPLAY_DEBUG (display, DMABUF, + "EGL supports %s dmabuf format %.4s:%#" G_GINT64_MODIFIER "x", + external_only[j] ? "external " : "", + (char *) &fourccs[i], + modifiers[j]); + if (modifiers[j] != DRM_FORMAT_MOD_LINEAR && (!external_only[j] || gdk_gl_context_get_use_es (context))) { - GDK_DISPLAY_DEBUG (display, DMABUF, - "%s EGL dmabuf format %.4s:%#" G_GINT64_MODIFIER "x", - external_only[j] ? "external " : "", - (char *) &fourccs[i], - modifiers[j]); - gdk_dmabuf_formats_builder_add_format (formats, fourccs[i], modifiers[j]); } if (!external_only[j])