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
| ||
|
Date: Sun, 31 Jul 2011 20:53:06 -0400 From: Arnaud Lacombe <lacombar@...il.com> To: Peter Foley <pefoley2@...izon.net> Cc: Linux Kernel Mailing List <linux-kernel@...r.kernel.org>, Linux Kbuild Mailing List <linux-kbuild@...r.kernel.org>, mmarek@...e.cz, rdunlap@...otime.net Subject: Re: [PATCH RFC] kconfig.h fix extraneous dependencies Hi, On Sun, Jul 31, 2011 at 8:36 PM, Peter Foley <pefoley2@...izon.net> wrote: > The introduction of kconfig.h caused fixdep to create three extraneous > dependencies. > include/config/.h > include/config/h.h > include/config/foo.h > > Remove them by eliminating instances of CONFIG_ > this seems ugly to me... I would rather try to fix fixdep, no ? - Arnaud > Signed-off-by: Peter Foley <pefoley2@...izon.net> > --- > > Suggestions for a better replacement for CONFIG_FOO would be appreciated. > > diff --git a/include/linux/kconfig.h b/include/linux/kconfig.h > index 067eda0..66a0659 100644 > --- a/include/linux/kconfig.h > +++ b/include/linux/kconfig.h > @@ -1,15 +1,15 @@ > -#ifndef __LINUX_KCONFIG_H > -#define __LINUX_KCONFIG_H > +#ifndef __LINUX_KCONFIG > +#define __LINUX_KCONFIG > > #include <generated/autoconf.h> > > /* > - * Helper macros to use CONFIG_ options in C expressions. Note that > + * Helper macros to use CONFIG options in C expressions. Note that > * these only work with boolean and tristate options. > */ > > /* > - * IS_ENABLED(CONFIG_FOO) evaluates to 1 if CONFIG_FOO is set to 'y' or 'm', > + * IS_ENABLED(CONFIG _FOO) evaluates to 1 if CONFIG _FOO is set to 'y' or 'm', > * 0 otherwise. > * > */ > @@ -17,16 +17,16 @@ > (__enabled_ ## option || __enabled_ ## option ## _MODULE) > > /* > - * IS_BUILTIN(CONFIG_FOO) evaluates to 1 if CONFIG_FOO is set to 'y', 0 > + * IS_BUILTIN(CONFIG _FOO) evaluates to 1 if CONFIG _FOO is set to 'y', 0 > * otherwise. For boolean options, this is equivalent to > - * IS_ENABLED(CONFIG_FOO). > + * IS_ENABLED(CONFIG _FOO). > */ > #define IS_BUILTIN(option) __enabled_ ## option > > /* > - * IS_MODULE(CONFIG_FOO) evaluates to 1 if CONFIG_FOO is set to 'm', 0 > + * IS_MODULE(CONFIG _FOO) evaluates to 1 if CONFIG _FOO is set to 'm', 0 > * otherwise. > */ > #define IS_MODULE(option) __enabled_ ## option ## _MODULE > > -#endif /* __LINUX_KCONFIG_H */ > +#endif /* __LINUX_KCONFIG */ > -- 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