From 49325969e75dd85366d9a149625f75c323163723 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Sat, 31 Jan 2009 23:33:54 +0000 Subject: [PATCH] modutils-24: code shrink function old new delta helper_create_alloced_section - 117 +117 print_load_map 474 450 -24 bb_init_module_24 4615 4545 -70 obj_create_alloced_section 131 30 -101 ------------------------------------------------------------------------------ (add/remove: 1/0 grow/shrink: 0/3 up/down: 117/-195) Total: -78 bytes --- modutils/modutils-24.c | 67 +++++++++++++++++++----------------------- 1 file changed, 30 insertions(+), 37 deletions(-) diff --git a/modutils/modutils-24.c b/modutils/modutils-24.c index 4e112a06f..451975ab9 100644 --- a/modutils/modutils-24.c +++ b/modutils/modutils-24.c @@ -2135,7 +2135,7 @@ obj_insert_section_load_order(struct obj_file *f, struct obj_section *sec) *p = sec; } -static struct obj_section *obj_create_alloced_section(struct obj_file *f, +static struct obj_section *helper_create_alloced_section(struct obj_file *f, const char *name, unsigned long align, unsigned long size) @@ -2155,31 +2155,29 @@ static struct obj_section *obj_create_alloced_section(struct obj_file *f, if (size) sec->contents = xzalloc(size); - obj_insert_section_load_order(f, sec); - return sec; } -static struct obj_section *obj_create_alloced_section_first(struct obj_file *f, +static struct obj_section *obj_create_alloced_section(struct obj_file *f, const char *name, unsigned long align, unsigned long size) { - int newidx = f->header.e_shnum++; struct obj_section *sec; - f->sections = xrealloc_vector(f->sections, 2, newidx); - f->sections[newidx] = sec = arch_new_section(); + sec = helper_create_alloced_section(f, name, align, size); + obj_insert_section_load_order(f, sec); + return sec; +} - sec->header.sh_type = SHT_PROGBITS; - sec->header.sh_flags = SHF_WRITE | SHF_ALLOC; - sec->header.sh_size = size; - sec->header.sh_addralign = align; - sec->name = name; - sec->idx = newidx; - if (size) - sec->contents = xzalloc(size); +static struct obj_section *obj_create_alloced_section_first(struct obj_file *f, + const char *name, + unsigned long align, + unsigned long size) +{ + struct obj_section *sec; + sec = helper_create_alloced_section(f, name, align, size); sec->load_next = f->load_order; f->load_order = sec; if (f->load_order_search_start == &f->load_order) @@ -2202,15 +2200,16 @@ static void *obj_extend_section(struct obj_section *sec, unsigned long more) /* Conditionally add the symbols from the given symbol set to the new module. */ -static int -add_symbols_from( struct obj_file *f, - int idx, struct new_module_symbol *syms, size_t nsyms) +static int add_symbols_from(struct obj_file *f, + int idx, + struct new_module_symbol *syms, + size_t nsyms) { struct new_module_symbol *s; size_t i; int used = 0; #ifdef SYMBOL_PREFIX - char *name_buf = 0; + char *name_buf = NULL; size_t name_alloced_size = 0; #endif #if ENABLE_FEATURE_CHECK_TAINTED_MODULE @@ -2900,7 +2899,7 @@ static void obj_check_undefineds(struct obj_file *f) for (i = 0; i < HASH_BUCKETS; ++i) { struct obj_symbol *sym; - for (sym = f->symtab[i]; sym; sym = sym->next) + for (sym = f->symtab[i]; sym; sym = sym->next) { if (sym->secidx == SHN_UNDEF) { if (ELF_ST_BIND(sym->info) == STB_WEAK) { sym->secidx = SHN_ABS; @@ -2910,6 +2909,7 @@ static void obj_check_undefineds(struct obj_file *f) bb_error_msg_and_die("unresolved symbol %s", sym->name); } } + } } } @@ -3074,7 +3074,7 @@ static int obj_relocate(struct obj_file *f, ElfW(Addr) base) ElfW(Addr) value = 0; struct obj_symbol *intsym = NULL; unsigned long symndx; - ElfW(Sym) * extsym = 0; + ElfW(Sym) *extsym = NULL; const char *errmsg; /* Attempt to find a value to use for this relocation. */ @@ -3634,7 +3634,8 @@ add_ksymoops_symbols(struct obj_file *f, const char *filename, version = get_module_version(f, str); /* -1 if not found */ name = xasprintf("%s%s_O%s_M%0*lX_V%d", symprefix, m_name, absolute_filename, - (int)(2 * sizeof(statbuf.st_mtime)), statbuf.st_mtime, + (int)(2 * sizeof(statbuf.st_mtime)), + (long)statbuf.st_mtime, version); sym = obj_add_symbol(f, name, -1, ELF_ST_INFO(STB_GLOBAL, STT_NOTYPE), @@ -3645,7 +3646,6 @@ add_ksymoops_symbols(struct obj_file *f, const char *filename, free(absolute_filename); #ifdef _NOT_SUPPORTED_ /* record where the persistent data is going, same address as previous symbol */ - if (f->persist) { name = xasprintf("%s%s_P%s", symprefix, m_name, f->persist); @@ -3656,7 +3656,6 @@ add_ksymoops_symbols(struct obj_file *f, const char *filename, } #endif /* tag the desired sections if size is non-zero */ - for (i = 0; i < ARRAY_SIZE(section_names); ++i) { sec = obj_find_section(f, section_names[i]); if (sec && sec->header.sh_size) { @@ -3678,11 +3677,11 @@ static void print_load_map(struct obj_file *f) struct obj_section *sec; #if ENABLE_FEATURE_INSMOD_LOAD_MAP_FULL struct obj_symbol **all, **p; - int i, nsyms, *loaded; + int i, nsyms; + char *loaded; /* array of booleans */ struct obj_symbol *sym; #endif /* Report on the section layout. */ - printf("Sections: Size %-*s Align\n", (int) (2 * sizeof(void *)), "Address"); @@ -3704,14 +3703,12 @@ static void print_load_map(struct obj_file *f) } #if ENABLE_FEATURE_INSMOD_LOAD_MAP_FULL /* Quick reference which section indices are loaded. */ - i = f->header.e_shnum; - loaded = alloca(sizeof(int) * i); + loaded = alloca(i * sizeof(loaded[0])); while (--i >= 0) loaded[i] = ((f->sections[i]->header.sh_flags & SHF_ALLOC) != 0); /* Collect the symbols we'll be listing. */ - for (nsyms = i = 0; i < HASH_BUCKETS; ++i) for (sym = f->symtab[i]; sym; sym = sym->next) if (sym->secidx <= SHN_HIRESERVE @@ -3720,7 +3717,7 @@ static void print_load_map(struct obj_file *f) ++nsyms; } - all = alloca(nsyms * sizeof(struct obj_symbol *)); + all = alloca(nsyms * sizeof(all[0])); for (i = 0, p = all; i < HASH_BUCKETS; ++i) for (sym = f->symtab[i]; sym; sym = sym->next) @@ -3760,7 +3757,7 @@ static void print_load_map(struct obj_file *f) } if (ELF_ST_BIND(sym->info) == STB_LOCAL) - type = tolower(type); + type |= 0x20; /* tolower. safe for '?' too */ printf("%0*lx %c %s\n", (int) (2 * sizeof(void *)), value, type, sym->name); @@ -3799,10 +3796,7 @@ int FAST_FUNC bb_init_module_24(const char *m_filename, const char *options) f = obj_load(fp, LOADBITS); - if (get_modinfo_value(f, "kernel_version") == NULL) - m_has_modinfo = 0; - else - m_has_modinfo = 1; + m_has_modinfo = (get_modinfo_value(f, "kernel_version") != NULL); #if ENABLE_FEATURE_INSMOD_VERSION_CHECKING /* Version correspondence? */ @@ -3846,13 +3840,12 @@ int FAST_FUNC bb_init_module_24(const char *m_filename, const char *options) add_kernel_symbols(f); /* Allocate common symbols, symbol tables, and string tables. */ - new_create_this_module(f, m_name); obj_check_undefineds(f); obj_allocate_commons(f); check_tainted_module(f, m_name); - /* done with the module name, on to the optional var=value arguments */ + /* Done with the module name, on to the optional var=value arguments */ new_process_module_arguments(f, options); arch_create_got(f); -- 2.25.1