Messages in this thread Patch in this message | | | Subject | Re: CONFIG_REORDER Kconfig help strange sentence. | From | Rusty Russell <> | Date | Tue, 13 Mar 2007 17:37:35 +1100 |
| |
On Tue, 2007-03-13 at 00:56 +0100, Andi Kleen wrote: > On Tue, Mar 13, 2007 at 10:18:03AM +1100, Rusty Russell wrote: > > OK, this confused me: > > > > Function reordering (REORDER) [N/y/?] (NEW) ? > > > > This option enables the toolchain to reorder functions for a more > > optimal TLB usage. If you have pretty much any version of binutils, > > this can increase your kernel build time by roughly one minute. > > > > "If you have pretty much any version of binutils"? Huh? > > > > You mean "This will slow your kernel build by about a minute"? > > Yes. Lots of sections seem to trigger some quadratic behaviour in ld. > > It might be fixed in some unreleased CVS version though (not 100% sure) > > -Andi
OK, well here is a patch for the moment.
== Clarify CONFIG_REORDER explanation
if (1 && X) => if (X).
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
diff -r de5618b5e562 arch/x86_64/Kconfig --- a/arch/x86_64/Kconfig Tue Mar 13 11:41:55 2007 +1100 +++ b/arch/x86_64/Kconfig Tue Mar 13 17:27:05 2007 +1100 @@ -632,8 +632,8 @@ config REORDER default n help This option enables the toolchain to reorder functions for a more - optimal TLB usage. If you have pretty much any version of binutils, - this can increase your kernel build time by roughly one minute. + optimal TLB usage. This will slow your kernel build by + roughly one minute. config K8_NB def_bool y
- 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/
| |