Date: Wed, 1 Dec 2010 03:35:19 +0000 (UTC) From: Alan Cox <alc@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/vm vm_page.c Message-ID: <201012010335.oB13ZcAb059777@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
alc 2010-12-01 03:35:19 UTC
FreeBSD src repository
Modified files:
sys/vm vm_page.c
Log:
SVN rev 216090 on 2010-12-01 03:35:19Z by alc
Correct an error in the allocation of the vm_page_dump array in
vm_page_startup(). Specifically, the dump_avail array should be used
instead of the phys_avail array to calculate the size of vm_page_dump. For
example, the pages for the message buffer are allocated prior to
vm_page_startup() by subtracting them from the last entry in the phys_avail
array, but the first thing that vm_page_startup() does after creating the
vm_page_dump array is to set the bits corresponding to the message buffer
pages in that array. However, these bits might not actually exist in the
array, because the size of the array is determined by the current value in
the last entry of the phys_avail array. In general, the only reason why
this doesn't always result in an out-of-bounds array access is that the size
of the vm_page_dump array is rounded up to the next page boundary. This
change eliminates that dependence on rounding (and luck).
MFC after: 6 weeks
Revision Changes Path
1.425 +5 -4 src/sys/vm/vm_page.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201012010335.oB13ZcAb059777>