All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: qemu-devel@nongnu.org
Cc: Gerd Hoffmann <kraxel@redhat.com>
Subject: [Qemu-devel] [PULL 02/11] sdl2: stop flickering
Date: Fri,  9 Oct 2015 10:18:25 +0200	[thread overview]
Message-ID: <1444378714-15190-3-git-send-email-kraxel@redhat.com> (raw)
In-Reply-To: <1444378714-15190-1-git-send-email-kraxel@redhat.com>

Optimizing updates by copying the dirty rectangle
only do not work because of double-buffering.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Max Reitz <mreitz@redhat.com>
---
 ui/sdl2-2d.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/ui/sdl2-2d.c b/ui/sdl2-2d.c
index d0b340f..191ee3b 100644
--- a/ui/sdl2-2d.c
+++ b/ui/sdl2-2d.c
@@ -45,10 +45,23 @@ void sdl2_2d_update(DisplayChangeListener *dcl,
         return;
     }
 
+    /*
+     * SDL2 seems to do some double-buffering, and trying to only
+     * update the changed areas results in only one of the two buffers
+     * being updated.  Which flickers alot.  So lets not try to be
+     * clever do a full update every time ...
+     */
+#if 0
     rect.x = x;
     rect.y = y;
     rect.w = w;
     rect.h = h;
+#else
+    rect.x = 0;
+    rect.y = 0;
+    rect.w = surface_width(surf);
+    rect.h = surface_height(surf);
+#endif
 
     SDL_UpdateTexture(scon->texture, NULL, surface_data(surf),
                       surface_stride(surf));
-- 
1.8.3.1

  parent reply	other threads:[~2015-10-09  8:18 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-09  8:18 [Qemu-devel] [PULL 00/11] virtio-gpu+gtk: add 3d rendering support Gerd Hoffmann
2015-10-09  8:18 ` [Qemu-devel] [PULL 01/11] shaders: initialize vertexes once Gerd Hoffmann
2015-10-09  8:18 ` Gerd Hoffmann [this message]
2015-10-09  8:18 ` [Qemu-devel] [PULL 03/11] ui/console: add opengl context and scanout support interfaces Gerd Hoffmann
2015-10-09  8:18 ` [Qemu-devel] [PULL 04/11] virtio-gpu: move iov free to virtio_gpu_cleanup_mapping_iov Gerd Hoffmann
2015-10-09  8:18 ` [Qemu-devel] [PULL 05/11] virtio-gpu: change licence from GPLv2 to GPLv2+ Gerd Hoffmann
2015-10-09  8:18 ` [Qemu-devel] [PULL 06/11] virtio-gpu: update headers for virgl/3d Gerd Hoffmann
2015-10-09  8:18 ` [Qemu-devel] [PULL 07/11] virtio-gpu: add 3d mode and virgl rendering support Gerd Hoffmann
2015-10-09  8:18 ` [Qemu-devel] [PULL 08/11] virtio-gpu: add cursor update tracepoint Gerd Hoffmann
2015-10-09  8:18 ` [Qemu-devel] [PULL 09/11] opengl: add egl-context.[ch] helpers Gerd Hoffmann
2015-10-09  8:18 ` [Qemu-devel] [PULL 10/11] gtk/opengl: add opengl context and scanout support (egl) Gerd Hoffmann
2015-10-09  8:18 ` [Qemu-devel] [PULL 11/11] gtk/opengl: add opengl context and scanout support (GtkGLArea) Gerd Hoffmann
2015-10-12 10:06 ` [Qemu-devel] [PULL 00/11] virtio-gpu+gtk: add 3d rendering support Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1444378714-15190-3-git-send-email-kraxel@redhat.com \
    --to=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.