diff options
| author | Jonathan Corbet <corbet@lwn.net> | 2008-05-15 11:07:52 -0600 |
|---|---|---|
| committer | Jonathan Corbet <corbet@lwn.net> | 2008-06-20 14:05:48 -0600 |
| commit | 309c4551c0fa0897d5343c36cbfbfa39f1f41b88 (patch) | |
| tree | 86c48c9b1653ad781323430ba324fdf52c2b81f1 /drivers/char | |
| parent | 1f439647a4072ec64bb2e4b9290cd7be6aee8328 (diff) | |
| download | kernel_samsung_crespo-309c4551c0fa0897d5343c36cbfbfa39f1f41b88.zip kernel_samsung_crespo-309c4551c0fa0897d5343c36cbfbfa39f1f41b88.tar.gz kernel_samsung_crespo-309c4551c0fa0897d5343c36cbfbfa39f1f41b88.tar.bz2 | |
misc: cdev lock_kernel() pushdown
misc_open() looks fine, but who knows what all of the misc drivers are
doing in their open() functions?
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Diffstat (limited to 'drivers/char')
| -rw-r--r-- | drivers/char/misc.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/char/misc.c b/drivers/char/misc.c index eaace0d..6e1563c 100644 --- a/drivers/char/misc.c +++ b/drivers/char/misc.c @@ -49,6 +49,7 @@ #include <linux/device.h> #include <linux/tty.h> #include <linux/kmod.h> +#include <linux/smp_lock.h> /* * Head entry for the doubly linked miscdevice list @@ -118,6 +119,7 @@ static int misc_open(struct inode * inode, struct file * file) int err = -ENODEV; const struct file_operations *old_fops, *new_fops = NULL; + lock_kernel(); mutex_lock(&misc_mtx); list_for_each_entry(c, &misc_list, list) { @@ -155,6 +157,7 @@ static int misc_open(struct inode * inode, struct file * file) fops_put(old_fops); fail: mutex_unlock(&misc_mtx); + unlock_kernel(); return err; } |