This patchset provides a proposed infrastructure to allow for the automatic
selection of a sink during CoreSight tracing operations.
Currently starting tracing using perf requires a sink selection on the
command line:-
sudo ./perf record -e cs_etm/@tmc_etr0/ --per-thread uname -a
After this set the infrastructure will be able to select a default sink:-
sudo ./perf record -e cs_etm// --per-thread uname -a
This matches with the default operation provided with perf and intelpt.
Where no sink is specified at the start of a trace session, the CoreSight
system will walk the connection graph from the source ETM, to find a
suitable sink using the first encountered highest priority device.
The CoreSight infrastructure is updated to define sink sub_types to
differentiate between sinks with built in buffers (ETB / ETF) - BUFFER type
, and those that use system memory (ETR) - SYSMEM - types.
SYSMEM types are considered higher priority.
When two sinks are found of equal priority, then the closest sink to the
source in terms of connection nodes is chosen.
The automatic sink selection will also operate if an ETM is enabled using
sysfs commands, and no sink is currently enabled.
Applies to Linux coresight/next branch
Changes since v2:
1) changed from explicit set priority value to using the sink_subtype in the
coresight_device as the selection priority value.
2) Added in search depth to the find algorithm to ensure that the closest sink
to the source with the highest priority is chosen.
3) A default sink is cached with the source when the search is first
undertaken, reducing the need for repeat searches for a given source.
Changes since v1:
1) Dropped the device-tree attribute labelling of sinks for selection and
implemented the priority schema preferring first encountered ETR,
after mailing list discussions.
2) Added in sysfs support for auto sink selection.
Mike Leach (6):
coresight: Fix comment in main header file.
coresight: Add default sink selection to CoreSight base
coresight: tmc: Update sink types for default selection.
coresight: etm: perf: Add default sink selection to etm perf
coresight: sysfs: Allow select default sink on source enable.
perf: cs-etm: Allow no CoreSight sink to be specified on command line
.../hwtracing/coresight/coresight-etm-perf.c | 17 +-
drivers/hwtracing/coresight/coresight-priv.h | 2 +
drivers/hwtracing/coresight/coresight-tmc.c | 3 +-
drivers/hwtracing/coresight/coresight.c | 148 +++++++++++++++++-
include/linux/coresight.h | 6 +-
tools/perf/arch/arm/util/cs-etm.c | 6 +-
6 files changed, 172 insertions(+), 10 deletions(-)
--
2.17.1
This series is mainly to add support for replicators
which lose context on removing AMBA clock like on SC7180
SoC where replicator in AOSS domain loses context.
Patch 1 and 2 are minor cleanups to use macros.
Patch 3 and 4 adds support for replicators which loses
context on removing AMBA clock.
Previous version is here - https://lore.kernel.org/patchwork/patch/1239923/
More discussion is found here - https://lore.kernel.org/patchwork/patch/1231182/
There were no patch 1 and patch 2 in v1.
Sai Prakash Ranjan (4):
coresight: replicator: Use CS_AMBA_ID macro for id table
coresight: catu: Use CS_AMBA_ID macro for id table
coresight: replicator: Reset replicator if context is lost
dt-bindings: arm: coresight: Add optional property to replicators
.../devicetree/bindings/arm/coresight.txt | 6 ++
drivers/hwtracing/coresight/coresight-catu.c | 5 +-
.../coresight/coresight-replicator.c | 66 +++++++++++--------
3 files changed, 46 insertions(+), 31 deletions(-)
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation
I updated debian to 0.14.1.
I am also trying out a dgit-based workflow for the package, so the
package uploaded to the archive is also available as
https://browse.dgit.debian.org/libopencsd.git/
You can pull down the latest version with:
From the archive, as tarball+diff: apt
From the archive, as git repo: dgit
From a URL, as tarball+diff: dget
None of this should make much difference to any of you, but this does
make it much easier for people used to a git workflow to contribute to
the debian package if need be. It should also mean that the debian
package matches the git history of upstream much more closely.
In case anyone cares I have chosen the dgit 'merge' workflow about of
about 11 possible schemes (https://wiki.debian.org/GitPackagingSurvey)
(debian never has just one way of doing anything :-)
For those not familiar: dgit is a tool to provide bi-direcitonal
translation between the debian archive format (.dsc, tarball and
debian diff) and git repositories. Package maintainers can continue to
use either system and the git/tarball views of the world are
synchonised at upload.
I've also added a development view of the package on debian's gitlab instance:
https://salsa.debian.org/wookey/libopencsd
I would prefer patches as patches in bug reports, but I have left the
'merge request' thingy turned on so I guess people can use that too
(but I don't know how to drive it yet :-).
I reserve the right to give up with all this complicated fancy stuff
and go back to uploading debianised tarballs, which is quick, simple
and I understand it properly :-) but this stuff should make things a
little more accessible. I'd be interested to hear if anyone actually
finds it useful (nearly everyone on this list will be working with
upstream primarily so rarely/never fiddle with the debian packages, I
presume).
----- Forwarded message from Debian testing watch <noreply(a)release.debian.org> -----
Date: Tue, 19 May 2020 04:39:06 +0000
From: Debian testing watch <noreply(a)release.debian.org>
To: libopencsd(a)packages.debian.org
Subject: libopencsd 0.14.1-1 MIGRATED to testing
FYI: The status of the libopencsd source package
in Debian's testing distribution has changed.
Previous version: 0.14.0-1
Current version: 0.14.1-1
--
This email is automatically generated once a day. As the installation of
new packages into testing happens multiple times a day you will receive
later changes on the next day.
See https://release.debian.org/testing-watch/ for more information.
----- End forwarded message -----
Wookey
--
Principal hats: Linaro, Debian, Wookware, ARM
http://wookware.org/
Hi Poonam,
Please CC the coresight mailing list (as I did) when asking questions
- there is a lot of well informed people on there that can also help
you.
On Thu, 23 Jan 2020 at 22:33, Poonam Aggrwal <poonam.aggrwal(a)nxp.com> wrote:
>
> Hello Mathieu
>
>
>
> Greetings!
>
>
>
> I have started to take a look at the Linux coresight framework, and get this enabled on a NXP ARMv8 device.
>
>
>
> Can you share some documentation on the configs required to be enabled and the device tree nodes?
For V8 we have to reference implementation - ARM Juno and the
dragonboard 410c. I highly recommend purchasing the latter (because
it is very cheap) in order to get an understanding of what a working
coresight system look like. It is much easier to start from a working
example than nothing at all. Other than that the coresight bindings
[1] are full of good examples. I would also have a look at the DT for
Juno [2] and the dragonboard[3]. The HOWTO.md [4] on github is a
really good starting point when you'll get to test things out.
[1]. https://elixir.bootlin.com/linux/latest/source/Documentation/devicetree/bin…
[2]. https://elixir.bootlin.com/linux/latest/source/arch/arm64/boot/dts/arm/juno…
[3]. https://elixir.bootlin.com/linux/latest/source/arch/arm64/boot/dts/qcom/msm…
[4]. https://github.com/Linaro/OpenCSD/blob/master/HOWTO.md
>
> To start I am looking to enable the ARMv8 ETM tracing.
Before going further I advise you to look at the source and sink
configuration on your platform. Up to now we've been working with
configurations where sources share a single sink (N:1 topology).
Newer SoC will have one source per sink (1:1 topology). At this time
only the former is supported by the framework. Supporting 1:1
topologies would require a fair amount of refactoring, something we
haven't had the opportunity to do for lack of HW platform to work
with.
Regards,
Mathieu
>
> Is there a reference which I can check in Linux for device tree and config.
>
>
>
> Many Thanks
>
> Poonam
Hi,
I wanted to reach out to you about the urgent delivery of PPE kits.
We are FDA approved importer and have FDA approved supply for PPE kits and
can deliver most of the items within 10-14 days.
- KN95 Mask
- 3Ply Mask
- Isolated Gowns
- Hazmat Suits
- Goggles
- Gloves
- Contactless Thermometer
- Face Shield
- Swabs
- Hand Sanitizer ( Sea Freight only)
- Rapid Test Kits (FOB China only)
Please check our website: www.ppekits.us
For Catalogs with a lot of products and information please check:
www.ppekits.us/enquiry/
Reach out to us anytime on our toll-free number 855 525 2642
We have offices in the USA, U.K., Australia and we can ship your PPE
worldwide.
We have delivered millions of pieces during COVID-19
Thank you,
--
The Dioz Group of Companies - Emergencyessentials
Office in the UK, USA, Australia.
HQ: 8730 Wilshire Blvd, Penthouse
Beverly Hills, CA 90211
Website: www.ppekits.us
Free Call: 855 525 2642
You may unsubscribe <http://url5466.ppe-kit.site/ls/click?upn=Hc2sKy24YAJb-2BZxdqVW2BaRu3Jk6G-2B…> to stop
receiving our emails.
From: Tingwei Zhang <tingwei(a)codeaurora.org>
On some Qualcomm Technologies Inc. SoCs like SC7180, there
exists a hardware errata where the APSS (Application Processor
SubSystem)/CPU watchdog counter is stopped when ETM register
TRCPDCR.PU=1. Since the ETMs share the same power domain as
that of respective CPU cores, they are powered on when the
CPU core is powered on. So we can disable powering up of the
trace unit after checking for this errata via new property
called "qcom,tupwr-disable".
Signed-off-by: Tingwei Zhang <tingwei(a)codeaurora.org>
Co-developed-by: Sai Prakash Ranjan <saiprakash.ranjan(a)codeaurora.org>
Signed-off-by: Sai Prakash Ranjan <saiprakash.ranjan(a)codeaurora.org>
---
.../devicetree/bindings/arm/coresight.txt | 6 ++++
drivers/hwtracing/coresight/coresight-etm4x.c | 29 ++++++++++++-------
2 files changed, 25 insertions(+), 10 deletions(-)
diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt
index 846f6daae71b..d2030128fe46 100644
--- a/Documentation/devicetree/bindings/arm/coresight.txt
+++ b/Documentation/devicetree/bindings/arm/coresight.txt
@@ -108,6 +108,12 @@ its hardware characteristcs.
* arm,cp14: must be present if the system accesses ETM/PTM management
registers via co-processor 14.
+ * qcom,tupwr-disable: boolean. Indicates that trace unit power up can
+ be disabled on Qualcomm Technologies Inc. systems where ETMs are in
+ the same power domain as their CPU cores. This property is required
+ to identify such systems with hardware errata where the CPU watchdog
+ counter is stopped when TRCPDCR.PU=1.
+
* Optional property for TMC:
* arm,buffer-size: size of contiguous buffer space for TMC ETR
diff --git a/drivers/hwtracing/coresight/coresight-etm4x.c b/drivers/hwtracing/coresight/coresight-etm4x.c
index fb0f5f4f3a91..6886b44f6947 100644
--- a/drivers/hwtracing/coresight/coresight-etm4x.c
+++ b/drivers/hwtracing/coresight/coresight-etm4x.c
@@ -104,6 +104,11 @@ struct etm4_enable_arg {
int rc;
};
+static inline bool etm4_can_disable_tupwr(struct device *dev)
+{
+ return fwnode_property_present(dev_fwnode(dev), "qcom,tupwr-disable");
+}
+
static int etm4_enable_hw(struct etmv4_drvdata *drvdata)
{
int i, rc;
@@ -196,12 +201,14 @@ static int etm4_enable_hw(struct etmv4_drvdata *drvdata)
writel_relaxed(config->vmid_mask0, drvdata->base + TRCVMIDCCTLR0);
writel_relaxed(config->vmid_mask1, drvdata->base + TRCVMIDCCTLR1);
- /*
- * Request to keep the trace unit powered and also
- * emulation of powerdown
- */
- writel_relaxed(readl_relaxed(drvdata->base + TRCPDCR) | TRCPDCR_PU,
- drvdata->base + TRCPDCR);
+ if (!etm4_can_disable_tupwr(etm_dev)) {
+ /*
+ * Request to keep the trace unit powered and also
+ * emulation of powerdown
+ */
+ writel_relaxed(readl_relaxed(drvdata->base + TRCPDCR) | TRCPDCR_PU,
+ drvdata->base + TRCPDCR);
+ }
/* Enable the trace unit */
writel_relaxed(1, drvdata->base + TRCPRGCTLR);
@@ -476,10 +483,12 @@ static void etm4_disable_hw(void *info)
CS_UNLOCK(drvdata->base);
- /* power can be removed from the trace unit now */
- control = readl_relaxed(drvdata->base + TRCPDCR);
- control &= ~TRCPDCR_PU;
- writel_relaxed(control, drvdata->base + TRCPDCR);
+ if (!etm4_can_disable_tupwr(etm_dev)) {
+ /* power can be removed from the trace unit now */
+ control = readl_relaxed(drvdata->base + TRCPDCR);
+ control &= ~TRCPDCR_PU;
+ writel_relaxed(control, drvdata->base + TRCPDCR);
+ }
control = readl_relaxed(drvdata->base + TRCPRGCTLR);
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation