xiaoxiang781216 commented on code in PR #11125: URL: https://github.com/apache/nuttx/pull/11125#discussion_r1383065140
########## fs/inode/fs_files.c: ########## @@ -583,6 +605,7 @@ int fs_getfilep(int fd, FAR struct file **filep) ret = nxmutex_lock(&list->fl_lock); Review Comment: kmm_xx may switch out due to fail to hold mm_lock, and then the critical section also auto leaves temporarily. Before the scheduler switch in this thread and rehold the critical section again, other threads may grip the critical section and enter the same code path. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
