Changeset 18369 in vbox for trunk/src/VBox/VMM/VMMR0

Timestamp:

Mar 27, 2009 3:24:07 AM (16 years ago)

Author:

vboxsync

Message:

GMMR0: Baka tori! Bitmap was a wee bit too small.

File:

• trunk/src/VBox/VMM/VMMR0/GMMR0.cpp (modified) (3 diffs)

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

@@ -513 +513 @@
     uint32_t            idChunkPrev;
     /** Chunk ID allocation bitmap.
-     * Bits of allocated IDs are set, free ones are cleared.
+     * Bits of allocated IDs are set, free ones are clear.
      * The NIL id (0) is marked allocated. */
-    uint32_t            bmChunkId[(GMM_CHUNKID_LAST + 32) >> 10];
+    uint32_t            bmChunkId[(GMM_CHUNKID_LAST + 1 + 31) / 32];
 } GMM;
 /** Pointer to the GMM instance. */
@@ -1373 +1373 @@
         if (idChunk > NIL_GMM_CHUNKID)
         {
-            AssertMsgReturn(!ASMAtomicBitTestAndSet(&pGMM->bmChunkId[0], idChunk), ("%#x\n", idChunk), NIL_GVM_HANDLE);
+            AssertMsgReturn(!ASMAtomicBitTestAndSet(&pGMM->bmChunkId[0], idChunk), ("%#x\n", idChunk), NIL_GMM_CHUNKID);
             return pGMM->idChunkPrev = idChunk;
         }
@@ -1384 +1384 @@
     idChunk = ASMBitFirstClear(&pGMM->bmChunkId[0], GMM_CHUNKID_LAST + 1);
     AssertMsgReturn(idChunk > NIL_GMM_CHUNKID, ("%#x\n", idChunk), NIL_GVM_HANDLE);
-    AssertMsgReturn(!ASMAtomicBitTestAndSet(&pGMM->bmChunkId[0], idChunk), ("%#x\n", idChunk), NIL_GVM_HANDLE);
+    AssertMsgReturn(!ASMAtomicBitTestAndSet(&pGMM->bmChunkId[0], idChunk), ("%#x\n", idChunk), NIL_GMM_CHUNKID);
 
     return pGMM->idChunkPrev = idChunk;