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

[kernel] Remove thread_data pointer from TCB

Browse Source 
The TCB is always stored at a constant offset within the thread object.
So instead of carrying an extra pointer, just implement thread::from_tcb
to get the thread.
This commit is contained in:
Justin C. Miller
2020-07-19 17:01:15 -07:00
parent ef5c333030
commit c3abe035c8
5 changed files with 20 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/kernel/scheduler.cpp
View File

@@ -116,7 +116,7 @@ load_process_image(const void *image_start, size_t bytes, TCB *tcb)
// process code and load it
page_manager *pager = page_manager::get();
thread *th = tcb->thread_data;
thread *th = thread::from_tcb(tcb);
log::debug(logs::loader, "Loading task! ELF: %016lx [%d]", image_start, bytes);
@@ -279,7 +279,7 @@ void scheduler::prune(uint64_t now)
// move them to the appropriate lists.
auto *tcb = m_blocked.front();
while (tcb) {
thread *th = tcb->thread_data;
thread *th = thread::from_tcb(tcb);
uint8_t priority = tcb->priority;
bool ready = th->has_state(thread::state::ready);
@@ -311,7 +311,7 @@ scheduler::check_promotions(uint64_t now)
{
for (auto &pri_list : m_runlists) {
for (auto *tcb : pri_list) {
const thread *th = m_current->thread_data;
const thread *th = thread::from_tcb(m_current);
const bool constant = th->has_state(thread::state::constant);
if (constant)
continue;
@@ -345,7 +345,7 @@ scheduler::schedule()
uint8_t priority = m_current->priority;
uint32_t remaining = m_apic->stop_timer();
m_current->time_left = remaining;
thread *th = m_current->thread_data;
thread *th = thread::from_tcb(m_current);
const bool constant = th->has_state(thread::state::constant);
if (remaining == 0) {