diff options
author | Andreas Kling <awesomekling@gmail.com> | 2019-03-23 22:59:08 +0100 |
---|---|---|
committer | Andreas Kling <awesomekling@gmail.com> | 2019-03-23 22:59:08 +0100 |
commit | e561ab1b0b0c3e37d729ce6620f375ac8fefc0b6 (patch) | |
tree | 630600533980f9873370d35f0e286504dbcaa9a8 /Kernel | |
parent | 7f1757b16c82f7398e2386d84d9bae877d5fc39b (diff) | |
download | serenity-e561ab1b0b0c3e37d729ce6620f375ac8fefc0b6.zip |
Kernel+LibC: Add a simple create_thread() syscall.
It takes two parameters, a function pointer for the entry function,
and a void* argument to be passed to that function on the new thread.
Diffstat (limited to 'Kernel')
-rw-r--r-- | Kernel/Process.cpp | 17 | ||||
-rw-r--r-- | Kernel/Process.h | 1 | ||||
-rw-r--r-- | Kernel/Syscall.cpp | 2 | ||||
-rw-r--r-- | Kernel/Syscall.h | 1 | ||||
-rw-r--r-- | Kernel/Thread.cpp | 15 | ||||
-rw-r--r-- | Kernel/Thread.h | 3 | ||||
-rw-r--r-- | Kernel/i386.cpp | 3 | ||||
-rwxr-xr-x | Kernel/sync.sh | 1 |
8 files changed, 38 insertions, 5 deletions
diff --git a/Kernel/Process.cpp b/Kernel/Process.cpp index 4ac23417ad..6f771297d1 100644 --- a/Kernel/Process.cpp +++ b/Kernel/Process.cpp @@ -406,7 +406,7 @@ int Process::do_exec(String path, Vector<String> arguments, Vector<String> envir main_thread().m_tss.gs = 0x23; main_thread().m_tss.ss = 0x23; main_thread().m_tss.cr3 = page_directory().cr3(); - main_thread().make_userspace_stack(move(arguments), move(environment)); + main_thread().make_userspace_stack_for_main_thread(move(arguments), move(environment)); main_thread().m_tss.ss0 = 0x10; main_thread().m_tss.esp0 = old_esp0; main_thread().m_tss.ss2 = m_pid; @@ -2452,3 +2452,18 @@ int Process::thread_count() const }); return count; } + +int Process::sys$create_thread(int(*entry)(void*), void* argument) +{ + if (!validate_read((const void*)entry, sizeof(void*))) + return -EFAULT; + auto* thread = new Thread(*this); + auto& tss = thread->tss(); + tss.eip = (dword)entry; + tss.eflags = 0x0202; + tss.cr3 = page_directory().cr3(); + thread->make_userspace_stack_for_secondary_thread(argument); + + thread->set_state(Thread::State::Runnable); + return 0; +} diff --git a/Kernel/Process.h b/Kernel/Process.h index 20084c4e73..40a4c4234a 100644 --- a/Kernel/Process.h +++ b/Kernel/Process.h @@ -178,6 +178,7 @@ public: int sys$getsockopt(const Syscall::SC_getsockopt_params*); int sys$setsockopt(const Syscall::SC_setsockopt_params*); int sys$restore_signal_mask(dword mask); + int sys$create_thread(int(*)(void*), void*); int sys$create_shared_buffer(pid_t peer_pid, int, void** buffer); void* sys$get_shared_buffer(int shared_buffer_id); diff --git a/Kernel/Syscall.cpp b/Kernel/Syscall.cpp index ffb59b5386..fcc5845787 100644 --- a/Kernel/Syscall.cpp +++ b/Kernel/Syscall.cpp @@ -235,6 +235,8 @@ static dword handle(RegisterDump& regs, dword function, dword arg1, dword arg2, return current->process().sys$getsockopt((const SC_getsockopt_params*)arg1); case Syscall::SC_setsockopt: return current->process().sys$setsockopt((const SC_setsockopt_params*)arg1); + case Syscall::SC_create_thread: + return current->process().sys$create_thread((int(*)(void*))arg1, (void*)arg2); default: kprintf("<%u> int0x82: Unknown function %u requested {%x, %x, %x}\n", current->process().pid(), function, arg1, arg2, arg3); break; diff --git a/Kernel/Syscall.h b/Kernel/Syscall.h index 417ce12dd6..8124cbc694 100644 --- a/Kernel/Syscall.h +++ b/Kernel/Syscall.h @@ -92,6 +92,7 @@ __ENUMERATE_SYSCALL(recvfrom) \ __ENUMERATE_SYSCALL(getsockopt) \ __ENUMERATE_SYSCALL(setsockopt) \ + __ENUMERATE_SYSCALL(create_thread) \ namespace Syscall { diff --git a/Kernel/Thread.cpp b/Kernel/Thread.cpp index c3e61bf0c9..cdc840ffff 100644 --- a/Kernel/Thread.cpp +++ b/Kernel/Thread.cpp @@ -435,7 +435,7 @@ void Thread::push_value_on_stack(dword value) *stack_ptr = value; } -void Thread::make_userspace_stack(Vector<String> arguments, Vector<String> environment) +void Thread::make_userspace_stack_for_main_thread(Vector<String> arguments, Vector<String> environment) { auto* region = m_process.allocate_region(LinearAddress(), default_userspace_stack_size, "stack"); ASSERT(region); @@ -482,12 +482,23 @@ void Thread::make_userspace_stack(Vector<String> arguments, Vector<String> envir push_value_on_stack(0); } +void Thread::make_userspace_stack_for_secondary_thread(void *argument) +{ + auto* region = m_process.allocate_region(LinearAddress(), default_userspace_stack_size, String::format("Thread %u Stack", tid())); + ASSERT(region); + m_stack_top3 = region->laddr().offset(default_userspace_stack_size).get(); + m_tss.esp = m_stack_top3; + + // NOTE: The stack needs to be 16-byte aligned. + push_value_on_stack((dword)argument); + push_value_on_stack(0); +} + Thread* Thread::clone(Process& process) { auto* clone = new Thread(process); memcpy(clone->m_signal_action_data, m_signal_action_data, sizeof(m_signal_action_data)); clone->m_signal_mask = m_signal_mask; - //clone->m_tss = m_tss; clone->m_fpu_state = m_fpu_state; clone->m_has_used_fpu = m_has_used_fpu; return clone; diff --git a/Kernel/Thread.h b/Kernel/Thread.h index 85be55caa7..c4a0095eef 100644 --- a/Kernel/Thread.h +++ b/Kernel/Thread.h @@ -116,7 +116,8 @@ public: void set_default_signal_dispositions(); void push_value_on_stack(dword); - void make_userspace_stack(Vector<String> arguments, Vector<String> environment); + void make_userspace_stack_for_main_thread(Vector<String> arguments, Vector<String> environment); + void make_userspace_stack_for_secondary_thread(void* argument); Thread* clone(Process&); diff --git a/Kernel/i386.cpp b/Kernel/i386.cpp index 44103f0b82..3133b9ae4e 100644 --- a/Kernel/i386.cpp +++ b/Kernel/i386.cpp @@ -273,9 +273,10 @@ void exception_14_handler(RegisterDumpWithExceptionCode& regs) auto response = MM.handle_page_fault(PageFault(regs.exception_code, LinearAddress(faultAddress))); if (response == PageFaultResponse::ShouldCrash) { - kprintf("%s(%u) unrecoverable page fault, %s laddr=%p\n", + kprintf("%s(%u:%u) unrecoverable page fault, %s laddr=%p\n", current->process().name().characters(), current->pid(), + current->tid(), regs.exception_code & 2 ? "write" : "read", faultAddress); dump(regs); diff --git a/Kernel/sync.sh b/Kernel/sync.sh index 52e94cf382..2cd3585008 100755 --- a/Kernel/sync.sh +++ b/Kernel/sync.sh @@ -89,6 +89,7 @@ cp -v ../Applications/About/About mnt/bin/About cp -v ../Applications/TextEditor/TextEditor mnt/bin/TextEditor cp -v ../Applications/IRCClient/IRCClient mnt/bin/IRCClient ln -s IRCClient mnt/bin/irc +ln -s FileManager mnt/bin/fm cp -v ../Servers/LookupServer/LookupServer mnt/bin/LookupServer cp -v ../Servers/WindowServer/WindowServer mnt/bin/WindowServer cp -v kernel.map mnt/ |