Messages in this thread Patch in this message |  | | | From | Keith Owens <> | | Subject | Re: [PATCH] All Symbols in /proc/kallsyms | | Date | Mon, 10 May 2004 16:05:32 +1000 |
On Mon, 10 May 2004 15:28:37 +1000, Rusty Russell <rusty@rustcorp.com.au> wrote: >Debuggers (ie. xmon) can use kallsyms, but they want all symbols, not >just functions. Fair enough. > >Name: Debugging Option to Put text symbols in kallsyms >Status: Tested on 2.6.6-rc3-bk11 > >kallsyms contains only function names, but some debuggers (eg. xmon on >PPC/PPC64) use it to lookup symbols: it'd be much nicer if it included >data symbols too.
The patch needs some more work. You are including absolute symbols that are outside the kernel text and data space, which can be very misleading. For debugging, you want all symbols, even if they have the same address as the previous symbol.
The kdb 2.6.6 patch for scripts/kallsyms follows. I cannot remember why I excluded kallsyms_addresses, kallsyms_num_syms and kallsyms_names, there must have been a good reason at the time.
Index: linux/scripts/kallsyms.c --- linux.orig/scripts/kallsyms.c Mon May 10 12:33:21 2004 +++ linux/scripts/kallsyms.c Mon May 10 14:07:41 2004 @@ -6,6 +6,7 @@ * of the GNU General Public License, incorporated herein by reference. * * Usage: nm -n vmlinux | scripts/kallsyms > symbols.S + * If CONFIG_KDB is defined, generate all symbols for kdb. */ #include <stdio.h> @@ -21,7 +22,12 @@ struct sym_entry { static struct sym_entry *table; static int size, cnt; -static unsigned long long _stext, _etext, _sinittext, _einittext; +static unsigned long long _stext, _etext, _sinittext, _einittext, _end; +#ifdef CONFIG_KDB +#define kdb 1 +#else +#define kdb 0 +#endif static void usage(void) @@ -51,12 +57,18 @@ read_symbol(FILE *in, struct sym_entry * static int symbol_valid(struct sym_entry *s) { - if ((s->addr < _stext || s->addr > _etext) + if ((s->addr < _stext || (kdb && s->addr > _end) || (!kdb && s->addr > _etext)) && (s->addr < _sinittext || s->addr > _einittext)) return 0; if (strstr(s->sym, "_compiled.")) return 0; + if (kdb) { + if (strcmp(s->sym, "kallsyms_addresses") == 0 || + strcmp(s->sym, "kallsyms_num_syms") == 0 || + strcmp(s->sym, "kallsyms_names") == 0) + return 0; + } return 1; } @@ -87,6 +99,8 @@ read_map(FILE *in) _sinittext = table[i].addr; if (strcmp(table[i].sym, "_einittext") == 0) _einittext = table[i].addr; + if (kdb && strcmp(table[i].sym, "_end") == 0) + _end = table[i].addr; } } @@ -115,7 +129,7 @@ write_src(void) if (!symbol_valid(&table[i])) continue; - if (table[i].addr == last_addr) + if (table[i].addr == last_addr && !kdb) continue; printf("\tPTR\t%#llx\n", table[i].addr); @@ -140,7 +154,7 @@ write_src(void) if (!symbol_valid(&table[i])) continue; - if (table[i].addr == last_addr) + if (table[i].addr == last_addr && !kdb) continue; for (k = 0; table[i].sym[k] && table[i].sym[k] == prev[k]; ++k) - 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/
|  |