From: "Zeng, ZhichaoX" <zhichaox.zeng@intel.com>
To: "dev@dpdk.org" <dev@dpdk.org>,
"Richardson, Bruce" <bruce.richardson@intel.com>
Cc: "Medvedkin, Vladimir" <vladimir.medvedkin@intel.com>,
"Cui, KaixinX" <kaixinx.cui@intel.com>,
"Yang, Qiming" <qiming.yang@intel.com>,
Yuying Zhang <Yuying.Zhang@intel.com>
Subject: RE: [PATCH v5] net/i40e: support FEC feature
Date: Tue, 18 Jun 2024 03:08:47 +0000 [thread overview]
Message-ID: <CO6PR11MB5602ADAD4BAADE0689A9AC2DF1CE2@CO6PR11MB5602.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20240411092945.1068587-1-zhichaox.zeng@intel.com>
Hi Bruce,
Kindly ping for review~ Thanks.
Regards
Zhichao
> -----Original Message-----
> From: Zeng, ZhichaoX <zhichaox.zeng@intel.com>
> Sent: Thursday, April 11, 2024 5:30 PM
> To: dev@dpdk.org
> Cc: Medvedkin, Vladimir <vladimir.medvedkin@intel.com>; Cui, KaixinX
> <kaixinx.cui@intel.com>; Zeng, ZhichaoX <zhichaox.zeng@intel.com>; Yang,
> Qiming <qiming.yang@intel.com>; Yuying Zhang <Yuying.Zhang@intel.com>
> Subject: [PATCH v5] net/i40e: support FEC feature
>
> This patch enabled querying Forward Error Correction(FEC) capabilities, set
> FEC mode and get current FEC mode functions.
>
> Signed-off-by: Qiming Yang <qiming.yang@intel.com>
> Signed-off-by: Zhichao Zeng <zhichaox.zeng@intel.com>
>
> ---
> v5: fix some judgments
> v4: fix some logic
> v3: optimize code details
> v2: update NIC feature document
> ---
> doc/guides/nics/features/i40e.ini | 1 +
> doc/guides/rel_notes/release_24_07.rst | 4 +
> drivers/net/i40e/i40e_ethdev.c | 237 +++++++++++++++++++++++++
> 3 files changed, 242 insertions(+)
>
> diff --git a/doc/guides/nics/features/i40e.ini
> b/doc/guides/nics/features/i40e.ini
> index ef7514c44b..4610444ace 100644
> --- a/doc/guides/nics/features/i40e.ini
> +++ b/doc/guides/nics/features/i40e.ini
> @@ -32,6 +32,7 @@ Traffic manager = Y
> CRC offload = Y
> VLAN offload = Y
> QinQ offload = P
> +FEC = Y
> L3 checksum offload = P
> L4 checksum offload = P
> Inner L3 checksum = P
> diff --git a/doc/guides/rel_notes/release_24_07.rst
> b/doc/guides/rel_notes/release_24_07.rst
> index a69f24cf99..1e65f70d6c 100644
> --- a/doc/guides/rel_notes/release_24_07.rst
> +++ b/doc/guides/rel_notes/release_24_07.rst
> @@ -55,6 +55,10 @@ New Features
> Also, make sure to start the actual text at the margin.
> =======================================================
>
> +* **Updated Intel i40e driver.**
> +
> + * Added support for configuring the Forward Error Correction(FEC)
> + mode, querying
> + * FEC capabilities and current FEC mode from a device.
>
> Removed Items
> -------------
> diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c
> index 380ce1a720..bc4a62f64b 100644
> --- a/drivers/net/i40e/i40e_ethdev.c
> +++ b/drivers/net/i40e/i40e_ethdev.c
> @@ -406,6 +406,10 @@ static void i40e_ethertype_filter_restore(struct
> i40e_pf *pf); static void i40e_tunnel_filter_restore(struct i40e_pf *pf); static
> void i40e_filter_restore(struct i40e_pf *pf); static void
> i40e_notify_all_vfs_link_status(struct rte_eth_dev *dev);
> +static int i40e_fec_get_capability(struct rte_eth_dev *dev,
> + struct rte_eth_fec_capa *speed_fec_capa, unsigned int num); static int
> +i40e_fec_get(struct rte_eth_dev *dev, uint32_t *fec_capa); static int
> +i40e_fec_set(struct rte_eth_dev *dev, uint32_t fec_capa);
>
> static const char *const valid_keys[] = {
> ETH_I40E_FLOATING_VEB_ARG,
> @@ -521,6 +525,9 @@ static const struct eth_dev_ops i40e_eth_dev_ops = {
> .tm_ops_get = i40e_tm_ops_get,
> .tx_done_cleanup = i40e_tx_done_cleanup,
> .get_monitor_addr = i40e_get_monitor_addr,
> + .fec_get_capability = i40e_fec_get_capability,
> + .fec_get = i40e_fec_get,
> + .fec_set = i40e_fec_set,
> };
>
> /* store statistics names and its offset in stats structure */ @@ -12297,6
> +12304,236 @@ i40e_cloud_filter_qinq_create(struct i40e_pf *pf)
> return ret;
> }
>
> +static int
> +i40e_fec_get_capability(struct rte_eth_dev *dev,
> + struct rte_eth_fec_capa *speed_fec_capa, __rte_unused unsigned int
> +num) {
> + struct i40e_hw *hw = I40E_DEV_PRIVATE_TO_HW(dev->data-
> >dev_private);
> +
> + if (hw->mac.type == I40E_MAC_X722 &&
> + !(hw->flags & I40E_HW_FLAG_X722_FEC_REQUEST_CAPABLE)) {
> + PMD_DRV_LOG(ERR, "Setting FEC encoding not supported
> by"
> + " firmware. Please update the NVM image.\n");
> + return -ENOTSUP;
> + }
> +
> + if (hw->device_id == I40E_DEV_ID_25G_SFP28 ||
> + hw->device_id == I40E_DEV_ID_25G_B) {
> + if (speed_fec_capa) {
> + speed_fec_capa->speed =
> RTE_ETH_SPEED_NUM_25G;
> + speed_fec_capa->capa =
> RTE_ETH_FEC_MODE_CAPA_MASK(NOFEC) |
> +
> RTE_ETH_FEC_MODE_CAPA_MASK(BASER) |
> +
> RTE_ETH_FEC_MODE_CAPA_MASK(AUTO) |
> +
> RTE_ETH_FEC_MODE_CAPA_MASK(RS);
> + }
> +
> + /* since HW only supports 25G */
> + return 1;
> + } else if (hw->device_id == I40E_DEV_ID_KX_X722) {
> + if (speed_fec_capa) {
> + speed_fec_capa->speed =
> RTE_ETH_SPEED_NUM_25G;
> + speed_fec_capa->capa =
> RTE_ETH_FEC_MODE_CAPA_MASK(AUTO) |
> +
> RTE_ETH_FEC_MODE_CAPA_MASK(RS);
> + }
> + return 1;
> + }
> +
> + return -ENOTSUP;
> +}
> +
> +static int
> +i40e_fec_get(struct rte_eth_dev *dev, uint32_t *fec_capa) {
> + struct i40e_hw *hw = I40E_DEV_PRIVATE_TO_HW(dev->data-
> >dev_private);
> + struct i40e_aq_get_phy_abilities_resp abilities = {0};
> + struct i40e_link_status link_status = {0};
> + uint8_t current_fec_mode = 0, fec_config = 0;
> + bool link_up, enable_lse;
> + int ret = 0;
> +
> + enable_lse = dev->data->dev_conf.intr_conf.lsc ? true : false;
> + /* Get link info */
> + ret = i40e_aq_get_link_info(hw, enable_lse, &link_status, NULL);
> + if (ret != I40E_SUCCESS) {
> + PMD_DRV_LOG(ERR, "Failed to get link information: %d\n",
> + ret);
> + return -ENOTSUP;
> + }
> +
> + link_up = link_status.link_info & I40E_AQ_LINK_UP;
> +
> + ret = i40e_aq_get_phy_capabilities(hw, false, false, &abilities,
> + NULL);
> + if (ret) {
> + PMD_DRV_LOG(ERR, "Failed to get PHY capabilities: %d\n",
> + ret);
> + return -ENOTSUP;
> + }
> +
> + /**
> + * If link is down and AUTO is enabled, AUTO is returned,
> + * otherwise, configured FEC mode is returned.
> + * If link is up, current FEC mode is returned.
> + */
> + fec_config = abilities.fec_cfg_curr_mod_ext_info;
> + current_fec_mode = link_status.fec_info;
> +
> + if (link_up) {
> + switch (current_fec_mode) {
> + case I40E_AQ_CONFIG_FEC_KR_ENA:
> + *fec_capa =
> RTE_ETH_FEC_MODE_TO_CAPA(RTE_ETH_FEC_BASER);
> + break;
> + case I40E_AQ_CONFIG_FEC_RS_ENA:
> + *fec_capa =
> RTE_ETH_FEC_MODE_TO_CAPA(RTE_ETH_FEC_RS);
> + break;
> + case 0:
> + *fec_capa =
> RTE_ETH_FEC_MODE_TO_CAPA(RTE_ETH_FEC_NOFEC);
> + break;
> + default:
> + return -EINVAL;
> + }
> + return 0;
> + }
> +
> + if (fec_config & I40E_AQ_ENABLE_FEC_AUTO) {
> + *fec_capa =
> RTE_ETH_FEC_MODE_TO_CAPA(RTE_ETH_FEC_AUTO);
> + return 0;
> + }
> +
> + uint32_t temp_fec_capa = 0;
> + if (fec_config & I40E_AQ_ENABLE_FEC_KR)
> + temp_fec_capa |=
> RTE_ETH_FEC_MODE_TO_CAPA(RTE_ETH_FEC_BASER);
> + if (fec_config & I40E_AQ_ENABLE_FEC_RS)
> + temp_fec_capa |=
> RTE_ETH_FEC_MODE_TO_CAPA(RTE_ETH_FEC_RS);
> + if (temp_fec_capa == 0)
> + temp_fec_capa =
> RTE_ETH_FEC_MODE_TO_CAPA(RTE_ETH_FEC_NOFEC);
> +
> + *fec_capa = temp_fec_capa;
> + return 0;
> +}
> +
> +static int
> +i40e_fec_set(struct rte_eth_dev *dev, uint32_t fec_capa) {
> + struct i40e_hw *hw = I40E_DEV_PRIVATE_TO_HW(dev->data-
> >dev_private);
> + struct i40e_aq_get_phy_abilities_resp abilities = {0};
> + struct i40e_aq_set_phy_config config = {0};
> + enum i40e_status_code status;
> + uint8_t req_fec = 0, fec_auto = 0, fec_kr = 0, fec_rs = 0;
> +
> + if (hw->device_id != I40E_DEV_ID_25G_SFP28 &&
> + hw->device_id != I40E_DEV_ID_25G_B &&
> + hw->device_id != I40E_DEV_ID_KX_X722) {
> + return -ENOTSUP;
> + }
> +
> + if (hw->mac.type == I40E_MAC_X722 &&
> + !(hw->flags & I40E_HW_FLAG_X722_FEC_REQUEST_CAPABLE)) {
> + PMD_DRV_LOG(ERR, "Setting FEC encoding not supported
> by"
> + " firmware. Please update the NVM image.\n");
> + return -ENOTSUP;
> + }
> +
> + /**
> + * Copy the current user PHY configuration. The current user PHY
> + * configuration is initialized during probe from PHY capabilities
> + * software mode, and updated on set PHY configuration.
> + */
> + if (fec_capa == 0)
> + return -EINVAL;
> +
> + if (fec_capa & RTE_ETH_FEC_MODE_CAPA_MASK(AUTO))
> + fec_auto = 1;
> +
> + if (fec_capa & RTE_ETH_FEC_MODE_CAPA_MASK(BASER))
> + fec_kr = 1;
> +
> + if (fec_capa & RTE_ETH_FEC_MODE_CAPA_MASK(RS))
> + fec_rs = 1;
> +
> + if (fec_auto) {
> + if (hw->mac.type == I40E_MAC_X722) {
> + PMD_DRV_LOG(ERR, "X722 Unsupported FEC mode:
> AUTO");
> + return -EINVAL;
> + }
> + if (fec_kr || fec_rs) {
> + if (fec_kr)
> + req_fec = I40E_AQ_SET_FEC_ABILITY_KR |
> +
> I40E_AQ_SET_FEC_REQUEST_KR;
> + if (fec_rs) {
> + if (hw->mac.type == I40E_MAC_X722) {
> + PMD_DRV_LOG(ERR, "X722
> Unsupported FEC mode: RS");
> + return -EINVAL;
> + }
> + req_fec = I40E_AQ_SET_FEC_ABILITY_RS |
> +
> I40E_AQ_SET_FEC_REQUEST_RS;
> + }
> + } else {
> + if (hw->mac.type == I40E_MAC_X722) {
> + req_fec = I40E_AQ_SET_FEC_ABILITY_KR |
> +
> I40E_AQ_SET_FEC_REQUEST_KR;
> + } else {
> + req_fec = I40E_AQ_SET_FEC_ABILITY_KR |
> +
> I40E_AQ_SET_FEC_REQUEST_KR |
> +
> I40E_AQ_SET_FEC_ABILITY_RS |
> +
> I40E_AQ_SET_FEC_REQUEST_RS;
> + }
> + }
> + } else {
> + if (fec_kr ^ fec_rs) {
> + if (fec_kr) {
> + req_fec = I40E_AQ_SET_FEC_ABILITY_KR |
> +
> I40E_AQ_SET_FEC_REQUEST_KR;
> + } else {
> + if (hw->mac.type == I40E_MAC_X722) {
> + PMD_DRV_LOG(ERR, "X722
> Unsupported FEC mode: RS");
> + return -EINVAL;
> + }
> + req_fec = I40E_AQ_SET_FEC_ABILITY_RS |
> +
> I40E_AQ_SET_FEC_REQUEST_RS;
> + }
> + } else {
> + return -EINVAL;
> + }
> + }
> +
> + /* Get the current phy config */
> + status = i40e_aq_get_phy_capabilities(hw, false, false, &abilities,
> + NULL);
> + if (status) {
> + PMD_DRV_LOG(ERR, "Failed to get PHY capabilities: %d\n",
> + status);
> + return -ENOTSUP;
> + }
> +
> + if (abilities.fec_cfg_curr_mod_ext_info != req_fec) {
> + config.phy_type = abilities.phy_type;
> + config.abilities = abilities.abilities |
> + I40E_AQ_PHY_ENABLE_ATOMIC_LINK;
> + config.phy_type_ext = abilities.phy_type_ext;
> + config.link_speed = abilities.link_speed;
> + config.eee_capability = abilities.eee_capability;
> + config.eeer = abilities.eeer_val;
> + config.low_power_ctrl = abilities.d3_lpan;
> + config.fec_config = req_fec &
> I40E_AQ_PHY_FEC_CONFIG_MASK;
> + status = i40e_aq_set_phy_config(hw, &config, NULL);
> + if (status) {
> + PMD_DRV_LOG(ERR, "Failed to set PHY
> capabilities: %d\n",
> + status);
> + return -ENOTSUP;
> + }
> + }
> +
> + status = i40e_update_link_info(hw);
> + if (status) {
> + PMD_DRV_LOG(ERR, "Failed to set PHY capabilities: %d\n",
> + status);
> + return -ENOTSUP;
> + }
> +
> + return 0;
> +}
> +
> RTE_LOG_REGISTER_SUFFIX(i40e_logtype_init, init, NOTICE);
> RTE_LOG_REGISTER_SUFFIX(i40e_logtype_driver, driver, NOTICE); #ifdef
> RTE_ETHDEV_DEBUG_RX
> --
> 2.34.1
prev parent reply other threads:[~2024-06-18 3:08 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-12-20 8:47 [PATCH] " Qiming Yang
2024-03-06 10:41 ` [PATCH v4] " Zhichao Zeng
2024-03-11 15:59 ` Medvedkin, Vladimir
2024-03-12 8:44 ` Zeng, ZhichaoX
2024-03-14 21:41 ` Medvedkin, Vladimir
2024-04-11 8:18 ` [PATCH v5] " Zhichao Zeng
2024-04-11 9:29 ` Zhichao Zeng
2024-06-18 3:08 ` Zeng, ZhichaoX [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CO6PR11MB5602ADAD4BAADE0689A9AC2DF1CE2@CO6PR11MB5602.namprd11.prod.outlook.com \
--to=zhichaox.zeng@intel.com \
--cc=Yuying.Zhang@intel.com \
--cc=bruce.richardson@intel.com \
--cc=dev@dpdk.org \
--cc=kaixinx.cui@intel.com \
--cc=qiming.yang@intel.com \
--cc=vladimir.medvedkin@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).