]> git.ipfire.org Git - thirdparty/haproxy.git/commit
MINOR: pools: call malloc_trim() under thread isolation
authorWilly Tarreau <w@1wt.eu>
Thu, 10 Jun 2021 06:40:16 +0000 (08:40 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 10 Jun 2021 15:46:50 +0000 (17:46 +0200)
commit26ed183556f3df9e9bde6f6d47143aa4a18ae463
tree68f442efc015bcb0498d52d459e6c878962656b7
parentc88914379da35c46d093f6d410b9507355aacd0a
MINOR: pools: call malloc_trim() under thread isolation

pool_gc() was adjusted to run under thread isolation by commit c0e2ff202
("MEDIUM: memory: make pool_gc() run under thread isolation") so that the
underlying malloc() and free() don't compete between threads during these
potentially aggressive moments (especially when mmap/munmap are involved).

Commit 88366c292 ("MEDIUM: pools: call malloc_trim() from pool_gc()")
later added a call to malloc_trim() but made it outside of the thread
isolation, which is contrary to the principle explained above. Also it
missed it in the locked version, meaning that those without a lockless
implementation cannot benefit from trimming.

This patch fixes that by calling it before thread_release() in both
places.
src/pool.c