Messages in this thread | | | From | Dmitry Rokosov <> | Subject | Re: [PATCH v1] regmap: introduce value tracing for regmap bulk operations | Date | Thu, 18 Aug 2022 17:44:23 +0000 |
| |
Hello Mark,
On Thu, Aug 18, 2022 at 04:43:45PM +0100, Mark Brown wrote: > On Thu, Aug 18, 2022 at 02:49:20PM +0100, Marc Zyngier wrote: > > > 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? > > At the minute we don't put the actual data for the bulk transfers into > the trace so the information simply isn't there.
What do you think about the patch? Can we use the separate trace event class, or do we have to add these tracepoints to some existing class, like regmap_block?
Appreciate any thoughts and feedback.
-- Thank you, Dmitry
| |