* Naik, Avadhut <avadn...@amd.com> wrote:
> On 3/29/2024 02:11, Ingo Molnar wrote:
> >
> > Please split out the other (capitalization) changes to the output into
> > a separate patch.
> >
> Okay. Will put the capitalization changes into a separate patch.
>
> > - While at it, don't forget to:
> >
> > s/ADDR/MISC/SYND
> > /addr/misc/synd
> >
> These are actually acronyms for Address, Miscellaneous and Syndrome registers.
I kept SYND capitalized in the patch:
> > + TP_printk("CPU: %d, MCGc/s: %llx/%llx, MC%d: %016Lx, IPID: %016Lx,
> > addr: %016Lx, misc: %016Lx, SYND: %016Lx, RIP: %02x:<%016Lx>, TSC: %llx,
> > vendor: %u, CPUID: %x, time: %llu, socket: %u, APIC: %x",
But I guess ADDR and MISC are fine too.
To move forward with this I've committed the cleanup in tip:x86/cpu:
ac5e80e94f5c x86/mce: Clean up TP_printk() output line of the 'mce_record'
tracepoint
... and please post extensions of the tracepoint on top of that sane(er) code.
Thanks,
Ingo
