Commit 0137b129 authored by Changyuan Lyu's avatar Changyuan Lyu Committed by Martin K. Petersen
Browse files

scsi: pm80xx: Add pm80xx_mpi_build_cmd() tracepoint 

pm8001_mpi_build_cmd() prepares and sends all commands to a controller.
Having pm80xx_mpi_build_cmd tracepoint can help us with latency issues.

this patch depends on patch "scsi: pm80xx: Add tracepoints".

Link: https://lore.kernel.org/r/20211115215750.131696-3-changyuanl@google.com


Acked-by: default avatarJack Wang <jinpu.wang@ionos.com>
Co-developed-by: default avatarIgor Pylypiv <ipylypiv@google.com>
Signed-off-by: default avatarIgor Pylypiv <ipylypiv@google.com>
Signed-off-by: default avatarChangyuan Lyu <changyuanl@google.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 8ceddda3

drivers/scsi/pm8001/pm8001_hwi.c

+5 −0
Original line number Diff line number Diff line
@@ -42,6 +42,7 @@ 
 #include "pm8001_hwi.h"

 #include "pm8001_chips.h"

 #include "pm8001_ctl.h"

 #include "pm80xx_tracepoints.h"



/**

 * read_main_config_table - read the configure table and save it.
@@ -1324,6 +1325,10 @@ int pm8001_mpi_build_cmd(struct pm8001_hba_info *pm8001_ha, 
	unsigned long flags;

	int q_index = circularQ - pm8001_ha->inbnd_q_tbl;

	int rv;

	u32 htag = le32_to_cpu(*(__le32 *)payload);



	trace_pm80xx_mpi_build_cmd(pm8001_ha->id, opCode, htag, q_index,

		circularQ->producer_idx, le32_to_cpu(circularQ->consumer_index));



	if (WARN_ON(q_index >= pm8001_ha->max_q_num))

		return -EINVAL;

drivers/scsi/pm8001/pm80xx_tracepoints.h

+28 −0
Original line number Diff line number Diff line
@@ -75,6 +75,34 @@ TRACE_EVENT(pm80xx_request_complete, 
		    __entry->running_req)

);



TRACE_EVENT(pm80xx_mpi_build_cmd,

	    TP_PROTO(u32 id, u32 opc, u32 htag, u32 qi, u32 pi, u32 ci),



	    TP_ARGS(id, opc, htag, qi, pi, ci),



	    TP_STRUCT__entry(

		    __field(u32, id)

		    __field(u32, opc)

		    __field(u32, htag)

		    __field(u32, qi)

		    __field(u32, pi)

		    __field(u32, ci)

		    ),



	    TP_fast_assign(

		    __entry->id = id;

		    __entry->opc = opc;

		    __entry->htag = htag;

		    __entry->qi = qi;

		    __entry->pi = pi;

		    __entry->ci = ci;

		    ),



	    TP_printk("ctlr_id = %u opc = %#x htag = %#x QI = %u PI = %u CI = %u",

		    __entry->id, __entry->opc, __entry->htag, __entry->qi,

		    __entry->pi, __entry->ci)

);



#endif /* _TRACE_PM80XX_H_ */



#undef TRACE_INCLUDE_PATH