Changeset 72253 in vbox

Timestamp:

May 17, 2018 6:33:58 PM (7 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

122700

Message:

NEM: Handle HvMessageTypeUnrecoverableException. bugref:9044

Location:

trunk

Files:

• include/iprt/nt/hyperv.h (modified) (1 diff)
• src/VBox/VMM/VMMAll/NEMAllNativeTemplate-win.cpp.h (modified) (4 diffs)

trunk/include/iprt/nt/hyperv.h

@@ -1470 +1470 @@
         uint64_t                            Payload[HV_MESSAGE_MAX_PAYLOAD_QWORD_COUNT];
 
-        /** Common header for X64 intercept messages. */
+        /** Common header for X64 intercept messages.
+         * The HvMessageTypeUnrecoverableException message only has this.  */
         HV_X64_INTERCEPT_MESSAGE_HEADER     X64InterceptHeader;
         /** HvMessageTypeGpaIntercept, HvMessageTypeUnmappedGpa. */

trunk/src/VBox/VMM/VMMAll/NEMAllNativeTemplate-win.cpp.h

@@ -1411 +1411 @@
  * @param   pVCpu           The cross context per CPU structure.
  * @param   pMsg            The message.
+ * @param   pCtx            The register context.
  * @param   pGVCpu          The global (ring-0) per CPU structure (NULL in r3).
  */
@@ -1534 +1535 @@
  * @param   pVCpu           The cross context per CPU structure.
  * @param   pMsg            The message.
+ * @param   pCtx            The register context.
  * @param   pGVCpu          The global (ring-0) per CPU structure (NULL in r3).
  */
@@ -1559 +1561 @@
     RT_NOREF(pVM, pGVCpu);
     return VINF_SUCCESS;
+}
+
+
+/**
+ * Deals with unrecoverable exception (triple fault).
+ *
+ * @returns Strict VBox status code.
+ * @param   pVCpu           The cross context per CPU structure.
+ * @param   pMsgHdr         The message header.
+ * @param   pCtx            The register context.
+ */
+NEM_TMPL_STATIC VBOXSTRICTRC nemHCWinHandleMessageUnrecoverableException(PVMCPU pVCpu,
+                                                                         HV_X64_INTERCEPT_MESSAGE_HEADER const *pMsgHdr,
+                                                                         PCPUMCTX pCtx)
+{
+    /*
+     * Assert message sanity.
+     */
+    //Assert(   pMsgHdr->InterceptAccessType == HV_INTERCEPT_ACCESS_EXECUTE
+    //       || pMsgHdr->InterceptAccessType == HV_INTERCEPT_ACCESS_READ   // READ & WRITE are probably not used here
+    //       || pMsgHdr->InterceptAccessType == HV_INTERCEPT_ACCESS_WRITE);
+    AssertMsg(pMsgHdr->InstructionLength < 0x10, ("%#x\n", pMsgHdr->InstructionLength));
+
+    /*
+     * Just copy the state we've got and handle it in the loop for now.
+     */
+    nemHCWinCopyStateFromX64Header(pVCpu, pCtx, pMsgHdr);
+    Log(("TripleExit/%u: %04x:%08RX64: RFL=%#RX64 -> VINF_EM_TRIPLE_FAULT\n",
+         pVCpu->idCpu, pMsgHdr->CsSegment.Selector, pMsgHdr->Rip, pMsgHdr->Rflags));
+    return VINF_EM_TRIPLE_FAULT;
 }
 
@@ -1606 +1638 @@
                 return nemHCWinHandleMessageInterruptWindow(pVM, pVCpu, &pMsg->X64InterruptWindow, pCtx, pGVCpu);
 
+            case HvMessageTypeUnrecoverableException:
+                Assert(pMsg->Header.PayloadSize == sizeof(pMsg->X64InterceptHeader));
+                return nemHCWinHandleMessageUnrecoverableException(pVCpu, &pMsg->X64InterceptHeader, pCtx);
+
             case HvMessageTypeInvalidVpRegisterValue:
-            case HvMessageTypeUnrecoverableException:
             case HvMessageTypeUnsupportedFeature:
             case HvMessageTypeTlbPageSizeMismatch: