usb: cdns3: change some trace event __dynamic_array() to __get_buf()
authorLinyu Yuan <quic_linyyuan@quicinc.com>
Mon, 27 Feb 2023 02:44:08 +0000 (10:44 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 9 Mar 2023 14:35:03 +0000 (15:35 +0100)
some __dynamic_array() buffer will only used at trace event output time,
change to __get_buf() which will allocate tempary trace seq buffer for
output purpose.

Signed-off-by: Linyu Yuan <quic_linyyuan@quicinc.com>
Link: https://lore.kernel.org/r/1677465850-1396-3-git-send-email-quic_linyyuan@quicinc.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/cdns3/cdns3-trace.h
drivers/usb/cdns3/cdnsp-trace.h

index 99057186393b1e645fd39a53b66874e592fceb98..40db89e3333cc4143778f6c2503fcf1a22b89be9 100644 (file)
@@ -100,13 +100,12 @@ DECLARE_EVENT_CLASS(cdns3_log_usb_irq,
        TP_STRUCT__entry(
                __field(enum usb_device_speed, speed)
                __field(u32, usb_ists)
-               __dynamic_array(char, str, CDNS3_MSG_MAX)
        ),
        TP_fast_assign(
                __entry->speed = cdns3_get_speed(priv_dev);
                __entry->usb_ists = usb_ists;
        ),
-       TP_printk("%s", cdns3_decode_usb_irq(__get_str(str), __entry->speed,
+       TP_printk("%s", cdns3_decode_usb_irq(__get_buf(CDNS3_MSG_MAX), __entry->speed,
                                             __entry->usb_ists))
 );
 
@@ -124,7 +123,6 @@ DECLARE_EVENT_CLASS(cdns3_log_epx_irq,
                __field(u32, ep_traddr)
                __field(u32, ep_last_sid)
                __field(u32, use_streams)
-               __dynamic_array(char, str, CDNS3_MSG_MAX)
        ),
        TP_fast_assign(
                __assign_str(ep_name, priv_ep->name);
@@ -134,7 +132,7 @@ DECLARE_EVENT_CLASS(cdns3_log_epx_irq,
                __entry->use_streams = priv_ep->use_streams;
        ),
        TP_printk("%s, ep_traddr: %08x ep_last_sid: %08x use_streams: %d",
-                 cdns3_decode_epx_irq(__get_str(str),
+                 cdns3_decode_epx_irq(__get_buf(CDNS3_MSG_MAX),
                                       __get_str(ep_name),
                                       __entry->ep_sts),
                  __entry->ep_traddr,
@@ -153,13 +151,12 @@ DECLARE_EVENT_CLASS(cdns3_log_ep0_irq,
        TP_STRUCT__entry(
                __field(int, ep_dir)
                __field(u32, ep_sts)
-               __dynamic_array(char, str, CDNS3_MSG_MAX)
        ),
        TP_fast_assign(
                __entry->ep_dir = priv_dev->selected_ep;
                __entry->ep_sts = ep_sts;
        ),
-       TP_printk("%s", cdns3_decode_ep0_irq(__get_str(str),
+       TP_printk("%s", cdns3_decode_ep0_irq(__get_buf(CDNS3_MSG_MAX),
                                             __entry->ep_dir,
                                             __entry->ep_sts))
 );
@@ -178,7 +175,6 @@ DECLARE_EVENT_CLASS(cdns3_log_ctrl,
                __field(u16, wValue)
                __field(u16, wIndex)
                __field(u16, wLength)
-               __dynamic_array(char, str, CDNS3_MSG_MAX)
        ),
        TP_fast_assign(
                __entry->bRequestType = ctrl->bRequestType;
@@ -187,7 +183,7 @@ DECLARE_EVENT_CLASS(cdns3_log_ctrl,
                __entry->wIndex = le16_to_cpu(ctrl->wIndex);
                __entry->wLength = le16_to_cpu(ctrl->wLength);
        ),
-       TP_printk("%s", usb_decode_ctrl(__get_str(str), CDNS3_MSG_MAX,
+       TP_printk("%s", usb_decode_ctrl(__get_buf(CDNS3_MSG_MAX), CDNS3_MSG_MAX,
                                        __entry->bRequestType,
                                        __entry->bRequest, __entry->wValue,
                                        __entry->wIndex, __entry->wLength)
index 5983dfb996537ef344eb6bcb086d4a479ec48214..4b51011eb00b0ff9aedf14ae167c8c2ad4871a21 100644 (file)
@@ -271,7 +271,6 @@ DECLARE_EVENT_CLASS(cdnsp_log_ctrl,
                __field(u16, wValue)
                __field(u16, wIndex)
                __field(u16, wLength)
-               __dynamic_array(char, str, CDNSP_MSG_MAX)
        ),
        TP_fast_assign(
                __entry->bRequestType = ctrl->bRequestType;
@@ -280,7 +279,7 @@ DECLARE_EVENT_CLASS(cdnsp_log_ctrl,
                __entry->wIndex = le16_to_cpu(ctrl->wIndex);
                __entry->wLength = le16_to_cpu(ctrl->wLength);
        ),
-       TP_printk("%s", usb_decode_ctrl(__get_str(str), CDNSP_MSG_MAX,
+       TP_printk("%s", usb_decode_ctrl(__get_buf(CDNSP_MSG_MAX), CDNSP_MSG_MAX,
                                        __entry->bRequestType,
                                        __entry->bRequest, __entry->wValue,
                                        __entry->wIndex, __entry->wLength)
@@ -345,7 +344,6 @@ DECLARE_EVENT_CLASS(cdnsp_log_trb,
                __field(u32, field3)
                __field(union cdnsp_trb *, trb)
                __field(dma_addr_t, trb_dma)
-               __dynamic_array(char, str, CDNSP_MSG_MAX)
        ),
        TP_fast_assign(
                __entry->type = ring->type;
@@ -359,7 +357,7 @@ DECLARE_EVENT_CLASS(cdnsp_log_trb,
 
        ),
        TP_printk("%s: %s trb: %p(%pad)", cdnsp_ring_type_string(__entry->type),
-                 cdnsp_decode_trb(__get_str(str), CDNSP_MSG_MAX,
+                 cdnsp_decode_trb(__get_buf(CDNSP_MSG_MAX), CDNSP_MSG_MAX,
                                   __entry->field0, __entry->field1,
                                   __entry->field2, __entry->field3),
                                   __entry->trb, &__entry->trb_dma
@@ -544,7 +542,6 @@ DECLARE_EVENT_CLASS(cdnsp_log_ep_ctx,
                __field(u32, info2)
                __field(u64, deq)
                __field(u32, tx_info)
-               __dynamic_array(char, str, CDNSP_MSG_MAX)
        ),
        TP_fast_assign(
                __entry->info = le32_to_cpu(ctx->ep_info);
@@ -552,7 +549,7 @@ DECLARE_EVENT_CLASS(cdnsp_log_ep_ctx,
                __entry->deq = le64_to_cpu(ctx->deq);
                __entry->tx_info = le32_to_cpu(ctx->tx_info);
        ),
-       TP_printk("%s", cdnsp_decode_ep_context(__get_str(str), CDNSP_MSG_MAX,
+       TP_printk("%s", cdnsp_decode_ep_context(__get_buf(CDNSP_MSG_MAX), CDNSP_MSG_MAX,
                                                __entry->info, __entry->info2,
                                                __entry->deq, __entry->tx_info)
        )
@@ -777,7 +774,6 @@ DECLARE_EVENT_CLASS(cdnsp_log_portsc,
                TP_STRUCT__entry(
                                __field(u32, portnum)
                                __field(u32, portsc)
-                               __dynamic_array(char, str, CDNSP_MSG_MAX)
                                ),
                TP_fast_assign(
                                __entry->portnum = portnum;
@@ -785,7 +781,7 @@ DECLARE_EVENT_CLASS(cdnsp_log_portsc,
                                ),
                TP_printk("port-%d: %s",
                          __entry->portnum,
-                         cdnsp_decode_portsc(__get_str(str), CDNSP_MSG_MAX,
+                         cdnsp_decode_portsc(__get_buf(CDNSP_MSG_MAX), CDNSP_MSG_MAX,
                                              __entry->portsc)
                        )
 );