From patchwork Fri Dec 2 19:19:20 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [V3] Guest stop notification Date: Fri, 02 Dec 2011 09:19:20 -0000 From: Eric B Munson X-Patchwork-Id: 128940 Message-Id: <1322853560-24152-1-git-send-email-emunson@mgebm.net> To: qemu-devel@nongnu.org Cc: ryanh@linux.vnet.ibm.com, aliguori@us.ibm.com, kvm@vger.kernel.org, Jan Kiszka , Marcelo Tosatti , Eric B Munson , Avi Kivity Often when a guest is stopped from the qemu console, it will report spurious soft lockup warnings on resume. There are kernel patches being discussed that will give the host the ability to tell the guest that it is being stopped and should ignore the soft lockup warning that generates. Signed-off-by: Eric B Munson Cc: Avi Kivity Cc: Marcelo Tosatti Cc: Jan Kiszka Cc: ryanh@linux.vnet.ibm.com Cc: aliguori@us.ibm.com Cc: kvm@vger.kernel.org --- Changes from V2: Move ioctl into hw/kvmclock.c so as other arches can use it as it is implemented Changes from V1: Remove unnecessary encapsulating function hw/kvmclock.c | 24 ++++++++++++++++++++++++ 1 files changed, 24 insertions(+), 0 deletions(-) diff --git a/hw/kvmclock.c b/hw/kvmclock.c index 5388bc4..756839f 100644 --- a/hw/kvmclock.c +++ b/hw/kvmclock.c @@ -16,6 +16,7 @@ #include "sysbus.h" #include "kvm.h" #include "kvmclock.h" +#include "cpu-all.h" #include #include @@ -69,11 +70,34 @@ static void kvmclock_vm_state_change(void *opaque, int running, } } +static void kvmclock_vm_state_change_vcpu(void *opaque, int running, + RunState state) +{ + int ret; + CPUState *penv = first_cpu; + + if (running) { + while (penv) { + ret = kvm_vcpu_ioctl(penv, KVM_GUEST_PAUSED, 0); + if (ret) { + if (ret != ENOSYS) { + fprintf(stderr, + "kvmclock_vm_state_change_vcpu: %s\n", + strerror(-ret)); + } + return; + } + penv = (CPUState *)penv->next_cpu; + } + } +} + static int kvmclock_init(SysBusDevice *dev) { KVMClockState *s = FROM_SYSBUS(KVMClockState, dev); qemu_add_vm_change_state_handler(kvmclock_vm_state_change, s); + qemu_add_vm_change_state_handler(kvmclock_vm_state_change_vcpu, NULL); return 0; }