mm_for_maps: shift down_read(mmap_sem) to the caller
commit17dc3e97d6d51df33cb6e35fabb62b91ef14cf2c
authorOleg Nesterov <oleg@redhat.com>
Fri, 10 Jul 2009 01:27:38 +0000 (10 03:27 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Sun, 16 Aug 2009 21:19:13 +0000 (16 14:19 -0700)
tree756968feee8f64e4d44264a3c0adfdd6eeae7cd9
parent95d7e670e3158b6a52a8279290a0d6f7047250b4
mm_for_maps: shift down_read(mmap_sem) to the caller

commit 00f89d218523b9bf6b522349c039d5ac80aa536d upstream.

mm_for_maps() takes ->mmap_sem after security checks, this looks
strange and obfuscates the locking rules. Move this lock to its
single caller, m_start().

Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Acked-by: Serge Hallyn <serue@us.ibm.com>
Signed-off-by: James Morris <jmorris@namei.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
fs/proc/base.c
fs/proc/task_mmu.c
fs/proc/task_nommu.c