On Fri, Aug 8, 2014 at 1:28 PM, Jarno Rajahalme <jrajaha...@nicira.com> wrote:
> diff --git a/datapath/flow_netlink.c b/datapath/flow_netlink.c
> index e4cf535..294e54c 100644
> --- a/datapath/flow_netlink.c
> +++ b/datapath/flow_netlink.c
> @@ -688,6 +688,11 @@ static int ovs_key_from_nlattrs(struct sw_flow_match
> *match, u64 attrs,
> ipv6_key->ipv6_frag, OVS_FRAG_TYPE_MAX);
> return -EINVAL;
> }
> + if (ntohl(ipv6_key->ipv6_label) & 0xFFF00000) {
It's marginally better if we put a htonl on the constant instead,
since the compiler can do it for us.
Otherwise, this looks fine:
Acked-by: Jesse Gross <je...@nicira.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
