diff options
author | Andi Kleen <ak@linux.intel.com> | 2012-03-16 15:01:02 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-03-16 16:30:10 -0400 |
commit | 647d558023e3868725b381b4dccbbeae3448fc28 (patch) | |
tree | fd69b070831ddbf7ad1c32dfa33d00f3e95878de /drivers/usb/gadget/f_fs.c | |
parent | 963940cf472d76eca2d36296e461202cc6997352 (diff) |
usb: gadget: f_fs: Remove lock is held before freeing checks
lock debugging already supports this, no need to do it explicitely.
Cc: balbi@ti.com
Cc: gregkh@linuxfoundation.org
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/usb/gadget/f_fs.c')
-rw-r--r-- | drivers/usb/gadget/f_fs.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/usb/gadget/f_fs.c b/drivers/usb/gadget/f_fs.c index 7e2216f1bbe6..7f445ec723bc 100644 --- a/drivers/usb/gadget/f_fs.c +++ b/drivers/usb/gadget/f_fs.c | |||
@@ -1258,9 +1258,7 @@ static void ffs_data_put(struct ffs_data *ffs) | |||
1258 | if (unlikely(atomic_dec_and_test(&ffs->ref))) { | 1258 | if (unlikely(atomic_dec_and_test(&ffs->ref))) { |
1259 | pr_info("%s(): freeing\n", __func__); | 1259 | pr_info("%s(): freeing\n", __func__); |
1260 | ffs_data_clear(ffs); | 1260 | ffs_data_clear(ffs); |
1261 | BUG_ON(mutex_is_locked(&ffs->mutex) || | 1261 | BUG_ON(waitqueue_active(&ffs->ev.waitq) || |
1262 | spin_is_locked(&ffs->ev.waitq.lock) || | ||
1263 | waitqueue_active(&ffs->ev.waitq) || | ||
1264 | waitqueue_active(&ffs->ep0req_completion.wait)); | 1262 | waitqueue_active(&ffs->ep0req_completion.wait)); |
1265 | kfree(ffs); | 1263 | kfree(ffs); |
1266 | } | 1264 | } |