tipc: convert legacy nl link prop set to nl compat

Convert setting of link proprieties to compat doit calls.

Commands converted in this patch:
TIPC_CMD_SET_LINK_TOL
TIPC_CMD_SET_LINK_PRI
TIPC_CMD_SET_LINK_WINDOW

Signed-off-by: Richard Alpe <richard.alpe@ericsson.com>
Reviewed-by: Erik Hugne <erik.hugne@ericsson.com>
Reviewed-by: Ying Xue <ying.xue@windriver.com>
Reviewed-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 1 insertions, 145 deletions

diff --git a/net/tipc/link.c b/net/tipc/link.c
index 2622fb99344a..ec136b5cc339 100644
--- a/net/tipc/link.c
+++ b/net/tipc/link.c
@@ -1959,150 +1959,6 @@ static struct tipc_node *tipc_link_find_owner(struct net *net,
 }
 
 /**
- * link_value_is_valid -- validate proposed link tolerance/priority/window
- *
- * @cmd: value type (TIPC_CMD_SET_LINK_*)
- * @new_value: the new value
- *
- * Returns 1 if value is within range, 0 if not.
- */
-static int link_value_is_valid(u16 cmd, u32 new_value)
-{
-        switch (cmd) {
-        case TIPC_CMD_SET_LINK_TOL:
-                return (new_value >= TIPC_MIN_LINK_TOL) &&
-                        (new_value <= TIPC_MAX_LINK_TOL);
-        case TIPC_CMD_SET_LINK_PRI:
-                return (new_value <= TIPC_MAX_LINK_PRI);
-        case TIPC_CMD_SET_LINK_WINDOW:
-                return (new_value >= TIPC_MIN_LINK_WIN) &&
-                        (new_value <= TIPC_MAX_LINK_WIN);
-        }
-        return 0;
-}
-
-/**
- * link_cmd_set_value - change priority/tolerance/window for link/bearer/media
- * @net: the applicable net namespace
- * @name: ptr to link, bearer, or media name
- * @new_value: new value of link, bearer, or media setting
- * @cmd: which link, bearer, or media attribute to set (TIPC_CMD_SET_LINK_*)
- *
- * Caller must hold RTNL lock to ensure link/bearer/media is not deleted.
- *
- * Returns 0 if value updated and negative value on error.
- */
-static int link_cmd_set_value(struct net *net, const char *name, u32 new_value,
-                              u16 cmd)
-{
-        struct tipc_node *node;
-        struct tipc_link *l_ptr;
-        struct tipc_bearer *b_ptr;
-        struct tipc_media *m_ptr;
-        int bearer_id;
-        int res = 0;
-
-        node = tipc_link_find_owner(net, name, &bearer_id);
-        if (node) {
-                tipc_node_lock(node);
-                l_ptr = node->links[bearer_id];
-
-                if (l_ptr) {
-                        switch (cmd) {
-                        case TIPC_CMD_SET_LINK_TOL:
-                                link_set_supervision_props(l_ptr, new_value);
-                                tipc_link_proto_xmit(l_ptr, STATE_MSG, 0, 0,
-                                                     new_value, 0, 0);
-                                break;
-                        case TIPC_CMD_SET_LINK_PRI:
-                                l_ptr->priority = new_value;
-                                tipc_link_proto_xmit(l_ptr, STATE_MSG, 0, 0,
-                                                     0, new_value, 0);
-                                break;
-                        case TIPC_CMD_SET_LINK_WINDOW:
-                                tipc_link_set_queue_limits(l_ptr, new_value);
-                                break;
-                        default:
-                                res = -EINVAL;
-                                break;
-                        }
-                }
-                tipc_node_unlock(node);
-                return res;
-        }
-
-        b_ptr = tipc_bearer_find(net, name);
-        if (b_ptr) {
-                switch (cmd) {
-                case TIPC_CMD_SET_LINK_TOL:
-                        b_ptr->tolerance = new_value;
-                        break;
-                case TIPC_CMD_SET_LINK_PRI:
-                        b_ptr->priority = new_value;
-                        break;
-                case TIPC_CMD_SET_LINK_WINDOW:
-                        b_ptr->window = new_value;
-                        break;
-                default:
-                        res = -EINVAL;
-                        break;
-                }
-                return res;
-        }
-
-        m_ptr = tipc_media_find(name);
-        if (!m_ptr)
-                return -ENODEV;
-        switch (cmd) {
-        case TIPC_CMD_SET_LINK_TOL:
-                m_ptr->tolerance = new_value;
-                break;
-        case TIPC_CMD_SET_LINK_PRI:
-                m_ptr->priority = new_value;
-                break;
-        case TIPC_CMD_SET_LINK_WINDOW:
-                m_ptr->window = new_value;
-                break;
-        default:
-                res = -EINVAL;
-                break;
-        }
-        return res;
-}
-
-struct sk_buff *tipc_link_cmd_config(struct net *net, const void *req_tlv_area,
-                                     int req_tlv_space, u16 cmd)
-{
-        struct tipc_link_config *args;
-        u32 new_value;
-        int res;
-
-        if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_LINK_CONFIG))
-                return tipc_cfg_reply_error_string(TIPC_CFG_TLV_ERROR);
-
-        args = (struct tipc_link_config *)TLV_DATA(req_tlv_area);
-        new_value = ntohl(args->value);
-
-        if (!link_value_is_valid(cmd, new_value))
-                return tipc_cfg_reply_error_string(
-                        "cannot change, value invalid");
-
-        if (!strcmp(args->name, tipc_bclink_name)) {
-                if ((cmd == TIPC_CMD_SET_LINK_WINDOW) &&
-                    (tipc_bclink_set_queue_limits(net, new_value) == 0))
-                        return tipc_cfg_reply_none();
-                return tipc_cfg_reply_error_string(TIPC_CFG_NOT_SUPPORTED
-                                                   " (cannot change setting on broadcast link)");
-        }
-
-        res = link_cmd_set_value(net, args->name, new_value, cmd);
-        if (res)
-                return tipc_cfg_reply_error_string("cannot change link setting");
-
-        return tipc_cfg_reply_none();
-}
-
-/**
  * link_reset_statistics - reset link statistics
  * @l_ptr: pointer to link
  */
@@ -2216,7 +2072,7 @@ int tipc_nl_link_set(struct sk_buff *skb, struct genl_info *info)
         struct tipc_link *link;
         struct tipc_node *node;
         struct nlattr *attrs[TIPC_NLA_LINK_MAX + 1];
-        struct net *net = genl_info_net(info);
+        struct net *net = sock_net(skb->sk);
 
         if (!info->attrs[TIPC_NLA_LINK])
                 return -EINVAL;