On Wed, 30 Sep 2015 14:34:05 -0400, John W. Linville wrote: > +#if IS_ENABLED(CONFIG_IPV6) > +static netdev_tx_t geneve6_xmit_skb(struct sk_buff *skb, struct net_device > *dev, > + struct ip_tunnel_info *info) > +{ > + struct geneve_dev *geneve = netdev_priv(dev); > + struct geneve_sock *gs6 = geneve->sock6; > + struct dst_entry *dst = NULL; > + struct flowi6 fl6; > + __u8 ttl; > + __be16 sport; > + bool udp_csum; > + int err; > + bool xnet = !net_eq(geneve->net, dev_net(geneve->dev)); > + > + if (geneve->collect_md) { > + if (unlikely(info && info->mode != IP_TUNNEL_INFO_TX)) {
Should be !(info->mode & IP_TUNNEL_INFO_TX). The rest looks good to me. Jiri -- Jiri Benc -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html