* [PATCH 1/2] memory: tegra-mc: add tracepoints for tegra mc tracing
2025-06-16 16:01 two tegra drier tracing patches Ben Dooks
@ 2025-06-16 16:01 ` Ben Dooks
2025-06-16 16:01 ` [PATCH 2/2] drivers/firware/tegra: add tracing to bpmp code Ben Dooks
1 sibling, 0 replies; 3+ messages in thread
From: Ben Dooks @ 2025-06-16 16:01 UTC (permalink / raw)
To: thierry.reding, jonathanh, rostedt, mhiramat, linux-tegra
Cc: linux-trace-kernel, linux-kernel, Ben Dooks
Add trace points to allow tracing of how the tegra_mc driver is being
called.
Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
---
drivers/memory/tegra/mc.c | 26 +++++++------
include/trace/events/tegra_mc.h | 69 +++++++++++++++++++++++++++++++++
2 files changed, 84 insertions(+), 11 deletions(-)
create mode 100644 include/trace/events/tegra_mc.h
diff --git a/drivers/memory/tegra/mc.c b/drivers/memory/tegra/mc.c
index bd5b58f1fd42..256cf0d0aa2d 100644
--- a/drivers/memory/tegra/mc.c
+++ b/drivers/memory/tegra/mc.c
@@ -21,6 +21,9 @@
#include "mc.h"
+#define CREATE_TRACE_POINTS
+#include <trace/events/tegra_mc.h>
+
static const struct of_device_id tegra_mc_of_match[] = {
#ifdef CONFIG_ARCH_TEGRA_2x_SOC
{ .compatible = "nvidia,tegra20-mc-gart", .data = &tegra20_mc_soc },
@@ -210,6 +213,8 @@ static int tegra_mc_hotreset_assert(struct reset_controller_dev *rcdev,
int retries = 500;
int err;
+ trace_tegra_mc_hotreset(mc->dev, id, true);
+
rst = tegra_mc_reset_find(mc, id);
if (!rst)
return -ENODEV;
@@ -269,6 +274,8 @@ static int tegra_mc_hotreset_deassert(struct reset_controller_dev *rcdev,
const struct tegra_mc_reset *rst;
int err;
+ trace_tegra_mc_hotreset(mc->dev, id, false);
+
rst = tegra_mc_reset_find(mc, id);
if (!rst)
return -ENODEV;
@@ -583,7 +590,7 @@ irqreturn_t tegra30_mc_handle_irq(int irq, void *data)
for_each_set_bit(bit, &status, 32) {
const char *error = tegra_mc_status_names[bit] ?: "unknown";
const char *client = "unknown", *desc;
- const char *direction, *secure;
+ bool secure, write;
u32 status_reg, addr_reg;
u32 intmask = BIT(bit);
phys_addr_t addr = 0;
@@ -652,15 +659,8 @@ irqreturn_t tegra30_mc_handle_irq(int irq, void *data)
}
#endif
- if (value & MC_ERR_STATUS_RW)
- direction = "write";
- else
- direction = "read";
-
- if (value & MC_ERR_STATUS_SECURITY)
- secure = "secure ";
- else
- secure = "";
+ write = value & MC_ERR_STATUS_RW;
+ secure = value & MC_ERR_STATUS_SECURITY;
id = value & mc->soc->client_id_mask;
@@ -710,8 +710,12 @@ irqreturn_t tegra30_mc_handle_irq(int irq, void *data)
value = mc_readl(mc, addr_reg);
addr |= value;
+ trace_tegra_mc_err(mc->dev, client, secure, write, addr,
+ error, desc, perm);
+
dev_err_ratelimited(mc->dev, "%s: %s%s @%pa: %s (%s%s)\n",
- client, secure, direction, &addr, error,
+ client, secure ? "secure" : "",
+ write ? "write" : "read", &addr, error,
desc, perm);
}
diff --git a/include/trace/events/tegra_mc.h b/include/trace/events/tegra_mc.h
new file mode 100644
index 000000000000..acf00a19240b
--- /dev/null
+++ b/include/trace/events/tegra_mc.h
@@ -0,0 +1,69 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+/* tegra memory controller tracepoints
+ *
+ * Copyright (c) 2025 Codethink Ltd.
+ * Ben Dooks <ben.dooks@codethink.co.uk>
+ */
+
+#undef TRACE_SYSTEM
+#define TRACE_SYSTEM tegra_mc
+
+#if !defined(_TRACE_TEGRA_MC_H) || defined(TRACE_HEADER_MULTI_READ)
+#define _TRACE_TEGRA_MC_H
+
+#include <linux/device.h>
+#include <linux/tracepoint.h>
+
+TRACE_EVENT(tegra_mc_hotreset,
+ TP_PROTO(struct device *dev, unsigned long id, bool assert),
+ TP_ARGS(dev, id, assert),
+ TP_STRUCT__entry(
+ __string(dev, dev_name(dev))
+ __field(unsigned long, id)
+ __field(bool, assert)),
+ TP_fast_assign(
+ __assign_str(dev);
+ __entry->id = id;
+ __entry->assert = assert;
+ ),
+ TP_printk("rcdev %s, id %lu, %s\n",
+ __get_str(dev), __entry->id, __entry->assert ? "assert" : "release")
+);
+
+TRACE_EVENT(tegra_mc_err,
+ TP_PROTO(struct device *dev, const char *client,
+ bool secure, bool write, phys_addr_t addr,
+ const char *error, const char *desc, const char *perm),
+ TP_ARGS(dev, client, secure, write, addr, error, desc, perm),
+ TP_STRUCT__entry(
+ __string(dev, dev_name(dev))
+ __field(const char *, client)
+ __field(bool, secure)
+ __field(bool, write)
+ __field(phys_addr_t, addr)
+ __field(const char *, error)
+ __field(const char *, desc)
+ __string(perm, perm)),
+ TP_fast_assign(
+ __assign_str(dev);
+ __entry->client = client;
+ __entry->secure = secure;
+ __entry->write = write;
+ __entry->addr = addr;
+ __entry->desc = desc;
+ __assign_str(perm);
+ ),
+ TP_printk("%s: %s: %s%c @%pa %s (%s%s)",
+ __get_str(dev),
+ __entry->client, __entry->secure ? "secure/" : "",
+ __entry->write ? 'w' : 'r',
+ &__entry->addr,
+ __entry->error,
+ __entry->desc,
+ __get_str(perm))
+);
+
+#endif /* _TRACE_TEGRA_MC_H */
+
+/* This part must be outside protection */
+#include <trace/define_trace.h>
--
2.37.2.352.g3c44437643
^ permalink raw reply related [flat|nested] 3+ messages in thread* [PATCH 2/2] drivers/firware/tegra: add tracing to bpmp code
2025-06-16 16:01 two tegra drier tracing patches Ben Dooks
2025-06-16 16:01 ` [PATCH 1/2] memory: tegra-mc: add tracepoints for tegra mc tracing Ben Dooks
@ 2025-06-16 16:01 ` Ben Dooks
1 sibling, 0 replies; 3+ messages in thread
From: Ben Dooks @ 2025-06-16 16:01 UTC (permalink / raw)
To: thierry.reding, jonathanh, rostedt, mhiramat, linux-tegra
Cc: linux-trace-kernel, linux-kernel, Ben Dooks
Add trace points to the communications with the bpmp
Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
---
drivers/firmware/tegra/bpmp.c | 16 +++-
include/trace/events/tegra_bpmp.h | 140 ++++++++++++++++++++++++++++++
2 files changed, 155 insertions(+), 1 deletion(-)
create mode 100644 include/trace/events/tegra_bpmp.h
diff --git a/drivers/firmware/tegra/bpmp.c b/drivers/firmware/tegra/bpmp.c
index c3a1dc344961..b7c005990269 100644
--- a/drivers/firmware/tegra/bpmp.c
+++ b/drivers/firmware/tegra/bpmp.c
@@ -20,6 +20,9 @@
#include "bpmp-private.h"
+#define CREATE_TRACE_POINTS
+#include <trace/events/tegra_bpmp.h>
+
#define MSG_ACK BIT(0)
#define MSG_RING BIT(1)
#define TAG_SZ 32
@@ -227,6 +230,7 @@ static ssize_t tegra_bpmp_channel_read(struct tegra_bpmp_channel *channel,
unlock:
up(&bpmp->threaded.lock);
+ trace_tegra_bpmp_channel_read(channel, data, size, ret, err);
return err;
}
@@ -240,6 +244,8 @@ static ssize_t __tegra_bpmp_channel_write(struct tegra_bpmp_channel *channel,
if (data && size > 0)
tegra_bpmp_mb_write(&channel->ob, data, size);
+ trace_tegra_bpmp_channel_write(channel, mrq, flags, data, size);
+
return tegra_bpmp_post_request(channel);
}
@@ -350,8 +356,12 @@ int tegra_bpmp_transfer_atomic(struct tegra_bpmp *bpmp,
if (err < 0)
return err;
- return __tegra_bpmp_channel_read(channel, msg->rx.data, msg->rx.size,
+ err = __tegra_bpmp_channel_read(channel, msg->rx.data, msg->rx.size,
&msg->rx.ret);
+
+ trace_tegra_bpmp_channel_read(channel, msg->rx.data, msg->rx.size,
+ &msg->rx.ret, err);
+ return err;
}
EXPORT_SYMBOL_GPL(tegra_bpmp_transfer_atomic);
@@ -368,6 +378,8 @@ int tegra_bpmp_transfer(struct tegra_bpmp *bpmp,
if (!tegra_bpmp_message_valid(msg))
return -EINVAL;
+ trace_tegra_bpmp_transfer(bpmp, msg);
+
if (bpmp->suspended) {
/* Reset BPMP IPC channels during resume based on flags passed */
if (msg->flags & TEGRA_BPMP_MESSAGE_RESET)
@@ -812,6 +824,7 @@ static int __maybe_unused tegra_bpmp_suspend(struct device *dev)
{
struct tegra_bpmp *bpmp = dev_get_drvdata(dev);
+ trace_tegra_bpmp_pm(bpmp, true);
bpmp->suspended = true;
return 0;
@@ -821,6 +834,7 @@ static int __maybe_unused tegra_bpmp_resume(struct device *dev)
{
struct tegra_bpmp *bpmp = dev_get_drvdata(dev);
+ trace_tegra_bpmp_pm(bpmp, false);
bpmp->suspended = false;
if (bpmp->soc->ops->resume)
diff --git a/include/trace/events/tegra_bpmp.h b/include/trace/events/tegra_bpmp.h
new file mode 100644
index 000000000000..63289e475951
--- /dev/null
+++ b/include/trace/events/tegra_bpmp.h
@@ -0,0 +1,140 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+/* tegra memory controller tracepoints
+ *
+ * Copyright (c) 2025 Codethink Ltd.
+ * Ben Dooks <ben.dooks@codethink.co.uk>
+ */
+
+#undef TRACE_SYSTEM
+#define TRACE_SYSTEM tegra_bpmp
+
+#if !defined(_TRACE_TEGRA_BPMP_H) || defined(TRACE_HEADER_MULTI_READ)
+#define _TRACE_TEGRA_BPMP_H
+
+#include <linux/tracepoint.h>
+
+#include <soc/tegra/bpmp.h>
+
+#ifdef CREATE_TRACE_POINTS
+static inline const char *tegra_bpmp_mrq_name(unsigned mrq)
+{
+ switch (mrq) {
+ case MRQ_PING: return "ping";
+ case MRQ_QUERY_TAG: return "query-tag";
+ case MRQ_THREADED_PING: return "threaded-ping";
+ case MRQ_DEBUGFS: return "debugfs";
+ case MRQ_RESET: return "reset";
+ case MRQ_I2C: return "i2c";
+ case MRQ_CLK: return "clk";
+ case MRQ_QUERY_ABI: return "query-abi";
+ case MRQ_THERMAL: return "thermal";
+ case MRQ_CPU_VHINT: return "cpu-vhint";
+ case MRQ_ABI_RATCHET: return "abi-ratchet";
+ case MRQ_EMC_DVFS_LATENCY: return "emc-dvfs-latency";
+ case MRQ_RINGBUF_CONSOLE: return "ringbuf-console";
+ case MRQ_PG: return "pg";
+ case MRQ_CPU_NDIV_LIMITS: return "cpu-ndiv-limits";
+ case MRQ_STRAP: return "strap";
+ case MRQ_UPHY: return "uphy";
+ case MRQ_CPU_AUTO_CC3: return "cpu-auto-cc3";
+ case MRQ_QUERY_FW_TAG: return "query-fw-tag";
+ case MRQ_FMON: return "fwmon";
+ case MRQ_EC: return "ec";
+ case MRQ_DEBUG: return "debug";
+ case MRQ_EMC_DVFS_EMCHUB: return "emc-dvfs-emchub";
+ case MRQ_BWMGR: return "bwmgr";
+ case MRQ_ISO_CLIENT: return "iso-client";
+ case MRQ_EMC_DISP_RFL: return "emc-disp-rfl";
+ case MRQ_TELEMETRY: return "telemetry";
+ case MRQ_PWR_LIMIT: return "pwr-limit";
+ case MRQ_GEARS: return "gears";
+ case MRQ_BWMGR_INT: return "bwmgr-int";
+ case MRQ_OC_STATUS: return "oc-status";
+ }
+
+ return "";
+}
+
+#define __pm_state(__s) ((__s) ? "suspended" : "active")
+#endif /* CREATE_TRACE_POINTS */
+
+TRACE_EVENT(tegra_bpmp_pm,
+ TP_PROTO(struct tegra_bpmp *bpmp, bool suspend),
+ TP_ARGS(bpmp, suspend),
+ TP_STRUCT__entry(
+ __string(dev, dev_name(bpmp->dev))
+ __field(bool, suspend)
+ __field(bool, prev)),
+ TP_fast_assign(
+ __assign_str(dev);
+ __entry->prev = bpmp->suspended;
+ __entry->suspend = suspend;
+ ),
+ TP_printk("bpmp %s: pm %s to %s",
+ __get_str(dev),
+ __pm_state(__entry->prev), __pm_state(__entry->suspend))
+);
+
+TRACE_EVENT(tegra_bpmp_transfer,
+ TP_PROTO(struct tegra_bpmp *bpmp,
+ struct tegra_bpmp_message *msg),
+ TP_ARGS(bpmp, msg),
+ TP_STRUCT__entry(
+ __string(dev, dev_name(bpmp->dev))
+ __field_struct(struct tegra_bpmp_message, msg)
+ ),
+ TP_fast_assign(
+ __assign_str(dev);
+ __entry->msg = *msg;
+ ),
+ TP_printk("bpmp %s: tx %px, %zd rx %px, %zd, mrq %u (%s), flags %lu",
+ __get_str(dev),
+ __entry->msg.tx.data, __entry->msg.tx.size,
+ __entry->msg.rx.data, __entry->msg.rx.size,
+ __entry->msg.mrq, tegra_bpmp_mrq_name(__entry->msg.mrq),
+ __entry->msg.flags)
+);
+
+TRACE_EVENT(tegra_bpmp_channel_write,
+ TP_PROTO(struct tegra_bpmp_channel *channel,
+ unsigned int mrq, unsigned long flags,
+ const void *data, size_t size),
+ TP_ARGS(channel, mrq, flags, data, size),
+ TP_STRUCT__entry(
+ __field(unsigned int, index)
+ __field(unsigned int, mrq)
+ __field(unsigned long, flags)
+ ),
+ TP_fast_assign(
+ __entry->index = channel->index;
+ __entry->mrq = mrq;
+ __entry->flags = flags;
+ ),
+ TP_printk("channel %u: mrq %u (%s), flags %lu",
+ __entry->index,
+ __entry->mrq, tegra_bpmp_mrq_name(__entry->mrq),
+ __entry->flags)
+);
+
+TRACE_EVENT(tegra_bpmp_channel_read,
+ TP_PROTO(struct tegra_bpmp_channel *channel,
+ void *data, size_t size, int *ret, int err),
+ TP_ARGS(channel, data, size, ret, err),
+ TP_STRUCT__entry(
+ __field(unsigned int, index)
+ __field(int, ret)
+ __field(int, err)
+ ),
+ TP_fast_assign(
+ __entry->index = channel->index;
+ __entry->ret = ret ? *ret : -ENOENT;
+ __entry->err = err;
+ ),
+ TP_printk("channel %u: read returned %d, err %d",
+ __entry->index, __entry->ret, __entry->err)
+);
+
+#endif /* _TRACE_TEGRA_BPMP_H */
+
+/* This part must be outside protection */
+#include <trace/define_trace.h>
--
2.37.2.352.g3c44437643
^ permalink raw reply related [flat|nested] 3+ messages in thread