Skip to contentSkip to conversation

avoid clobbering exit_reason

PATCH New Hollis Blanchard 1 participant 1 message Mar 8, 2008

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 avoid clobbering exit_reason

FYI, here's the fix for another nasty yet dumb bug in which we were losing MMIOs by clobbering exit_reason when a signal is pending:

arch/powerpc/kvm/powerpc.c +3 -1
@@ -388,7 +388,9 @@ int kvmppc_handle_exit(struct kvm_run *r
 
        kvmppc_check_and_deliver_interrupts(vcpu);
 
-       if (signal_pending(current)) {
+       /* Only check for signals if we aren't already exiting for some other
+        * reason. This keeps us from clobbering exit_reason. */
+       if (!(r & RESUME_HOST) && signal_pending(current)) {
                run->exit_reason = KVM_EXIT_INTR;
                r = (-EINTR << 2) | RESUME_HOST | (r & RESUME_FLAG_NV);
        }

As far as I know, there are currently no more showstopper bugs, so we can get back to making network IO work.

Updated patches are in the usual place: http://penguinppc.org/~hollisb/kvm/

Signature · 12 lines
Hollis Blanchard
IBM Linux Technology Center


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
kvm-ppc-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/kvm-ppc-devel