From: Bean Huo <bean...@micron.com>

Don't call trace_ufshcd_upiu() in case ufshba_upiu trace poit is not enabled.

Acked-by: Steven Rostedt (VMware) <rost...@goodmis.org>
Signed-off-by: Bean Huo <bean...@micron.com>
---
 drivers/scsi/ufs/ufshcd.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
index c68afd356abb..9c8fc46183c4 100644
--- a/drivers/scsi/ufs/ufshcd.c
+++ b/drivers/scsi/ufs/ufshcd.c
@@ -309,6 +309,9 @@ static void ufshcd_add_cmd_upiu_trace(struct ufs_hba *hba, 
unsigned int tag,
 {
        struct utp_upiu_req *rq = hba->lrb[tag].ucd_req_ptr;
 
+       if (!trace_ufshcd_upiu_enabled())
+               return;
+
        trace_ufshcd_upiu(dev_name(hba->dev), str_t, &rq->header, &rq->sc.cdb);
 }
 
@@ -317,6 +320,9 @@ static void ufshcd_add_query_upiu_trace(struct ufs_hba 
*hba, unsigned int tag,
 {
        struct utp_upiu_req *rq = hba->lrb[tag].ucd_req_ptr;
 
+       if (!trace_ufshcd_upiu_enabled())
+               return;
+
        trace_ufshcd_upiu(dev_name(hba->dev), str_t, &rq->header, &rq->qr);
 }
 
@@ -326,6 +332,9 @@ static void ufshcd_add_tm_upiu_trace(struct ufs_hba *hba, 
unsigned int tag,
        int off = (int)tag - hba->nutrs;
        struct utp_task_req_desc *descp = &hba->utmrdl_base_addr[off];
 
+       if (!trace_ufshcd_upiu_enabled())
+               return;
+
        trace_ufshcd_upiu(dev_name(hba->dev), str_t, &descp->req_header,
                        &descp->input_param1);
 }
-- 
2.17.1

Reply via email to