Add trace support to help debug eDMA problem.

Signed-off-by: Frank Li <[email protected]>
---
 drivers/dma/dw-edma/Makefile        |   3 +
 drivers/dma/dw-edma/dw-edma-core.c  |  12 +++
 drivers/dma/dw-edma/dw-edma-core.h  |   2 +
 drivers/dma/dw-edma/dw-edma-trace.c |   4 +
 drivers/dma/dw-edma/dw-edma-trace.h | 150 ++++++++++++++++++++++++++++++++++++
 5 files changed, 171 insertions(+)

diff --git a/drivers/dma/dw-edma/Makefile b/drivers/dma/dw-edma/Makefile
index 
83ab58f87760831883bcfad788306e1722634a83..3e31e7d92f3ecb577136bbb0e430801b6f8ff2b3
 100644
--- a/drivers/dma/dw-edma/Makefile
+++ b/drivers/dma/dw-edma/Makefile
@@ -1,9 +1,12 @@
 # SPDX-License-Identifier: GPL-2.0
 
+dw-edma-trace-$(CONFIG_TRACING)        := dw-edma-trace.o
+CFLAGS_dw-edma-trace.o         := -I$(src)
 obj-$(CONFIG_DW_EDMA)          += dw-edma.o
 dw-edma-$(CONFIG_DEBUG_FS)     := dw-edma-v0-debugfs.o \
                                   dw-hdma-v0-debugfs.o
 dw-edma-objs                   := dw-edma-core.o       \
                                   dw-edma-v0-core.o    \
+                                  ${dw-edma-trace-y} \
                                   dw-hdma-v0-core.o $(dw-edma-y)
 obj-$(CONFIG_DW_EDMA_PCIE)     += dw-edma-pcie.o
diff --git a/drivers/dma/dw-edma/dw-edma-core.c 
b/drivers/dma/dw-edma/dw-edma-core.c
index 
5aacd04bd2da4a65aabec48f6631f6f8882eecfd..339e372eb8cf60c3baa0de3e3576865e27d91716
 100644
--- a/drivers/dma/dw-edma/dw-edma-core.c
+++ b/drivers/dma/dw-edma/dw-edma-core.c
@@ -111,6 +111,12 @@ static void dw_edma_core_start(struct dw_edma_desc *desc, 
bool first)
                                     chan->ll_head, chan->cb,
                                     i == desc->nburst - 1 || free == 1);
 
+               trace_edma_fill_ll(chan, chan->ll_head,
+                                  desc->vd.tx.cookie,
+                                  desc->burst[i].sar,
+                                  desc->burst[i].dar, desc->burst[i].sz,
+                                  chan->cb);
+
                chan->ll_head++;
 
                if (chan->ll_head == chan->ll_max - 1) {
@@ -141,6 +147,8 @@ static int dw_edma_start_transfer(struct dw_edma_chan *chan)
 
                desc = vd2dw_edma_desc(vd);
 
+               trace_edma_start_desc(desc);
+
                if (desc->start_burst != desc->nburst) {
                        dw_edma_core_start(desc, !desc->start_burst);
                        ret = 1;
@@ -193,6 +201,7 @@ static void dw_edma_ll_clean_pending(struct dw_edma_chan 
*chan, int idx)
                                                            DMA_TRANS_NOERROR);
                                list_del(&vd->node);
                                vchan_cookie_complete(vd);
+                               trace_edma_complete_desc(desc);
                                chan->ll_end = desc->ll_end;
                        }
                } else {
@@ -348,6 +357,8 @@ dw_edma_device_tx_status(struct dma_chan *dchan, 
dma_cookie_t cookie,
        }
        spin_unlock_irqrestore(&chan->vc.lock, flags);
 
+       trace_edma_tx_status_info(chan, idx);
+
        /* check gain because dw_edma_ll_clean_pending() may update cookie */
        ret = dma_cookie_status(dchan, cookie, txstate);
        if (ret == DMA_COMPLETE)
@@ -609,6 +620,7 @@ static void dw_edma_done_interrupt(struct dw_edma_chan 
*chan)
 
        spin_lock_irqsave(&chan->vc.lock, flags);
        idx = dw_edma_core_ll_cur_idx(chan);
+       trace_edma_irq(chan, idx);
        if (idx != chan->cur_idx) {
                chan->cur_idx = idx;
                dw_edma_ll_clean_pending(chan, idx);
diff --git a/drivers/dma/dw-edma/dw-edma-core.h 
b/drivers/dma/dw-edma/dw-edma-core.h
index 
94d49f8359b99a9b0f8ca708edf81ca854dff4c2..ecc08dc0d34f4a86cc739dd12a1ce46ace58045c
 100644
--- a/drivers/dma/dw-edma/dw-edma-core.h
+++ b/drivers/dma/dw-edma/dw-edma-core.h
@@ -140,6 +140,8 @@ struct dw_edma {
        const struct dw_edma_core_ops   *core;
 };
 
+#include "dw-edma-trace.h"
+
 typedef void (*dw_edma_handler_t)(struct dw_edma_chan *);
 
 struct dw_edma_core_ops {
diff --git a/drivers/dma/dw-edma/dw-edma-trace.c 
b/drivers/dma/dw-edma/dw-edma-trace.c
new file mode 100644
index 
0000000000000000000000000000000000000000..2620ad61a9436a8d21b2408f3613c585fba0d9bb
--- /dev/null
+++ b/drivers/dma/dw-edma/dw-edma-trace.c
@@ -0,0 +1,4 @@
+// SPDX-License-Identifier: GPL-2.0
+
+#define CREATE_TRACE_POINTS
+#include "dw-edma-core.h"
diff --git a/drivers/dma/dw-edma/dw-edma-trace.h 
b/drivers/dma/dw-edma/dw-edma-trace.h
new file mode 100644
index 
0000000000000000000000000000000000000000..3be77b42b04947407536523d1535d1eb7d9bdf71
--- /dev/null
+++ b/drivers/dma/dw-edma/dw-edma-trace.h
@@ -0,0 +1,150 @@
+/* SPDX-License-Identifier: GPL-2.0+ */
+/*
+ * Copyright 2023 NXP.
+ */
+
+#undef TRACE_SYSTEM
+#define TRACE_SYSTEM dw_edma
+
+#if !defined(__LINUX_DW_EDMA_TRACE) || defined(TRACE_HEADER_MULTI_READ)
+#define __LINUX_DW_EDMA_TRACE
+
+#include <linux/types.h>
+#include <linux/tracepoint.h>
+
+DECLARE_EVENT_CLASS(edma_desc_info,
+       TP_PROTO(struct dw_edma_desc *desc),
+       TP_ARGS(desc),
+       TP_STRUCT__entry(
+               __field(u32, nburst)
+               __field(u32, start_burst)
+               __field(u32, ll_end)
+               __field(u32, cookie)
+               __field(u32, id)
+               __field(u8, dir)
+       ),
+       TP_fast_assign(
+               __entry->nburst = desc->nburst,
+               __entry->start_burst = desc->start_burst,
+               __entry->ll_end = desc->ll_end,
+               __entry->id = desc->chan->id,
+               __entry->dir = desc->chan->dir,
+               __entry->cookie = desc->vd.tx.cookie;
+       ),
+       TP_printk("chan %d%c desc %d, nburst %d, start_burst %d, ll_end %d\n",
+               __entry->id,
+               __entry->dir ? 'R': 'W',
+               __entry->cookie,
+               __entry->nburst,
+               __entry->start_burst,
+               __entry->ll_end)
+);
+
+DEFINE_EVENT(edma_desc_info, edma_start_desc,
+        TP_PROTO(struct dw_edma_desc *desc),
+        TP_ARGS(desc)
+);
+
+DEFINE_EVENT(edma_desc_info, edma_complete_desc,
+        TP_PROTO(struct dw_edma_desc *desc),
+        TP_ARGS(desc)
+);
+
+DECLARE_EVENT_CLASS(edma_ll_info,
+       TP_PROTO(struct dw_edma_chan *chan, int idx),
+       TP_ARGS(chan, idx),
+       TP_STRUCT__entry(
+               __field(u32, head)
+               __field(u32, end)
+               __field(u32, total)
+               __field(u32, index)
+               __field(u32, completed_cookie)
+               __field(u32, cookie)
+               __field(u32, id)
+               __field(u8, dir)
+       ),
+       TP_fast_assign(
+               __entry->head = chan->ll_head,
+               __entry->end = chan->ll_end,
+               __entry->total = chan->ll_max,
+               __entry->index = idx,
+               __entry->completed_cookie = chan->vc.chan.completed_cookie,
+               __entry->cookie = chan->vc.chan.cookie,
+               __entry->id = chan->id,
+               __entry->dir = chan->dir;
+       ),
+       TP_printk("chan %d%c head: %d end: %d: dma cur index: %d, complete 
cookie: %d, cookie: %d\n",
+               __entry->id,
+               __entry->dir ? 'R': 'W',
+               __entry->head,
+               __entry->end,
+               __entry->index,
+               __entry->completed_cookie,
+               __entry->cookie)
+);
+
+DEFINE_EVENT(edma_ll_info, edma_tx_status_info,
+        TP_PROTO(struct dw_edma_chan *chan, int idx),
+        TP_ARGS(chan, idx)
+);
+
+DEFINE_EVENT(edma_ll_info, edma_irq,
+        TP_PROTO(struct dw_edma_chan *chan, int idx),
+        TP_ARGS(chan, idx)
+);
+
+DEFINE_EVENT(edma_ll_info, emda_terminate_all,
+       TP_PROTO(struct dw_edma_chan *chan, int idx),
+       TP_ARGS(chan, idx)
+);
+
+DECLARE_EVENT_CLASS(edma_log_ll,
+       TP_PROTO(struct dw_edma_chan *chan, u32 idx, u32 cookie, u64 src, u64 
dest, u32 sz, bool flag),
+       TP_ARGS(chan, idx, cookie, src, dest, sz, flag),
+       TP_STRUCT__entry(
+               __field(u32, idx)
+               __field(u64, src)
+               __field(u64, dest)
+               __field(u32, sz)
+               __field(u32, id)
+               __field(u32, cookie)
+               __field(bool, flag)
+               __field(u8, dir)
+       ),
+       TP_fast_assign(
+               __entry->idx = idx,
+               __entry->src = src,
+               __entry->dest = dest,
+               __entry->sz = sz,
+               __entry->id = chan->id,
+               __entry->dir = chan->dir,
+               __entry->cookie = cookie,
+               __entry->flag = flag;
+       ),
+       TP_printk("chan %d%c %d [%d] %c src: %08llx dest: %08llx sz: %04x\n",
+               __entry->id,
+               __entry->dir ? 'R' : 'W',
+               __entry->cookie,
+               __entry->idx,
+               __entry->flag ? 'C' : 'c',
+               __entry->src,
+               __entry->dest,
+               __entry->sz)
+);
+
+DEFINE_EVENT(edma_log_ll, edma_fill_ll,
+       TP_PROTO(struct dw_edma_chan *chan, u32 idx, u32 cookie, u64 src, u64 
dest, u32 sz, bool flag),
+       TP_ARGS(chan, idx, cookie, src, dest, sz, flag)
+);
+
+#endif
+
+/* this part must be outside header guard */
+
+#undef TRACE_INCLUDE_PATH
+#define TRACE_INCLUDE_PATH .
+
+#undef TRACE_INCLUDE_FILE
+#define TRACE_INCLUDE_FILE dw-edma-trace
+
+#include <trace/define_trace.h>

-- 
2.34.1


Reply via email to