>-----Original Message----- >From: Steve Yang <[email protected]> >Sent: Wednesday, December 9, 2020 8:46 AM >To: [email protected] >Cc: [email protected]; [email protected]; >[email protected]; [email protected]; [email protected]; >[email protected]; [email protected]; [email protected]; >[email protected]; [email protected]; [email protected]; >[email protected]; [email protected]; Harman Kalra ><[email protected]>; Jerin Jacob Kollanukkaran <[email protected]>; >Nithin Kumar Dabilpuram <[email protected]>; Kiran Kumar >Kokkilagadda <[email protected]>; Rasesh Mody ><[email protected]>; Shahed Shaikh <[email protected]>; >[email protected]; Maciej Czekaj [C] <[email protected]>; >[email protected]; [email protected]; [email protected]; >[email protected]; [email protected]; [email protected]; >[email protected]; [email protected]; >[email protected]; [email protected]; [email protected]; >[email protected]; [email protected]; [email protected]; Sunil >Kumar Kori <[email protected]>; Vamsi Krishna Attunuru ><[email protected]>; [email protected]; >[email protected]; [email protected]; >[email protected]; [email protected]; >[email protected]; Steve Yang <[email protected]> >Subject: [EXT] [PATCH v1 09/12] net/octeontx2: fix the jumbo frame flag >condition for mtu > >External Email > >---------------------------------------------------------------------- >The jumbo frame uses the 'RTE_ETHER_MAX_LEN' as boundary condition, but >the Ether overhead is larger than 18 when it supports dual VLAN tags. >That will cause the jumbo flag rx offload is wrong when MTU size is >'RTE_ETHER_MTU'. > >This fix will change the boundary condition with 'RTE_ETHER_MTU'. > >Fixes: 0e2efd02db58 ("net/octeontx2: add MTU set operation") > >Signed-off-by: Steve Yang <[email protected]> >--- > drivers/net/octeontx2/otx2_ethdev_ops.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/drivers/net/octeontx2/otx2_ethdev_ops.c >b/drivers/net/octeontx2/otx2_ethdev_ops.c >index b36d37b9f7..170b8fbd91 100644 >--- a/drivers/net/octeontx2/otx2_ethdev_ops.c >+++ b/drivers/net/octeontx2/otx2_ethdev_ops.c >@@ -58,7 +58,7 @@ otx2_nix_mtu_set(struct rte_eth_dev *eth_dev, uint16_t >mtu) > if (rc) > return rc; > >- if (frame_size > RTE_ETHER_MAX_LEN) >+ if (mtu > RTE_ETHER_MTU) > dev->rx_offloads |= DEV_RX_OFFLOAD_JUMBO_FRAME; > else > dev->rx_offloads &= ~DEV_RX_OFFLOAD_JUMBO_FRAME; >-- >2.17.1
Acked-by: Sunil Kumar Kori <[email protected]>
