ip_gre: add the support for i/o_flags update via ioctl
authorXin Long <lucien.xin@gmail.com>
Tue, 7 Nov 2017 08:33:09 +0000 (16:33 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 10 Nov 2017 05:39:53 +0000 (14:39 +0900)
As patch 'ip_gre: add the support for i/o_flags update via netlink'
did for netlink, we also need to do the same job for these update
via ioctl.

This patch is to update i/o_flags and call ipgre_link_update to
recalculate these gre properties after ip_tunnel_ioctl does the
common update.

Signed-off-by: Xin Long <lucien.xin@gmail.com>
Acked-by: William Tu <u9012063@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/ip_gre.c

index 81e1e20af33bb9d07ec8c5ae55395ba9463d821b..bb6239169b1ab943a6418494dab5018843bcde3c 100644 (file)
@@ -800,17 +800,19 @@ static void ipgre_link_update(struct net_device *dev, bool set_mtu)
 static int ipgre_tunnel_ioctl(struct net_device *dev,
                              struct ifreq *ifr, int cmd)
 {
-       int err;
        struct ip_tunnel_parm p;
+       int err;
 
        if (copy_from_user(&p, ifr->ifr_ifru.ifru_data, sizeof(p)))
                return -EFAULT;
+
        if (cmd == SIOCADDTUNNEL || cmd == SIOCCHGTUNNEL) {
                if (p.iph.version != 4 || p.iph.protocol != IPPROTO_GRE ||
-                   p.iph.ihl != 5 || (p.iph.frag_off&htons(~IP_DF)) ||
-                   ((p.i_flags|p.o_flags)&(GRE_VERSION|GRE_ROUTING)))
+                   p.iph.ihl != 5 || (p.iph.frag_off & htons(~IP_DF)) ||
+                   ((p.i_flags | p.o_flags) & (GRE_VERSION | GRE_ROUTING)))
                        return -EINVAL;
        }
+
        p.i_flags = gre_flags_to_tnl_flags(p.i_flags);
        p.o_flags = gre_flags_to_tnl_flags(p.o_flags);
 
@@ -818,11 +820,22 @@ static int ipgre_tunnel_ioctl(struct net_device *dev,
        if (err)
                return err;
 
+       if (cmd == SIOCCHGTUNNEL) {
+               struct ip_tunnel *t = netdev_priv(dev);
+
+               t->parms.i_flags = p.i_flags;
+               t->parms.o_flags = p.o_flags;
+
+               if (strcmp(dev->rtnl_link_ops->kind, "erspan"))
+                       ipgre_link_update(dev, true);
+       }
+
        p.i_flags = gre_tnl_flags_to_gre_flags(p.i_flags);
        p.o_flags = gre_tnl_flags_to_gre_flags(p.o_flags);
 
        if (copy_to_user(ifr->ifr_ifru.ifru_data, &p, sizeof(p)))
                return -EFAULT;
+
        return 0;
 }