Messages in this thread | | | Subject | Re: [PATCH] clocksource: Store reg field within struct clocksource | From | Marc Gonzalez <> | Date | Tue, 24 Nov 2015 13:36:50 +0100 |
| |
On 19/11/2015 14:57, Thomas Gleixner wrote: > 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.
Thomas,
Will you commit that patch yourself?
Regards.
| |