[Libvir] [patch] qemu/kvm: use_system_powerdown instead of killing the vm hard

[Dan Kenigsberg]

On Wed, Jan 09, 2008 at 03:56:52PM +0000, Daniel P. Berrange wrote:
> On Wed, Jan 09, 2008 at 04:11:44PM +0100, Guido Guenther wrote:
> > On Tue, Jan 08, 2008 at 06:34:36PM +0000, Daniel P. Berrange wrote:
> > > Fair enough. Any idea if the 'system_reboot' command is also wired up
> > > in KVM yet ?  We should probably just add code to call it anyway since
> > > it'll be near identical code to that which you used for shutdown, and
> > > its better than leaving domainReboot driver method as a no-op.
> > There's no system_reboot in either kvm nor qemu. I see two solutions
> > for wireing up domainReboot(). Either qemu/kvm accepts a parameter to
> > system_powerdown to signal that we should restart the vm instead of
> > simply shutting down or we simply send 
> > "sendkey ctrl-alt-delete"
> > to the monitor which might be very fragile though. Both solutions depend
> > on the os in the vm doing the right thing (acpi event handling in the
> > first case).
> 
> Well there's a 'system_reset'  monitor command, but its unclear if
> its actually working - when i run it, it stops the guest from responding
> to any keyboard input and makes it take 100% CPU, but doesn't reboot :-(
> If that's not suitable i think the ctrl-alt-delete thing is probably the
> best we can do.

You are quicker than me... With an up-to-date kvm, system_reset seems to
be working fine (it was buggy in previous kvms).

Dan.