Compare commits

...

1 Commits

Author SHA1 Message Date
chao an
b96f993015 sched/sched_lock: remove null pointer check of rtcb
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>
2025-11-11 18:41:17 +08:00
2 changed files with 2 additions and 2 deletions

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)

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.