diff options
author | Arnd Bergmann <arnd@arndb.de> | 2010-07-03 22:02:25 (GMT) |
---|---|---|
committer | Frederic Weisbecker <fweisbec@gmail.com> | 2010-07-09 23:21:49 (GMT) |
commit | 5f202bd5ca64132cdd7f186656bc0221f257733d (patch) | |
tree | e53eac24bf18052955565370f3327da088fa1be7 /fs | |
parent | 5e3d20a68f63fc5a310687d81956c3b96e488b84 (diff) | |
download | linux-5f202bd5ca64132cdd7f186656bc0221f257733d.tar.xz |
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')
-rw-r--r-- | fs/exec.c | 7 |
1 files changed, 0 insertions, 7 deletions
@@ -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; |