Skip to content

Commit cc558d9

Browse files
AtkinsSJawesomekling
authored andcommitted
Kernel: Remove pledge syscall from the big lock
This already does all its dangerous work inside `with_mutable_protected_data()`.
1 parent ce78394 commit cc558d9

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

Kernel/API/Syscall.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ enum class NeedsBigProcessLock {
133133
S(perf_event, NeedsBigProcessLock::Yes) \
134134
S(perf_register_string, NeedsBigProcessLock::Yes) \
135135
S(pipe, NeedsBigProcessLock::No) \
136-
S(pledge, NeedsBigProcessLock::Yes) \
136+
S(pledge, NeedsBigProcessLock::No) \
137137
S(poll, NeedsBigProcessLock::Yes) \
138138
S(posix_fallocate, NeedsBigProcessLock::No) \
139139
S(prctl, NeedsBigProcessLock::Yes) \

Kernel/Syscalls/pledge.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ namespace Kernel {
1111

1212
ErrorOr<FlatPtr> Process::sys$pledge(Userspace<Syscall::SC_pledge_params const*> user_params)
1313
{
14-
VERIFY_PROCESS_BIG_LOCK_ACQUIRED(this);
14+
VERIFY_NO_PROCESS_BIG_LOCK(this);
1515
auto params = TRY(copy_typed_from_user(user_params));
1616

1717
if (params.promises.length > 1024 || params.execpromises.length > 1024)

0 commit comments

Comments
 (0)