BUG/MINOR: pools: don't report "limited to the first X entries" by default

With the fix in commit 982805e6a3 ("BUG/MINOR: pools: Fix the dump of
pools info to deal with buffers limitations"), the max count is now
compared to the number of dumped pools instead of the configured
numbered, and keeping >= is no longer valid because maxcnt is set by
default to the same value when not set, so this means that since this
patch we're always displaying "limited to the first X entries" where X
is the number of dumped entries even in the absence of any limitation.
Let's just fix the comparison to only show this when the limit is lower.

This must be backported to 3.2 where the patch above already is.

diff --git a/src/pool.c b/src/pool.c
index 594195d10812dc120e42eb5c91a8561ce22b4007..9d247eb582bf6bf0a300cd4077fd7a1a557550c3 100644 (file)
--- a/src/pool.c
+++ b/src/pool.c
@@ -1399,7 +1399,7 @@ int dump_pools_info(struct appctx *appctx)
 
        if (ctx->pool_idx == -1) {
                chunk_printf(&trash, "Dumping pools usage");
-               if (ctx->nbpools >= ctx->maxcnt)
+               if (ctx->nbpools > ctx->maxcnt)
                        chunk_appendf(&trash, " (limited to the first %u entries)", ctx->maxcnt);
                chunk_appendf(&trash, ". Use SIGQUIT to flush them.\n");