On Sat, Mar 26, 2016 at 09:11:38AM +0100, Thomas Monjalon wrote: > > v13: > > - Fix startup regression; revert flip of ETH_LINK_SPEED_FIXED and > > ETH_LINK_SPEED_AUTONEG values. ETH_LINK_SPEED_AUTONEG is now 0. > > As commented earlier, I would prefer avoiding this revert. > Comments are welcome.
I've tested v13 using testpmd and ixgbe. Performed the following checks: - Port start/stop - OK, link speed is displayed correctly. - Forcing 1G / duplex auto - OK. - Forcing 10G / duplex auto - OK. - Forcing 10G / duplex half - half is not allowed (expected). - Forcing 40G / duplex auto - unsupported, causes port start to fail (expected). - Forcing 1G / duplex full - OK. - Forcing 100 / duplex half - unsupported, causes port start to fail (expected). - Forcing 100 / duplex full - OK. - Traffic flows whenever link speed is supported/negotiated. Tested-by: Adrien Mazarguil <adrien.mazarguil at 6wind.com> -- Adrien Mazarguil 6WIND