Commit fa00077d authored by Subbaraya Sundeep's avatar Subbaraya Sundeep Committed by Paolo Abeni
Browse files

octeontx2: Add new tracepoint otx2_msg_status 



Apart from netdev interface Octeontx2 PF does the following:
1. Sends its own requests to AF and receives responses from AF.
2. Receives async messages from AF.
3. Forwards VF requests to AF, sends respective responses from AF to VFs.
4. Sends async messages to VFs.
This patch adds new tracepoint otx2_msg_status to display the status
of PF wrt mailbox handling.

Signed-off-by: default avatarSubbaraya Sundeep <sbhatta@marvell.com>
Link: https://patch.msgid.link/1747136408-30685-5-git-send-email-sbhatta@marvell.com


Signed-off-by: default avatarPaolo Abeni <pabeni@redhat.com>
parent 27d27a06

drivers/net/ethernet/marvell/octeontx2/af/rvu_trace.c

+1 −0
Original line number Diff line number Diff line
@@ -11,3 +11,4 @@ 
EXPORT_TRACEPOINT_SYMBOL(otx2_msg_alloc);

EXPORT_TRACEPOINT_SYMBOL(otx2_msg_interrupt);

EXPORT_TRACEPOINT_SYMBOL(otx2_msg_process);

EXPORT_TRACEPOINT_SYMBOL(otx2_msg_status);

drivers/net/ethernet/marvell/octeontx2/af/rvu_trace.h

+15 −0
Original line number Diff line number Diff line
@@ -118,6 +118,21 @@ TRACE_EVENT(otx2_msg_wait_rsp, 
		      __get_str(dev))

);



TRACE_EVENT(otx2_msg_status,

	    TP_PROTO(const struct pci_dev *pdev, const char *msg, u16 num_msgs),

	    TP_ARGS(pdev, msg, num_msgs),

	    TP_STRUCT__entry(__string(dev, pci_name(pdev))

			     __string(str, msg)

			     __field(u16, num_msgs)

	    ),

	    TP_fast_assign(__assign_str(dev);

			   __assign_str(str);

			   __entry->num_msgs = num_msgs;

	    ),

	    TP_printk("[%s] %s num_msgs:%d\n", __get_str(dev),

		      __get_str(str), __entry->num_msgs)

);



#endif /* __RVU_TRACE_H */



#undef TRACE_INCLUDE_PATH

drivers/net/ethernet/marvell/octeontx2/nic/otx2_pf.c

+18 −0
Original line number Diff line number Diff line
@@ -465,6 +465,9 @@ static void otx2_pfvf_mbox_handler(struct work_struct *work) 


	offset = ALIGN(sizeof(struct mbox_hdr), MBOX_MSG_ALIGN);



	trace_otx2_msg_status(pf->pdev, "PF-VF down queue handler(forwarding)",

			      vf_mbox->num_msgs);



	for (id = 0; id < vf_mbox->num_msgs; id++) {

		msg = (struct mbox_msghdr *)(mdev->mbase + mbox->rx_start +

					     offset);
@@ -503,6 +506,9 @@ static void otx2_pfvf_mbox_up_handler(struct work_struct *work) 


	offset = mbox->rx_start + ALIGN(sizeof(struct mbox_hdr), MBOX_MSG_ALIGN);



	trace_otx2_msg_status(pf->pdev, "PF-VF up queue handler(response)",

			      vf_mbox->up_num_msgs);



	for (id = 0; id < vf_mbox->up_num_msgs; id++) {

		msg = mdev->mbase + offset;
@@ -819,6 +825,9 @@ static void otx2_pfaf_mbox_handler(struct work_struct *work) 
	offset = mbox->rx_start + ALIGN(sizeof(*rsp_hdr), MBOX_MSG_ALIGN);

	pf = af_mbox->pfvf;



	trace_otx2_msg_status(pf->pdev, "PF-AF down queue handler(response)",

			      num_msgs);



	for (id = 0; id < num_msgs; id++) {

		msg = (struct mbox_msghdr *)(mdev->mbase + offset);

		otx2_process_pfaf_mbox_msg(pf, msg);
@@ -974,6 +983,9 @@ static void otx2_pfaf_mbox_up_handler(struct work_struct *work) 


	offset = mbox->rx_start + ALIGN(sizeof(*rsp_hdr), MBOX_MSG_ALIGN);



	trace_otx2_msg_status(pf->pdev, "PF-AF up queue handler(notification)",

			      num_msgs);



	for (id = 0; id < num_msgs; id++) {

		msg = (struct mbox_msghdr *)(mdev->mbase + offset);
@@ -1023,6 +1035,9 @@ static irqreturn_t otx2_pfaf_mbox_intr_handler(int irq, void *pf_irq) 


		trace_otx2_msg_interrupt(pf->pdev, "UP message from AF to PF",

					 BIT_ULL(0));



		trace_otx2_msg_status(pf->pdev, "PF-AF up work queued(interrupt)",

				      hdr->num_msgs);

	}



	if (mbox_data & MBOX_DOWN_MSG) {
@@ -1039,6 +1054,9 @@ static irqreturn_t otx2_pfaf_mbox_intr_handler(int irq, void *pf_irq) 


		trace_otx2_msg_interrupt(pf->pdev, "DOWN reply from AF to PF",

					 BIT_ULL(0));



		trace_otx2_msg_status(pf->pdev, "PF-AF down work queued(interrupt)",

				      hdr->num_msgs);

	}



	return IRQ_HANDLED;