Kernel: Remove colonel special-case from Process::for_each_thread

Since each Process now has its own list of threads, we don't need
to treat colonel any different anymore. This also means that it
reports all kernel threads, not just the idle threads.

1 files changed, 5 insertions, 16 deletions

diff --git a/Kernel/Process.h b/Kernel/Process.h
index 5b851ace20..ae43f88aea 100644
--- a/Kernel/Process.h
+++ b/Kernel/Process.h
@@ -700,22 +700,11 @@ inline void Process::for_each_child(Callback callback)
 template<typename Callback>
 inline IterationDecision Process::for_each_thread(Callback callback) const
 {
-    if (pid() == 0) {
-        // NOTE: Special case the colonel process, since its main thread is not in the global thread table.
-        Processor::for_each(
-            [&](Processor& proc) -> IterationDecision {
-                auto idle_thread = proc.idle_thread();
-                if (idle_thread != nullptr)
-                    return callback(*idle_thread);
-                return IterationDecision::Continue;
-            });
-    } else {
-        ScopedSpinLock thread_list_lock(m_thread_list_lock);
-        for (auto& thread : m_thread_list) {
-            IterationDecision decision = callback(thread);
-            if (decision != IterationDecision::Continue)
-                return decision;
-        }
+    ScopedSpinLock thread_list_lock(m_thread_list_lock);
+    for (auto& thread : m_thread_list) {
+        IterationDecision decision = callback(thread);
+        if (decision != IterationDecision::Continue)
+            return decision;
     }
     return IterationDecision::Continue;
 }