From: Serhey Popovych <serhe.popovych@gmail.com>
To: netdev@vger.kernel.org
Cc: dsahern@gmail.com
Subject: [PATCH iproute2-next v3 0/8] iplink: Improve iplink_parse()
Date: Thu, 22 Feb 2018 15:01:58 +0200 [thread overview]
Message-ID: <1519304526-18848-1-git-send-email-serhe.popovych@gmail.com> (raw)
This is main routine to parse ip-link(8) configuration parameters.
Main reason to improve it is to pass network device @name, @dev and
other parameters to kind specific ->parse_opt() function so they can use
this information.
For example later we will extend iplink_get() to parse netlink
attributes deeper and replace open coded rtnl_talk() in ip/tunnel
modules to simplify getting existing tunnel information.
Among main change there is a number of patches to prepare for it
that improve iplink_parse() in some way.
See individual patch description message for more information.
v3
Move vxlan/veth ifinfomsg save/restore to separate patch to
make clear change that perform most of request buffer setups
and checks in iplink_parse().
Update commit message descriptions and extra new line from
"utils: Introduce and use nodev() helper routine" patch.
v2
Terminate via exit() when failing to parse command line arguments
to help identify failing line in batch mode.
Thanks,
Serhii
Serhey Popovych (8):
utils: Introduce and use nodev() helper routine
iplink: Correctly report error when network device isn't found
iplink: Use "dev" and "name" parameters interchangeable when possible
iplink: Follow documented behaviour when "index" is given
veth,vxcan: Save/reinitialize/restore whole @struct ifinfomsg
iplink: Perform most of request buffer setups and checks in
iplink_parse()
iplink: Move data structures to block of their users
iplink: Reduce number of arguments to iplink_parse()
bridge/fdb.c | 17 ++--
bridge/link.c | 8 +-
bridge/mdb.c | 19 ++---
bridge/vlan.c | 7 +-
include/utils.h | 1 +
ip/ip6tunnel.c | 6 +-
ip/ip_common.h | 17 +++-
ip/ipaddress.c | 7 +-
ip/iplink.c | 200 +++++++++++++++++++++++++++-------------------
ip/iplink_bond.c | 8 +-
ip/iplink_bond_slave.c | 4 +-
ip/iplink_bridge.c | 11 ++-
ip/iplink_bridge_slave.c | 4 +-
ip/iplink_can.c | 4 +-
ip/iplink_geneve.c | 4 +-
ip/iplink_hsr.c | 4 +-
ip/iplink_ipoib.c | 4 +-
ip/iplink_ipvlan.c | 4 +-
ip/iplink_macvlan.c | 4 +-
ip/iplink_vlan.c | 4 +-
ip/iplink_vrf.c | 5 +-
ip/iplink_vxcan.c | 41 +++-------
ip/iplink_vxlan.c | 11 ++-
ip/iplink_xdp.c | 7 +-
ip/ipmacsec.c | 4 +-
ip/ipmroute.c | 7 +-
ip/ipneigh.c | 14 ++--
ip/ipntable.c | 6 +-
ip/iproute.c | 36 +++------
ip/iproute_lwtunnel.c | 4 +-
ip/iptunnel.c | 6 +-
ip/link_gre.c | 43 +++++-----
ip/link_gre6.c | 43 +++++-----
ip/link_ip6tnl.c | 40 +++++-----
ip/link_iptnl.c | 40 +++++-----
ip/link_veth.c | 41 +++-------
ip/link_vti.c | 43 +++++-----
ip/link_vti6.c | 43 +++++-----
lib/utils.c | 6 ++
tc/m_mirred.c | 6 +-
tc/tc_class.c | 14 ++--
tc/tc_filter.c | 18 ++---
tc/tc_qdisc.c | 12 +--
43 files changed, 404 insertions(+), 423 deletions(-)
--
1.7.10.4
next reply other threads:[~2018-02-22 13:02 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-22 13:01 Serhey Popovych [this message]
2018-02-22 13:01 ` [PATCH iproute2-next v3 1/8] utils: Introduce and use nodev() helper routine Serhey Popovych
2018-02-22 13:02 ` [PATCH iproute2-next v3 2/8] iplink: Correctly report error when network device isn't found Serhey Popovych
2018-02-23 23:41 ` David Ahern
2018-02-25 12:07 ` Serhey Popovych
2018-02-22 13:02 ` [PATCH iproute2-next v3 3/8] iplink: Use "dev" and "name" parameters interchangeable when possible Serhey Popovych
2018-02-22 13:02 ` [PATCH iproute2-next v3 4/8] iplink: Follow documented behaviour when "index" is given Serhey Popovych
2018-02-22 13:02 ` [PATCH iproute2-next v3 5/8] veth,vxcan: Save/reinitialize/restore whole @struct ifinfomsg Serhey Popovych
2018-02-26 3:28 ` David Ahern
2018-02-26 15:48 ` Serhey Popovych
2018-02-26 15:57 ` Serhey Popovych
2018-02-22 13:02 ` [PATCH iproute2-next v3 6/8] iplink: Perform most of request buffer setups and checks in iplink_parse() Serhey Popovych
2018-02-26 3:31 ` David Ahern
2018-02-26 15:51 ` Serhey Popovych
2018-02-22 13:02 ` [PATCH iproute2-next v3 7/8] iplink: Move data structures to block of their users Serhey Popovych
2018-02-22 13:02 ` [PATCH iproute2-next v3 8/8] iplink: Reduce number of arguments to iplink_parse() Serhey Popovych
2018-02-26 3:35 ` David Ahern
2018-02-26 15:44 ` Serhey Popovych
2018-02-26 16:07 ` Serhey Popovych
2018-02-26 18:06 ` Stephen Hemminger
2018-02-26 18:20 ` Serhey Popovych
2018-02-26 18:27 ` David Ahern
2018-02-26 18:38 ` Serhey Popovych
2018-02-26 20:09 ` Serhey Popovych
2018-02-26 21:35 ` Stephen Hemminger
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1519304526-18848-1-git-send-email-serhe.popovych@gmail.com \
--to=serhe.popovych@gmail.com \
--cc=dsahern@gmail.com \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).