]> git.ipfire.org Git - thirdparty/haproxy.git/commit
MINOR: activity: make the memory profiling hash size configurable at build time
authorWilly Tarreau <w@1wt.eu>
Thu, 27 Jun 2024 15:53:18 +0000 (17:53 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 27 Jun 2024 16:01:27 +0000 (18:01 +0200)
commit290659ffd3a2eead918adc387e8842c59fbff2e7
tree688bcc1adf389e43d7145bf7026b69e42ba1b3ec
parenteec804804212374739556175f81b234d7cc8c6f0
MINOR: activity: make the memory profiling hash size configurable at build time

The MEMPROF_HASH_BITS variable was set to 10 without a possibility to
change it (beyond patching the code). After seeing a few reports already
with "other" being listed and a list with close to 1024 entries, it looks
like it's about time to either increase the hash size, or at least make
it configurable for special cases. As a reminder, in order to remain
fast, the algorithm searches no more than 16 places after the hash, so
when a table is almost full, searches are long and new places are rare.

The present patch just makes it possible to redefine it by passing
"-DMEMPROF_HASH_BITS=11" or "-DMEMPROF_HASH_BITS=12" in CFLAGS, and
moves the definition to defaults.h to make it easier to find. Such
values should be way sufficient for the vast majority of use cases.
Maybe in the future we'd change the default. At least this version
should be backported to ease rebuilds, say, till 2.8 or so.
include/haproxy/activity-t.h
include/haproxy/defaults.h