[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20090819001424.4C1144730F@magilla.sf.frob.com>
Date: Tue, 18 Aug 2009 17:14:24 -0700 (PDT)
From: Roland McGrath <roland@...hat.com>
To: James Bottomley <James.Bottomley@...senPartnership.com>
Cc: Rusty Russell <rusty@...tcorp.com.au>,
Helge Deller <deller@....de>,
linux-parisc <linux-parisc@...r.kernel.org>,
linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: kernel segv with 2.6.31-rc6 ?
> Actually, for parisc, its not reasonable. It's expected that our modules
> have multiple text sections (we have to use -ffunction-sections to
> generate them in order that the PCREL17 jump stubs can be interleaved).
I don't think you need what you think you need. Having lots of sections in
your .o's when you compile is fine. These should be combined by the linker
script that creates the .ko, however. Unless I am missing something, there
is no purpose to this section distinction at insmod time--it's only
important for the relative layout of the parts of the .ko's text, which
winds up contiguous whether laid out that way at ld -r (.ko creation) time
or at insmod time.
> Even with the duplicate name, though, the module should be perfectly
> loadable.
But its /sys/module/foo/sections/ virtual directory becomes useless,
as a single name space can no longer describe what sections it has.
So perhaps it is then proper for add_sect_attrs() to punt on it.
But that reduces the functionality you get from CONFIG_KALLSYMS.
Thanks,
Roland
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists