From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1KYnmx-0003ks-PB
	for qemu-devel@nongnu.org; Thu, 28 Aug 2008 16:08:15 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1KYnmw-0003kN-9M
	for qemu-devel@nongnu.org; Thu, 28 Aug 2008 16:08:15 -0400
Received: from [199.232.76.173] (port=52015 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1KYnmw-0003kI-1f
	for qemu-devel@nongnu.org; Thu, 28 Aug 2008 16:08:14 -0400
Received: from wx-out-0506.google.com ([66.249.82.229]:47875)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <anthony@codemonkey.ws>) id 1KYnmv-0001hc-3e
	for qemu-devel@nongnu.org; Thu, 28 Aug 2008 16:08:13 -0400
Received: by wx-out-0506.google.com with SMTP id h29so169091wxd.4
	for <qemu-devel@nongnu.org>; Thu, 28 Aug 2008 13:08:11 -0700 (PDT)
Message-ID: <48B7057B.2050203@codemonkey.ws>
Date: Thu, 28 Aug 2008 15:07:23 -0500
From: Anthony Liguori <anthony@codemonkey.ws>
MIME-Version: 1.0
Subject: Re: [Qemu-devel] [PATCH 0 of 3] vnc and vga improvements
References: <48B6C8FB.9090106@eu.citrix.com>
In-Reply-To: <48B6C8FB.9090106@eu.citrix.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Reply-To: qemu-devel@nongnu.org
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: qemu-devel@nongnu.org

Stefano Stabellini wrote:
> Hi all,
> This is a three patch series coming from xen-unstable to improve vnc
> and vga performances.
>
> The first patch implements dynamic colour depth changes in vnc.c:
> this way the vnc server can change its own internal colour depth at run
> time to follow any guest resolution change.
>
> The second patch implements the WMVi vnc extension in the qemu vnc
> server, so that we can also notify a vnc client when we change
> internal colour depth and offload any possible colour conversion to the
> client.
>
> The third patch implements sharing of the display pixel buffer between
> vnc.c and vga.c, in order to save a lot of memcpy's.
>
> The idea is that vnc.c (and in the near future sdl.c too) strictly
> follows the guest display resolution and notifies the client of any
> change. As a consequence we can save two colour conversions: one between
> vga and vnc, another one between the vnc server and the vnc client.
>   

I have mixed feelings about this.  On the one hand, reducing the copying 
is a good thing.  On the other hand, we pretty much make it impossible 
to ever support multiple clients.

I'm going to be travelling tomorrow but I'll look through these patches 
in more detail over the weekend.

Regards,

Anthony Liguori

> For the moment this is all, but I'll start working on the sdl shared
> buffer in the next days.
>
> Cheers,
>
> Stefano Stabellini
>
>
>