bus: mhi: host: Change the trace string for the userspace tools mapping
authorKrishna chaitanya chundru <quic_krichai@quicinc.com>
Sun, 18 Feb 2024 08:43:39 +0000 (14:13 +0530)
committerManivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Thu, 22 Feb 2024 05:15:19 +0000 (10:45 +0530)
User space tools can't map strings if we use directly, as the string
address is internal to kernel.

So add trace point strings for the user space tools to map strings
properly.

Suggested-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Krishna chaitanya chundru <quic_krichai@quicinc.com>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Link: https://lore.kernel.org/r/20240218-ftrace_string-v1-1-27da85c1f844@quicinc.com
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
drivers/bus/mhi/host/main.c
drivers/bus/mhi/host/trace.h

index 2d38f6005da659b6e665e90f4fc17095b6230eb2..15d657af9b5b8685bfa0a70b0c4dd2c498c3bf06 100644 (file)
@@ -1340,7 +1340,7 @@ static int mhi_update_channel_state(struct mhi_controller *mhi_cntrl,
        enum mhi_cmd_type cmd = MHI_CMD_NOP;
        int ret;
 
-       trace_mhi_channel_command_start(mhi_cntrl, mhi_chan, to_state, "Updating");
+       trace_mhi_channel_command_start(mhi_cntrl, mhi_chan, to_state, TPS("Updating"));
        switch (to_state) {
        case MHI_CH_STATE_TYPE_RESET:
                write_lock_irq(&mhi_chan->lock);
@@ -1407,7 +1407,7 @@ static int mhi_update_channel_state(struct mhi_controller *mhi_cntrl,
                write_unlock_irq(&mhi_chan->lock);
        }
 
-       trace_mhi_channel_command_end(mhi_cntrl, mhi_chan, to_state, "Updated");
+       trace_mhi_channel_command_end(mhi_cntrl, mhi_chan, to_state, TPS("Updated"));
 exit_channel_update:
        mhi_cntrl->runtime_put(mhi_cntrl);
        mhi_device_put(mhi_cntrl->mhi_dev);
index d12a98d4427283e60b4448314af3c2f78d0d8c85..368515dcb22d157db44c6458354390ad29ea1ffc 100644 (file)
@@ -84,6 +84,8 @@ DEV_ST_TRANSITION_LIST
 #define dev_st_trans(a, b)             { DEV_ST_TRANSITION_##a, b },
 #define dev_st_trans_end(a, b)         { DEV_ST_TRANSITION_##a, b }
 
+#define TPS(x) tracepoint_string(x)
+
 TRACE_EVENT(mhi_gen_tre,
 
        TP_PROTO(struct mhi_controller *mhi_cntrl, struct mhi_chan *mhi_chan,