Changeset 41898 in vbox for trunk/src/VBox

Timestamp:

Jun 23, 2012 6:44:29 PM (13 years ago)

Author:

vboxsync

Message:

patmRemovePatchFromPage: No need to move the whole array up one entry, just move the last entry. It's not like this is sorted or anything.

File:

• trunk/src/VBox/VMM/VMMR3/PATM.cpp (modified) (1 diff)

trunk/src/VBox/VMM/VMMR3/PATM.cpp

@@ -4678 +4678 @@
             {
                 /* close the gap between the remaining pointers. */
-                uint32_t cAfter = pPatchPage->cCount - i - 1;
-                if (cAfter)
-                    memmove(&pPatchPage->papPatch[i], &pPatchPage->papPatch[i + 1], sizeof(pPatchPage->papPatch[i]) * cAfter);
-                pPatchPage->papPatch[--pPatchPage->cCount] = NULL;
+                uint32_t cNew = --pPatchPage->cCount;
+                if (i < cNew)
+                    pPatchPage->papPatch[i] = pPatchPage->papPatch[cNew];
+                pPatchPage->papPatch[cNew] = NULL;
                 return VINF_SUCCESS;
             }