From: Serhey Popovych <serhe.popovych@gmail.com>
To: netdev@vger.kernel.org
Cc: dsahern@gmail.com
Subject: [PATCH iproute2-next v4 0/9] ipaddress: Make print_linkinfo_brief() static
Date: Thu, 15 Feb 2018 20:33:59 +0200 [thread overview]
Message-ID: <1518719648-30134-1-git-send-email-serhe.popovych@gmail.com> (raw)
With this series I propose to make print_linkinfo_brief() static in
favor of print_linkinfo() as single point for linkinfo printing.
Changes presented with this series tested using following script:
\#!/bin/bash
iproute2_dir="$1"
iface='eth0.2'
pushd "$iproute2_dir" &>/dev/null
for i in new old; do
DIR="/tmp/$i"
mkdir -p "$DIR"
ln -snf ip.$i ip/ip
# normal
ip/ip link show >"$DIR/ip-link-show"
ip/ip -4 addr show >"$DIR/ip-4-addr-show"
ip/ip -6 addr show >"$DIR/ip-6-addr-show"
ip/ip addr show dev "$iface" >"$DIR/ip-addr-show-$iface"
# brief
ip/ip -br link show >"$DIR/ip-br-link-show"
ip/ip -br -4 addr show >"$DIR/ip-br-4-addr-show"
ip/ip -br -6 addr show >"$DIR/ip-br-6-addr-show"
ip/ip -br addr show dev "$iface" >"$DIR/ip-br-addr-show-$iface"
done
rm -f ip/ip
diff -urN /tmp/{old,new} |sed -n -Ee'/^(-{3}|\+{3})[[:space:]]+/!p'
rc=$?
popd &>/dev/null
exit $rc
Expected results : <no output>
Actual results : <no output>
Although test coverage is far from ideal in my opinion it covers most
important aspects of the changes presented by the series.
All this work is done in prepare of iplink_get() enhancements to support
attribute parse that finally will be used to simplify ip/tunnel
RTM_GETLINK code.
As always reviews, comments, suggestions and criticism is welcome.
v4
Print master network device name using correct color.
v3
Fixed subject line.
v2
Rebased to current iproute2-next/master. No changes.
Thanks,
Serhii
Serhey Popovych (9):
ipaddress: Abstract IFA_LABEL matching code
ipaddress: ll_map: Replace ll_idx_n2a() with ll_index_to_name()
utils: Reimplement ll_idx_n2a() and introduce ll_idx_a2n()
ipaddress: Improve print_linkinfo()
ipaddress: Simplify print_linkinfo_brief() and it's usage
lib: Correct object file dependencies
utils: Introduce and use get_ifname_rta()
utils: Introduce and use print_name_and_link() to print name@link
ipaddress: Make print_linkinfo_brief() static
bridge/link.c | 21 ++---
include/ll_map.h | 4 +-
include/utils.h | 5 ++
ip/ip_common.h | 2 -
ip/ipaddress.c | 231 ++++++++++++++----------------------------------------
ip/iplink.c | 5 +-
lib/Makefile | 4 +-
lib/ll_map.c | 31 +++++---
lib/utils.c | 68 ++++++++++++++++
9 files changed, 166 insertions(+), 205 deletions(-)
--
1.7.10.4
next reply other threads:[~2018-02-15 18:34 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-02-15 18:33 Serhey Popovych [this message]
2018-02-15 18:34 ` [PATCH iproute2-next v4 1/9] ipaddress: Abstract IFA_LABEL matching code Serhey Popovych
2018-02-15 18:34 ` [PATCH iproute2-next v4 2/9] ipaddress: ll_map: Replace ll_idx_n2a() with ll_index_to_name() Serhey Popovych
2018-02-15 18:34 ` [PATCH iproute2-next v4 3/9] utils: Reimplement ll_idx_n2a() and introduce ll_idx_a2n() Serhey Popovych
2018-02-15 18:34 ` [PATCH iproute2-next v4 4/9] ipaddress: Improve print_linkinfo() Serhey Popovych
2018-02-15 18:34 ` [PATCH iproute2-next v4 5/9] ipaddress: Simplify print_linkinfo_brief() and it's usage Serhey Popovych
2018-02-15 18:34 ` [PATCH iproute2-next v4 6/9] lib: Correct object file dependencies Serhey Popovych
2018-02-15 18:34 ` [PATCH iproute2-next v4 7/9] utils: Introduce and use get_ifname_rta() Serhey Popovych
2018-02-15 19:54 ` David Ahern
2018-02-15 21:25 ` Serhey Popovych
2018-02-15 18:34 ` [PATCH iproute2-next v4 8/9] utils: Introduce and use print_name_and_link() to print name@link Serhey Popovych
2018-02-15 18:59 ` Stephen Hemminger
2018-02-15 19:27 ` David Ahern
2018-02-15 18:34 ` [PATCH iproute2-next v4 9/9] ipaddress: Make print_linkinfo_brief() static Serhey Popovych
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=1518719648-30134-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).