RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-10-31 17:32:44 +00:00
Code Issues Activity Actions

Kernel: Mark sys$adjtime() as not needing the big lock

Browse source 
This syscall works on global kernel state and so doesn't need protection
from threads in the same process.
This commit is contained in:
Andreas Kling 2022-04-04 00:39:43 +02:00
parent 4306422f29
commit 12b612ab14
2 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Kernel/API/Syscall.h
View file

@ -42,7 +42,7 @@ enum class NeedsBigProcessLock {
#define ENUMERATE_SYSCALLS(S) \ #define ENUMERATE_SYSCALLS(S) \
S(accept4, NeedsBigProcessLock::Yes) \ S(accept4, NeedsBigProcessLock::Yes) \
S(access, NeedsBigProcessLock::Yes) \ S(access, NeedsBigProcessLock::Yes) \
S(adjtime, NeedsBigProcessLock::Yes) \ S(adjtime, NeedsBigProcessLock::No) \
S(alarm, NeedsBigProcessLock::Yes) \ S(alarm, NeedsBigProcessLock::Yes) \
S(allocate_tls, NeedsBigProcessLock::Yes) \ S(allocate_tls, NeedsBigProcessLock::Yes) \
S(anon_create, NeedsBigProcessLock::No) \ S(anon_create, NeedsBigProcessLock::No) \

2
Kernel/Syscalls/clock.cpp
View file

@ -93,7 +93,7 @@ ErrorOr<FlatPtr> Process::sys$clock_nanosleep(Userspace<Syscall::SC_clock_nanosl
ErrorOr<FlatPtr> Process::sys$adjtime(Userspace<timeval const*> user_delta, Userspace<timeval*> user_old_delta) ErrorOr<FlatPtr> Process::sys$adjtime(Userspace<timeval const*> user_delta, Userspace<timeval*> user_old_delta)
{ {
VERIFY_PROCESS_BIG_LOCK_ACQUIRED(this); VERIFY_NO_PROCESS_BIG_LOCK(this);
if (user_old_delta) { if (user_old_delta) {
timespec old_delta_ts = TimeManagement::the().remaining_epoch_time_adjustment(); timespec old_delta_ts = TimeManagement::the().remaining_epoch_time_adjustment();
timeval old_delta; timeval old_delta;