[ros-dev] [ros-diffs] [arty] 56441: [NTOSKRNL] Remove PAGEOPs and use the argument to pass relevant information where needed, including generally a section page table entry or a PFN_NUMBER. In its place, the code looks ...

[Aleksey Bragin]

Truly epic. I can't believe the concept of "PageOps" is gone now.

WBR,
Aleksey Bragin.

On 28.04.2012 22:10, Alex Ionescu wrote:
> Patch of the Year.
>
> Best regards,
> Alex Ionescu
>
>
> On Sat, Apr 28, 2012 at 10:56 AM, <arty at svn.reactos.org 
> <mailto:arty at svn.reactos.org>> wrote:
>
>     Author: arty
>     Date: Sat Apr 28 02:56:31 2012
>     New Revision: 56441
>
>     URL: http://svn.reactos.org/svn/reactos?rev=56441&view=rev
>     <http://svn.reactos.org/svn/reactos?rev=56441&view=rev>
>     Log:
>     [NTOSKRNL]
>     Remove PAGEOPs and use the argument to pass relevant information
>     where needed,
>     including generally a section page table entry or a PFN_NUMBER.
>
>     In its place, the code looks for an MM_WAIT_ENTRY in the
>     appropriate place and
>     waits, returning STATUS_MM_RESTART_OPERATION.  Rather than copying
>     the pagein
>     and CoW operations in the PAGEOP contention case, we just punt
>     back and restart
>     the operation, resulting in simpler code.
>
>     Modified:
>        trunk/reactos/ntoskrnl/CMakeLists.txt
>        trunk/reactos/ntoskrnl/cache/section/sptab.c
>        trunk/reactos/ntoskrnl/include/internal/mm.h
>        trunk/reactos/ntoskrnl/mm/mminit.c
>        trunk/reactos/ntoskrnl/mm/rmap.c
>        trunk/reactos/ntoskrnl/mm/section.c
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.reactos.org/pipermail/ros-dev/attachments/20120428/a77249b9/attachment.html>