On my side I fixed the stability problem I was seeing and pushed a new branch, "perf-opencsd-4.5-rc5", to my repository on g.l.o [1]. The new header is in [2]. I moved the snapshot information from the CPU specific portion to the global header. That way we avoid duplication.
But doing so breaks [3] where CS_ETM_SNAPSHOT now has the wrong index value. Tor, please help me fix that.
Last but not least I generated a new bundle[4] for testing.
Have a look and tell me what you think.
In the mean time I'll continue working on adding the sink in the PMU options from the cmd line.
Thanks, Mathieu
[1]. http://git.linaro.org/git-ro/people/mathieu.poirier/coresight.git [2]. tools/perf/util/cs-etm.h [3]. tools/perf/util/cs-etm.c [4]. http://people.linaro.org/~mathieu.poirier/openCSD/uname.v4.user.feb24.tgz
On 24 February 2016 at 11:36, Jeremiassen, Tor tor@ti.com wrote:
I pushed my latest changes to github.
Main thing is that you now need to define an env variable CSTRACE_PATH with the full path to the ref_trace_decoder directory under OpenCSD before you build – otherwise the trace decoder will be disabled.
e.g., setenv CSTRACE_PATH /home/tor/linaro/git/OpenCSD/ref_trace_decoder
Tor
Tor Jeremiassen, Ph.D.
Senior Member Technical Staff SDO Foundational Tools Texas Instruments Ph: 832 939 2356 13905 University Lane Fax: 832 939 2015 Sugar Land, TX 77479 Email: tor@ti.com
I fixed it and pushed changes to [1] at branch perf-opencsd-4.5-rc5.
It now works for the new layout, it is NOT backward compatible to the old bundles.
Tor
--- Tor Jeremiassen, Ph.D. Senior Member Technical Staff SDO Foundational Tools Texas Instruments Ph: 832 939 2356 13905 University Lane Fax: 832 939 2015 Sugar Land, TX 77479 Email: tor@ti.com
[1]. http://github.com/tor-jeremiassen/OpenCSD.git
-----Original Message----- From: Mathieu Poirier [mailto:mathieu.poirier@linaro.org] Sent: Wednesday, February 24, 2016 1:28 PM To: Jeremiassen, Tor Cc: Mike Leach; coresight@lists.linaro.org Subject: Re: git hub
On my side I fixed the stability problem I was seeing and pushed a new branch, "perf-opencsd-4.5-rc5", to my repository on g.l.o [1]. The new header is in [2]. I moved the snapshot information from the CPU specific portion to the global header. That way we avoid duplication.
But doing so breaks [3] where CS_ETM_SNAPSHOT now has the wrong index value. Tor, please help me fix that.
Last but not least I generated a new bundle[4] for testing.
Have a look and tell me what you think.
In the mean time I'll continue working on adding the sink in the PMU options from the cmd line.
Thanks, Mathieu
[1]. http://git.linaro.org/git-ro/people/mathieu.poirier/coresight.git [2]. tools/perf/util/cs-etm.h [3]. tools/perf/util/cs-etm.c [4]. http://people.linaro.org/~mathieu.poirier/openCSD/uname.v4.user.feb24.tgz
On 24 February 2016 at 11:36, Jeremiassen, Tor tor@ti.com wrote:
I pushed my latest changes to github.
Main thing is that you now need to define an env variable CSTRACE_PATH with the full path to the ref_trace_decoder directory under OpenCSD before you build – otherwise the trace decoder will be disabled.
e.g., setenv CSTRACE_PATH /home/tor/linaro/git/OpenCSD/ref_trace_decoder
Tor
Tor Jeremiassen, Ph.D.
Senior Member Technical Staff SDO Foundational Tools Texas Instruments Ph: 832 939 2356 13905 University Lane Fax: 832 939 2015 Sugar Land, TX 77479 Email: tor@ti.com
On 24 February 2016 at 14:54, Jeremiassen, Tor tor@ti.com wrote:
I fixed it and pushed changes to [1] at branch perf-opencsd-4.5-rc5.
It now works for the new layout, it is NOT backward compatible to the old bundles.
Right, that's the downside.
Thanks for the follow-up, Mathieu
Tor
Tor Jeremiassen, Ph.D. Senior Member Technical Staff SDO Foundational Tools Texas Instruments Ph: 832 939 2356 13905 University Lane Fax: 832 939 2015 Sugar Land, TX 77479 Email: tor@ti.com
[1]. http://github.com/tor-jeremiassen/OpenCSD.git
-----Original Message----- From: Mathieu Poirier [mailto:mathieu.poirier@linaro.org] Sent: Wednesday, February 24, 2016 1:28 PM To: Jeremiassen, Tor Cc: Mike Leach; coresight@lists.linaro.org Subject: Re: git hub
On my side I fixed the stability problem I was seeing and pushed a new branch, "perf-opencsd-4.5-rc5", to my repository on g.l.o [1]. The new header is in [2]. I moved the snapshot information from the CPU specific portion to the global header. That way we avoid duplication.
But doing so breaks [3] where CS_ETM_SNAPSHOT now has the wrong index value. Tor, please help me fix that.
Last but not least I generated a new bundle[4] for testing.
Have a look and tell me what you think.
In the mean time I'll continue working on adding the sink in the PMU options from the cmd line.
Thanks, Mathieu
[1]. http://git.linaro.org/git-ro/people/mathieu.poirier/coresight.git [2]. tools/perf/util/cs-etm.h [3]. tools/perf/util/cs-etm.c [4]. http://people.linaro.org/~mathieu.poirier/openCSD/uname.v4.user.feb24.tgz
On 24 February 2016 at 11:36, Jeremiassen, Tor tor@ti.com wrote:
I pushed my latest changes to github.
Main thing is that you now need to define an env variable CSTRACE_PATH with the full path to the ref_trace_decoder directory under OpenCSD before you build – otherwise the trace decoder will be disabled.
e.g., setenv CSTRACE_PATH /home/tor/linaro/git/OpenCSD/ref_trace_decoder
Tor
Tor Jeremiassen, Ph.D.
Senior Member Technical Staff SDO Foundational Tools Texas Instruments Ph: 832 939 2356 13905 University Lane Fax: 832 939 2015 Sugar Land, TX 77479 Email: tor@ti.com
-
Jeremiassen, Tor -
Mathieu Poirier