diff options
author | Oliver Neukum <oliver@neukum.org> | 2010-01-13 09:33:15 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-03-02 17:54:23 -0500 |
commit | 86266452f80545285c14e20a8024f79c4fb88a86 (patch) | |
tree | ebb0a287f9bf189737d4924536d18b36492fd330 /drivers/usb/class | |
parent | f9de332ebf9df71892d52f7eb64af101a647349f (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.c | 3 | ||||
-rw-r--r-- | drivers/usb/class/usblp.c | 3 | ||||
-rw-r--r-- | drivers/usb/class/usbtmc.c | 3 |
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); |
549 | out: | 551 | out: |
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 | } |
435 | out: | 437 | out: |
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 | ||
130 | exit: | 132 | exit: |
133 | unlock_kernel(); | ||
131 | return retval; | 134 | return retval; |
132 | } | 135 | } |
133 | 136 | ||