qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] Rate limit vnc_write_pixels_generic


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PATCH] Rate limit vnc_write_pixels_generic
Date: Mon, 07 May 2012 09:19:57 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20120329 Thunderbird/11.0.1

On 05/07/2012 08:58 AM, Philipp Hahn wrote:
Only print the error message once per change and also include the actual
unsupported color depth in bytes per pixel in the error message.

Signed-off-by: Philipp Hahn<address@hidden>

Under what conditions are you seeing this happen?

Regards,

Anthony Liguori

---
  ui/vnc.c |    8 +++++++-
  1 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/ui/vnc.c b/ui/vnc.c
index deb9ecd..9dcff9b 100644
--- a/ui/vnc.c
+++ b/ui/vnc.c
@@ -641,6 +641,7 @@ static void vnc_write_pixels_generic(VncState *vs, struct 
PixelFormat *pf,
                                       void *pixels1, int size)
  {
      uint8_t buf[4];
+    static uint8_t last_error_bpp = 4; /* any of the supported formats */

      if (pf->bytes_per_pixel == 4) {
          uint32_t *pixels = pixels1;
@@ -667,7 +668,12 @@ static void vnc_write_pixels_generic(VncState *vs, struct 
PixelFormat *pf,
              vnc_write(vs, buf, vs->clientds.pf.bytes_per_pixel);
          }
      } else {
-        fprintf(stderr, "vnc_write_pixels_generic: VncState color depth not 
supported\n");
+        if (last_error_bpp != pf->bytes_per_pixel) {
+            last_error_bpp = pf->bytes_per_pixel;
+            fprintf(stderr,
+                    "vnc_write_pixels_generic: VncState color depth %d not 
supported\n",
+                    pf->bytes_per_pixel);
+        }
      }
  }





reply via email to

[Prev in Thread] Current Thread [Next in Thread]