Good morning,
On Thu, Feb 25, 2021 at 10:42:58AM +0100, Arnd Bergmann wrote:
> From: Arnd Bergmann
>
> clang-12 fails to build the etm4x driver with -fsanitize=array-bounds:
>
> :1:7: error: expected constant expression in '.inst' directive
> .inst (0xd520|2) << 19) | ((1) << 16) |
void * and
> use a local variable with the correct type so that everything works
> properly. With this, we can remove casting to rproc_handle_resource_t
> for these functions.
>
> Signed-off-by: Jindong Yue
> Reviewed-by: Peng Fan
> Reviewed-by: Sami Tolvanen
Revie
pru_rproc_debug_ss_set, "%llu\n");
Reviewed-by: Mathieu Poirier
>
> /*
> --
> 1.8.3.1
>
On Thu, Jan 28, 2021 at 05:09:30PM +, Mike Leach wrote:
> Loaded coresight configurations are registered in the cs_etm\cs_config sub
> directory. This extends the etm-perf code to handle these registrations,
> and the cs_syscfg driver to perform the registration on load.
>
> Signed-off-by:
From: Suzuki K Poulose
The PID of the task could be traced as VMID when the kernel is running
at EL2. Teach the decoder to look for VMID when the CONTEXTIDR (Arm32)
or CONTEXTIDR_EL1 (Arm64) is invalid but we have a valid VMID.
Cc: Mike Leach
Cc: Mathieu Poirier
Cc: Al Grant
Signed-off
ot;contextid2"; otherwise, it
dynamically sets config based on PMU format "contextid".
Cc: Mike Leach
Cc: Mathieu Poirier
Cc: Al Grant
Signed-off-by: Suzuki K Poulose
Co-developed-by: Leo Yan
Signed-off-by: Leo Yan
Reviewed-by: Mike Leach
Reviewed-by: Mathieu Poirier
Link: https://lore.
From: Leo Yan
To get the changes in the commit:
"coresight: etm-perf: Clarify comment on perf options".
Signed-off-by: Leo Yan
Reviewed-by: Suzuki K Poulose
Reviewed-by: Mathieu Poirier
Link: https://lore.kernel.org/r/20210213113220.292229-2-leo@linaro.org
---
tools/inc
gned-off-by: Leo Yan
Reviewed-by: Mike Leach
Reviewed-by: Mathieu Poirier
Link: https://lore.kernel.org/r/20210213113220.292229-3-leo@linaro.org
---
tools/perf/arch/arm/util/cs-etm.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/perf/arch/arm/util/cs-etm.
From: Mike Leach
The current fixed metadata version format (version 0), means that adding
metadata parameter items renders files from a previous version of perf
unreadable. Per CPU parameters appear in a fixed order, but there is no
field to indicate the number of ETM parameters per CPU.
This
From: Leo Yan
This patch adds helper function cs_etm__get_pid_fmt(), by passing
parameter "traceID", it returns the PID format.
Signed-off-by: Leo Yan
Reviewed-by: Mathieu Poirier
Reviewed-by: Suzuki K Poulose
Link: https://lore.kernel.org/r/20210213113220.292229-5-leo@
Good day Arnaldo,
I noticed there is a couple of patchsets [1][2] that haven't made it
to your tree for the coming v5.12 cycle. Do you think that can still
be done?
I tallied the patches here to make it easier for you to pick up.
Applies cleanly on perf/core (84b7725536d8)
Thanks,
Mathieu
Good day James,
I have received your patchset and added it to my queue. On the flip side it
will be 3 to 4 weeks (from today) before I get a chance to look at it. As such
I suggest you don't wait on me before addressing the issues found by Leo.
Thanks,
Mathieu
On Fri, Feb 12, 2021 at
-by: Mathieu Poirier
---
New for V6:
- Double free of the cached table has been fixed.
- rproc_reset_loaded_rsc_table() has seen a complete re-write.
- rproc_stop() now calls rproc_reset_loaded_rsc_table() rather than
dealing with the cached. This allows to properly shutdown a
remote processor
The panic handler operation of registered remote processors
should also be called when remote processors have been
attached to.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_core.c | 6 +-
1 file changed, 5 insertions
This patch takes into account scenarios where a remote processor
has been attached to when receiving a "start" command from sysfs.
As with the case with the running state, the command can't be
carried out if the remote processor is already in operation.
Signed-off-by: Mathieu Poirier
Allow a remote processor that was started by another entity to be
switched off by the remoteproc core. For that to happen a
rproc::ops::stop() operation needs to be available.
Signed-off-by: Mathieu Poirier
---
New for V6:
- Removed state check in rproc_shutdown() as it is already done
Refactor function rproc_del() and rproc_cdev_release() to take
into account the current state of the remote processor when choosing
the state to transition to.
Signed-off-by: Mathieu Poirier
---
New for V6:
- The RPROC_RUNNING -> RPROC_DETACHED transition is no longer permitted.
to av
From: Arnaud POULIQUEN
Some actions such as memory resources reallocation are needed when
trying to reattach a co-processor. Use the prepare() operation for
these actions.
Co-developed-by: Mathieu Poirier
Signed-off-by: Mathieu Poirier
Signed-off-by: Arnaud POULIQUEN
---
drivers/remoteproc
te processor while
the latter is kept operating.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
---
drivers/remoteproc/remoteproc_core.c | 30
1 file changed, 30 insertions(+)
diff --git a/drivers/remoteproc/remoteproc_core.c
b/drivers/remoteproc/remoteproc_
Introduce function rproc_detach() to enable the remoteproc
core to release the resources associated with a remote processor
without stopping its operation.
Signed-off-by: Mathieu Poirier
---
New for V6:
- Checking for rproc->state has been removed. They have been moved to
calling functi
This patch introduces the capability to detach a remote processor
that has been attached by the remoteproc core. For that to happen
a rproc::ops::detach() operation needs to be available.
Signed-off-by: Mathieu Poirier
---
New for V6:
- The RPROC_RUNNING -> RPROC_DETACHED transit
Add an new detach() operation in order to support scenarios where
the remoteproc core is going away but the remote processor is
kept operating. This could be the case when the system is
rebooted or when the platform driver is removed.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_core.c | 20 +---
drivers/remoteproc/remoteproc_sysfs.c | 5 +
include/linux/remoteproc.h| 2 --
3 files changed, 2 insertions(+), 25 deletions(-)
diff --git
needs to be available
at a later time than the platform driver's probe() function.
Signed-off-by: Mathieu Poirier
---
drivers/remoteproc/stm32_rproc.c | 141 +++
1 file changed, 68 insertions(+), 73 deletions(-)
diff --git a/drivers/remoteproc/stm32_rproc.c b/drivers
Add a new get_loaded_rsc_table() operation in order to support
scenarios where the remoteproc core has booted a remote processor
and detaches from it. When re-attaching to the remote processor,
the core needs to know where the resource table has been placed
in memory.
Signed-off-by: Mathieu
Rename function rproc_actuate() to rproc_attach(). That way it is
easy to understand that it does the opposite of rproc_detach().
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_core.c | 8
1 file changed, 4
Add a new RPROC_ATTACHED state to take into account scenarios
where the remoteproc core needs to attach to a remote processor
that is booted by another entity.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_sysfs.c | 1
.12-rc1 when it
comes out next week.
Thanks,
Mathieu
Arnaud POULIQUEN (1):
remoteproc: stm32: Move memory parsing to rproc_ops
Mathieu Poirier (15):
remoteproc: Remove useless check in rproc_del()
remoteproc: Rename function rproc_actuate()
remoteproc: Add new RPROC_ATTACHED state
remotep
when in fact it
no longer exist.
Invariably calling rproc_shutdown() is fine since it will return
immediately if the remote processor has already been switched
off.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_core.c | 4
On Tue, 23 Feb 2021 at 09:49, Sami Tolvanen wrote:
>
> On Tue, Feb 23, 2021 at 8:41 AM Mathieu Poirier
> wrote:
> >
> > On Mon, 22 Feb 2021 at 15:48, Sami Tolvanen wrote:
> > >
> > > Hi,
> > >
> > > On Sun, Feb 21, 2021 at 11:18 PM Jin
On Mon, 22 Feb 2021 at 15:48, Sami Tolvanen wrote:
>
> Hi,
>
> On Sun, Feb 21, 2021 at 11:18 PM Jindong Yue wrote:
> >
> > There are four different callback functions that are used for the
> > rproc_handle_resource_t callback that all have different second
> > parameter types.
> >
> >
Good morning,
I have received your patchset but currently don't see having the time
to look at it before the middle of March.
Thanks,
Mathieu
On Tue, 23 Feb 2021 at 08:44, Ben Levinsky wrote:
>
> R5 is included in Xilinx Zynq UltraScale MPSoC so by adding this
> remotproc driver, we can boot
On Thu, Jan 28, 2021 at 05:09:29PM +, Mike Leach wrote:
> Adds a set of generic support functions that allow devices to set and save
> features values on the device, and enable and disable configurations.
>
> Additional functions for other common operations including feature
> reset.
>
>
On Thu, Jan 28, 2021 at 05:09:28PM +, Mike Leach wrote:
> API for individual devices to register with the syscfg management
> system is added.
>
> Devices register with matching information, and any features or
> configurations that match will be loaded into the device.
>
> The feature and
On Thu, Jan 28, 2021 at 05:09:27PM +, Mike Leach wrote:
> Creates an system management API to allow complex configurations and
> features to be programmed into a CoreSight infrastructure.
>
> A feature is defined as a programming set for a device or class of
> devices.
>
> A configuration is
Hi Mike,
On Thu, Jan 28, 2021 at 05:09:28PM +, Mike Leach wrote:
> API for individual devices to register with the syscfg management
> system is added.
>
> Devices register with matching information, and any features or
> configurations that match will be loaded into the device.
>
> The
[...]
> +/**
> + * List entry for Coresight devices that are registered as supporting complex
> + * config operations.
> + *
> + * @csdev: The registered device.
> + * @match_info: The matching type information.
> + * @ops: Operations supported by the registered device.
> + * @item:list
On Thu, Jan 28, 2021 at 05:09:27PM +, Mike Leach wrote:
> Creates an system management API to allow complex configurations and
> features to be programmed into a CoreSight infrastructure.
>
> A feature is defined as a programming set for a device or class of
> devices.
>
> A configuration is
aid out in
configfs look good to me. Things can still change at some point in the future
but for now:
Reviewed-by: Mathieu Poirier
>
> diff --git a/Documentation/trace/coresight/coresight-config.rst
> b/Documentation/trace/coresight/coresight-config.rst
> new file mode 10064
mbers of parameters than in use at the
> time perf was built.
>
> Signed-off-by: Mike Leach
> Reviewed-by: Leo Yan
> Tested-by: Leo Yan
>
Reviewed-by: Mathieu Poirier
Please rebase this patch on 5.12-rc1 when it comes out and resend to Arnaldo.
Thanks,
Mathieu
>
On Wed, Feb 17, 2021 at 09:34:13PM +, Ben Levinsky wrote:
> 1 more for the pnode_id comment
>
[...]
> > > > > +static int parse_tcm_banks(struct rproc *rproc)
> > > > > +{
> > > > > + int i, num_banks;
> > > > > + struct zynqmp_r5_rproc *z_rproc = rproc->priv;
> > > > > + struct
On Wed, Feb 17, 2021 at 07:11:37PM +, Ben Levinsky wrote:
> Hi Mathieu,
>
> I worked on this further and had 1 more comment on the prepare/unprepare for
> TCM banks below
>
[...]
> > > > + */
> > > > +static int tcm_mem_release(struct rproc *rproc, struct rproc_mem_entry
> > > *mem)
> > >
On Mon, Feb 15, 2021 at 02:10:10PM +0100, Arnaud POULIQUEN wrote:
> Hi Mathieu,
>
> On 2/12/21 12:46 AM, Mathieu Poirier wrote:
> > Add a new get_loaded_rsc_table() operation in order to support
> > scenarios where the remoteproc core has booted a remote processor
> >
On Tue, Feb 16, 2021 at 04:10:18PM +0530, Anshuman Khandual wrote:
>
>
> On 2/5/21 12:04 AM, Mathieu Poirier wrote:
> > On Thu, Jan 28, 2021 at 09:16:34AM +, Suzuki K Poulose wrote:
> >> On 1/27/21 8:55 AM, Anshuman Khandual wrote:
> >>> Add suppor
On Mon, Feb 15, 2021 at 05:58:37PM +, Mike Leach wrote:
> Hi Mathieu,
>
> On Mon, 15 Feb 2021 at 16:56, Mathieu Poirier
> wrote:
> >
> > On Mon, Feb 15, 2021 at 04:27:26PM +, Mike Leach wrote:
> > > HI Anshuman
> > >
> > > On Wed, 2
On Sun, Jan 10, 2021 at 06:02:50PM -0800, Ben Levinsky wrote:
> R5 is included in Xilinx Zynq UltraScale MPSoC so by adding this
> remotproc driver, we can boot the R5 sub-system in two different
> configurations -
> * Split
> * Lockstep
>
> The Xilinx R5 Remoteproc Driver boots the
Good day,
I have started to review this set - as with previous revisions comments will
come over several days.
On Sun, Jan 10, 2021 at 06:02:50PM -0800, Ben Levinsky wrote:
> R5 is included in Xilinx Zynq UltraScale MPSoC so by adding this
> remotproc driver, we can boot the R5 sub-system in two
t; > percpu source and sink devices which are exclusively linked and dependent.
> > Build the path directly and skip connection scanning for such devices.
> >
> > Cc: Mathieu Poirier
> > Cc: Mike Leach
> > Cc: Suzuki K Poulose
> > Signed-off-
On Mon, Feb 15, 2021 at 10:26:38AM +, Mike Leach wrote:
> Reviewed-by: Mike Leach
I will pick this up when 5.12-rc1 comes out.
Thanks,
Mathieu
>
> On Sat, 13 Feb 2021 at 11:28, Sai Prakash Ranjan
> wrote:
> >
> > Add ETM PID for Cortex-A78 to the list of supported ETMs.
> >
> >
.P. The TRBE can also generate a CPU
> private interrupt (PPI) on address translation errors and when the buffer
> is full. Overall implementation here is inspired from the Arm SPE driver.
>
> Cc: Mathieu Poirier
> Cc: Mike Leach
> Cc: Suzuki K Poulose
> Signed-off-by: Anshuman Kha
On Fri, Feb 12, 2021 at 11:13:01AM +0530, Anshuman Khandual wrote:
>
>
> On 2/11/21 12:30 AM, Mathieu Poirier wrote:
> > On Wed, Jan 27, 2021 at 02:25:35PM +0530, Anshuman Khandual wrote:
> >> Trace Buffer Extension (TRBE) implements a trace buffer per CPU which is
> &
[...]
> >
> >
> >> + if (nr_pages < 2)
> >> + return NULL;
> >> +
> >> + buf = kzalloc_node(sizeof(*buf), GFP_KERNEL, trbe_alloc_node(event));
> >> + if (IS_ERR(buf))
> >> + return ERR_PTR(-ENOMEM);
> >> +
> >> + pglist = kcalloc(nr_pages, sizeof(*pglist), GFP_KERNEL);
>
This patch introduces the capability to detach a remote processor
that has been attached to or booted by the remoteproc core. For
that to happen a rproc::ops::detach() operation need to be
available.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
Refactor function rproc_del() and rproc_cdev_release() to take
into account the policy specified in the device tree.
Signed-off-by: Mathieu Poirier
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_cdev.c | 18 +++---
drivers/remoteproc/remoteproc_core.c | 36
The panic handler operation of registered remote processors
should also be called when remote processors have been
attached to.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_core.c | 6 +-
1 file changed, 5 insertions
-by: Arnaud POULIQUEN
Signed-off-by: Mathieu Poirier
---
drivers/remoteproc/remoteproc_core.c | 70 ++
drivers/remoteproc/remoteproc_elf_loader.c | 24 +++-
include/linux/remoteproc.h | 3 +
3 files changed, 95 insertions(+), 2 deletions(-)
diff --git
This patch takes into account scenarios where a remote processor
has been attached to when receiving a "start" command from sysfs.
As with the "running" case, the command can't be carried out if the
remote processor is already in operation.
Signed-off-by: Mathieu Poirier
This patch introduces the capability to stop a remote processor
that has been attached to by the remoteproc core. For that to
happen a rproc::ops::stop() operation need to be available.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc
Introduce function rproc_detach() to enable the remoteproc
core to release the resources associated with a remote processor
without stopping its operation.
Signed-off-by: Mathieu Poirier
---
New for V5:
- Fixed comment about rproc_actuate() that no longer exists.
- Added call
Add a return value to function rproc_shutdown() in order to
properly deal with error conditions that may occur.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_core.c | 19 ++-
include/linux/remoteproc.h
Add an new detach() operation in order to support scenarios where
the remoteproc core is going away but the remote processor is
kept operating. This could be the case when the system is
rebooted or when the platform driver is removed.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
te processor while
the latter is kept operating.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
---
New for V5:
- Removed fancy error recovery when ops->detach() fails to replicate what is
done in rproc->stop().
---
drivers/remoteproc/remoteproc_core.c | 30 ++
From: Arnaud POULIQUEN
Some actions such as memory resources reallocation are needed when
trying to reattach a co-processor. Use the prepare() operation for
these actions.
Co-developed-by: Mathieu Poirier
Signed-off-by: Mathieu Poirier
Signed-off-by: Arnaud POULIQUEN
---
drivers/remoteproc
Add a new RPROC_ATTACHED state to take into account scenarios
where the remoteproc core needs to attach to a remote processor
that is booted by another entity.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_sysfs.c | 1
needs to be available
at a later time than the platform driver's probe() function.
Signed-off-by: Mathieu Poirier
---
New for V5:
- stm32_rproc_get_loaded_rsc_table() now returns a resource table pointer.
---
drivers/remoteproc/stm32_rproc.c | 141 +++
1 file changed
Add a new get_loaded_rsc_table() operation in order to support
scenarios where the remoteproc core has booted a remote processor
and detaches from it. When re-attaching to the remote processor,
the core needs to know where the resource table has been placed
in memory.
Signed-off-by: Mathieu
-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_core.c | 20 +---
drivers/remoteproc/remoteproc_sysfs.c | 5 +
include/linux/remoteproc.h| 2 --
3 files changed, 2 insertions(+), 25 deletions(-)
diff --git
when in fact it
no longer exist.
Invariably calling rproc_shutdown() is fine since it will return
immediately if the remote processor has already been switched
off.
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_core.c | 4
The state of the remote processor may have changed between the
time a call to rproc_shutdown() was made and the time it is
executed. To avoid moving forward with an operation that may
have been cancelled, recheck while holding the mutex.
Cc:
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
(1):
remoteproc: stm32: Move memory parsing to rproc_ops
Mathieu Poirier (18):
dt-bindings: remoteproc: Add bindind to support autonomous processors
remoteproc: Re-check state in rproc_shutdown()
remoteproc: Remove useless check in rproc_del()
remoteproc: Rename function rproc_actuate
Rename function rproc_actuate() to rproc_attach(). That way it is
easy to understand that it does the opposite of rproc_detach().
Signed-off-by: Mathieu Poirier
Reviewed-by: Peng Fan
Reviewed-by: Arnaud Pouliquen
---
drivers/remoteproc/remoteproc_core.c | 8
1 file changed, 4
if "autonomous-on-core-reboot" is specified in the remote
processor DT node, the remoteproc core will detach the remote processor
rather than switching it off.
Signed-off-by: Mathieu Poirier
Reviewed-by: Rob Herring
---
.../bindings/remoteproc/remoteproc-core.yaml | 27 +
.P. The TRBE can also generate a CPU
> private interrupt (PPI) on address translation errors and when the buffer
> is full. Overall implementation here is inspired from the Arm SPE driver.
>
> Cc: Mathieu Poirier
> Cc: Mike Leach
> Cc: Suzuki K Poulose
> Signed-off-by: Anshuman Kha
.
Signed-off-by: Leo Yan
Reviewed-by: Suzuki K Poulose
Reviewed-by: Mike Leach
Message-Id: <20210206150833.42120-9-leo@linaro.org>
Signed-off-by: Mathieu Poirier
---
Documentation/trace/coresight/coresight.rst | 32 +
1 file changed, 32 insertions(+)
diff
as for the option that enables PID
tracing. I.e,
contextid == contextid1, on EL1 kernel.
contextid == contextid2, on EL2 kernel.
Cc: Mathieu Poirier
Cc: Al Grant
Cc: Mike Leach
Cc: Leo Yan
Signed-off-by: Suzuki K Poulose
[ Added two config fo
Poulose
Message-Id: <20210206150833.42120-2-leo@linaro.org>
Signed-off-by: Mathieu Poirier
---
.../hwtracing/coresight/coresight-etm-perf.c| 5 -
include/linux/coresight-pmu.h | 17 -
2 files changed, 16 insertions(+), 6 deletions(-)
diff
Good morning,
My previous submission had the wrong baseline and as such was missing a patch.
This set applies properly on [1].
Thanks for the patience,
Mathieu
[1]. 48139bad913d ACRN: update MAINTAINERS: mailing list is subscribers-only
Leo Yan (2):
coresight: etm-perf: Clarify comment on
.P. The TRBE can also generate a CPU
> private interrupt (PPI) on address translation errors and when the buffer
> is full. Overall implementation here is inspired from the Arm SPE driver.
>
> Cc: Mathieu Poirier
> Cc: Mike Leach
> Cc: Suzuki K Poulose
> Signed-off-by: Anshuman Kha
On Wed, Feb 10, 2021 at 09:42:29AM +0530, Anshuman Khandual wrote:
>
>
> On 2/9/21 11:09 PM, Mathieu Poirier wrote:
> > On Fri, Feb 05, 2021 at 10:53:30AM -0700, Mathieu Poirier wrote:
> >> On Wed, Jan 27, 2021 at 02:25:35PM +0530, Anshuman Khandual wrote:
> >&
.
Signed-off-by: Leo Yan
Reviewed-by: Suzuki K Poulose
Reviewed-by: Mike Leach
Message-Id: <20210206150833.42120-9-leo@linaro.org>
Signed-off-by: Mathieu Poirier
---
Documentation/trace/coresight/coresight.rst | 32 +
1 file changed, 32 insertions(+)
diff
as for the option that enables PID
tracing. I.e,
contextid == contextid1, on EL1 kernel.
contextid == contextid2, on EL2 kernel.
Cc: Mathieu Poirier
Cc: Al Grant
Cc: Mike Leach
Cc: Leo Yan
Signed-off-by: Suzuki K Poulose
[ Added two config fo
Hi Greg,
Please see if you can add these 2 patches to your 5.12 tally. No worries
if you think it is too close to the merge window, I'll simply queue them
for the next one.
Thanks,
Mathieu
Leo Yan (1):
Documentation: coresight: Add PID tracing description
Suzuki K Poulose (1):
coresight:
On Fri, Feb 05, 2021 at 10:53:30AM -0700, Mathieu Poirier wrote:
> On Wed, Jan 27, 2021 at 02:25:35PM +0530, Anshuman Khandual wrote:
> > Trace Buffer Extension (TRBE) implements a trace buffer per CPU which is
> > accessible via the system registers. The TRBE supports diffe
On Wed, Feb 03, 2021 at 08:58:15AM +0100, Arnaud POULIQUEN wrote:
>
>
> On 2/2/21 11:42 PM, Mathieu Poirier wrote:
> > On Tue, Feb 02, 2021 at 09:54:13AM +0100, Arnaud POULIQUEN wrote:
> >>
> >>
> >> On 2/2/21 1:49 AM, Mathieu Poirier wrote:
> >>
On Sat, Feb 06, 2021 at 11:08:33PM +0800, Leo Yan wrote:
> After support the PID tracing for the kernel in EL1 or EL2, the usage
> gets more complicated.
>
> This patch gives description for the PMU formats of contextID configs,
> this can help users to understand how to control the knobs for PID
> +#define ETM_SET_OPT_CTXTID (1 << 0)
> +#define ETM_SET_OPT_TS (1 << 1)
> +#define ETM_SET_OPT_MASK (ETM_SET_OPT_CTXTID | ETM_SET_OPT_TS)
> +
I would much rather see this fixed with the BIT() macro as it is done in the
rest of this set than defining new constant.
ve a valid VMID.
>
> Cc: Mike Leach
> Cc: Mathieu Poirier
> Cc: Al Grant
> Signed-off-by: Suzuki K Poulose
> Co-developed-by: Leo Yan
> Signed-off-by: Leo Yan
Reviewed-by: Mathieu Poirier
> ---
> .../perf/util/cs-etm-decoder/cs-etm-decoder.c | 38
On Sat, Feb 06, 2021 at 11:08:31PM +0800, Leo Yan wrote:
> This patch adds helper function cs_etm__get_pid_fmt(), by passing
> parameter "traceID", it returns the PID format.
>
> Signed-off-by: Leo Yan
Reviewed-by: Mathieu Poirier
> ---
> t
cified any configs
> from "contextid", "contextid1" or "contextid2"; otherwise, it
> dynamically sets config based on PMU format "contextid".
>
> Cc: Mike Leach
> Cc: Mathieu Poirier
> Cc: Al Grant
> Signed-off-by: Suzuki K Poulose
On Sat, Feb 06, 2021 at 11:08:27PM +0800, Leo Yan wrote:
> To get the changes in the commit:
>
> "coresight: etm-perf: Clarify comment on perf options".
>
> Signed-off-by: Leo Yan
> Reviewed-by: Suzuki K Poulose
Reviewed-by: Mathieu Poirier
> ---
> t
apply/patch:76: new blank line at EOF.
+
warning: 1 line adds whitespace errors.
> Cc: Mathieu Poirier
> Cc: Mike Leach
> Cc: Suzuki K Poulose
> Signed-off-by: Anshuman Khandual
> ---
> Changes in V3:
>
> - Added new DT bindings document TRBE.yaml
> - Changed TRB
are not present for certain
> > percpu source and sink devices which are exclusively linked and dependent.
> > Build the path directly and skip connection scanning for such devices.
> >
> > Cc: Mathieu Poirier
> > Cc: Mike Leach
> > Cc: Suzuki K Poulose
> > S
On Thu, 4 Feb 2021 at 08:11, Arnaud POULIQUEN
wrote:
>
>
>
> On 2/4/21 1:44 PM, kernel test robot wrote:
> > Hi Arnaud,
> >
> > I love your patch! Yet something to improve:
> >
> > [auto build test ERROR on linus/master]
> > [also build test ERROR on v5.11-rc6 next-20210125]
> > [cannot apply to
by free_aux() as long as the event is active and tracing,
> also provides us with access to the critical information
> needed to wind up a session even in the absence of an active
> output_handle.
>
> This is not an issue for the legacy sinks as none of them supports
> an IRQ and is centrally ha
mbers of parameters than in use at the
> time perf was built.
>
> Signed-off-by: Mike Leach
> Reviewed-by: Leo Yan
> Tested-by: Leo Yan
>
Reviewed-by: Mathieu Poirier
Arnaldo - please consider for inclusing in your tree.
Thanks,
Mathieu
> ---
>
> Changes si
On Tue, Feb 02, 2021 at 09:54:13AM +0100, Arnaud POULIQUEN wrote:
>
>
> On 2/2/21 1:49 AM, Mathieu Poirier wrote:
> > On Wed, Jan 27, 2021 at 10:21:24AM +0100, Arnaud POULIQUEN wrote:
> >> Hi Mathieu
> >>
> >> On 12/18/20 6:32 PM, Mathieu Poirier wr
ource status register is programmed
> before the tracing is turned on. For the moment simply write to
> it indicating TraceActive.
>
> Cc: Mathieu Poirier
> Cc: Mike Leach
> Signed-off-by: Suzuki K Poulose
> Signed-off-by: Anshuman Khandual
> ---
> drivers
he
> PE OS lock
>
> Cc: Mathieu Poirier
> Cc: Mike Leach
> Signed-off-by: Suzuki K Poulose
> Signed-off-by: Anshuman Khandual
> ---
> drivers/hwtracing/coresight/coresight-etm4x-core.c | 50
> ++
> drivers/hwtracing/coresight/coresight-etm
ers. Re-arrange the ETMv4x list
> to share the common definitions and add the ETE sysreg support.
>
> Cc: Mike Leach
> Cc: Mathieu Poirier
> Signed-off-by: Suzuki K Poulose
> Signed-off-by: Anshuman Khandual
> ---
> drivers/hwtracing/coresight/coresight-etm4x-core.c | 32 +
he
> PE OS lock
>
> Cc: Mathieu Poirier
> Cc: Mike Leach
> Signed-off-by: Suzuki K Poulose
> Signed-off-by: Anshuman Khandual
> ---
> drivers/hwtracing/coresight/coresight-etm4x-core.c | 50
> ++
> drivers/hwtracing/coresight/coresight-etm
301 - 400 of 5387 matches
Mail list logo