Messages in this thread | | | From | Roman Zippel <> | Subject | Re: [PATCH 7/8] Remove current_tick_length() | Date | Wed, 26 Mar 2008 02:53:20 +0100 |
| |
Hi,
On Tuesday 18. March 2008, john stultz wrote:
> On Sat, 2008-03-15 at 18:14 +0100, Roman Zippel wrote: > > Hi, > > > > On Sat, 15 Mar 2008, Ray Lee wrote: > > > Then make the original function an inline. With -O2 it should compile > > > to exactly the same thing. > > > > This would also defeat John's intention of keeping the value static. > > Well, don't mistake me for being fanatical about it. Having the values > be static is cleaner, but if its a real performance issue, then clearly > performance wins.
There are two aspects, such functions tend to generate slightly larger and slower code.
> I do like Ray's suggestion, and think using the inline'd function is > preferred to the raw variable, as it better establishes through use if > nothing else, the read-only nature of the value outside of ntp.
In other languages one would use private or protected for this, but we don't have this. I'm not too fond of an inline function, as it would mark it as some kind of public API, which it isn't. It's just an internal value used by the timekeeping code, which happens to be needed by a few source files. If you want to make a little more private, it would be better to move it to header under kernel/time/.
bye, Roman
| |