[Qemu-devel] [BUG latest master] - qemu segfaults when issuing screendump cmd

[Qemu-devel] [BUG latest master] - qemu segfaults when issuing screendump cmd

[Lucas Meneghel Rodrigues]

Latest qemu.git master is failing big time to pass sanity checks:

https://bugs.launchpad.net/qemu/+bug/1169254

We've had some issues with the test jobs, and finally managed to
stabilize the grid, so we don't have results for the last couple of
weeks to bisect the problem.

Re: [Qemu-devel] [BUG latest master] - qemu segfaults when issuing screendump cmd

[Gerd Hoffmann]

On 04/15/13 18:49, Lucas Meneghel Rodrigues wrote:
> Latest qemu.git master is failing big time to pass sanity checks:
> 
> https://bugs.launchpad.net/qemu/+bug/1169254
> 
> We've had some issues with the test jobs, and finally managed to
> stabilize the grid, so we don't have results for the last couple of
> weeks to bisect the problem.

Can you try this?
  git://git.kraxel.org/qemu rebase/pixman

[ I'm about to prepare a pull req, was planning
   to do it last week but got sick ... ]

cheers,
  Gerd

Re: [Qemu-devel] [BUG latest master] - qemu segfaults when issuing screendump cmd

[Lucas Meneghel Rodrigues]

On Tue, 2013-04-16 at 09:39 +0200, Gerd Hoffmann wrote:
> On 04/15/13 18:49, Lucas Meneghel Rodrigues wrote:
> > Latest qemu.git master is failing big time to pass sanity checks:
> > 
> > https://bugs.launchpad.net/qemu/+bug/1169254
> > 
> > We've had some issues with the test jobs, and finally managed to
> > stabilize the grid, so we don't have results for the last couple of
> > weeks to bisect the problem.
> 
> Can you try this?
>   git://git.kraxel.org/qemu rebase/pixman
> 
> [ I'm about to prepare a pull req, was planning
>    to do it last week but got sick ... ]

Yes, in your branch, the problem is solved. Anthony came up with an even
simpler reproducer:

$ (sleep 1; echo -e 'screendump abc.ppm\nquit') | MALLOC_PERTURB_=1 x86_64-softmmu/qemu-system-x86_64 -monitor stdio -vnc :0
QEMU 1.4.50 monitor - type 'help' for more information
(qemu) screendump abc.ppm
(qemu) quit

With current master, qemu crashes.

[lmr@thinkpad-t420s qemu-gerd]$ (sleep 1; echo -e 'screendump abc.ppm\nquit') | MALLOC_PERTURB_=1 x86_64-softmmu/qemu-system-x86_64 -monitor stdio -vnc :0
QEMU 1.4.50 monitor - type 'help' for more information
(qemu) screendump abc.ppm
Segmentation fault

Re: [Qemu-devel] [BUG latest master] - qemu segfaults when issuing screendump cmd

[Gerd Hoffmann]

>> Can you try this?
>>   git://git.kraxel.org/qemu rebase/pixman
>>

> With current master, qemu crashes.

FYI: pull req sent, anthony merged, so it is fixed in master now.

autotest might need adaptions nevertheless, on headless configurations
(-vga none) qemu starts throwing errors now (due to lack of a screen it
can dump from) instead of silently doing nothing.

cheers,
  Gerd

Re: [Qemu-devel] [BUG latest master] - qemu segfaults when issuing screendump cmd

[Lucas Meneghel Rodrigues]

On Thu, 2013-04-18 at 08:53 +0200, Gerd Hoffmann wrote:
> >> Can you try this?
> >>   git://git.kraxel.org/qemu rebase/pixman
> >>
> 
> > With current master, qemu crashes.
> 
> FYI: pull req sent, anthony merged, so it is fixed in master now.
> 
> autotest might need adaptions nevertheless, on headless configurations
> (-vga none) qemu starts throwing errors now (due to lack of a screen it
> can dump from) instead of silently doing nothing.

Oh, absolutely. An error at least is normal program operation/behavior,
so we'll adapt it. It's rather different than a segmentation fault,
which is a bug regardless of the circumstances.

Thanks, I'll verify and mark the bug as fixed.