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 69cca4f67e8e71a485247c9b15c4794cbff28bf5..7ca3854a2ac8c62a5ee34df16d4a3b277a20e7ee 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 2691cd57b8a8244b89857c043631445d55d031ab..ec381d444bc3651da5019407caa2f553e9bc763d 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 aefe1869572edaa275f1bb28d0895004f36c72b0..9842ec08dd5cd210bf937bbbbf4f95e8404ccd6c 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