* [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link @ 2017-01-10 12:32 Christos Ricudis 2017-01-10 15:28 ` Olivier MATZ 0 siblings, 1 reply; 16+ messages in thread From: Christos Ricudis @ 2017-01-10 12:32 UTC (permalink / raw) To: dev Hello, Using a X710 based 4-port 4x10Gbit NIC, I have came across the following issue on the i40e PMD: When an optical SFP+ (Finisar FTLX8571D3BCL) is used with no active link partner on the other end of the link (or fiber completely disconnected from the SFP+), i40e_aq_get_phy_capabilities() (called by i40e_dev_sync_phy_type() on port initialization), fails with a 0x05 return value (EIO) on the AQ response structure. The struct i40e_aq_get_phy_abilities_resp buffer passed to the Get Phy Abilities command is unmodified upon return. This prevents DPDK 16.11 from initializing the port, and ultimately fails with the following error: PMD: eth_i40e_dev_init(): Failed to sync phy type: -95 The change introducing this issue was http://dpdk.org/ml/archives/dev/2016-September/047663.html Reading the X710 datasheet, I notice that no specific mention is given on the meaning of EIO as a response to Get PHY Abilities command (opcode 0x0600), whereas in most other commands, an explicit mention of the meaning of the possible error status responses is given. This behaviour is the same across the following NVM releases: FW 4.33 API 1.2 NVM 04.04.02 eetrack 800018a6 FW 4.40 API 1.4 NVM 04.05.03 eetrack 80001cd8 FW 5.0 API 1.5 NVM 05.00.04 eetrack 800024da I will try to get around the issue by falling back to PHY capabilities detection using the device ID in the case i40e_aq_get_phy_capabilities() fails, but conceptually the capabilities of the PHY should not be dependent on whether PHY detects an active link or not. I’d be happy to do more testing on this issue per your recommendations. Moreover, while trying to debug this issue, I managed to get both 3 NIC adapters on my test system on a state where the PHY has apparently died - no link indication at all on any ports. A reboot solved this, and I am now trying to replicate this behaviour under more controlled conditions. Best regards, Christos Ricudis. ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-01-10 12:32 [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link Christos Ricudis @ 2017-01-10 15:28 ` Olivier MATZ 2017-01-11 1:08 ` Zhang, Helin 0 siblings, 1 reply; 16+ messages in thread From: Olivier MATZ @ 2017-01-10 15:28 UTC (permalink / raw) To: Christos Ricudis; +Cc: dev, helin.zhang, jingjing.wu Hi Christos, +CC i40e maintainers On Tue, 10 Jan 2017 20:32:26 +0800, Christos Ricudis <ricudis.christos@gmail.com> wrote: > Hello, > > Using a X710 based 4-port 4x10Gbit NIC, I have came across the > following issue on the i40e PMD: > > When an optical SFP+ (Finisar FTLX8571D3BCL) is used with no active > link partner on the other end of the link (or fiber completely > disconnected from the SFP+), i40e_aq_get_phy_capabilities() (called > by i40e_dev_sync_phy_type() on port initialization), fails with a > 0x05 return value (EIO) on the AQ response structure. The struct > i40e_aq_get_phy_abilities_resp buffer passed to the Get Phy Abilities > command is unmodified upon return. > > This prevents DPDK 16.11 from initializing the port, and ultimately > fails with the following error: > > PMD: eth_i40e_dev_init(): Failed to sync phy type: -95 > > The change introducing this issue was > http://dpdk.org/ml/archives/dev/2016-September/047663.html > > Reading the X710 datasheet, I notice that no specific mention is > given on the meaning of EIO as a response to Get PHY Abilities > command (opcode 0x0600), whereas in most other commands, an explicit > mention of the meaning of the possible error status responses is > given. > > This behaviour is the same across the following NVM releases: > > FW 4.33 API 1.2 NVM 04.04.02 eetrack 800018a6 > FW 4.40 API 1.4 NVM 04.05.03 eetrack 80001cd8 > FW 5.0 API 1.5 NVM 05.00.04 eetrack 800024da > > I will try to get around the issue by falling back to PHY > capabilities detection using the device ID in the case > i40e_aq_get_phy_capabilities() fails, but conceptually the > capabilities of the PHY should not be dependent on whether PHY > detects an active link or not. > > I’d be happy to do more testing on this issue per your > recommendations. > > Moreover, while trying to debug this issue, I managed to get both 3 > NIC adapters on my test system on a state where the PHY has > apparently died - no link indication at all on any ports. A reboot > solved this, and I am now trying to replicate this behaviour under > more controlled conditions. > I'm currently running into a similar issue (I think). I can reproduce it with testpmd with the following case: set link_check off port stop 0 # don't wait between these 2 commands port start 0 I added some logs that are displayed after the port start: PMD: i40e_set_tx_function(): Vector tx finally be used. PMD: i40e_set_rx_function(): Vector rx enabled, please make sure RX burst size no less than 4 (port=0). PMD: i40e_dev_rx_queue_start(): >> PMD: i40e_dev_tx_queue_start(): >> PMD: i40e_dev_start(): applying link settings... PMD: i40e_apply_link_speed(): abilities = 38, speed = 0 PMD: i40e_phy_conf_link(): i40e_aq_get_phy_capabilities failed -7 PMD: i40e_dev_start(): Fail to apply link setting PMD: i40e_dev_clear_queues(): >> The -7 corresponds to I40E_ERR_UNKNOWN_PHY. This happens in i40e_aq_get_phy_capabilities() in the following code, which makes me think it's the same problem than yours: if (hw->aq.asq_last_status == I40E_AQ_RC_EIO) status = I40E_ERR_UNKNOWN_PHY; A workaround in my usecase is to wait a bit between the stop and the start. Any help is welcome. Regards, Olivier ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-01-10 15:28 ` Olivier MATZ @ 2017-01-11 1:08 ` Zhang, Helin 2017-01-11 3:00 ` Christos Ricudis 0 siblings, 1 reply; 16+ messages in thread From: Zhang, Helin @ 2017-01-11 1:08 UTC (permalink / raw) To: Olivier MATZ, Christos Ricudis, Rowden, Aaron F; +Cc: dev, Wu, Jingjing Hi Aaron Is the SFP+ (Finisar FTLX8571D3BCL) supported and validated by Intel? It seems there is some PHY issue in this case. Regards, Helin -----Original Message----- From: Olivier MATZ [mailto:olivier.matz@6wind.com] Sent: Tuesday, January 10, 2017 11:29 PM To: Christos Ricudis <ricudis.christos@gmail.com> Cc: dev@dpdk.org; Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com> Subject: Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link Hi Christos, +CC i40e maintainers On Tue, 10 Jan 2017 20:32:26 +0800, Christos Ricudis <ricudis.christos@gmail.com> wrote: > Hello, > > Using a X710 based 4-port 4x10Gbit NIC, I have came across the > following issue on the i40e PMD: > > When an optical SFP+ (Finisar FTLX8571D3BCL) is used with no active > link partner on the other end of the link (or fiber completely > disconnected from the SFP+), i40e_aq_get_phy_capabilities() (called by > i40e_dev_sync_phy_type() on port initialization), fails with a > 0x05 return value (EIO) on the AQ response structure. The struct > i40e_aq_get_phy_abilities_resp buffer passed to the Get Phy Abilities > command is unmodified upon return. > > This prevents DPDK 16.11 from initializing the port, and ultimately > fails with the following error: > > PMD: eth_i40e_dev_init(): Failed to sync phy type: -95 > > The change introducing this issue was > http://dpdk.org/ml/archives/dev/2016-September/047663.html > > Reading the X710 datasheet, I notice that no specific mention is given > on the meaning of EIO as a response to Get PHY Abilities command > (opcode 0x0600), whereas in most other commands, an explicit mention > of the meaning of the possible error status responses is given. > > This behaviour is the same across the following NVM releases: > > FW 4.33 API 1.2 NVM 04.04.02 eetrack 800018a6 FW 4.40 API 1.4 NVM > 04.05.03 eetrack 80001cd8 FW 5.0 API 1.5 NVM 05.00.04 eetrack 800024da > > I will try to get around the issue by falling back to PHY capabilities > detection using the device ID in the case > i40e_aq_get_phy_capabilities() fails, but conceptually the > capabilities of the PHY should not be dependent on whether PHY detects > an active link or not. > > I’d be happy to do more testing on this issue per your > recommendations. > > Moreover, while trying to debug this issue, I managed to get both 3 > NIC adapters on my test system on a state where the PHY has apparently > died - no link indication at all on any ports. A reboot solved this, > and I am now trying to replicate this behaviour under more controlled > conditions. > I'm currently running into a similar issue (I think). I can reproduce it with testpmd with the following case: set link_check off port stop 0 # don't wait between these 2 commands port start 0 I added some logs that are displayed after the port start: PMD: i40e_set_tx_function(): Vector tx finally be used. PMD: i40e_set_rx_function(): Vector rx enabled, please make sure RX burst size no less than 4 (port=0). PMD: i40e_dev_rx_queue_start(): >> PMD: i40e_dev_tx_queue_start(): >> PMD: i40e_dev_start(): applying link settings... PMD: i40e_apply_link_speed(): abilities = 38, speed = 0 PMD: i40e_phy_conf_link(): i40e_aq_get_phy_capabilities failed -7 PMD: i40e_dev_start(): Fail to apply link setting PMD: i40e_dev_clear_queues(): >> The -7 corresponds to I40E_ERR_UNKNOWN_PHY. This happens in i40e_aq_get_phy_capabilities() in the following code, which makes me think it's the same problem than yours: if (hw->aq.asq_last_status == I40E_AQ_RC_EIO) status = I40E_ERR_UNKNOWN_PHY; A workaround in my usecase is to wait a bit between the stop and the start. Any help is welcome. Regards, Olivier ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-01-11 1:08 ` Zhang, Helin @ 2017-01-11 3:00 ` Christos Ricudis [not found] ` <FC7A99FCE8F15942AB8D6F91FF66D8F694836A60@ORSMSX112.amr.corp.intel.com> 0 siblings, 1 reply; 16+ messages in thread From: Christos Ricudis @ 2017-01-11 3:00 UTC (permalink / raw) To: Zhang, Helin; +Cc: Olivier MATZ, Rowden, Aaron F, dev, Wu, Jingjing Hi Helin, > On 11 Jan 2017, at 09:08, Zhang, Helin <helin.zhang@intel.com> wrote: > > Hi Aaron > > Is the SFP+ (Finisar FTLX8571D3BCL) supported and validated by Intel? > It seems there is some PHY issue in this case. As the original reporter of this issue, I will test with validated SFP+s and will report on my testing. Shouldn’t unsupported SFP+s be blacklisted in the I40E driver? Best regards, Christos Ricudis > > Regards, > Helin > > -----Original Message----- > From: Olivier MATZ [mailto:olivier.matz@6wind.com] > Sent: Tuesday, January 10, 2017 11:29 PM > To: Christos Ricudis <ricudis.christos@gmail.com> > Cc: dev@dpdk.org; Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com> > Subject: Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link > > Hi Christos, > > +CC i40e maintainers > > On Tue, 10 Jan 2017 20:32:26 +0800, Christos Ricudis <ricudis.christos@gmail.com> wrote: >> Hello, >> >> Using a X710 based 4-port 4x10Gbit NIC, I have came across the >> following issue on the i40e PMD: >> >> When an optical SFP+ (Finisar FTLX8571D3BCL) is used with no active >> link partner on the other end of the link (or fiber completely >> disconnected from the SFP+), i40e_aq_get_phy_capabilities() (called by >> i40e_dev_sync_phy_type() on port initialization), fails with a >> 0x05 return value (EIO) on the AQ response structure. The struct >> i40e_aq_get_phy_abilities_resp buffer passed to the Get Phy Abilities >> command is unmodified upon return. >> >> This prevents DPDK 16.11 from initializing the port, and ultimately >> fails with the following error: >> >> PMD: eth_i40e_dev_init(): Failed to sync phy type: -95 >> >> The change introducing this issue was >> http://dpdk.org/ml/archives/dev/2016-September/047663.html >> >> Reading the X710 datasheet, I notice that no specific mention is given >> on the meaning of EIO as a response to Get PHY Abilities command >> (opcode 0x0600), whereas in most other commands, an explicit mention >> of the meaning of the possible error status responses is given. >> >> This behaviour is the same across the following NVM releases: >> >> FW 4.33 API 1.2 NVM 04.04.02 eetrack 800018a6 FW 4.40 API 1.4 NVM >> 04.05.03 eetrack 80001cd8 FW 5.0 API 1.5 NVM 05.00.04 eetrack 800024da >> >> I will try to get around the issue by falling back to PHY capabilities >> detection using the device ID in the case >> i40e_aq_get_phy_capabilities() fails, but conceptually the >> capabilities of the PHY should not be dependent on whether PHY detects >> an active link or not. >> >> I’d be happy to do more testing on this issue per your >> recommendations. >> >> Moreover, while trying to debug this issue, I managed to get both 3 >> NIC adapters on my test system on a state where the PHY has apparently >> died - no link indication at all on any ports. A reboot solved this, >> and I am now trying to replicate this behaviour under more controlled >> conditions. >> > > I'm currently running into a similar issue (I think). I can reproduce it with testpmd with the following case: > > set link_check off > port stop 0 > # don't wait between these 2 commands > port start 0 > > I added some logs that are displayed after the port start: > > PMD: i40e_set_tx_function(): Vector tx finally be used. > PMD: i40e_set_rx_function(): Vector rx enabled, please make sure RX > burst size no less than 4 (port=0). > PMD: i40e_dev_rx_queue_start(): >> > PMD: i40e_dev_tx_queue_start(): >> > PMD: i40e_dev_start(): applying link settings... > PMD: i40e_apply_link_speed(): abilities = 38, speed = 0 > PMD: i40e_phy_conf_link(): i40e_aq_get_phy_capabilities failed -7 > PMD: i40e_dev_start(): Fail to apply link setting > PMD: i40e_dev_clear_queues(): >> > > The -7 corresponds to I40E_ERR_UNKNOWN_PHY. This happens in > i40e_aq_get_phy_capabilities() in the following code, which makes me think it's the same problem than yours: > > if (hw->aq.asq_last_status == I40E_AQ_RC_EIO) > status = I40E_ERR_UNKNOWN_PHY; > > A workaround in my usecase is to wait a bit between the stop and the start. > > Any help is welcome. > > Regards, > Olivier > > ^ permalink raw reply [flat|nested] 16+ messages in thread
[parent not found: <FC7A99FCE8F15942AB8D6F91FF66D8F694836A60@ORSMSX112.amr.corp.intel.com>]
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link [not found] ` <FC7A99FCE8F15942AB8D6F91FF66D8F694836A60@ORSMSX112.amr.corp.intel.com> @ 2017-01-12 13:55 ` Olivier MATZ 2017-01-13 13:24 ` Olivier Matz 2017-01-18 10:15 ` Christos Ricudis 0 siblings, 2 replies; 16+ messages in thread From: Olivier MATZ @ 2017-01-12 13:55 UTC (permalink / raw) To: Rowden, Aaron F Cc: Christos Ricudis, Zhang, Helin, Olivier MATZ, dev, Wu, Jingjing Hi, On Wed, 11 Jan 2017 20:51:58 +0000, "Rowden, Aaron F" <aaron.f.rowden@intel.com> wrote: > Hi Helin, > > I'm checking on this to see why it could be failing but I don’t think > this is one part of formal validation. Intel modules are always what > is recommended. > > Aaron > > > Hi Helin, > > > > > On 11 Jan 2017, at 09:08, Zhang, Helin <helin.zhang@intel.com> > > > wrote: > > > > > > Hi Aaron > > > > > > Is the SFP+ (Finisar FTLX8571D3BCL) supported and validated by > > > Intel? It seems there is some PHY issue in this case. > > > > As the original reporter of this issue, I will test with validated > > SFP+s and will report on my testing. > > > > Shouldn’t unsupported SFP+s be blacklisted in the I40E driver? > > Just to let you know that in my case the SFP are Intel ones. Maybe it's a different issue. I see there are some i40e fixes in the net-next repo, I'll give a try with this version. Regards, Olivier ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-01-12 13:55 ` Olivier MATZ @ 2017-01-13 13:24 ` Olivier Matz 2017-01-17 12:50 ` Wu, Jingjing 2017-01-18 10:15 ` Christos Ricudis 1 sibling, 1 reply; 16+ messages in thread From: Olivier Matz @ 2017-01-13 13:24 UTC (permalink / raw) To: Olivier MATZ Cc: Rowden, Aaron F, Christos Ricudis, Zhang, Helin, dev, Wu, Jingjing Hi, On Thu, 12 Jan 2017 14:55:54 +0100, Olivier MATZ <olivier.matz@dev.6wind.com> wrote: > Hi, > > On Wed, 11 Jan 2017 20:51:58 +0000, "Rowden, Aaron F" > <aaron.f.rowden@intel.com> wrote: > > Hi Helin, > > > > I'm checking on this to see why it could be failing but I don’t > > think this is one part of formal validation. Intel modules are > > always what is recommended. > > > > Aaron > > > > > Hi Helin, > > > > > > > On 11 Jan 2017, at 09:08, Zhang, Helin <helin.zhang@intel.com> > > > > wrote: > > > > > > > > Hi Aaron > > > > > > > > Is the SFP+ (Finisar FTLX8571D3BCL) supported and validated by > > > > Intel? It seems there is some PHY issue in this case. > > > > > > As the original reporter of this issue, I will test with validated > > > SFP+s and will report on my testing. > > > > > > Shouldn’t unsupported SFP+s be blacklisted in the I40E driver? > > > > > Just to let you know that in my case the SFP are Intel ones. > Maybe it's a different issue. > > I see there are some i40e fixes in the net-next repo, I'll give a try > with this version. The issue still exists in net-next. I did a git bissect, and the commit that introduces the issue is: f4668a33efe5 ("net/i40e: fix link status change interrupt") [1] If I revert it (with some conflicts), the problem I described in [2] disappear. Helin, Jinging, do you know what would be the consequences of reverting this patch? Christos, I don't know if it also helps for yor issue. If no, sorry for having squatted your topic, the symptoms looked quite similar at first glance. Thanks, Olivier [1] http://dpdk.org/browse/dpdk/commit/?id=f4668a33efe5 [2] http://dpdk.org/ml/archives/dev/2017-January/054401.html ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-01-13 13:24 ` Olivier Matz @ 2017-01-17 12:50 ` Wu, Jingjing 0 siblings, 0 replies; 16+ messages in thread From: Wu, Jingjing @ 2017-01-17 12:50 UTC (permalink / raw) To: Olivier Matz, Olivier MATZ Cc: Rowden, Aaron F, Christos Ricudis, Zhang, Helin, dev Hi, Oliver Let me Christos reports this issue is introduced by the http://dpdk.org/ml/archives/dev/2016-September/047663.html Which is the commit commit edfb226f69bf68d514c9abae4b99b98aeb7c6a32 Author: Qi Zhang <qi.z.zhang@intel.com> Date: Tue Sep 27 09:37:21 2016 +0800 net/i40e: use PHY type to check PHY capability Using device ID to check PHY capability is not extensible. Now we are using PHY type to detect PHY capability. All link speeds supported by the device are encoded into PHY type, and PHY type value can be read by admin queue "get_phy_capability" command at initialization stage. While, your issue seems been introduce by commit f4668a33efe5311301e10bc6e3709a2eccb446ad Author: Qiming Yang <qiming.yang@intel.com> Date: Fri Nov 4 17:10:50 2016 +0800 net/i40e: fix link status change interrupt Previously, link status interrupt in i40e is achieved by checking LINK_STAT_CHANGE_MASK in PFINT_ICR0 register which is provided only for diagnostic use. Instead, drivers need to get the link status change notification by using LSE (Link Status Event). This patch enables LSE and calls LSC callback when the event is received. This patch also removes the processing on LINK_STAT_CHANGE_MASK. As my understanding, the first commit is try to get the phy type during initialization time, it may cause error according to different PHYs. But about the second one, I think the changes are only related to LSC (Link status change interrupt). And I didn’t get the relationship between your issue and this patch. Anyway, About the question " the consequences of reverting this patch", the answer is It may affect the feature Link status change interrupt. Without the patch, there is an issue like: When lsc is enabled, and link status change happens, interrupt is trigger, but the link status queried by API rte_eth_link_get is not changed at all immediately. Thanks Jingjing > -----Original Message----- > From: Olivier Matz [mailto:olivier.matz@6wind.com] > Sent: Friday, January 13, 2017 9:25 PM > To: Olivier MATZ <olivier.matz@dev.6wind.com> > Cc: Rowden, Aaron F <aaron.f.rowden@intel.com>; Christos Ricudis > <ricudis.christos@gmail.com>; Zhang, Helin <helin.zhang@intel.com>; > dev@dpdk.org; Wu, Jingjing <jingjing.wu@intel.com> > Subject: Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ > with no link > > Hi, > > On Thu, 12 Jan 2017 14:55:54 +0100, Olivier MATZ > <olivier.matz@dev.6wind.com> wrote: > > Hi, > > > > On Wed, 11 Jan 2017 20:51:58 +0000, "Rowden, Aaron F" > > <aaron.f.rowden@intel.com> wrote: > > > Hi Helin, > > > > > > I'm checking on this to see why it could be failing but I don’t > > > think this is one part of formal validation. Intel modules are > > > always what is recommended. > > > > > > Aaron > > > > > > > Hi Helin, > > > > > > > > > On 11 Jan 2017, at 09:08, Zhang, Helin <helin.zhang@intel.com> > > > > > wrote: > > > > > > > > > > Hi Aaron > > > > > > > > > > Is the SFP+ (Finisar FTLX8571D3BCL) supported and validated by > > > > > Intel? It seems there is some PHY issue in this case. > > > > > > > > As the original reporter of this issue, I will test with validated > > > > SFP+s and will report on my testing. > > > > > > > > Shouldn’t unsupported SFP+s be blacklisted in the I40E driver? > > > > > > > > Just to let you know that in my case the SFP are Intel ones. > > Maybe it's a different issue. > > > > I see there are some i40e fixes in the net-next repo, I'll give a try > > with this version. > > The issue still exists in net-next. > > I did a git bissect, and the commit that introduces the issue is: > f4668a33efe5 ("net/i40e: fix link status change interrupt") [1] > > If I revert it (with some conflicts), the problem I described in [2] disappear. > > Helin, Jinging, do you know what would be the consequences of reverting this > patch? > > Christos, I don't know if it also helps for yor issue. If no, sorry for having > squatted your topic, the symptoms looked quite similar at first glance. > > Thanks, > Olivier > > > [1] http://dpdk.org/browse/dpdk/commit/?id=f4668a33efe5 > [2] http://dpdk.org/ml/archives/dev/2017-January/054401.html ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-01-12 13:55 ` Olivier MATZ 2017-01-13 13:24 ` Olivier Matz @ 2017-01-18 10:15 ` Christos Ricudis 2017-02-05 15:30 ` Ivan Nardi 1 sibling, 1 reply; 16+ messages in thread From: Christos Ricudis @ 2017-01-18 10:15 UTC (permalink / raw) To: Olivier MATZ; +Cc: Rowden, Aaron F, Zhang, Helin, dev, Wu, Jingjing Hi, > On 12 Jan 2017, at 21:55, Olivier MATZ <olivier.matz@6wind.com> wrote: > > Hi, > > On Wed, 11 Jan 2017 20:51:58 +0000, "Rowden, Aaron F" > <aaron.f.rowden@intel.com> wrote: >> Hi Helin, >> >> I'm checking on this to see why it could be failing but I don’t think >> this is one part of formal validation. Intel modules are always what >> is recommended. >> >> Aaron >> >>> Hi Helin, >>> >>>> On 11 Jan 2017, at 09:08, Zhang, Helin <helin.zhang@intel.com> >>>> wrote: >>>> >>>> Hi Aaron >>>> >>>> Is the SFP+ (Finisar FTLX8571D3BCL) supported and validated by >>>> Intel? It seems there is some PHY issue in this case. >>> >>> As the original reporter of this issue, I will test with validated >>> SFP+s and will report on my testing. >>> >>> Shouldn’t unsupported SFP+s be blacklisted in the I40E driver? >>> > > Just to let you know that in my case the SFP are Intel ones. > Maybe it's a different issue. > > I see there are some i40e fixes in the net-next repo, I'll give a try > with this version. > > Regards, > Olivier After further testing, I can confirm that this issue persists with supported Intel SFPs (Intel FTLX8571D3BCV-IT). As for the changeset introducing this issue - we had failure reports with previous DPDK versions, probably related to LSE handling, but these weren’t properly investigated. The change in 16.11 which calls get_phy_capability too early in initialization stage might have alleviated the issue making it easier for us to detect and confirm. Best regards, Christos Ricudis. ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-01-18 10:15 ` Christos Ricudis @ 2017-02-05 15:30 ` Ivan Nardi 2017-02-05 20:19 ` Ivan Nardi 0 siblings, 1 reply; 16+ messages in thread From: Ivan Nardi @ 2017-02-05 15:30 UTC (permalink / raw) To: dev Cc: Olivier MATZ, Christos Ricudis, Rowden, Aaron F, Zhang, Helin, Wu, Jingjing Hi guys any updates on this issue? We are facing a very similar problem. We have a server with 4 nics X710 4*10Gbit and the dpdk randomly failed to start with the error: PMD: eth_i40e_dev_init(): FW 4.40 API 1.4 NVM 04.05.03 eetrack 80001cd8 PMD: eth_i40e_dev_init(): Failed to sync phy type: -95 It happens randomly (sometimes it works properly, sometimes not), the "failed" port index is random too and it happens whether the fibers have been connected or not. We are using dpdk 16.11. Any help would be appreciated Thanks in advance Ivan On 18 January 2017 at 11:15, Christos Ricudis <ricudis.christos@gmail.com> wrote: > Hi, > > > On 12 Jan 2017, at 21:55, Olivier MATZ <olivier.matz@6wind.com> wrote: > > > > Hi, > > > > On Wed, 11 Jan 2017 20:51:58 +0000, "Rowden, Aaron F" > > <aaron.f.rowden@intel.com> wrote: > >> Hi Helin, > >> > >> I'm checking on this to see why it could be failing but I don’t think > >> this is one part of formal validation. Intel modules are always what > >> is recommended. > >> > >> Aaron > >> > >>> Hi Helin, > >>> > >>>> On 11 Jan 2017, at 09:08, Zhang, Helin <helin.zhang@intel.com> > >>>> wrote: > >>>> > >>>> Hi Aaron > >>>> > >>>> Is the SFP+ (Finisar FTLX8571D3BCL) supported and validated by > >>>> Intel? It seems there is some PHY issue in this case. > >>> > >>> As the original reporter of this issue, I will test with validated > >>> SFP+s and will report on my testing. > >>> > >>> Shouldn’t unsupported SFP+s be blacklisted in the I40E driver? > >>> > > > > Just to let you know that in my case the SFP are Intel ones. > > Maybe it's a different issue. > > > > I see there are some i40e fixes in the net-next repo, I'll give a try > > with this version. > > > > Regards, > > Olivier > > After further testing, I can confirm that this issue persists with > supported Intel SFPs (Intel FTLX8571D3BCV-IT). > > As for the changeset introducing this issue - we had failure reports with > previous DPDK versions, probably related to LSE handling, but these weren’t > properly investigated. The change in 16.11 which calls get_phy_capability > too early in initialization stage might have alleviated the issue making it > easier for us to detect and confirm. > > Best regards, > Christos Ricudis. > > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-02-05 15:30 ` Ivan Nardi @ 2017-02-05 20:19 ` Ivan Nardi 2017-02-06 1:04 ` Zhang, Qi Z 0 siblings, 1 reply; 16+ messages in thread From: Ivan Nardi @ 2017-02-05 20:19 UTC (permalink / raw) To: dev Cc: Olivier MATZ, Christos Ricudis, Rowden, Aaron F, Zhang, Helin, Wu, Jingjing HI same issue with 17.02-rc2 It seems to me the problem I am facing is similar to the ones reported in these mails; if not, I apologize to have used this thread Ivan On 5 February 2017 at 16:30, Ivan Nardi <nardi.ivan@gmail.com> wrote: > Hi guys > any updates on this issue? > We are facing a very similar problem. > We have a server with 4 nics X710 4*10Gbit and the dpdk randomly failed to > start with the error: > > PMD: eth_i40e_dev_init(): FW 4.40 API 1.4 NVM 04.05.03 eetrack 80001cd8 > PMD: eth_i40e_dev_init(): Failed to sync phy type: -95 > > It happens randomly (sometimes it works properly, sometimes not), the > "failed" port index is random too and it happens whether the fibers have > been connected or not. > > We are using dpdk 16.11. > > Any help would be appreciated > Thanks in advance > > Ivan > > On 18 January 2017 at 11:15, Christos Ricudis <ricudis.christos@gmail.com> > wrote: > >> Hi, >> >> > On 12 Jan 2017, at 21:55, Olivier MATZ <olivier.matz@6wind.com> wrote: >> > >> > Hi, >> > >> > On Wed, 11 Jan 2017 20:51:58 +0000, "Rowden, Aaron F" >> > <aaron.f.rowden@intel.com> wrote: >> >> Hi Helin, >> >> >> >> I'm checking on this to see why it could be failing but I don’t think >> >> this is one part of formal validation. Intel modules are always what >> >> is recommended. >> >> >> >> Aaron >> >> >> >>> Hi Helin, >> >>> >> >>>> On 11 Jan 2017, at 09:08, Zhang, Helin <helin.zhang@intel.com> >> >>>> wrote: >> >>>> >> >>>> Hi Aaron >> >>>> >> >>>> Is the SFP+ (Finisar FTLX8571D3BCL) supported and validated by >> >>>> Intel? It seems there is some PHY issue in this case. >> >>> >> >>> As the original reporter of this issue, I will test with validated >> >>> SFP+s and will report on my testing. >> >>> >> >>> Shouldn’t unsupported SFP+s be blacklisted in the I40E driver? >> >>> >> > >> > Just to let you know that in my case the SFP are Intel ones. >> > Maybe it's a different issue. >> > >> > I see there are some i40e fixes in the net-next repo, I'll give a try >> > with this version. >> > >> > Regards, >> > Olivier >> >> After further testing, I can confirm that this issue persists with >> supported Intel SFPs (Intel FTLX8571D3BCV-IT). >> >> As for the changeset introducing this issue - we had failure reports with >> previous DPDK versions, probably related to LSE handling, but these weren’t >> properly investigated. The change in 16.11 which calls get_phy_capability >> too early in initialization stage might have alleviated the issue making it >> easier for us to detect and confirm. >> >> Best regards, >> Christos Ricudis. >> >> > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-02-05 20:19 ` Ivan Nardi @ 2017-02-06 1:04 ` Zhang, Qi Z 2017-02-06 1:36 ` Zhang, Helin 2017-02-06 21:06 ` Ivan Nardi 0 siblings, 2 replies; 16+ messages in thread From: Zhang, Qi Z @ 2017-02-06 1:04 UTC (permalink / raw) To: Ivan Nardi, dev Cc: Olivier MATZ, Christos Ricudis, Rowden, Aaron F, Zhang, Helin, Wu, Jingjing Hi Ivan: I'm looking at this issue, but I can't repeat it on my environment both with X710x4 and XL710x1 Not sure if you could try below things to help narrow down this issue. 1) move i40e_dev_sync_phy_type call after i40e_set_fc call, to see if the problem still exist, since without i40e_dev_sync_phy_type, i40e_set_fc is the first place i40e_aq_get_phy_capabilities get called and we didn't see this issue before 16.11. 2) if above change works, at least we have a work around, if above still fail, please modify the parameter of i40e_aq_get_phy_capabilities in i40e_dev_sync_phy_type as below and check result. - status = i40e_aq_get_phy_capabilities(hw, false, true, &phy_ab, - NULL); + status = i40e_aq_get_phy_capabilities(hw, false, false, &phy_ab, + NULL); Thank you! Regards Qi > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ivan Nardi > Sent: Monday, February 6, 2017 4:19 AM > To: dev@dpdk.org > Cc: Olivier MATZ <olivier.matz@6wind.com>; Christos Ricudis > <ricudis.christos@gmail.com>; Rowden, Aaron F > <aaron.f.rowden@intel.com>; Zhang, Helin <helin.zhang@intel.com>; Wu, > Jingjing <jingjing.wu@intel.com> > Subject: Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ > with no link > > HI > same issue with 17.02-rc2 > It seems to me the problem I am facing is similar to the ones reported in > these mails; if not, I apologize to have used this thread > > Ivan > > On 5 February 2017 at 16:30, Ivan Nardi <nardi.ivan@gmail.com> wrote: > > > Hi guys > > any updates on this issue? > > We are facing a very similar problem. > > We have a server with 4 nics X710 4*10Gbit and the dpdk randomly > > failed to start with the error: > > > > PMD: eth_i40e_dev_init(): FW 4.40 API 1.4 NVM 04.05.03 eetrack > > 80001cd8 > > PMD: eth_i40e_dev_init(): Failed to sync phy type: -95 > > > > It happens randomly (sometimes it works properly, sometimes not), the > > "failed" port index is random too and it happens whether the fibers > > have been connected or not. > > > > We are using dpdk 16.11. > > > > Any help would be appreciated > > Thanks in advance > > > > Ivan > > > > On 18 January 2017 at 11:15, Christos Ricudis > > <ricudis.christos@gmail.com> > > wrote: > > > >> Hi, > >> > >> > On 12 Jan 2017, at 21:55, Olivier MATZ <olivier.matz@6wind.com> > wrote: > >> > > >> > Hi, > >> > > >> > On Wed, 11 Jan 2017 20:51:58 +0000, "Rowden, Aaron F" > >> > <aaron.f.rowden@intel.com> wrote: > >> >> Hi Helin, > >> >> > >> >> I'm checking on this to see why it could be failing but I don’t > >> >> think this is one part of formal validation. Intel modules are > >> >> always what is recommended. > >> >> > >> >> Aaron > >> >> > >> >>> Hi Helin, > >> >>> > >> >>>> On 11 Jan 2017, at 09:08, Zhang, Helin <helin.zhang@intel.com> > >> >>>> wrote: > >> >>>> > >> >>>> Hi Aaron > >> >>>> > >> >>>> Is the SFP+ (Finisar FTLX8571D3BCL) supported and validated by > >> >>>> Intel? It seems there is some PHY issue in this case. > >> >>> > >> >>> As the original reporter of this issue, I will test with > >> >>> validated > >> >>> SFP+s and will report on my testing. > >> >>> > >> >>> Shouldn’t unsupported SFP+s be blacklisted in the I40E driver? > >> >>> > >> > > >> > Just to let you know that in my case the SFP are Intel ones. > >> > Maybe it's a different issue. > >> > > >> > I see there are some i40e fixes in the net-next repo, I'll give a > >> > try with this version. > >> > > >> > Regards, > >> > Olivier > >> > >> After further testing, I can confirm that this issue persists with > >> supported Intel SFPs (Intel FTLX8571D3BCV-IT). > >> > >> As for the changeset introducing this issue - we had failure reports > >> with previous DPDK versions, probably related to LSE handling, but > >> these weren’t properly investigated. The change in 16.11 which calls > >> get_phy_capability too early in initialization stage might have > >> alleviated the issue making it easier for us to detect and confirm. > >> > >> Best regards, > >> Christos Ricudis. > >> > >> > > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-02-06 1:04 ` Zhang, Qi Z @ 2017-02-06 1:36 ` Zhang, Helin 2017-02-07 14:56 ` Olivier MATZ 2017-02-06 21:06 ` Ivan Nardi 1 sibling, 1 reply; 16+ messages in thread From: Zhang, Helin @ 2017-02-06 1:36 UTC (permalink / raw) To: Zhang, Qi Z, Ivan Nardi, dev Cc: Olivier MATZ, Christos Ricudis, Rowden, Aaron F, Wu, Jingjing Hi guys There may have firmware dependencies, new DPDK version + old firmware version may not work. Please refer to below link to see what type of firmware has been validated with 16.11 DPDK. http://dpdk.org/doc/guides/rel_notes/release_16_11.html#tested-nics Then first, you need to upgrade your firmware. Then let us know if the issue is still there. Thanks! Regards, Helin -----Original Message----- From: Zhang, Qi Z Sent: Monday, February 6, 2017 9:05 AM To: Ivan Nardi <nardi.ivan@gmail.com>; dev@dpdk.org Cc: Olivier MATZ <olivier.matz@6wind.com>; Christos Ricudis <ricudis.christos@gmail.com>; Rowden, Aaron F <aaron.f.rowden@intel.com>; Zhang, Helin <helin.zhang@intel.com>; Wu, Jingjing <jingjing.wu@intel.com> Subject: RE: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link Hi Ivan: I'm looking at this issue, but I can't repeat it on my environment both with X710x4 and XL710x1 Not sure if you could try below things to help narrow down this issue. 1) move i40e_dev_sync_phy_type call after i40e_set_fc call, to see if the problem still exist, since without i40e_dev_sync_phy_type, i40e_set_fc is the first place i40e_aq_get_phy_capabilities get called and we didn't see this issue before 16.11. 2) if above change works, at least we have a work around, if above still fail, please modify the parameter of i40e_aq_get_phy_capabilities in i40e_dev_sync_phy_type as below and check result. - status = i40e_aq_get_phy_capabilities(hw, false, true, &phy_ab, - NULL); + status = i40e_aq_get_phy_capabilities(hw, false, false, &phy_ab, + NULL); Thank you! Regards Qi > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ivan Nardi > Sent: Monday, February 6, 2017 4:19 AM > To: dev@dpdk.org > Cc: Olivier MATZ <olivier.matz@6wind.com>; Christos Ricudis > <ricudis.christos@gmail.com>; Rowden, Aaron F > <aaron.f.rowden@intel.com>; Zhang, Helin <helin.zhang@intel.com>; Wu, > Jingjing <jingjing.wu@intel.com> > Subject: Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when > using SFP+ with no link > > HI > same issue with 17.02-rc2 > It seems to me the problem I am facing is similar to the ones reported > in these mails; if not, I apologize to have used this thread > > Ivan > > On 5 February 2017 at 16:30, Ivan Nardi <nardi.ivan@gmail.com> wrote: > > > Hi guys > > any updates on this issue? > > We are facing a very similar problem. > > We have a server with 4 nics X710 4*10Gbit and the dpdk randomly > > failed to start with the error: > > > > PMD: eth_i40e_dev_init(): FW 4.40 API 1.4 NVM 04.05.03 eetrack > > 80001cd8 > > PMD: eth_i40e_dev_init(): Failed to sync phy type: -95 > > > > It happens randomly (sometimes it works properly, sometimes not), > > the "failed" port index is random too and it happens whether the > > fibers have been connected or not. > > > > We are using dpdk 16.11. > > > > Any help would be appreciated > > Thanks in advance > > > > Ivan > > > > On 18 January 2017 at 11:15, Christos Ricudis > > <ricudis.christos@gmail.com> > > wrote: > > > >> Hi, > >> > >> > On 12 Jan 2017, at 21:55, Olivier MATZ <olivier.matz@6wind.com> > wrote: > >> > > >> > Hi, > >> > > >> > On Wed, 11 Jan 2017 20:51:58 +0000, "Rowden, Aaron F" > >> > <aaron.f.rowden@intel.com> wrote: > >> >> Hi Helin, > >> >> > >> >> I'm checking on this to see why it could be failing but I don’t > >> >> think this is one part of formal validation. Intel modules are > >> >> always what is recommended. > >> >> > >> >> Aaron > >> >> > >> >>> Hi Helin, > >> >>> > >> >>>> On 11 Jan 2017, at 09:08, Zhang, Helin <helin.zhang@intel.com> > >> >>>> wrote: > >> >>>> > >> >>>> Hi Aaron > >> >>>> > >> >>>> Is the SFP+ (Finisar FTLX8571D3BCL) supported and validated by > >> >>>> Intel? It seems there is some PHY issue in this case. > >> >>> > >> >>> As the original reporter of this issue, I will test with > >> >>> validated > >> >>> SFP+s and will report on my testing. > >> >>> > >> >>> Shouldn’t unsupported SFP+s be blacklisted in the I40E driver? > >> >>> > >> > > >> > Just to let you know that in my case the SFP are Intel ones. > >> > Maybe it's a different issue. > >> > > >> > I see there are some i40e fixes in the net-next repo, I'll give a > >> > try with this version. > >> > > >> > Regards, > >> > Olivier > >> > >> After further testing, I can confirm that this issue persists with > >> supported Intel SFPs (Intel FTLX8571D3BCV-IT). > >> > >> As for the changeset introducing this issue - we had failure > >> reports with previous DPDK versions, probably related to LSE > >> handling, but these weren’t properly investigated. The change in > >> 16.11 which calls get_phy_capability too early in initialization > >> stage might have alleviated the issue making it easier for us to detect and confirm. > >> > >> Best regards, > >> Christos Ricudis. > >> > >> > > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-02-06 1:36 ` Zhang, Helin @ 2017-02-07 14:56 ` Olivier MATZ 2017-02-07 14:58 ` Zhang, Helin 2017-02-12 16:13 ` Ivan Nardi 0 siblings, 2 replies; 16+ messages in thread From: Olivier MATZ @ 2017-02-07 14:56 UTC (permalink / raw) To: Zhang, Helin Cc: Zhang, Qi Z, Ivan Nardi, dev, Olivier MATZ, Christos Ricudis, Rowden, Aaron F, Wu, Jingjing Hi Helin, On Mon, 6 Feb 2017 01:36:45 +0000, "Zhang, Helin" <helin.zhang@intel.com> wrote: > Hi guys > > There may have firmware dependencies, new DPDK version + old firmware > version may not work. Please refer to below link to see what type of > firmware has been validated with 16.11 DPDK. > http://dpdk.org/doc/guides/rel_notes/release_16_11.html#tested-nics > Then first, you need to upgrade your firmware. Then let us know if > the issue is still there. Thanks! > I updated the firmware, and I confirm it solves my issue described at: http://dpdk.org/ml/archives/dev/2017-January/054401.html Thanks, Olivier ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-02-07 14:56 ` Olivier MATZ @ 2017-02-07 14:58 ` Zhang, Helin 2017-02-12 16:13 ` Ivan Nardi 1 sibling, 0 replies; 16+ messages in thread From: Zhang, Helin @ 2017-02-07 14:58 UTC (permalink / raw) To: Olivier MATZ Cc: Zhang, Qi Z, Ivan Nardi, dev, Christos Ricudis, Rowden, Aaron F, Wu, Jingjing Hi Oliver Very good to know that good news! Thank you! Regards, Helin -----Original Message----- From: Olivier MATZ [mailto:olivier.matz@6wind.com] Sent: Tuesday, February 7, 2017 10:56 PM To: Zhang, Helin <helin.zhang@intel.com> Cc: Zhang, Qi Z <qi.z.zhang@intel.com>; Ivan Nardi <nardi.ivan@gmail.com>; dev@dpdk.org; Olivier MATZ <olivier.matz@6wind.com>; Christos Ricudis <ricudis.christos@gmail.com>; Rowden, Aaron F <aaron.f.rowden@intel.com>; Wu, Jingjing <jingjing.wu@intel.com> Subject: Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link Hi Helin, On Mon, 6 Feb 2017 01:36:45 +0000, "Zhang, Helin" <helin.zhang@intel.com> wrote: > Hi guys > > There may have firmware dependencies, new DPDK version + old firmware > version may not work. Please refer to below link to see what type of > firmware has been validated with 16.11 DPDK. > http://dpdk.org/doc/guides/rel_notes/release_16_11.html#tested-nics > Then first, you need to upgrade your firmware. Then let us know if the > issue is still there. Thanks! > I updated the firmware, and I confirm it solves my issue described at: http://dpdk.org/ml/archives/dev/2017-January/054401.html Thanks, Olivier ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-02-07 14:56 ` Olivier MATZ 2017-02-07 14:58 ` Zhang, Helin @ 2017-02-12 16:13 ` Ivan Nardi 1 sibling, 0 replies; 16+ messages in thread From: Ivan Nardi @ 2017-02-12 16:13 UTC (permalink / raw) To: Olivier MATZ Cc: Zhang, Helin, Zhang, Qi Z, dev, Christos Ricudis, Rowden, Aaron F, Wu, Jingjing Hi I am not able to reproduce the issue anymore with nvm 5.05 + 17.02-rc2 (with LSC disabled) Lesson learned: always upgrade to latest versions first. Thanks to everyone for the support Regards Ivan On 7 February 2017 at 15:56, Olivier MATZ <olivier.matz@6wind.com> wrote: > Hi Helin, > > On Mon, 6 Feb 2017 01:36:45 +0000, "Zhang, Helin" > <helin.zhang@intel.com> wrote: >> Hi guys >> >> There may have firmware dependencies, new DPDK version + old firmware >> version may not work. Please refer to below link to see what type of >> firmware has been validated with 16.11 DPDK. >> http://dpdk.org/doc/guides/rel_notes/release_16_11.html#tested-nics >> Then first, you need to upgrade your firmware. Then let us know if >> the issue is still there. Thanks! >> > > I updated the firmware, and I confirm it solves my issue described > at: http://dpdk.org/ml/archives/dev/2017-January/054401.html > > Thanks, > Olivier ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link 2017-02-06 1:04 ` Zhang, Qi Z 2017-02-06 1:36 ` Zhang, Helin @ 2017-02-06 21:06 ` Ivan Nardi 1 sibling, 0 replies; 16+ messages in thread From: Ivan Nardi @ 2017-02-06 21:06 UTC (permalink / raw) To: Zhang, Qi Z Cc: dev, Olivier MATZ, Christos Ricudis, Rowden, Aaron F, Zhang, Helin, Wu, Jingjing Thanks for the suggestions! I'll try them and I will report back the results in the next days. Regards Ivan On 6 February 2017 at 02:04, Zhang, Qi Z <qi.z.zhang@intel.com> wrote: > Hi Ivan: > > I'm looking at this issue, but I can't repeat it on my environment both with X710x4 and XL710x1 > Not sure if you could try below things to help narrow down this issue. > > 1) move i40e_dev_sync_phy_type call after i40e_set_fc call, to see if the problem still exist, since without i40e_dev_sync_phy_type, i40e_set_fc is the first place i40e_aq_get_phy_capabilities get called and we didn't see this issue before 16.11. > > 2) if above change works, at least we have a work around, if above still fail, please modify the parameter of i40e_aq_get_phy_capabilities in i40e_dev_sync_phy_type as below and check result. > - status = i40e_aq_get_phy_capabilities(hw, false, true, &phy_ab, > - NULL); > + status = i40e_aq_get_phy_capabilities(hw, false, false, &phy_ab, > + NULL); > > Thank you! > > Regards > Qi > >> -----Original Message----- >> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Ivan Nardi >> Sent: Monday, February 6, 2017 4:19 AM >> To: dev@dpdk.org >> Cc: Olivier MATZ <olivier.matz@6wind.com>; Christos Ricudis >> <ricudis.christos@gmail.com>; Rowden, Aaron F >> <aaron.f.rowden@intel.com>; Zhang, Helin <helin.zhang@intel.com>; Wu, >> Jingjing <jingjing.wu@intel.com> >> Subject: Re: [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ >> with no link >> >> HI >> same issue with 17.02-rc2 >> It seems to me the problem I am facing is similar to the ones reported in >> these mails; if not, I apologize to have used this thread >> >> Ivan >> >> On 5 February 2017 at 16:30, Ivan Nardi <nardi.ivan@gmail.com> wrote: >> >> > Hi guys >> > any updates on this issue? > >> > We are facing a very similar problem. >> > We have a server with 4 nics X710 4*10Gbit and the dpdk randomly >> > failed to start with the error: >> > >> > PMD: eth_i40e_dev_init(): FW 4.40 API 1.4 NVM 04.05.03 eetrack >> > 80001cd8 >> > PMD: eth_i40e_dev_init(): Failed to sync phy type: -95 >> > >> > It happens randomly (sometimes it works properly, sometimes not), the >> > "failed" port index is random too and it happens whether the fibers >> > have been connected or not. >> > >> > We are using dpdk 16.11. >> > >> > Any help would be appreciated >> > Thanks in advance >> > >> > Ivan >> > >> > On 18 January 2017 at 11:15, Christos Ricudis >> > <ricudis.christos@gmail.com> >> > wrote: >> > >> >> Hi, >> >> >> >> > On 12 Jan 2017, at 21:55, Olivier MATZ <olivier.matz@6wind.com> >> wrote: >> >> > >> >> > Hi, >> >> > >> >> > On Wed, 11 Jan 2017 20:51:58 +0000, "Rowden, Aaron F" >> >> > <aaron.f.rowden@intel.com> wrote: >> >> >> Hi Helin, >> >> >> >> >> >> I'm checking on this to see why it could be failing but I don’t >> >> >> think this is one part of formal validation. Intel modules are >> >> >> always what is recommended. >> >> >> >> >> >> Aaron >> >> >> >> >> >>> Hi Helin, >> >> >>> >> >> >>>> On 11 Jan 2017, at 09:08, Zhang, Helin <helin.zhang@intel.com> >> >> >>>> wrote: >> >> >>>> >> >> >>>> Hi Aaron >> >> >>>> >> >> >>>> Is the SFP+ (Finisar FTLX8571D3BCL) supported and validated by >> >> >>>> Intel? It seems there is some PHY issue in this case. >> >> >>> >> >> >>> As the original reporter of this issue, I will test with >> >> >>> validated >> >> >>> SFP+s and will report on my testing. >> >> >>> >> >> >>> Shouldn’t unsupported SFP+s be blacklisted in the I40E driver? >> >> >>> >> >> > >> >> > Just to let you know that in my case the SFP are Intel ones. >> >> > Maybe it's a different issue. >> >> > >> >> > I see there are some i40e fixes in the net-next repo, I'll give a >> >> > try with this version. >> >> > >> >> > Regards, >> >> > Olivier >> >> >> >> After further testing, I can confirm that this issue persists with >> >> supported Intel SFPs (Intel FTLX8571D3BCV-IT). >> >> >> >> As for the changeset introducing this issue - we had failure reports >> >> with previous DPDK versions, probably related to LSE handling, but >> >> these weren’t properly investigated. The change in 16.11 which calls >> >> get_phy_capability too early in initialization stage might have >> >> alleviated the issue making it easier for us to detect and confirm. >> >> >> >> Best regards, >> >> Christos Ricudis. >> >> >> >> >> > ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2017-02-12 16:13 UTC | newest] Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-01-10 12:32 [dpdk-dev] i40e_aq_get_phy_capabilities() fails when using SFP+ with no link Christos Ricudis 2017-01-10 15:28 ` Olivier MATZ 2017-01-11 1:08 ` Zhang, Helin 2017-01-11 3:00 ` Christos Ricudis [not found] ` <FC7A99FCE8F15942AB8D6F91FF66D8F694836A60@ORSMSX112.amr.corp.intel.com> 2017-01-12 13:55 ` Olivier MATZ 2017-01-13 13:24 ` Olivier Matz 2017-01-17 12:50 ` Wu, Jingjing 2017-01-18 10:15 ` Christos Ricudis 2017-02-05 15:30 ` Ivan Nardi 2017-02-05 20:19 ` Ivan Nardi 2017-02-06 1:04 ` Zhang, Qi Z 2017-02-06 1:36 ` Zhang, Helin 2017-02-07 14:56 ` Olivier MATZ 2017-02-07 14:58 ` Zhang, Helin 2017-02-12 16:13 ` Ivan Nardi 2017-02-06 21:06 ` Ivan Nardi
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).