From: Rafael da Veiga Cabral <verganic@gmail.com>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] PNG screendump alternative
Date: Mon, 19 Mar 2012 22:11:08 -0300 [thread overview]
Message-ID: <4F67D92C.6040203@gmail.com> (raw)
In-Reply-To: <4F67A0CC.1030009@codemonkey.ws>
On 03/19/2012 06:10 PM, Anthony Liguori wrote:
> On 03/17/2012 05:45 PM, Rafael wrote:
>> Hi,
>>
>> I had already started to work on supporting PNG dump output on
>> screendump when got to know there was another effort to do the same
>> (see Add PNG screendump thread from Daniel P. Berrange). Anyway, I'd
>> like to share my implementation which is a little bit different. It
>> does not use any resource/lib besides libpng and qemu itself and is
>> based on qemu-kvm 0.15.1 ppm_save code.
>>
>> I'm using that for a personal project developed at Mandriva which uses
>> QMP directly to communicate with qemu VMs rather than using the
>> libvirt although its usage is considered for the future. There is
>> something more to show but it is just QMP related and soon I'll have
>> it done to share.
>>
>> Comments and suggestions are appreciated.
>
> I think the added value of this is minimal compared to the alternative
> implementation (just run convert on the PPM).
>
> Can you talk a bit more about your use-case any why post-processing
> the PPM isn't a reasonable alternative?
>
It's because my use-case requires a screenshoot on each second or less.
It's a functional testing tool based on what is happening on the vm
screen. The testing stuff is based on basic image processing of the
screens. Therefore as screen are grabbed all time the less CPU and Disk
I/O are required the better, for example, by changing from PPM to PNG
at least 2.2 Megas of disk writing are saved on each second. Think about
a session of 15 minutes or more (15 * 60 * 2.2). I've also thought
that by saving directly from the screen buffer to png should save some
CPU cycles rather than saving to PPM first and converting it to PNG on
the qemu runtime.
> Regards,
>
> Anthony Liguori
>
>>
>> Cheers
>> Rafael Cabral
>
Thanks Anthony
Rafael Cabral
prev parent reply other threads:[~2012-03-20 1:11 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-17 22:45 [Qemu-devel] [PATCH] PNG screendump alternative Rafael
2012-03-19 20:50 ` Luiz Capitulino
2012-03-20 0:57 ` Rafael da Veiga Cabral
2012-03-19 21:10 ` Anthony Liguori
2012-03-20 1:11 ` Rafael da Veiga Cabral [this message]
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=4F67D92C.6040203@gmail.com \
--to=verganic@gmail.com \
--cc=anthony@codemonkey.ws \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).