From: Sam Ravnborg <sam@ravnborg.org>
To: Meelis Roos <mroos@linux.ee>, rth@twiddle.net, ink@jurassic.park.msu.ru
Cc: Linux Kernel list <linux-kernel@vger.kernel.org>
Subject: Re: alpha modpost warnings
Date: Wed, 6 Jun 2007 21:21:02 +0200 [thread overview]
Message-ID: <20070606192102.GA6828@uranus.ravnborg.org> (raw)
In-Reply-To: <20070605053502.GB21745@uranus.ravnborg.org>
On Tue, Jun 05, 2007 at 07:35:02AM +0200, Sam Ravnborg wrote:
> > WARNING: drivers/built-in.o(.data.rel+0x1a80): Section mismatch: reference to .init.text:serial8250_console_setup (between 'serial8250_console' and '__param_arr_probe_rsa')
I wondered why this one showed up on alpha but nowhere else.
We already whitelist (do not warn about) references to .init.text from
variables named *_console IFF the variable is placed in section .data
But from the above we can see that serial8250_console is placed in .data.rel
Now I wonder why the alpha toolchain suddenly uses a .data.rel section.
The exact commandline to build the .c file is:
alpha-unknown-linux-gnu-gcc -Wp,-MD,drivers/serial/.8250.o.d -nostdinc -isystem /opt/crosstool/gcc-4.1.0-glibc-2.3.6/alpha-unknown-linux-gnu/lib/gcc/alpha-unknown-linux-gnu/4.1.0/include -D__KERNEL__ -Iinclude -Iinclude2 -I/home/sam/kernel/kbuild.git/include -include include/linux/autoconf.h -I/home/sam/kernel/kbuild.git/drivers/serial -Idrivers/serial -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs -fno-strict-aliasing -fno-common -O2 -pipe -mno-fp-regs -ffixed-8 -msmall-data -mcpu=ev6 -Wa,-mev6 -fomit-frame-pointer -fno-stack-protector -Wdeclaration-after-statement -Wno-pointer-sign -D"KBUILD_STR(s)=#s" -D"KBUILD_BASENAME=KBUILD_STR(8250)" -D"KBUILD_MODNAME=KBUILD_STR(8250)" -c -o drivers/serial/8250.o /home/sam/kernel/kbuild.git/drivers/serial/8250.c
No use of -PIC or somethign like that.
Then I tried to reproduce it here - no luck.
I do not see the warning from serial/8250.o here?
I thought that I had all warnings reproduced but in the noise I must have
missed that I did not reproduce the 8250.o warning.
I will consider my patch for 8250.c bad and will drop it.
The others I will forward to akpm.
Anyone that can explain why alpha uses .data.rel in the above case -
but not with my toolchain?
$> gcc --version
alpha-unknown-linux-gnu-gcc (GCC) 4.1.0
Copyright (C) 2006 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
Sam
next prev parent reply other threads:[~2007-06-06 19:22 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-04 10:11 alpha modpost warnings Meelis Roos
2007-06-04 10:43 ` Sam Ravnborg
2007-06-05 5:35 ` Sam Ravnborg
2007-06-05 16:32 ` Richard Henderson
2007-06-06 13:41 ` Meelis Roos
2007-06-06 19:21 ` Sam Ravnborg [this message]
2007-06-07 20:29 ` [PATCH] alpha: fix trivial section mismatch warnings Sam Ravnborg
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20070606192102.GA6828@uranus.ravnborg.org \
--to=sam@ravnborg.org \
--cc=ink@jurassic.park.msu.ru \
--cc=linux-kernel@vger.kernel.org \
--cc=mroos@linux.ee \
--cc=rth@twiddle.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox