Messages in this thread | | | Date | Thu, 25 Sep 2003 20:11:27 +0200 | From | Sam Ravnborg <> | Subject | Re: [2.6 patch] add a config option for -Os compilation |
| |
On Thu, Sep 25, 2003 at 04:38:45PM +0200, Adrian Bunk wrote: > --- linux-2.6.0-test5-mm4/init/Kconfig.old 2003-09-25 14:38:18.000000000 +0200 > +++ linux-2.6.0-test5-mm4/init/Kconfig 2003-09-25 14:47:12.000000000 +0200 > @@ -65,6 +65,16 @@ > > menu "General setup" > > +config OPTIMIZE_FOR_SIZE > + bool "Optimize for size" if EXPERIMENTAL > + default y if ARM || H8300 > + default n > + help > + Enabling this option will pass "-Os" instead of "-O2" to gcc > + resulting in a smaller kernel. > + > + If unsure, say N. > +
This is a general file, and it is wrong to include architecture specific knowledge here. I recall that Roman Zippel introduced "enable" for exactly this purpose.
I looked into Documentation/kbuild/kconfig-language.txt but did not see it documented - Roman?
Another comment about the naming of the config symbol. We keep getting more config symbols controlling options to GCC. A naming scheme like: CONFIG_CC_OPTIMIZE_FOR_SIZE, CONFIG_CC_DEBUG_INFO is preferable since it give better context information. Let's start with this option, and rename the others later.
Sam - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |