tipc: make tipc random value aware of net namespace
authorYing Xue <ying.xue@windriver.com>
Fri, 9 Jan 2015 07:27:12 +0000 (15:27 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 12 Jan 2015 21:24:33 +0000 (16:24 -0500)
After namespace is supported, each namespace should own its private
random value. So the global variable representing the random value
must be moved to tipc_net structure.

Signed-off-by: Ying Xue <ying.xue@windriver.com>
Tested-by: Tero Aho <Tero.Aho@coriant.com>
Reviewed-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/tipc/core.c
net/tipc/core.h
net/tipc/discover.c
net/tipc/link.c

index 4a8b7955e0e0d4689eee3c4cbac1142698594d32..674bd26985289f5f5cda583a2244456bd520e5c3 100644 (file)
@@ -44,9 +44,6 @@
 
 #include <linux/module.h>
 
-/* global variables used by multiple sub-systems within TIPC */
-int tipc_random __read_mostly;
-
 /* configurable TIPC parameters */
 int tipc_net_id __read_mostly;
 int sysctl_tipc_rmem[3] __read_mostly; /* min/default/max */
@@ -58,6 +55,7 @@ static int __net_init tipc_init_net(struct net *net)
 
        tn->net_id = 4711;
        tn->own_addr = 0;
+       get_random_bytes(&tn->random, sizeof(int));
        INIT_LIST_HEAD(&tn->node_list);
        spin_lock_init(&tn->node_list_lock);
 
@@ -109,8 +107,6 @@ static int __init tipc_init(void)
                              TIPC_CRITICAL_IMPORTANCE;
        sysctl_tipc_rmem[2] = TIPC_CONN_OVERLOAD_LIMIT;
 
-       get_random_bytes(&tipc_random, sizeof(tipc_random));
-
        err = tipc_netlink_start();
        if (err)
                goto out_netlink;
index 639f562dddf36c2b037b42d9eb2b596c6743025e..817b2e9d42279185721300079ecb38228e478cd7 100644 (file)
@@ -76,14 +76,10 @@ extern int tipc_net_id __read_mostly;
 extern int sysctl_tipc_rmem[3] __read_mostly;
 extern int sysctl_tipc_named_timeout __read_mostly;
 
-/*
- * Other global variables
- */
-extern int tipc_random __read_mostly;
-
 struct tipc_net {
        u32 own_addr;
        int net_id;
+       int random;
 
        /* Node table and node list */
        spinlock_t node_list_lock;
index 786411dea61cd56796183db419e1cae9fd9a147e..5b40cb89ff0aaa5492ba5c51b38df6bdc3bc85a8 100644 (file)
@@ -88,7 +88,7 @@ static void tipc_disc_init_msg(struct net *net, struct sk_buff *buf, u32 type,
        msg = buf_msg(buf);
        tipc_msg_init(net, msg, LINK_CONFIG, type, INT_H_SIZE, dest_domain);
        msg_set_non_seq(msg, 1);
-       msg_set_node_sig(msg, tipc_random);
+       msg_set_node_sig(msg, tn->random);
        msg_set_dest_domain(msg, dest_domain);
        msg_set_bc_netid(msg, tn->net_id);
        b_ptr->media->addr2msg(msg_media_addr(msg), &b_ptr->addr);
index 997256769065c7035d9ea69a49f3d07376b423c1..193bc1560677a4237fdc9304f3e4ac38f3f7b1b3 100644 (file)
@@ -289,7 +289,7 @@ struct tipc_link *tipc_link_create(struct tipc_node *n_ptr,
        tipc_msg_init(n_ptr->net, msg, LINK_PROTOCOL, RESET_MSG, INT_H_SIZE,
                      l_ptr->addr);
        msg_set_size(msg, sizeof(l_ptr->proto_msg));
-       msg_set_session(msg, (tipc_random & 0xffff));
+       msg_set_session(msg, (tn->random & 0xffff));
        msg_set_bearer_id(msg, b_ptr->identity);
        strcpy((char *)msg_data(msg), if_name);