system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-07-22 10:36:24 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

Kernel: Disable lock rank enforcement by default for now

Browse source 
There are a few violations with signal handling that I won't be able to
fix it until later this week. So lets put lock rank enforcement under a
debug option for now so other folks don't hit these crashes until rank
enforcement is more fleshed out.
This commit is contained in:
Brian Gianforcaro 2021-09-14 11:05:17 -07:00 committed by Brian Gianforcaro
parent 44cc6e1662
commit fbb31b4519
3 changed files with 15 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
Kernel/Debug.h.in
View file

@ -170,6 +170,10 @@
#cmakedefine01 LOCK_IN_CRITICAL_DEBUG
#endif
#ifndef LOCK_RANK_ENFORCEMENT
#cmakedefine01 LOCK_RANK_ENFORCEMENT
#endif
#ifndef LOCK_RESTORE_DEBUG
#cmakedefine01 LOCK_RESTORE_DEBUG
#endif

16
Kernel/Locking/LockRank.cpp
View file

@ -14,16 +14,20 @@ namespace Kernel {
void track_lock_acquire(LockRank rank)
{
auto thread = Thread::current();
if (thread && !thread->is_crashing())
thread->track_lock_acquire(rank);
if constexpr (LOCK_RANK_ENFORCEMENT) {
auto thread = Thread::current();
if (thread && !thread->is_crashing())
thread->track_lock_acquire(rank);
}
}
void track_lock_release(LockRank rank)
{
auto thread = Thread::current();
if (thread && !thread->is_crashing())
thread->track_lock_release(rank);
if constexpr (LOCK_RANK_ENFORCEMENT) {
auto thread = Thread::current();
if (thread && !thread->is_crashing())
thread->track_lock_release(rank);
}
}
}

1
Meta/CMake/all_the_debug_macros.cmake
View file

@ -102,6 +102,7 @@ set(LINE_EDITOR_DEBUG ON)
set(LOCAL_SOCKET_DEBUG ON)
set(LOCK_DEBUG ON)
set(LOCK_IN_CRITICAL_DEBUG ON)
set(LOCK_RANK_ENFORCEMENT ON)
set(LOCK_RESTORE_DEBUG ON)
set(LOCK_TRACE_DEBUG ON)
set(LOOKUPSERVER_DEBUG ON)