[ros-diffs] [sir_richard] 45625: [NTOS]: Fix premature setting of the page location of a newly allocated MDL page. Because the zeroing of non-zeroed pages is done after the initial page allocation loop (and by checking which list the page was on), setting the page location as ActiveAndValid made the zero loop believe the page was unzeroed (even though it could've come from the zeroed list), causing a superflous second zero operation.

sir_richard at svn.reactos.org sir_richard at svn.reactos.org
Fri Feb 19 19:29:29 CET 2010


Author: sir_richard
Date: Fri Feb 19 19:29:29 2010
New Revision: 45625

URL: http://svn.reactos.org/svn/reactos?rev=45625&view=rev
Log:
[NTOS]: Fix premature setting of the page location of a newly allocated MDL page. Because the zeroing of non-zeroed pages is done after the initial page allocation loop (and by checking which list the page was on), setting the page location as ActiveAndValid made the zero loop believe the page was unzeroed (even though it could've come from the zeroed list), causing a superflous second zero operation.

Modified:
    trunk/reactos/ntoskrnl/mm/freelist.c

Modified: trunk/reactos/ntoskrnl/mm/freelist.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/freelist.c?rev=45625&r1=45624&r2=45625&view=diff
==============================================================================
--- trunk/reactos/ntoskrnl/mm/freelist.c [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/freelist.c [iso-8859-1] Fri Feb 19 19:29:29 2010
@@ -480,7 +480,6 @@
             Pfn1->u3.e1.StartOfAllocation = 1;
             Pfn1->u3.e1.EndOfAllocation = 1;
             Pfn1->u3.e2.ReferenceCount = 1;
-            Pfn1->u3.e1.PageLocation = ActiveAndValid;
             Pfn1->SavedSwapEntry = 0;
             
             //




More information about the Ros-diffs mailing list