[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot |
Date: |
Wed, 8 Jul 2009 14:56:07 +0100 |
User-agent: |
Mutt/1.4.1i |
On Wed, Jul 08, 2009 at 08:47:19AM -0500, Anthony Liguori wrote:
> Daniel P. Berrange wrote:
> >
> >Only other option I think of is a pure paravirt shutdown/reboot
> >handler, ala Xen. That sucks in different ways though, namely
> >needing a driver in the guest, which is probably worse.
> >
>
> What do you think about Avi's suggestion of using -no-shutdown, doing a
> system_powerdown, waiting to receive a powerdown (you can poll 'info
> status' too to see when you entered into the paused state), and then
> doing a system_reset? I'd suggesting doing a timeout too while waiting
> to avoid waiting forever.
That suffers from the same problem as a system_reboot command in that
the guest may never start the shutdown. If that's an acceptable
limitation, then we might as well implement it directly in QEMU as
system_reboot so its easily available to everyone. Make 'system_reboot'
take an optional timeout arg (default to 60 seconds) after which it
cancels its plans.
Regards,
Dainel
--
|: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :|
|: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Gleb Natapov, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Anthony Liguori, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Daniel P. Berrange, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Anthony Liguori, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot,
Daniel P. Berrange <=
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Anthony Liguori, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Ryan Harper, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Anthony Liguori, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Ryan Harper, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Anthony Liguori, 2009/07/08
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Avi Kivity, 2009/07/08
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Amit Shah, 2009/07/16
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Jamie Lokier, 2009/07/16
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Amit Shah, 2009/07/16
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Gleb Natapov, 2009/07/08