]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "GTP: add support for flow based tunneling API"
authorJonas Bonn <jonas@norrbonn.se>
Wed, 3 Feb 2021 07:07:59 +0000 (08:07 +0100)
committerJakub Kicinski <kuba@kernel.org>
Thu, 4 Feb 2021 17:29:57 +0000 (09:29 -0800)
commitcab8ec6cb4df7b40fe21ec92752f14fcef3595d3
tree88a1fa10535f12d1995871e68547c08cdf994de4
parentee439014ee63343defd80374dd162ed9cd0c4272
Revert "GTP: add support for flow based tunneling API"

This reverts commit bf85ef7deb780588b0572fe42c87606f6cf657d1.

This patch was committed without maintainer approval and despite a number
of unaddressed concerns from review.  There are several issues that
impede the acceptance of this patch and that make a reversion of this
particular instance of these changes the best way forward:

i)  the patch contains several logically separate changes that would be
better served as smaller patches (for review purposes)
ii) functionality like the handling of end markers has been introduced
without further explanation
iii) symmetry between the handling of GTPv0 and GTPv1 has been
unnecessarily broken
iv) the patchset produces 'broken' packets when extension headers are
included
v) there are no available userspace tools to allow for testing this
functionality
vi) there is an unaddressed Coverity report against the patch concering
memory leakage
vii) most importantly, the patch contains a large amount of superfluous
churn that impedes other ongoing work with this driver

This patch will be reworked into a series that aligns with other
ongoing work and facilitates review.

Signed-off-by: Jonas Bonn <jonas@norrbonn.se>
Acked-by: Harald Welte <laforge@gnumonks.org>
Acked-by: Pravin B Shelar <pshelar@ovn.org>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/gtp.c
include/uapi/linux/gtp.h
include/uapi/linux/if_link.h
include/uapi/linux/if_tunnel.h
tools/include/uapi/linux/if_link.h