diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2010-07-04 00:02:25 +0200 |
|---|---|---|
| committer | Frederic Weisbecker <fweisbec@gmail.com> | 2010-07-10 01:21:49 +0200 |
| commit | 5f202bd5ca64132cdd7f186656bc0221f257733d (patch) | |
| tree | e53eac24bf18052955565370f3327da088fa1be7 /fs/exec.c | |
| parent | 5e3d20a68f63fc5a310687d81956c3b96e488b84 (diff) | |
| download | lwn-5f202bd5ca64132cdd7f186656bc0221f257733d.tar.gz lwn-5f202bd5ca64132cdd7f186656bc0221f257733d.zip | |
do_coredump: Do not take BKL
core_pattern is not actually protected and hasn't been
ever since we introduced procfs support for sysctl -- a
_long_ time. Don't take it here either.
Also nothing inside do_coredump appears to require bkl
protection.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
[ remove smp_lock.h headers ]
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Diffstat (limited to 'fs/exec.c')
| -rw-r--r-- | fs/exec.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/fs/exec.c b/fs/exec.c index e19de6a80339..2619c767c089 100644 --- a/fs/exec.c +++ b/fs/exec.c @@ -28,7 +28,6 @@ #include <linux/mm.h> #include <linux/stat.h> #include <linux/fcntl.h> -#include <linux/smp_lock.h> #include <linux/swap.h> #include <linux/string.h> #include <linux/init.h> @@ -1891,13 +1890,7 @@ void do_coredump(long signr, int exit_code, struct pt_regs *regs) */ clear_thread_flag(TIF_SIGPENDING); - /* - * lock_kernel() because format_corename() is controlled by sysctl, which - * uses lock_kernel() - */ - lock_kernel(); ispipe = format_corename(corename, signr); - unlock_kernel(); if (ispipe) { int dump_count; |