nuttx/nuttx-update
1
2
Fork 1
Code Issues 1 Pull requests 1 Projects Releases Packages Wiki Activity Actions

bt_buf: use small lock in wireless/bluetooth/bt_buf.c

Browse source 
reason:
We would like to replace the big lock with a small lock.

Signed-off-by: hujun5 <hujun5@xiaomi.com>
This commit is contained in:
hujun5 2024-12-18 09:54:59 +08:00 committed by Xiang Xiao
parent dbeedf9229
commit 366332a948
1 changed files with 11 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
wireless/bluetooth/bt_buf.c
View file

@ -109,6 +109,7 @@ static struct bt_buf_s
g_buf_pool[CONFIG_BLUETOOTH_BUFFER_PREALLOC]; g_buf_pool[CONFIG_BLUETOOTH_BUFFER_PREALLOC];
static bool g_poolinit = false; static bool g_poolinit = false;
static spinlock_t g_buf_lock = SP_UNLOCKED;
/**************************************************************************** /****************************************************************************
* Public Functions * Public Functions
@ -238,7 +239,7 @@ FAR struct bt_buf_s *bt_buf_alloc(enum bt_buf_type_e type,
* then try the list of messages reserved for interrupt handlers * then try the list of messages reserved for interrupt handlers
*/ */
flags = spin_lock_irqsave(NULL); /* Always necessary in SMP mode */ flags = spin_lock_irqsave(&g_buf_lock); /* Always necessary in SMP mode */
if (up_interrupt_context()) if (up_interrupt_context())
{ {
#if CONFIG_BLUETOOTH_BUFFER_PREALLOC > CONFIG_BLUETOOTH_BUFFER_IRQRESERVE #if CONFIG_BLUETOOTH_BUFFER_PREALLOC > CONFIG_BLUETOOTH_BUFFER_IRQRESERVE
@ -249,7 +250,7 @@ FAR struct bt_buf_s *bt_buf_alloc(enum bt_buf_type_e type,
buf = g_buf_free; buf = g_buf_free;
g_buf_free = buf->flink; g_buf_free = buf->flink;
spin_unlock_irqrestore(NULL, flags); spin_unlock_irqrestore(&g_buf_lock, flags);
pool = POOL_BUFFER_GENERAL; pool = POOL_BUFFER_GENERAL;
} }
else else
@ -262,13 +263,13 @@ FAR struct bt_buf_s *bt_buf_alloc(enum bt_buf_type_e type,
buf = g_buf_free_irq; buf = g_buf_free_irq;
g_buf_free_irq = buf->flink; g_buf_free_irq = buf->flink;
spin_unlock_irqrestore(NULL, flags); spin_unlock_irqrestore(&g_buf_lock, flags);
pool = POOL_BUFFER_IRQ; pool = POOL_BUFFER_IRQ;
} }
else else
#endif #endif
{ {
spin_unlock_irqrestore(NULL, flags); spin_unlock_irqrestore(&g_buf_lock, flags);
return NULL; return NULL;
} }
} }
@ -285,7 +286,7 @@ FAR struct bt_buf_s *bt_buf_alloc(enum bt_buf_type_e type,
buf = g_buf_free; buf = g_buf_free;
g_buf_free = buf->flink; g_buf_free = buf->flink;
spin_unlock_irqrestore(NULL, flags); spin_unlock_irqrestore(&g_buf_lock, flags);
pool = POOL_BUFFER_GENERAL; pool = POOL_BUFFER_GENERAL;
} }
else else
@ -295,7 +296,7 @@ FAR struct bt_buf_s *bt_buf_alloc(enum bt_buf_type_e type,
* will have to allocate one from the kernel memory pool. * will have to allocate one from the kernel memory pool.
*/ */
spin_unlock_irqrestore(NULL, flags); spin_unlock_irqrestore(&g_buf_lock, flags);
buf = (FAR struct bt_buf_s *) buf = (FAR struct bt_buf_s *)
kmm_malloc((sizeof (struct bt_buf_s))); kmm_malloc((sizeof (struct bt_buf_s)));
@ -425,10 +426,10 @@ void bt_buf_release(FAR struct bt_buf_s *buf)
* list from interrupt handlers. * list from interrupt handlers.
*/ */
flags = spin_lock_irqsave(NULL); flags = spin_lock_irqsave(&g_buf_lock);
buf->flink = g_buf_free; buf->flink = g_buf_free;
g_buf_free = buf; g_buf_free = buf;
spin_unlock_irqrestore(NULL, flags); spin_unlock_irqrestore(&g_buf_lock, flags);
} }
else else
#endif #endif
@ -444,10 +445,10 @@ void bt_buf_release(FAR struct bt_buf_s *buf)
* list from interrupt handlers. * list from interrupt handlers.
*/ */
flags = spin_lock_irqsave(NULL); flags = spin_lock_irqsave(&g_buf_lock);
buf->flink = g_buf_free_irq; buf->flink = g_buf_free_irq;
g_buf_free_irq = buf; g_buf_free_irq = buf;
spin_unlock_irqrestore(NULL, flags); spin_unlock_irqrestore(&g_buf_lock, flags);
} }
else else
#endif #endif