From nobody Mon Feb 9 09:54:34 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1692015009; cv=none; d=zohomail.com; s=zohoarc; b=bd8lfoSOxhqLn93GAc74R3vwlJmS6kwpV2krPEQ5F0xk4r4cSzMLZgJHHx5PXelQfbzppDRtjRe4QPRkDUINNBgZTM0UDZIqcDSpcI1oa+0dz+dvjzXUkThW9ti59/qtQWKfnumJpnI8N7j0Zq4VlrlCYTdCe5qeAg9Un/1eQKc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1692015009; h=Content-Transfer-Encoding:Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:To; bh=djAaU1ZNtNEKZKPQsWR0+nJJZnoNRb16+ycYq7odmi8=; b=XdpDmbMx8QfnwUIMItkB1hsvDAwnsJZ6fMrbx1CQQ7DMIWKOGHyb1+zLXSFYufPS/3NaXaRdtCMaAMknrz9ayyOs8dFN80u464vb1+UFqcemuQ6fK9iIusx605p6GO9EomaZzu+EI6tcSco6N+ZsAU6yQ7XRRzA6Eqv6930BqrI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1692015009826343.1606000472693; Mon, 14 Aug 2023 05:10:09 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qVWNe-0002a1-Bu; Mon, 14 Aug 2023 08:09:06 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qVWNc-0002Zm-Fi for qemu-devel@nongnu.org; Mon, 14 Aug 2023 08:09:04 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qVWNa-0005Q8-Ld for qemu-devel@nongnu.org; Mon, 14 Aug 2023 08:09:04 -0400 Received: from mimecast-mx02.redhat.com (66.187.233.73 [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-539-xh9-0FJ8NzO0yqZthMRQvg-1; Mon, 14 Aug 2023 08:07:41 -0400 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.rdu2.redhat.com [10.11.54.8]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 0DAFE2A59579 for ; Mon, 14 Aug 2023 12:07:41 +0000 (UTC) Received: from toolbox.redhat.com (unknown [10.45.226.149]) by smtp.corp.redhat.com (Postfix) with ESMTP id 14D5FC15BAD; Mon, 14 Aug 2023 12:07:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1692014941; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=djAaU1ZNtNEKZKPQsWR0+nJJZnoNRb16+ycYq7odmi8=; b=AqPeXhw60UGVqQQW04pxH8SgSbcpekDCnHDG5g9Wt5EUhALqFuTUJubNCMOJJbT0Tff5u0 WiygFhWvhbzuBilv0eUSi4YYPBi3Alf5YoaU0B0gFoGXbXvY2lFxJ+y9taDD7cufC+RLk4 uV5tuFAh7vYWd2icE93Eqv6hglgBvSc= X-MC-Unique: xh9-0FJ8NzO0yqZthMRQvg-1 From: Bilal Elmoussaoui To: qemu-devel@nongnu.org Cc: marcandre.lureau@redhat.com, Gerd Hoffmann , Bilal Elmoussaoui , Christian Hergert Subject: [PATCH] ui/dbus: implement damage regions for GL Date: Mon, 14 Aug 2023 14:07:39 +0200 Message-ID: <20230814120739.89213-1-belmouss@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 3.1 on 10.11.54.8 Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=170.10.129.124; envelope-from=belmouss@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1692015011414100001 Content-Type: text/plain; charset="utf-8" Currently, when using `-display dbus,gl=3Don` all updates to the client become "full scanout" updates, meaning there is no way for the client to limit damage regions to the display server. Instead of using an "update count", this patch tracks the damage region and propagates it to the client. This was less of an issue when clients were using GtkGLArea for rendering, as you'd be doing full-surface redraw. To be efficient, the client needs both a DMA-BUF and the damage region to be updated. Co-authored-by: Christian Hergert Signed-off-by: Bilal Elmoussaoui --- ui/dbus-listener.c | 32 ++++++++++++++++++++++++-------- 1 file changed, 24 insertions(+), 8 deletions(-) diff --git a/ui/dbus-listener.c b/ui/dbus-listener.c index 30917271ab..d015e8d759 100644 --- a/ui/dbus-listener.c +++ b/ui/dbus-listener.c @@ -26,6 +26,9 @@ #include "qapi/error.h" #include "sysemu/sysemu.h" #include "dbus.h" +#ifdef CONFIG_OPENGL +#include +#endif #ifdef G_OS_UNIX #include #endif @@ -59,12 +62,15 @@ struct _DBusDisplayListener { =20 QemuDBusDisplay1Listener *proxy; =20 +#ifdef CONFIG_OPENGL + /* Keep track of the damage region */ + pixman_region32_t gl_damage; +#endif + DisplayChangeListener dcl; DisplaySurface *ds; enum share_kind ds_share; =20 - int gl_updates; - bool ds_mapped; bool can_share_map; =20 @@ -539,11 +545,16 @@ static void dbus_gl_refresh(DisplayChangeListener *dc= l) return; } =20 - if (ddl->gl_updates) { - dbus_call_update_gl(dcl, 0, 0, - surface_width(ddl->ds), surface_height(ddl->ds= )); - ddl->gl_updates =3D 0; + int n_rects =3D pixman_region32_n_rects(&ddl->gl_damage); + + for (int i =3D 0; i < n_rects; i++) { + pixman_box32_t *box; + box =3D pixman_region32_rectangles(&ddl->gl_damage, NULL) + i; + + dbus_call_update_gl(dcl, box->x1, box->y1, + box->x2 - box->x1, box->y2 - box->y1); } + pixman_region32_clear(&ddl->gl_damage); } #endif /* OPENGL */ =20 @@ -558,7 +569,10 @@ static void dbus_gl_gfx_update(DisplayChangeListener *= dcl, { DBusDisplayListener *ddl =3D container_of(dcl, DBusDisplayListener, dc= l); =20 - ddl->gl_updates++; + pixman_region32_t rect_region; + pixman_region32_init_rect(&rect_region, x, y, w, h); + pixman_region32_union(&ddl->gl_damage, &ddl->gl_damage, &rect_region); + pixman_region32_fini(&rect_region); } #endif =20 @@ -933,7 +947,9 @@ dbus_display_listener_new(const char *bus_name, g_object_unref(ddl); return NULL; } - +#ifdef CONFIG_OPENGL + pixman_region32_init(&ddl->gl_damage); +#endif ddl->bus_name =3D g_strdup(bus_name); ddl->conn =3D conn; ddl->console =3D console; --=20 2.41.0