tipc: Eliminate obsolete support for "not running" mode
authorAllan Stephens <allan.stephens@windriver.com>
Tue, 8 Nov 2011 18:18:59 +0000 (13:18 -0500)
committerPaul Gortmaker <paul.gortmaker@windriver.com>
Fri, 24 Feb 2012 22:05:17 +0000 (17:05 -0500)
Removes all references to TIPC's "not running" mode, since the
removal of support for the native API means that there is no longer
any way to interact with TIPC if it has not been initialized.

The changes made consist of removing mode-based checks that are no
longer needed, along with any associated code lying on non-executable
control paths.

Signed-off-by: Allan Stephens <allan.stephens@windriver.com>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
net/tipc/config.c
net/tipc/core.c
net/tipc/core.h

index 69cca4f..7ca3854 100644 (file)
@@ -252,11 +252,8 @@ static struct sk_buff *cfg_set_max_ports(void)
        if (value < 127 || value > 65535)
                return tipc_cfg_reply_error_string(TIPC_CFG_INVALID_VALUE
                                                   " (max ports must be 127-65535)");
-       if (tipc_mode != TIPC_NOT_RUNNING)
-               return tipc_cfg_reply_error_string(TIPC_CFG_NOT_SUPPORTED
-                       " (cannot change max ports while TIPC is active)");
-       tipc_max_ports = value;
-       return tipc_cfg_reply_none();
+       return tipc_cfg_reply_error_string(TIPC_CFG_NOT_SUPPORTED
+               " (cannot change max ports while TIPC is active)");
 }
 
 static struct sk_buff *cfg_set_netid(void)
index 2691cd5..ec381d4 100644 (file)
@@ -53,7 +53,7 @@
 
 /* global variables used by multiple sub-systems within TIPC */
 
-int tipc_mode = TIPC_NOT_RUNNING;
+int tipc_mode;
 int tipc_random;
 
 const char tipc_alphabet[] =
@@ -125,11 +125,6 @@ int tipc_core_start_net(unsigned long addr)
 
 static void tipc_core_stop(void)
 {
-       if (tipc_mode != TIPC_NODE_MODE)
-               return;
-
-       tipc_mode = TIPC_NOT_RUNNING;
-
        tipc_netlink_stop();
        tipc_handler_stop();
        tipc_cfg_stop();
@@ -148,9 +143,6 @@ static int tipc_core_start(void)
 {
        int res;
 
-       if (tipc_mode != TIPC_NOT_RUNNING)
-               return -ENOPROTOOPT;
-
        get_random_bytes(&tipc_random, sizeof(tipc_random));
        tipc_mode = TIPC_NODE_MODE;
 
index aefe186..9842ec0 100644 (file)
@@ -132,7 +132,7 @@ void tipc_msg_dbg(struct print_buf *, struct tipc_msg *, const char *);
 /*
  * TIPC operating mode routines
  */
-#define TIPC_NOT_RUNNING  0
+
 #define TIPC_NODE_MODE    1
 #define TIPC_NET_MODE     2