Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot

[Ryan Harper <ryanh@us.ibm.com>]

* Daniel P. Berrange <berrange@redhat.com> [2009-07-08 08:58]:
> 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. 

This seems the most reasonable thing to me.  As already mentioned,
system_powerdown already is non-deterministic since the guest could
ignore the ACPI event.



-- 
Ryan Harper
Software Engineer; Linux Technology Center
IBM Corp., Austin, Tx
ryanh@us.ibm.com