Patchwork [v2,22/24] kvm: Leave kvm_cpu_exec directly after KVM_EXIT_SHUTDOWN

login
register
mail settings
Submitter Jan Kiszka
Date Feb. 1, 2011, 9:16 p.m.
Message ID <cb84efb7be0666f5888434ce83c156c13559bbfc.1296594961.git.jan.kiszka@web.de>
Download mbox | patch
Permalink /patch/81404/
State New
Headers show

Comments

Jan Kiszka - Feb. 1, 2011, 9:16 p.m.
From: Jan Kiszka <jan.kiszka@siemens.com>

The reset we issue on KVM_EXIT_SHUTDOWN implies that we should also
leave the VCPU loop. As we now check for exit_request which is set by
qemu_system_reset_request, this bug is no longer critical. Still it's an
unneeded extra turn.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
---
 kvm-all.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

Patch

diff --git a/kvm-all.c b/kvm-all.c
index cf54256..35860df 100644
--- a/kvm-all.c
+++ b/kvm-all.c
@@ -959,7 +959,6 @@  int kvm_cpu_exec(CPUState *env)
         case KVM_EXIT_SHUTDOWN:
             DPRINTF("shutdown\n");
             qemu_system_reset_request();
-            ret = 1;
             break;
         case KVM_EXIT_UNKNOWN:
             fprintf(stderr, "KVM: unknown exit, hardware reason %" PRIx64 "\n",