mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* kbuild: Spelling/grammar fixes for config DEBUG_SECTION_MISMATCH
@ 2008-01-30 10:13 Geert Uytterhoeven
  2008-01-30 16:08 ` Randy Dunlap
  0 siblings, 1 reply; 3+ messages in thread
From: Geert Uytterhoeven @ 2008-01-30 10:13 UTC (permalink / raw)
  To: Sam Ravnborg; +Cc: linux-kbuild, Linux Kernel Development

[-- Attachment #1: Type: TEXT/PLAIN, Size: 2207 bytes --]

kbuild: Spelling/grammar fixes for config DEBUG_SECTION_MISMATCH

Signed-off-by: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>
---
 lib/Kconfig.debug |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -90,19 +90,19 @@ config DEBUG_SECTION_MISMATCH
 	  most likely result in an oops.
 	  In the code functions and variables are annotated with
 	  __init, __devinit etc. (see full list in include/linux/init.h)
-	  which result in the code/data being placed in specific sections.
-	  The section mismatch anaylsis are always done after a full
+	  which results in the code/data being placed in specific sections.
+	  The section mismatch analysis is always done after a full
 	  kernel build but enabling this options will in addition
 	  do the following:
 	  - Add the option -fno-inline-functions-called-once to gcc
 	    When inlining a function annotated __init in a non-init
 	    function we would loose the section information and thus
 	    the analysis would not catch the illegal reference.
-	    This options tell gcc to inline less but will also
+	    This option tells gcc to inline less but will also
 	    result in a larger kernel.
 	  - Run the section mismatch analysis for each module/built-in.o
 	    When we run the section mismatch analysis on vmlinux.o we
-	    looses valueable information about where the mismatch was
+	    loose valueable information about where the mismatch was
 	    introduced.
 	    Running the analysis for each module/built-in.o file
 	    will tell where the mismatch happens much closer to the

With kind regards,

Geert Uytterhoeven
Software Architect

Sony Network and Software Technology Center Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium

Phone:    +32 (0)2 700 8453
Fax:      +32 (0)2 700 8622
E-mail:   Geert.Uytterhoeven@sonycom.com
Internet: http://www.sony-europe.com/

Sony Network and Software Technology Center Europe
A division of Sony Service Centre (Europe) N.V.
Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium
VAT BE 0413.825.160 · RPR Brussels
Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: kbuild: Spelling/grammar fixes for config DEBUG_SECTION_MISMATCH
  2008-01-30 10:13 kbuild: Spelling/grammar fixes for config DEBUG_SECTION_MISMATCH Geert Uytterhoeven
@ 2008-01-30 16:08 ` Randy Dunlap
  2008-02-02 18:06   ` Sam Ravnborg
  0 siblings, 1 reply; 3+ messages in thread
From: Randy Dunlap @ 2008-01-30 16:08 UTC (permalink / raw)
  To: Geert Uytterhoeven; +Cc: Sam Ravnborg, linux-kbuild, Linux Kernel Development

On Wed, 30 Jan 2008 11:13:23 +0100 (CET) Geert Uytterhoeven wrote:

> kbuild: Spelling/grammar fixes for config DEBUG_SECTION_MISMATCH
> 
> Signed-off-by: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>
> ---
>  lib/Kconfig.debug |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -90,19 +90,19 @@ config DEBUG_SECTION_MISMATCH
>  	  most likely result in an oops.
>  	  In the code functions and variables are annotated with
>  	  __init, __devinit etc. (see full list in include/linux/init.h)
> -	  which result in the code/data being placed in specific sections.
> -	  The section mismatch anaylsis are always done after a full
> +	  which results in the code/data being placed in specific sections.
> +	  The section mismatch analysis is always done after a full
>  	  kernel build but enabling this options will in addition

	                                 option

>  	  do the following:
>  	  - Add the option -fno-inline-functions-called-once to gcc
>  	    When inlining a function annotated __init in a non-init
>  	    function we would loose the section information and thus

	                      lose

>  	    the analysis would not catch the illegal reference.
> -	    This options tell gcc to inline less but will also
> +	    This option tells gcc to inline less but will also
>  	    result in a larger kernel.
>  	  - Run the section mismatch analysis for each module/built-in.o
>  	    When we run the section mismatch analysis on vmlinux.o we
> -	    looses valueable information about where the mismatch was
> +	    loose valueable information about where the mismatch was

	    lose valuable

>  	    introduced.
>  	    Running the analysis for each module/built-in.o file
>  	    will tell where the mismatch happens much closer to the

Thanks.  I had a note to myself to do this as well.

---
~Randy

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: kbuild: Spelling/grammar fixes for config DEBUG_SECTION_MISMATCH
  2008-01-30 16:08 ` Randy Dunlap
@ 2008-02-02 18:06   ` Sam Ravnborg
  0 siblings, 0 replies; 3+ messages in thread
From: Sam Ravnborg @ 2008-02-02 18:06 UTC (permalink / raw)
  To: Randy Dunlap; +Cc: Geert Uytterhoeven, linux-kbuild, Linux Kernel Development

On Wed, Jan 30, 2008 at 08:08:11AM -0800, Randy Dunlap wrote:
> On Wed, 30 Jan 2008 11:13:23 +0100 (CET) Geert Uytterhoeven wrote:
> 
> > kbuild: Spelling/grammar fixes for config DEBUG_SECTION_MISMATCH
> > 
> > Signed-off-by: Geert Uytterhoeven <Geert.Uytterhoeven@sonycom.com>

Applied a combined patch - thanks for fixing my bad grammar/spelling.

	Sam

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-02-02 18:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-01-30 10:13 kbuild: Spelling/grammar fixes for config DEBUG_SECTION_MISMATCH Geert Uytterhoeven
2008-01-30 16:08 ` Randy Dunlap
2008-02-02 18:06   ` Sam Ravnborg

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome