[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] vnc regression with -vga vmware
From: |
Serge Hallyn |
Subject: |
Re: [Qemu-devel] vnc regression with -vga vmware |
Date: |
Mon, 17 Mar 2014 12:35:19 -0500 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Quoting Peter Lieven (address@hidden):
>
> On 17.03.2014 16:19, Serge Hallyn wrote:
> >Quoting Peter Lieven (address@hidden):
> >>I found 2 issues:
> >>
> >>- with vmware VGA the server surface and the client desktop size are out of
> >>sync for some time
> >>at a resolution change. the server surface gets updates for x coordinates
> >>that are out of bound
> >>for the client.
> >>- the max width of the client (2360) is not dividable by 16
> >>(VNC_DIRTY_PIXELS_PER_BIT).
> >>
> >>I will try to fix this in ui/vnc but we should definetly look for the root
> >>cause.
> >Thanks, Peter!
> >
> >-serge
>
>
> The vmware vga driver seems to do some nasty things.
> I receive the msg like this (independent of ui/vnc):
> vmsvga_update_rect: update x was < 0 (-65)
>
> Can you try the following patch:
Indeed, with this patch it seems quite stable, thanks.
-serge
> diff --git a/hw/display/vmware_vga.c b/hw/display/vmware_vga.c
> index bd2c108..6ae3348 100644
> --- a/hw/display/vmware_vga.c
> +++ b/hw/display/vmware_vga.c
> @@ -25,6 +25,7 @@
> #include "hw/loader.h"
> #include "trace.h"
> #include "ui/console.h"
> +#include "ui/vnc.h"
> #include "hw/pci/pci.h"
>
> #undef VERBOSE
> @@ -218,7 +219,7 @@ enum {
>
> /* These values can probably be changed arbitrarily. */
> #define SVGA_SCRATCH_SIZE 0x8000
> -#define SVGA_MAX_WIDTH 2360
> +#define SVGA_MAX_WIDTH ROUND_UP(2360,
> VNC_DIRTY_PIXELS_PER_BIT)
> #define SVGA_MAX_HEIGHT 1770
>
> #ifdef VERBOSE
> diff --git a/ui/vnc.c b/ui/vnc.c
> index 9c84b3e..5925774 100644
> --- a/ui/vnc.c
> +++ b/ui/vnc.c
> @@ -888,7 +888,7 @@ static int vnc_update_client(VncState *vs, int has_dirty,
> bool sync)
> VncDisplay *vd = vs->vd;
> VncJob *job;
> int y;
> - int height;
> + int height, width;
> int n = 0;
>
> if (vs->output.offset && !vs->audio_cap && !vs->force_update)
> @@ -907,6 +907,7 @@ static int vnc_update_client(VncState *vs, int has_dirty,
> bool sync)
> job = vnc_job_new(vs);
>
> height = MIN(pixman_image_get_height(vd->server), vs->client_height);
> + width = MIN(pixman_image_get_width(vd->server), vs->client_width);
>
> y = 0;
> for (;;) {
> @@ -925,8 +926,11 @@ static int vnc_update_client(VncState *vs, int
> has_dirty, bool sync)
> VNC_DIRTY_BPL(vs), x);
> bitmap_clear(vs->dirty[y], x, x2 - x);
> h = find_and_clear_dirty_height(vs, y, x, x2, height);
> - n += vnc_job_add_rect(job, x * VNC_DIRTY_PIXELS_PER_BIT, y,
> - (x2 - x) * VNC_DIRTY_PIXELS_PER_BIT, h);
> + x2 = MIN(x2, width / VNC_DIRTY_PIXELS_PER_BIT);
> + if (x2 > x) {
> + n += vnc_job_add_rect(job, x * VNC_DIRTY_PIXELS_PER_BIT, y,
> + (x2 - x) * VNC_DIRTY_PIXELS_PER_BIT,
> h);
> + }
> }
>
> vnc_job_push(job);
>
> Peter
>
>
>
- [Qemu-devel] vnc regression with -vga vmware, Serge Hallyn, 2014/03/14
- Re: [Qemu-devel] vnc regression with -vga vmware, Gerd Hoffmann, 2014/03/17
- Re: [Qemu-devel] vnc regression with -vga vmware, Peter Lieven, 2014/03/17
- Re: [Qemu-devel] vnc regression with -vga vmware, Peter Lieven, 2014/03/17
- Re: [Qemu-devel] vnc regression with -vga vmware, Serge Hallyn, 2014/03/17
- Re: [Qemu-devel] vnc regression with -vga vmware, Peter Lieven, 2014/03/17
- Re: [Qemu-devel] vnc regression with -vga vmware, Serge Hallyn, 2014/03/17
- Re: [Qemu-devel] vnc regression with -vga vmware, Peter Lieven, 2014/03/17
- Re: [Qemu-devel] vnc regression with -vga vmware,
Serge Hallyn <=
- Re: [Qemu-devel] vnc regression with -vga vmware, Peter Lieven, 2014/03/17