[dpdk-dev] [PATCH] net/ixgbe: fix busy polling while fiber link update
Laurent Hardy
laurent.hardy at 6wind.com
Mon Oct 15 12:43:23 CEST 2018
Hi Wei,
On 10/12/2018 09:36 AM, Zhao1, Wei wrote:
>>>> Hi, laurent.hardy
>>>> You are the author for the patch (* net/ixgbe: ensure link status is
>> updated), why do you implement code that way?
>>>> Is that must that set up link with wait?
>>>>
>>>> ixgbe_setup_link(hw, speed, true);
>>>>
>> The main issue which has lead to this patch has been reported through a test
>> case with the autoneg enabled (which has been also reported in the pmd
>> test provided along with the patch:
>> http://patches.dpdk.org/comment/46253/).
>> In this context, without the flag set the patch wasn't effective.
> My question is whether we can change to:
>
> ixgbe_setup_link(hw, speed, false);
>
> in your patch, it is " ixgbe_setup_link(hw, speed, true);"
> some user may need flexible in wait for complete.
>
>
Yes, the parameter (autoneg_wait_to_complete) could be change to false.
I redo the test following test plan provided through
http://patches.dpdk.org/comment/46253/, with flag set to false
and speed defined to 1Gb on switch side.
In this case both ports goes properly up (nic used is 82599ES
10-Gigabit) with a correct link speed.
More information about the dev
mailing list