[dpdk-dev] [PATCH] common/octeontx2: fix link event message size

Harman Kalra hkalra at marvell.com
Fri Jul 30 18:08:06 CEST 2021


Due to wrong size of mbox message allocated for sending link status
to the VF, incorrect link status is observed.

Fixes: cb8d769fb6fe ("common/octeontx2: send link event to VF")

Signed-off-by: Harman Kalra <hkalra at marvell.com>
---
 drivers/common/octeontx2/otx2_dev.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/common/octeontx2/otx2_dev.c b/drivers/common/octeontx2/otx2_dev.c
index 1485e2b357..ce4f0e7ca9 100644
--- a/drivers/common/octeontx2/otx2_dev.c
+++ b/drivers/common/octeontx2/otx2_dev.c
@@ -172,14 +172,17 @@ af_pf_wait_msg(struct otx2_dev *dev, uint16_t vf, int num_msg)
 			/* Send link status to VF */
 			struct cgx_link_user_info linfo;
 			struct mbox_msghdr *vf_msg;
+			size_t sz;
 
 			/* Get the link status */
 			if (dev->ops && dev->ops->link_status_get)
 				dev->ops->link_status_get(dev, &linfo);
 
+			sz = RTE_ALIGN(otx2_mbox_id2size(
+				MBOX_MSG_CGX_LINK_EVENT), MBOX_MSG_ALIGN);
 			/* Prepare the message to be sent */
 			vf_msg = otx2_mbox_alloc_msg(&dev->mbox_vfpf_up, vf,
-						     size);
+						     sz);
 			otx2_mbox_req_init(MBOX_MSG_CGX_LINK_EVENT, vf_msg);
 			memcpy((uint8_t *)vf_msg + sizeof(struct mbox_msghdr),
 			       &linfo, sizeof(struct cgx_link_user_info));
-- 
2.18.0



More information about the dev mailing list