[2.6 patch] include/asm-mips/processor.h: "extern inline" -> "static inline"

[2.6 patch] include/asm-mips/processor.h: "extern inline" -> "static inline"

Post by Ralf Baech » Tue, 10 Jul 2007 19:40:08



The idea was to have a linker error in case gcc should deciede for some
reason not to inline this function which as I understand will continue
to be the behaviour of gcc 4.3?

Ralf
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to XXXX@XXXXX.COM
More majordomo info at http://www.yqcomputer.com/
Please read the FAQ at http://www.yqcomputer.com/
 
 
 

[2.6 patch] include/asm-mips/processor.h: "extern inline" -> "static inline"

Post by Adrian Bun » Tue, 10 Jul 2007 21:30:17


If that's the intention, please use __always_inline instead.


In C99 (and therefore in gcc >= 4.3), "extern inline" means that the
function should be compiled inline where the inline definition is seen,
and that the compiler should also emit a copy of the function body with
an externally visible symbol.

You don't want this.


cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to XXXX@XXXXX.COM
More majordomo info at http://www.yqcomputer.com/
Please read the FAQ at http://www.yqcomputer.com/

 
 
 

[2.6 patch] include/asm-mips/processor.h: "extern inline" -> "static inline"

Post by Ralf Baech » Tue, 10 Jul 2007 22:20:10


Uh, indeed. Thanks for the reminder. Patch applied.

Ralf
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to XXXX@XXXXX.COM
More majordomo info at http://www.yqcomputer.com/
Please read the FAQ at http://www.yqcomputer.com/