bridge: fdb: add support for src_vni option

[mirror_iproute2.git] / tc / m_vlan.c

diff --git a/tc/m_vlan.c b/tc/m_vlan.c
index cccb4996b05f3f80ebf6faa18917df164e15c8b9..412f6aa1000e86fd2fff02039dace7912a997c45 100644 (file)
--- a/tc/m_vlan.c
+++ b/tc/m_vlan.c
@@ -27,13 +27,14 @@ static const char * const action_names[] = {
 
 static void explain(void)
 {
-       fprintf(stderr, "Usage: vlan pop\n");
-       fprintf(stderr, "       vlan push [ protocol VLANPROTO ] id VLANID [ priority VLANPRIO ] [CONTROL]\n");
-       fprintf(stderr, "       vlan modify [ protocol VLANPROTO ] id VLANID [ priority VLANPRIO ] [CONTROL]\n");
-       fprintf(stderr, "       VLANPROTO is one of 802.1Q or 802.1AD\n");
-       fprintf(stderr, "            with default: 802.1Q\n");
-       fprintf(stderr, "       CONTROL := reclassify | pipe | drop | continue | pass |\n");
-       fprintf(stderr, "                  goto chain <CHAIN_INDEX>\n");
+       fprintf(stderr,
+               "Usage: vlan pop\n"
+               "       vlan push [ protocol VLANPROTO ] id VLANID [ priority VLANPRIO ] [CONTROL]\n"
+               "       vlan modify [ protocol VLANPROTO ] id VLANID [ priority VLANPRIO ] [CONTROL]\n"
+               "       VLANPROTO is one of 802.1Q or 802.1AD\n"
+               "            with default: 802.1Q\n"
+               "       CONTROL := reclassify | pipe | drop | continue | pass |\n"
+               "                  goto chain <CHAIN_INDEX>\n");
 }
 
 static void usage(void)
@@ -47,6 +48,14 @@ static bool has_push_attribs(int action)
        return action == TCA_VLAN_ACT_PUSH || action == TCA_VLAN_ACT_MODIFY;
 }
 
+static void unexpected(const char *arg)
+{
+       fprintf(stderr,
+               "unexpected \"%s\" - action already specified\n",
+               arg);
+       explain();
+}
+
 static int parse_vlan(struct action_util *a, int *argc_p, char ***argv_p,
                      int tca_id, struct nlmsghdr *n)
 {
@@ -70,57 +79,42 @@ static int parse_vlan(struct action_util *a, int *argc_p, char ***argv_p,
        while (argc > 0) {
                if (matches(*argv, "pop") == 0) {
                        if (action) {
-                               fprintf(stderr, "unexpected \"%s\" - action already specified\n",
-                                       *argv);
-                               explain();
+                               unexpected(*argv);
                                return -1;
                        }
                        action = TCA_VLAN_ACT_POP;
                } else if (matches(*argv, "push") == 0) {
                        if (action) {
-                               fprintf(stderr, "unexpected \"%s\" - action already specified\n",
-                                       *argv);
-                               explain();
+                               unexpected(*argv);
                                return -1;
                        }
                        action = TCA_VLAN_ACT_PUSH;
                } else if (matches(*argv, "modify") == 0) {
                        if (action) {
-                               fprintf(stderr, "unexpected \"%s\" - action already specified\n",
-                                       *argv);
-                               explain();
+                               unexpected(*argv);
                                return -1;
                        }
                        action = TCA_VLAN_ACT_MODIFY;
                } else if (matches(*argv, "id") == 0) {
-                       if (!has_push_attribs(action)) {
-                               fprintf(stderr, "\"%s\" is only valid for push/modify\n",
-                                       *argv);
-                               explain();
-                               return -1;
-                       }
+                       if (!has_push_attribs(action))
+                               invarg("only valid for push/modify", *argv);
+
                        NEXT_ARG();
                        if (get_u16(&id, *argv, 0))
                                invarg("id is invalid", *argv);
                        id_set = 1;
                } else if (matches(*argv, "protocol") == 0) {
-                       if (!has_push_attribs(action)) {
-                               fprintf(stderr, "\"%s\" is only valid for push/modify\n",
-                                       *argv);
-                               explain();
-                               return -1;
-                       }
+                       if (!has_push_attribs(action))
+                               invarg("only valid for push/modify", *argv);
+
                        NEXT_ARG();
                        if (ll_proto_a2n(&proto, *argv))
                                invarg("protocol is invalid", *argv);
                        proto_set = 1;
                } else if (matches(*argv, "priority") == 0) {
-                       if (!has_push_attribs(action)) {
-                               fprintf(stderr, "\"%s\" is only valid for push/modify\n",
-                                       *argv);
-                               explain();
-                               return -1;
-                       }
+                       if (!has_push_attribs(action))
+                               invarg("only valid for push/modify", *argv);
+
                        NEXT_ARG();
                        if (get_u8(&prio, *argv, 0) || (prio & ~0x7))
                                invarg("prio is invalid", *argv);
@@ -157,8 +151,7 @@ static int parse_vlan(struct action_util *a, int *argc_p, char ***argv_p,
        }
 
        parm.v_action = action;
-       tail = NLMSG_TAIL(n);
-       addattr_l(n, MAX_MSG, tca_id, NULL, 0);
+       tail = addattr_nest(n, MAX_MSG, tca_id);
        addattr_l(n, MAX_MSG, TCA_VLAN_PARMS, &parm, sizeof(parm));
        if (id_set)
                addattr_l(n, MAX_MSG, TCA_VLAN_PUSH_VLAN_ID, &id, 2);
@@ -175,7 +168,7 @@ static int parse_vlan(struct action_util *a, int *argc_p, char ***argv_p,
        if (prio_set)
                addattr8(n, MAX_MSG, TCA_VLAN_PUSH_VLAN_PRIORITY, prio);
 
-       tail->rta_len = (char *)NLMSG_TAIL(n) - (char *)tail;
+       addattr_nest_end(n, tail);
 
        *argc_p = argc;
        *argv_p = argv;
@@ -195,39 +188,40 @@ static int print_vlan(struct action_util *au, FILE *f, struct rtattr *arg)
        parse_rtattr_nested(tb, TCA_VLAN_MAX, arg);
 
        if (!tb[TCA_VLAN_PARMS]) {
-               fprintf(f, "[NULL vlan parameters]");
+               print_string(PRINT_FP, NULL, "%s", "[NULL vlan parameters]");
                return -1;
        }
        parm = RTA_DATA(tb[TCA_VLAN_PARMS]);
 
-       fprintf(f, " vlan");
+       print_string(PRINT_ANY, "kind", "%s ", "vlan");
+       print_string(PRINT_ANY, "vlan_action", " %s",
+                    action_names[parm->v_action]);
 
        switch (parm->v_action) {
-       case TCA_VLAN_ACT_POP:
-               fprintf(f, " pop");
-               break;
        case TCA_VLAN_ACT_PUSH:
        case TCA_VLAN_ACT_MODIFY:
-               fprintf(f, " %s", action_names[parm->v_action]);
                if (tb[TCA_VLAN_PUSH_VLAN_ID]) {
                        val = rta_getattr_u16(tb[TCA_VLAN_PUSH_VLAN_ID]);
-                       fprintf(f, " id %u", val);
+                       print_uint(PRINT_ANY, "id", " id %u", val);
                }
                if (tb[TCA_VLAN_PUSH_VLAN_PROTOCOL]) {
-                       fprintf(f, " protocol %s",
-                               ll_proto_n2a(rta_getattr_u16(tb[TCA_VLAN_PUSH_VLAN_PROTOCOL]),
-                                            b1, sizeof(b1)));
+                       __u16 proto;
+
+                       proto = rta_getattr_u16(tb[TCA_VLAN_PUSH_VLAN_PROTOCOL]);
+                       print_string(PRINT_ANY, "protocol", " protocol %s",
+                                    ll_proto_n2a(proto, b1, sizeof(b1)));
                }
                if (tb[TCA_VLAN_PUSH_VLAN_PRIORITY]) {
                        val = rta_getattr_u8(tb[TCA_VLAN_PUSH_VLAN_PRIORITY]);
-                       fprintf(f, " priority %u", val);
+                       print_uint(PRINT_ANY, "priority", " priority %u", val);
                }
                break;
        }
        print_action_control(f, " ", parm->action, "");
 
-       fprintf(f, "\n\t index %u ref %d bind %d", parm->index, parm->refcnt,
-               parm->bindcnt);
+       print_uint(PRINT_ANY, "index", "\n\t index %u", parm->index);
+       print_int(PRINT_ANY, "ref", " ref %d", parm->refcnt);
+       print_int(PRINT_ANY, "bind", " bind %d", parm->bindcnt);
 
        if (show_stats) {
                if (tb[TCA_VLAN_TM]) {
@@ -237,7 +231,7 @@ static int print_vlan(struct action_util *au, FILE *f, struct rtattr *arg)
                }
        }
 
-       fprintf(f, "\n ");
+       print_string(PRINT_FP, NULL, "%s", "\n");
 
        return 0;
 }