Kernel: Remove big lock from sys$connect

This syscall doesn't access any unprotected shared data.
Author: https://github.com/IdanHo Commit: https://github.com/SerenityOS/serenity/commit/e620487b66 Pull-request: https://github.com/SerenityOS/serenity/pull/13594
2022-04-09 22:51:14 +03:00 · 2022-04-09 22:51:14 +03:00 · e620487b66 · 2024-07-17 14:13:44 +09:00
commit e620487b66
parent 9547a8e8a2
2 changed files with 2 additions and 2 deletions
--- a/Kernel/API/Syscall.h
+++ b/Kernel/API/Syscall.h
@ -55,7 +55,7 @@ enum class NeedsBigProcessLock {
    S(clock_nanosleep, NeedsBigProcessLock::No)             \
    S(clock_settime, NeedsBigProcessLock::No)               \
    S(close, NeedsBigProcessLock::No)                       \
-    S(connect, NeedsBigProcessLock::Yes)                    \
+    S(connect, NeedsBigProcessLock::No)                     \
    S(create_inode_watcher, NeedsBigProcessLock::Yes)       \
    S(create_thread, NeedsBigProcessLock::Yes)              \
    S(dbgputstr, NeedsBigProcessLock::No)                   \
--- a/Kernel/Syscalls/socket.cpp
+++ b/Kernel/Syscalls/socket.cpp
@ -145,7 +145,7 @@ ErrorOr<FlatPtr> Process::sys$accept4(Userspace<Syscall::SC_accept4_params const

 ErrorOr<FlatPtr> Process::sys$connect(int sockfd, Userspace<sockaddr const*> user_address, socklen_t user_address_size)
 {
-    VERIFY_PROCESS_BIG_LOCK_ACQUIRED(this)
+    VERIFY_NO_PROCESS_BIG_LOCK(this)

    auto description = TRY(open_file_description(sockfd));
    if (!description->is_socket())