Re: (hacky) [PATCH] silence MODPOST section mismatch warnings

[Sam Ravnborg]

On Thu, May 10, 2007 at 09:54:27PM +0100, Russell King wrote:
> On Thu, May 10, 2007 at 01:51:47PM -0700, David Miller wrote:
> > From: Chris Wedgwood <cw@xxxxxxxx>
> > Date: Thu, 10 May 2007 13:34:18 -0700
> > 
> > > MODPOST seems to be spewing bogus warnings.  It's not clear how best
> > > to fix it so perhaps we should silence it for now?
> > 
> > Most of them are legitimate, the only one that needs sorting
> > is the mm/slab.c case and people are working on that.
> > 
> > The rest are useful and I've been working to fix things up
> > on sparc64 and the networking, and in fact I'm very happy
> > about these notifications.
> > 
> > Please don't apply a sledgehammer to this issue, thanks.
> 
> I've not had one accurate one on ARM yet.

To be able to test my modpost changes I am building arm for all configs.
I can see a number of legitime warnings for the ARM code.
Those I have investigated has all been of the case where functions
are used only during init but not marked init.
So no real bugs just potential savings and consistency.
Shall I try to fix them up and drop you a path?

Samples (with old warning format):

WARNING: drivers/built-in.o - Section mismatch: reference to .init.text:pcmcia_assabet_init from .text between 'sa11x0_drv_pcmcia_probe' (at offset 0x49ed8) an
d 'assabet_pcmcia_socket_state'
WARNING: drivers/built-in.o - Section mismatch: reference to .init.text:pcmcia_collie_init from .text between 'sa11x0_drv_pcmcia_probe' (at offset 0x5f250) and
 'sharpsl_pcmcia_init_reset'
WARNING: drivers/pcmcia/sa1100_cs.o - Section mismatch: reference to .init.text:pcmcia_cerf_init from .text between 'sa11x0_drv_pcmcia_probe' (at offset 0xc) a
nd 'cerf_pcmcia_socket_state'

WARNING: arch/arm/mach-at91/built-in.o - Section mismatch: reference to .init.data: from .text after 'nand_partitions' (at offset 0xbdc)
WARNING: arch/arm/mach-at91/built-in.o - Section mismatch: reference to .init.data: from .text between 'nand_partitions' (at offset 0xbdc) and 'ads7843_pendown
_state'
WARNING: arch/arm/mach-at91/built-in.o - Section mismatch: reference to .init.data: from .text between 'nand_partitions' (at offset 0xdf4) and 'at91_leds_event
'


	Sam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/