Re: [dpdk-dev] [PATCH] bonding: fix lacp negotiation failed

2019-03-19 Thread Kevin Traynor
On 15/03/2019 09:06, Liang Zhang wrote: > When monitor(port-mirroring) traffic from other lacp port-channel, > rx_machine_update may recieving other lacp negotiation packets. > Thus bond mode 4 will negotiation failed. > Please add a 'Fixes:' tag, and if appropriate for backport to stable branche

Re: [dpdk-dev] [PATCH] bonding: fix lacp negotiation failed

2019-03-18 Thread Chas Williams
On 3/15/19 5:06 AM, Liang Zhang wrote: When monitor(port-mirroring) traffic from other lacp port-channel, rx_machine_update may recieving other lacp negotiation packets. Thus bond mode 4 will negotiation failed. Signed-off-by: Liang Zhang Acked-by: Chas Williams --- drivers/net/bonding/r

[dpdk-dev] [PATCH] bonding: fix lacp negotiation failed

2019-03-15 Thread Liang Zhang
When monitor(port-mirroring) traffic from other lacp port-channel, rx_machine_update may recieving other lacp negotiation packets. Thus bond mode 4 will negotiation failed. Signed-off-by: Liang Zhang --- drivers/net/bonding/rte_eth_bond_8023ad.c | 12 ++-- drivers/net/bonding/rte

Re: [dpdk-dev] [PATCH] bonding: fix lacp negotiation failed

2019-03-13 Thread Chas Williams
At a minimum you need to address the long lines in your patch. See http://mails.dpdk.org/archives/test-report/2019-March/076663.html otherwise this looks fine to me. On 3/13/19 9:23 AM, Liang Zhang wrote: When monitor(port-mirroring) traffic from other lacp port-channel, rx_machine_update may

[dpdk-dev] [PATCH] bonding: fix lacp negotiation failed

2019-03-13 Thread Liang Zhang
When monitor(port-mirroring) traffic from other lacp port-channel, rx_machine_update may recieving other lacp negotiation packets. Thus bond mode 4 will negotiation failed. Signed-off-by: Liang Zhang --- drivers/net/bonding/rte_eth_bond_8023ad.c | 9 +++-- drivers/net/bonding/rte_eth