Re: [PATCH net-next 0/4] bonding: final set of netlink patches

[Ding Tianhong <dthxman@gmail.com>]

于 2013/12/28 15:15, Scott Feldman 写道:
> The following series implements the last set of bonding netlink attributes
> for 802.3ad mode:
> 
> 	lacp_rate
> 	ad_select
> 	ad_info, nest of:
> 		ad_aggregator
> 		ad_num_ports
> 		ad_actor_key
> 		ad_partner_key
> 		ad_partner_mac
> 
> The last patch adds an additional netlink attribute, active_slaves, which
> is a nested list of ifindices for current active slaves.  We're using this
> list to enable/disable hashing of ports in a hardware LAG implementation.
> In the same way bonding driver includes/excludes ports for 802.3ad egress
> hashing, hardware ports are included/excluded from egress hashing by
> hardware based on port active status.  Yes, data path offloaded to
> hardware, control path remains in kernel via bonding driver.
> 
> ---
> 
> Scott Feldman (4):
>       bonding: add lacp_rate attribute netlink support
>       bonding: add ad_select attribute netlink support
>       bonding: add ad_info attribute netlink support
>       bonding: add active_slaves attribute
> 
> 
>  drivers/net/bonding/bond_netlink.c |  100 +++++++++++++++++++++++++++++++++
>  drivers/net/bonding/bond_options.c |   51 +++++++++++++++++
>  drivers/net/bonding/bond_sysfs.c   |  108 +++++++++++++++++++++---------------
>  drivers/net/bonding/bonding.h      |    2 +
>  include/uapi/linux/if_link.h       |   15 +++++
>  5 files changed, 229 insertions(+), 47 deletions(-)
> 

Reviewed-by: Ding Tianhong <dingtianhong@huawei.com>