RDMA/hns: Fix endian issue in trace events

Avoid using __le32 directly in trace events to fix sparse complains:

drivers/infiniband/hw/hns/./hns_roce_trace.h:48:1: sparse: sparse: cast to restricted __le32
drivers/infiniband/hw/hns/./hns_roce_trace.h:48:1: sparse: sparse: restricted __le32 degrades to integer
drivers/infiniband/hw/hns/./hns_roce_trace.h:48:1: sparse: sparse: restricted __le32 degrades to integer
drivers/infiniband/hw/hns/./hns_roce_trace.h:90:1: sparse: sparse: cast to restricted __le32
drivers/infiniband/hw/hns/./hns_roce_trace.h:90:1: sparse: sparse: restricted __le32 degrades to integer
drivers/infiniband/hw/hns/./hns_roce_trace.h:90:1: sparse: sparse: restricted __le32 degrades to integer
drivers/infiniband/hw/hns/./hns_roce_trace.h:173:1: sparse: sparse: cast to restricted __le32
drivers/infiniband/hw/hns/./hns_roce_trace.h:173:1: sparse: sparse: restricted __le32 degrades to integer
drivers/infiniband/hw/hns/./hns_roce_trace.h:173:1: sparse: sparse: restricted __le32 degrades to integer

Fixes: 6c98c8670806 ("RDMA/hns: Add trace for WQE dumping")
Fixes: 1e63e2f96613 ("RDMA/hns: Add trace for AEQE dumping")
Fixes: 6bd18dabf1c9 ("RDMA/hns: Add trace for CMDQ dumping")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202505170327.TNOpreil-lkp@intel.com/
Signed-off-by: Junxian Huang <huangjunxian6@hisilicon.com>
Link: https://patch.msgid.link/20250523023433.2171003-1-huangjunxian6@hisilicon.com
Signed-off-by: Leon Romanovsky <leon@kernel.org>

diff --git a/drivers/infiniband/hw/hns/hns_roce_trace.h b/drivers/infiniband/hw/hns/hns_roce_trace.h
index 23cbdbaeffaa41562eaa47e9875c09cb062a69c3..59ceb591b3a1a259e749bd81a524e665c4e76e34 100644 (file)
--- a/drivers/infiniband/hw/hns/hns_roce_trace.h
+++ b/drivers/infiniband/hw/hns/hns_roce_trace.h
@@ -52,7 +52,7 @@ DECLARE_EVENT_CLASS(wqe_template,
 
                    TP_STRUCT__entry(__field(unsigned long, qpn)
                                     __field(u32, idx)
-                                    __array(__le32, wqe,
+                                    __array(u32, wqe,
                                             MAX_WQE_SIZE / sizeof(__le32))
                                     __field(u32, len)
                                     __field(u64, id)
@@ -62,9 +62,10 @@ DECLARE_EVENT_CLASS(wqe_template,
                    TP_fast_assign(__entry->qpn = qpn;
                                   __entry->idx = idx;
                                   __entry->id = id;
-                                  memcpy(__entry->wqe, wqe, len);
                                   __entry->len = len / sizeof(__le32);
                                   __entry->type = type;
+                                  for (int i = 0; i < __entry->len; i++)
+                                       __entry->wqe[i] = le32_to_cpu(((__le32 *)wqe)[i]);
                                   ),
 
                    TP_printk("%s 0x%lx wqe(0x%x/0x%llx): %s",
@@ -92,14 +93,15 @@ TRACE_EVENT(hns_ae_info,
            TP_ARGS(event_type, aeqe, len),
 
            TP_STRUCT__entry(__field(int, event_type)
-                            __array(__le32, aeqe,
+                            __array(u32, aeqe,
                                     HNS_ROCE_V3_EQE_SIZE / sizeof(__le32))
                             __field(u32, len)
            ),
 
            TP_fast_assign(__entry->event_type = event_type;
                           __entry->len = len / sizeof(__le32);
-                          memcpy(__entry->aeqe, aeqe, len);
+                          for (int i = 0; i < __entry->len; i++)
+                               __entry->aeqe[i] = le32_to_cpu(((__le32 *)aeqe)[i]);
            ),
 
            TP_printk("event %2d aeqe: %s", __entry->event_type,
@@ -179,14 +181,15 @@ DECLARE_EVENT_CLASS(cmdq,
                                     __field(u16, opcode)
                                     __field(u16, flag)
                                     __field(u16, retval)
-                                    __array(__le32, data, 6)
+                                    __array(u32, data, 6)
                    ),
 
                    TP_fast_assign(__assign_str(dev_name);
                                   __entry->opcode = le16_to_cpu(desc->opcode);
                                   __entry->flag = le16_to_cpu(desc->flag);
                                   __entry->retval = le16_to_cpu(desc->retval);
-                                  memcpy(__entry->data, desc->data, 6 * sizeof(__le32));
+                                  for (int i = 0; i < 6; i++)
+                                       __entry->data[i] = le32_to_cpu(desc->data[i]);
                    ),
 
                    TP_printk("%s cmdq opcode:0x%x, flag:0x%x, retval:0x%x, data:%s\n",