aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/usb/class
diff options
context:
space:
mode:
authorOliver Neukum <oliver@neukum.org>2010-01-13 09:33:15 -0500
committerGreg Kroah-Hartman <gregkh@suse.de>2010-03-02 17:54:23 -0500
commit86266452f80545285c14e20a8024f79c4fb88a86 (patch)
treeebb0a287f9bf189737d4924536d18b36492fd330 /drivers/usb/class
parentf9de332ebf9df71892d52f7eb64af101a647349f (diff)
USB: Push BKL on open down into the drivers
Straightforward push into the drivers to allow auditing individual drivers separately Signed-off-by: Oliver Neukum <oliver@neukum.org> Acked-by: Mauro Carvalho Chehab <mchehab@redhat.com> Cc: Jiri Kosina <jkosina@suse.cz> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/class')
-rw-r--r--drivers/usb/class/cdc-wdm.c3
-rw-r--r--drivers/usb/class/usblp.c3
-rw-r--r--drivers/usb/class/usbtmc.c3
3 files changed, 9 insertions, 0 deletions
diff --git a/drivers/usb/class/cdc-wdm.c b/drivers/usb/class/cdc-wdm.c
index 18aafcb08fc8..b75a3d8bb02f 100644
--- a/drivers/usb/class/cdc-wdm.c
+++ b/drivers/usb/class/cdc-wdm.c
@@ -19,6 +19,7 @@
19#include <linux/uaccess.h> 19#include <linux/uaccess.h>
20#include <linux/bitops.h> 20#include <linux/bitops.h>
21#include <linux/poll.h> 21#include <linux/poll.h>
22#include <linux/smp_lock.h>
22#include <linux/usb.h> 23#include <linux/usb.h>
23#include <linux/usb/cdc.h> 24#include <linux/usb/cdc.h>
24#include <asm/byteorder.h> 25#include <asm/byteorder.h>
@@ -516,6 +517,7 @@ static int wdm_open(struct inode *inode, struct file *file)
516 struct usb_interface *intf; 517 struct usb_interface *intf;
517 struct wdm_device *desc; 518 struct wdm_device *desc;
518 519
520 lock_kernel();
519 mutex_lock(&wdm_mutex); 521 mutex_lock(&wdm_mutex);
520 intf = usb_find_interface(&wdm_driver, minor); 522 intf = usb_find_interface(&wdm_driver, minor);
521 if (!intf) 523 if (!intf)
@@ -548,6 +550,7 @@ static int wdm_open(struct inode *inode, struct file *file)
548 usb_autopm_put_interface(desc->intf); 550 usb_autopm_put_interface(desc->intf);
549out: 551out:
550 mutex_unlock(&wdm_mutex); 552 mutex_unlock(&wdm_mutex);
553 unlock_kernel();
551 return rv; 554 return rv;
552} 555}
553 556
diff --git a/drivers/usb/class/usblp.c b/drivers/usb/class/usblp.c
index 93b5f85d7ceb..d53f9499f936 100644
--- a/drivers/usb/class/usblp.c
+++ b/drivers/usb/class/usblp.c
@@ -56,6 +56,7 @@
56#include <linux/slab.h> 56#include <linux/slab.h>
57#include <linux/lp.h> 57#include <linux/lp.h>
58#include <linux/mutex.h> 58#include <linux/mutex.h>
59#include <linux/smp_lock.h>
59#undef DEBUG 60#undef DEBUG
60#include <linux/usb.h> 61#include <linux/usb.h>
61 62
@@ -395,6 +396,7 @@ static int usblp_open(struct inode *inode, struct file *file)
395 if (minor < 0) 396 if (minor < 0)
396 return -ENODEV; 397 return -ENODEV;
397 398
399 lock_kernel();
398 mutex_lock (&usblp_mutex); 400 mutex_lock (&usblp_mutex);
399 401
400 retval = -ENODEV; 402 retval = -ENODEV;
@@ -434,6 +436,7 @@ static int usblp_open(struct inode *inode, struct file *file)
434 } 436 }
435out: 437out:
436 mutex_unlock (&usblp_mutex); 438 mutex_unlock (&usblp_mutex);
439 unlock_kernel();
437 return retval; 440 return retval;
438} 441}
439 442
diff --git a/drivers/usb/class/usbtmc.c b/drivers/usb/class/usbtmc.c
index 8588c0937a89..426bfc72b9b4 100644
--- a/drivers/usb/class/usbtmc.c
+++ b/drivers/usb/class/usbtmc.c
@@ -26,6 +26,7 @@
26#include <linux/uaccess.h> 26#include <linux/uaccess.h>
27#include <linux/kref.h> 27#include <linux/kref.h>
28#include <linux/mutex.h> 28#include <linux/mutex.h>
29#include <linux/smp_lock.h>
29#include <linux/usb.h> 30#include <linux/usb.h>
30#include <linux/usb/tmc.h> 31#include <linux/usb/tmc.h>
31 32
@@ -113,6 +114,7 @@ static int usbtmc_open(struct inode *inode, struct file *filp)
113 struct usbtmc_device_data *data; 114 struct usbtmc_device_data *data;
114 int retval = 0; 115 int retval = 0;
115 116
117 lock_kernel();
116 intf = usb_find_interface(&usbtmc_driver, iminor(inode)); 118 intf = usb_find_interface(&usbtmc_driver, iminor(inode));
117 if (!intf) { 119 if (!intf) {
118 printk(KERN_ERR KBUILD_MODNAME 120 printk(KERN_ERR KBUILD_MODNAME
@@ -128,6 +130,7 @@ static int usbtmc_open(struct inode *inode, struct file *filp)
128 filp->private_data = data; 130 filp->private_data = data;
129 131
130exit: 132exit:
133 unlock_kernel();
131 return retval; 134 return retval;
132} 135}
133 136