On 03/09/2020 09:54, Tom Parkin wrote:
> This series of patches makes the following cleanups and improvements to
> the l2tp code:
>
> * various API tweaks to remove unused parameters from function calls
> * lightly refactor the l2tp transmission path to capture more error
> conditions in the data plane statistics
> * repurpose the "magic feather" validation in l2tp to check for
> sk_user_data (ab)use as opposed to refcount debugging
> * remove some duplicated code
>
> Tom Parkin (6):
> l2tp: remove header length param from l2tp_xmit_skb
> l2tp: drop data_len argument from l2tp_xmit_core
> l2tp: drop net argument from l2tp_tunnel_create
> l2tp: capture more tx errors in data plane stats
> l2tp: make magic feather checks more useful
> l2tp: avoid duplicated code in l2tp_tunnel_closeall
>
> net/l2tp/l2tp_core.c | 134 +++++++++++++++++++---------------------
> net/l2tp/l2tp_core.h | 10 ++-
> net/l2tp/l2tp_eth.c | 2 +-
> net/l2tp/l2tp_ip.c | 2 +-
> net/l2tp/l2tp_ip6.c | 2 +-
> net/l2tp/l2tp_netlink.c | 2 +-
> net/l2tp/l2tp_ppp.c | 15 ++++-
> 7 files changed, 87 insertions(+), 80 deletions(-)
>
Reviewed-by: James Chapman <jchap...@katalix.com>