justin/jsix_import
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

[kernel] Don't keep creating new processes for kernel tasks

Browse Source 
The scheduler was making new process objects every time it made kernel
task threads. Don't do that.
This commit is contained in:
Justin C. Miller
2020-08-02 18:13:09 -07:00
parent e1b1b5d357
commit b98334db28
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/kernel/scheduler.cpp
View File

@@ -47,6 +47,7 @@ scheduler::scheduler(lapic *apic) :
reinterpret_cast<uintptr_t>(&idle_stack_end)); reinterpret_cast<uintptr_t>(&idle_stack_end));
log::debug(logs::task, "Idle thread koid %llx", idle->koid()); log::debug(logs::task, "Idle thread koid %llx", idle->koid());
log::debug(logs::task, "Kernel PML4 %llx", pml4);
auto *tcb = idle->tcb(); auto *tcb = idle->tcb();
m_runlists[max_priority].push_back(tcb); m_runlists[max_priority].push_back(tcb);
@@ -181,20 +182,16 @@ void
scheduler::create_kernel_task(void (*task)(), uint8_t priority, bool constant) scheduler::create_kernel_task(void (*task)(), uint8_t priority, bool constant)
{ {
page_table *pml4 = page_manager::get()->get_kernel_pml4(); page_table *pml4 = page_manager::get()->get_kernel_pml4();
thread *th = create_process(pml4, false); thread *th = m_kernel_process->create_thread(priority, false);
auto *tcb = th->tcb(); auto *tcb = th->tcb();
uint16_t kcs = (1 << 3) | 0; // Kernel CS is GDT entry 1, ring 0
uint16_t kss = (2 << 3) | 0; // Kernel SS is GDT entry 2, ring 0
th->add_thunk_kernel(reinterpret_cast<uintptr_t>(task)); th->add_thunk_kernel(reinterpret_cast<uintptr_t>(task));
tcb->priority = priority; tcb->time_left = quantum(priority);
tcb->pml4 = page_manager::get()->get_kernel_pml4();
if (constant) if (constant)
th->set_state(thread::state::constant); th->set_state(thread::state::constant);
m_runlists[default_priority].push_back(tcb); m_runlists[priority].push_back(tcb);
log::debug(logs::task, "Creating kernel task: thread %llx pri %d", th->koid(), tcb->priority); log::debug(logs::task, "Creating kernel task: thread %llx pri %d", th->koid(), tcb->priority);
log::debug(logs::task, " RSP0 %016lx", tcb->rsp0); log::debug(logs::task, " RSP0 %016lx", tcb->rsp0);
@@ -337,8 +334,11 @@ scheduler::schedule()
bsp_cpu_data.p = &th->parent(); bsp_cpu_data.p = &th->parent();
thread *next_thread = thread::from_tcb(m_current); thread *next_thread = thread::from_tcb(m_current);
log::debug(logs::task, "Scheduler switching threads %llx->%llx, priority %d time left %d @ %lld.", log::debug(logs::task, "Scheduler switching threads %llx->%llx",
th->koid(), next_thread->koid(), m_current->priority, m_current->time_left, m_clock); th->koid(), next_thread->koid());
log::debug(logs::task, " priority %d time left %d @ %lld.",
m_current->priority, m_current->time_left, m_clock);
log::debug(logs::task, " PML4 %llx", m_current->pml4);
task_switch(m_current); task_switch(m_current);
} }