linux-kernel - Re: [patch] flex_array: export symbols to modules

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 2 Dec 2010 15:56:42 +0900
From:	Paul Mundt <lethal@...ux-sh.org>
To:	Am??rico Wang <xiyou.wangcong@...il.com>
Cc:	David Rientjes <rientjes@...gle.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Alex Bennee <kernel-hacker@...nee.com>,
	Dave Hansen <dave@...ux.vnet.ibm.com>,
	linux-kernel@...r.kernel.org
Subject: Re: [patch] flex_array: export symbols to modules

On Thu, Dec 02, 2010 at 02:57:35PM +0800, Am??rico Wang wrote:
> On Thu, Dec 02, 2010 at 03:30:55PM +0900, Paul Mundt wrote:
> >On Wed, Dec 01, 2010 at 05:16:05PM -0800, David Rientjes wrote:
> >> On Wed, 1 Dec 2010, Alex Bennee wrote:
> >> > I want to be able to use the flex_array functions in my module so I
> >> > wanted to EXPORT_SYMBOL the API.
> >> > 
> >> > I added the appropriate EXPORT_SYMBOL runes into lib/flex_array.c and
> >> > added externs in the header files but after re-building there is still
> >> > no sign of them in Module.symvers.
> >> 
> >> You shouldn't need to add any externs in include/linux/flex_array.h, they 
> >> should already be publically declared.
> >> 
> >> Perhaps you forgot to include linux/module.h in lib/flex_array.c?
> >> 
> >> flex_array: export symbols to modules
> >> 
> >> Add EXPORT_SYMBOL() for all publically accessible flex array functions.
> >> 
> >> Cc: Dave Hansen <dave@...ux.vnet.ibm.com>
> >> Signed-off-by: David Rientjes <rientjes@...gle.com>
> >> ---
> >>  lib/flex_array.c |   10 ++++++++++
> >>  1 files changed, 10 insertions(+), 0 deletions(-)
> >> 
> >This will be ineffective so long as flex_array.o remains lib-y. It will
> >need to be moved to obj-y if modules are to also make use of it.
> 
> Not moved, it needs to be added to obj-y.

Wrong, try again. Unless something has recently changed, an object can
not be both lib-y and obj-y. The object needs to be moved for modules to
make use of it, period.
--
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/