Messages in this thread | | | Date | Fri, 12 Jan 2007 16:38:52 +0000 | From | Russell King <> | Subject | Re: [patch 2.6.20-rc4-git] remove modpost false warnings on ARM |
| |
On Fri, Jan 12, 2007 at 08:31:36AM -0800, David Brownell wrote: > Index: at91/scripts/mod/modpost.c > =================================================================== > --- at91.orig/scripts/mod/modpost.c 2007-01-11 22:51:49.000000000 -0800 > +++ at91/scripts/mod/modpost.c 2007-01-12 04:20:00.000000000 -0800 > @@ -679,6 +679,26 @@ static Elf_Sym *find_elf_symbol(struct e > } > > /* > + * If there's no name there, ignore it; likewise, ignore it if it's > + * one of the magic symbols emitted used by current ARM tools. > + * > + * Otherwise if find_symbols_between() returns those symbols, they'll > + * fail the whitelist tests and cause lots of false alarms ... fixable > + * only by shrinking __exit and __init sections into __text, bloating > + * the kernel (which is especially evil on embedded platforms). > + */ > +static int is_valid_name(struct elf_info *elf, Elf_Sym *sym) > +{ > + const char *name = elf->strtab + sym->st_name; > + > + if (!name || !strlen(name)) > + return 0; > + if (strcmp(name, "$a") == 0 || strcmp(name, "$d") == 0) > + return 0;
A more correct test would be that found in kallsyms.c:
/* * This ignores the intensely annoying "mapping symbols" found * in ARM ELF files: $a, $t and $d. */ static inline int is_arm_mapping_symbol(const char *str) { return str[0] == '$' && strchr("atd", str[1]) && (str[2] == '\0' || str[2] == '.'); }
Suggest that code is re-used here (as well as in other tools such as oprofile, readprofile, etc.)
-- Russell King Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/ maintainer of: - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |