Re: [PATCH v1] regmap: introduce value tracing for regmap bulk operations

From: Marc Zyngier
Date: Thu Aug 18 2022 - 09:49:29 EST


On Thu, 18 Aug 2022 13:15:00 +0100,
Dmitry Rokosov <DDRokosov@xxxxxxxxxxxxxx> wrote:
>
> +Thomas, Rasmus and Marc
>
> On Tue, Aug 16, 2022 at 06:14:48PM +0000, Dmitry Rokosov wrote:
> > Currently, only one-register io operations support tracepoints with
> > value logging. For the regmap bulk operations developer can view
> > hw_start/hw_done tracepoints with starting reg number and registers
> > count to be reading or writing. This patch injects tracepoints with
> > dumping registers values in the hex format to regmap bulk reading
> > and writing.

I don't care much about regmap as a MMIO backend, but it strikes me as
odd that you end up with multiple ways of logging the same stuff (with
a memcpy in the middle of it).

Why can't this be done with a small amount of trace post-processing?

M.

--
Without deviation from the norm, progress is not possible.