Commit b0ab2fab authored by Jiri Pirko's avatar Jiri Pirko Committed by David S. Miller

rtnetlink: allow to register ops without ops->setup set

So far, it is assumed that ops->setup is filled up. But there might be
case that ops might make sense even without ->setup. In that case,
forbid to newlink and dellink.

This allows to register simple rtnl link ops containing only ->kind.
That allows consistent way of passing device kind (either device-kind or
slave-kind) to userspace.
Signed-off-by: default avatarJiri Pirko <jiri@resnulli.us>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 9bf2b8c2
...@@ -7091,7 +7091,7 @@ static void __net_exit default_device_exit_batch(struct list_head *net_list) ...@@ -7091,7 +7091,7 @@ static void __net_exit default_device_exit_batch(struct list_head *net_list)
rtnl_lock_unregistering(net_list); rtnl_lock_unregistering(net_list);
list_for_each_entry(net, net_list, exit_list) { list_for_each_entry(net, net_list, exit_list) {
for_each_netdev_reverse(net, dev) { for_each_netdev_reverse(net, dev) {
if (dev->rtnl_link_ops) if (dev->rtnl_link_ops && dev->rtnl_link_ops->dellink)
dev->rtnl_link_ops->dellink(dev, &dev_kill_list); dev->rtnl_link_ops->dellink(dev, &dev_kill_list);
else else
unregister_netdevice_queue(dev, &dev_kill_list); unregister_netdevice_queue(dev, &dev_kill_list);
......
...@@ -299,7 +299,12 @@ int __rtnl_link_register(struct rtnl_link_ops *ops) ...@@ -299,7 +299,12 @@ int __rtnl_link_register(struct rtnl_link_ops *ops)
if (rtnl_link_ops_get(ops->kind)) if (rtnl_link_ops_get(ops->kind))
return -EEXIST; return -EEXIST;
if (!ops->dellink) /* The check for setup is here because if ops
* does not have that filled up, it is not possible
* to use the ops for creating device. So do not
* fill up dellink as well. That disables rtnl_dellink.
*/
if (ops->setup && !ops->dellink)
ops->dellink = unregister_netdevice_queue; ops->dellink = unregister_netdevice_queue;
list_add_tail(&ops->list, &link_ops); list_add_tail(&ops->list, &link_ops);
...@@ -1777,7 +1782,7 @@ static int rtnl_dellink(struct sk_buff *skb, struct nlmsghdr *nlh) ...@@ -1777,7 +1782,7 @@ static int rtnl_dellink(struct sk_buff *skb, struct nlmsghdr *nlh)
return -ENODEV; return -ENODEV;
ops = dev->rtnl_link_ops; ops = dev->rtnl_link_ops;
if (!ops) if (!ops || !ops->dellink)
return -EOPNOTSUPP; return -EOPNOTSUPP;
ops->dellink(dev, &list_kill); ops->dellink(dev, &list_kill);
...@@ -2038,6 +2043,9 @@ static int rtnl_newlink(struct sk_buff *skb, struct nlmsghdr *nlh) ...@@ -2038,6 +2043,9 @@ static int rtnl_newlink(struct sk_buff *skb, struct nlmsghdr *nlh)
return -EOPNOTSUPP; return -EOPNOTSUPP;
} }
if (!ops->setup)
return -EOPNOTSUPP;
if (!ifname[0]) if (!ifname[0])
snprintf(ifname, IFNAMSIZ, "%s%%d", ops->kind); snprintf(ifname, IFNAMSIZ, "%s%%d", ops->kind);
......
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