mbox series

[net-next,0/7] GTP

Message ID 20210202065159.227049-1-jonas@norrbonn.se
Headers show
Series GTP | expand

Message

Jonas Bonn Feb. 2, 2021, 6:51 a.m. UTC
There's ongoing work in this driver to provide support for IPv6, GRO,
GSO, and "collect metadata" mode operation.  In order to facilitate this
work going forward, this short series accumulates already ACK:ed patches
that are ready for the next merge window.

All of these patches should be uncontroversial at this point, including
the first one in the series that reverts a recently added change to
introduce "collect metadata" mode.  As that patch produces 'broken'
packets when common GTP headers are in place, it seems better to revert
it and rethink things a bit before inclusion.

/Jonas

Jonas Bonn (7):
  Revert "GTP: add support for flow based tunneling API"
  gtp: set initial MTU
  gtp: include role in link info
  gtp: really check namespaces before xmit
  gtp: drop unnecessary call to skb_dst_drop
  gtp: set device type
  gtp: update rx_length_errors for abnormally short packets

 drivers/net/gtp.c                  | 540 +++++++++--------------------
 include/uapi/linux/gtp.h           |  12 -
 include/uapi/linux/if_link.h       |   1 -
 include/uapi/linux/if_tunnel.h     |   1 -
 tools/include/uapi/linux/if_link.h |   1 -
 5 files changed, 158 insertions(+), 397 deletions(-)

Comments

Jonas Bonn Feb. 3, 2021, 7:07 a.m. UTC | #1
There's ongoing work in this driver to provide support for IPv6, GRO,
GSO, and "collect metadata" mode operation.  In order to facilitate this
work going forward, this short series accumulates already ACK:ed patches
that are ready for the next merge window.

All of these patches should be uncontroversial at this point, including
the first one in the series that reverts a recently added change to
introduce "collect metadata" mode.  As that patch produces 'broken'
packets when common GTP headers are in place, it seems better to revert
it and rethink things a bit before inclusion.

Changes in v2:
- fix fallout from "one rebase too many"

/Jonas


Jonas Bonn (7):
  Revert "GTP: add support for flow based tunneling API"
  gtp: set initial MTU
  gtp: include role in link info
  gtp: really check namespaces before xmit
  gtp: drop unnecessary call to skb_dst_drop
  gtp: set device type
  gtp: update rx_length_errors for abnormally short packets

 drivers/net/gtp.c                  | 544 +++++++++--------------------
 include/uapi/linux/gtp.h           |  12 -
 include/uapi/linux/if_link.h       |   1 -
 include/uapi/linux/if_tunnel.h     |   1 -
 tools/include/uapi/linux/if_link.h |   1 -
 5 files changed, 160 insertions(+), 399 deletions(-)
Pravin Shelar Feb. 4, 2021, 6:09 a.m. UTC | #2
On Mon, Feb 1, 2021 at 10:53 PM Jonas Bonn <jonas@norrbonn.se> wrote:
>

> There's ongoing work in this driver to provide support for IPv6, GRO,

> GSO, and "collect metadata" mode operation.  In order to facilitate this

> work going forward, this short series accumulates already ACK:ed patches

> that are ready for the next merge window.

>

> All of these patches should be uncontroversial at this point, including

> the first one in the series that reverts a recently added change to

> introduce "collect metadata" mode.  As that patch produces 'broken'

> packets when common GTP headers are in place, it seems better to revert

> it and rethink things a bit before inclusion.

>

Acked-by: Pravin B Shelar <pshelar@ovn.org>