[ros-diffs] [tkreuzer] 58047: [NTOSKRNL] In NtAllocateVirtualMemory, when MEM_RESET is passed, silence the DPRINT and also return STATUS_SUCCESS, since MEM_RESET is only an optimization (tells the memory manag...

Author: tkreuzer
Date: Sat Dec 29 21:40:41 2012
New Revision: 58047

URL: http://svn.reactos.org/svn/reactos?rev=58047&view=rev
Log:
[NTOSKRNL]
In NtAllocateVirtualMemory, when MEM_RESET is passed, silence the DPRINT and also return
STATUS_SUCCESS, since MEM_RESET is only an optimization (tells the memory manager that the
contents shouldn't be written to the pagefile) and it won't hurt (except the
performance) to pretend success. On the other hand paging out is probably never done
anyway due to broken Mm.
This fixes spamming the log by Firefox. It does not fix the hangs that firfox suffers
from.

Modified:
    trunk/reactos/ntoskrnl/mm/ARM3/virtual.c

Modified: trunk/reactos/ntoskrnl/mm/ARM3/virtual.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/virtual.c…
==============================================================================
--- trunk/reactos/ntoskrnl/mm/ARM3/virtual.c [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/ARM3/virtual.c [iso-8859-1] Sat Dec 29 21:40:41 2012
@@ -3861,8 +3861,9 @@
     }
     if ((AllocationType & MEM_RESET) == MEM_RESET)
     {
-        DPRINT1("MEM_RESET not supported\n");
-        Status = STATUS_INVALID_PARAMETER;
+        /// @todo HACK: pretend success
+        DPRINT("MEM_RESET not supported\n");
+        Status = STATUS_SUCCESS;
         goto FailPathNoLock;
     }
     if (Process->VmTopDown == 1)