Messages in this thread | | | Date | Thu, 19 Nov 2015 14:57:30 +0100 (CET) | From | Thomas Gleixner <> | Subject | Re: [PATCH] clocksource: Store reg field within struct clocksource |
| |
On Thu, 19 Nov 2015, Marc Gonzalez wrote: > On 19/11/2015 12:14, Thomas Gleixner wrote: > > > So yes, the alignment of the clocksource struct is not longer > > relevant. The case where we access clocksource->max_cycles is when > > CONFIG_DEBUG_TIMEKEEPING is enabled, which imposes worse performance > > problems to timekeeping than the extra cacheline. > > > > So the simple solution for this issue is indeed the one liner below. > > It would make sense to also remove the comment emphasizing the > alignment requirement.
Indeed.
| |