Re: [PATCH v2 0/2] perf cs-etm: Set time on synthesised samples to preserve ordering

From: Arnaldo Carvalho de Melo
Date: Fri Apr 16 2021 - 11:19:00 EST


Em Fri, Apr 16, 2021 at 09:07:09AM -0600, Mathieu Poirier escreveu:
> Hi James,
>
> On Fri, Apr 16, 2021 at 01:56:30PM +0300, James Clark wrote:
> > Changes since v1:
> > * Improved variable name from etm_timestamp -> cs_timestamp
> > * Fixed ordering of Signed-off-by
> >
>
> You forgot to add the RB and AB you received. Since Arnaldo is responsible for
> the perf tools subsystem, please send another revision.



Yep, please collect Reported-by and Acked-by as you go sending new
versions of a patchset, the last one I don't have a problem collecting
myself, but if you have to resend, please collect the feedback tags.

- Arnaldo

> Thanks,
> Mathieu
>
> > James Clark (2):
> > perf cs-etm: Refactor timestamp variable names
> > perf cs-etm: Set time on synthesised samples to preserve ordering
> >
> > .../perf/util/cs-etm-decoder/cs-etm-decoder.c | 18 +++----
> > tools/perf/util/cs-etm.c | 52 ++++++++++---------
> > tools/perf/util/cs-etm.h | 4 +-
> > 3 files changed, 39 insertions(+), 35 deletions(-)
> >
> > --
> > 2.28.0
> >

--

- Arnaldo