From 16c979f9adf226036187cc4e5777d9511bfe615c Mon Sep 17 00:00:00 2001 From: Helin Zhang Date: Thu, 30 Apr 2015 23:03:08 +0800 Subject: [PATCH] i40e: disable setting of PHY configuration There was a known link issue on 40G ports on NVM version (FVL3E), when setting phy configuration. As a workaround, setting of phy configuration should be disabled. The impact is that the link cannot be forcedly configured, which doesn't affect any feature functions. The workaround can be removed when a formal fix is ready later. Test report: http://www.dpdk.org/ml/archives/dev/2015-May/017384.html Signed-off-by: Helin Zhang Acked-by: Jingjing Wu Acked-by: Jijiang Liu Tested-by: Min Cao --- lib/librte_pmd_i40e/i40e_ethdev.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/lib/librte_pmd_i40e/i40e_ethdev.c b/lib/librte_pmd_i40e/i40e_ethdev.c index 43762f2bf8..c08de4ffc1 100644 --- a/lib/librte_pmd_i40e/i40e_ethdev.c +++ b/lib/librte_pmd_i40e/i40e_ethdev.c @@ -791,6 +791,10 @@ i40e_phy_conf_link(struct i40e_hw *hw, uint8_t abilities, uint8_t force_speed) I40E_LINK_SPEED_100MB; int ret = -ENOTSUP; + /* Skip it on 40G interfaces, as a workaround for the link issue */ + if (i40e_is_40G_device(hw->device_id)) + return I40E_SUCCESS; + status = i40e_aq_get_phy_capabilities(hw, false, false, &phy_ab, NULL); if (status) -- 2.20.1