diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2022-06-12 16:52:57 +0200 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2022-06-29 16:51:52 +0200 |
commit | b01f31954f1c7c4601925173ae2638b572224e9a (patch) | |
tree | 1eb4c844727641e5f2682539f9f127581b4b82df /src/shared/enable-mempool.c | |
parent | f63d1b0efa64fe716c2855a0410ac47ad67f7dec (diff) | |
download | systemd-b01f31954f1c7c4601925173ae2638b572224e9a.tar.gz |
Turn mempool_enabled() into a weak symbol
Before we had the following scheme:
mempool_enabled() would check mempool_use_allowed, and
libsystemd-shared would be linked with a .c file that provides mempool_use_allowed=true,
while other things would linked with a different .c file with mempool_use_allowed=false.
In the new scheme, mempool_enabled() itself is a weak symbol. If it's
not found, we assume false. So it only needs to be provided for libsystemd-shared,
where it can return false or true.
test-set-disable-mempool is libshared, so it gets the symbol. But then we
actually disable the mempool via envvar. mempool_enable() is called to check
its return value directly.
Diffstat (limited to 'src/shared/enable-mempool.c')
-rw-r--r-- | src/shared/enable-mempool.c | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/src/shared/enable-mempool.c b/src/shared/enable-mempool.c index 1abfccbd81..fd582c0e78 100644 --- a/src/shared/enable-mempool.c +++ b/src/shared/enable-mempool.c @@ -1,5 +1,19 @@ /* SPDX-License-Identifier: LGPL-2.1-or-later */ +#include <stdbool.h> + +#include "env-util.h" #include "mempool.h" +#include "process-util.h" + +bool mempool_enabled(void) { + static int cache = -1; + + if (!is_main_thread()) + return false; + + if (cache < 0) + cache = getenv_bool("SYSTEMD_MEMPOOL") != 0; -const bool mempool_use_allowed = true; + return cache; +} |