From: Vivek Goyal On Mon, Nov 06, 2006 at 03:48:57PM +0100, Haavard Skinnemoen wrote: > On Mon, 23 Oct 2006 15:37:48 -0400 > Vivek Goyal wrote: > > > Add the _text symbol definitions to the architectures which don't > > define it otherwise linker will fail. > > I get lots of this when building latest -mm for avr32: > > .tmp_kallsyms2.S:643: Warning: right operand is a bignum; integer 0 > assumed > > Reverting this patch makes the warnings go away. I think it's > because on avr32, .init is the first section, not .text, so many of the > offsets become negative and kallsyms doesn't seem to handle this very > well. Here's a few lines from .tmp_kallsyms2.S: > > kallsyms_addresses: > PTR _text + 0xffffffffffff4000 > PTR _text + 0xffffffffffff4000 > o On some platforms like avr32, section init comes before .text and not necessarily a symbol's relative position w.r.t _text is positive. In such cases assembler detects the overflow and emits warning. This patch fixes it. Signed-off-by: Vivek Goyal Cc: Andi Kleen Cc: Haavard Skinnemoen Signed-off-by: Andrew Morton --- scripts/kallsyms.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff -puN scripts/kallsyms.c~fix-x86_64-mm-i386-reloc-kallsyms scripts/kallsyms.c --- a/scripts/kallsyms.c~fix-x86_64-mm-i386-reloc-kallsyms +++ a/scripts/kallsyms.c @@ -277,8 +277,12 @@ static void write_src(void) output_label("kallsyms_addresses"); for (i = 0; i < table_cnt; i++) { if (toupper(table[i].sym[0]) != 'A') { - printf("\tPTR\t_text + %#llx\n", - table[i].addr - _text); + if (_text <= table[i].addr) + printf("\tPTR\t_text + %#llx\n", + table[i].addr - _text); + else + printf("\tPTR\t_text - %#llx\n", + _text - table[i].addr); } else { printf("\tPTR\t%#llx\n", table[i].addr); } _