diff options
author | Brian Gianforcaro <bgianf@serenityos.org> | 2021-05-31 02:38:50 -0700 |
---|---|---|
committer | Linus Groh <mail@linusgroh.de> | 2021-05-31 11:30:47 +0100 |
commit | 26cb64573c75bab5f9d049fc6bd841d24670624b (patch) | |
tree | 68b5d4fa0fbb9211a6709b278d78e9c0ffe069f4 /CMakeLists.txt | |
parent | d838a02e74ffd6af59ce7c27971fb193491a492c (diff) | |
download | serenity-26cb64573c75bab5f9d049fc6bd841d24670624b.zip |
CMake: Hide KMALLOC_VERIFY_NO_SPINLOCK_HELD so folks don't find it
Since I introduced this functionality there has been a steady stream of
people building with `ALL_THE_DEBUG_MACROS` and trying to boot the
system, and immediately hitting this assert. I have no idea why people
try to build with all the debugging enabled, but I'm tired of seeing the
bug reports about asserts we know are going to happen at this point.
So I'm hiding this value under the new ENABLE_ALL_DEBUG_FACILITIES flag
instead. This is only set by CI, and hopefully no-one will try to build
with this thing (It's documented as not recommended).
Fixes: #7527
Diffstat (limited to 'CMakeLists.txt')
-rw-r--r-- | CMakeLists.txt | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/CMakeLists.txt b/CMakeLists.txt index 3c5b4abb28..ac9fe49211 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -102,6 +102,12 @@ endif() if (ENABLE_ALL_DEBUG_FACILITIES) set(ENABLE_ALL_THE_DEBUG_MACROS ON) set(ENABLE_EXTRA_KERNEL_DEBUG_SYMBOLS ON) + + # Immediately finds violations during boot, shouldn't be discoverable + # by people who aren't working on fixing issues. Use this check to make + # sure this code continues to build instead of all_debug_macros to avoid + # people filing bugs. + set(KMALLOC_VERIFY_NO_SPINLOCK_HELD ON) endif() if (ENABLE_ALL_THE_DEBUG_MACROS) |