Re: [PATCH v4 net-next 0/5] bonding:use latest lacp_rate andad_select and delete unused ad_timer and arp_mon_pt

[David Miller]

From: Jay Vosburgh <fubar@xxxxxxxxxx>
Date: Thu, 09 Jun 2011 14:58:11 -0700

> Weiping Pan <panweiping3@xxxxxxxxx> wrote:
> 
>>There is bug that when you modify lacp_rate via sysfs,
>>802.3ad won't use the new value of lacp_rate to transmit packets.
>>This is because port->actor_oper_port_state isn't changed.
>>
>>As for ad_select, it can work,
>>but both struct bond_params and ad_bond_info have lacp_fast and ad_select,
>>they are duplicate and need extra synchronization.
>>802.3ad can get them from bond_params directly every time.
>>
>>And ad_timer and arp_mon_pt aren't used any more, just delete them.
>>
>>changelog:
>>v2:
>>add bond_3ad_update_lacp_rate() as a helper function,
>>and hold bond->lock when iterates slave list.
>>
>>v3:
>>delete duplicate lacp_fast and agg_select_mode from struct ad_bond_info.
>>
>>v4:
>>delete unused ad_timer and arp_mon_pt.
> 
> 	All patches in the series look reasonable.
> 
> Signed-off-by: Jay Vosburgh <fubar@xxxxxxxxxx>

All applied, thanks everyone.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/