summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArnd Bergmann <arnd@arndb.de>2010-07-04 00:02:25 +0200
committerFrederic Weisbecker <fweisbec@gmail.com>2010-07-10 01:21:49 +0200
commit5f202bd5ca64132cdd7f186656bc0221f257733d (patch)
treee53eac24bf18052955565370f3327da088fa1be7
parentinit: Remove the BKL from startup code (diff)
downloadlinux-5f202bd5ca64132cdd7f186656bc0221f257733d.tar.xz
linux-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>
-rw-r--r--fs/exec.c7
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;