Re: [RFC PATCH 0/8] EDAC, mce_amd: Add a tracepoint for the decoded error

From: Ingo Molnar
Date: Fri Jul 28 2017 - 02:38:05 EST



* Borislav Petkov <bp@xxxxxxxxx> wrote:

> BUT(!), I just realized, I *think* I can address this much more elegantly:
> extend trace_mce_record() by adding the decoded string as its last argument. And
> that's fine, I'm being told, because adding arguments to the tracepoints is not
> a big deal, removing them is hard. And actually, we have added args before, come
> to think of it:

Yeah, structured, append-only ABIs are elegant - that's what perf uses too.

> I agree with the rest but you're obviously preaching to the choir.
>
> :-)

Had do vent my (non kernel tree integrated) Linux tooling frustration!! ;-)

Thanks,

Ingo