Re: [PATCH net-next 1/2] net: batman-adv: Treat NET_XMIT_CN as transmit successfully
From: Feng Gao <hidden>
Date: 2016-12-17 02:17:24
Also in:
batman
From: Feng Gao <hidden>
Date: 2016-12-17 02:17:24
Also in:
batman
On Fri, Dec 16, 2016 at 5:19 PM, Sven Eckelmann [off-list ref] wrote:
On Montag, 21. November 2016 23:00:32 CET fgao-KlmEoCYek3zQT0dZR+AlfA@public.gmane.org wrote:quoted
From: Gao Feng <redacted> The tc could return NET_XMIT_CN as one congestion notification, but it does not mean the packet is lost. Other modules like ipvlan, macvlan, and others treat NET_XMIT_CN as success too. So batman-adv should add the NET_XMIT_CN check. Signed-off-by: Gao Feng <redacted> --- net/batman-adv/distributed-arp-table.c | 2 +- net/batman-adv/fragmentation.c | 2 +- net/batman-adv/routing.c | 10 +++++----- net/batman-adv/soft-interface.c | 2 +- net/batman-adv/tp_meter.c | 2 +- 5 files changed, 9 insertions(+), 9 deletions(-)David marked your patches as "derefered" after "under review" and did not apply them directly. Also Florian Westphal didn't continue the discussion about the direction you should choose. The patches were therefore queued up in the in batman-adv 671630d6aad0..eab7617142d2. They will be submitted later(tm) in a pull request to David.
I get it. Thanks Sven. Regards Feng
Thanks,
Sven