OrgApache/incubator-nuttx
OrgApache
/
incubator-nuttx
mirror of https://github.com/apache/incubator-nuttx.git
1
0
Fork 0
Code Issues Releases Wiki Activity

ramlog: unify the lock to critical section 

Signed-off-by: ligd <liguiding1@xiaomi.com>
This commit is contained in:
ligd 2024-07-25 22:50:23 +08:00 committed by archer
parent 28efb282d8
commit 78d93eb628
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
drivers/syslog/ramlog.c
View File

@ -510,7 +510,7 @@ static int ramlog_file_ioctl(FAR struct file *filep, int cmd,
irqstate_t flags; irqstate_t flags;
int ret = 0; int ret = 0;
flags = spin_lock_irqsave(NULL); flags = enter_critical_section();
switch (cmd) switch (cmd)
{ {
@ -528,7 +528,7 @@ static int ramlog_file_ioctl(FAR struct file *filep, int cmd,
break; break;
} }
spin_unlock_irqrestore(NULL, flags); leave_critical_section(flags);
return ret; return ret;
} }
@ -615,11 +615,11 @@ static int ramlog_file_open(FAR struct file *filep)
nxsem_init(&upriv->rl_waitsem, 0, 0); nxsem_init(&upriv->rl_waitsem, 0, 0);
#endif #endif
flags = spin_lock_irqsave(NULL); flags = enter_critical_section();
list_add_tail(&priv->rl_list, &upriv->rl_node); list_add_tail(&priv->rl_list, &upriv->rl_node);
upriv->rl_tail = header->rl_head > priv->rl_bufsize ? upriv->rl_tail = header->rl_head > priv->rl_bufsize ?
header->rl_head - priv->rl_bufsize : 0; header->rl_head - priv->rl_bufsize : 0;
spin_unlock_irqrestore(NULL, flags); leave_critical_section(flags);
filep->f_priv = upriv; filep->f_priv = upriv;
return 0; return 0;
@ -636,9 +636,9 @@ static int ramlog_file_close(FAR struct file *filep)
/* Get exclusive access to the rl_tail index */ /* Get exclusive access to the rl_tail index */
flags = spin_lock_irqsave(NULL); flags = enter_critical_section();
list_delete(&upriv->rl_node); list_delete(&upriv->rl_node);
spin_unlock_irqrestore(NULL, flags); leave_critical_section(flags);
#ifndef CONFIG_RAMLOG_NONBLOCKING #ifndef CONFIG_RAMLOG_NONBLOCKING
nxsem_destroy(&upriv->rl_waitsem); nxsem_destroy(&upriv->rl_waitsem);