[src] cvs commit: src/sys/vm vm_contig.c

Nate Lawson nate at root.org
Fri Mar 5 10:12:47 PST 2004


On Thu, 4 Mar 2004, Alan Cox wrote:
>   FreeBSD src repository
>
>   Modified files:
>     sys/vm               vm_contig.c
>   Log:
>   In the last revision, I introduced a physical contiguity check that is both
>   unnecessary and wrong.  While it is necessary to verify that the page is
>   still free after dropping and reacquiring the free page queue lock, the
>   physical contiguity of the page can not change, making this check
>   unnecessary.  This check was wrong in that it could cause an out-of-bounds
>   array access.
>
>   Tested by:      rwatson
>
>   Revision  Changes    Path
>   1.32      +1 -3      src/sys/vm/vm_contig.c

Reported, tested by:	marcel
Initial incorrect fix:	njl

-Nate


More information about the cvs-all mailing list