Changeset 105714 in vbox for trunk/src/VBox/VMM

Timestamp:

Aug 17, 2024 12:10:53 AM (5 months ago)

Author:

vboxsync

Message:

VMM/IEM: Used the new iemPollTimers in iemNativeHlpReturnBreakViaLookup and iemNativeHlpReturnBreakViaLookupWithTlb. bugref:10656

File:

• trunk/src/VBox/VMM/VMMAll/IEMAllN8veRecompiler.cpp (modified) (4 diffs)

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

@@ -212 +212 @@
                     /* Do polling. */
                     uint64_t const cTbExecNative = pVCpu->iem.s.cTbExecNative;
-                    if (   RT_LIKELY(cTbExecNative & 511)
-                        || !TMTimerPollBoolWith32BitMilliTS(pVCpu->CTX_SUFF(pVM), pVCpu, &pVCpu->iem.s.msRecompilerPollNow) )
+                    if (   RT_LIKELY((int32_t)--pVCpu->iem.s.cTbsTillNextTimerPoll > 0)
+                        || iemPollTimers(pVCpu->CTX_SUFF(pVM), pVCpu) == VINF_SUCCESS)
                     {
                         /*
                          * Success. Update statistics and switch to the next TB.
                          */
-                        pVCpu->iem.s.cTbExecNative = cTbExecNative + 1;
                         if (a_fWithIrqCheck)
                             STAM_REL_COUNTER_INC(&pVCpu->iem.s.StatNativeTbExitDirectLinking1Irq);
@@ -228 +227 @@
                         pVCpu->iem.s.pCurTbR3          = pNewTb;
                         pVCpu->iem.s.ppTbLookupEntryR3 = IEMTB_GET_TB_LOOKUP_TAB_ENTRY(pNewTb, 0);
+                        pVCpu->iem.s.cTbExecNative    += 1;
                         Log10(("iemNativeHlpReturnBreakViaLookupWithPc: match at %04x:%08RX64 (%RGp): pTb=%p[%#x]-> %p\n",
                                pVCpu->cpum.GstCtx.cs.Sel, pVCpu->cpum.GstCtx.rip, GCPhysPc, pTb, idxTbLookup, pNewTb));
@@ -342 +342 @@
                 {
                     /* Do polling. */
-                    uint64_t const cTbExecNative = pVCpu->iem.s.cTbExecNative;
-                    if (   RT_LIKELY(cTbExecNative & 511)
-                        || !TMTimerPollBoolWith32BitMilliTS(pVCpu->CTX_SUFF(pVM), pVCpu, &pVCpu->iem.s.msRecompilerPollNow) )
+                    if (   RT_LIKELY((int32_t)--pVCpu->iem.s.cTbsTillNextTimerPoll > 0)
+                        || iemPollTimers(pVCpu->CTX_SUFF(pVM), pVCpu) == VINF_SUCCESS)
                     {
                         /*
                          * Success. Update statistics and switch to the next TB.
                          */
-                        pVCpu->iem.s.cTbExecNative = cTbExecNative + 1;
                         if (a_fWithIrqCheck)
                             STAM_REL_COUNTER_INC(&pVCpu->iem.s.StatNativeTbExitDirectLinking2Irq);
@@ -359 +357 @@
                         pVCpu->iem.s.pCurTbR3          = pNewTb;
                         pVCpu->iem.s.ppTbLookupEntryR3 = IEMTB_GET_TB_LOOKUP_TAB_ENTRY(pNewTb, 0);
+                        pVCpu->iem.s.cTbExecNative    += 1;
                         Log10(("iemNativeHlpReturnBreakViaLookupWithTlb: match at %04x:%08RX64 (%RGp): pTb=%p[%#x]-> %p\n",
                                pVCpu->cpum.GstCtx.cs.Sel, pVCpu->cpum.GstCtx.rip, GCPhysPc, pTb, idxTbLookup, pNewTb));