On Thu, 18 Jan 2018 16:04:27 +0200 Serhey Popovych <serhe.popov...@gmail.com> wrote:
> Continue improvements to tunnel modules. Final goal > is to make merge IP and IPv6 variants and make that > transition as transparent as possible. > > Everything within this series is open for your comments, > suggestions and criticism. > > See individual patch description message for details. > > v2 > 1) Fix checkpatch issues: no assignment in condtional > anymore. > > 2) Better code abstraction: introduce tnl_print_encap() > helper and get rid of duplicated code for printing > tunnel encapsulation options. > > Patch with subject > "ip/tunnel: Abstract tunnel encapsulation options printing" > replaces two previous in v1 series > "ip/tunnel: Use print_string() and simplify encap option printing" > and > "ip/tunnel: Minor cleanups in print routines" > > 3) Include patch with subject > "tunnel: Return constant string without copying it" > in the series: it is related to tunneling code too. > > Thanks, > Serhii > > Serhey Popovych (9): > iplink: Use ll_index_to_name() instead of if_indextoname() > ip/tunnel: Correct and unify ttl/hoplimit printing > ip/tunnel: Simplify and unify tos printing > ip/tunnel: Use print_0xhex() instead of print_string() > ip/tunnel: Abstract tunnel encapsulation options printing > gre/tunnel: Print erspan_index using print_uint() > vti/tunnel: Unify ikey/okey printing > vti6/tunnel: Unify and simplify link type help functions > tunnel: Return constant string without copying it > > bridge/fdb.c | 5 +- > bridge/link.c | 19 +++---- > ip/ip6tunnel.c | 5 +- > ip/iplink_bond.c | 38 ++++++-------- > ip/iplink_geneve.c | 38 ++++++-------- > ip/iplink_vxlan.c | 49 ++++++++---------- > ip/iproute_lwtunnel.c | 11 ++--- > ip/iptunnel.c | 2 +- > ip/link_gre.c | 132 > +++++++++++-------------------------------------- > ip/link_gre6.c | 103 +++++++++----------------------------- > ip/link_ip6tnl.c | 105 ++++++++++----------------------------- > ip/link_iptnl.c | 128 +++++++++++------------------------------------ > ip/link_vti.c | 42 ++++++++-------- > ip/link_vti6.c | 64 +++++++++++++++--------- > ip/tunnel.c | 116 ++++++++++++++++++++++++++++++++++++------- > ip/tunnel.h | 5 ++ > 16 files changed, 340 insertions(+), 522 deletions(-) > This looks fine. Thanks for following up. Applied.
