Commit c8632fc3 authored by Petr Machata's avatar Petr Machata Committed by David S. Miller

net: ip6_gre: Split up ip6gre_changelink()

Extract from ip6gre_changelink() a reusable function
ip6gre_changelink_common(). This will allow introduction of
ERSPAN-specific _changelink() function with not a lot of code
duplication.

Fixes: 5a963eb6 ("ip6_gre: Add ERSPAN native tunnel support")
Signed-off-by: default avatarPetr Machata <petrm@mellanox.com>
Acked-by: default avatarWilliam Tu <u9012063@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 7fa38a7c
...@@ -1924,37 +1924,52 @@ static int ip6gre_newlink(struct net *src_net, struct net_device *dev, ...@@ -1924,37 +1924,52 @@ static int ip6gre_newlink(struct net *src_net, struct net_device *dev,
return err; return err;
} }
static int ip6gre_changelink(struct net_device *dev, struct nlattr *tb[], static struct ip6_tnl *
struct nlattr *data[], ip6gre_changelink_common(struct net_device *dev, struct nlattr *tb[],
struct netlink_ext_ack *extack) struct nlattr *data[], struct __ip6_tnl_parm *p_p,
struct netlink_ext_ack *extack)
{ {
struct ip6_tnl *t, *nt = netdev_priv(dev); struct ip6_tnl *t, *nt = netdev_priv(dev);
struct net *net = nt->net; struct net *net = nt->net;
struct ip6gre_net *ign = net_generic(net, ip6gre_net_id); struct ip6gre_net *ign = net_generic(net, ip6gre_net_id);
struct __ip6_tnl_parm p;
struct ip_tunnel_encap ipencap; struct ip_tunnel_encap ipencap;
if (dev == ign->fb_tunnel_dev) if (dev == ign->fb_tunnel_dev)
return -EINVAL; return ERR_PTR(-EINVAL);
if (ip6gre_netlink_encap_parms(data, &ipencap)) { if (ip6gre_netlink_encap_parms(data, &ipencap)) {
int err = ip6_tnl_encap_setup(nt, &ipencap); int err = ip6_tnl_encap_setup(nt, &ipencap);
if (err < 0) if (err < 0)
return err; return ERR_PTR(err);
} }
ip6gre_netlink_parms(data, &p); ip6gre_netlink_parms(data, p_p);
t = ip6gre_tunnel_locate(net, &p, 0); t = ip6gre_tunnel_locate(net, p_p, 0);
if (t) { if (t) {
if (t->dev != dev) if (t->dev != dev)
return -EEXIST; return ERR_PTR(-EEXIST);
} else { } else {
t = nt; t = nt;
} }
return t;
}
static int ip6gre_changelink(struct net_device *dev, struct nlattr *tb[],
struct nlattr *data[],
struct netlink_ext_ack *extack)
{
struct ip6gre_net *ign = net_generic(dev_net(dev), ip6gre_net_id);
struct __ip6_tnl_parm p;
struct ip6_tnl *t;
t = ip6gre_changelink_common(dev, tb, data, &p, extack);
if (IS_ERR(t))
return PTR_ERR(t);
ip6gre_tunnel_unlink(ign, t); ip6gre_tunnel_unlink(ign, t);
ip6gre_tnl_change(t, &p, !tb[IFLA_MTU]); ip6gre_tnl_change(t, &p, !tb[IFLA_MTU]);
ip6gre_tunnel_link(ign, t); ip6gre_tunnel_link(ign, t);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment