linux-kernel - Re: [Patch] Fix section mismatch of memory hotplug related code.

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]

Message-ID: <20070506175326.GA9855@uranus.ravnborg.org>
Date:	Sun, 6 May 2007 19:53:26 +0200
From:	Sam Ravnborg <sam@...nborg.org>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	Yasunori Goto <y-goto@...fujitsu.com>,
	Linux Kernel ML <linux-kernel@...r.kernel.org>,
	Miles Lane <miles.lane@...il.com>
Subject: Re: [Patch] Fix section mismatch of memory hotplug related code.

On Fri, May 04, 2007 at 05:04:21PM -0700, Andrew Morton wrote:
> On Thu, 05 Apr 2007 17:01:02 +0900
> Yasunori Goto <y-goto@...fujitsu.com> wrote:
> 
> > Hello.
> > 
> > This is to fix many section mismatches of code related to memory hotplug.
> > I checked compile with memory hotplug on/off on ia64 and x86-64 box.
> > 
> > ..
> >
> > ===================================================================
> > --- meminit.orig/drivers/acpi/numa.c	2007-04-04 20:15:58.000000000 +0900
> > +++ meminit/drivers/acpi/numa.c	2007-04-04 20:56:34.000000000 +0900
> > @@ -228,7 +228,7 @@ int __init acpi_numa_init(void)
> >  	return 0;
> >  }
> >  
> > -int acpi_get_pxm(acpi_handle h)
> > +int __meminit acpi_get_pxm(acpi_handle h)
> >  {
> >  	unsigned long pxm;
> >  	acpi_status status;
> > @@ -246,7 +246,7 @@ int acpi_get_pxm(acpi_handle h)
> >  }
> >  EXPORT_SYMBOL(acpi_get_pxm);
> >  
> > -int acpi_get_node(acpi_handle *handle)
> > +int __meminit acpi_get_node(acpi_handle *handle)
> >  {
> >  	int pxm, node = -1;
> 
> It doesn't make a lot of sense to export an __init symbol to modules.  I
> guess it's OK in this case, but we get warnings:

It seems wrong to me to first tell linker to discard the code after init and
next to export the symbol to make it available for any module anytime.

Both function are relatively small so better avoid playing games and
drop the __meminit tag.

	Sam
-
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/