Re: [Qemu-devel] [Bug 1021649] Re: qemu 1.1.0 waits for a keypress at boot

[Stefano Stabellini]

On Wed, 25 Jul 2012, Michael Tokarev wrote:
> Stefano, Paul, can you take a look please?
> 
> https://bugs.launchpad.net/bugs/1021649

That is a very good bug triage that you did!

However "main_loop_wait: block indefinitely" only increases the maximum
select timeout of QEMU's main_loop.
That mean that if one or more emulators have bugs and don't get
notifications correctly they might hang.
The reason why it only reproduces with nographic is that both sdl and vnc
introduce a gui_timer that wakes QEMU up every 30ms.

So the question is: why is kernel_irqchip=on required to repro the bug?
It strikes me as a bug in kernel_irqchip that prevents QEMU from being
waken up when it should.

CC'ing Avi.


> Thanks,
> 
> /mjt
> 
> On 25.07.2012 12:25, Michael Tokarev wrote:
> > Jamie Heilman (the debian bug #680719 reporter) bisected this issue to
> > this commit:
> > 
> > 7c7db75576bd5a31508208f153c5aada64b2c8df is the first bad commit
> > commit 7c7db75576bd5a31508208f153c5aada64b2c8df
> > Author: Stefano Stabellini <address@hidden>
> > Date:   Fri Apr 13 19:35:04 2012 +0100
> > 
> >     main_loop_wait: block indefinitely
> >     
> >     - remove qemu_calculate_timeout;
> >     
> >     - explicitly size timeout to uint32_t;
> >     
> >     - introduce slirp_update_timeout;
> >     
> >     - pass NULL as timeout argument to select in case timeout is the maximum
> >     value;
> >     
> >     Signed-off-by: Stefano Stabellini <address@hidden>
> >     Acked-by: Paul Brook <address@hidden>
> >     Signed-off-by: Anthony Liguori <address@hidden>
> > 
> 
>