From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:38928) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S78lr-0001jL-6X for qemu-devel@nongnu.org; Mon, 12 Mar 2012 13:10:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1S78lm-00020X-2Q for qemu-devel@nongnu.org; Mon, 12 Mar 2012 13:10:54 -0400 Received: from mx1.redhat.com ([209.132.183.28]:18651) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S78ll-00020P-Qm for qemu-devel@nongnu.org; Mon, 12 Mar 2012 13:10:50 -0400 Date: Mon, 12 Mar 2012 17:10:43 +0000 From: "Daniel P. Berrange" Message-ID: <20120312171043.GC20435@redhat.com> References: <1331557893-30806-1-git-send-email-marcandre.lureau@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1331557893-30806-1-git-send-email-marcandre.lureau@redhat.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH/RFC 0/7] Screendump to UNIX socket & in PNG format Reply-To: "Daniel P. Berrange" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?utf-8?Q?Marc-Andr=C3=A9?= Lureau Cc: =?utf-8?Q?Marc-Andr=C3=A9?= Lureau , qemu-devel@nongnu.org On Mon, Mar 12, 2012 at 02:11:26PM +0100, Marc-Andr=C3=A9 Lureau wrote: > Hi, >=20 > The current screendump command can only save to disk very large PPM fil= es. >=20 > The following patches add support for screendump in a UNIX socket, > following the syntax used for migration URI: "unix:/path/to/socket". To facilitate interaction with security frameworks like SELinbux and AppArmour, it is even better for the mgmt app to just be able to pass a pre-opened FD across the monitor, rather than using UNIX sockets. > The last 3 patches add support for dumping in PNG format. This can > reduce the size of the image by a great factor (x10 is not unusual), > and is also a more convenient format than PPM. Currently, it dumps > in PNG by checking if the path ends with ".png", we may want to > have a seperate option for that instead, or use this syntax only > if the path begins with "unix:" (or any foo:) for example. We can let it guess format by default, but we should provide a way to force the format via an optional arg. > Next, I would like to add support for a scaling factor too (a typical > use case is to show a small thumbnail of the desktop). Specifying > only the requested "width" or "height" should be supported. Should > we rely on pixman to do this work? My suggestion would be to use Gdk-Pixbuf, since that also gives you suppo= rt for saving to arbitrary file formats, as well as scaling, etc. Daniel --=20 |: http://berrange.com -o- http://www.flickr.com/photos/dberrange= / :| |: http://libvirt.org -o- http://virt-manager.or= g :| |: http://autobuild.org -o- http://search.cpan.org/~danberr= / :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vn= c :|