iplink: cleanup style errors

Fix long strings causing checkpatch warnings
This commit is contained in:
Stephen Hemminger 2016-10-09 19:24:38 -07:00
parent 56e9f0ab19
commit 6773bcc227
1 changed files with 56 additions and 53 deletions

View File

@ -45,60 +45,62 @@ static int iplink_have_newlink(void);
void iplink_usage(void) void iplink_usage(void)
{ {
if (iplink_have_newlink()) { if (iplink_have_newlink()) {
fprintf(stderr, "Usage: ip link add [link DEV] [ name ] NAME\n"); fprintf(stderr,
fprintf(stderr, " [ txqueuelen PACKETS ]\n"); "Usage: ip link add [link DEV] [ name ] NAME\n"
fprintf(stderr, " [ address LLADDR ]\n"); " [ txqueuelen PACKETS ]\n"
fprintf(stderr, " [ broadcast LLADDR ]\n"); " [ address LLADDR ]\n"
fprintf(stderr, " [ mtu MTU ] [index IDX ]\n"); " [ broadcast LLADDR ]\n"
fprintf(stderr, " [ numtxqueues QUEUE_COUNT ]\n"); " [ mtu MTU ] [index IDX ]\n"
fprintf(stderr, " [ numrxqueues QUEUE_COUNT ]\n"); " [ numtxqueues QUEUE_COUNT ]\n"
fprintf(stderr, " type TYPE [ ARGS ]\n"); " [ numrxqueues QUEUE_COUNT ]\n"
fprintf(stderr, " ip link delete { DEVICE | dev DEVICE | group DEVGROUP } type TYPE [ ARGS ]\n"); " type TYPE [ ARGS ]\n"
fprintf(stderr, "\n"); " ip link delete { DEVICE | dev DEVICE | group DEVGROUP } type TYPE [ ARGS ]\n"
fprintf(stderr, " ip link set { DEVICE | dev DEVICE | group DEVGROUP }\n"); "\n"
fprintf(stderr, " [ { up | down } ]\n"); " ip link set { DEVICE | dev DEVICE | group DEVGROUP }\n"
fprintf(stderr, " [ type TYPE ARGS ]\n"); " [ { up | down } ]\n"
" [ type TYPE ARGS ]\n");
} else } else
fprintf(stderr, "Usage: ip link set DEVICE [ { up | down } ]\n"); fprintf(stderr, "Usage: ip link set DEVICE [ { up | down } ]\n");
fprintf(stderr, " [ arp { on | off } ]\n"); fprintf(stderr,
fprintf(stderr, " [ dynamic { on | off } ]\n"); " [ arp { on | off } ]\n"
fprintf(stderr, " [ multicast { on | off } ]\n"); " [ dynamic { on | off } ]\n"
fprintf(stderr, " [ allmulticast { on | off } ]\n"); " [ multicast { on | off } ]\n"
fprintf(stderr, " [ promisc { on | off } ]\n"); " [ allmulticast { on | off } ]\n"
fprintf(stderr, " [ trailers { on | off } ]\n"); " [ promisc { on | off } ]\n"
fprintf(stderr, " [ txqueuelen PACKETS ]\n"); " [ trailers { on | off } ]\n"
fprintf(stderr, " [ name NEWNAME ]\n"); " [ txqueuelen PACKETS ]\n"
fprintf(stderr, " [ address LLADDR ]\n"); " [ name NEWNAME ]\n"
fprintf(stderr, " [ broadcast LLADDR ]\n"); " [ address LLADDR ]\n"
fprintf(stderr, " [ mtu MTU ]\n"); " [ broadcast LLADDR ]\n"
fprintf(stderr, " [ netns { PID | NAME } ]\n"); " [ mtu MTU ]\n"
fprintf(stderr, " [ link-netnsid ID ]\n"); " [ netns { PID | NAME } ]\n"
fprintf(stderr, " [ alias NAME ]\n"); " [ link-netnsid ID ]\n"
fprintf(stderr, " [ vf NUM [ mac LLADDR ]\n"); " [ alias NAME ]\n"
fprintf(stderr, " [ vlan VLANID [ qos VLAN-QOS ] [ proto VLAN-PROTO ] ]\n"); " [ vf NUM [ mac LLADDR ]\n"
" [ vlan VLANID [ qos VLAN-QOS ] [ proto VLAN-PROTO ] ]\n"
fprintf(stderr, " [ rate TXRATE ]\n"); " [ rate TXRATE ]\n"
fprintf(stderr, " [ max_tx_rate TXRATE ]\n"); " [ max_tx_rate TXRATE ]\n"
fprintf(stderr, " [ min_tx_rate TXRATE ]\n"); " [ min_tx_rate TXRATE ]\n"
fprintf(stderr, " [ spoofchk { on | off} ]\n"); " [ spoofchk { on | off} ]\n"
fprintf(stderr, " [ query_rss { on | off} ]\n"); " [ query_rss { on | off} ]\n"
fprintf(stderr, " [ state { auto | enable | disable} ] ]\n"); " [ state { auto | enable | disable} ] ]\n"
fprintf(stderr, " [ trust { on | off} ] ]\n"); " [ trust { on | off} ] ]\n"
fprintf(stderr, " [ master DEVICE ][ vrf NAME ]\n"); " [ master DEVICE ][ vrf NAME ]\n"
fprintf(stderr, " [ nomaster ]\n"); " [ nomaster ]\n"
fprintf(stderr, " [ addrgenmode { eui64 | none | stable_secret | random } ]\n"); " [ addrgenmode { eui64 | none | stable_secret | random } ]\n"
fprintf(stderr, " [ protodown { on | off } ]\n"); " [ protodown { on | off } ]\n"
fprintf(stderr, " ip link show [ DEVICE | group GROUP ] [up] [master DEV] [vrf NAME] [type TYPE]\n"); " ip link show [ DEVICE | group GROUP ] [up] [master DEV] [vrf NAME] [type TYPE]\n");
if (iplink_have_newlink()) { if (iplink_have_newlink()) {
fprintf(stderr, " ip link help [ TYPE ]\n"); fprintf(stderr,
fprintf(stderr, "\n"); " ip link help [ TYPE ]\n\n"
fprintf(stderr, "TYPE := { vlan | veth | vcan | dummy | ifb | macvlan | macvtap |\n"); "TYPE := { vlan | veth | vcan | dummy | ifb | macvlan | macvtap |\n"
fprintf(stderr, " bridge | bond | ipoib | ip6tnl | ipip | sit | vxlan |\n"); " bridge | bond | ipoib | ip6tnl | ipip | sit | vxlan |\n"
fprintf(stderr, " gre | gretap | ip6gre | ip6gretap | vti | nlmon |\n"); " gre | gretap | ip6gre | ip6gretap | vti | nlmon |\n"
fprintf(stderr, " bond_slave | ipvlan | geneve | bridge_slave | vrf | macsec }\n"); " bond_slave | ipvlan | geneve | bridge_slave | vrf | macsec }\n");
} }
exit(-1); exit(-1);
} }
@ -291,8 +293,7 @@ static void iplink_parse_vf_vlan_info(int vf, int *argcp, char ***argvp,
SPRINT_BUF(b2); SPRINT_BUF(b2);
char msg[64 + sizeof(b1) + sizeof(b2)]; char msg[64 + sizeof(b1) + sizeof(b2)];
sprintf(msg, "Invalid \"vlan protocol\"" sprintf(msg, "Invalid \"vlan protocol\" value - supported %s, %s\n",
" value - supported %s, %s\n",
ll_proto_n2a(htons(ETH_P_8021Q), ll_proto_n2a(htons(ETH_P_8021Q),
b1, sizeof(b1)), b1, sizeof(b1)),
ll_proto_n2a(htons(ETH_P_8021AD), ll_proto_n2a(htons(ETH_P_8021AD),
@ -859,8 +860,9 @@ static int iplink_modify(int cmd, unsigned int flags, int argc, char **argv)
&group, sizeof(group)); &group, sizeof(group));
else { else {
if (argc) { if (argc) {
fprintf(stderr, "Garbage instead of arguments \"%s ...\". Try \"ip link help\".\n", fprintf(stderr,
*argv); "Garbage instead of arguments \"%s ...\". Try \"ip link help\".\n",
*argv);
return -1; return -1;
} }
if (flags & NLM_F_CREATE) { if (flags & NLM_F_CREATE) {
@ -950,8 +952,9 @@ static int iplink_modify(int cmd, unsigned int flags, int argc, char **argv)
} else if (argc) { } else if (argc) {
if (matches(*argv, "help") == 0) if (matches(*argv, "help") == 0)
usage(); usage();
fprintf(stderr, "Garbage instead of arguments \"%s ...\". Try \"ip link help\".\n", fprintf(stderr,
*argv); "Garbage instead of arguments \"%s ...\". Try \"ip link help\".\n",
*argv);
return -1; return -1;
} }
addattr_nest_end(&req.n, linkinfo); addattr_nest_end(&req.n, linkinfo);