[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <c3a75731-15b4-e052-1840-a881b05862df@arm.com>
Date: Mon, 19 Apr 2021 11:18:31 +0300
From: James Clark <james.clark@....com>
To: Arnaldo Carvalho de Melo <acme@...nel.org>,
Mathieu Poirier <mathieu.poirier@...aro.org>
Cc: coresight@...ts.linaro.org, al.grant@....com,
branislav.rankov@....com, denik@...omium.org,
suzuki.poulose@....com, Mike Leach <mike.leach@...aro.org>,
Leo Yan <leo.yan@...aro.org>,
Mark Rutland <mark.rutland@....com>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
Jiri Olsa <jolsa@...hat.com>,
Namhyung Kim <namhyung@...nel.org>,
John Garry <john.garry@...wei.com>,
Will Deacon <will@...nel.org>,
linux-arm-kernel@...ts.infradead.org,
linux-perf-users@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 0/2] perf cs-etm: Set time on synthesised samples to
preserve ordering
On 16/04/2021 18:16, Arnaldo Carvalho de Melo wrote:
> 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
>
Ok thanks, I will keep that in mind for the future. I wasn't sure if they still
applied or not as it was a new version.
Thanks
James
>> 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
>>>
>
Powered by blists - more mailing lists