tipc: call tsk_set_importance from tipc_topsrv_create_listener

Avoid using kernel_setsockopt for the TIPC_IMPORTANCE option when we can
just use the internal helper.  The only change needed is to pass a struct
sock instead of tipc_sock, which is private to socket.c

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/net/tipc/socket.c b/net/tipc/socket.c
index d6b67d07d22ecfeb05d552a736604a94ede03d77..3734cdbedc9cca8c7e85629e2ac910e620d93d70 100644 (file)
--- a/net/tipc/socket.c
+++ b/net/tipc/socket.c
@@ -196,17 +196,17 @@ static int tsk_importance(struct tipc_sock *tsk)
        return msg_importance(&tsk->phdr);
 }
 
-static int tsk_set_importance(struct tipc_sock *tsk, int imp)
+static struct tipc_sock *tipc_sk(const struct sock *sk)
 {
-       if (imp > TIPC_CRITICAL_IMPORTANCE)
-               return -EINVAL;
-       msg_set_importance(&tsk->phdr, (u32)imp);
-       return 0;
+       return container_of(sk, struct tipc_sock, sk);
 }
 
-static struct tipc_sock *tipc_sk(const struct sock *sk)
+int tsk_set_importance(struct sock *sk, int imp)
 {
-       return container_of(sk, struct tipc_sock, sk);
+       if (imp > TIPC_CRITICAL_IMPORTANCE)
+               return -EINVAL;
+       msg_set_importance(&tipc_sk(sk)->phdr, (u32)imp);
+       return 0;
 }
 
 static bool tsk_conn_cong(struct tipc_sock *tsk)
@@ -2721,7 +2721,7 @@ static int tipc_accept(struct socket *sock, struct socket *new_sock, int flags,
        /* Connect new socket to it's peer */
        tipc_sk_finish_conn(new_tsock, msg_origport(msg), msg_orignode(msg));
 
-       tsk_set_importance(new_tsock, msg_importance(msg));
+       tsk_set_importance(new_sk, msg_importance(msg));
        if (msg_named(msg)) {
                new_tsock->conn_type = msg_nametype(msg);
                new_tsock->conn_instance = msg_nameinst(msg);
@@ -3139,7 +3139,7 @@ static int tipc_setsockopt(struct socket *sock, int lvl, int opt,
 
        switch (opt) {
        case TIPC_IMPORTANCE:
-               res = tsk_set_importance(tsk, value);
+               res = tsk_set_importance(sk, value);
                break;
        case TIPC_SRC_DROPPABLE:
                if (sock->type != SOCK_STREAM)

diff --git a/net/tipc/socket.h b/net/tipc/socket.h
index 235b9679acee4415455be63c775f6f6587331b1a..b11575afc66fe645003c6ca2475c4e3b874355de 100644 (file)
--- a/net/tipc/socket.h
+++ b/net/tipc/socket.h
@@ -75,4 +75,6 @@ u32 tipc_sock_get_portid(struct sock *sk);
 bool tipc_sk_overlimit1(struct sock *sk, struct sk_buff *skb);
 bool tipc_sk_overlimit2(struct sock *sk, struct sk_buff *skb);
 
+int tsk_set_importance(struct sock *sk, int imp);
+
 #endif

diff --git a/net/tipc/topsrv.c b/net/tipc/topsrv.c
index 446af7bbd13e68ccc225fca69fbd40dc75c30d1c..1489cfb941d8ef0d83cbc567fcf1e989c3b58e21 100644 (file)
--- a/net/tipc/topsrv.c
+++ b/net/tipc/topsrv.c
@@ -497,7 +497,6 @@ static void tipc_topsrv_listener_data_ready(struct sock *sk)
 
 static int tipc_topsrv_create_listener(struct tipc_topsrv *srv)
 {
-       int imp = TIPC_CRITICAL_IMPORTANCE;
        struct socket *lsock = NULL;
        struct sockaddr_tipc saddr;
        struct sock *sk;
@@ -514,8 +513,9 @@ static int tipc_topsrv_create_listener(struct tipc_topsrv *srv)
        sk->sk_user_data = srv;
        write_unlock_bh(&sk->sk_callback_lock);
 
-       rc = kernel_setsockopt(lsock, SOL_TIPC, TIPC_IMPORTANCE,
-                              (char *)&imp, sizeof(imp));
+       lock_sock(sk);
+       rc = tsk_set_importance(sk, TIPC_CRITICAL_IMPORTANCE);
+       release_sock(sk);
        if (rc < 0)
                goto err;