Re: CONFIG_OPTIMIZE_INLINING

[Ingo Molnar <mingo@elte.hu>]

* David Miller <davem@davemloft.net> wrote:

> From: Ingo Molnar <mingo@elte.hu>
> Date: Sun, 27 Apr 2008 07:59:43 +0200
> 
> > ... as i saw no reason why this feature, which i found rather 
> > useful, should be delayed another year or so. I'd be more than happy 
> > to promote this feature back to lib/Kconfig.debug, sparc64 interest 
> > would make that a strong argument.
> 
> So you caved in to FUD in order to pad your commit and signoff count?

David, stop this nonsense already.

Had you read the lkml discussion thread i linked to:

  http://lkml.org/lkml/2008/3/3/122

... you'd realize that the first thing i did was to make it a generic 
feature. But it was claimed on lkml that this would break an unknown 
number of architectures:

| > > And what we should do is to attack the excessive wrong usage of 
| > > inlines in .c files, not messing with a global #define in a way 
| > > that the results on 24 architectures with 7 different releases of 
| > > gcc would be unpredictable.

Where were you in that thread defending my position? Andrew showed 
interest, gave me feedback and i incorporated that feedback.

I am fully willing and ready to move this feature to a generic file (via 
the patch below), in fact i implemented it that way from the get go, but 
only:

   _if other architecture maintainers want it too_

Nobody countered the "it would break other architectures" position in 
that discussion, and i'm not interested in endless bikeshed painting 
either.

lets look at a counter-example where other architecture maintainers 
showed interest: the recent bitops generalizations we did for x86 and 
extended to all other architectures. It was welcome by other maintainers 
and we did it that way. See the merge commit 9b79ed952bd734 for an 
overview of this topic:

    x86, bitops: select the generic bitmap search functions
    x86: include/asm-x86/pgalloc.h/bitops.h: checkpatch cleanups - formatting
    x86: finalize bitops unification
    x86, UML: remove x86-specific implementations of find_first_bit
    x86: optimize find_first_bit for small bitmaps
    x86: switch 64-bit to generic find_first_bit
    x86: generic versions of find_first_(zero_)bit, convert i386
    bitops: use __fls for fls64 on 64-bit archs
    generic: implement __fls on all 64-bit archs
    generic: introduce a generic __fls implementation
    x86: merge the simple bitops and move them to bitops.h
    x86, generic: optimize find_next_(zero_)bit for small constant-size bitmap
    x86, uml: fix uml with generic find_next_bit for x86
    x86: change x86 to use generic find_next_bit
    uml: Kconfig cleanup
    uml: fix build error

	Ingo

-------------------------------->
Subject: generic: optimize inlining
From: Ingo Molnar <mingo@elte.hu>
Date: Sun Apr 27 09:36:29 CEST 2008

Suggested-by: David Miller <davem@davemloft.net>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
---
 arch/x86/Kconfig.debug |   12 ------------
 init/Kconfig           |   13 +++++++++++++
 2 files changed, 13 insertions(+), 12 deletions(-)

Index: linux/arch/x86/Kconfig.debug
===================================================================
--- linux.orig/arch/x86/Kconfig.debug
+++ linux/arch/x86/Kconfig.debug
@@ -355,15 +355,3 @@ config CPA_DEBUG
 
 endmenu
 
-config OPTIMIZE_INLINING
-	bool "Allow gcc to uninline functions marked 'inline'"
-	default y
-	help
-	  This option determines if the kernel forces gcc to inline the functions
-	  developers have marked 'inline'. Doing so takes away freedom from gcc to
-	  do what it thinks is best, which is desirable for the gcc 3.x series of
-	  compilers. The gcc 4.x series have a rewritten inlining algorithm and
-	  disabling this option will generate a smaller kernel there. Hopefully
-	  this algorithm is so good that allowing gcc4 to make the decision can
-	  become the default in the future, until then this option is there to
-	  test gcc for this.
Index: linux/init/Kconfig
===================================================================
--- linux.orig/init/Kconfig
+++ linux/init/Kconfig
@@ -520,6 +520,19 @@ config CC_OPTIMIZE_FOR_SIZE
 
 	  If unsure, say N.
 
+config OPTIMIZE_INLINING
+	bool "Allow gcc to uninline functions marked 'inline'"
+	default y
+	help
+	  This option determines if the kernel forces gcc to inline the functions
+	  developers have marked 'inline'. Doing so takes away freedom from gcc to
+	  do what it thinks is best, which is desirable for the gcc 3.x series of
+	  compilers. The gcc 4.x series have a rewritten inlining algorithm and
+	  disabling this option will generate a smaller kernel there. Hopefully
+	  this algorithm is so good that allowing gcc4 to make the decision can
+	  become the default in the future, until then this option is there to
+	  test gcc for this.
+
 config SYSCTL
 	bool