]> git.baikalelectronics.ru Git - kernel.git/commit
geneve: change from tx_error to tx_dropped on missing metadata
authorJiri Benc <jbenc@redhat.com>
Wed, 3 Jun 2020 09:12:14 +0000 (11:12 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 4 Jun 2020 22:45:15 +0000 (15:45 -0700)
commit24693e3d25e67090db13136d3c73b89e8b640b46
tree64165837c35e7994b555542bf75dd4f16ce9bdd6
parent62a91688b98c9f618770efdd303055cde9a112d7
geneve: change from tx_error to tx_dropped on missing metadata

If the geneve interface is in collect_md (external) mode, it can't send any
packets submitted directly to its net interface, as such packets won't have
metadata attached. This is expected.

However, the kernel itself sends some packets to the interface, most
notably, IPv6 DAD, IPv6 multicast listener reports, etc. This is not wrong,
as tunnel metadata can be specified in routing table (although technically,
that has never worked for IPv6, but hopefully will be fixed eventually) and
then the interface must correctly participate in IPv6 housekeeping.

The problem is that any such attempt increases the tx_error counter. Just
bringing up a geneve interface with IPv6 enabled is enough to see a number
of tx_errors. That causes confusion among users, prompting them to find
a network error where there is none.

Change the counter used to tx_dropped. That better conveys the meaning
(there's nothing wrong going on, just some packets are getting dropped) and
hopefully will make admins panic less.

Signed-off-by: Jiri Benc <jbenc@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/geneve.c