tipc: eliminate functions tipc_port_init and tipc_port_destroy
authorJon Paul Maloy <jon.maloy@ericsson.com>
Fri, 22 Aug 2014 22:09:13 +0000 (18:09 -0400)
committerDavid S. Miller <davem@davemloft.net>
Sat, 23 Aug 2014 18:18:34 +0000 (11:18 -0700)
After the latest changes to the socket/port layer the existence of
the functions tipc_port_init() and tipc_port_destroy() cannot be
justified. They are both called only once, from tipc_sk_create() and
tipc_sk_delete() respectively, and their functionality can better be
merged into the latter two functions.

This also entails that all remaining references to port_lock now are
made from inside socket.c, something that will make it easier to remove
this lock.

Signed-off-by: Jon Maloy <jon.maloy@ericsson.com>
Reviewed-by: Erik Hugne <erik.hugne@ericsson.com>
Reviewed-by: Ying Xue <ying.xue@windriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/tipc/port.c
net/tipc/port.h
net/tipc/socket.c

index 1074ccb0c76f1b42889ccf9beacd9a25c545a1a8..1efa2982d2d2f2b52cc289a7b10758a55d04cfd0 100644 (file)
 
 #define MAX_REJECT_SIZE 1024
 
-DEFINE_SPINLOCK(tipc_port_list_lock);
-
-static LIST_HEAD(ports);
-
 /**
  * tipc_port_peer_msg - verify message was sent by connected port's peer
  *
@@ -67,73 +63,6 @@ int tipc_port_peer_msg(struct tipc_port *p_ptr, struct tipc_msg *msg)
                (!peernode && (orignode == tipc_own_addr));
 }
 
-/* tipc_port_init - intiate TIPC port and lock it
- *
- * Returns obtained reference if initialization is successful, zero otherwise
- */
-u32 tipc_port_init(struct tipc_port *p_ptr,
-                  const unsigned int importance)
-{
-       struct tipc_msg *msg;
-       u32 ref;
-
-       ref = tipc_ref_acquire(p_ptr, &p_ptr->lock);
-       if (!ref) {
-               pr_warn("Port registration failed, ref. table exhausted\n");
-               return 0;
-       }
-
-       p_ptr->max_pkt = MAX_PKT_DEFAULT;
-       p_ptr->ref = ref;
-       INIT_LIST_HEAD(&p_ptr->subscription.nodesub_list);
-       INIT_LIST_HEAD(&p_ptr->publications);
-       INIT_LIST_HEAD(&p_ptr->port_list);
-
-       /*
-        * Must hold port list lock while initializing message header template
-        * to ensure a change to node's own network address doesn't result
-        * in template containing out-dated network address information
-        */
-       spin_lock_bh(&tipc_port_list_lock);
-       msg = &p_ptr->phdr;
-       tipc_msg_init(msg, importance, TIPC_NAMED_MSG, NAMED_H_SIZE, 0);
-       msg_set_origport(msg, ref);
-       list_add_tail(&p_ptr->port_list, &ports);
-       spin_unlock_bh(&tipc_port_list_lock);
-       return ref;
-}
-
-void tipc_port_destroy(struct tipc_port *p_ptr)
-{
-       struct sk_buff *buf = NULL;
-       struct tipc_msg *msg = NULL;
-       u32 peer_node;
-
-       tipc_withdraw(p_ptr, 0, NULL);
-
-       spin_lock_bh(p_ptr->lock);
-       tipc_ref_discard(p_ptr->ref);
-       spin_unlock_bh(p_ptr->lock);
-
-       k_cancel_timer(&p_ptr->timer);
-       if (p_ptr->connected) {
-               peer_node = tipc_port_peernode(p_ptr);
-               buf = tipc_msg_create(TIPC_CRITICAL_IMPORTANCE, TIPC_CONN_MSG,
-                                     SHORT_H_SIZE, 0, peer_node,
-                                     tipc_own_addr, tipc_port_peerport(p_ptr),
-                                     p_ptr->ref, TIPC_ERR_NO_PORT);
-               if (buf) {
-                       msg = buf_msg(buf);
-                       tipc_link_xmit(buf, peer_node, msg_link_selector(msg));
-               }
-               tipc_node_remove_conn(peer_node, p_ptr->ref);
-       }
-       spin_lock_bh(&tipc_port_list_lock);
-       list_del(&p_ptr->port_list);
-       spin_unlock_bh(&tipc_port_list_lock);
-       k_term_timer(&p_ptr->timer);
-}
-
 static int port_print(struct tipc_port *p_ptr, char *buf, int len, int full_id)
 {
        struct publication *publ;
@@ -194,7 +123,7 @@ struct sk_buff *tipc_port_get_ports(void)
        pb_len = ULTRA_STRING_MAX_LEN;
 
        spin_lock_bh(&tipc_port_list_lock);
-       list_for_each_entry(p_ptr, &ports, port_list) {
+       list_for_each_entry(p_ptr, &tipc_socks, port_list) {
                spin_lock_bh(p_ptr->lock);
                str_len += port_print(p_ptr, pb, pb_len, 0);
                spin_unlock_bh(p_ptr->lock);
@@ -213,7 +142,7 @@ void tipc_port_reinit(void)
        struct tipc_msg *msg;
 
        spin_lock_bh(&tipc_port_list_lock);
-       list_for_each_entry(p_ptr, &ports, port_list) {
+       list_for_each_entry(p_ptr, &tipc_socks, port_list) {
                msg = &p_ptr->phdr;
                msg_set_prevnode(msg, tipc_own_addr);
                msg_set_orignode(msg, tipc_own_addr);
index c92e1721f672e5a756b780d222a0525837d1715c..4c6cfdd363516236b6bc3dbae8478f5a1aa1bda1 100644 (file)
@@ -63,7 +63,6 @@
  * @probing_state:
  * @probing_interval:
  * @timer_ref:
- * @subscription: "node down" subscription used to terminate failed connections
  */
 struct tipc_port {
        spinlock_t *lock;
@@ -80,11 +79,10 @@ struct tipc_port {
        u32 probing_state;
        u32 probing_interval;
        struct timer_list timer;
-       struct tipc_node_subscr subscription;
 };
 
+extern struct list_head tipc_socks;
 extern spinlock_t tipc_port_list_lock;
-struct tipc_port_list;
 
 /*
  * TIPC port manipulation routines
index 686a11be5c588ab0126a29acc6c596d5851ca4a9..f2be4c2e20bbb5dec4a25fe1e0d7894a1770beb3 100644 (file)
@@ -63,6 +63,9 @@ static const struct proto_ops msg_ops;
 static struct proto tipc_proto;
 static struct proto tipc_proto_kern;
 
+DEFINE_SPINLOCK(tipc_port_list_lock);
+LIST_HEAD(tipc_socks);
+
 /*
  * Revised TIPC socket locking policy:
  *
@@ -156,6 +159,7 @@ static int tipc_sk_create(struct net *net, struct socket *sock,
        struct sock *sk;
        struct tipc_sock *tsk;
        struct tipc_port *port;
+       struct tipc_msg *msg;
        u32 ref;
 
        /* Validate arguments */
@@ -191,18 +195,28 @@ static int tipc_sk_create(struct net *net, struct socket *sock,
 
        tsk = tipc_sk(sk);
        port = &tsk->port;
-
-       ref = tipc_port_init(port, TIPC_LOW_IMPORTANCE);
+       ref = tipc_ref_acquire(port, &port->lock);
        if (!ref) {
-               pr_warn("Socket registration failed, ref. table exhausted\n");
-               sk_free(sk);
+               pr_warn("Socket create failed; reference table exhausted\n");
                return -ENOMEM;
        }
+       port->max_pkt = MAX_PKT_DEFAULT;
+       port->ref = ref;
+       INIT_LIST_HEAD(&port->publications);
+       INIT_LIST_HEAD(&port->port_list);
+
+       /* Guard against race during node address update */
+       spin_lock_bh(&tipc_port_list_lock);
+       msg = &port->phdr;
+       tipc_msg_init(msg, TIPC_LOW_IMPORTANCE, TIPC_NAMED_MSG,
+                     NAMED_H_SIZE, 0);
+       msg_set_origport(msg, ref);
+       list_add_tail(&port->port_list, &tipc_socks);
+       spin_unlock_bh(&tipc_port_list_lock);
 
        /* Finish initializing socket data structures */
        sock->ops = ops;
        sock->state = state;
-
        sock_init_data(sock, sk);
        k_init_timer(&port->timer, (Handler)tipc_sk_timeout, ref);
        sk->sk_backlog_rcv = tipc_backlog_rcv;
@@ -330,6 +344,7 @@ static int tipc_release(struct socket *sock)
         * Reject all unreceived messages, except on an active connection
         * (which disconnects locally & sends a 'FIN+' to peer)
         */
+       dnode = tipc_port_peernode(port);
        while (sock->state != SS_DISCONNECTING) {
                buf = __skb_dequeue(&sk->sk_receive_queue);
                if (buf == NULL)
@@ -341,18 +356,31 @@ static int tipc_release(struct socket *sock)
                            (sock->state == SS_CONNECTED)) {
                                sock->state = SS_DISCONNECTING;
                                port->connected = 0;
-                               tipc_node_remove_conn(tipc_port_peernode(port),
-                                                     port->ref);
+                               tipc_node_remove_conn(dnode, port->ref);
                        }
                        if (tipc_msg_reverse(buf, &dnode, TIPC_ERR_NO_PORT))
                                tipc_link_xmit(buf, dnode, 0);
                }
        }
 
-       /* Destroy TIPC port; also disconnects an active connection and
-        * sends a 'FIN-' to peer.
-        */
-       tipc_port_destroy(port);
+       tipc_withdraw(port, 0, NULL);
+       spin_lock_bh(port->lock);
+       tipc_ref_discard(port->ref);
+       spin_unlock_bh(port->lock);
+       k_cancel_timer(&port->timer);
+       if (port->connected) {
+               buf = tipc_msg_create(TIPC_CRITICAL_IMPORTANCE, TIPC_CONN_MSG,
+                                     SHORT_H_SIZE, 0, dnode, tipc_own_addr,
+                                     tipc_port_peerport(port),
+                                     port->ref, TIPC_ERR_NO_PORT);
+               if (buf)
+                       tipc_link_xmit(buf, dnode, port->ref);
+               tipc_node_remove_conn(dnode, port->ref);
+       }
+       spin_lock_bh(&tipc_port_list_lock);
+       list_del(&port->port_list);
+       spin_unlock_bh(&tipc_port_list_lock);
+       k_term_timer(&port->timer);
 
        /* Discard any remaining (connection-based) messages in receive queue */
        __skb_queue_purge(&sk->sk_receive_queue);
@@ -360,7 +388,6 @@ static int tipc_release(struct socket *sock)
        /* Reject any messages that accumulated in backlog queue */
        sock->state = SS_DISCONNECTING;
        release_sock(sk);
-
        sock_put(sk);
        sock->sk = NULL;