michaelh/zephyr
1
0
Fork 0
Code Releases Activity

kernel: Revert: "Add static threads to k_thread_foreach functions"

Browse source 
Revert commit mistakenly iterating over static threads in
k_thread_foreach functions. The static threads where already included
in the for-loop, and is now duplicated.

This reverts commit bd3b4b0caf.

Signed-off-by: Joakim Andersson <joakim.andersson@nordicsemi.no>
This commit is contained in:
Joakim Andersson 2020-04-27 16:47:26 +02:00 • committed by Carles Cufí
commit aac2220d29
1 changed files with 0 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

14
kernel/thread.c
View file

@ -55,15 +55,9 @@ void k_thread_foreach(k_thread_user_cb_t user_cb, void *user_data)
* k_thread_abort from user_cb.
*/
key = k_spin_lock(&z_thread_monitor_lock);
_FOREACH_STATIC_THREAD(thread_data) {
user_cb(thread_data->init_thread, user_data);
}
for (thread = _kernel.threads; thread; thread = thread->next_thread) {
user_cb(thread, user_data);
}
k_spin_unlock(&z_thread_monitor_lock, key);
#endif
}
@ -77,19 +71,11 @@ void k_thread_foreach_unlocked(k_thread_user_cb_t user_cb, void *user_data)
__ASSERT(user_cb != NULL, "user_cb can not be NULL");
key = k_spin_lock(&z_thread_monitor_lock);
_FOREACH_STATIC_THREAD(thread_data) {
k_spin_unlock(&z_thread_monitor_lock, key);
user_cb(thread_data->init_thread, user_data);
key = k_spin_lock(&z_thread_monitor_lock);
}
for (thread = _kernel.threads; thread; thread = thread->next_thread) {
k_spin_unlock(&z_thread_monitor_lock, key);
user_cb(thread, user_data);
key = k_spin_lock(&z_thread_monitor_lock);
}
k_spin_unlock(&z_thread_monitor_lock, key);
#endif
}