BUG/MINOR: threads: Initialize maxthrpertgroup earlier.

Give global.maxthrpertgroup its default value at global creation,
instead of later when we're trying to detect the thread count.
It is used when verifying the configuration file validity, and if it was
not set in the config file, in a few corner cases, the value of 0 would
be used, which would then reject perfectly fine configuration files.

This should be backported to 3.3.
This commit is contained in:
Olivier Houchard 2026-02-06 02:49:26 +01:00
parent 9e023ae930
commit 9766211cf0
2 changed files with 1 additions and 3 deletions

View File

@ -205,6 +205,7 @@ struct global global = {
#endif
/* by default allow clients which use a privileged port for TCP only */
.clt_privileged_ports = HA_PROTO_TCP,
.maxthrpertgroup = MAX_THREADS_PER_GROUP,
/* others NULL OK */
};

View File

@ -1641,9 +1641,6 @@ void thread_detect_count(void)
if (global.nbtgroups)
grp_min = grp_max = global.nbtgroups;
if (!global.maxthrpertgroup)
global.maxthrpertgroup = MAX_THREADS_PER_GROUP;
#if defined(USE_THREAD)
/* Adjust to boot settings if not forced */
if (thr_min <= thread_cpus_enabled_at_boot && thread_cpus_enabled_at_boot < thr_max)