Eric Paris 0bd0f9fb19 [PATCH] hugetlb: fix race in set_max_huge_pages for multiple updaters of nr_huge_pages
If there are multiple updaters to /proc/sys/vm/nr_hugepages simultaneously
it is possible for the nr_huge_pages variable to become incorrect.  There
is no locking in the set_max_huge_pages function around
alloc_fresh_huge_page which is able to update nr_huge_pages.  Two callers
to alloc_fresh_huge_page could race against each other as could a call to
alloc_fresh_huge_page and a call to update_and_free_page.  This patch just
expands the area covered by the hugetlb_lock to cover the call into
alloc_fresh_huge_page.  I'm not sure how we could say that a sysctl section
is performance critical where more specific locking would be needed.

My reproducer was to run a couple copies of the following script
simultaneously

while [ true ]; do
	echo 1000 > /proc/sys/vm/nr_hugepages
	echo 500 > /proc/sys/vm/nr_hugepages
	echo 750 > /proc/sys/vm/nr_hugepages
	echo 100 > /proc/sys/vm/nr_hugepages
	echo 0 > /proc/sys/vm/nr_hugepages
done

and then watch /proc/meminfo and eventually you will see things like

HugePages_Total:     100
HugePages_Free:      109

After applying the patch all seemed well.

Signed-off-by: Eric Paris <eparis@redhat.com>
Acked-by: William Irwin <wli@holomorphy.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
2005-11-22 09:13:43 -08:00
..
2005-10-29 21:40:39 -07:00
2005-06-24 00:06:41 -07:00
2005-10-28 08:16:51 -07:00
2005-04-16 15:20:36 -07:00
2005-11-22 09:13:42 -08:00
2005-11-22 09:13:42 -08:00
2005-10-28 08:16:47 -07:00
2005-04-16 15:20:36 -07:00
2005-10-29 21:40:42 -07:00
2005-11-22 09:13:42 -08:00
2005-10-29 21:40:42 -07:00
2005-04-16 15:20:36 -07:00
2005-11-07 07:53:37 -08:00
2005-11-22 09:13:42 -08:00
2005-10-29 21:40:42 -07:00
2005-11-22 09:13:42 -08:00
2005-10-29 21:40:35 -07:00
2005-11-13 18:14:12 -08:00