Re: [2.6 patch] kernel/sched*: optimize inlining

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Adrian Bunk
Date: Monday, May 12, 2008 - 12:45 am

On Mon, May 12, 2008 at 08:57:59AM +0200, Sam Ravnborg wrote:

I'm saying since 2005 that breaking the semantics of "inline" in the 
kernel can cause problems - and noone cared.

Now I did choose an example that highlights the possible problem of 
changing the semantics of "inline" [1], defended it, and let other 
people (like you) argue against it.

This thread is AFAIR the only time someone other than me said
"Stop! That inline here might be required!".

What I'll do with non-provoking patches will be to remove the > 95%
inline's that are not required and hope people like you now have a
rough understanding that this plus removing the "optimized inlining"
stuff is the safe way to go.


cu
Adrian

[1] in the few places that really need it a hint is not sufficient;
    if you think it's required somewhere you have to audit the code
    and use always_inline _before_ the "optimized inlining" gets offered

-- 

       "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

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[2.6 patch] kernel/sched*: optimize inlining, Adrian Bunk, (Sun May 11, 2:21 am)
Re: [2.6 patch] kernel/sched*: optimize inlining, Andreas Mohr, (Sun May 11, 4:18 am)
Re: [2.6 patch] kernel/sched*: optimize inlining, Adrian Bunk, (Sun May 11, 5:52 am)
Re: [2.6 patch] kernel/sched*: optimize inlining, Ray Lee, (Sun May 11, 10:22 am)
Re: [2.6 patch] kernel/sched*: optimize inlining, Sam Ravnborg, (Sun May 11, 10:52 am)
Re: [2.6 patch] kernel/sched*: optimize inlining, Adrian Bunk, (Sun May 11, 11:49 am)
Re: [2.6 patch] kernel/sched*: optimize inlining, Sam Ravnborg, (Sun May 11, 12:42 pm)
Re: [2.6 patch] kernel/sched*: optimize inlining, Adrian Bunk, (Sun May 11, 1:00 pm)
Re: [2.6 patch] kernel/sched*: optimize inlining, Sam Ravnborg, (Sun May 11, 1:38 pm)
Re: [2.6 patch] kernel/sched*: optimize inlining, Adrian Bunk, (Sun May 11, 2:19 pm)
Re: [2.6 patch] kernel/sched*: optimize inlining, Sam Ravnborg, (Sun May 11, 2:44 pm)
Re: [2.6 patch] kernel/sched*: optimize inlining, Adrian Bunk, (Sun May 11, 3:11 pm)
Re: [2.6 patch] kernel/sched*: optimize inlining, Sam Ravnborg, (Sun May 11, 11:57 pm)
Re: [2.6 patch] kernel/sched*: optimize inlining, Adrian Bunk, (Mon May 12, 12:45 am)