1 |
commit: af56301d75988a12c5347b8e9db6812d36f157b4 |
2 |
Author: Robin H. Johnson <robbat2 <AT> gentoo <DOT> org> |
3 |
AuthorDate: Tue Nov 14 19:16:28 2017 +0000 |
4 |
Commit: Robin H. Johnson <robbat2 <AT> gentoo <DOT> org> |
5 |
CommitDate: Tue Nov 14 19:16:28 2017 +0000 |
6 |
URL: https://gitweb.gentoo.org/proj/netifrc.git/commit/?id=af56301d |
7 |
|
8 |
net/iproute2: refactor veinfo+CMD to _cmd helper. |
9 |
|
10 |
Signed-off-by: Robin H. Johnson <robbat2 <AT> gentoo.org> |
11 |
|
12 |
net/iproute2.sh | 77 +++++++++++++++++++++++++-------------------------------- |
13 |
1 file changed, 34 insertions(+), 43 deletions(-) |
14 |
|
15 |
diff --git a/net/iproute2.sh b/net/iproute2.sh |
16 |
index 7dea195..6d7a3b5 100644 |
17 |
--- a/net/iproute2.sh |
18 |
+++ b/net/iproute2.sh |
19 |
@@ -8,16 +8,21 @@ iproute2_depend() |
20 |
after ifconfig |
21 |
} |
22 |
|
23 |
+# This helper exists for the common pattern of 'veinfo $CMD ; $CMD' |
24 |
+_cmd() |
25 |
+{ |
26 |
+ veinfo "${@}" |
27 |
+ "${@}" |
28 |
+} |
29 |
+ |
30 |
_up() |
31 |
{ |
32 |
- veinfo ip link set dev "${IFACE}" up |
33 |
- ip link set dev "${IFACE}" up |
34 |
+ _cmd ip link set dev "${IFACE}" up |
35 |
} |
36 |
|
37 |
_down() |
38 |
{ |
39 |
- veinfo ip link set dev "${IFACE}" down |
40 |
- ip link set dev "${IFACE}" down |
41 |
+ _cmd ip link set dev "${IFACE}" down |
42 |
} |
43 |
|
44 |
_exists() |
45 |
@@ -59,8 +64,7 @@ _set_flag() |
46 |
flag=${flag#-} |
47 |
opt="off" |
48 |
fi |
49 |
- veinfo ip link set dev "${IFACE}" "${flag}" "${opt}" |
50 |
- ip link set dev "${IFACE}" "${flag}" "${opt}" |
51 |
+ _cmd ip link set dev "${IFACE}" "${flag}" "${opt}" |
52 |
} |
53 |
|
54 |
_get_mac_address() |
55 |
@@ -82,8 +86,7 @@ _get_mac_address() |
56 |
|
57 |
_set_mac_address() |
58 |
{ |
59 |
- veinfo ip link set dev "${IFACE}" address "$1" |
60 |
- ip link set dev "${IFACE}" address "$1" |
61 |
+ _cmd ip link set dev "${IFACE}" address "$1" |
62 |
} |
63 |
|
64 |
_get_inet_addresses() |
65 |
@@ -102,8 +105,7 @@ _get_inet_address() |
66 |
_add_address() |
67 |
{ |
68 |
if [ "$1" = "127.0.0.1/8" -a "${IFACE}" = "lo" ]; then |
69 |
- veinfo ip addr add "$@" dev "${IFACE}" 2>/dev/null |
70 |
- ip addr add "$@" dev "${IFACE}" 2>/dev/null |
71 |
+ _cmd ip addr add "$@" dev "${IFACE}" 2>/dev/null |
72 |
return 0 |
73 |
fi |
74 |
local x |
75 |
@@ -179,8 +181,7 @@ _add_address() |
76 |
|
77 |
# This must appear on a single line, continuations cannot be used |
78 |
set -- "${address}${netmask:+/}${netmask}" ${peer:+peer} ${peer} ${broadcast:+broadcast} ${broadcast} ${anycast:+anycast} ${anycast} ${label:+label} ${label} ${scope:+scope} ${scope} dev "${IFACE}" ${valid_lft:+valid_lft} $valid_lft ${preferred_lft:+preferred_lft} $preferred_lft $confflaglist |
79 |
- veinfo ip addr add "$@" |
80 |
- ip addr add "$@" |
81 |
+ _cmd ip addr add "$@" |
82 |
rc=$? |
83 |
# Check return code in some cases |
84 |
if [ $rc -ne 0 ]; then |
85 |
@@ -251,8 +252,7 @@ _add_route() |
86 |
fgrep -sq "${cmd%% *}" |
87 |
route_already_exists=$? |
88 |
|
89 |
- veinfo ip ${family} route append ${cmd} dev "${IFACE}" |
90 |
- ip ${family} route append ${cmd} dev "${IFACE}" |
91 |
+ _cmd ip ${family} route append ${cmd} dev "${IFACE}" |
92 |
rc=$? |
93 |
# Check return code in some cases |
94 |
if [ $rc -ne 0 ]; then |
95 |
@@ -278,13 +278,10 @@ _add_route() |
96 |
|
97 |
_delete_addresses() |
98 |
{ |
99 |
- veinfo ip addr flush dev "${IFACE}" scope global 2>/dev/null |
100 |
- ip addr flush dev "${IFACE}" scope global 2>/dev/null |
101 |
- veinfo ip addr flush dev "${IFACE}" scope site 2>/dev/null |
102 |
- ip addr flush dev "${IFACE}" scope site 2>/dev/null |
103 |
+ _cmd ip addr flush dev "${IFACE}" scope global 2>/dev/null |
104 |
+ _cmd ip addr flush dev "${IFACE}" scope site 2>/dev/null |
105 |
if [ "${IFACE}" != "lo" ]; then |
106 |
- veinfo ip addr flush dev "${IFACE}" scope host 2>/dev/null |
107 |
- ip addr flush dev "${IFACE}" scope host 2>/dev/null |
108 |
+ _cmd ip addr flush dev "${IFACE}" scope host 2>/dev/null |
109 |
fi |
110 |
return 0 |
111 |
} |
112 |
@@ -294,10 +291,10 @@ _has_carrier() |
113 |
LC_ALL=C ip link show dev "${IFACE}" | grep -q "LOWER_UP" |
114 |
} |
115 |
|
116 |
+# Used by iproute2, ip6rd & ip6to4 |
117 |
_tunnel() |
118 |
{ |
119 |
- veinfo ip tunnel "$@" |
120 |
- ip tunnel "$@" |
121 |
+ _cmd ip $FAMILY tunnel "$@" |
122 |
rc=$? |
123 |
# TODO: check return code in some cases |
124 |
return $rc |
125 |
@@ -349,8 +346,7 @@ _iproute2_policy_routing() |
126 |
_ip_rule_runner -4 add "${rules}" |
127 |
fi |
128 |
fi |
129 |
- veinfo ip -4 route flush table cache dev "${IFACE}" |
130 |
- ip -4 route flush table cache dev "${IFACE}" |
131 |
+ FAMILY="-4" IFACE="${IFACE}" _iproute2_route_flush |
132 |
fi |
133 |
|
134 |
# Kernel may not have IPv6 built in |
135 |
@@ -365,8 +361,7 @@ _iproute2_policy_routing() |
136 |
_ip_rule_runner -6 add "${rules}" |
137 |
fi |
138 |
fi |
139 |
- veinfo ip -6 route flush table cache dev "${IFACE}" |
140 |
- ip -6 route flush table cache dev "${IFACE}" |
141 |
+ FAMILY="-6" IFACE="${IFACE}" _iproute2_route_flush |
142 |
fi |
143 |
} |
144 |
|
145 |
@@ -388,8 +383,7 @@ iproute2_pre_start() |
146 |
esac |
147 |
|
148 |
ebegin "Creating tunnel ${IFVAR}" |
149 |
- veinfo ip ${ipproto} tunnel add ${tunnel} name "${IFACE}" |
150 |
- ip ${ipproto} tunnel add ${tunnel} name "${IFACE}" |
151 |
+ _cmd ip ${ipproto} tunnel add ${tunnel} name "${IFACE}" |
152 |
eend $? || return 1 |
153 |
_up |
154 |
fi |
155 |
@@ -404,8 +398,7 @@ iproute2_pre_start() |
156 |
esac |
157 |
|
158 |
ebegin "Creating interface ${IFVAR}" |
159 |
- veinfo ip ${ipproto} link add "${IFACE}" ${link} |
160 |
- ip ${ipproto} link add "${IFACE}" ${link} |
161 |
+ _cmd ip ${ipproto} link add "${IFACE}" ${link} |
162 |
eend $? || return 1 |
163 |
_up |
164 |
fi |
165 |
@@ -414,16 +407,14 @@ iproute2_pre_start() |
166 |
local mtu= |
167 |
eval mtu=\$mtu_${IFVAR} |
168 |
if [ -n "${mtu}" ]; then |
169 |
- veinfo ip link set dev "${IFACE}" mtu "${mtu}" |
170 |
- ip link set dev "${IFACE}" mtu "${mtu}" |
171 |
+ _cmd ip link set dev "${IFACE}" mtu "${mtu}" |
172 |
fi |
173 |
|
174 |
# TX Queue Length support |
175 |
local len= |
176 |
eval len=\$txqueuelen_${IFVAR} |
177 |
if [ -n "${len}" ]; then |
178 |
- veinfo ip link set dev "${IFACE}" txqueuelen "${len}" |
179 |
- ip link set dev "${IFACE}" txqueuelen "${len}" |
180 |
+ _cmd ip link set dev "${IFACE}" txqueuelen "${len}" |
181 |
fi |
182 |
|
183 |
local policyroute_order= |
184 |
@@ -436,18 +427,20 @@ iproute2_pre_start() |
185 |
|
186 |
_iproute2_tunnel_delete() { |
187 |
ebegin "Destroying tunnel $1" |
188 |
- veinfo ip $2 tunnel del "$1" |
189 |
- ip $2 tunnel del "$1" |
190 |
+ _tunnel del "$1" |
191 |
eend $? |
192 |
} |
193 |
|
194 |
_iproute2_link_delete() { |
195 |
ebegin "Destroying interface $1" |
196 |
- veinfo ip link del dev "$1" |
197 |
- ip link del dev "$1" |
198 |
+ _cmd ip $FAMILY link del dev "$1" |
199 |
eend $? |
200 |
} |
201 |
|
202 |
+_iproute2_route_flush() { |
203 |
+ _cmd ip $FAMILY route flush table cache dev "${IFACE}" |
204 |
+} |
205 |
+ |
206 |
_iproute2_ipv6_tentative() |
207 |
{ |
208 |
[ -n "$(LC_ALL=C ip -family inet6 addr show dev ${IFACE} tentative)" ] && return 0 |
209 |
@@ -502,8 +495,7 @@ iproute2_post_stop() |
210 |
|
211 |
# Only do something if the interface actually exist |
212 |
if _exists; then |
213 |
- veinfo ip -4 route flush table cache dev "${IFACE}" |
214 |
- ip -4 route flush table cache dev "${IFACE}" |
215 |
+ FAMILY="-4" IFACE="${IFACE}" _iproute2_route_flush |
216 |
fi |
217 |
fi |
218 |
|
219 |
@@ -517,8 +509,7 @@ iproute2_post_stop() |
220 |
|
221 |
# Only do something if the interface actually exist |
222 |
if _exists; then |
223 |
- veinfo ip -6 route flush table cache dev "${IFACE}" |
224 |
- ip -6 route flush table cache dev "${IFACE}" |
225 |
+ FAMILY="-6" IFACE="${IFACE}" _iproute2_route_flush |
226 |
fi |
227 |
fi |
228 |
|
229 |
@@ -527,7 +518,7 @@ iproute2_post_stop() |
230 |
if [ -n "$(ip tunnel show "${IFACE}" 2>/dev/null)" ]; then |
231 |
_iproute2_tunnel_delete "${IFACE}" |
232 |
elif [ -n "$(ip -6 tunnel show "${IFACE}" 2>/dev/null)" ]; then |
233 |
- _iproute2_tunnel_delete "${IFACE}" -6 |
234 |
+ FAMILY=-6 _iproute2_tunnel_delete "${IFACE}" |
235 |
fi |
236 |
|
237 |
[ -n "$(ip link show "${IFACE}" type gretap 2>/dev/null)" ] && _iproute2_link_delete "${IFACE}" |