Re: [Qemu-devel] [PATCH v2 4/5] console: pass Monitor to vga_hw_screen_dump/hw_vga_dump

[Gerd Hoffmann <kraxel@redhat.com>]

  Hi,

>> Some solutions that come to my mind:
>>
>>  1. Pool the screendump file creation from a timer.
>>
>>     Cons: it may return before the file is fully written to disk
>>
> 
> We know what the file size should be, so we can poll for the actual
> size. Actually why do we need to poll? we could add a
> "internal.screendump.complete" or "internal-query-screendump", no?

Marc-Andre currently looks at adding support for other file formats.
I think it would be good to team up with him.

First, with this applied you will not know the size in advance.  Also
one of the approaches discussed is to allow passing in a file handle.
That is a possible way to handle async screendumps too: just write to
the passed file handle and close it when done.  Obvious drawback is that
this will not cover the classic way of specifying the output filename as
argument.

cheers,
  Gerd