summaryrefslogtreecommitdiffstats
path: root/include/net
diff options
context:
space:
mode:
authorLibor Pechacek <lpechacek@suse.cz>2014-04-29 20:38:34 +0200
committerMarcel Holtmann <marcel@holtmann.org>2014-05-05 19:25:06 -0700
commit86aae6c7b577654b7293374973985a153e0c147e (patch)
tree2bc969c42c78d2fa817c0bdb0d0a85dcf692d075 /include/net
parent22e70786413ed05950207eda7be420c280b776d7 (diff)
downloadop-kernel-dev-86aae6c7b577654b7293374973985a153e0c147e.zip
op-kernel-dev-86aae6c7b577654b7293374973985a153e0c147e.tar.gz
Bluetooth: Convert RFCOMM spinlocks into mutexes
Enabling CONFIG_DEBUG_ATOMIC_SLEEP has shown that some rfcomm functions acquiring spinlocks call sleeping locks further in the chain. Converting the offending spinlocks into mutexes makes sleeping safe. Signed-off-by: Libor Pechacek <lpechacek@suse.cz> Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'include/net')
-rw-r--r--include/net/bluetooth/rfcomm.h6
1 files changed, 3 insertions, 3 deletions
diff --git a/include/net/bluetooth/rfcomm.h b/include/net/bluetooth/rfcomm.h
index 2611cc3..578b831 100644
--- a/include/net/bluetooth/rfcomm.h
+++ b/include/net/bluetooth/rfcomm.h
@@ -173,7 +173,7 @@ struct rfcomm_dlc {
struct sk_buff_head tx_queue;
struct timer_list timer;
- spinlock_t lock;
+ struct mutex lock;
unsigned long state;
unsigned long flags;
atomic_t refcnt;
@@ -244,8 +244,8 @@ int rfcomm_dlc_get_modem_status(struct rfcomm_dlc *d, u8 *v24_sig);
void rfcomm_dlc_accept(struct rfcomm_dlc *d);
struct rfcomm_dlc *rfcomm_dlc_exists(bdaddr_t *src, bdaddr_t *dst, u8 channel);
-#define rfcomm_dlc_lock(d) spin_lock(&d->lock)
-#define rfcomm_dlc_unlock(d) spin_unlock(&d->lock)
+#define rfcomm_dlc_lock(d) mutex_lock(&d->lock)
+#define rfcomm_dlc_unlock(d) mutex_unlock(&d->lock)
static inline void rfcomm_dlc_hold(struct rfcomm_dlc *d)
{
OpenPOWER on IntegriCloud