Skip to content

Commit

Permalink
sched/spin_lock: rename raw_spin_lock to spin_lock_notrace
Browse files Browse the repository at this point in the history 
Signed-off-by: hujun5 <hujun5@xiaomi.com>
  • Loading branch information
@hujun260
hujun260 committed Feb 1, 2025
1 parent 80e545a commit 59ebddf
Show file tree
Hide file tree
Showing 11 changed files with 96 additions and 96 deletions.
16 changes: 8 additions & 8 deletions drivers/note/note_driver.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -1841,7 +1841,7 @@ void sched_note_filter_mode(FAR struct note_filter_named_mode_s *oldm,
irqstate_t irq_mask;
FAR struct note_driver_s **driver;

irq_mask = raw_spin_lock_irqsave(&g_note_lock);
irq_mask = spin_lock_irqsave_notrace(&g_note_lock);

if (oldm != NULL)
{
Expand Down Expand Up @@ -1877,7 +1877,7 @@ void sched_note_filter_mode(FAR struct note_filter_named_mode_s *oldm,
}
}

raw_spin_unlock_irqrestore(&g_note_lock, irq_mask);
spin_unlock_irqrestore_notrace(&g_note_lock, irq_mask);
}

/****************************************************************************
Expand Down Expand Up @@ -1907,7 +1907,7 @@ void sched_note_filter_syscall(FAR struct note_filter_named_syscall_s *oldf,
irqstate_t irq_mask;
FAR struct note_driver_s **driver;

irq_mask = raw_spin_lock_irqsave(&g_note_lock);
irq_mask = spin_lock_irqsave_notrace(&g_note_lock);

if (oldf != NULL)
{
Expand Down Expand Up @@ -1943,7 +1943,7 @@ void sched_note_filter_syscall(FAR struct note_filter_named_syscall_s *oldf,
}
}

raw_spin_unlock_irqrestore(&g_note_lock, irq_mask);
spin_unlock_irqrestore_notrace(&g_note_lock, irq_mask);
}
#endif

Expand Down Expand Up @@ -1974,7 +1974,7 @@ void sched_note_filter_irq(FAR struct note_filter_named_irq_s *oldf,
irqstate_t irq_mask;
FAR struct note_driver_s **driver;

irq_mask = raw_spin_lock_irqsave(&g_note_lock);
irq_mask = spin_lock_irqsave_notrace(&g_note_lock);

if (oldf != NULL)
{
Expand Down Expand Up @@ -2010,7 +2010,7 @@ void sched_note_filter_irq(FAR struct note_filter_named_irq_s *oldf,
}
}

raw_spin_unlock_irqrestore(&g_note_lock, irq_mask);
spin_unlock_irqrestore_notrace(&g_note_lock, irq_mask);
}
#endif

Expand Down Expand Up @@ -2041,7 +2041,7 @@ void sched_note_filter_tag(FAR struct note_filter_named_tag_s *oldf,
FAR struct note_driver_s **driver;
irqstate_t irq_mask;

irq_mask = raw_spin_lock_irqsave(&g_note_lock);
irq_mask = spin_lock_irqsave_notrace(&g_note_lock);

if (oldf != NULL)
{
Expand Down Expand Up @@ -2077,7 +2077,7 @@ void sched_note_filter_tag(FAR struct note_filter_named_tag_s *oldf,
}
}

raw_spin_unlock_irqrestore(&g_note_lock, irq_mask);
spin_unlock_irqrestore_notrace(&g_note_lock, irq_mask);
}
#endif

Expand Down
26 changes: 13 additions & 13 deletions drivers/note/noteram_driver.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -467,9 +467,9 @@ static ssize_t noteram_read(FAR struct file *filep, FAR char *buffer,

if (ctx->mode == NOTERAM_MODE_READ_BINARY)
{
flags = raw_spin_lock_irqsave(&drv->lock);
flags = spin_lock_irqsave_notrace(&drv->lock);
ret = noteram_get(drv, (FAR uint8_t *)buffer, buflen);
raw_spin_unlock_irqrestore(&drv->lock, flags);
spin_unlock_irqrestore_notrace(&drv->lock, flags);
}
else
{
Expand All @@ -481,9 +481,9 @@ static ssize_t noteram_read(FAR struct file *filep, FAR char *buffer,

/* Get the next note (removing it from the buffer) */

flags = raw_spin_lock_irqsave(&drv->lock);
flags = spin_lock_irqsave_notrace(&drv->lock);
ret = noteram_get(drv, note, sizeof(note));
raw_spin_unlock_irqrestore(&drv->lock, flags);
spin_unlock_irqrestore_notrace(&drv->lock, flags);
if (ret <= 0)
{
return ret;
Expand All @@ -508,7 +508,7 @@ static int noteram_ioctl(FAR struct file *filep, int cmd, unsigned long arg)
{
int ret = -ENOSYS;
FAR struct noteram_driver_s *drv = filep->f_inode->i_private;
irqstate_t flags = raw_spin_lock_irqsave(&drv->lock);
irqstate_t flags = spin_lock_irqsave_notrace(&drv->lock);

/* Handle the ioctl commands */

Expand Down Expand Up @@ -600,7 +600,7 @@ static int noteram_ioctl(FAR struct file *filep, int cmd, unsigned long arg)
break;
}

raw_spin_unlock_irqrestore(&drv->lock, flags);
spin_unlock_irqrestore_notrace(&drv->lock, flags);
return ret;
}

Expand All @@ -622,7 +622,7 @@ static int noteram_poll(FAR struct file *filep, FAR struct pollfd *fds,
DEBUGASSERT(inode != NULL && inode->i_private != NULL);
drv = inode->i_private;

flags = raw_spin_lock_irqsave(&drv->lock);
flags = spin_lock_irqsave_notrace(&drv->lock);

/* Ignore waits that do not include POLLIN */

Expand Down Expand Up @@ -655,7 +655,7 @@ static int noteram_poll(FAR struct file *filep, FAR struct pollfd *fds,

if (noteram_unread_length(drv) > 0)
{
raw_spin_unlock_irqrestore(&drv->lock, flags);
spin_unlock_irqrestore_notrace(&drv->lock, flags);
poll_notify(&drv->pfd, 1, POLLIN);
return ret;
}
Expand All @@ -666,7 +666,7 @@ static int noteram_poll(FAR struct file *filep, FAR struct pollfd *fds,
}

errout:
raw_spin_unlock_irqrestore(&drv->lock, flags);
spin_unlock_irqrestore_notrace(&drv->lock, flags);
return ret;
}

Expand Down Expand Up @@ -698,11 +698,11 @@ static void noteram_add(FAR struct note_driver_s *driver,
unsigned int space;
irqstate_t flags;

flags = raw_spin_lock_irqsave(&drv->lock);
flags = spin_lock_irqsave_notrace(&drv->lock);

if (drv->ni_overwrite == NOTERAM_MODE_OVERWRITE_OVERFLOW)
{
raw_spin_unlock_irqrestore(&drv->lock, flags);
spin_unlock_irqrestore_notrace(&drv->lock, flags);
return;
}

Expand All @@ -716,7 +716,7 @@ static void noteram_add(FAR struct note_driver_s *driver,
/* Stop recording if not in overwrite mode */

drv->ni_overwrite = NOTERAM_MODE_OVERWRITE_OVERFLOW;
raw_spin_unlock_irqrestore(&drv->lock, flags);
spin_unlock_irqrestore_notrace(&drv->lock, flags);
return;
}

Expand All @@ -737,7 +737,7 @@ static void noteram_add(FAR struct note_driver_s *driver,
memcpy(drv->ni_buffer + head, note, space);
memcpy(drv->ni_buffer, buf + space, notelen - space);
drv->ni_head = noteram_next(drv, head, NOTE_ALIGN(notelen));
raw_spin_unlock_irqrestore(&drv->lock, flags);
spin_unlock_irqrestore_notrace(&drv->lock, flags);
poll_notify(&drv->pfd, 1, POLLIN);
}

Expand Down
8 changes: 4 additions & 4 deletions drivers/note/noterpmsg_driver.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -180,15 +180,15 @@ static bool noterpmsg_transfer(FAR struct noterpmsg_driver_s *drv,
static void noterpmsg_work(FAR void *priv)
{
FAR struct noterpmsg_driver_s *drv = priv;
irqstate_t flags = raw_spin_lock_irqsave(&drv->lock);
irqstate_t flags = spin_lock_irqsave_notrace(&drv->lock);

if (!noterpmsg_transfer(drv, false))
{
work_queue(HPWORK, &drv->work, noterpmsg_work, drv,
NOTE_RPMSG_WORK_DELAY);
}

raw_spin_unlock_irqrestore(&drv->lock, flags);
spin_unlock_irqrestore_notrace(&drv->lock, flags);
}

static void noterpmsg_add(FAR struct note_driver_s *driver,
Expand All @@ -199,7 +199,7 @@ static void noterpmsg_add(FAR struct note_driver_s *driver,
irqstate_t flags;
size_t space;

flags = raw_spin_lock_irqsave(&drv->lock);
flags = spin_lock_irqsave_notrace(&drv->lock);

space = CONFIG_DRIVERS_NOTERPMSG_BUFSIZE - noterpmsg_length(drv);
if (space < notelen)
Expand Down Expand Up @@ -236,7 +236,7 @@ static void noterpmsg_add(FAR struct note_driver_s *driver,
NOTE_RPMSG_WORK_DELAY);
}

raw_spin_unlock_irqrestore(&drv->lock, flags);
spin_unlock_irqrestore_notrace(&drv->lock, flags);
}

static int noterpmsg_ept_cb(FAR struct rpmsg_endpoint *ept,
Expand Down
4 changes: 2 additions & 2 deletions drivers/segger/config/SEGGER_RTT_Conf.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -89,11 +89,11 @@ extern spinlock_t g_segger_lock;

/* Lock RTT (nestable) (i.e. disable interrupts) */

#define SEGGER_RTT_LOCK() irqstate_t __flags = raw_spin_lock_irqsave(&g_segger_lock)
#define SEGGER_RTT_LOCK() irqstate_t __flags = spin_lock_irqsave_notrace(&g_segger_lock)

/* Unlock RTT (nestable) (i.e. enable previous interrupt lock state) */

#define SEGGER_RTT_UNLOCK() raw_spin_unlock_irqrestore(&g_segger_lock, __flags)
#define SEGGER_RTT_UNLOCK() spin_unlock_irqrestore_notrace(&g_segger_lock, __flags)

/* Disable RTT SEGGER_RTT_WriteSkipNoLock */

Expand Down
8 changes: 4 additions & 4 deletions drivers/syslog/syslog_intbuffer.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ void syslog_add_intbuffer(FAR const char *buffer, size_t buflen)

/* Disable concurrent modification from interrupt handling logic */

flags = raw_spin_lock_irqsave(&g_syslog_intbuffer.splock);
flags = spin_lock_irqsave_notrace(&g_syslog_intbuffer.splock);

space = circbuf_space(&g_syslog_intbuffer.circ);

Expand All @@ -172,7 +172,7 @@ void syslog_add_intbuffer(FAR const char *buffer, size_t buflen)
buffer + space, buflen - space);
}

raw_spin_unlock_irqrestore(&g_syslog_intbuffer.splock, flags);
spin_unlock_irqrestore_notrace(&g_syslog_intbuffer.splock, flags);
}

/****************************************************************************
Expand All @@ -198,9 +198,9 @@ void syslog_flush_intbuffer(bool force)
{
irqstate_t flags;

flags = raw_spin_lock_irqsave(&g_syslog_intbuffer.splock);
flags = spin_lock_irqsave_notrace(&g_syslog_intbuffer.splock);
syslog_flush_internal(force, sizeof(g_syslog_intbuffer.buffer));
raw_spin_unlock_irqrestore(&g_syslog_intbuffer.splock, flags);
spin_unlock_irqrestore_notrace(&g_syslog_intbuffer.splock, flags);
}

#endif /* CONFIG_SYSLOG_INTBUFFER */
18 changes: 9 additions & 9 deletions fs/inode/fs_files.c
View file
Original file line number Diff line number Diff line change
Expand Up @@ -72,9 +72,9 @@ static FAR struct file *files_fget_by_index(FAR struct filelist *list,
FAR struct file *filep;
irqstate_t flags;

flags = raw_spin_lock_irqsave(&list->fl_lock);
flags = spin_lock_irqsave_notrace(&list->fl_lock);
filep = &list->fl_files[l1][l2];
raw_spin_unlock_irqrestore(&list->fl_lock, flags);
spin_unlock_irqrestore_notrace(&list->fl_lock, flags);

#ifdef CONFIG_FS_REFCOUNT
if (filep->f_inode != NULL)
Expand Down Expand Up @@ -164,7 +164,7 @@ static int files_extend(FAR struct filelist *list, size_t row)
}
while (++i < row);

flags = raw_spin_lock_irqsave(&list->fl_lock);
flags = spin_lock_irqsave_notrace(&list->fl_lock);

/* To avoid race condition, if the file list is updated by other threads
* and list rows is greater or equal than temp list,
Expand All @@ -173,7 +173,7 @@ static int files_extend(FAR struct filelist *list, size_t row)

if (orig_rows != list->fl_rows && list->fl_rows >= row)
{
raw_spin_unlock_irqrestore(&list->fl_lock, flags);
spin_unlock_irqrestore_notrace(&list->fl_lock, flags);

for (j = orig_rows; j < i; j++)
{
Expand All @@ -195,7 +195,7 @@ static int files_extend(FAR struct filelist *list, size_t row)
list->fl_files = files;
list->fl_rows = row;

raw_spin_unlock_irqrestore(&list->fl_lock, flags);
spin_unlock_irqrestore_notrace(&list->fl_lock, flags);

if (tmp != NULL && tmp != &list->fl_prefile)
{
Expand Down Expand Up @@ -565,21 +565,21 @@ int file_allocate_from_tcb(FAR struct tcb_s *tcb, FAR struct inode *inode,

/* Find free file */

flags = raw_spin_lock_irqsave(&list->fl_lock);
flags = spin_lock_irqsave_notrace(&list->fl_lock);

for (; ; i++, j = 0)
{
if (i >= list->fl_rows)
{
raw_spin_unlock_irqrestore(&list->fl_lock, flags);
spin_unlock_irqrestore_notrace(&list->fl_lock, flags);

ret = files_extend(list, i + 1);
if (ret < 0)
{
return ret;
}

flags = raw_spin_lock_irqsave(&list->fl_lock);
flags = spin_lock_irqsave_notrace(&list->fl_lock);
}

do
Expand Down Expand Up @@ -608,7 +608,7 @@ int file_allocate_from_tcb(FAR struct tcb_s *tcb, FAR struct inode *inode,
}

found:
raw_spin_unlock_irqrestore(&list->fl_lock, flags);
spin_unlock_irqrestore_notrace(&list->fl_lock, flags);

if (addref)
{
Expand Down
2 changes: 1 addition & 1 deletion include/nuttx/irq.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@
do \
{ \
g_cpu_irqset = 0; \
raw_spin_unlock(&g_cpu_irqlock); \
spin_unlock_notrace(&g_cpu_irqlock); \
} \
while (0)
#endif
Expand Down
Loading

0 comments on commit 59ebddf

Please sign in to comment.