Commit 9aaf20cb authored by Jonathan Corbet's avatar Jonathan Corbet

videopix: BKL pushdown

Add explicit lock_kernel() calls to vfc_open().
Signed-off-by: default avatarJonathan Corbet <corbet@lwn.net>
parent 5e9829ad
...@@ -24,6 +24,7 @@ ...@@ -24,6 +24,7 @@
#include <linux/spinlock.h> #include <linux/spinlock.h>
#include <linux/mutex.h> #include <linux/mutex.h>
#include <linux/mm.h> #include <linux/mm.h>
#include <linux/smp_lock.h>
#include <asm/openprom.h> #include <asm/openprom.h>
#include <asm/oplib.h> #include <asm/oplib.h>
...@@ -178,14 +179,17 @@ static int vfc_open(struct inode *inode, struct file *file) ...@@ -178,14 +179,17 @@ static int vfc_open(struct inode *inode, struct file *file)
{ {
struct vfc_dev *dev; struct vfc_dev *dev;
lock_kernel();
spin_lock(&vfc_dev_lock); spin_lock(&vfc_dev_lock);
dev = vfc_get_dev_ptr(iminor(inode)); dev = vfc_get_dev_ptr(iminor(inode));
if (dev == NULL) { if (dev == NULL) {
spin_unlock(&vfc_dev_lock); spin_unlock(&vfc_dev_lock);
unlock_kernel();
return -ENODEV; return -ENODEV;
} }
if (dev->busy) { if (dev->busy) {
spin_unlock(&vfc_dev_lock); spin_unlock(&vfc_dev_lock);
unlock_kernel();
return -EBUSY; return -EBUSY;
} }
...@@ -202,6 +206,7 @@ static int vfc_open(struct inode *inode, struct file *file) ...@@ -202,6 +206,7 @@ static int vfc_open(struct inode *inode, struct file *file)
vfc_captstat_reset(dev); vfc_captstat_reset(dev);
vfc_unlock_device(dev); vfc_unlock_device(dev);
unlock_kernel();
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment