Fix to substract the macsec sectag size from port MTU during InitializePort (sonic-net#2789)

judyjoseph · web-flow · commit 1e9a8b7f4309 · 2023-05-30T23:26:38.000-07:00
diff --git a/orchagent/portsorch.cpp b/orchagent/portsorch.cpp
@@ -1194,7 +1194,8 @@ bool PortsOrch::getPortMtu(const Port& port, sai_uint32_t &mtu)
 
     mtu = attr.value.u32 - (uint32_t)(sizeof(struct ether_header) + FCS_LEN + VLAN_TAG_LEN);
 
-    if (isMACsecPort(port.m_port_id))
+    /* Reduce the default MTU got from ASIC by MAX_MACSEC_SECTAG_SIZE */
+    if (mtu > MAX_MACSEC_SECTAG_SIZE)
     {
         mtu -= MAX_MACSEC_SECTAG_SIZE;
     }

Original file line number	Diff line number	Diff line change
`@@ -1194,7 +1194,8 @@ bool PortsOrch::getPortMtu(const Port& port, sai_uint32_t &mtu)`
`1194`	`1194`
`1195`	`1195`	`mtu = attr.value.u32 - (uint32_t)(sizeof(struct ether_header) + FCS_LEN + VLAN_TAG_LEN);`
`1196`	`1196`
`1197`		`- if (isMACsecPort(port.m_port_id))`
	`1197`	`+ /* Reduce the default MTU got from ASIC by MAX_MACSEC_SECTAG_SIZE */`
	`1198`	`+ if (mtu > MAX_MACSEC_SECTAG_SIZE)`
`1198`	`1199`	`{`
`1199`	`1200`	`mtu -= MAX_MACSEC_SECTAG_SIZE;`
`1200`	`1201`	`}`