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

[Daniel P. Berrange]

On Wed, Jan 09, 2008 at 06:18:50PM +0200, Dan Kenigsberg wrote:
> 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).

Ok, I'm using an ancient KVM from Fedora 8 GA - kvm-36. On the basis that
recent KVM does work, and we currently have no-impl of the reboot API to
causes regressions on, I say we use  system_reset for QEMU/KVM.

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=|