x86/shadow: Drop stale adjustment in the PAE second-half search
authorAndrew Cooper <andrew.cooper3@citrix.com>
Fri, 2 Dec 2016 18:23:02 +0000 (18:23 +0000)
committerAndrew Cooper <andrew.cooper3@citrix.com>
Mon, 5 Dec 2016 14:29:01 +0000 (14:29 +0000)
This shouldn't have been present in c/s 29a57c992 "x86/emul: Rework emulator
event injection".  It was a leftover from a previous version of the series.

This conditional has no effect on the behaviour following it, as both
X86EMUL_EXCEPTION and X86EMUL_UNHANDLEABLE fall into the same "return back to
guest" path.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Acked-by: Tim Deegan <tim@xen.org>
xen/arch/x86/mm/shadow/multi.c

index 2696396e42eea7c2f86c8fe72dc9c956f769b85c..f494f7bd38b3913f9b22812e130c59db5bb8928f 100644 (file)
@@ -3475,9 +3475,6 @@ static int sh_page_fault(struct vcpu *v,
             v->arch.paging.last_write_was_pt = 0;
             r = x86_emulate(&emul_ctxt.ctxt, emul_ops);
 
-            if ( r == X86EMUL_EXCEPTION && emul_ctxt.ctxt.event_pending )
-                r = X86EMUL_UNHANDLEABLE;
-
             /*
              * Only continue the search for the second half if there are no
              * exceptions or pending actions.  Otherwise, give up and re-enter