Changeset 97117 in vbox

Timestamp:

Oct 12, 2022 3:08:10 PM (2 years ago)

Author:

vboxsync

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

154079

Message:

IEM: Removed todo, testcase confirmed that CMPXCHG8B implementation was right.

Location:

trunk/src/VBox/VMM/VMMAll

Files:

• IEMAll.cpp (modified) (1 diff)
• IEMAllInstructionsTwoByte0f.cpp.h (modified) (1 diff)

trunk/src/VBox/VMM/VMMAll/IEMAll.cpp

@@ -4171 +4171 @@
 VBOXSTRICTRC iemRaiseMathFault(PVMCPUCC pVCpu)
 {
-    return iemRaiseXcptOrInt(pVCpu, 0, X86_XCPT_MF, IEM_XCPT_FLAGS_T_CPU_XCPT, 0, 0);
+    if (pVCpu->cpum.GstCtx.cr0 & X86_CR0_NE)
+        return iemRaiseXcptOrInt(pVCpu, 0, X86_XCPT_MF, IEM_XCPT_FLAGS_T_CPU_XCPT, 0, 0);
+    else
+    {
+        /* Convert a #MF into a FERR -> IRQ 13. See @bugref{6117}. */
+        PDMIsaSetIrq(pVCpu->CTX_SUFF(pVM), 13 /* u8Irq */, 1 /* u8Level */, 0 /* uTagSrc */);
+        iemRegUpdateRipAndClearRF(pVCpu);
+        return VINF_SUCCESS;
+    }
 }
 

trunk/src/VBox/VMM/VMMAll/IEMAllInstructionsTwoByte0f.cpp.h

@@ -12142 +12142 @@
     IEM_MC_COMMIT_EFLAGS(EFlags);
     IEM_MC_IF_EFL_BIT_NOT_SET(X86_EFL_ZF)
-        /** @todo Testcase: Check effect of cmpxchg8b on bits 63:32 in rax and rdx. */
         IEM_MC_STORE_GREG_U32(X86_GREG_xAX, u64EaxEdx.s.Lo);
         IEM_MC_STORE_GREG_U32(X86_GREG_xDX, u64EaxEdx.s.Hi);