Re: [PATCH 1/2] init.h: Remove __dev* sections from the kernel

2012-11-30 Thread Greg Kroah-Hartman
On Fri, Nov 30, 2012 at 12:39:43PM -0800, H. Peter Anvin wrote: > On 11/29/2012 10:43 AM, Greg Kroah-Hartman wrote: > > > >-/* Used for HOTPLUG */ > >-#define __devinit__section(.devinit.text) __cold notrace > >-#define __devinitdata__section(.devinit.data) > >-#define __devinitconst

Re: [PATCH 1/2] init.h: Remove __dev* sections from the kernel

2012-11-30 Thread Greg Kroah-Hartman
On Fri, Nov 30, 2012 at 12:39:01PM -0800, H. Peter Anvin wrote: > On 11/29/2012 10:43 AM, Greg Kroah-Hartman wrote: > >From: Greg Kroah-Hartman > > > >With the recent work to remove CONFIG_HOTPLUG, we are starting to get a > >bunch of __devinit section warnings, despite CONFIG_HOTPLUG always being

Re: [PATCH 1/2] init.h: Remove __dev* sections from the kernel

2012-11-30 Thread Bill Pemberton
H. Peter Anvin writes: > > On 11/29/2012 10:43 AM, Greg Kroah-Hartman wrote: > > From: Greg Kroah-Hartman > > > > With the recent work to remove CONFIG_HOTPLUG, we are starting to get a > > bunch of __devinit section warnings, despite CONFIG_HOTPLUG always being > > enabled. So, stop marking the

Re: [PATCH 1/2] init.h: Remove __dev* sections from the kernel

2012-11-30 Thread H. Peter Anvin
On 11/29/2012 10:43 AM, Greg Kroah-Hartman wrote: -/* Used for HOTPLUG */ -#define __devinit__section(.devinit.text) __cold notrace -#define __devinitdata__section(.devinit.data) -#define __devinitconst __constsection(.devinit.rodata) -#define __devexit__section(.devexit.te

Re: [PATCH 1/2] init.h: Remove __dev* sections from the kernel

2012-11-30 Thread H. Peter Anvin
On 11/29/2012 10:43 AM, Greg Kroah-Hartman wrote: From: Greg Kroah-Hartman With the recent work to remove CONFIG_HOTPLUG, we are starting to get a bunch of __devinit section warnings, despite CONFIG_HOTPLUG always being enabled. So, stop marking the sections entirely, by defining them away the

Re: [PATCH 1/2] init.h: Remove __dev* sections from the kernel

2012-11-30 Thread Greg Kroah-Hartman
On Fri, Nov 30, 2012 at 07:11:56PM +0100, Sam Ravnborg wrote: > On Thu, Nov 29, 2012 at 10:43:28AM -0800, Greg Kroah-Hartman wrote: > > From: Greg Kroah-Hartman > > > > With the recent work to remove CONFIG_HOTPLUG, we are starting to get a > > bunch of __devinit section warnings, despite CONFIG_

Re: [PATCH 1/2] init.h: Remove __dev* sections from the kernel

2012-11-30 Thread Sam Ravnborg
On Thu, Nov 29, 2012 at 10:43:28AM -0800, Greg Kroah-Hartman wrote: > From: Greg Kroah-Hartman > > With the recent work to remove CONFIG_HOTPLUG, we are starting to get a > bunch of __devinit section warnings, despite CONFIG_HOTPLUG always being > enabled. So, stop marking the sections entirely,

[PATCH 1/2] init.h: Remove __dev* sections from the kernel

2012-11-29 Thread Greg Kroah-Hartman
From: Greg Kroah-Hartman With the recent work to remove CONFIG_HOTPLUG, we are starting to get a bunch of __devinit section warnings, despite CONFIG_HOTPLUG always being enabled. So, stop marking the sections entirely, by defining them away the section markings in init.h Signed-off-by: Greg Kro