From 15f9ac3c59e9bef455b75ab97020f41155c4ea7a Mon Sep 17 00:00:00 2001
From: Willy Tarreau <w@1wt.eu>
Date: Sat, 8 May 2021 12:30:50 +0200
Subject: [PATCH] REORG: mworker: move proc_self from global to mworker

Only mworker uses proc_self, and it was declared in global.h, forcing
users of global.h to include mworker and its dependencies.

Moving it to mworker reduces the preprocessed size of version.c from
170 to 125kB by shrinking the number of local includes from 30 to 16
and the number of system includes from 147 to 132.
---
 include/haproxy/cli.h     | 1 +
 include/haproxy/global.h  | 2 --
 include/haproxy/mworker.h | 2 ++
 src/haproxy.c             | 2 --
 src/mworker.c             | 1 +
 5 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/haproxy/cli.h b/include/haproxy/cli.h
index 0a4433b69..31020430e 100644
--- a/include/haproxy/cli.h
+++ b/include/haproxy/cli.h
@@ -27,6 +27,7 @@
 #include <haproxy/channel-t.h>
 #include <haproxy/cli-t.h>
 #include <haproxy/global.h>
+#include <haproxy/mworker-t.h>
 #include <haproxy/stream-t.h>
 
 
diff --git a/include/haproxy/global.h b/include/haproxy/global.h
index 547a2b8ce..a095a5268 100644
--- a/include/haproxy/global.h
+++ b/include/haproxy/global.h
@@ -24,7 +24,6 @@
 
 #include <haproxy/api-t.h>
 #include <haproxy/global-t.h>
-#include <haproxy/mworker-t.h>
 
 extern const char *build_features;
 extern struct global global;
@@ -49,7 +48,6 @@ extern char *localpeer;
 extern unsigned int warned;     /* bitfield of a few warnings to emit just once */
 extern volatile unsigned long sleeping_thread_mask;
 extern struct list proc_list; /* list of process in mworker mode */
-extern struct mworker_proc *proc_self; /* process structure of current process */
 extern int master; /* 1 if in master, 0 otherwise */
 extern unsigned int rlim_fd_cur_at_boot;
 extern unsigned int rlim_fd_max_at_boot;
diff --git a/include/haproxy/mworker.h b/include/haproxy/mworker.h
index edc612ec3..434077c0c 100644
--- a/include/haproxy/mworker.h
+++ b/include/haproxy/mworker.h
@@ -17,6 +17,8 @@
 #include <haproxy/mworker-t.h>
 #include <haproxy/signal-t.h>
 
+extern struct mworker_proc *proc_self;
+
 void mworker_proc_list_to_env();
 void mworker_env_to_proc_list();
 
diff --git a/src/haproxy.c b/src/haproxy.c
index 3de53bfce..41715c105 100644
--- a/src/haproxy.c
+++ b/src/haproxy.c
@@ -252,8 +252,6 @@ unsigned int rlim_fd_max_at_boot = 0;
 /* per-boot randomness */
 unsigned char boot_seed[20];        /* per-boot random seed (160 bits initially) */
 
-struct mworker_proc *proc_self = NULL;
-
 static void *run_thread_poll_loop(void *data);
 
 /* bitfield of a few warnings to emit just once (WARN_*) */
diff --git a/src/mworker.c b/src/mworker.c
index e3c579b57..b69c1adcb 100644
--- a/src/mworker.c
+++ b/src/mworker.c
@@ -42,6 +42,7 @@
 
 static int exitcode = -1;
 static int max_reloads = -1; /* number max of reloads a worker can have until they are killed */
+struct mworker_proc *proc_self = NULL; /* process structure of current process */
 
 /* ----- children processes handling ----- */