tipc: remove active flag from tipc_bearer structure
authorYing Xue <ying.xue@windriver.com>
Thu, 27 Mar 2014 04:54:34 +0000 (12:54 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 27 Mar 2014 17:08:37 +0000 (13:08 -0400)
After the allocation of tipc_bearer structure instance is converted
from statical way to dynamical way, we identify whether a certain
tipc_bearer structure pointer is valid by checking whether the pointer
is NULL or not. So the active flag in tipc_bearer structure becomes
redundant.

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

index b4f8c62a27775c7861aa4157864f5177df777e23..47bb07aac16cf8a73bdbc47a15a3cefea27ff86b 100644 (file)
@@ -669,7 +669,7 @@ void tipc_bcbearer_sort(void)
 
        for (b_index = 0; b_index < MAX_BEARERS; b_index++) {
                struct tipc_bearer *b = bearer_list[b_index];
-               if (!b || !b->active || !b->nodes.count)
+               if (!b || !b->nodes.count)
                        continue;
 
                if (!bp_temp[b->priority].primary)
index 7ff98efd4890cd9520388222eb307d0c563e69e4..826b7010ab696278a7676a7277bfb6f50004b5af 100644 (file)
@@ -179,7 +179,7 @@ struct tipc_bearer *tipc_bearer_find(const char *name)
 
        for (i = 0; i < MAX_BEARERS; i++) {
                b_ptr = bearer_list[i];
-               if (b_ptr && b_ptr->active && (!strcmp(b_ptr->name, name)))
+               if (b_ptr && (!strcmp(b_ptr->name, name)))
                        return b_ptr;
        }
        return NULL;
@@ -204,7 +204,7 @@ struct sk_buff *tipc_bearer_get_names(void)
                        b = bearer_list[j];
                        if (!b)
                                continue;
-                       if (b->active && (b->media == media_info_array[i])) {
+                       if (b->media == media_info_array[i]) {
                                tipc_cfg_append_tlv(buf, TIPC_TLV_BEARER_NAME,
                                                    b->name,
                                                    strlen(b->name) + 1);
@@ -288,7 +288,7 @@ restart:
        with_this_prio = 1;
        for (i = MAX_BEARERS; i-- != 0; ) {
                b_ptr = bearer_list[i];
-               if (!b_ptr || !b_ptr->active) {
+               if (!b_ptr) {
                        bearer_id = i;
                        continue;
                }
@@ -333,7 +333,6 @@ restart:
        b_ptr->tolerance = m_ptr->tolerance;
        b_ptr->window = m_ptr->window;
        b_ptr->net_plane = bearer_id + 'A';
-       b_ptr->active = 1;
        b_ptr->priority = priority;
 
        res = tipc_disc_create(b_ptr, &b_ptr->bcast_addr, disc_domain);
@@ -628,7 +627,7 @@ void tipc_bearer_stop(void)
 
        for (i = 0; i < MAX_BEARERS; i++) {
                b_ptr = bearer_list[i];
-               if (b_ptr && b_ptr->active) {
+               if (b_ptr) {
                        bearer_disable(b_ptr, true);
                        bearer_list[i] = NULL;
                }
index f4e72caccb21a1f323d3f55b38fe757626d827e2..3f6d7d0f059ba1f1f4ff2891b939b88c2352adf4 100644 (file)
@@ -118,7 +118,6 @@ struct tipc_media {
  * @tolerance: default link tolerance for bearer
  * @identity: array index of this bearer within TIPC bearer array
  * @link_req: ptr to (optional) structure making periodic link setup requests
- * @active: non-zero if bearer structure is represents a bearer
  * @net_plane: network plane ('A' through 'H') currently associated with bearer
  * @nodes: indicates which nodes in cluster can be reached through bearer
  *
@@ -138,7 +137,6 @@ struct tipc_bearer {
        u32 tolerance;
        u32 identity;
        struct tipc_link_req *link_req;
-       int active;
        char net_plane;
        struct tipc_node_map nodes;
 };
index a42f4a1d3cd1d4cd2acd2251a1ab6000660cb3d1..882c5c9c52f973dbbb176d3593053bc4746e7a4b 100644 (file)
@@ -1458,10 +1458,6 @@ void tipc_rcv(struct sk_buff *head, struct tipc_bearer *b_ptr)
                head = head->next;
                buf->next = NULL;
 
-               /* Ensure bearer is still enabled */
-               if (unlikely(!b_ptr->active))
-                       goto discard;
-
                /* Ensure message is well-formed */
                if (unlikely(!link_recv_buf_validate(buf)))
                        goto discard;