On Wed, Mar 27, 2019 at 08:40:18AM -0600, Mathieu Poirier wrote:
On Wed, 27 Mar 2019 at 01:52, Leo Yan leo.yan@linaro.org wrote:
On Mon, Mar 25, 2019 at 03:56:16PM -0600, Mathieu Poirier wrote:
This is the second revision of a patchset that adds support for CPU-wide trace scenarios and as such, it is now possible to issue the following commands:
# perf record -e cs_etm/@20070000.etr/ -C 2,3 $COMMAND # perf record -e cs_etm/@20070000.etr/ -a $COMMANDThe solution is designed to work for both 1:1 and N:1 source/sink topologies, though the former hasn't been tested for lack of access to HW.
Most of the changes revolve around allowing more than one event to use a sink when operated from perf. More specifically the first event to use a sink switches it on while the last one is tasked to aggregate traces and switching off the device.
This is the kernel part of the solution, with the user space portion to be released in a later set. All patches (user and kernel) have been rebased on v5.1-rc2 and are hosted here[1]. Everything has been tested on Juno and 410c dragonboard platforms.
FWIW, gave the testing on my Hikey620 board and these patches also work well with below commands:
# perf record -e cs_etm/@f6402000.etf/ -a uname # perf record -e cs_etm/@f6402000.etf/ -C 0,4 uname
# perf record -e cs_etm/@f6404000.etr/ -a uname # perf record -e cs_etm/@f6404000.etr/ -C 1,2,7 uname
Should I add your "Tested-by:" to the patches then?
Yes. Please add my test tag:
Tested-by: Leo Yan leo.yan@linaro.org
Thanks, Leo Yan