Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1442363 > unrolled thread

Re: [PATCH RESEND 2/2] perf tools: adding coresight etm PMU record capabilities

Started byAdrian Hunter <adrian.hunter@intel.com>
First post2016-07-13 13:50 +0200
Last post2016-07-13 19:30 +0200
Articles 4 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH RESEND 2/2] perf tools: adding coresight etm PMU record  capabilities Adrian Hunter <adrian.hunter@intel.com> - 2016-07-13 13:50 +0200
    Re: [PATCH RESEND 2/2] perf tools: adding coresight etm PMU record capabilities Mathieu Poirier <mathieu.poirier@linaro.org> - 2016-07-13 17:20 +0200
      Re: [PATCH RESEND 2/2] perf tools: adding coresight etm PMU record  capabilities Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-07-13 17:30 +0200
        Re: [PATCH RESEND 2/2] perf tools: adding coresight etm PMU record capabilities Mathieu Poirier <mathieu.poirier@linaro.org> - 2016-07-13 19:30 +0200

#1442363 — Re: [PATCH RESEND 2/2] perf tools: adding coresight etm PMU record capabilities

FromAdrian Hunter <adrian.hunter@intel.com>
Date2016-07-13 13:50 +0200
SubjectRe: [PATCH RESEND 2/2] perf tools: adding coresight etm PMU record capabilities
Message-ID<rUqPU-2ww-19@gated-at.bofh.it>
On 17/06/16 20:02, Mathieu Poirier wrote:
> Coresight ETMs are IP blocks used to perform HW assisted tracing
> on a CPU core.  This patch introduce the required auxiliary API
> functions allowing the perf core to interact with a tracer.
> 
> Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
> ---
>  MAINTAINERS                         |   3 +
>  tools/perf/arch/arm/util/Build      |   2 +-
>  tools/perf/arch/arm/util/auxtrace.c |  54 ++++
>  tools/perf/arch/arm/util/cs-etm.c   | 559 ++++++++++++++++++++++++++++++++++++
>  tools/perf/arch/arm/util/cs-etm.h   |  23 ++
>  tools/perf/arch/arm64/util/Build    |   4 +
>  tools/perf/util/auxtrace.c          |   1 +
>  tools/perf/util/auxtrace.h          |   1 +
>  tools/perf/util/cs-etm.h            |  74 +++++
>  9 files changed, 720 insertions(+), 1 deletion(-)
>  create mode 100644 tools/perf/arch/arm/util/auxtrace.c
>  create mode 100644 tools/perf/arch/arm/util/cs-etm.c
>  create mode 100644 tools/perf/arch/arm/util/cs-etm.h
>  create mode 100644 tools/perf/util/cs-etm.h
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index d3451007718b..4fa03b03bfc5 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1086,6 +1086,9 @@ F:	Documentation/trace/coresight.txt
>  F:	Documentation/devicetree/bindings/arm/coresight.txt
>  F:	Documentation/ABI/testing/sysfs-bus-coresight-devices-*
>  F:	tools/perf/arch/arm/util/pmu.c
> +F:	tools/perf/arch/arm/util/auxtrace.c
> +F:	tools/perf/arch/arm/util/cs_etm.c
> +F:	tools/perf/arch/arm/util/cs_etm.h

File names have changed from having '_' to '-'
Also FWIW there is now tools/perf/util/cs-etm.h

As last time, there is no decoder, which still begs the question: is there
anything you can actually do with the perf.data file?

Otherwise:

Acked-by: Adrian Hunter <adrian.hunter@intel.com>

[toc] | [next] | [standalone]

#1442544 — Re: [PATCH RESEND 2/2] perf tools: adding coresight etm PMU record capabilities

FromMathieu Poirier <mathieu.poirier@linaro.org>
Date2016-07-13 17:20 +0200
SubjectRe: [PATCH RESEND 2/2] perf tools: adding coresight etm PMU record capabilities
Message-ID<rUu78-4Nq-15@gated-at.bofh.it>
In reply to#1442363
On 13 July 2016 at 05:40, Adrian Hunter <adrian.hunter@intel.com> wrote:
> On 17/06/16 20:02, Mathieu Poirier wrote:
>> Coresight ETMs are IP blocks used to perform HW assisted tracing
>> on a CPU core.  This patch introduce the required auxiliary API
>> functions allowing the perf core to interact with a tracer.
>>
>> Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
>> ---
>>  MAINTAINERS                         |   3 +
>>  tools/perf/arch/arm/util/Build      |   2 +-
>>  tools/perf/arch/arm/util/auxtrace.c |  54 ++++
>>  tools/perf/arch/arm/util/cs-etm.c   | 559 ++++++++++++++++++++++++++++++++++++
>>  tools/perf/arch/arm/util/cs-etm.h   |  23 ++
>>  tools/perf/arch/arm64/util/Build    |   4 +
>>  tools/perf/util/auxtrace.c          |   1 +
>>  tools/perf/util/auxtrace.h          |   1 +
>>  tools/perf/util/cs-etm.h            |  74 +++++
>>  9 files changed, 720 insertions(+), 1 deletion(-)
>>  create mode 100644 tools/perf/arch/arm/util/auxtrace.c
>>  create mode 100644 tools/perf/arch/arm/util/cs-etm.c
>>  create mode 100644 tools/perf/arch/arm/util/cs-etm.h
>>  create mode 100644 tools/perf/util/cs-etm.h
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index d3451007718b..4fa03b03bfc5 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -1086,6 +1086,9 @@ F:      Documentation/trace/coresight.txt
>>  F:   Documentation/devicetree/bindings/arm/coresight.txt
>>  F:   Documentation/ABI/testing/sysfs-bus-coresight-devices-*
>>  F:   tools/perf/arch/arm/util/pmu.c
>> +F:   tools/perf/arch/arm/util/auxtrace.c
>> +F:   tools/perf/arch/arm/util/cs_etm.c
>> +F:   tools/perf/arch/arm/util/cs_etm.h
>
> File names have changed from having '_' to '-'

Yes, I will change that.

> Also FWIW there is now tools/perf/util/cs-etm.h

Ok

>
> As last time, there is no decoder, which still begs the question: is there
> anything you can actually do with the perf.data file?

We are proceeding incrementally - this is only the first set in a
suite of upcoming submission.  The whole solution can be found here
[1] and the decoding library, called OpenCSD, here [2].  File
"HOWTO.md" in the latter details how to compile the components in
order to build a working environment.

>
> Otherwise:
>
> Acked-by: Adrian Hunter <adrian.hunter@intel.com>
>

Thanks for your time,
Mathieu

[1]. https://github.com/Linaro/OpenCSD/commits/perf-opencsd-4.7-rc4
[2]. https://github.com/Linaro/OpenCSD/tree/opencsd-0v003

[toc] | [prev] | [next] | [standalone]

#1442545

FromArnaldo Carvalho de Melo <acme@kernel.org>
Date2016-07-13 17:30 +0200
Message-ID<rUugO-4RD-23@gated-at.bofh.it>
In reply to#1442544
Em Wed, Jul 13, 2016 at 09:11:46AM -0600, Mathieu Poirier escreveu:
> [1]. https://github.com/Linaro/OpenCSD/commits/perf-opencsd-4.7-rc4
> [2]. https://github.com/Linaro/OpenCSD/tree/opencsd-0v003

https://github.com/Linaro/OpenCSD/tree/opencsd-0v003 points to:

How the Library is used in Linux perf

The library and additional infrastructure for programming CoreSight
components has been integrated with the standard linux perfomance
analysis tool perf.

See HOWTO.md for details.

------------------------

But the HOWTO.md link gives me a 404 :-\

https://github.com/Linaro/OpenCSD/blob/opencsd-0v003/@ref%20howto_perf

- Arnaldo

[toc] | [prev] | [next] | [standalone]

#1442653 — Re: [PATCH RESEND 2/2] perf tools: adding coresight etm PMU record capabilities

FromMathieu Poirier <mathieu.poirier@linaro.org>
Date2016-07-13 19:30 +0200
SubjectRe: [PATCH RESEND 2/2] perf tools: adding coresight etm PMU record capabilities
Message-ID<rUw8V-67L-7@gated-at.bofh.it>
In reply to#1442545
On 13 July 2016 at 09:15, Arnaldo Carvalho de Melo <acme@kernel.org> wrote:
> Em Wed, Jul 13, 2016 at 09:11:46AM -0600, Mathieu Poirier escreveu:
>> [1]. https://github.com/Linaro/OpenCSD/commits/perf-opencsd-4.7-rc4
>> [2]. https://github.com/Linaro/OpenCSD/tree/opencsd-0v003
>
> https://github.com/Linaro/OpenCSD/tree/opencsd-0v003 points to:
>
> How the Library is used in Linux perf
>
> The library and additional infrastructure for programming CoreSight
> components has been integrated with the standard linux perfomance
> analysis tool perf.
>
> See HOWTO.md for details.
>
> ------------------------
>
> But the HOWTO.md link gives me a 404 :-\
>
> https://github.com/Linaro/OpenCSD/blob/opencsd-0v003/@ref%20howto_perf

Apologies for that.  This here will work:

https://github.com/Linaro/OpenCSD/blob/opencsd-0v003/HOWTO.md

I will do a respin of the patchset with Adrian' comments taken into account.

>
> - Arnaldo

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web