mirrors/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity.git synced 2025-01-24 02:12:09 -05:00
Code Issues Projects Releases Packages Wiki Activity

Kernel: Make sure we never put the colonel thread in the runnable list.

Browse source 
This would cause it to get scheduled unnecessarily.
This commit is contained in:
Andreas Kling 2019-05-18 20:24:55 +02:00
parent 64a4f3df69
commit 7900da9667
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
Kernel/Scheduler.cpp
View file

@ -232,6 +232,9 @@ bool Scheduler::pick_next()
}
#endif
if (g_runnable_threads->is_empty())
return context_switch(s_colonel_process->main_thread());
auto* previous_head = g_runnable_threads->head();
for (;;) {
// Move head to tail.

4
Kernel/Thread.cpp
View file

@ -546,6 +546,7 @@ bool Thread::is_thread(void* ptr)
void Thread::set_thread_list(InlineLinkedList<Thread>* thread_list)
{
ASSERT(pid() != 0);
if (m_thread_list == thread_list)
return;
if (m_thread_list)
@ -558,5 +559,6 @@ void Thread::set_thread_list(InlineLinkedList<Thread>* thread_list)
void Thread::set_state(State new_state)
{
m_state = new_state;
set_thread_list(thread_list_for_state(new_state));
if (m_process.pid() != 0)
set_thread_list(thread_list_for_state(new_state));
}