michaelh/zephyr
1
0
Fork 0
Code Releases Activity

kernel: Use _reschedule() instead of _Swap() where possible

Browse source 
These two spots were duplicating logic that is already done inside
_reschedule(), which is the cleaner, less dangerous API.  Use it where
possible when outside the scheduler internals.

Signed-off-by: Andy Ross <andrew.j.ross@intel.com>
This commit is contained in:
Andy Ross 2018-07-19 10:41:01 -07:00 • committed by Anas Nashif
commit 53cae5f471
2 changed files with 2 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

2
kernel/thread.c
View file

@ -540,7 +540,7 @@ void _impl_k_thread_suspend(struct k_thread *thread)
sys_trace_thread_suspend(thread); sys_trace_thread_suspend(thread);
if (thread == _current) { if (thread == _current) {
(void)_Swap(key); _reschedule(key);
} else { } else {
irq_unlock(key); irq_unlock(key);
} }

12
kernel/thread_abort.c
View file

@ -37,17 +37,7 @@ void _impl_k_thread_abort(k_tid_t thread)
_k_thread_single_abort(thread); _k_thread_single_abort(thread);
_thread_monitor_exit(thread); _thread_monitor_exit(thread);
if (_is_in_isr()) { _reschedule(key);
irq_unlock(key);
} else {
if (_current == thread) {
(void)_Swap(key);
CODE_UNREACHABLE;
}
/* The abort handler might have altered the ready queue. */
_reschedule(key);
}
} }
#endif #endif