diff options
author | James Clark <[email protected]> | 2023-10-23 14:15:49 +0100 |
---|---|---|
committer | Namhyung Kim <[email protected]> | 2023-10-26 10:58:10 -0700 |
commit | 0b783d2e82d827af73c779e8e8f95d07e992b451 (patch) | |
tree | 8a9242947b871040c853d5f49be17c54ef5c21cb /lib/test-string_helpers.c | |
parent | 4ece2a7e88e1624f52c56d44394e4af1a13b1137 (diff) |
perf tests: test_arm_coresight: Simplify source iteration
There are two reasons to do this, firstly there is a shellcheck warning
in cs_etm_dev_name(), which can be completely deleted. And secondly the
current iteration method doesn't support systems with both ETE and ETM
because it picks one or the other. There isn't a known system with this
configuration, but it could happen in the future.
Iterating over all the sources for each CPU can be done by going through
/sys/bus/event_source/devices/cs_etm/cpu* and following the symlink back
to the Coresight device in /sys/bus/coresight/devices. This will work
whether the device is ETE, ETM or any future name, and is much simpler
and doesn't require any hard coded version numbers
Suggested-by: Suzuki K Poulose <[email protected]>
Signed-off-by: James Clark <[email protected]>
Acked-by: Ian Rogers <[email protected]>
Tested-by: Leo Yan <[email protected]>
Cc: [email protected]
Cc: Kajol Jain <[email protected]>
Cc: Anushree Mathur <[email protected]>
Cc: Tiezhu Yang <[email protected]>
Cc: [email protected]
Cc: [email protected]
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Namhyung Kim <[email protected]>
Diffstat (limited to 'lib/test-string_helpers.c')
0 files changed, 0 insertions, 0 deletions