[PATCH v2 36/45] common/sfc_efx/base: correct MAC PDU calculation on Medford4
Ivan Malov
ivan.malov at arknetworks.am
Wed Apr 23 17:59:53 CEST 2025
For managing MAC PDU (max. frame size), client drivers apply
EFX macros to switch between PDU and SDU forms. These macros
include a workaround for a bug that dates back to Siena NICs.
Starting with Medford4, the bug is no longer there and it is
wrong to use the macros, so provide users with a replacement.
The new APIs will either include the said workaround or omit
it, depending on whether support for netport MCDI is present.
Signed-off-by: Ivan Malov <ivan.malov at arknetworks.am>
Reviewed-by: Andy Moreton <andy.moreton at amd.com>
Reviewed-by: Pieter Jansen Van Vuuren <pieter.jansen-van-vuuren at amd.com>
---
drivers/common/sfc_efx/base/efx.h | 25 +++++++++++++++++++++++
drivers/common/sfc_efx/base/efx_mac.c | 19 +++++++++++++++--
drivers/common/sfc_efx/base/efx_nic.c | 3 +++
drivers/common/sfc_efx/sfc_base_symbols.c | 1 +
4 files changed, 46 insertions(+), 2 deletions(-)
diff --git a/drivers/common/sfc_efx/base/efx.h b/drivers/common/sfc_efx/base/efx.h
index 6ca108cffe..73dc38f84e 100644
--- a/drivers/common/sfc_efx/base/efx.h
+++ b/drivers/common/sfc_efx/base/efx.h
@@ -668,24 +668,45 @@ typedef enum efx_link_mode_e {
#define EFX_MAC_SDU_MAX 9202
+/*
+ * NOTE: the PDU macros implement an obsolete workaround that is needed for
+ * MC_CMD_SET_MAC; do not use the PDU macros for the netport MCDI commands,
+ * which do not use the workaround.
+ */
+
#define EFX_MAC_PDU_ADJUSTMENT \
(/* EtherII */ 14 \
+ /* VLAN */ 4 \
+ /* CRC */ 4 \
+ /* bug16011 */ 16) \
+/* NOTE: this macro is deprecated; use efx_mac_pdu_from_sdu(). */
#define EFX_MAC_PDU(_sdu) \
EFX_P2ROUNDUP(size_t, (_sdu) + EFX_MAC_PDU_ADJUSTMENT, 8)
/*
* Due to the EFX_P2ROUNDUP in EFX_MAC_PDU(), EFX_MAC_SDU_FROM_PDU() may give
* the SDU rounded up slightly.
+ *
+ * NOTE: do not use this macro in new code as it is
+ * incorrect for the netport MCDI commands.
*/
#define EFX_MAC_SDU_FROM_PDU(_pdu) ((_pdu) - EFX_MAC_PDU_ADJUSTMENT)
#define EFX_MAC_PDU_MIN 60
+
+/* NOTE: this macro is deprecated; use encp->enc_mac_pdu_max. */
#define EFX_MAC_PDU_MAX EFX_MAC_PDU(EFX_MAC_SDU_MAX)
+/*
+ * For use with efx_mac_pdu_set(), convert the given SDU value to its PDU form.
+ */
+LIBEFX_API
+extern size_t
+efx_mac_pdu_from_sdu(
+ __in efx_nic_t *enp,
+ __in size_t sdu);
+
LIBEFX_API
extern __checkReturn efx_rc_t
efx_mac_pdu_get(
@@ -1729,6 +1750,10 @@ typedef struct efx_nic_cfg_s {
efx_nic_dma_mapping_t enc_dma_mapping;
/* Physical ports shared by PFs */
efx_port_usage_t enc_port_usage;
+ /* Minimum MAC PDU value to use with efx_mac_pdu_set() */
+ uint32_t enc_mac_pdu_min;
+ /* Maximum MAC PDU value to use with efx_mac_pdu_set() */
+ uint32_t enc_mac_pdu_max;
} efx_nic_cfg_t;
#define EFX_PCI_VF_INVALID 0xffff
diff --git a/drivers/common/sfc_efx/base/efx_mac.c b/drivers/common/sfc_efx/base/efx_mac.c
index 9a0cf64fd9..69f053ce44 100644
--- a/drivers/common/sfc_efx/base/efx_mac.c
+++ b/drivers/common/sfc_efx/base/efx_mac.c
@@ -114,6 +114,20 @@ static const efx_mac_ops_t __efx_mac_medford4_ops = {
};
#endif /* EFSYS_OPT_MEDFORD4 */
+ size_t
+efx_mac_pdu_from_sdu(
+ __in efx_nic_t *enp,
+ __in size_t sdu)
+{
+ if (efx_np_supported(enp) != B_FALSE) {
+ /* PDU size for netport MCDI capable adaptors. */
+ return sdu + 14 /* ETH */ + 4 /* VLAN */ + 4 /* FCS */;
+ } else {
+ /* PDU size for legacy MC_CMD_SET_MAC command. */
+ return EFX_MAC_PDU(sdu);
+ }
+}
+
__checkReturn efx_rc_t
efx_mac_pdu_set(
__in efx_nic_t *enp,
@@ -121,6 +135,7 @@ efx_mac_pdu_set(
{
efx_port_t *epp = &(enp->en_port);
const efx_mac_ops_t *emop = epp->ep_emop;
+ efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
uint32_t old_pdu;
efx_rc_t rc;
@@ -128,12 +143,12 @@ efx_mac_pdu_set(
EFSYS_ASSERT3U(enp->en_mod_flags, &, EFX_MOD_PORT);
EFSYS_ASSERT(emop != NULL);
- if (pdu < EFX_MAC_PDU_MIN) {
+ if (pdu < encp->enc_mac_pdu_min) {
rc = EINVAL;
goto fail1;
}
- if (pdu > EFX_MAC_PDU_MAX) {
+ if (pdu > encp->enc_mac_pdu_max) {
rc = EINVAL;
goto fail2;
}
diff --git a/drivers/common/sfc_efx/base/efx_nic.c b/drivers/common/sfc_efx/base/efx_nic.c
index 1ec684da40..1c25270792 100644
--- a/drivers/common/sfc_efx/base/efx_nic.c
+++ b/drivers/common/sfc_efx/base/efx_nic.c
@@ -491,6 +491,9 @@ efx_nic_probe(
encp->enc_features = enp->en_features;
+ encp->enc_mac_pdu_max = efx_mac_pdu_from_sdu(enp, EFX_MAC_SDU_MAX);
+ encp->enc_mac_pdu_min = EFX_MAC_PDU_MIN;
+
if ((rc = efx_phy_probe(enp)) != 0)
goto fail2;
diff --git a/drivers/common/sfc_efx/sfc_base_symbols.c b/drivers/common/sfc_efx/sfc_base_symbols.c
index ae6605632d..0e74034031 100644
--- a/drivers/common/sfc_efx/sfc_base_symbols.c
+++ b/drivers/common/sfc_efx/sfc_base_symbols.c
@@ -59,6 +59,7 @@ RTE_EXPORT_INTERNAL_SYMBOL(efx_intr_trigger)
RTE_EXPORT_INTERNAL_SYMBOL(efx_intr_status_line)
RTE_EXPORT_INTERNAL_SYMBOL(efx_intr_status_message)
RTE_EXPORT_INTERNAL_SYMBOL(efx_intr_fatal)
+RTE_EXPORT_INTERNAL_SYMBOL(efx_mac_pdu_from_sdu)
RTE_EXPORT_INTERNAL_SYMBOL(efx_mac_pdu_set)
RTE_EXPORT_INTERNAL_SYMBOL(efx_mac_pdu_get)
RTE_EXPORT_INTERNAL_SYMBOL(efx_mac_addr_set)
--
2.39.5
More information about the dev
mailing list