[PATCH 26/74] lto, sound: Fix export symbols for !CONFIG_MODULES
From: Andi Kleen The new LTO EXPORT_SYMBOL references symbols even without CONFIG_MODULES. Since these functions are macros in this case this doesn't work. Add a ifdef to fix the build. Cc: ti...@suse.de Signed-off-by: Andi Kleen --- sound/core/seq/seq_device.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/sound/core/seq/seq_device.c b/sound/core/seq/seq_device.c index 5cf8d65..60e8fc1 100644 --- a/sound/core/seq/seq_device.c +++ b/sound/core/seq/seq_device.c @@ -569,5 +569,7 @@ EXPORT_SYMBOL(snd_seq_device_load_drivers); EXPORT_SYMBOL(snd_seq_device_new); EXPORT_SYMBOL(snd_seq_device_register_driver); EXPORT_SYMBOL(snd_seq_device_unregister_driver); +#ifdef CONFIG_MODULES EXPORT_SYMBOL(snd_seq_autoload_lock); EXPORT_SYMBOL(snd_seq_autoload_unlock); +#endif -- 1.7.7.6 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 26/74] lto, sound: Fix export symbols for !CONFIG_MODULES
At Sat, 18 Aug 2012 19:56:22 -0700, Andi Kleen wrote: > > From: Andi Kleen > > The new LTO EXPORT_SYMBOL references symbols even without CONFIG_MODULES. > Since these functions are macros in this case this doesn't work. > Add a ifdef to fix the build. > > Cc: ti...@suse.de > Signed-off-by: Andi Kleen Reviewed-by: Takashi Iwai I haven't seen the background, so let me ask a dumb question: is it a 3.6 fix or for 3.7? And shall I apply this one to sound git tree, or would you like to apply all in a single tree? thanks, Takashi > --- > sound/core/seq/seq_device.c |2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/sound/core/seq/seq_device.c b/sound/core/seq/seq_device.c > index 5cf8d65..60e8fc1 100644 > --- a/sound/core/seq/seq_device.c > +++ b/sound/core/seq/seq_device.c > @@ -569,5 +569,7 @@ EXPORT_SYMBOL(snd_seq_device_load_drivers); > EXPORT_SYMBOL(snd_seq_device_new); > EXPORT_SYMBOL(snd_seq_device_register_driver); > EXPORT_SYMBOL(snd_seq_device_unregister_driver); > +#ifdef CONFIG_MODULES > EXPORT_SYMBOL(snd_seq_autoload_lock); > EXPORT_SYMBOL(snd_seq_autoload_unlock); > +#endif > -- > 1.7.7.6 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 26/74] lto, sound: Fix export symbols for !CONFIG_MODULES
On Mon, Aug 20, 2012 at 10:30:29AM +0200, Takashi Iwai wrote: > At Sat, 18 Aug 2012 19:56:22 -0700, > Andi Kleen wrote: > > > > From: Andi Kleen > > > > The new LTO EXPORT_SYMBOL references symbols even without CONFIG_MODULES. > > Since these functions are macros in this case this doesn't work. > > Add a ifdef to fix the build. > > > > Cc: ti...@suse.de > > Signed-off-by: Andi Kleen > > Reviewed-by: Takashi Iwai > > I haven't seen the background, so let me ask a dumb question: > is it a 3.6 fix or for 3.7? I don't strictly need it for 3.6, 3.7 is ok. > > And shall I apply this one to sound git tree, or would you like to > apply all in a single tree? Please apply it in yours. -Andi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 26/74] lto, sound: Fix export symbols for !CONFIG_MODULES
At Mon, 20 Aug 2012 11:45:45 +0200, Andi Kleen wrote: > > On Mon, Aug 20, 2012 at 10:30:29AM +0200, Takashi Iwai wrote: > > At Sat, 18 Aug 2012 19:56:22 -0700, > > Andi Kleen wrote: > > > > > > From: Andi Kleen > > > > > > The new LTO EXPORT_SYMBOL references symbols even without CONFIG_MODULES. > > > Since these functions are macros in this case this doesn't work. > > > Add a ifdef to fix the build. > > > > > > Cc: ti...@suse.de > > > Signed-off-by: Andi Kleen > > > > Reviewed-by: Takashi Iwai > > > > I haven't seen the background, so let me ask a dumb question: > > is it a 3.6 fix or for 3.7? > > I don't strictly need it for 3.6, 3.7 is ok. > > > > > And shall I apply this one to sound git tree, or would you like to > > apply all in a single tree? > > Please apply it in yours. OK, applied now. Thanks. Takashi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/