MAJOR: config: disable support for nbproc and nbthread in parallel
When 1.8 was released, we wanted to support both nbthread and nbproc to
observe how things would go. Since then it appeared obvious that the two
are never used together because of the pain to configure affinity in this
case, and instead of bringing benefits, it brings the limitations of both
models, and causes multiple threads to compete for the same CPU. In
addition, it costs a lot to support both in parallel, so let's get rid
of this once for all.
diff --git a/src/cfgparse.c b/src/cfgparse.c
index 4c8b1fe..85d8dd5 100644
--- a/src/cfgparse.c
+++ b/src/cfgparse.c
@@ -2200,6 +2200,12 @@
if (!global.tune.requri_len)
global.tune.requri_len = REQURI_LEN;
+ if (global.nbproc > 1 && global.nbthread > 1) {
+ ha_alert("config : cannot enable multiple processes if multiple threads are configured. Please use either nbproc or nbthread but not both.\n");
+ err_code |= ERR_ALERT | ERR_FATAL;
+ goto out;
+ }
+
pool_head_requri = create_pool("requri", global.tune.requri_len , MEM_F_SHARED);
pool_head_capture = create_pool("capture", global.tune.cookie_len, MEM_F_SHARED);