Aksal/nuttx
3
1
Fork 1
mirror of https://github.com/apache/nuttx.git synced 2026-10-10 07:40:27 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

fs/inode: Name the inode tree lock in the comments, not g_inode_sem.

Browse source 
Three comments say that the caller of inode_search() and
_inode_linktarget() holds the g_inode_sem semaphore.  That semaphore no
longer exists.  The caller holds the inode tree lock, from inode_lock()
or inode_rlock().  No code change.

Assisted-by: Claude Code:claude-opus-5-5
Signed-off-by: Marco Casaroli <marco.casaroli@gmail.com>
This commit is contained in:
Marco Casaroli 2026-10-08 12:20:07 +02:00 • committed by Alan C. Assis
parent 810c12770e
commit 7e6c2eaf00
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
fs/inode/fs_inodesearch.c
View file

@ -150,7 +150,7 @@ static int _inode_compare(FAR const char *fname, FAR struct inode *inode)
* the soft link.
*
* Assumptions:
* The caller holds the g_inode_sem semaphore
* The caller holds the inode tree lock (inode_lock() or inode_rlock())
*
****************************************************************************/
@ -577,7 +577,7 @@ int inode_search_setup(FAR struct inode_search_s *desc,
* that link WILL be deferenced unconditionally.
*
* Assumptions:
* The caller holds the g_inode_sem semaphore
* The caller holds the inode tree lock (inode_lock() or inode_rlock())
* The descriptor was initialized with inode_search_setup()
*
****************************************************************************/

2
fs/inode/inode.h
View file

@ -221,7 +221,7 @@ void inode_runlock(void);
* cares about existence (e.g. inode_reserve).
*
* Assumptions:
* The caller holds the g_inode_sem semaphore
* The caller holds the inode tree lock (inode_lock() or inode_rlock())
* The descriptor was initialized with inode_search_setup()
*
****************************************************************************/