[libvirt] [PATCH] qemu: Avoid loop of fake reboots

Jiri Denemark jdenemar at redhat.com
Wed Sep 21 14:58:56 UTC 2011


On Wed, Sep 21, 2011 at 08:17:32 -0600, Eric Blake wrote:
> On 09/21/2011 08:08 AM, Jiri Denemark wrote:
> > Once virDomainReboot is called for a domain, guest OS initiated shutdown
> > would always result in reboot instead of shutdown. Only
> > virDomainShutdown would actually shutd such domain down. That's because
> > we forgot to reset fakeReboot flag once we asked the domain to reboot.
> > ---
> >   src/qemu/qemu_process.c |    1 +
> >   1 files changed, 1 insertions(+), 0 deletions(-)
> >
> > diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> > index 3baaa19..bd49b21 100644
> > --- a/src/qemu/qemu_process.c
> > +++ b/src/qemu/qemu_process.c
> > @@ -445,6 +445,7 @@ qemuProcessHandleShutdown(qemuMonitorPtr mon ATTRIBUTE_UNUSED,
> >
> >       priv->gotShutdown = true;
> >       if (priv->fakeReboot) {
> > +        priv->fakeReboot = false;
> 
> ACK.  Looks like another code path similar to the bug fixed in c2e9fab2.

Yep. Thanks and pushed.

Jirka




More information about the libvir-list mailing list