vxlan: advertise link netns in fdb messages
authorNicolas Dichtel <nicolas.dichtel@6wind.com>
Mon, 26 Jan 2015 13:10:53 +0000 (14:10 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 28 Jan 2015 01:11:07 +0000 (17:11 -0800)
Previous commit is based on a wrong assumption, fdb messages are always sent
into the netns where the interface stands (see vxlan_fdb_notify()).

These fdb messages doesn't embed the rtnl attribute IFLA_LINK_NETNSID, thus we
need to add it (useful to interpret NDA_IFINDEX or NDA_DST for example).

Note also that vxlan_nlmsg_size() was not updated.

Fixes: 193523bf9373 ("vxlan: advertise netns of vxlan dev in fdb msg")
Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/vxlan.c
include/uapi/linux/neighbour.h

index 87736e6..31bac2a 100644 (file)
@@ -341,8 +341,8 @@ static int vxlan_fdb_info(struct sk_buff *skb, struct vxlan_dev *vxlan,
        ndm->ndm_type = RTN_UNICAST;
 
        if (!net_eq(dev_net(vxlan->dev), vxlan->net) &&
-           nla_put_s32(skb, NDA_NDM_IFINDEX_NETNSID,
-                       peernet2id(vxlan->net, dev_net(vxlan->dev))))
+           nla_put_s32(skb, NDA_LINK_NETNSID,
+                       peernet2id(dev_net(vxlan->dev), vxlan->net)))
                goto nla_put_failure;
 
        if (send_eth && nla_put(skb, NDA_LLADDR, ETH_ALEN, &fdb->eth_addr))
@@ -385,6 +385,7 @@ static inline size_t vxlan_nlmsg_size(void)
                + nla_total_size(sizeof(__be16)) /* NDA_PORT */
                + nla_total_size(sizeof(__be32)) /* NDA_VNI */
                + nla_total_size(sizeof(__u32)) /* NDA_IFINDEX */
+               + nla_total_size(sizeof(__s32)) /* NDA_LINK_NETNSID */
                + nla_total_size(sizeof(struct nda_cacheinfo));
 }
 
index 38f2368..3873a35 100644 (file)
@@ -25,7 +25,7 @@ enum {
        NDA_VNI,
        NDA_IFINDEX,
        NDA_MASTER,
-       NDA_NDM_IFINDEX_NETNSID,
+       NDA_LINK_NETNSID,
        __NDA_MAX
 };