summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorDenys Vlasenko <vda.linux@googlemail.com>2018-01-10 13:32:20 +0100
committerDenys Vlasenko <vda.linux@googlemail.com>2018-01-10 13:32:20 +0100
commitf2539c78d2e507150dea06e8702c5ab8713d2b49 (patch)
treefade1ab37811045bff900534dc75f4be77664cc9
parentbaa41c785551ae0580526298aa6fadf4534fc8c0 (diff)
init: stop using static data
function old new delta init_action_list 4 - -4 new_init_action 148 142 -6 ------------------------------------------------------------------------------ (add/remove: 0/1 grow/shrink: 0/1 up/down: 0/-10) Total: -10 bytes text data bss dec hex filename 927839 481 6832 935152 e44f0 busybox_old 927833 481 6824 935138 e44e2 busybox_unstripped Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r--init/init.c40
1 files changed, 24 insertions, 16 deletions
diff --git a/init/init.c b/init/init.c
index 6f3374eac..cac165fc7 100644
--- a/init/init.c
+++ b/init/init.c
@@ -128,6 +128,7 @@
#define DEBUG_SEGV_HANDLER 0
#include "libbb.h"
+#include "common_bufsiz.h"
#include <syslog.h>
#include <sys/resource.h>
#ifdef __linux__
@@ -203,7 +204,6 @@
*/
#define RESTART 0x80
-
/* A linked list of init_actions, to be read from inittab */
struct init_action {
struct init_action *next;
@@ -213,11 +213,17 @@ struct init_action {
char command[1];
};
-static struct init_action *init_action_list = NULL;
-
+struct globals {
+ struct init_action *init_action_list;
#if !ENABLE_FEATURE_INIT_SYSLOG
-static const char *log_console = VC_5;
+ const char *log_console;
#endif
+} FIX_ALIASING;
+#define G (*(struct globals*)bb_common_bufsiz1)
+#define INIT_G() do { \
+ setup_common_bufsiz(); \
+ IF_NOT_FEATURE_INIT_SYSLOG(G.log_console = VC_5;) \
+} while (0)
enum {
L_LOG = 0x1,
@@ -265,10 +271,10 @@ static void message(int where, const char *fmt, ...)
if (log_fd < 0) {
log_fd = STDERR_FILENO;
- if (log_console) {
- log_fd = device_open(log_console, O_WRONLY | O_NONBLOCK | O_NOCTTY);
+ if (G.log_console) {
+ log_fd = device_open(G.log_console, O_WRONLY | O_NONBLOCK | O_NOCTTY);
if (log_fd < 0) {
- bb_error_msg("can't log to %s", log_console);
+ bb_error_msg("can't log to %s", G.log_console);
where = L_CONSOLE;
} else {
close_on_exec_on(log_fd);
@@ -328,7 +334,7 @@ static void console_init(void)
if (!s || strcmp(s, "linux") == 0)
putenv((char*)"TERM=vt102");
# if !ENABLE_FEATURE_INIT_SYSLOG
- log_console = NULL;
+ G.log_console = NULL;
# endif
} else
#endif
@@ -562,7 +568,7 @@ static struct init_action *mark_terminated(pid_t pid)
if (pid > 0) {
update_utmp_DEAD_PROCESS(pid);
- for (a = init_action_list; a; a = a->next) {
+ for (a = G.init_action_list; a; a = a->next) {
if (a->pid == pid) {
a->pid = 0;
return a;
@@ -596,7 +602,7 @@ static void run_actions(int action_type)
{
struct init_action *a;
- for (a = init_action_list; a; a = a->next) {
+ for (a = G.init_action_list; a; a = a->next) {
if (!(a->action_type & action_type))
continue;
@@ -630,7 +636,7 @@ static void new_init_action(uint8_t action_type, const char *command, const char
* To achieve that, if we find a matching entry, we move it
* to the end.
*/
- nextp = &init_action_list;
+ nextp = &G.init_action_list;
while ((a = *nextp) != NULL) {
/* Don't enter action if it's already in the list.
* This prevents losing running RESPAWNs.
@@ -845,7 +851,7 @@ static void exec_restart_action(void)
{
struct init_action *a;
- for (a = init_action_list; a; a = a->next) {
+ for (a = G.init_action_list; a; a = a->next) {
if (!(a->action_type & RESTART))
continue;
@@ -923,7 +929,7 @@ static void reload_inittab(void)
message(L_LOG, "reloading /etc/inittab");
/* Disable old entries */
- for (a = init_action_list; a; a = a->next)
+ for (a = G.init_action_list; a; a = a->next)
a->action_type = 0;
/* Append new entries, or modify existing entries
@@ -936,14 +942,14 @@ static void reload_inittab(void)
#if ENABLE_FEATURE_KILL_REMOVED
/* Kill stale entries */
/* Be nice and send SIGTERM first */
- for (a = init_action_list; a; a = a->next)
+ for (a = G.init_action_list; a; a = a->next)
if (a->action_type == 0 && a->pid != 0)
kill(a->pid, SIGTERM);
if (CONFIG_FEATURE_KILL_DELAY) {
/* NB: parent will wait in NOMMU case */
if ((BB_MMU ? fork() : vfork()) == 0) { /* child */
sleep(CONFIG_FEATURE_KILL_DELAY);
- for (a = init_action_list; a; a = a->next)
+ for (a = G.init_action_list; a; a = a->next)
if (a->action_type == 0 && a->pid != 0)
kill(a->pid, SIGKILL);
_exit(EXIT_SUCCESS);
@@ -955,7 +961,7 @@ static void reload_inittab(void)
* We never rerun SYSINIT entries anyway,
* removing them too saves a few bytes
*/
- nextp = &init_action_list;
+ nextp = &G.init_action_list;
while ((a = *nextp) != NULL) {
/*
* Why pid == 0 check?
@@ -1046,6 +1052,8 @@ static void sleep_much(void)
int init_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
int init_main(int argc UNUSED_PARAM, char **argv)
{
+ INIT_G();
+
if (argv[1] && strcmp(argv[1], "-q") == 0) {
return kill(1, SIGHUP);
}