[dpdk-dev] [PATCH v8 00/15] common linkstatus functions

Ferruh Yigit ferruh.yigit at intel.com
Fri Mar 16 11:31:04 CET 2018


On 3/15/2018 5:38 PM, Ferruh Yigit wrote:
> On 1/26/2018 2:01 AM, Stephen Hemminger wrote:
>> While writing hyper-v driver, noticed a lot of unnecessary
>> duplication of code in drivers for handling the eth_dev link status
>> information. While consolidating this, it also became obvious that
>> some drivers behave differently for no good reason.
>>
>> It also was a good chance to introduce atomic exchange primitives
>> in EAL because there are other places using cmpset where not
>> necessary (such as bonding).
>>
>> Mostly only compile tested only, don't have all of the hardware
>> available (except ixgbe and virtio) to test.
>>
>> v8
>>   - fix typo in 32bit exchange
>>   - fix build of dpaa2
>>   - go back to same return value as original code
>>   - reduce number of lines changed
>>
>> v7
>>   - add exchange functions for PPC64
>>   - move linkstatus helpers to rte_ethdev_driver
>>   - rebase to 18.02
>>
>> Stephen Hemminger (15):
>>   eal: introduce atomic exchange operation
>>   ethdev: add linkstatus get/set helper functions
>>   net/virtio: use eth_linkstatus_set
>>   net/vmxnet3: use rte_eth_linkstatus_set
>>   net/dpaa2: use rte_eth_linkstatus_set
>>   net/nfp: use rte_eth_linkstatus functions
>>   net/e1000: use rte_eth_linkstatus helpers
>>   net/ixgbe: use rte_eth_linkstatus functions
>>   net/sfc: use new rte_eth_linkstatus functions
>>   net/i40e: use rte_eth_linkstatus functions
>>   net/liquidio: use rte_eth_linkstatus_set
>>   net/thunderx: use rte_eth_linkstatus_set
>>   net/szedata: use _rte_eth_linkstatus_set
>>   net/octeontx: use rte_eth_linkstatus_set
>>   net/enic: use rte_eth_linkstatus_set
> 
> For series
> Reviewed-by: Ferruh Yigit <ferruh.yigit at intel.com>

Series applied to dpdk-next-net/master, thanks.

(Need to resolve some conflicts, specially in i40e and vmxnet3, please verify
final patches.)


More information about the dev mailing list