diff options
| author | Liav A <liavalb@gmail.com> | 2021-09-11 15:57:41 +0300 |
|---|---|---|
| committer | Andreas Kling <kling@serenityos.org> | 2021-09-11 19:05:00 +0200 |
| commit | 04ba31b8c5ec71bf57cc34d58f8dd1d08bea87ab (patch) | |
| tree | c3956413c9aa902e35175fcc8b82e118829d9caf /Kernel/Process.cpp | |
| parent | b92871f7ef293a9149aa7c952dd50bf1fadcaf55 (diff) | |
| download | serenity-04ba31b8c5ec71bf57cc34d58f8dd1d08bea87ab.zip | |
Kernel+Userland: Remove loadable kernel moduless
These interfaces are broken for about 9 months, maybe longer than that.
At this point, this is just a dead code nobody tests or tries to use, so
let's remove it instead of keeping a stale code just for the sake of
keeping it and hoping someone will fix it.
To better justify this, I read that OpenBSD removed loadable kernel
modules in 5.7 release (2014), mainly for the same reason we do -
nobody used it so they had no good reason to maintain it.
Still, OpenBSD had LKMs being effectively working, which is not the
current state in our project for a long time.
An arguably better approach to minimize the Kernel image size is to
allow dropping drivers and features while compiling a new image.
Diffstat (limited to 'Kernel/Process.cpp')
| -rw-r--r-- | Kernel/Process.cpp | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/Kernel/Process.cpp b/Kernel/Process.cpp index 62014dfcc9..1b53fc2e7d 100644 --- a/Kernel/Process.cpp +++ b/Kernel/Process.cpp @@ -25,7 +25,6 @@ #include <Kernel/Memory/AnonymousVMObject.h> #include <Kernel/Memory/PageDirectory.h> #include <Kernel/Memory/SharedInodeVMObject.h> -#include <Kernel/Module.h> #include <Kernel/PerformanceEventBuffer.h> #include <Kernel/PerformanceManager.h> #include <Kernel/Process.h> @@ -45,7 +44,6 @@ static void create_signal_trampoline(); RecursiveSpinlock g_profiling_lock; static Atomic<pid_t> next_pid; static Singleton<SpinlockProtected<Process::List>> s_processes; -READONLY_AFTER_INIT HashMap<String, OwnPtr<Module>>* g_modules; READONLY_AFTER_INIT Memory::Region* g_signal_trampoline_region; static Singleton<MutexProtected<String>> s_hostname; @@ -72,8 +70,6 @@ ProcessID Process::allocate_pid() UNMAP_AFTER_INIT void Process::initialize() { - g_modules = new HashMap<String, OwnPtr<Module>>; - next_pid.store(0, AK::MemoryOrder::memory_order_release); // Note: This is called before scheduling is initialized, and before APs are booted. |