Commit 488b41d0 authored by Jamal Hadi Salim's avatar Jamal Hadi Salim Committed by Stephen Hemminger

tc: flower no need to specify the ethertype

since all tc classifiers are required to specify ethertype as part of grammar
By not allowing eth_type to be specified we remove contradiction for
example when a user specifies:
tc filter add ... priority xxx protocol ip flower eth_type ipv6
This patch removes that contradiction
Signed-off-by: default avatarJamal Hadi Salim <jhs@mojatatu.com>
parent 8f80d450
...@@ -31,7 +31,7 @@ static void explain(void) ...@@ -31,7 +31,7 @@ static void explain(void)
fprintf(stderr, " MATCH := { indev DEV-NAME | \n"); fprintf(stderr, " MATCH := { indev DEV-NAME | \n");
fprintf(stderr, " dst_mac MAC-ADDR | \n"); fprintf(stderr, " dst_mac MAC-ADDR | \n");
fprintf(stderr, " src_mac MAC-ADDR | \n"); fprintf(stderr, " src_mac MAC-ADDR | \n");
fprintf(stderr, " eth_type [ipv4 | ipv6 | ETH-TYPE ] | \n"); fprintf(stderr, " [ipv4 | ipv6 ] | \n");
fprintf(stderr, " ip_proto [tcp | udp | IP-PROTO ] | \n"); fprintf(stderr, " ip_proto [tcp | udp | IP-PROTO ] | \n");
fprintf(stderr, " dst_ip [ IPV4-ADDR | IPV6-ADDR ] | \n"); fprintf(stderr, " dst_ip [ IPV4-ADDR | IPV6-ADDR ] | \n");
fprintf(stderr, " src_ip [ IPV4-ADDR | IPV6-ADDR ] | \n"); fprintf(stderr, " src_ip [ IPV4-ADDR | IPV6-ADDR ] | \n");
...@@ -60,29 +60,6 @@ static int flower_parse_eth_addr(char *str, int addr_type, int mask_type, ...@@ -60,29 +60,6 @@ static int flower_parse_eth_addr(char *str, int addr_type, int mask_type,
return 0; return 0;
} }
static int flower_parse_eth_type(char *str, int type, __be16 *p_eth_type,
struct nlmsghdr *n)
{
int ret;
__be16 eth_type;
if (matches(str, "ipv4") == 0) {
eth_type = htons(ETH_P_IP);
} else if (matches(str, "ipv6") == 0) {
eth_type = htons(ETH_P_IPV6);
} else {
__u16 tmp;
ret = get_u16(&tmp, str, 16);
if (ret)
return -1;
eth_type = htons(tmp);
}
addattr16(n, MAX_MSG, type, eth_type);
*p_eth_type = eth_type;
return 0;
}
static int flower_parse_ip_proto(char *str, __be16 eth_type, int type, static int flower_parse_ip_proto(char *str, __be16 eth_type, int type,
__u8 *p_ip_proto, struct nlmsghdr *n) __u8 *p_ip_proto, struct nlmsghdr *n)
{ {
...@@ -188,12 +165,9 @@ static int flower_parse_opt(struct filter_util *qu, char *handle, ...@@ -188,12 +165,9 @@ static int flower_parse_opt(struct filter_util *qu, char *handle,
int ret; int ret;
struct tcmsg *t = NLMSG_DATA(n); struct tcmsg *t = NLMSG_DATA(n);
struct rtattr *tail; struct rtattr *tail;
__be16 eth_type = 0; __be16 eth_type = TC_H_MIN(t->tcm_info);
__u8 ip_proto = 0xff; __u8 ip_proto = 0xff;
if (argc == 0)
return 0;
if (handle) { if (handle) {
ret = get_u32(&t->tcm_handle, handle, 0); ret = get_u32(&t->tcm_handle, handle, 0);
if (ret) { if (ret) {
...@@ -205,6 +179,11 @@ static int flower_parse_opt(struct filter_util *qu, char *handle, ...@@ -205,6 +179,11 @@ static int flower_parse_opt(struct filter_util *qu, char *handle,
tail = (struct rtattr *) (((void *) n) + NLMSG_ALIGN(n->nlmsg_len)); tail = (struct rtattr *) (((void *) n) + NLMSG_ALIGN(n->nlmsg_len));
addattr_l(n, MAX_MSG, TCA_OPTIONS, NULL, 0); addattr_l(n, MAX_MSG, TCA_OPTIONS, NULL, 0);
if (argc == 0) {
/*at minimal we will match all ethertype packets */
goto parse_done;
}
while (argc > 0) { while (argc > 0) {
if (matches(*argv, "classid") == 0 || if (matches(*argv, "classid") == 0 ||
matches(*argv, "flowid") == 0) { matches(*argv, "flowid") == 0) {
...@@ -244,15 +223,6 @@ static int flower_parse_opt(struct filter_util *qu, char *handle, ...@@ -244,15 +223,6 @@ static int flower_parse_opt(struct filter_util *qu, char *handle,
fprintf(stderr, "Illegal \"src_mac\"\n"); fprintf(stderr, "Illegal \"src_mac\"\n");
return -1; return -1;
} }
} else if (matches(*argv, "eth_type") == 0) {
NEXT_ARG();
ret = flower_parse_eth_type(*argv,
TCA_FLOWER_KEY_ETH_TYPE,
&eth_type, n);
if (ret < 0) {
fprintf(stderr, "Illegal \"eth_type\"\n");
return -1;
}
} else if (matches(*argv, "ip_proto") == 0) { } else if (matches(*argv, "ip_proto") == 0) {
NEXT_ARG(); NEXT_ARG();
ret = flower_parse_ip_proto(*argv, eth_type, ret = flower_parse_ip_proto(*argv, eth_type,
...@@ -323,6 +293,14 @@ static int flower_parse_opt(struct filter_util *qu, char *handle, ...@@ -323,6 +293,14 @@ static int flower_parse_opt(struct filter_util *qu, char *handle,
argc--; argv++; argc--; argv++;
} }
parse_done:
ret = addattr16(n, MAX_MSG, TCA_FLOWER_KEY_ETH_TYPE, eth_type);
if (ret) {
fprintf(stderr, "Illegal \"eth_type\"(0x%x)\n",
ntohs(eth_type));
return -1;
}
tail->rta_len = (((void*)n)+n->nlmsg_len) - (void*)tail; tail->rta_len = (((void*)n)+n->nlmsg_len) - (void*)tail;
return 0; return 0;
...@@ -489,7 +467,8 @@ static int flower_print_opt(struct filter_util *qu, FILE *f, ...@@ -489,7 +467,8 @@ static int flower_print_opt(struct filter_util *qu, FILE *f,
if (tb[TCA_FLOWER_CLASSID]) { if (tb[TCA_FLOWER_CLASSID]) {
SPRINT_BUF(b1); SPRINT_BUF(b1);
fprintf(f, "classid %s ", fprintf(f, "classid %s ",
sprint_tc_classid(rta_getattr_u32(tb[TCA_FLOWER_CLASSID]), b1)); sprint_tc_classid(rta_getattr_u32(tb[TCA_FLOWER_CLASSID]),
b1));
} }
if (tb[TCA_FLOWER_INDEV]) { if (tb[TCA_FLOWER_INDEV]) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment