Changeset 73277 in vbox for trunk/src/VBox/VMM/VMMAll/PGMAllPool.cpp

Timestamp:

Jul 20, 2018 6:30:34 PM (6 years ago)

Author:

vboxsync

Message:

PGM: Eliminated the unused VERR_PGM_POOL_FLUSHED and VERR_PGM_POOL_CLEARED status codes. Found checking out PGMHCChangeMode. bugref:9044

File:

• trunk/src/VBox/VMM/VMMAll/PGMAllPool.cpp (modified) (12 diffs)

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

@@ -93 +93 @@
  * Flushes a chain of pages sharing the same access monitor.
  *
- * @returns VBox status code suitable for scheduling.
  * @param   pPool       The pool.
  * @param   pPage       A page in the chain.
- * @todo VBOXSTRICTRC
- */
-int pgmPoolMonitorChainFlush(PPGMPOOL pPool, PPGMPOOLPAGE pPage)
+ */
+void pgmPoolMonitorChainFlush(PPGMPOOL pPool, PPGMPOOLPAGE pPage)
 {
     LogFlow(("pgmPoolMonitorChainFlush: Flush page %RGp type=%d\n", pPage->GCPhys, pPage->enmKind));
@@ -119 +117 @@
      * Iterate the list flushing each shadow page.
      */
-    int rc = VINF_SUCCESS;
     for (;;)
     {
@@ -134 +131 @@
         pPage = &pPool->aPages[idx];
     }
-    return rc;
 }
 
@@ -868 +864 @@
      * First, do the flushing.
      */
-    int rc = pgmPoolMonitorChainFlush(pPool, pPage);
+    pgmPoolMonitorChainFlush(pPool, pPage);
 
     /*
@@ -874 +870 @@
      * Must do this in raw mode (!); XP boot will fail otherwise.
      */
+    int rc = VINF_SUCCESS;
     VBOXSTRICTRC rc2 = EMInterpretInstructionDisasState(pVCpu, pDis, pRegFrame, pvFault, EMCODETYPE_ALL);
     if (rc2 == VINF_SUCCESS)
@@ -879 +876 @@
     else if (rc2 == VINF_EM_RESCHEDULE)
     {
-        if (rc == VINF_SUCCESS)
-            rc = VBOXSTRICTRC_VAL(rc2);
+        rc = VBOXSTRICTRC_VAL(rc2);
 # ifndef IN_RING3
         VMCPU_FF_SET(pVCpu, VMCPU_FF_TO_R3);
@@ -1486 +1482 @@
         }
         else
-        {
-            /* ASSUME that VERR_PGM_POOL_CLEARED can be ignored here and that FFs will deal with it in due time. */
             pgmPoolMonitorChainFlush(pPool, pPage);
-        }
 
         STAM_PROFILE_STOP_EX(&pPool->CTX_SUFF_Z(StatMonitor), &pPool->CTX_MID_Z(StatMonitor,FlushPage), a);
@@ -2920 +2913 @@
  * @returns VBox status code.
  * @retval  VINF_SUCCESS if successfully added.
- * @retval  <del>VERR_PGM_POOL_FLUSHED</del> if the pool was flushed - this is
- *          not returned anywhere anymore.
+ *
  * @param   pPool       The pool.
  * @param   iUser       The user index.
@@ -2951 +2943 @@
  * @returns VBox status code.
  * @retval  VINF_SUCCESS if successfully added.
- * @retval  <del>VERR_PGM_POOL_FLUSHED</del> if the pool was flushed - this is
- *          not returned anywhere anymore.
  *
  * @param   pPool       The pool.
@@ -3041 +3031 @@
  * @returns VBox status code.
  * @retval  VINF_SUCCESS if successfully added.
- * @retval  <del>VERR_PGM_POOL_FLUSHED</del> if the pool was flushed - this is
- *          not returned anywhere anymore.
  *
  * @param   pPool       The pool.
@@ -5104 +5092 @@
  * @returns VBox status code.
  * @retval  VINF_SUCCESS on success.
- * @retval  <del>VERR_PGM_POOL_FLUSHED</del> if the pool was flushed - this is
- *          not returned anywhere anymore.
  *
  * @param   pPool       The pool.
@@ -5157 +5143 @@
  * @retval  VINF_SUCCESS if a NEW page was allocated.
  * @retval  VINF_PGM_CACHED_PAGE if a CACHED page was returned.
- * @retval  <del>VERR_PGM_POOL_FLUSHED</del> if the pool was flushed - this is
- *          not returned anywhere anymore.
  *
  * @param   pVM         The cross context VM structure.