Fri, Oct 12, 2018 at 03:54:42PM CEST, amritha.namb...@intel.com wrote:

[...]

>@@ -1516,6 +1625,22 @@ static int flower_print_opt(struct filter_util *qu, 
>FILE *f,
>       if (nl_type >= 0)
>               flower_print_port("src_port", tb[nl_type]);
> 
>+      if (flower_port_range_attr_type(ip_proto, FLOWER_ENDPOINT_DST, &range)
>+          == 0) {
>+              flower_print_port_range("dst_port_min",
>+                                      tb[range.min_port_type]);
>+              flower_print_port_range("dst_port_max",
>+                                      tb[range.max_port_type]);

The input and output of iproute2 utils, tc included should be in sync.
So you need to print "range x-y" here.


>+      }
>+
>+      if (flower_port_range_attr_type(ip_proto, FLOWER_ENDPOINT_SRC, &range)
>+          == 0) {
>+              flower_print_port_range("src_port_min",
>+                                      tb[range.min_port_type]);
>+              flower_print_port_range("src_port_max",
>+                                      tb[range.max_port_type]);
>+      }
>+
>       flower_print_tcp_flags("tcp_flags", tb[TCA_FLOWER_KEY_TCP_FLAGS],
>                              tb[TCA_FLOWER_KEY_TCP_FLAGS_MASK]);
> 
>

Reply via email to