Mirrors_Framework/nuttx
1
0
Fork 0
mirror of https://github.com/apache/nuttx.git synced 2025-12-11 04:47:47 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

sched/sched_lock: remove null pointer check of rtcb

Browse Source 
In previous commits to xiaomi, runtime null pointer checks for RTCB were removed.
This could cause `sched_lock/unlock` exceptions when `DEBUG_ASSERTIONS` was disabled.
In this commit, we removed all RTCB checks to improve performance by 1 comparison cycle
References to null pointers in the RTCB will rely on hardware exception or MPU/MMU protection.

| commit d94cb53d6c (HEAD, origin/master, origin/HEAD)
| Author: hujun5 <hujun5@xiaomi.com>
| Date:   Thu Feb 6 15:06:00 2025 +0800
|
|     sched_lock: remove the check for whether tcb is NULL
|
|     Remove Redundant Checks
|
|     Signed-off-by: hujun5 <hujun5@xiaomi.com>

Signed-off-by: chao an <anchao.archer@bytedance.com>
This commit is contained in:
chao an
2025-11-11 18:31:37 +08:00
parent d94cb53d6c
commit b96f993015
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
sched/sched/sched_lock.c
View File

@@ -76,7 +76,7 @@ void sched_lock(void)
* integer type.
*/
DEBUGASSERT(rtcb && rtcb->lockcount < MAX_LOCK_COUNT);
DEBUGASSERT(rtcb->lockcount < MAX_LOCK_COUNT);
/* A counter is used to support locking. This allows nested lock
* operations on this thread (on any CPU)

2
sched/sched/sched_unlock.c
View File

@@ -64,7 +64,7 @@ void sched_unlock(void)
/* rtcb may be NULL only during early boot-up phases */
DEBUGASSERT(rtcb && rtcb->lockcount > 0);
DEBUGASSERT(rtcb->lockcount > 0);
/* Check if the lock counter has decremented to zero. If so,
* then pre-emption has been re-enabled.