Kernel: Disable big process lock for sys$sync

This syscall doesn't touch any intra-process shared resources and only calls VirtualFileSystem::sync, which is self-locking.
Author: https://github.com/IdanHo Commit: https://github.com/SerenityOS/serenity/commit/9d21c796712 Pull-request: https://github.com/SerenityOS/serenity/pull/9264
2021-08-07 16:06:07 +03:00 · 2021-08-07 16:06:07 +03:00 · 9d21c79671 · 2024-07-18 07:18:48 +09:00
commit 9d21c79671
parent 0d468f2282
2 changed files with 2 additions and 2 deletions
--- a/Kernel/API/Syscall.h
+++ b/Kernel/API/Syscall.h
@ -106,7 +106,7 @@ enum class NeedsBigProcessLock {
    S(mkdir, NeedsBigProcessLock::Yes)                      \
    S(times, NeedsBigProcessLock::Yes)                      \
    S(utime, NeedsBigProcessLock::Yes)                      \
-    S(sync, NeedsBigProcessLock::Yes)                       \
+    S(sync, NeedsBigProcessLock::No)                        \
    S(ptsname, NeedsBigProcessLock::Yes)                    \
    S(select, NeedsBigProcessLock::Yes)                     \
    S(unlink, NeedsBigProcessLock::Yes)                     \
--- a/Kernel/Syscalls/sync.cpp
+++ b/Kernel/Syscalls/sync.cpp
@ -11,7 +11,7 @@ namespace Kernel {

 KResultOr<FlatPtr> Process::sys$sync()
 {
-    VERIFY_PROCESS_BIG_LOCK_ACQUIRED(this)
+    VERIFY_NO_PROCESS_BIG_LOCK(this)
    REQUIRE_PROMISE(stdio);
    VirtualFileSystem::sync();
    return 0;