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

2008-01-14 Thread Daniel P. Berrange
On Mon, Jan 14, 2008 at 02:01:40PM +0100, Guido Guenther wrote: > On Mon, Jan 14, 2008 at 12:28:09PM +, Richard W.M. Jones wrote: > > Guido Guenther wrote: > >> "sendkey ctrl-alt-delete" > >> to the monitor which might be very fragile though. Both solutions depend > >> on the os in the vm doing

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

2008-01-14 Thread Guido Guenther
On Mon, Jan 14, 2008 at 12:28:09PM +, Richard W.M. Jones wrote: > Guido Guenther wrote: >> "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). > > That's no

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

2008-01-14 Thread Richard W.M. Jones
Guido Guenther wrote: On Wed, Jan 09, 2008 at 03:56:52PM +, Daniel P. Berrange wrote: 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 rebo

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

2008-01-14 Thread Richard W.M. Jones
Guido Guenther wrote: "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). That's not going to work on Windows NT. Rich. -- Emerging Technologies, Red Hat - http://

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

2008-01-09 Thread Guido Guenther
On Wed, Jan 09, 2008 at 03:56:52PM +, Daniel P. Berrange wrote: > 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

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

2008-01-09 Thread 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 +, 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 +, Daniel P. Berrange wrote: > > > > Fair enough. An

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

2008-01-09 Thread Dan Kenigsberg
On Wed, Jan 09, 2008 at 03:56:52PM +, 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 +, Daniel P. Berrange wrote: > > > Fair enough. Any idea if the 'system_reboot' command is also wired up > > > in KVM yet

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

2008-01-09 Thread Daniel P. Berrange
On Tue, Jan 08, 2008 at 12:15:56PM +0100, Guido Guenther wrote: > Hi, > currently domainShutdown kills qemu/kvm instances hard which is not very > filesystem friendly. However recent kvm git acquired system_powerdown to > shutdown the system gracefully by simulating an acpi power button press. > We

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

2008-01-09 Thread Dan Kenigsberg
On Wed, Jan 09, 2008 at 04:11:44PM +0100, Guido Guenther wrote: > On Tue, Jan 08, 2008 at 06:34:36PM +, 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 nea

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

2008-01-09 Thread Daniel P. Berrange
On Wed, Jan 09, 2008 at 04:11:44PM +0100, Guido Guenther wrote: > On Tue, Jan 08, 2008 at 06:34:36PM +, 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 nea

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

2008-01-09 Thread Guido Guenther
On Tue, Jan 08, 2008 at 06:34:36PM +, 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 t

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

2008-01-08 Thread Daniel P. Berrange
On Tue, Jan 08, 2008 at 05:55:45PM +0100, Guido Guenther wrote: > On Tue, Jan 08, 2008 at 04:37:02PM +, Daniel P. Berrange wrote: > > On Tue, Jan 08, 2008 at 12:15:56PM +0100, Guido Guenther wrote: > > > Hi, > > > currently domainShutdown kills qemu/kvm instances hard which is not very > > > fi

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

2008-01-08 Thread Guido Guenther
On Tue, Jan 08, 2008 at 04:37:02PM +, Daniel P. Berrange wrote: > On Tue, Jan 08, 2008 at 12:15:56PM +0100, Guido Guenther wrote: > > Hi, > > currently domainShutdown kills qemu/kvm instances hard which is not very > > filesystem friendly. However recent kvm git acquired system_powerdown to > >

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

2008-01-08 Thread Daniel P. Berrange
On Tue, Jan 08, 2008 at 12:15:56PM +0100, Guido Guenther wrote: > Hi, > currently domainShutdown kills qemu/kvm instances hard which is not very > filesystem friendly. However recent kvm git acquired system_powerdown to > shutdown the system gracefully by simulating an acpi power button press. > We

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

2008-01-08 Thread Guido Guenther
Hi, currently domainShutdown kills qemu/kvm instances hard which is not very filesystem friendly. However recent kvm git acquired system_powerdown to shutdown the system gracefully by simulating an acpi power button press. We can now use this in libvirt: diff --git a/src/qemu_driver.c b/src/qemu_