diff options
| author | Idan Horowitz <idan.horowitz@gmail.com> | 2023-04-04 14:13:41 +0300 |
|---|---|---|
| committer | Idan Horowitz <idan.horowitz@gmail.com> | 2023-04-06 20:30:03 +0300 |
| commit | d1082a00b768218f45eb7c1c43e47b5452a5b67f (patch) | |
| tree | 80c2a564c002808b4eac90be470fd638c5de8454 | |
| parent | 2f79d0e8b9888fc3dd35db119d3a951373a7ac58 (diff) | |
| download | serenity-d1082a00b768218f45eb7c1c43e47b5452a5b67f.zip | |
Kernel: Mark sys$set_mmap_name as not needing the big lock
All accesses to shared mutable data are already serialized behind the
process address space spinlock.
| -rw-r--r-- | Kernel/API/Syscall.h | 2 | ||||
| -rw-r--r-- | Kernel/Syscalls/mmap.cpp | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/Kernel/API/Syscall.h b/Kernel/API/Syscall.h index 3b91d57c03..ab74efa75f 100644 --- a/Kernel/API/Syscall.h +++ b/Kernel/API/Syscall.h @@ -157,7 +157,7 @@ enum class NeedsBigProcessLock { S(scheduler_set_parameters, NeedsBigProcessLock::No) \ S(sendfd, NeedsBigProcessLock::No) \ S(sendmsg, NeedsBigProcessLock::Yes) \ - S(set_mmap_name, NeedsBigProcessLock::Yes) \ + S(set_mmap_name, NeedsBigProcessLock::No) \ S(set_thread_name, NeedsBigProcessLock::No) \ S(setegid, NeedsBigProcessLock::No) \ S(seteuid, NeedsBigProcessLock::No) \ diff --git a/Kernel/Syscalls/mmap.cpp b/Kernel/Syscalls/mmap.cpp index 31b84f3ccc..8e66f4ca03 100644 --- a/Kernel/Syscalls/mmap.cpp +++ b/Kernel/Syscalls/mmap.cpp @@ -441,7 +441,7 @@ ErrorOr<FlatPtr> Process::sys$madvise(Userspace<void*> address, size_t size, int ErrorOr<FlatPtr> Process::sys$set_mmap_name(Userspace<Syscall::SC_set_mmap_name_params const*> user_params) { - VERIFY_PROCESS_BIG_LOCK_ACQUIRED(this); + VERIFY_NO_PROCESS_BIG_LOCK(this); TRY(require_promise(Pledge::stdio)); auto params = TRY(copy_typed_from_user(user_params)); |