On Tue, Nov 14, 2017 at 2:50 PM, Thomas Gleixner <t...@linutronix.de> wrote: > > Something like the untested patch below should do the storage and converts > the primary timestamp from local_clock() to clock monotonic.
This may work. > - u64 ts_nsec; /* timestamp in nanoseconds */ > + /* Timestamps in nanoseconds */ > + union { > + u64 ts_nsec; > + struct timestamps ts; > + }; But I'd skip the union, and just search-and-replace the ts_nsec -> ts.mono. You did that in at least one place anyway. Because as it is, that's just too subtle and hard to read. Don't make it worse by having a magical "ts_nsec is also ts.mono, but you have to look in two different files to see that. That file is messy enough as-is. Linus