Re: [PATCH v7 2/2] perf cs-etm: Split --dump-raw-trace by AUX records

From: Mathieu Poirier
Date: Tue Jul 20 2021 - 11:53:40 EST


On Mon, Jul 19, 2021 at 05:10:34PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Mon, Jul 19, 2021 at 10:33:36AM -0600, Mathieu Poirier escreveu:
> > Hi Arnaldo,
> >
> > On Wed, Jul 14, 2021 at 02:45:34PM -0300, Arnaldo Carvalho de Melo wrote:
> > > Em Mon, Jul 05, 2021 at 01:33:38PM -0600, Mathieu Poirier escreveu:
> > > > On Thu, Jun 24, 2021 at 05:43:03PM +0100, James Clark wrote:
> > > > > for example:
> > > > >
> > > > > Idx:808; ID:1c; I_BAD_SEQUENCE : Invalid Sequence in packet.[I_ASYNC]
> > > > > ...
> > > > > PKTP_ETMV4I_0016 : 0x0014 (OCSD_ERR_INVALID_PCKT_HDR) [Invalid packet header]; TrcIdx=822
> > > > >
> > > > > Signed-off-by: James Clark <james.clark@xxxxxxx>
> > > > > ---
> > > > > tools/perf/util/cs-etm.c | 20 ++++++++++++++++++--
> > > > > 1 file changed, 18 insertions(+), 2 deletions(-)
> > > > >
> > > >
> > > > Reviewed-by: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx>
> > >
> > >
> > >
> > > Thanks, applied.
> >
> > This patch is the second of a two patch series[1][2]. The first one was applied and
> > is in Linus' current master branch. But I can't find the second one in
> > either your core branch or Linus', and that despite your reply above that it has been
> > applied. As such I am guessing something went wrong with the process.
> >
> > How do you want to proceed - should we resend the second patch[2] or you still
> > have it somewhere in your Inbox?
>
> My bad, its no in tmp.perf/urgent, after testing together with other
> patches will move to perf/urgent and by the end of this week I'll submit
> it to Linus, sorry.
>

Very good - thanks for the follow up.

> - Arnaldo
>
> > Thanks,
> > Mathieu
> >
> > [1]. https://lists.linaro.org/pipermail/coresight/2021-June/006575.html
> > [2]. https://lists.linaro.org/pipermail/coresight/2021-June/006576.html
> >
> > >
> > > - Arnaldo
> > >
>
> --
>
> - Arnaldo