tree 1048c36448ebe8da7f0f087e553f6cdf8ddf1788
parent 636efb39bab076adccb2d78ed95805b664514877
author Willy Tarreau <w@1wt.eu> 1623313295 +0200
committer Willy Tarreau <w@1wt.eu> 1625752942 +0200
encoding latin1

MEDIUM: pools: use a single pool_gc() function for locked and lockless

Locked and lockless shared pools don't need to use a different pool_gc()
function because this function isolates itself during the operation, so
we do not need to rely on DWCAS nor any atomic operation in fact. Let's
just get rid of the lockless one in favor of the simple one. This should
even result in a faster execution.

The ifdefs were slightly moved so that we can have pool_gc() defined
as soon as there are global pools, this avoids duplicating the function.

(cherry picked from commit 9b3ed51371693f70bab07e09cdd34e26e3a46d93)
Signed-off-by: Willy Tarreau <w@1wt.eu>
(cherry picked from commit a206cf93eb17e2d0f8184b91a0775ada763e7f71)
[wt: for 2.3, this was simplified, only the lockless version was
 reworked to work without the DWCAS, very similarly to the one in
 2.4 except that accounting still has to be done]
Signed-off-by: Willy Tarreau <w@1wt.eu>
(cherry picked from commit 294da644124403050eb1c4dd0aa1b7eb06372d9d)
Signed-off-by: Willy Tarreau <w@1wt.eu>
(cherry picked from commit 629c3683e2f774457ebedbff2c48c3e66331474b)
[wt: a number of previous optimizations such as 3e853ea74 ("MINOR:
     pools: release the pool's lock during the malloc/free calls"
     were not part of 2.0), hence some differences in the patch and
     its context, though the result is the same]
Signed-off-by: Willy Tarreau <w@1wt.eu>