Changeset 5033

Timestamp:

09/26/07 06:48:51 (1 year ago)

Author:

vboxsync

Message:

GMM...

Files:

• trunk/include/VBox/gmm.h (modified) (2 diffs)
• trunk/src/VBox/VMM/Makefile.kmk (modified) (3 diffs)
• trunk/src/VBox/VMM/VMMR0/GMMR0.cpp (modified) (11 diffs)
• trunk/src/VBox/VMM/VMMR0/GMMR0Internal.h (added)

trunk/include/VBox/gmm.h

@@ -20 +20 @@
-
-#include <VBox/types.h>
+#include <VBox/gvmm.h>
-
-#include "gvmm.h"
-
-/** @defgroup   grp_gmm     GMM - The Global Memory Manager
@@ -54 +54 @@
-
-
+/**
+ * Over-commitment policy.
+ */
+typedef enum GMMOCPOLICY
+{
+    /** The usual invalid 0 value. */
+    GMMOCPOLICY_INVALID = 0,
+    /** No over-commitment, fully backed.
+     * The GMM guarantees that it will be able to allocate all of the 
+     * guest RAM for a VM with OC policy. */
+    GMMOCPOLICY_NO_OC,
+    /** to-be-determined. */
+    GMMOCPOLICY_TBD,
+    /** The end of the valid policy range. */
+    GMMOCPOLICY_END,
+    /** The usual 32-bit hack. */
+    GMMOCPOLICY_32BIT_HACK = 0x7fffffff
+} GMMOCPOLICY;
+
+/**
+ * VM / Memory priority.
+ */
+typedef enum GMMPRORITY
+{
+    /** The usual invalid 0 value. */
+    GMMPRORITY_INVALID = 0, 
+    /** High - avoid interrupting it if at all possible */
+    GMMPRORITY_HIGH,
+    /** Normal - pause, save it or kill it. */
+    GMMPRORITY_NORMAL,
+    /** Low - save or kill it. */
+    GMMPRORITY_LOW,
+    /** The end of the valid priority range. */
+    GMMPRORITY_END = 0, 
+    /** The custom 32-bit type blowup. */
+    GMMPRORITY_32BIT_HACK = 0x7fffffff
+} GMMPRIORITY;
+
+
-/** @} */
-

trunk/src/VBox/VMM/Makefile.kmk

@@ -313 +313 @@
-        $(PATH_SUB_CURRENT) \
-        PATM
-
+IN_GMM_R0 
-ifdef VBOX_WITH_IDT_PATCHING
-VMMR0_DEFS     += VBOX_WITH_IDT_PATCHING
@@ -327 +327 @@
-        VMMR0/CPUMR0.cpp \
-        VMMR0/DBGFR0.cpp \
+        VMMR0/GMMR0.cpp \
-        VMMR0/GVMMR0.cpp \
-        VMMR0/HWACCMR0.cpp \
@@ -366 +367 @@
-        VMMAll/VMAll.cpp \
-
+
-ifeq ($(VBOX_LDR_FMT),pe)
-VMMR0_SOURCES += VMMR0/VMMR0.def

trunk/src/VBox/VMM/VMMR0/GMMR0.cpp

@@ -16 +16 @@
- *
- */
+
-
-/** @page pg_gmm    GMM - The Global Memory Manager
@@ -140 +141 @@
-*   Header Files                                                               *
-*******************************************************************************/
-
-
+
+
+
+
+
+
+
+
+
+
+
-
-
@@ -155 +165 @@
- *
- * Because of the different layout on 32-bit and 64-bit hosts, macros
-set and get
+get and set some of
- */
-typedef union GMMPAGE
@@ -347 +357 @@
-    uint16_t        au16Reserved;
-    /** The pages. */
-IZE
+HIFT
-} GMMCHUNK;
-
@@ -359 +369 @@
-    uint32_t        idChunk;
-    /** Pointer to the chunk. */
-G
+M
-} GMMCHUNKTLBE;
-/** Pointer to an allocation chunk TLB entry. */
@@ -415 +425 @@
-     * This is determined at initialization time. */
-    bool                fLegacyMode;
+    /** The number of active VMs. */
+    uint16_t            cActiveVMs;
-} GMM;
-/** Pointer to the GMM instance. */
@@ -423 +435 @@
-
-
+/*******************************************************************************
+*   Global Variables                                                           *
+*******************************************************************************/
+/** Pointer to the GMM instance data. */
+static PGMM g_pGMM = NULL;
+
+/** Macro for obtaining and validating the g_pGMM pointer.
+ * On failure it will return from the invoking function with the specified return value.
+ *
+ * @param   pGMM    The name of the pGMM variable.
+ * @param   rc      The return value on failure. Use VERR_INTERNAL_ERROR for
+ *                  VBox status codes.
+ */
+#define GMM_GET_VALID_INSTANCE(pGMM, rc) \
+    do { \
+        (pGMM) = g_pGMM; \
+        AssertPtrReturn((pGMM), (rc)); \
+        AssertMsgReturn((pGMM)->u32Magic == GMM_MAGIC, ("%p - %#x\n", (pGMM), (pGMM)->u32Magic), (rc)); \
+    } while (0)
+
+/** Macro for obtaining and validating the g_pGMM pointer, void function variant.
+ * On failure it will return from the invoking function.
+ *
+ * @param   pGMM    The name of the pGMM variable.
+ */
+#define GMM_GET_VALID_INSTANCE_VOID(pGMM) \
+    do { \
+        (pGMM) = g_pGMM; \
+        AssertPtrReturnVoid((pGMM)); \
+        AssertMsgReturnVoid((pGMM)->u32Magic == GMM_MAGIC, ("%p - %#x\n", (pGMM), (pGMM)->u32Magic)); \
+    } while (0)
+
-
-/*******************************************************************************
-*   Internal Functions                                                         *
-*******************************************************************************/
-
+
+
-
-
@@ -507 +552 @@
-    g_pGMM = NULL;
-    pGMM->u32Magic++;
-EventDestroy(&
+FastMutexDestroy(
-    pGMM->Mtx = NIL_RTSEMFASTMUTEX;
-
-    /* free any chunks still hanging around. */
-pGMM->
+&pGMM->p
-
-    /* finally the instance data itself. */
@@ -526 +571 @@
- * @param   pvGMM   The GMM handle.
- */
- int
-
- 
+(int)
+
+
-
-    if (pChunk->cFree != (GMM_CHUNK_SIZE >> PAGE_SHIFT))
@@ -553 +598 @@
-
-/**
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-
-    int rc =  RTSemFastMutexRequest(pGMM->Mtx);
@@ -571 +628 @@
-
-    /*
-
-
-
+
+
+
+
+
-     */
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-
-    RTSemFastMutexRelease(pGMM->Mtx);
+
+    /* trash the data */
+    pGVM->gmm.s.cRAMPages = 0;
+    pGVM->gmm.s.cPrivatePages = 0;
+    pGVM->gmm.s.cSharedPages = 0;
+    pGVM->gmm.s.enmPolicy = GMMOCPOLICY_INVALID;
+    pGVM->gmm.s.enmPriority = GMMPRORITY_INVALID;
+
-    LogFlow(("GMMR0CleanupVM: returns\n"));
-}
-
-
+/**
+ * RTAvlU32DoWithAll callback.
+ *
+ * @returns 0
+ * @param   pNode   The node to destroy.
+ * @param   pvhGVM  The GVM::hSelf value.
+ */
+static DECLCALLBACK(int) gmmR0FreeVMPagesInChunk(PAVLU32NODECORE pNode, void *pvhGVM)
+{
+    PGMMCHUNK pChunk = (PGMMCHUNK)pNode;
+    uint16_t hGVM = (uintptr_t)pvhGVM;
+
+#ifndef VBOx_STRICT
+    if (pChunk->cFree != (GMM_CHUNK_SIZE >> PAGE_SHIFT))
+#endif
+    {
+        /*
+         * Perform some internal checks while we're scanning.
+         */
+        unsigned cPrivate = 0;
+        unsigned cShared = 0;
+        unsigned cFree = 0;
+
+        unsigned iPage = (GMM_CHUNK_SIZE >> PAGE_SHIFT);
+        while (iPage-- > 0)
+            if (GMM_PAGE_IS_PRIVATE(&pChunk->aPages[iPage]))
+
+            {
+                if (pChunk->aPages[iPage].Private.hGVM == hGVM)
+                {
+                    /* Free it. */
+                    pChunk->aPages[iPage].u = 0;
+                    pChunk->aPages[iPage].Free.idNext = pChunk->idFreeHead;
+                    pChunk->aPages[iPage].Free.u2State = GMM_PAGE_STATE_FREE;
+                    pChunk->idFreeHead = iPage;
+                    pChunk->cPrivate--;
+                    pChunk->cFree++;
+                    cFree++;
+                }
+                else
+                    cPrivate++;
+            }
+            else if (GMM_PAGE_IS_FREE(&pChunk->aPages[iPage]))
+                cFree++;
+            else
+                cShared++;
+
+        /* 
+         * Did it add up?
+         */
+        if (RT_UNLIKELY(    pChunk->cFree != cFree
+                        ||  pChunk->cPrivate != cPrivate
+                        ||  pChunk->cShared != cShared))
+        {
+            SUPR0Printf("GMM: Chunk %p/%#x has bogus stats - free=%d/%d private=%d/%d shared=%d/%d\n", 
+                        pChunk->cFree, cFree, pChunk->cPrivate, cPrivate, pChunk->cShared, cShared);
+            pChunk->cFree = cFree;
+            pChunk->cPrivate = cPrivate;
+            pChunk->cShared = cShared;
+        }
+    }
+
+    return 0;
+}
+
+
+