Hi Sven,

On Mon, Nov 21, 2016 at 4:16 PM, Sven Eckelmann [off-list ref] wrote:

On Montag, 21. November 2016 08:39:39 CET fgao@ikuai8.com wrote:

quoted

From: Gao Feng <redacted>

The tc could return NET_XMIT_CN as one congestion notification, but
it does not mean the packe is lost. Other modules like ipvlan,

s/packe/packet/

What's this mean?

quoted

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               | 2 +-
 net/batman-adv/tp_meter.c              | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

[...]

Not sure how the batman-adv maintainers see this - but this patch needs
an additional patch for net-next to also add it to the parts which were
rewritten.

Kind regards,
        Sven

Ok. I would commit another patch to net-next.

Best Regards
Feng