Changeset 22707 in vbox for trunk

Timestamp:

Sep 2, 2009 11:02:48 AM (15 years ago)

Author:

vboxsync

Message:

REM: Double the size of aHandlerNotifications and make sure there are at least 48 free entries in REMNotifyHandlerPhysicalFlushIfAlmostFull. This should hopefully get rid of the rare cases where we need to flush the notifications during pgmMapActivateCR3 in RC.

Location:

trunk

Files:

• include/VBox/vm.h (modified) (1 diff)
• src/VBox/VMM/REMInternal.h (modified) (1 diff)
• src/VBox/VMM/VMMAll/REMAll.cpp (modified) (2 diffs)

trunk/include/VBox/vm.h

@@ -973 +973 @@
 /** @def VM_REM_SIZE
  * Must be multiple of 32 and coherent with REM_ENV_SIZE from REMInternal.h. */
-# define VM_REM_SIZE        0x10A00
+# define VM_REM_SIZE        0x11100
         char        padding[VM_REM_SIZE];   /* multiple of 32 */
     } rem;

trunk/src/VBox/VMM/REMInternal.h

@@ -175 +175 @@
     /** Array of recorded handler noticications.
      * These are replayed when entering REM. */
-    REMHANDLERNOTIFICATION  aHandlerNotifications[32];
+    REMHANDLERNOTIFICATION  aHandlerNotifications[64];
     volatile uint32_t       idxPendingList;
     volatile uint32_t       idxFreeList;

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

@@ -214 +214 @@
 
     /*
-     * Less than 20 items means we should flush.
+     * Less than 48 items means we should flush.
      */
     uint32_t cFree = 0;
@@ -222 +222 @@
     {
         Assert(idx < RT_ELEMENTS(pVM->rem.s.aHandlerNotifications));
-        if (++cFree > 20)
+        if (++cFree >= 48)
             return;
     }