From 60be4eb653d3e44dd0d34caa107e012cf8bd819c Mon Sep 17 00:00:00 2001 From: Andrew Boie Date: Fri, 14 Feb 2020 12:03:48 -0800 Subject: [PATCH] kernel: remove comment in k_thread_abort() z_reschedule_unlocked() is a no-op if the caller is cooperative, because the logic that maintains the ready queue ensures that the co-op thread is always at the front unless some special handling is done like in k_yield(), which does not happen here. Signed-off-by: Andrew Boie --- kernel/thread_abort.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/kernel/thread_abort.c b/kernel/thread_abort.c index 152962ebf38..6500827a158 100644 --- a/kernel/thread_abort.c +++ b/kernel/thread_abort.c @@ -42,14 +42,6 @@ void z_impl_k_thread_abort(k_tid_t thread) z_swap(&lock, k_spin_lock(&lock)); } else { - /* Really, there's no good reason for this to be a - * scheduling point if we aren't aborting _current (by - * definition, no higher priority thread is runnable, - * because we're running!). But it always has been - * and is thus part of our API, and we have tests that - * rely on k_thread_abort() scheduling out of - * cooperative threads. - */ z_reschedule_unlocked(); } }