Re: [PATCH v2 0/7] perf arm-spe: Enable timestamp

2021-04-07 Thread Leo Yan
On Wed, Apr 07, 2021 at 04:28:40PM +0300, Adrian Hunter wrote: > On 7/04/21 4:15 pm, Leo Yan wrote: > > Hi Al, > > > > On Tue, Apr 06, 2021 at 09:38:32AM +, Al Grant wrote: > > > > [...] > > > >>> This patch set is to enable timestamp for Arm SPE trace. It reads out TSC > >>> parameters fro

Re: [PATCH v2 0/7] perf arm-spe: Enable timestamp

2021-04-07 Thread Adrian Hunter
On 7/04/21 4:15 pm, Leo Yan wrote: > Hi Al, > > On Tue, Apr 06, 2021 at 09:38:32AM +, Al Grant wrote: > > [...] > >>> This patch set is to enable timestamp for Arm SPE trace. It reads out TSC >>> parameters from mmap page and stores into auxtrace info structure; >> >> Why not synthesize a P

Re: [PATCH v2 0/7] perf arm-spe: Enable timestamp

2021-04-07 Thread Leo Yan
Hi Al, On Tue, Apr 06, 2021 at 09:38:32AM +, Al Grant wrote: [...] > > This patch set is to enable timestamp for Arm SPE trace. It reads out TSC > > parameters from mmap page and stores into auxtrace info structure; > > Why not synthesize a PERF_RECORD_TIME_CONV - isn't that specifically t

RE: [PATCH v2 0/7] perf arm-spe: Enable timestamp

2021-04-06 Thread Al Grant
a ; > Namhyung Kim ; Adrian Hunter > ; linux-arm-ker...@lists.infradead.org; linux- > ker...@vger.kernel.org > Cc: leo....@linaro.org > Subject: [PATCH v2 0/7] perf arm-spe: Enable timestamp > > As we know, the timestamp is important for AUX trace; it's mainly used to >

[PATCH v2 0/7] perf arm-spe: Enable timestamp

2021-04-03 Thread Leo Yan
As we know, the timestamp is important for AUX trace; it's mainly used to correlate between perf events and AUX trace, allows to generate events with time ordered manner. There have several good examples of enabling timestamp for AUX trace (like Intel-pt, Intel-bts, etc). Since the conversion bet