Changeset 105098 in vbox

Timestamp:

Jul 2, 2024 12:46:06 PM (5 months ago)

Author:

vboxsync

Message:

VMM/IEM: iret logging. bugref:10715

File:

• trunk/src/VBox/VMM/VMMAll/IEMAllCImpl.cpp (modified) (21 diffs)

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

@@ -3451 +3451 @@
     if (pVCpu->cpum.GstCtx.eflags.Bits.u1NT)
     {
-        Log(("iretq with NT=1 (eflags=%#x) -> #GP(0)\n", pVCpu->cpum.GstCtx.eflags.u));
+        Log(("iret/64 with NT=1 (eflags=%#x) -> #GP(0)\n", pVCpu->cpum.GstCtx.eflags.u));
         return iemRaiseGeneralProtectionFault0(pVCpu);
     }
@@ -3508 +3508 @@
     else
         return rcStrict;
-    Log7(("iretq stack: cs:rip=%04x:%016RX64 rflags=%016RX64 ss:rsp=%04x:%016RX64\n", uNewCs, uNewRip, uNewFlags, uNewSs, uNewRsp));
+    Log7(("iret/64 stack: cs:rip=%04x:%016RX64 rflags=%016RX64 ss:rsp=%04x:%016RX64\n", uNewCs, uNewRip, uNewFlags, uNewSs, uNewRsp));
 
     /*
@@ -3516 +3516 @@
     if (!(uNewCs & X86_SEL_MASK_OFF_RPL))
     {
-        Log(("iret %04x:%016RX64/%04x:%016RX64 -> invalid CS selector, #GP(0)\n", uNewCs, uNewRip, uNewSs, uNewRsp));
+        Log(("iret/64 %04x:%016RX64/%04x:%016RX64 -> invalid CS selector, #GP(0)\n", uNewCs, uNewRip, uNewSs, uNewRsp));
         return iemRaiseGeneralProtectionFault0(pVCpu);
     }
@@ -3524 +3524 @@
     if (rcStrict != VINF_SUCCESS)
     {
-        Log(("iret %04x:%016RX64/%04x:%016RX64 - rcStrict=%Rrc when fetching CS\n",
+        Log(("iret/64 %04x:%016RX64/%04x:%016RX64 - rcStrict=%Rrc when fetching CS\n",
              uNewCs, uNewRip, uNewSs, uNewRsp, VBOXSTRICTRC_VAL(rcStrict)));
         return rcStrict;
@@ -3533 +3533 @@
         || !(DescCS.Legacy.Gen.u4Type & X86_SEL_TYPE_CODE))
     {
-        Log(("iret %04x:%016RX64/%04x:%016RX64 - CS is not a code segment T=%u T=%#xu -> #GP\n",
+        Log(("iret/64 %04x:%016RX64/%04x:%016RX64 - CS is not a code segment T=%u T=%#xu -> #GP\n",
              uNewCs, uNewRip, uNewSs, uNewRsp, DescCS.Legacy.Gen.u1DescType, DescCS.Legacy.Gen.u4Type));
         return iemRaiseGeneralProtectionFaultBySelector(pVCpu, uNewCs);
@@ -3544 +3544 @@
         if ((uNewCs & X86_SEL_RPL) != DescCS.Legacy.Gen.u2Dpl)
         {
-            Log(("iret %04x:%016RX64 - RPL != DPL (%d) -> #GP\n", uNewCs, uNewRip, DescCS.Legacy.Gen.u2Dpl));
+            Log(("iret/64 %04x:%016RX64 - RPL != DPL (%d) -> #GP\n", uNewCs, uNewRip, DescCS.Legacy.Gen.u2Dpl));
             return iemRaiseGeneralProtectionFaultBySelector(pVCpu, uNewCs);
         }
@@ -3550 +3550 @@
     else if ((uNewCs & X86_SEL_RPL) < DescCS.Legacy.Gen.u2Dpl)
     {
-        Log(("iret %04x:%016RX64 - RPL < DPL (%d) -> #GP\n", uNewCs, uNewRip, DescCS.Legacy.Gen.u2Dpl));
+        Log(("iret/64 %04x:%016RX64 - RPL < DPL (%d) -> #GP\n", uNewCs, uNewRip, DescCS.Legacy.Gen.u2Dpl));
         return iemRaiseGeneralProtectionFaultBySelector(pVCpu, uNewCs);
     }
     if ((uNewCs & X86_SEL_RPL) < IEM_GET_CPL(pVCpu))
     {
-        Log(("iret %04x:%016RX64 - RPL < CPL (%d) -> #GP\n", uNewCs, uNewRip, IEM_GET_CPL(pVCpu)));
+        Log(("iret/64 %04x:%016RX64 - RPL < CPL (%d) -> #GP\n", uNewCs, uNewRip, IEM_GET_CPL(pVCpu)));
         return iemRaiseGeneralProtectionFaultBySelector(pVCpu, uNewCs);
     }
@@ -3562 +3562 @@
     if (!DescCS.Legacy.Gen.u1Present)
     {
-        Log(("iret %04x:%016RX64/%04x:%016RX64 - CS not present -> #NP\n", uNewCs, uNewRip, uNewSs, uNewRsp));
+        Log(("iret/64 %04x:%016RX64/%04x:%016RX64 - CS not present -> #NP\n", uNewCs, uNewRip, uNewSs, uNewRsp));
         return iemRaiseSelectorNotPresentBySelector(pVCpu, uNewCs);
     }
@@ -3576 +3576 @@
             || uNewCpl > 2) /** @todo verify SS=0 impossible for ring-3. */
         {
-            Log(("iret %04x:%016RX64/%04x:%016RX64 -> invalid SS selector, #GP(0)\n", uNewCs, uNewRip, uNewSs, uNewRsp));
+            Log(("iret/64 %04x:%016RX64/%04x:%016RX64 -> invalid SS selector, #GP(0)\n", uNewCs, uNewRip, uNewSs, uNewRsp));
             return iemRaiseGeneralProtectionFault0(pVCpu);
         }
@@ -3587 +3587 @@
         if (rcStrict != VINF_SUCCESS)
         {
-            Log(("iret %04x:%016RX64/%04x:%016RX64 - %Rrc when fetching SS\n",
+            Log(("iret/64 %04x:%016RX64/%04x:%016RX64 - %Rrc when fetching SS\n",
                  uNewCs, uNewRip, uNewSs, uNewRsp, VBOXSTRICTRC_VAL(rcStrict)));
             return rcStrict;
@@ -3596 +3596 @@
     if ((uNewSs & X86_SEL_RPL) != (uNewCs & X86_SEL_RPL))
     {
-        Log(("iret %04x:%016RX64/%04x:%016RX64 -> SS.RPL != CS.RPL -> #GP\n", uNewCs, uNewRip, uNewSs, uNewRsp));
+        Log(("iret/64 %04x:%016RX64/%04x:%016RX64 -> SS.RPL != CS.RPL -> #GP\n", uNewCs, uNewRip, uNewSs, uNewRsp));
         return iemRaiseGeneralProtectionFaultBySelector(pVCpu, uNewSs);
     }
@@ -3607 +3607 @@
         if (DescSS.Legacy.Gen.u2Dpl != (uNewCs & X86_SEL_RPL))
         {
-            Log(("iret %04x:%016RX64/%04x:%016RX64 -> SS.DPL (%d) != CS.RPL -> #GP\n",
+            Log(("iret/64 %04x:%016RX64/%04x:%016RX64 -> SS.DPL (%d) != CS.RPL -> #GP\n",
                  uNewCs, uNewRip, uNewSs, uNewRsp, DescSS.Legacy.Gen.u2Dpl));
             return iemRaiseGeneralProtectionFaultBySelector(pVCpu, uNewSs);
@@ -3615 +3615 @@
         if (!DescSS.Legacy.Gen.u1DescType)
         {
-            Log(("iret %04x:%016RX64/%04x:%016RX64 -> SS is system segment (%#x) -> #GP\n",
+            Log(("iret/64 %04x:%016RX64/%04x:%016RX64 -> SS is system segment (%#x) -> #GP\n",
                  uNewCs, uNewRip, uNewSs, uNewRsp, DescSS.Legacy.Gen.u4Type));
             return iemRaiseGeneralProtectionFaultBySelector(pVCpu, uNewSs);
@@ -3621 +3621 @@
         if ((DescSS.Legacy.Gen.u4Type & (X86_SEL_TYPE_CODE | X86_SEL_TYPE_WRITE)) != X86_SEL_TYPE_WRITE)
         {
-            Log(("iret %04x:%016RX64/%04x:%016RX64 - not writable data segment (%#x) -> #GP\n",
+            Log(("iret/64 %04x:%016RX64/%04x:%016RX64 - not writable data segment (%#x) -> #GP\n",
                  uNewCs, uNewRip, uNewSs, uNewRsp, DescSS.Legacy.Gen.u4Type));
             return iemRaiseGeneralProtectionFaultBySelector(pVCpu, uNewSs);
@@ -3629 +3629 @@
         if (!DescSS.Legacy.Gen.u1Present)
         {
-            Log(("iret %04x:%016RX64/%04x:%016RX64 -> SS not present -> #SS\n", uNewCs, uNewRip, uNewSs, uNewRsp));
+            Log(("iret/64 %04x:%016RX64/%04x:%016RX64 -> SS not present -> #SS\n", uNewCs, uNewRip, uNewSs, uNewRsp));
             return iemRaiseStackSelectorNotPresentBySelector(pVCpu, uNewSs);
         }
@@ -3640 +3640 @@
         if (!IEM_IS_CANONICAL(uNewRip))
         {
-            Log(("iret %04x:%016RX64/%04x:%016RX64 -> RIP is not canonical -> #GP(0)\n",
-                 uNewCs, uNewRip, uNewSs, uNewRsp));
+            Log(("iret/64 %04x:%016RX64/%04x:%016RX64 -> RIP is not canonical -> #GP(0)\n", uNewCs, uNewRip, uNewSs, uNewRsp));
             return iemRaiseNotCanonical(pVCpu);
         }
@@ -3649 +3648 @@
         else
         {
-            Log(("iret %04x:%016RX64/%04x:%016RX64 -> both L and D are set -> #GP(0)\n", uNewCs, uNewRip, uNewSs, uNewRsp));
+            Log(("iret/64 %04x:%016RX64/%04x:%016RX64 -> both L and D are set -> #GP(0)\n", uNewCs, uNewRip, uNewSs, uNewRsp));
             return iemRaiseGeneralProtectionFault0(pVCpu);
         }
@@ -3657 +3656 @@
         if (uNewRip > cbLimitCS)
         {
-            Log(("iret %04x:%016RX64/%04x:%016RX64 -> EIP is out of bounds (%#x) -> #GP(0)\n",
+            Log(("iret/64 %04x:%016RX64/%04x:%016RX64 -> EIP is out of bounds (%#x) -> #GP(0)\n",
                  uNewCs, uNewRip, uNewSs, uNewRsp, cbLimitCS));
             /** @todo Which is it, \#GP(0) or \#GP(sel)? */
@@ -3696 +3695 @@
     fEFlagsNew         |= uNewFlags & fEFlagsMask;
 #ifdef DBGFTRACE_ENABLED
-    RTTraceBufAddMsgF(pVCpu->CTX_SUFF(pVM)->CTX_SUFF(hTraceBuf), "iret/%ul%u %08llx -> %04x:%04llx %llx %04x:%04llx",
+    RTTraceBufAddMsgF(pVCpu->CTX_SUFF(pVM)->CTX_SUFF(hTraceBuf), "iret/64/%ul%u %08llx -> %04x:%04llx %llx %04x:%04llx",
                       IEM_GET_CPL(pVCpu), uNewCpl, pVCpu->cpum.GstCtx.rip, uNewCs, uNewRip, uNewFlags, uNewSs, uNewRsp);
 #endif
@@ -3720 +3719 @@
         pVCpu->cpum.GstCtx.ss.u32Limit   = UINT32_MAX;
         pVCpu->cpum.GstCtx.ss.u64Base    = 0;
-        Log2(("iretq new SS: NULL\n"));
+        Log2(("iret/64 new SS: NULL\n"));
     }
     else
@@ -3728 +3727 @@
         pVCpu->cpum.GstCtx.ss.u32Limit   = cbLimitSs;
         pVCpu->cpum.GstCtx.ss.u64Base    = X86DESC_BASE(&DescSS.Legacy);
-        Log2(("iretq new SS: base=%#RX64 lim=%#x attr=%#x\n", pVCpu->cpum.GstCtx.ss.u64Base, pVCpu->cpum.GstCtx.ss.u32Limit, pVCpu->cpum.GstCtx.ss.Attr.u));
+        Log2(("iret/64 new SS: base=%#RX64 lim=%#x attr=%#x\n", pVCpu->cpum.GstCtx.ss.u64Base, pVCpu->cpum.GstCtx.ss.u32Limit, pVCpu->cpum.GstCtx.ss.Attr.u));
     }