Kernel: Remove big lock from sys$statvfs

This syscall doesn't access any unprotected shared data.
Author: https://github.com/IdanHo Commit: https://github.com/SerenityOS/serenity/commit/d4ce43cf45 Pull-request: https://github.com/SerenityOS/serenity/pull/13594
2022-04-09 23:07:20 +03:00 · 2022-04-09 23:07:20 +03:00 · d4ce43cf45 · 2024-07-17 14:13:15 +09:00
commit d4ce43cf45
parent 4ae93179f1
2 changed files with 2 additions and 2 deletions
--- a/Kernel/API/Syscall.h
+++ b/Kernel/API/Syscall.h
@ -175,7 +175,7 @@ enum class NeedsBigProcessLock {
    S(socket, NeedsBigProcessLock::Yes)                     \
    S(socketpair, NeedsBigProcessLock::Yes)                 \
    S(stat, NeedsBigProcessLock::No)                        \
-    S(statvfs, NeedsBigProcessLock::Yes)                    \
+    S(statvfs, NeedsBigProcessLock::No)                     \
    S(symlink, NeedsBigProcessLock::No)                     \
    S(sync, NeedsBigProcessLock::No)                        \
    S(sysconf, NeedsBigProcessLock::No)                     \
--- a/Kernel/Syscalls/statvfs.cpp
+++ b/Kernel/Syscalls/statvfs.cpp
@ -41,7 +41,7 @@ ErrorOr<FlatPtr> Process::do_statvfs(FileSystem const& fs, Custody const* custod

 ErrorOr<FlatPtr> Process::sys$statvfs(Userspace<Syscall::SC_statvfs_params const*> user_params)
 {
-    VERIFY_PROCESS_BIG_LOCK_ACQUIRED(this)
+    VERIFY_NO_PROCESS_BIG_LOCK(this)
    TRY(require_promise(Pledge::rpath));
    auto params = TRY(copy_typed_from_user(user_params));