Add sysfs node to reset all the sources and sinks. When mltiple sources are enabled, write 1 to reset_source_sink node to disable all sources and sinks.
Signed-off-by: Mao Jinlong quic_jinlmao@quicinc.com --- drivers/hwtracing/coresight/coresight-core.c | 106 +++++++++++++++++-- 1 file changed, 99 insertions(+), 7 deletions(-)
diff --git a/drivers/hwtracing/coresight/coresight-core.c b/drivers/hwtracing/coresight/coresight-core.c index d3bf82c0de1d..06741ed2dee7 100644 --- a/drivers/hwtracing/coresight/coresight-core.c +++ b/drivers/hwtracing/coresight/coresight-core.c @@ -458,6 +458,28 @@ static bool coresight_disable_source(struct coresight_device *csdev) return !csdev->enable; }
+/** + * coresight_get_source - Get the source from the path + * + * @path: The list of devices. + * + * Returns the soruce csdev. + * + */ +static struct coresight_device *coresight_get_source(struct list_head *path) +{ + struct coresight_device *csdev; + + if (!path) + return NULL; + + csdev = list_first_entry(path, struct coresight_node, link)->csdev; + if (csdev->type != CORESIGHT_DEV_TYPE_SOURCE) + return NULL; + + return csdev; +} + /* * coresight_disable_path_from : Disable components in the given path beyond * @nd in the list. If @nd is NULL, all the components, except the SOURCE are @@ -1159,20 +1181,18 @@ int coresight_enable(struct coresight_device *csdev) } EXPORT_SYMBOL_GPL(coresight_enable);
-void coresight_disable(struct coresight_device *csdev) +static void _coresight_disable(struct coresight_device *csdev) { int cpu, ret; struct list_head *path = NULL; u32 hash;
- mutex_lock(&coresight_mutex); - ret = coresight_validate_source(csdev, __func__); if (ret) - goto out; + return;
if (!csdev->enable || !coresight_disable_source(csdev)) - goto out; + return;
switch (csdev->subtype.source_subtype) { case CORESIGHT_DEV_SUBTYPE_SOURCE_PROC: @@ -1187,7 +1207,7 @@ void coresight_disable(struct coresight_device *csdev) path = idr_find(&path_idr, hash); if (path == NULL) { pr_err("Path is not found for %s\n", dev_name(&csdev->dev)); - goto out; + return; } idr_remove(&path_idr, hash); break; @@ -1198,8 +1218,12 @@ void coresight_disable(struct coresight_device *csdev)
coresight_disable_path(path); coresight_release_path(path); +}
-out: +void coresight_disable(struct coresight_device *csdev) +{ + mutex_lock(&coresight_mutex); + _coresight_disable(csdev); mutex_unlock(&coresight_mutex); } EXPORT_SYMBOL_GPL(coresight_disable); @@ -1761,8 +1785,76 @@ char *coresight_alloc_device_name(struct coresight_dev_list *dict, } EXPORT_SYMBOL_GPL(coresight_alloc_device_name);
+/* + * Set the sink active status to false. + */ +static int coresight_reset_sink(struct device *dev, void *data) +{ + struct coresight_device *csdev = to_coresight_device(dev); + + if ((csdev->type == CORESIGHT_DEV_TYPE_SINK || + csdev->type == CORESIGHT_DEV_TYPE_LINKSINK) && + csdev->activated) + csdev->activated = false; + + return 0; +} + +static void coresight_reset_all_sink(void) +{ + bus_for_each_dev(&coresight_bustype, NULL, NULL, coresight_reset_sink); +} + +static ssize_t reset_source_sink_store(struct bus_type *bus, + const char *buf, size_t size) +{ + int id, cpu, ret = 0; + unsigned long val; + struct coresight_device *csdev; + struct list_head *path = NULL; + + ret = kstrtoul(buf, 10, &val); + if (ret) + return ret; + + mutex_lock(&coresight_mutex); + + /* Disable all per cpu sources */ + for_each_present_cpu(cpu) { + path = per_cpu(tracer_path, cpu); + if (path) { + csdev = coresight_get_source(path); + if (!csdev) + continue; + } + _coresight_disable(csdev); + } + + /* Disable all sources which aren't associated with CPU */ + idr_for_each_entry (&path_idr, path, id) { + csdev = coresight_get_source(path); + if (!csdev) + continue; + + _coresight_disable(csdev); + } + /* Reset all activated sinks */ + coresight_reset_all_sink(); + + mutex_unlock(&coresight_mutex); + return size; +} +static BUS_ATTR_WO(reset_source_sink); + +static struct attribute *coresight_reset_source_sink_attrs[] = { + &bus_attr_reset_source_sink.attr, + NULL, +}; +ATTRIBUTE_GROUPS(coresight_reset_source_sink); + struct bus_type coresight_bustype = { .name = "coresight", + .bus_groups = coresight_reset_source_sink_groups, };
static int __init coresight_init(void)
Hi
As this is a sysfs only update - would it not be easier to simply use a shell script to iterate through coresight/devices/ looking for disable_source / disable_sink files and setting those accordingly?
See tools/perf/tests/shell/test_arm_coresight.sh for an example of a script that does similar iteration to test coresight in perf
Regards
Mike
On Wed, 8 Feb 2023 at 11:16, Mao Jinlong quic_jinlmao@quicinc.com wrote:
Add sysfs node to reset all the sources and sinks. When mltiple sources are enabled, write 1 to reset_source_sink node to disable all sources and sinks.
Signed-off-by: Mao Jinlong quic_jinlmao@quicinc.com
drivers/hwtracing/coresight/coresight-core.c | 106 +++++++++++++++++-- 1 file changed, 99 insertions(+), 7 deletions(-)
diff --git a/drivers/hwtracing/coresight/coresight-core.c b/drivers/hwtracing/coresight/coresight-core.c index d3bf82c0de1d..06741ed2dee7 100644 --- a/drivers/hwtracing/coresight/coresight-core.c +++ b/drivers/hwtracing/coresight/coresight-core.c @@ -458,6 +458,28 @@ static bool coresight_disable_source(struct coresight_device *csdev) return !csdev->enable; }
+/**
- coresight_get_source - Get the source from the path
- @path: The list of devices.
- Returns the soruce csdev.
- */
+static struct coresight_device *coresight_get_source(struct list_head *path) +{
struct coresight_device *csdev;
if (!path)
return NULL;
csdev = list_first_entry(path, struct coresight_node, link)->csdev;
if (csdev->type != CORESIGHT_DEV_TYPE_SOURCE)
return NULL;
return csdev;
+}
/*
- coresight_disable_path_from : Disable components in the given path beyond
- @nd in the list. If @nd is NULL, all the components, except the SOURCE are
@@ -1159,20 +1181,18 @@ int coresight_enable(struct coresight_device *csdev) } EXPORT_SYMBOL_GPL(coresight_enable);
-void coresight_disable(struct coresight_device *csdev) +static void _coresight_disable(struct coresight_device *csdev) { int cpu, ret; struct list_head *path = NULL; u32 hash;
mutex_lock(&coresight_mutex);
ret = coresight_validate_source(csdev, __func__); if (ret)
goto out;
return; if (!csdev->enable || !coresight_disable_source(csdev))
goto out;
return; switch (csdev->subtype.source_subtype) { case CORESIGHT_DEV_SUBTYPE_SOURCE_PROC:
@@ -1187,7 +1207,7 @@ void coresight_disable(struct coresight_device *csdev) path = idr_find(&path_idr, hash); if (path == NULL) { pr_err("Path is not found for %s\n", dev_name(&csdev->dev));
goto out;
return; } idr_remove(&path_idr, hash); break;
@@ -1198,8 +1218,12 @@ void coresight_disable(struct coresight_device *csdev)
coresight_disable_path(path); coresight_release_path(path);
+}
-out: +void coresight_disable(struct coresight_device *csdev) +{
mutex_lock(&coresight_mutex);
_coresight_disable(csdev); mutex_unlock(&coresight_mutex);
} EXPORT_SYMBOL_GPL(coresight_disable); @@ -1761,8 +1785,76 @@ char *coresight_alloc_device_name(struct coresight_dev_list *dict, } EXPORT_SYMBOL_GPL(coresight_alloc_device_name);
+/*
- Set the sink active status to false.
- */
+static int coresight_reset_sink(struct device *dev, void *data) +{
struct coresight_device *csdev = to_coresight_device(dev);
if ((csdev->type == CORESIGHT_DEV_TYPE_SINK ||
csdev->type == CORESIGHT_DEV_TYPE_LINKSINK) &&
csdev->activated)
csdev->activated = false;
return 0;
+}
+static void coresight_reset_all_sink(void) +{
bus_for_each_dev(&coresight_bustype, NULL, NULL, coresight_reset_sink);
+}
+static ssize_t reset_source_sink_store(struct bus_type *bus,
const char *buf, size_t size)
+{
int id, cpu, ret = 0;
unsigned long val;
struct coresight_device *csdev;
struct list_head *path = NULL;
ret = kstrtoul(buf, 10, &val);
if (ret)
return ret;
mutex_lock(&coresight_mutex);
/* Disable all per cpu sources */
for_each_present_cpu(cpu) {
path = per_cpu(tracer_path, cpu);
if (path) {
csdev = coresight_get_source(path);
if (!csdev)
continue;
}
_coresight_disable(csdev);
}
/* Disable all sources which aren't associated with CPU */
idr_for_each_entry (&path_idr, path, id) {
csdev = coresight_get_source(path);
if (!csdev)
continue;
_coresight_disable(csdev);
}
/* Reset all activated sinks */
coresight_reset_all_sink();
mutex_unlock(&coresight_mutex);
return size;
+} +static BUS_ATTR_WO(reset_source_sink);
+static struct attribute *coresight_reset_source_sink_attrs[] = {
&bus_attr_reset_source_sink.attr,
NULL,
+}; +ATTRIBUTE_GROUPS(coresight_reset_source_sink);
struct bus_type coresight_bustype = { .name = "coresight",
.bus_groups = coresight_reset_source_sink_groups,
};
static int __init coresight_init(void)
2.39.0
Quick correction - you need to look for enable_source / enable_sink files and disable those that are currently '1'
Mike
On Wed, 8 Feb 2023 at 16:16, Mike Leach mike.leach@linaro.org wrote:
Hi
As this is a sysfs only update - would it not be easier to simply use a shell script to iterate through coresight/devices/ looking for disable_source / disable_sink files and setting those accordingly?
See tools/perf/tests/shell/test_arm_coresight.sh for an example of a script that does similar iteration to test coresight in perf
Regards
Mike
On Wed, 8 Feb 2023 at 11:16, Mao Jinlong quic_jinlmao@quicinc.com wrote:
Add sysfs node to reset all the sources and sinks. When mltiple sources are enabled, write 1 to reset_source_sink node to disable all sources and sinks.
Signed-off-by: Mao Jinlong quic_jinlmao@quicinc.com
drivers/hwtracing/coresight/coresight-core.c | 106 +++++++++++++++++-- 1 file changed, 99 insertions(+), 7 deletions(-)
diff --git a/drivers/hwtracing/coresight/coresight-core.c b/drivers/hwtracing/coresight/coresight-core.c index d3bf82c0de1d..06741ed2dee7 100644 --- a/drivers/hwtracing/coresight/coresight-core.c +++ b/drivers/hwtracing/coresight/coresight-core.c @@ -458,6 +458,28 @@ static bool coresight_disable_source(struct coresight_device *csdev) return !csdev->enable; }
+/**
- coresight_get_source - Get the source from the path
- @path: The list of devices.
- Returns the soruce csdev.
- */
+static struct coresight_device *coresight_get_source(struct list_head *path) +{
struct coresight_device *csdev;
if (!path)
return NULL;
csdev = list_first_entry(path, struct coresight_node, link)->csdev;
if (csdev->type != CORESIGHT_DEV_TYPE_SOURCE)
return NULL;
return csdev;
+}
/*
- coresight_disable_path_from : Disable components in the given path beyond
- @nd in the list. If @nd is NULL, all the components, except the SOURCE are
@@ -1159,20 +1181,18 @@ int coresight_enable(struct coresight_device *csdev) } EXPORT_SYMBOL_GPL(coresight_enable);
-void coresight_disable(struct coresight_device *csdev) +static void _coresight_disable(struct coresight_device *csdev) { int cpu, ret; struct list_head *path = NULL; u32 hash;
mutex_lock(&coresight_mutex);
ret = coresight_validate_source(csdev, __func__); if (ret)
goto out;
return; if (!csdev->enable || !coresight_disable_source(csdev))
goto out;
return; switch (csdev->subtype.source_subtype) { case CORESIGHT_DEV_SUBTYPE_SOURCE_PROC:
@@ -1187,7 +1207,7 @@ void coresight_disable(struct coresight_device *csdev) path = idr_find(&path_idr, hash); if (path == NULL) { pr_err("Path is not found for %s\n", dev_name(&csdev->dev));
goto out;
return; } idr_remove(&path_idr, hash); break;
@@ -1198,8 +1218,12 @@ void coresight_disable(struct coresight_device *csdev)
coresight_disable_path(path); coresight_release_path(path);
+}
-out: +void coresight_disable(struct coresight_device *csdev) +{
mutex_lock(&coresight_mutex);
_coresight_disable(csdev); mutex_unlock(&coresight_mutex);
} EXPORT_SYMBOL_GPL(coresight_disable); @@ -1761,8 +1785,76 @@ char *coresight_alloc_device_name(struct coresight_dev_list *dict, } EXPORT_SYMBOL_GPL(coresight_alloc_device_name);
+/*
- Set the sink active status to false.
- */
+static int coresight_reset_sink(struct device *dev, void *data) +{
struct coresight_device *csdev = to_coresight_device(dev);
if ((csdev->type == CORESIGHT_DEV_TYPE_SINK ||
csdev->type == CORESIGHT_DEV_TYPE_LINKSINK) &&
csdev->activated)
csdev->activated = false;
return 0;
+}
+static void coresight_reset_all_sink(void) +{
bus_for_each_dev(&coresight_bustype, NULL, NULL, coresight_reset_sink);
+}
+static ssize_t reset_source_sink_store(struct bus_type *bus,
const char *buf, size_t size)
+{
int id, cpu, ret = 0;
unsigned long val;
struct coresight_device *csdev;
struct list_head *path = NULL;
ret = kstrtoul(buf, 10, &val);
if (ret)
return ret;
mutex_lock(&coresight_mutex);
/* Disable all per cpu sources */
for_each_present_cpu(cpu) {
path = per_cpu(tracer_path, cpu);
if (path) {
csdev = coresight_get_source(path);
if (!csdev)
continue;
}
_coresight_disable(csdev);
}
/* Disable all sources which aren't associated with CPU */
idr_for_each_entry (&path_idr, path, id) {
csdev = coresight_get_source(path);
if (!csdev)
continue;
_coresight_disable(csdev);
}
/* Reset all activated sinks */
coresight_reset_all_sink();
mutex_unlock(&coresight_mutex);
return size;
+} +static BUS_ATTR_WO(reset_source_sink);
+static struct attribute *coresight_reset_source_sink_attrs[] = {
&bus_attr_reset_source_sink.attr,
NULL,
+}; +ATTRIBUTE_GROUPS(coresight_reset_source_sink);
struct bus_type coresight_bustype = { .name = "coresight",
.bus_groups = coresight_reset_source_sink_groups,
};
static int __init coresight_init(void)
2.39.0
-- Mike Leach Principal Engineer, ARM Ltd. Manchester Design Centre. UK
On 08/02/2023 16:20, Mike Leach wrote:
Quick correction - you need to look for enable_source / enable_sink files and disable those that are currently '1'
Mike
On Wed, 8 Feb 2023 at 16:16, Mike Leach mike.leach@linaro.org wrote:
Hi
As this is a sysfs only update - would it not be easier to simply use a shell script to iterate through coresight/devices/ looking for disable_source / disable_sink files and setting those accordingly?
See tools/perf/tests/shell/test_arm_coresight.sh for an example of a script that does similar iteration to test coresight in perf
+1
Suzuki
On 2/9/2023 12:36 AM, Suzuki K Poulose wrote:
On 08/02/2023 16:20, Mike Leach wrote:
Quick correction - you need to look for enable_source / enable_sink files and disable those that are currently '1'
Mike
On Wed, 8 Feb 2023 at 16:16, Mike Leach mike.leach@linaro.org wrote:
Hi
As this is a sysfs only update - would it not be easier to simply use a shell script to iterate through coresight/devices/ looking for disable_source / disable_sink files and setting those accordingly?
See tools/perf/tests/shell/test_arm_coresight.sh for an example of a script that does similar iteration to test coresight in perf
+1
Suzuki
Hi Mike & Suzuki,
Sometimes user just want to have some quick test from PC with adb commands. It is very easy to reset all sources and sinks' status by command below. echo 1 > /sys/bus/coresight/reset_source_sink
Preparing the script for test is not easy for users who are not familiar with the coresight framework.
Thanks Jinlong Mao
Hi,
On Thu, 9 Feb 2023 at 03:02, Jinlong Mao quic_jinlmao@quicinc.com wrote:
On 2/9/2023 12:36 AM, Suzuki K Poulose wrote:
On 08/02/2023 16:20, Mike Leach wrote:
Quick correction - you need to look for enable_source / enable_sink files and disable those that are currently '1'
Mike
On Wed, 8 Feb 2023 at 16:16, Mike Leach mike.leach@linaro.org wrote:
Hi
As this is a sysfs only update - would it not be easier to simply use a shell script to iterate through coresight/devices/ looking for disable_source / disable_sink files and setting those accordingly?
See tools/perf/tests/shell/test_arm_coresight.sh for an example of a script that does similar iteration to test coresight in perf
+1
Suzuki
Hi Mike & Suzuki,
Sometimes user just want to have some quick test from PC with adb commands. It is very easy to reset all sources and sinks' status by command below. echo 1 > /sys/bus/coresight/reset_source_sink
Users of coresight via sysfs will have to know how to use the coresight infrastructure in order to enable the sources and sinks in the first place - e.g echo 1 > /sys/bus/coresight/devices/tmc_etr0/enable_sink echo 1 > /sys/bus/coresight/devices/etm0/enable_source
Given that they are aware of which sources and sinks they enabled - disabling them should be simple.
Preparing the script for test is not easy for users who are not familiar with the coresight framework.
If there is a genuine use case were a user has opened so many sources on the command line that they need a simpler way of closing them than repeating the enabled commands with an echo 0 > ... then any script could be shipped as part of kernel/tools/coresight or kernel/samples/coresight - they would not have to write it themselves, and just run it from the command line - for example :- ./kernel/tools/coresight/scripts/sysfs_disable_sources_and_sinks.sh
Realistically users will only try out a couple of devices as the usefulness of the sysfs interface is really limited to testing or board bring up. Any complex use with sysfs - as in the coresight tests I mentioned earlier is really going to be done by scripting.
Regards
Mike
Thanks Jinlong Mao
Hi Mike & Suzuki,
On 2/9/2023 10:56 PM, Mike Leach wrote:
Hi,
On Thu, 9 Feb 2023 at 03:02, Jinlong Mao quic_jinlmao@quicinc.com wrote:
On 2/9/2023 12:36 AM, Suzuki K Poulose wrote:
On 08/02/2023 16:20, Mike Leach wrote:
Quick correction - you need to look for enable_source / enable_sink files and disable those that are currently '1'
Mike
On Wed, 8 Feb 2023 at 16:16, Mike Leach mike.leach@linaro.org wrote:
Hi
As this is a sysfs only update - would it not be easier to simply use a shell script to iterate through coresight/devices/ looking for disable_source / disable_sink files and setting those accordingly?
See tools/perf/tests/shell/test_arm_coresight.sh for an example of a script that does similar iteration to test coresight in perf
+1
Suzuki
Hi Mike & Suzuki,
Sometimes user just want to have some quick test from PC with adb commands. It is very easy to reset all sources and sinks' status by command below. echo 1 > /sys/bus/coresight/reset_source_sink
Users of coresight via sysfs will have to know how to use the coresight infrastructure in order to enable the sources and sinks in the first place - e.g echo 1 > /sys/bus/coresight/devices/tmc_etr0/enable_sink echo 1 > /sys/bus/coresight/devices/etm0/enable_source
Given that they are aware of which sources and sinks they enabled - disabling them should be simple.
Preparing the script for test is not easy for users who are not familiar with the coresight framework.
If there is a genuine use case were a user has opened so many sources on the command line that they need a simpler way of closing them than repeating the enabled commands with an echo 0 > ... then any script could be shipped as part of kernel/tools/coresight or kernel/samples/coresight - they would not have to write it themselves, and just run it from the command line - for example :- ./kernel/tools/coresight/scripts/sysfs_disable_sources_and_sinks.sh
Realistically users will only try out a couple of devices as the usefulness of the sysfs interface is really limited to testing or board bring up. Any complex use with sysfs - as in the coresight tests I mentioned earlier is really going to be done by scripting.
Regards
Mike
There is also requirement that reset all the sources and sinks in an user space daemon. For such requirement, I think it is better to use only once sysfs node instead of iterating through coresight/device folder in an user space daemon.
Thanks Jinlong Mao
Thanks Jinlong Mao
On 3/1/2023 11:04 PM, Jinlong Mao wrote:
Hi Mike & Suzuki,
On 2/9/2023 10:56 PM, Mike Leach wrote:
Hi,
On Thu, 9 Feb 2023 at 03:02, Jinlong Mao quic_jinlmao@quicinc.com wrote:
On 2/9/2023 12:36 AM, Suzuki K Poulose wrote:
On 08/02/2023 16:20, Mike Leach wrote:
Quick correction - you need to look for enable_source / enable_sink files and disable those that are currently '1'
Mike
On Wed, 8 Feb 2023 at 16:16, Mike Leach mike.leach@linaro.org wrote:
Hi
As this is a sysfs only update - would it not be easier to simply use a shell script to iterate through coresight/devices/ looking for disable_source / disable_sink files and setting those accordingly?
See tools/perf/tests/shell/test_arm_coresight.sh for an example of a script that does similar iteration to test coresight in perf
+1
Suzuki
Hi Mike & Suzuki,
Sometimes user just want to have some quick test from PC with adb commands. It is very easy to reset all sources and sinks' status by command below. echo 1 > /sys/bus/coresight/reset_source_sink
Users of coresight via sysfs will have to know how to use the coresight infrastructure in order to enable the sources and sinks in the first place - e.g echo 1 > /sys/bus/coresight/devices/tmc_etr0/enable_sink echo 1 > /sys/bus/coresight/devices/etm0/enable_source
Given that they are aware of which sources and sinks they enabled - disabling them should be simple.
Preparing the script for test is not easy for users who are not familiar with the coresight framework.
If there is a genuine use case were a user has opened so many sources on the command line that they need a simpler way of closing them than repeating the enabled commands with an echo 0 > ... then any script could be shipped as part of kernel/tools/coresight or kernel/samples/coresight - they would not have to write it themselves, and just run it from the command line - for example :- ./kernel/tools/coresight/scripts/sysfs_disable_sources_and_sinks.sh
Realistically users will only try out a couple of devices as the usefulness of the sysfs interface is really limited to testing or board bring up. Any complex use with sysfs - as in the coresight tests I mentioned earlier is really going to be done by scripting.
Regards
Mike
There is also requirement that reset all the sources and sinks in an user space daemon. For such requirement, I think it is better to use only once sysfs node instead of iterating through coresight/device folder in an user space daemon.
Hi Mike & Suzuki,
In our internal build, there is binary executable which can configure coresight source/sink. Before running the case, it will disable all the sources and sinks by writing reset_source_sink node to avoid any other source packet's impact.
Thanks Jinlong Mao
Thanks Jinlong Mao
Thanks Jinlong Mao
CoreSight mailing list -- coresight@lists.linaro.org To unsubscribe send an email to coresight-leave@lists.linaro.org
On 17/03/2023 06:18, Jinlong Mao wrote:
On 3/1/2023 11:04 PM, Jinlong Mao wrote:
Hi Mike & Suzuki,
On 2/9/2023 10:56 PM, Mike Leach wrote:
Hi,
On Thu, 9 Feb 2023 at 03:02, Jinlong Mao quic_jinlmao@quicinc.com wrote:
On 2/9/2023 12:36 AM, Suzuki K Poulose wrote:
On 08/02/2023 16:20, Mike Leach wrote:
Quick correction - you need to look for enable_source / enable_sink files and disable those that are currently '1'
Mike
On Wed, 8 Feb 2023 at 16:16, Mike Leach mike.leach@linaro.org wrote: > Hi > > As this is a sysfs only update - would it not be easier to simply > use > a shell script to iterate through coresight/devices/ looking for > disable_source / disable_sink files and setting those accordingly? > > See tools/perf/tests/shell/test_arm_coresight.sh for an example of a > script that does similar iteration to test coresight in perf >
+1
Suzuki
Hi Mike & Suzuki,
Sometimes user just want to have some quick test from PC with adb commands. It is very easy to reset all sources and sinks' status by command below. echo 1 > /sys/bus/coresight/reset_source_sink
Users of coresight via sysfs will have to know how to use the coresight infrastructure in order to enable the sources and sinks in the first place - e.g echo 1 > /sys/bus/coresight/devices/tmc_etr0/enable_sink echo 1 > /sys/bus/coresight/devices/etm0/enable_source
Given that they are aware of which sources and sinks they enabled - disabling them should be simple.
Preparing the script for test is not easy for users who are not familiar with the coresight framework.
If there is a genuine use case were a user has opened so many sources on the command line that they need a simpler way of closing them than repeating the enabled commands with an echo 0 > ... then any script could be shipped as part of kernel/tools/coresight or kernel/samples/coresight - they would not have to write it themselves, and just run it from the command line - for example :- ./kernel/tools/coresight/scripts/sysfs_disable_sources_and_sinks.sh
Realistically users will only try out a couple of devices as the usefulness of the sysfs interface is really limited to testing or board bring up. Any complex use with sysfs - as in the coresight tests I mentioned earlier is really going to be done by scripting.
Regards
Mike
There is also requirement that reset all the sources and sinks in an user space daemon. For such requirement, I think it is better to use only once sysfs node instead of iterating through coresight/device folder in an user space daemon.
Hi Mike & Suzuki,
In our internal build, there is binary executable which can configure coresight source/sink. Before running the case, it will disable all the sources and sinks by writing reset_source_sink node to avoid any other source packet's impact.
How does that justify this patch ? Your internal build depending on something is your code. It looks like there is more to these patches than what you are disclosing. e.g., with the dummy device series.
Please could you paint a complete picture and call out the dependencies / requirements for what you are trying to achieve ?
Kind regards Suzuki
Thanks Jinlong Mao
Thanks Jinlong Mao
Thanks Jinlong Mao
CoreSight mailing list -- coresight@lists.linaro.org To unsubscribe send an email to coresight-leave@lists.linaro.org