From 9f699958dc668a2a9c505ffa6b6366acaad4c234 Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Fri, 18 Feb 2022 18:31:53 +0100 Subject: [PATCH] MINOR: pools: mark most static pool configuration variables as read-mostly The mem_poison_byte, mem_fail_rate, using_default_allocator and the pools list are all only set once at boot time and never changed later, while they're heavily used at run time. Let's optimize their usage from all threads by marking them read-mostly so that them reside in a shared cache line. --- src/pool.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/pool.c b/src/pool.c index cc92245c7..60be27d0d 100644 --- a/src/pool.c +++ b/src/pool.c @@ -35,14 +35,14 @@ THREAD_LOCAL size_t pool_cache_bytes = 0; /* total cache size */ THREAD_LOCAL size_t pool_cache_count = 0; /* #cache objects */ #endif -static struct list pools = LIST_HEAD_INIT(pools); -int mem_poison_byte = -1; +static struct list pools __read_mostly = LIST_HEAD_INIT(pools); +int mem_poison_byte __read_mostly = -1; #ifdef DEBUG_FAIL_ALLOC -static int mem_fail_rate = 0; +static int mem_fail_rate __read_mostly = 0; #endif -static int using_default_allocator = 1; +static int using_default_allocator __read_mostly = 1; static int(*my_mallctl)(const char *, void *, size_t *, void *, size_t) = NULL; /* ask the allocator to trim memory pools.