From c1dc5d947384a338d7630e035c42a76c43eeb340 Mon Sep 17 00:00:00 2001 From: John Beppu Date: Fri, 11 Feb 2000 12:43:20 +0000 Subject: [PATCH] reduced .bss size by dynmaically allocating a certain large array instead of letting it be static. objdump -t busybox \ | grep .bss \ | sed 's/^.*\.bss //' \ | grep -v ABS \ #| perl -e 'while(<>) { @x = split; @y = reverse split(//, $x[0]); for ($i=0; $i<@y; $i++) { $s += $y[$i] * (16 ** $i); if ($y[$i] && $i > 2) { print "> $y[$i] * 16 ** $i $x[1]\n"; } } } print "$s\n";' --- fsck_minix.c | 25 +++++++++++++++++++++++-- util-linux/fsck_minix.c | 25 +++++++++++++++++++++++-- 2 files changed, 46 insertions(+), 4 deletions(-) diff --git a/fsck_minix.c b/fsck_minix.c index 084c76d36..9da79748f 100644 --- a/fsck_minix.c +++ b/fsck_minix.c @@ -143,9 +143,10 @@ static struct termios termios; static int termios_set = 0; /* File-name data */ -#define MAX_DEPTH 50 +#define MAX_DEPTH 32 static int name_depth = 0; -static char name_list[MAX_DEPTH][PATH_MAX + 1]; +// static char name_list[MAX_DEPTH][PATH_MAX + 1]; +static char **name_list; static char *inode_buffer = NULL; @@ -1240,12 +1241,32 @@ static void check2(void) } #endif +/* Wed Feb 9 15:17:06 MST 2000 */ +/* dynamically allocate name_list (instead of making it static) */ +static void alloc_name_list(void) +{ + int i; + + name_list = malloc(sizeof(char *) * MAX_DEPTH); + for (i = 0; i < MAX_DEPTH; i++) { + name_list[i] = malloc(sizeof(char) * PATH_MAX + 1); + } +} + +static void free_name_list(void) +{ + if (name_list) free(name_list); +} + extern int fsck_minix_main(int argc, char **argv) { struct termios tmp; int count; int retcode = 0; + alloc_name_list(); + atexit(free_name_list); + if (argc && *argv) program_name = *argv; if (INODE_SIZE * MINIX_INODES_PER_BLOCK != BLOCK_SIZE) diff --git a/util-linux/fsck_minix.c b/util-linux/fsck_minix.c index 084c76d36..9da79748f 100644 --- a/util-linux/fsck_minix.c +++ b/util-linux/fsck_minix.c @@ -143,9 +143,10 @@ static struct termios termios; static int termios_set = 0; /* File-name data */ -#define MAX_DEPTH 50 +#define MAX_DEPTH 32 static int name_depth = 0; -static char name_list[MAX_DEPTH][PATH_MAX + 1]; +// static char name_list[MAX_DEPTH][PATH_MAX + 1]; +static char **name_list; static char *inode_buffer = NULL; @@ -1240,12 +1241,32 @@ static void check2(void) } #endif +/* Wed Feb 9 15:17:06 MST 2000 */ +/* dynamically allocate name_list (instead of making it static) */ +static void alloc_name_list(void) +{ + int i; + + name_list = malloc(sizeof(char *) * MAX_DEPTH); + for (i = 0; i < MAX_DEPTH; i++) { + name_list[i] = malloc(sizeof(char) * PATH_MAX + 1); + } +} + +static void free_name_list(void) +{ + if (name_list) free(name_list); +} + extern int fsck_minix_main(int argc, char **argv) { struct termios tmp; int count; int retcode = 0; + alloc_name_list(); + atexit(free_name_list); + if (argc && *argv) program_name = *argv; if (INODE_SIZE * MINIX_INODES_PER_BLOCK != BLOCK_SIZE)