On Wed, Feb 22, 2012 at 02:58:10PM +0100, Gerd Hoffmann wrote: > > And the only user of screen_dump_filename is: > > > > static void vga_save_dpy_update(DisplayState *ds, > > int x, int y, int w, int h) > > { > > if (screen_dump_filename) { > > ppm_save(screen_dump_filename, ds->surface); > > upstream/master this here: > > screen_dump_filename = NULL; >
That's wrong, you'll get the screenshot after the first update, who's to say it is fully rendered? > > } > > } > > > This seems to work, only tested with -vga qxl and in vga mode: > > The corner case where this fails is when console switching is needed, > i.e. switch to monitor console via ctrl-alt-2, then type the screenshot > command there ... Are you talking about sdl console or linux guest console? ctrl-alt-2 or ctrl-alt-F2? I can try both. > > _______________________________________________ > Spice-devel mailing list > spice-de...@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/spice-devel