Messages in this thread | | | Subject | Re: [PATCH] clocksource: Store reg field within struct clocksource | From | Marc Gonzalez <> | Date | Thu, 19 Nov 2015 13:41:12 +0100 |
| |
On 19/11/2015 12:21, Russell King wrote:
> When I wrote the MMIO clocksource implementation, there was no > ____cacheline_aligned on struct clocksource, and the arrangement I came > to for the structure put the 'reg' and 'read' within the same cache line > (note that the MMIO clocksource pre-dates Thomas' rearrangement of struct > clocksource and the addition of the cache line alignment.) The original > layout did not have any padding gaps.
For the record, I pointed out the chronology in a previous discussion. But Thomas didn't comment at the time :-(
http://thread.gmane.org/gmane.linux.ports.arm.kernel/402968/focus=403604
Mason wrote: > Oh and while I have your attention ;-) I have alignment-related > questions about clocksource_mmio_init() (commit 442c8176d2) wrt > Thomas Gleixner's 369db4c952 patch. (I think the two patches > do not play nice.)
Regards.
| |