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

[James Clark]

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
>>>
>