diff options
Diffstat (limited to 'net/tipc')
| -rw-r--r-- | net/tipc/bcast.c | 161 | ||||
| -rw-r--r-- | net/tipc/bcast.h | 16 | ||||
| -rw-r--r-- | net/tipc/bearer.c | 190 | ||||
| -rw-r--r-- | net/tipc/bearer.h | 77 | ||||
| -rw-r--r-- | net/tipc/config.c | 13 | ||||
| -rw-r--r-- | net/tipc/core.c | 2 | ||||
| -rw-r--r-- | net/tipc/discover.c | 27 | ||||
| -rw-r--r-- | net/tipc/discover.h | 8 | ||||
| -rw-r--r-- | net/tipc/eth_media.c | 159 | ||||
| -rw-r--r-- | net/tipc/link.c | 346 | ||||
| -rw-r--r-- | net/tipc/link.h | 63 | ||||
| -rw-r--r-- | net/tipc/msg.c | 9 | ||||
| -rw-r--r-- | net/tipc/msg.h | 16 | ||||
| -rw-r--r-- | net/tipc/name_distr.c | 14 | ||||
| -rw-r--r-- | net/tipc/name_table.c | 15 | ||||
| -rw-r--r-- | net/tipc/name_table.h | 10 | ||||
| -rw-r--r-- | net/tipc/net.c | 7 | ||||
| -rw-r--r-- | net/tipc/node.c | 25 | ||||
| -rw-r--r-- | net/tipc/node.h | 12 | ||||
| -rw-r--r-- | net/tipc/port.c | 8 | ||||
| -rw-r--r-- | net/tipc/port.h | 4 | ||||
| -rw-r--r-- | net/tipc/ref.c | 3 | ||||
| -rw-r--r-- | net/tipc/socket.c | 3 | ||||
| -rw-r--r-- | net/tipc/subscr.c | 46 | ||||
| -rw-r--r-- | net/tipc/subscr.h | 10 | 
25 files changed, 699 insertions, 545 deletions
diff --git a/net/tipc/bcast.c b/net/tipc/bcast.c index 28908f54459..8eb87b11d10 100644 --- a/net/tipc/bcast.c +++ b/net/tipc/bcast.c @@ -46,7 +46,7 @@  #define BCLINK_WIN_DEFAULT 20		/* bcast link window size (default) */  /** - * struct bcbearer_pair - a pair of bearers used by broadcast link + * struct tipc_bcbearer_pair - a pair of bearers used by broadcast link   * @primary: pointer to primary bearer   * @secondary: pointer to secondary bearer   * @@ -54,13 +54,13 @@   * to be paired.   */ -struct bcbearer_pair { +struct tipc_bcbearer_pair {  	struct tipc_bearer *primary;  	struct tipc_bearer *secondary;  };  /** - * struct bcbearer - bearer used by broadcast link + * struct tipc_bcbearer - bearer used by broadcast link   * @bearer: (non-standard) broadcast bearer structure   * @media: (non-standard) broadcast media structure   * @bpairs: array of bearer pairs @@ -74,38 +74,40 @@ struct bcbearer_pair {   * prevented through use of the spinlock "bc_lock".   */ -struct bcbearer { +struct tipc_bcbearer {  	struct tipc_bearer bearer; -	struct media media; -	struct bcbearer_pair bpairs[MAX_BEARERS]; -	struct bcbearer_pair bpairs_temp[TIPC_MAX_LINK_PRI + 1]; +	struct tipc_media media; +	struct tipc_bcbearer_pair bpairs[MAX_BEARERS]; +	struct tipc_bcbearer_pair bpairs_temp[TIPC_MAX_LINK_PRI + 1];  	struct tipc_node_map remains;  	struct tipc_node_map remains_new;  };  /** - * struct bclink - link used for broadcast messages + * struct tipc_bclink - link used for broadcast messages   * @link: (non-standard) broadcast link structure   * @node: (non-standard) node structure representing b'cast link's peer node + * @bcast_nodes: map of broadcast-capable nodes   * @retransmit_to: node that most recently requested a retransmit   *   * Handles sequence numbering, fragmentation, bundling, etc.   */ -struct bclink { -	struct link link; +struct tipc_bclink { +	struct tipc_link link;  	struct tipc_node node; +	struct tipc_node_map bcast_nodes;  	struct tipc_node *retransmit_to;  }; +static struct tipc_bcbearer bcast_bearer; +static struct tipc_bclink bcast_link; -static struct bcbearer *bcbearer; -static struct bclink *bclink; -static struct link *bcl; -static DEFINE_SPINLOCK(bc_lock); +static struct tipc_bcbearer *bcbearer = &bcast_bearer; +static struct tipc_bclink *bclink = &bcast_link; +static struct tipc_link *bcl = &bcast_link.link; -/* broadcast-capable node map */ -struct tipc_node_map tipc_bcast_nmap; +static DEFINE_SPINLOCK(bc_lock);  const char tipc_bclink_name[] = "broadcast-link"; @@ -113,11 +115,6 @@ static void tipc_nmap_diff(struct tipc_node_map *nm_a,  			   struct tipc_node_map *nm_b,  			   struct tipc_node_map *nm_diff); -static u32 buf_seqno(struct sk_buff *buf) -{ -	return msg_seqno(buf_msg(buf)); -} -  static u32 bcbuf_acks(struct sk_buff *buf)  {  	return (u32)(unsigned long)TIPC_SKB_CB(buf)->handle; @@ -133,6 +130,19 @@ static void bcbuf_decr_acks(struct sk_buff *buf)  	bcbuf_set_acks(buf, bcbuf_acks(buf) - 1);  } +void tipc_bclink_add_node(u32 addr) +{ +	spin_lock_bh(&bc_lock); +	tipc_nmap_add(&bclink->bcast_nodes, addr); +	spin_unlock_bh(&bc_lock); +} + +void tipc_bclink_remove_node(u32 addr) +{ +	spin_lock_bh(&bc_lock); +	tipc_nmap_remove(&bclink->bcast_nodes, addr); +	spin_unlock_bh(&bc_lock); +}  static void bclink_set_last_sent(void)  { @@ -222,14 +232,36 @@ void tipc_bclink_acknowledge(struct tipc_node *n_ptr, u32 acked)  	struct sk_buff *next;  	unsigned int released = 0; -	if (less_eq(acked, n_ptr->bclink.acked)) -		return; -  	spin_lock_bh(&bc_lock); -	/* Skip over packets that node has previously acknowledged */ - +	/* Bail out if tx queue is empty (no clean up is required) */  	crs = bcl->first_out; +	if (!crs) +		goto exit; + +	/* Determine which messages need to be acknowledged */ +	if (acked == INVALID_LINK_SEQ) { +		/* +		 * Contact with specified node has been lost, so need to +		 * acknowledge sent messages only (if other nodes still exist) +		 * or both sent and unsent messages (otherwise) +		 */ +		if (bclink->bcast_nodes.count) +			acked = bcl->fsm_msg_cnt; +		else +			acked = bcl->next_out_no; +	} else { +		/* +		 * Bail out if specified sequence number does not correspond +		 * to a message that has been sent and not yet acknowledged +		 */ +		if (less(acked, buf_seqno(crs)) || +		    less(bcl->fsm_msg_cnt, acked) || +		    less_eq(acked, n_ptr->bclink.acked)) +			goto exit; +	} + +	/* Skip over packets that node has previously acknowledged */  	while (crs && less_eq(buf_seqno(crs), n_ptr->bclink.acked))  		crs = crs->next; @@ -237,7 +269,15 @@ void tipc_bclink_acknowledge(struct tipc_node *n_ptr, u32 acked)  	while (crs && less_eq(buf_seqno(crs), acked)) {  		next = crs->next; -		bcbuf_decr_acks(crs); + +		if (crs != bcl->next_out) +			bcbuf_decr_acks(crs); +		else { +			bcbuf_set_acks(crs, 0); +			bcl->next_out = next; +			bclink_set_last_sent(); +		} +  		if (bcbuf_acks(crs) == 0) {  			bcl->first_out = next;  			bcl->out_queue_size--; @@ -256,6 +296,7 @@ void tipc_bclink_acknowledge(struct tipc_node *n_ptr, u32 acked)  	}  	if (unlikely(released && !list_empty(&bcl->waiting_ports)))  		tipc_link_wakeup_ports(bcl, 0); +exit:  	spin_unlock_bh(&bc_lock);  } @@ -267,7 +308,7 @@ void tipc_bclink_acknowledge(struct tipc_node *n_ptr, u32 acked)  static void bclink_send_ack(struct tipc_node *n_ptr)  { -	struct link *l_ptr = n_ptr->active_links[n_ptr->addr & 1]; +	struct tipc_link *l_ptr = n_ptr->active_links[n_ptr->addr & 1];  	if (l_ptr != NULL)  		tipc_link_send_proto_msg(l_ptr, STATE_MSG, 0, 0, 0, 0, 0); @@ -402,13 +443,19 @@ int tipc_bclink_send_msg(struct sk_buff *buf)  	spin_lock_bh(&bc_lock); +	if (!bclink->bcast_nodes.count) { +		res = msg_data_sz(buf_msg(buf)); +		buf_discard(buf); +		goto exit; +	} +  	res = tipc_link_send_buf(bcl, buf); -	if (likely(res > 0)) +	if (likely(res >= 0)) {  		bclink_set_last_sent(); - -	bcl->stats.queue_sz_counts++; -	bcl->stats.accu_queue_sz += bcl->out_queue_size; - +		bcl->stats.queue_sz_counts++; +		bcl->stats.accu_queue_sz += bcl->out_queue_size; +	} +exit:  	spin_unlock_bh(&bc_lock);  	return res;  } @@ -572,13 +619,13 @@ static int tipc_bcbearer_send(struct sk_buff *buf,  	if (likely(!msg_non_seq(buf_msg(buf)))) {  		struct tipc_msg *msg; -		bcbuf_set_acks(buf, tipc_bcast_nmap.count); +		bcbuf_set_acks(buf, bclink->bcast_nodes.count);  		msg = buf_msg(buf);  		msg_set_non_seq(msg, 1);  		msg_set_mc_netid(msg, tipc_net_id);  		bcl->stats.sent_info++; -		if (WARN_ON(!tipc_bcast_nmap.count)) { +		if (WARN_ON(!bclink->bcast_nodes.count)) {  			dump_stack();  			return 0;  		} @@ -586,7 +633,7 @@ static int tipc_bcbearer_send(struct sk_buff *buf,  	/* Send buffer over bearers until all targets reached */ -	bcbearer->remains = tipc_bcast_nmap; +	bcbearer->remains = bclink->bcast_nodes;  	for (bp_index = 0; bp_index < MAX_BEARERS; bp_index++) {  		struct tipc_bearer *p = bcbearer->bpairs[bp_index].primary; @@ -630,8 +677,8 @@ static int tipc_bcbearer_send(struct sk_buff *buf,  void tipc_bcbearer_sort(void)  { -	struct bcbearer_pair *bp_temp = bcbearer->bpairs_temp; -	struct bcbearer_pair *bp_curr; +	struct tipc_bcbearer_pair *bp_temp = bcbearer->bpairs_temp; +	struct tipc_bcbearer_pair *bp_curr;  	int b_index;  	int pri; @@ -752,25 +799,13 @@ int tipc_bclink_set_queue_limits(u32 limit)  	return 0;  } -int tipc_bclink_init(void) +void tipc_bclink_init(void)  { -	bcbearer = kzalloc(sizeof(*bcbearer), GFP_ATOMIC); -	bclink = kzalloc(sizeof(*bclink), GFP_ATOMIC); -	if (!bcbearer || !bclink) { -		warn("Broadcast link creation failed, no memory\n"); -		kfree(bcbearer); -		bcbearer = NULL; -		kfree(bclink); -		bclink = NULL; -		return -ENOMEM; -	} -  	INIT_LIST_HEAD(&bcbearer->bearer.cong_links);  	bcbearer->bearer.media = &bcbearer->media;  	bcbearer->media.send_msg = tipc_bcbearer_send;  	sprintf(bcbearer->media.name, "tipc-broadcast"); -	bcl = &bclink->link;  	INIT_LIST_HEAD(&bcl->waiting_ports);  	bcl->next_out_no = 1;  	spin_lock_init(&bclink->node.lock); @@ -780,22 +815,16 @@ int tipc_bclink_init(void)  	bcl->b_ptr = &bcbearer->bearer;  	bcl->state = WORKING_WORKING;  	strlcpy(bcl->name, tipc_bclink_name, TIPC_MAX_LINK_NAME); - -	return 0;  }  void tipc_bclink_stop(void)  {  	spin_lock_bh(&bc_lock); -	if (bcbearer) { -		tipc_link_stop(bcl); -		bcl = NULL; -		kfree(bclink); -		bclink = NULL; -		kfree(bcbearer); -		bcbearer = NULL; -	} +	tipc_link_stop(bcl);  	spin_unlock_bh(&bc_lock); + +	memset(bclink, 0, sizeof(*bclink)); +	memset(bcbearer, 0, sizeof(*bcbearer));  } @@ -864,9 +893,9 @@ static void tipc_nmap_diff(struct tipc_node_map *nm_a,   * tipc_port_list_add - add a port to a port list, ensuring no duplicates   */ -void tipc_port_list_add(struct port_list *pl_ptr, u32 port) +void tipc_port_list_add(struct tipc_port_list *pl_ptr, u32 port)  { -	struct port_list *item = pl_ptr; +	struct tipc_port_list *item = pl_ptr;  	int i;  	int item_sz = PLSIZE;  	int cnt = pl_ptr->count; @@ -898,10 +927,10 @@ void tipc_port_list_add(struct port_list *pl_ptr, u32 port)   *   */ -void tipc_port_list_free(struct port_list *pl_ptr) +void tipc_port_list_free(struct tipc_port_list *pl_ptr)  { -	struct port_list *item; -	struct port_list *next; +	struct tipc_port_list *item; +	struct tipc_port_list *next;  	for (item = pl_ptr->next; item; item = next) {  		next = item->next; diff --git a/net/tipc/bcast.h b/net/tipc/bcast.h index 06740da5ae6..b009666c60b 100644 --- a/net/tipc/bcast.h +++ b/net/tipc/bcast.h @@ -51,20 +51,18 @@ struct tipc_node_map {  	u32 map[MAX_NODES / WSIZE];  }; -extern struct tipc_node_map tipc_bcast_nmap; -  #define PLSIZE 32  /** - * struct port_list - set of node local destination ports + * struct tipc_port_list - set of node local destination ports   * @count: # of ports in set (only valid for first entry in list)   * @next: pointer to next entry in list   * @ports: array of port references   */ -struct port_list { +struct tipc_port_list {  	int count; -	struct port_list *next; +	struct tipc_port_list *next;  	u32 ports[PLSIZE];  }; @@ -85,11 +83,13 @@ static inline int tipc_nmap_equal(struct tipc_node_map *nm_a, struct tipc_node_m  	return !memcmp(nm_a, nm_b, sizeof(*nm_a));  } -void tipc_port_list_add(struct port_list *pl_ptr, u32 port); -void tipc_port_list_free(struct port_list *pl_ptr); +void tipc_port_list_add(struct tipc_port_list *pl_ptr, u32 port); +void tipc_port_list_free(struct tipc_port_list *pl_ptr); -int  tipc_bclink_init(void); +void tipc_bclink_init(void);  void tipc_bclink_stop(void); +void tipc_bclink_add_node(u32 addr); +void tipc_bclink_remove_node(u32 addr);  struct tipc_node *tipc_bclink_retransmit_to(void);  void tipc_bclink_acknowledge(struct tipc_node *n_ptr, u32 acked);  int  tipc_bclink_send_msg(struct sk_buff *buf); diff --git a/net/tipc/bearer.c b/net/tipc/bearer.c index e2202de3d93..329fb659fae 100644 --- a/net/tipc/bearer.c +++ b/net/tipc/bearer.c @@ -41,7 +41,7 @@  #define MAX_ADDR_STR 32 -static struct media media_list[MAX_MEDIA]; +static struct tipc_media *media_list[MAX_MEDIA];  static u32 media_count;  struct tipc_bearer tipc_bearers[MAX_BEARERS]; @@ -65,17 +65,31 @@ static int media_name_valid(const char *name)  }  /** - * media_find - locates specified media object by name + * tipc_media_find - locates specified media object by name   */ -static struct media *media_find(const char *name) +struct tipc_media *tipc_media_find(const char *name)  { -	struct media *m_ptr;  	u32 i; -	for (i = 0, m_ptr = media_list; i < media_count; i++, m_ptr++) { -		if (!strcmp(m_ptr->name, name)) -			return m_ptr; +	for (i = 0; i < media_count; i++) { +		if (!strcmp(media_list[i]->name, name)) +			return media_list[i]; +	} +	return NULL; +} + +/** + * media_find_id - locates specified media object by type identifier + */ + +static struct tipc_media *media_find_id(u8 type) +{ +	u32 i; + +	for (i = 0; i < media_count; i++) { +		if (media_list[i]->type_id == type) +			return media_list[i];  	}  	return NULL;  } @@ -86,87 +100,34 @@ static struct media *media_find(const char *name)   * Bearers for this media type must be activated separately at a later stage.   */ -int  tipc_register_media(u32 media_type, -			 char *name, -			 int (*enable)(struct tipc_bearer *), -			 void (*disable)(struct tipc_bearer *), -			 int (*send_msg)(struct sk_buff *, -					 struct tipc_bearer *, -					 struct tipc_media_addr *), -			 char *(*addr2str)(struct tipc_media_addr *a, -					   char *str_buf, int str_size), -			 struct tipc_media_addr *bcast_addr, -			 const u32 bearer_priority, -			 const u32 link_tolerance,  /* [ms] */ -			 const u32 send_window_limit) +int tipc_register_media(struct tipc_media *m_ptr)  { -	struct media *m_ptr; -	u32 media_id; -	u32 i;  	int res = -EINVAL;  	write_lock_bh(&tipc_net_lock); -	if (tipc_mode != TIPC_NET_MODE) { -		warn("Media <%s> rejected, not in networked mode yet\n", name); +	if (!media_name_valid(m_ptr->name))  		goto exit; -	} -	if (!media_name_valid(name)) { -		warn("Media <%s> rejected, illegal name\n", name); +	if ((m_ptr->bcast_addr.media_id != m_ptr->type_id) || +	    !m_ptr->bcast_addr.broadcast)  		goto exit; -	} -	if (!bcast_addr) { -		warn("Media <%s> rejected, no broadcast address\n", name); +	if (m_ptr->priority > TIPC_MAX_LINK_PRI)  		goto exit; -	} -	if ((bearer_priority < TIPC_MIN_LINK_PRI) || -	    (bearer_priority > TIPC_MAX_LINK_PRI)) { -		warn("Media <%s> rejected, illegal priority (%u)\n", name, -		     bearer_priority); +	if ((m_ptr->tolerance < TIPC_MIN_LINK_TOL) || +	    (m_ptr->tolerance > TIPC_MAX_LINK_TOL))  		goto exit; -	} -	if ((link_tolerance < TIPC_MIN_LINK_TOL) || -	    (link_tolerance > TIPC_MAX_LINK_TOL)) { -		warn("Media <%s> rejected, illegal tolerance (%u)\n", name, -		     link_tolerance); +	if (media_count >= MAX_MEDIA)  		goto exit; -	} - -	media_id = media_count++; -	if (media_id >= MAX_MEDIA) { -		warn("Media <%s> rejected, media limit reached (%u)\n", name, -		     MAX_MEDIA); -		media_count--; +	if (tipc_media_find(m_ptr->name) || media_find_id(m_ptr->type_id))  		goto exit; -	} -	for (i = 0; i < media_id; i++) { -		if (media_list[i].type_id == media_type) { -			warn("Media <%s> rejected, duplicate type (%u)\n", name, -			     media_type); -			media_count--; -			goto exit; -		} -		if (!strcmp(name, media_list[i].name)) { -			warn("Media <%s> rejected, duplicate name\n", name); -			media_count--; -			goto exit; -		} -	} -	m_ptr = &media_list[media_id]; -	m_ptr->type_id = media_type; -	m_ptr->send_msg = send_msg; -	m_ptr->enable_bearer = enable; -	m_ptr->disable_bearer = disable; -	m_ptr->addr2str = addr2str; -	memcpy(&m_ptr->bcast_addr, bcast_addr, sizeof(*bcast_addr)); -	strcpy(m_ptr->name, name); -	m_ptr->priority = bearer_priority; -	m_ptr->tolerance = link_tolerance; -	m_ptr->window = send_window_limit; +	media_list[media_count] = m_ptr; +	media_count++;  	res = 0;  exit:  	write_unlock_bh(&tipc_net_lock); +	if (res) +		warn("Media <%s> registration error\n", m_ptr->name);  	return res;  } @@ -176,27 +137,19 @@ exit:  void tipc_media_addr_printf(struct print_buf *pb, struct tipc_media_addr *a)  { -	struct media *m_ptr; -	u32 media_type; -	u32 i; +	char addr_str[MAX_ADDR_STR]; +	struct tipc_media *m_ptr; -	media_type = ntohl(a->type); -	for (i = 0, m_ptr = media_list; i < media_count; i++, m_ptr++) { -		if (m_ptr->type_id == media_type) -			break; -	} - -	if ((i < media_count) && (m_ptr->addr2str != NULL)) { -		char addr_str[MAX_ADDR_STR]; +	m_ptr = media_find_id(a->media_id); -		tipc_printf(pb, "%s(%s)", m_ptr->name, -			    m_ptr->addr2str(a, addr_str, sizeof(addr_str))); -	} else { -		unchar *addr = (unchar *)&a->dev_addr; +	if (m_ptr && !m_ptr->addr2str(a, addr_str, sizeof(addr_str))) +		tipc_printf(pb, "%s(%s)", m_ptr->name, addr_str); +	else { +		u32 i; -		tipc_printf(pb, "UNKNOWN(%u)", media_type); -		for (i = 0; i < (sizeof(*a) - sizeof(a->type)); i++) -			tipc_printf(pb, "-%02x", addr[i]); +		tipc_printf(pb, "UNKNOWN(%u)", a->media_id); +		for (i = 0; i < sizeof(a->value); i++) +			tipc_printf(pb, "-%02x", a->value[i]);  	}  } @@ -207,7 +160,6 @@ void tipc_media_addr_printf(struct print_buf *pb, struct tipc_media_addr *a)  struct sk_buff *tipc_media_get_names(void)  {  	struct sk_buff *buf; -	struct media *m_ptr;  	int i;  	buf = tipc_cfg_reply_alloc(MAX_MEDIA * TLV_SPACE(TIPC_MAX_MEDIA_NAME)); @@ -215,9 +167,10 @@ struct sk_buff *tipc_media_get_names(void)  		return NULL;  	read_lock_bh(&tipc_net_lock); -	for (i = 0, m_ptr = media_list; i < media_count; i++, m_ptr++) { -		tipc_cfg_append_tlv(buf, TIPC_TLV_MEDIA_NAME, m_ptr->name, -				    strlen(m_ptr->name) + 1); +	for (i = 0; i < media_count; i++) { +		tipc_cfg_append_tlv(buf, TIPC_TLV_MEDIA_NAME, +				    media_list[i]->name, +				    strlen(media_list[i]->name) + 1);  	}  	read_unlock_bh(&tipc_net_lock);  	return buf; @@ -232,7 +185,7 @@ struct sk_buff *tipc_media_get_names(void)   */  static int bearer_name_validate(const char *name, -				struct bearer_name *name_parts) +				struct tipc_bearer_names *name_parts)  {  	char name_copy[TIPC_MAX_BEARER_NAME];  	char *media_name; @@ -276,10 +229,10 @@ static int bearer_name_validate(const char *name,  }  /** - * bearer_find - locates bearer object with matching bearer name + * tipc_bearer_find - locates bearer object with matching bearer name   */ -static struct tipc_bearer *bearer_find(const char *name) +struct tipc_bearer *tipc_bearer_find(const char *name)  {  	struct tipc_bearer *b_ptr;  	u32 i; @@ -318,7 +271,6 @@ struct tipc_bearer *tipc_bearer_find_interface(const char *if_name)  struct sk_buff *tipc_bearer_get_names(void)  {  	struct sk_buff *buf; -	struct media *m_ptr;  	struct tipc_bearer *b_ptr;  	int i, j; @@ -327,10 +279,10 @@ struct sk_buff *tipc_bearer_get_names(void)  		return NULL;  	read_lock_bh(&tipc_net_lock); -	for (i = 0, m_ptr = media_list; i < media_count; i++, m_ptr++) { +	for (i = 0; i < media_count; i++) {  		for (j = 0; j < MAX_BEARERS; j++) {  			b_ptr = &tipc_bearers[j]; -			if (b_ptr->active && (b_ptr->media == m_ptr)) { +			if (b_ptr->active && (b_ptr->media == media_list[i])) {  				tipc_cfg_append_tlv(buf, TIPC_TLV_BEARER_NAME,  						    b_ptr->name,  						    strlen(b_ptr->name) + 1); @@ -366,7 +318,7 @@ void tipc_bearer_remove_dest(struct tipc_bearer *b_ptr, u32 dest)  static int bearer_push(struct tipc_bearer *b_ptr)  {  	u32 res = 0; -	struct link *ln, *tln; +	struct tipc_link *ln, *tln;  	if (b_ptr->blocked)  		return 0; @@ -412,7 +364,8 @@ void tipc_continue(struct tipc_bearer *b_ptr)   * bearer.lock is busy   */ -static void tipc_bearer_schedule_unlocked(struct tipc_bearer *b_ptr, struct link *l_ptr) +static void tipc_bearer_schedule_unlocked(struct tipc_bearer *b_ptr, +						struct tipc_link *l_ptr)  {  	list_move_tail(&l_ptr->link_list, &b_ptr->cong_links);  } @@ -425,7 +378,7 @@ static void tipc_bearer_schedule_unlocked(struct tipc_bearer *b_ptr, struct link   * bearer.lock is free   */ -void tipc_bearer_schedule(struct tipc_bearer *b_ptr, struct link *l_ptr) +void tipc_bearer_schedule(struct tipc_bearer *b_ptr, struct tipc_link *l_ptr)  {  	spin_lock_bh(&b_ptr->lock);  	tipc_bearer_schedule_unlocked(b_ptr, l_ptr); @@ -438,7 +391,8 @@ void tipc_bearer_schedule(struct tipc_bearer *b_ptr, struct link *l_ptr)   * and if there is, try to resolve it before returning.   * 'tipc_net_lock' is read_locked when this function is called   */ -int tipc_bearer_resolve_congestion(struct tipc_bearer *b_ptr, struct link *l_ptr) +int tipc_bearer_resolve_congestion(struct tipc_bearer *b_ptr, +					struct tipc_link *l_ptr)  {  	int res = 1; @@ -457,7 +411,7 @@ int tipc_bearer_resolve_congestion(struct tipc_bearer *b_ptr, struct link *l_ptr   * tipc_bearer_congested - determines if bearer is currently congested   */ -int tipc_bearer_congested(struct tipc_bearer *b_ptr, struct link *l_ptr) +int tipc_bearer_congested(struct tipc_bearer *b_ptr, struct tipc_link *l_ptr)  {  	if (unlikely(b_ptr->blocked))  		return 1; @@ -473,8 +427,8 @@ int tipc_bearer_congested(struct tipc_bearer *b_ptr, struct link *l_ptr)  int tipc_enable_bearer(const char *name, u32 disc_domain, u32 priority)  {  	struct tipc_bearer *b_ptr; -	struct media *m_ptr; -	struct bearer_name b_name; +	struct tipc_media *m_ptr; +	struct tipc_bearer_names b_names;  	char addr_string[16];  	u32 bearer_id;  	u32 with_this_prio; @@ -486,7 +440,7 @@ int tipc_enable_bearer(const char *name, u32 disc_domain, u32 priority)  		     name);  		return -ENOPROTOOPT;  	} -	if (!bearer_name_validate(name, &b_name)) { +	if (!bearer_name_validate(name, &b_names)) {  		warn("Bearer <%s> rejected, illegal name\n", name);  		return -EINVAL;  	} @@ -511,10 +465,10 @@ int tipc_enable_bearer(const char *name, u32 disc_domain, u32 priority)  	write_lock_bh(&tipc_net_lock); -	m_ptr = media_find(b_name.media_name); +	m_ptr = tipc_media_find(b_names.media_name);  	if (!m_ptr) {  		warn("Bearer <%s> rejected, media <%s> not registered\n", name, -		     b_name.media_name); +		     b_names.media_name);  		goto exit;  	} @@ -561,6 +515,8 @@ restart:  	b_ptr->identity = bearer_id;  	b_ptr->media = m_ptr; +	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; @@ -590,11 +546,11 @@ exit:  int tipc_block_bearer(const char *name)  {  	struct tipc_bearer *b_ptr = NULL; -	struct link *l_ptr; -	struct link *temp_l_ptr; +	struct tipc_link *l_ptr; +	struct tipc_link *temp_l_ptr;  	read_lock_bh(&tipc_net_lock); -	b_ptr = bearer_find(name); +	b_ptr = tipc_bearer_find(name);  	if (!b_ptr) {  		warn("Attempt to block unknown bearer <%s>\n", name);  		read_unlock_bh(&tipc_net_lock); @@ -625,8 +581,8 @@ int tipc_block_bearer(const char *name)  static void bearer_disable(struct tipc_bearer *b_ptr)  { -	struct link *l_ptr; -	struct link *temp_l_ptr; +	struct tipc_link *l_ptr; +	struct tipc_link *temp_l_ptr;  	info("Disabling bearer <%s>\n", b_ptr->name);  	spin_lock_bh(&b_ptr->lock); @@ -648,7 +604,7 @@ int tipc_disable_bearer(const char *name)  	int res;  	write_lock_bh(&tipc_net_lock); -	b_ptr = bearer_find(name); +	b_ptr = tipc_bearer_find(name);  	if (b_ptr == NULL) {  		warn("Attempt to disable unknown bearer <%s>\n", name);  		res = -EINVAL; diff --git a/net/tipc/bearer.h b/net/tipc/bearer.h index d696f9e414e..d3eac56b8c2 100644 --- a/net/tipc/bearer.h +++ b/net/tipc/bearer.h @@ -43,32 +43,45 @@  #define MAX_MEDIA	2  /* + * Identifiers associated with TIPC message header media address info + * + * - address info field is 20 bytes long + * - media type identifier located at offset 3 + * - remaining bytes vary according to media type + */ + +#define TIPC_MEDIA_ADDR_SIZE	20 +#define TIPC_MEDIA_TYPE_OFFSET	3 + +/*   * Identifiers of supported TIPC media types   */  #define TIPC_MEDIA_TYPE_ETH	1  /* - * Destination address structure used by TIPC bearers when sending messages - * - * IMPORTANT: The fields of this structure MUST be stored using the specified - * byte order indicated below, as the structure is exchanged between nodes - * as part of a link setup process. + * struct tipc_media_addr - destination address used by TIPC bearers + * @value: address info (format defined by media) + * @media_id: TIPC media type identifier + * @broadcast: non-zero if address is a broadcast address   */ +  struct tipc_media_addr { -	__be32  type;			/* bearer type (network byte order) */ -	union { -		__u8   eth_addr[6];	/* 48 bit Ethernet addr (byte array) */ -	} dev_addr; +	u8 value[TIPC_MEDIA_ADDR_SIZE]; +	u8 media_id; +	u8 broadcast;  };  struct tipc_bearer;  /** - * struct media - TIPC media information available to internal users + * struct tipc_media - TIPC media information available to internal users   * @send_msg: routine which handles buffer transmission   * @enable_bearer: routine which enables a bearer   * @disable_bearer: routine which disables a bearer - * @addr2str: routine which converts bearer's address to string form + * @addr2str: routine which converts media address to string + * @str2addr: routine which converts media address from string + * @addr2msg: routine which converts media address to protocol message area + * @msg2addr: routine which converts media address from protocol message area   * @bcast_addr: media address used in broadcasting   * @priority: default link (and bearer) priority   * @tolerance: default time (in ms) before declaring link failure @@ -77,14 +90,16 @@ struct tipc_bearer;   * @name: media name   */ -struct media { +struct tipc_media {  	int (*send_msg)(struct sk_buff *buf,  			struct tipc_bearer *b_ptr,  			struct tipc_media_addr *dest);  	int (*enable_bearer)(struct tipc_bearer *b_ptr);  	void (*disable_bearer)(struct tipc_bearer *b_ptr); -	char *(*addr2str)(struct tipc_media_addr *a, -			  char *str_buf, int str_size); +	int (*addr2str)(struct tipc_media_addr *a, char *str_buf, int str_size); +	int (*str2addr)(struct tipc_media_addr *a, char *str_buf); +	int (*addr2msg)(struct tipc_media_addr *a, char *msg_area); +	int (*msg2addr)(struct tipc_media_addr *a, char *msg_area);  	struct tipc_media_addr bcast_addr;  	u32 priority;  	u32 tolerance; @@ -103,6 +118,8 @@ struct media {   * @name: bearer name (format = media:interface)   * @media: ptr to media structure associated with bearer   * @priority: default link priority for bearer + * @window: default window size for bearer + * @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   * @links: list of non-congested links associated with bearer @@ -122,10 +139,12 @@ struct tipc_bearer {  	struct tipc_media_addr addr;		/* initalized by media */  	char name[TIPC_MAX_BEARER_NAME];  	spinlock_t lock; -	struct media *media; +	struct tipc_media *media;  	u32 priority; +	u32 window; +	u32 tolerance;  	u32 identity; -	struct link_req *link_req; +	struct tipc_link_req *link_req;  	struct list_head links;  	struct list_head cong_links;  	int active; @@ -133,28 +152,19 @@ struct tipc_bearer {  	struct tipc_node_map nodes;  }; -struct bearer_name { +struct tipc_bearer_names {  	char media_name[TIPC_MAX_MEDIA_NAME];  	char if_name[TIPC_MAX_IF_NAME];  }; -struct link; +struct tipc_link;  extern struct tipc_bearer tipc_bearers[];  /*   * TIPC routines available to supported media types   */ -int tipc_register_media(u32 media_type, -		 char *media_name, int (*enable)(struct tipc_bearer *), -		 void (*disable)(struct tipc_bearer *), -		 int (*send_msg)(struct sk_buff *, -			struct tipc_bearer *, struct tipc_media_addr *), -		 char *(*addr2str)(struct tipc_media_addr *a, -			char *str_buf, int str_size), -		 struct tipc_media_addr *bcast_addr, const u32 bearer_priority, -		 const u32 link_tolerance,  /* [ms] */ -		 const u32 send_window_limit); +int tipc_register_media(struct tipc_media *m_ptr);  void tipc_recv_msg(struct sk_buff *buf, struct tipc_bearer *tb_ptr); @@ -170,16 +180,21 @@ int tipc_disable_bearer(const char *name);  int  tipc_eth_media_start(void);  void tipc_eth_media_stop(void); +int tipc_media_set_priority(const char *name, u32 new_value); +int tipc_media_set_window(const char *name, u32 new_value);  void tipc_media_addr_printf(struct print_buf *pb, struct tipc_media_addr *a);  struct sk_buff *tipc_media_get_names(void);  struct sk_buff *tipc_bearer_get_names(void);  void tipc_bearer_add_dest(struct tipc_bearer *b_ptr, u32 dest);  void tipc_bearer_remove_dest(struct tipc_bearer *b_ptr, u32 dest); -void tipc_bearer_schedule(struct tipc_bearer *b_ptr, struct link *l_ptr); +void tipc_bearer_schedule(struct tipc_bearer *b_ptr, struct tipc_link *l_ptr); +struct tipc_bearer *tipc_bearer_find(const char *name);  struct tipc_bearer *tipc_bearer_find_interface(const char *if_name); -int tipc_bearer_resolve_congestion(struct tipc_bearer *b_ptr, struct link *l_ptr); -int tipc_bearer_congested(struct tipc_bearer *b_ptr, struct link *l_ptr); +struct tipc_media *tipc_media_find(const char *name); +int tipc_bearer_resolve_congestion(struct tipc_bearer *b_ptr, +				   struct tipc_link *l_ptr); +int tipc_bearer_congested(struct tipc_bearer *b_ptr, struct tipc_link *l_ptr);  void tipc_bearer_stop(void);  void tipc_bearer_lock_push(struct tipc_bearer *b_ptr); diff --git a/net/tipc/config.c b/net/tipc/config.c index b25a396b7e1..4785bf26cdf 100644 --- a/net/tipc/config.c +++ b/net/tipc/config.c @@ -184,13 +184,12 @@ static struct sk_buff *cfg_set_own_addr(void)  						   " (cannot change node address once assigned)");  	/* -	 * Must release all spinlocks before calling start_net() because -	 * Linux version of TIPC calls eth_media_start() which calls -	 * register_netdevice_notifier() which may block! -	 * -	 * Temporarily releasing the lock should be harmless for non-Linux TIPC, -	 * but Linux version of eth_media_start() should really be reworked -	 * so that it can be called with spinlocks held. +	 * Must temporarily release configuration spinlock while switching into +	 * networking mode as it calls tipc_eth_media_start(), which may sleep. +	 * Releasing the lock is harmless as other locally-issued configuration +	 * commands won't occur until this one completes, and remotely-issued +	 * configuration commands can't be received until a local configuration +	 * command to enable the first bearer is received and processed.  	 */  	spin_unlock_bh(&config_lock); diff --git a/net/tipc/core.c b/net/tipc/core.c index c21331d58fd..2691cd57b8a 100644 --- a/net/tipc/core.c +++ b/net/tipc/core.c @@ -99,8 +99,8 @@ struct sk_buff *tipc_buf_acquire(u32 size)  static void tipc_core_stop_net(void)  { -	tipc_eth_media_stop();  	tipc_net_stop(); +	tipc_eth_media_stop();  }  /** diff --git a/net/tipc/discover.c b/net/tipc/discover.c index f2fb96e86ee..a00e5f81156 100644 --- a/net/tipc/discover.c +++ b/net/tipc/discover.c @@ -45,7 +45,7 @@  /** - * struct link_req - information about an ongoing link setup request + * struct tipc_link_req - information about an ongoing link setup request   * @bearer: bearer issuing requests   * @dest: destination address for request messages   * @domain: network domain to which links can be established @@ -54,7 +54,7 @@   * @timer: timer governing period between requests   * @timer_intv: current interval between requests (in ms)   */ -struct link_req { +struct tipc_link_req {  	struct tipc_bearer *bearer;  	struct tipc_media_addr dest;  	u32 domain; @@ -84,7 +84,7 @@ static struct sk_buff *tipc_disc_init_msg(u32 type,  		msg_set_non_seq(msg, 1);  		msg_set_dest_domain(msg, dest_domain);  		msg_set_bc_netid(msg, tipc_net_id); -		msg_set_media_addr(msg, &b_ptr->addr); +		b_ptr->media->addr2msg(&b_ptr->addr, msg_media_addr(msg));  	}  	return buf;  } @@ -120,7 +120,7 @@ static void disc_dupl_alert(struct tipc_bearer *b_ptr, u32 node_addr,  void tipc_disc_recv_msg(struct sk_buff *buf, struct tipc_bearer *b_ptr)  {  	struct tipc_node *n_ptr; -	struct link *link; +	struct tipc_link *link;  	struct tipc_media_addr media_addr, *addr;  	struct sk_buff *rbuf;  	struct tipc_msg *msg = buf_msg(buf); @@ -130,12 +130,15 @@ void tipc_disc_recv_msg(struct sk_buff *buf, struct tipc_bearer *b_ptr)  	u32 type = msg_type(msg);  	int link_fully_up; -	msg_get_media_addr(msg, &media_addr); +	media_addr.broadcast = 1; +	b_ptr->media->msg2addr(&media_addr, msg_media_addr(msg));  	buf_discard(buf);  	/* Validate discovery message from requesting node */  	if (net_id != tipc_net_id)  		return; +	if (media_addr.broadcast) +		return;  	if (!tipc_addr_domain_valid(dest))  		return;  	if (!tipc_addr_node_valid(orig)) @@ -215,7 +218,7 @@ void tipc_disc_recv_msg(struct sk_buff *buf, struct tipc_bearer *b_ptr)   * and is either not currently searching or is searching at a slow rate   */ -static void disc_update(struct link_req *req) +static void disc_update(struct tipc_link_req *req)  {  	if (!req->num_nodes) {  		if ((req->timer_intv == TIPC_LINK_REQ_INACTIVE) || @@ -231,7 +234,7 @@ static void disc_update(struct link_req *req)   * @req: ptr to link request structure   */ -void tipc_disc_add_dest(struct link_req *req) +void tipc_disc_add_dest(struct tipc_link_req *req)  {  	req->num_nodes++;  } @@ -241,7 +244,7 @@ void tipc_disc_add_dest(struct link_req *req)   * @req: ptr to link request structure   */ -void tipc_disc_remove_dest(struct link_req *req) +void tipc_disc_remove_dest(struct tipc_link_req *req)  {  	req->num_nodes--;  	disc_update(req); @@ -252,7 +255,7 @@ void tipc_disc_remove_dest(struct link_req *req)   * @req: ptr to link request structure   */ -static void disc_send_msg(struct link_req *req) +static void disc_send_msg(struct tipc_link_req *req)  {  	if (!req->bearer->blocked)  		tipc_bearer_send(req->bearer, req->buf, &req->dest); @@ -265,7 +268,7 @@ static void disc_send_msg(struct link_req *req)   * Called whenever a link setup request timer associated with a bearer expires.   */ -static void disc_timeout(struct link_req *req) +static void disc_timeout(struct tipc_link_req *req)  {  	int max_delay; @@ -313,7 +316,7 @@ exit:  int tipc_disc_create(struct tipc_bearer *b_ptr,  		     struct tipc_media_addr *dest, u32 dest_domain)  { -	struct link_req *req; +	struct tipc_link_req *req;  	req = kmalloc(sizeof(*req), GFP_ATOMIC);  	if (!req) @@ -342,7 +345,7 @@ int tipc_disc_create(struct tipc_bearer *b_ptr,   * @req: ptr to link request structure   */ -void tipc_disc_delete(struct link_req *req) +void tipc_disc_delete(struct tipc_link_req *req)  {  	k_cancel_timer(&req->timer);  	k_term_timer(&req->timer); diff --git a/net/tipc/discover.h b/net/tipc/discover.h index a3af595b86c..75b67c403aa 100644 --- a/net/tipc/discover.h +++ b/net/tipc/discover.h @@ -37,13 +37,13 @@  #ifndef _TIPC_DISCOVER_H  #define _TIPC_DISCOVER_H -struct link_req; +struct tipc_link_req;  int tipc_disc_create(struct tipc_bearer *b_ptr, struct tipc_media_addr *dest,  		     u32 dest_domain); -void tipc_disc_delete(struct link_req *req); -void tipc_disc_add_dest(struct link_req *req); -void tipc_disc_remove_dest(struct link_req *req); +void tipc_disc_delete(struct tipc_link_req *req); +void tipc_disc_add_dest(struct tipc_link_req *req); +void tipc_disc_remove_dest(struct tipc_link_req *req);  void tipc_disc_recv_msg(struct sk_buff *buf, struct tipc_bearer *b_ptr);  #endif diff --git a/net/tipc/eth_media.c b/net/tipc/eth_media.c index e728d4ce2a1..527e3f0e165 100644 --- a/net/tipc/eth_media.c +++ b/net/tipc/eth_media.c @@ -38,28 +38,45 @@  #include "bearer.h"  #define MAX_ETH_BEARERS		MAX_BEARERS -#define ETH_LINK_PRIORITY	TIPC_DEF_LINK_PRI -#define ETH_LINK_TOLERANCE	TIPC_DEF_LINK_TOL -#define ETH_LINK_WINDOW		TIPC_DEF_LINK_WIN + +#define ETH_ADDR_OFFSET	4	/* message header offset of MAC address */  /**   * struct eth_bearer - Ethernet bearer data structure   * @bearer: ptr to associated "generic" bearer structure   * @dev: ptr to associated Ethernet network device   * @tipc_packet_type: used in binding TIPC to Ethernet driver + * @cleanup: work item used when disabling bearer   */  struct eth_bearer {  	struct tipc_bearer *bearer;  	struct net_device *dev;  	struct packet_type tipc_packet_type; +	struct work_struct cleanup;  }; +static struct tipc_media eth_media_info;  static struct eth_bearer eth_bearers[MAX_ETH_BEARERS];  static int eth_started;  static struct notifier_block notifier;  /** + * eth_media_addr_set - initialize Ethernet media address structure + * + * Media-dependent "value" field stores MAC address in first 6 bytes + * and zeroes out the remaining bytes. + */ + +static void eth_media_addr_set(struct tipc_media_addr *a, char *mac) +{ +	memcpy(a->value, mac, ETH_ALEN); +	memset(a->value + ETH_ALEN, 0, sizeof(a->value) - ETH_ALEN); +	a->media_id = TIPC_MEDIA_TYPE_ETH; +	a->broadcast = !memcmp(mac, eth_media_info.bcast_addr.value, ETH_ALEN); +} + +/**   * send_msg - send a TIPC message out over an Ethernet interface   */ @@ -85,7 +102,7 @@ static int send_msg(struct sk_buff *buf, struct tipc_bearer *tb_ptr,  	skb_reset_network_header(clone);  	clone->dev = dev; -	dev_hard_header(clone, dev, ETH_P_TIPC, &dest->dev_addr.eth_addr, +	dev_hard_header(clone, dev, ETH_P_TIPC, dest->value,  			dev->dev_addr, clone->len);  	dev_queue_xmit(clone);  	return 0; @@ -172,22 +189,41 @@ static int enable_bearer(struct tipc_bearer *tb_ptr)  	tb_ptr->usr_handle = (void *)eb_ptr;  	tb_ptr->mtu = dev->mtu;  	tb_ptr->blocked = 0; -	tb_ptr->addr.type = htonl(TIPC_MEDIA_TYPE_ETH); -	memcpy(&tb_ptr->addr.dev_addr, dev->dev_addr, ETH_ALEN); +	eth_media_addr_set(&tb_ptr->addr, (char *)dev->dev_addr);  	return 0;  }  /** + * cleanup_bearer - break association between Ethernet bearer and interface + * + * This routine must be invoked from a work queue because it can sleep. + */ + +static void cleanup_bearer(struct work_struct *work) +{ +	struct eth_bearer *eb_ptr = +		container_of(work, struct eth_bearer, cleanup); + +	dev_remove_pack(&eb_ptr->tipc_packet_type); +	dev_put(eb_ptr->dev); +	eb_ptr->dev = NULL; +} + +/**   * disable_bearer - detach TIPC bearer from an Ethernet interface   * - * We really should do dev_remove_pack() here, but this function can not be - * called at tasklet level. => Use eth_bearer->bearer as a flag to throw away - * incoming buffers, & postpone dev_remove_pack() to eth_media_stop() on exit. + * Mark Ethernet bearer as inactive so that incoming buffers are thrown away, + * then get worker thread to complete bearer cleanup.  (Can't do cleanup + * here because cleanup code needs to sleep and caller holds spinlocks.)   */  static void disable_bearer(struct tipc_bearer *tb_ptr)  { -	((struct eth_bearer *)tb_ptr->usr_handle)->bearer = NULL; +	struct eth_bearer *eb_ptr = (struct eth_bearer *)tb_ptr->usr_handle; + +	eb_ptr->bearer = NULL; +	INIT_WORK(&eb_ptr->cleanup, cleanup_bearer); +	schedule_work(&eb_ptr->cleanup);  }  /** @@ -246,17 +282,81 @@ static int recv_notification(struct notifier_block *nb, unsigned long evt,   * eth_addr2str - convert Ethernet address to string   */ -static char *eth_addr2str(struct tipc_media_addr *a, char *str_buf, int str_size) +static int eth_addr2str(struct tipc_media_addr *a, char *str_buf, int str_size) +{ +	if (str_size < 18)	/* 18 = strlen("aa:bb:cc:dd:ee:ff\0") */ +		return 1; + +	sprintf(str_buf, "%pM", a->value); +	return 0; +} + +/** + * eth_str2addr - convert string to Ethernet address + */ + +static int eth_str2addr(struct tipc_media_addr *a, char *str_buf) +{ +	char mac[ETH_ALEN]; +	int r; + +	r = sscanf(str_buf, "%02x:%02x:%02x:%02x:%02x:%02x", +		       (u32 *)&mac[0], (u32 *)&mac[1], (u32 *)&mac[2], +		       (u32 *)&mac[3], (u32 *)&mac[4], (u32 *)&mac[5]); + +	if (r != ETH_ALEN) +		return 1; + +	eth_media_addr_set(a, mac); +	return 0; +} + +/** + * eth_str2addr - convert Ethernet address format to message header format + */ + +static int eth_addr2msg(struct tipc_media_addr *a, char *msg_area)  { -	unchar *addr = (unchar *)&a->dev_addr; +	memset(msg_area, 0, TIPC_MEDIA_ADDR_SIZE); +	msg_area[TIPC_MEDIA_TYPE_OFFSET] = TIPC_MEDIA_TYPE_ETH; +	memcpy(msg_area + ETH_ADDR_OFFSET, a->value, ETH_ALEN); +	return 0; +} -	if (str_size < 18) -		*str_buf = '\0'; -	else -		sprintf(str_buf, "%pM", addr); -	return str_buf; +/** + * eth_str2addr - convert message header address format to Ethernet format + */ + +static int eth_msg2addr(struct tipc_media_addr *a, char *msg_area) +{ +	if (msg_area[TIPC_MEDIA_TYPE_OFFSET] != TIPC_MEDIA_TYPE_ETH) +		return 1; + +	eth_media_addr_set(a, msg_area + ETH_ADDR_OFFSET); +	return 0;  } +/* + * Ethernet media registration info + */ + +static struct tipc_media eth_media_info = { +	.send_msg	= send_msg, +	.enable_bearer	= enable_bearer, +	.disable_bearer	= disable_bearer, +	.addr2str	= eth_addr2str, +	.str2addr	= eth_str2addr, +	.addr2msg	= eth_addr2msg, +	.msg2addr	= eth_msg2addr, +	.bcast_addr	= { { 0xff, 0xff, 0xff, 0xff, 0xff, 0xff }, +			    TIPC_MEDIA_TYPE_ETH, 1 }, +	.priority	= TIPC_DEF_LINK_PRI, +	.tolerance	= TIPC_DEF_LINK_TOL, +	.window		= TIPC_DEF_LINK_WIN, +	.type_id	= TIPC_MEDIA_TYPE_ETH, +	.name		= "eth" +}; +  /**   * tipc_eth_media_start - activate Ethernet bearer support   * @@ -266,21 +366,12 @@ static char *eth_addr2str(struct tipc_media_addr *a, char *str_buf, int str_size  int tipc_eth_media_start(void)  { -	struct tipc_media_addr bcast_addr;  	int res;  	if (eth_started)  		return -EINVAL; -	bcast_addr.type = htonl(TIPC_MEDIA_TYPE_ETH); -	memset(&bcast_addr.dev_addr, 0xff, ETH_ALEN); - -	memset(eth_bearers, 0, sizeof(eth_bearers)); - -	res = tipc_register_media(TIPC_MEDIA_TYPE_ETH, "eth", -				  enable_bearer, disable_bearer, send_msg, -				  eth_addr2str, &bcast_addr, ETH_LINK_PRIORITY, -				  ETH_LINK_TOLERANCE, ETH_LINK_WINDOW); +	res = tipc_register_media(ð_media_info);  	if (res)  		return res; @@ -298,22 +389,10 @@ int tipc_eth_media_start(void)  void tipc_eth_media_stop(void)  { -	int i; -  	if (!eth_started)  		return; +	flush_scheduled_work();  	unregister_netdevice_notifier(¬ifier); -	for (i = 0; i < MAX_ETH_BEARERS ; i++) { -		if (eth_bearers[i].bearer) { -			eth_bearers[i].bearer->blocked = 1; -			eth_bearers[i].bearer = NULL; -		} -		if (eth_bearers[i].dev) { -			dev_remove_pack(ð_bearers[i].tipc_packet_type); -			dev_put(eth_bearers[i].dev); -		} -	} -	memset(ð_bearers, 0, sizeof(eth_bearers));  	eth_started = 0;  } diff --git a/net/tipc/link.c b/net/tipc/link.c index ae98a72da11..ac1832a66f8 100644 --- a/net/tipc/link.c +++ b/net/tipc/link.c @@ -71,35 +71,36 @@  #define START_CHANGEOVER 100000u  /** - * struct link_name - deconstructed link name + * struct tipc_link_name - deconstructed link name   * @addr_local: network address of node at this end   * @if_local: name of interface at this end   * @addr_peer: network address of node at far end   * @if_peer: name of interface at far end   */ -struct link_name { +struct tipc_link_name {  	u32 addr_local;  	char if_local[TIPC_MAX_IF_NAME];  	u32 addr_peer;  	char if_peer[TIPC_MAX_IF_NAME];  }; -static void link_handle_out_of_seq_msg(struct link *l_ptr, +static void link_handle_out_of_seq_msg(struct tipc_link *l_ptr,  				       struct sk_buff *buf); -static void link_recv_proto_msg(struct link *l_ptr, struct sk_buff *buf); -static int  link_recv_changeover_msg(struct link **l_ptr, struct sk_buff **buf); -static void link_set_supervision_props(struct link *l_ptr, u32 tolerance); +static void link_recv_proto_msg(struct tipc_link *l_ptr, struct sk_buff *buf); +static int  link_recv_changeover_msg(struct tipc_link **l_ptr, +				     struct sk_buff **buf); +static void link_set_supervision_props(struct tipc_link *l_ptr, u32 tolerance);  static int  link_send_sections_long(struct tipc_port *sender,  				    struct iovec const *msg_sect,  				    u32 num_sect, unsigned int total_len,  				    u32 destnode); -static void link_check_defragm_bufs(struct link *l_ptr); -static void link_state_event(struct link *l_ptr, u32 event); -static void link_reset_statistics(struct link *l_ptr); -static void link_print(struct link *l_ptr, const char *str); -static void link_start(struct link *l_ptr); -static int link_send_long_buf(struct link *l_ptr, struct sk_buff *buf); +static void link_check_defragm_bufs(struct tipc_link *l_ptr); +static void link_state_event(struct tipc_link *l_ptr, u32 event); +static void link_reset_statistics(struct tipc_link *l_ptr); +static void link_print(struct tipc_link *l_ptr, const char *str); +static void link_start(struct tipc_link *l_ptr); +static int link_send_long_buf(struct tipc_link *l_ptr, struct sk_buff *buf);  /*   *  Simple link routines @@ -110,7 +111,7 @@ static unsigned int align(unsigned int i)  	return (i + 3) & ~3u;  } -static void link_init_max_pkt(struct link *l_ptr) +static void link_init_max_pkt(struct tipc_link *l_ptr)  {  	u32 max_pkt; @@ -127,14 +128,14 @@ static void link_init_max_pkt(struct link *l_ptr)  	l_ptr->max_pkt_probes = 0;  } -static u32 link_next_sent(struct link *l_ptr) +static u32 link_next_sent(struct tipc_link *l_ptr)  {  	if (l_ptr->next_out) -		return msg_seqno(buf_msg(l_ptr->next_out)); +		return buf_seqno(l_ptr->next_out);  	return mod(l_ptr->next_out_no);  } -static u32 link_last_sent(struct link *l_ptr) +static u32 link_last_sent(struct tipc_link *l_ptr)  {  	return mod(link_next_sent(l_ptr) - 1);  } @@ -143,28 +144,29 @@ static u32 link_last_sent(struct link *l_ptr)   *  Simple non-static link routines (i.e. referenced outside this file)   */ -int tipc_link_is_up(struct link *l_ptr) +int tipc_link_is_up(struct tipc_link *l_ptr)  {  	if (!l_ptr)  		return 0;  	return link_working_working(l_ptr) || link_working_unknown(l_ptr);  } -int tipc_link_is_active(struct link *l_ptr) +int tipc_link_is_active(struct tipc_link *l_ptr)  {  	return	(l_ptr->owner->active_links[0] == l_ptr) ||  		(l_ptr->owner->active_links[1] == l_ptr);  }  /** - * link_name_validate - validate & (optionally) deconstruct link name + * link_name_validate - validate & (optionally) deconstruct tipc_link name   * @name - ptr to link name string   * @name_parts - ptr to area for link name components (or NULL if not needed)   *   * Returns 1 if link name is valid, otherwise 0.   */ -static int link_name_validate(const char *name, struct link_name *name_parts) +static int link_name_validate(const char *name, +				struct tipc_link_name *name_parts)  {  	char name_copy[TIPC_MAX_LINK_NAME];  	char *addr_local; @@ -238,7 +240,7 @@ static int link_name_validate(const char *name, struct link_name *name_parts)   * tipc_node_delete() is called.)   */ -static void link_timeout(struct link *l_ptr) +static void link_timeout(struct tipc_link *l_ptr)  {  	tipc_node_lock(l_ptr->owner); @@ -287,7 +289,7 @@ static void link_timeout(struct link *l_ptr)  	tipc_node_unlock(l_ptr->owner);  } -static void link_set_timer(struct link *l_ptr, u32 time) +static void link_set_timer(struct tipc_link *l_ptr, u32 time)  {  	k_start_timer(&l_ptr->timer, time);  } @@ -301,11 +303,11 @@ static void link_set_timer(struct link *l_ptr, u32 time)   * Returns pointer to link.   */ -struct link *tipc_link_create(struct tipc_node *n_ptr, +struct tipc_link *tipc_link_create(struct tipc_node *n_ptr,  			      struct tipc_bearer *b_ptr,  			      const struct tipc_media_addr *media_addr)  { -	struct link *l_ptr; +	struct tipc_link *l_ptr;  	struct tipc_msg *msg;  	char *if_name;  	char addr_string[16]; @@ -343,7 +345,7 @@ struct link *tipc_link_create(struct tipc_node *n_ptr,  	l_ptr->checkpoint = 1;  	l_ptr->peer_session = INVALID_SESSION;  	l_ptr->b_ptr = b_ptr; -	link_set_supervision_props(l_ptr, b_ptr->media->tolerance); +	link_set_supervision_props(l_ptr, b_ptr->tolerance);  	l_ptr->state = RESET_UNKNOWN;  	l_ptr->pmsg = (struct tipc_msg *)&l_ptr->proto_msg; @@ -355,7 +357,7 @@ struct link *tipc_link_create(struct tipc_node *n_ptr,  	strcpy((char *)msg_data(msg), if_name);  	l_ptr->priority = b_ptr->priority; -	tipc_link_set_queue_limits(l_ptr, b_ptr->media->window); +	tipc_link_set_queue_limits(l_ptr, b_ptr->window);  	link_init_max_pkt(l_ptr); @@ -382,7 +384,7 @@ struct link *tipc_link_create(struct tipc_node *n_ptr,   * to avoid a potential deadlock situation.   */ -void tipc_link_delete(struct link *l_ptr) +void tipc_link_delete(struct tipc_link *l_ptr)  {  	if (!l_ptr) {  		err("Attempt to delete non-existent link\n"); @@ -401,7 +403,7 @@ void tipc_link_delete(struct link *l_ptr)  	kfree(l_ptr);  } -static void link_start(struct link *l_ptr) +static void link_start(struct tipc_link *l_ptr)  {  	tipc_node_lock(l_ptr->owner);  	link_state_event(l_ptr, STARTING_EVT); @@ -418,7 +420,7 @@ static void link_start(struct link *l_ptr)   * has abated.   */ -static int link_schedule_port(struct link *l_ptr, u32 origport, u32 sz) +static int link_schedule_port(struct tipc_link *l_ptr, u32 origport, u32 sz)  {  	struct tipc_port *p_ptr; @@ -440,7 +442,7 @@ exit:  	return -ELINKCONG;  } -void tipc_link_wakeup_ports(struct link *l_ptr, int all) +void tipc_link_wakeup_ports(struct tipc_link *l_ptr, int all)  {  	struct tipc_port *p_ptr;  	struct tipc_port *temp_p_ptr; @@ -475,7 +477,7 @@ exit:   * @l_ptr: pointer to link   */ -static void link_release_outqueue(struct link *l_ptr) +static void link_release_outqueue(struct tipc_link *l_ptr)  {  	struct sk_buff *buf = l_ptr->first_out;  	struct sk_buff *next; @@ -494,7 +496,7 @@ static void link_release_outqueue(struct link *l_ptr)   * @l_ptr: pointer to link   */ -void tipc_link_reset_fragments(struct link *l_ptr) +void tipc_link_reset_fragments(struct tipc_link *l_ptr)  {  	struct sk_buff *buf = l_ptr->defragm_buf;  	struct sk_buff *next; @@ -512,7 +514,7 @@ void tipc_link_reset_fragments(struct link *l_ptr)   * @l_ptr: pointer to link   */ -void tipc_link_stop(struct link *l_ptr) +void tipc_link_stop(struct tipc_link *l_ptr)  {  	struct sk_buff *buf;  	struct sk_buff *next; @@ -537,7 +539,7 @@ void tipc_link_stop(struct link *l_ptr)  	l_ptr->proto_msg_queue = NULL;  } -void tipc_link_reset(struct link *l_ptr) +void tipc_link_reset(struct tipc_link *l_ptr)  {  	struct sk_buff *buf;  	u32 prev_state = l_ptr->state; @@ -597,7 +599,7 @@ void tipc_link_reset(struct link *l_ptr)  } -static void link_activate(struct link *l_ptr) +static void link_activate(struct tipc_link *l_ptr)  {  	l_ptr->next_in_no = l_ptr->stats.recv_info = 1;  	tipc_node_link_up(l_ptr->owner, l_ptr); @@ -610,9 +612,9 @@ static void link_activate(struct link *l_ptr)   * @event: state machine event to process   */ -static void link_state_event(struct link *l_ptr, unsigned event) +static void link_state_event(struct tipc_link *l_ptr, unsigned event)  { -	struct link *other; +	struct tipc_link *other;  	u32 cont_intv = l_ptr->continuity_interval;  	if (!l_ptr->started && (event != STARTING_EVT)) @@ -784,7 +786,7 @@ static void link_state_event(struct link *l_ptr, unsigned event)   * the tail of an existing one.   */ -static int link_bundle_buf(struct link *l_ptr, +static int link_bundle_buf(struct tipc_link *l_ptr,  			   struct sk_buff *bundler,  			   struct sk_buff *buf)  { @@ -813,7 +815,7 @@ static int link_bundle_buf(struct link *l_ptr,  	return 1;  } -static void link_add_to_outqueue(struct link *l_ptr, +static void link_add_to_outqueue(struct tipc_link *l_ptr,  				 struct sk_buff *buf,  				 struct tipc_msg *msg)  { @@ -834,7 +836,7 @@ static void link_add_to_outqueue(struct link *l_ptr,  		l_ptr->stats.max_queue_sz = l_ptr->out_queue_size;  } -static void link_add_chain_to_outqueue(struct link *l_ptr, +static void link_add_chain_to_outqueue(struct tipc_link *l_ptr,  				       struct sk_buff *buf_chain,  				       u32 long_msgno)  { @@ -859,7 +861,7 @@ static void link_add_chain_to_outqueue(struct link *l_ptr,   * has failed, and from link_send()   */ -int tipc_link_send_buf(struct link *l_ptr, struct sk_buff *buf) +int tipc_link_send_buf(struct tipc_link *l_ptr, struct sk_buff *buf)  {  	struct tipc_msg *msg = buf_msg(buf);  	u32 size = msg_size(msg); @@ -954,7 +956,7 @@ int tipc_link_send_buf(struct link *l_ptr, struct sk_buff *buf)  int tipc_link_send(struct sk_buff *buf, u32 dest, u32 selector)  { -	struct link *l_ptr; +	struct tipc_link *l_ptr;  	struct tipc_node *n_ptr;  	int res = -ELINKCONG; @@ -988,7 +990,7 @@ int tipc_link_send(struct sk_buff *buf, u32 dest, u32 selector)  void tipc_link_send_names(struct list_head *message_list, u32 dest)  {  	struct tipc_node *n_ptr; -	struct link *l_ptr; +	struct tipc_link *l_ptr;  	struct sk_buff *buf;  	struct sk_buff *temp_buf; @@ -1027,7 +1029,7 @@ void tipc_link_send_names(struct list_head *message_list, u32 dest)   * Link is locked. Returns user data length.   */ -static int link_send_buf_fast(struct link *l_ptr, struct sk_buff *buf, +static int link_send_buf_fast(struct tipc_link *l_ptr, struct sk_buff *buf,  			      u32 *used_max_pkt)  {  	struct tipc_msg *msg = buf_msg(buf); @@ -1061,7 +1063,7 @@ static int link_send_buf_fast(struct link *l_ptr, struct sk_buff *buf,   */  int tipc_send_buf_fast(struct sk_buff *buf, u32 destnode)  { -	struct link *l_ptr; +	struct tipc_link *l_ptr;  	struct tipc_node *n_ptr;  	int res;  	u32 selector = msg_origport(buf_msg(buf)) & 1; @@ -1100,7 +1102,7 @@ int tipc_link_send_sections_fast(struct tipc_port *sender,  				 u32 destaddr)  {  	struct tipc_msg *hdr = &sender->phdr; -	struct link *l_ptr; +	struct tipc_link *l_ptr;  	struct sk_buff *buf;  	struct tipc_node *node;  	int res; @@ -1195,7 +1197,7 @@ static int link_send_sections_long(struct tipc_port *sender,  				   unsigned int total_len,  				   u32 destaddr)  { -	struct link *l_ptr; +	struct tipc_link *l_ptr;  	struct tipc_node *node;  	struct tipc_msg *hdr = &sender->phdr;  	u32 dsz = total_len; @@ -1342,7 +1344,7 @@ reject:  /*   * tipc_link_push_packet: Push one unsent packet to the media   */ -u32 tipc_link_push_packet(struct link *l_ptr) +u32 tipc_link_push_packet(struct tipc_link *l_ptr)  {  	struct sk_buff *buf = l_ptr->first_out;  	u32 r_q_size = l_ptr->retransm_queue_size; @@ -1354,7 +1356,7 @@ u32 tipc_link_push_packet(struct link *l_ptr)  	if (r_q_size && buf) {  		u32 last = lesser(mod(r_q_head + r_q_size),  				  link_last_sent(l_ptr)); -		u32 first = msg_seqno(buf_msg(buf)); +		u32 first = buf_seqno(buf);  		while (buf && less(first, r_q_head)) {  			first = mod(first + 1); @@ -1403,7 +1405,7 @@ u32 tipc_link_push_packet(struct link *l_ptr)  	if (buf) {  		struct tipc_msg *msg = buf_msg(buf);  		u32 next = msg_seqno(msg); -		u32 first = msg_seqno(buf_msg(l_ptr->first_out)); +		u32 first = buf_seqno(l_ptr->first_out);  		if (mod(next - first) < l_ptr->queue_limit[0]) {  			msg_set_ack(msg, mod(l_ptr->next_in_no - 1)); @@ -1426,7 +1428,7 @@ u32 tipc_link_push_packet(struct link *l_ptr)   * push_queue(): push out the unsent messages of a link where   *               congestion has abated. Node is locked   */ -void tipc_link_push_queue(struct link *l_ptr) +void tipc_link_push_queue(struct tipc_link *l_ptr)  {  	u32 res; @@ -1470,7 +1472,8 @@ static void link_reset_all(unsigned long addr)  	read_unlock_bh(&tipc_net_lock);  } -static void link_retransmit_failure(struct link *l_ptr, struct sk_buff *buf) +static void link_retransmit_failure(struct tipc_link *l_ptr, +					struct sk_buff *buf)  {  	struct tipc_msg *msg = buf_msg(buf); @@ -1514,7 +1517,7 @@ static void link_retransmit_failure(struct link *l_ptr, struct sk_buff *buf)  	}  } -void tipc_link_retransmit(struct link *l_ptr, struct sk_buff *buf, +void tipc_link_retransmit(struct tipc_link *l_ptr, struct sk_buff *buf,  			  u32 retransmits)  {  	struct tipc_msg *msg; @@ -1558,7 +1561,7 @@ void tipc_link_retransmit(struct link *l_ptr, struct sk_buff *buf,  		} else {  			tipc_bearer_schedule(l_ptr->b_ptr, l_ptr);  			l_ptr->stats.bearer_congs++; -			l_ptr->retransm_queue_head = msg_seqno(buf_msg(buf)); +			l_ptr->retransm_queue_head = buf_seqno(buf);  			l_ptr->retransm_queue_size = retransmits;  			return;  		} @@ -1571,7 +1574,7 @@ void tipc_link_retransmit(struct link *l_ptr, struct sk_buff *buf,   * link_insert_deferred_queue - insert deferred messages back into receive chain   */ -static struct sk_buff *link_insert_deferred_queue(struct link *l_ptr, +static struct sk_buff *link_insert_deferred_queue(struct tipc_link *l_ptr,  						  struct sk_buff *buf)  {  	u32 seq_no; @@ -1579,7 +1582,7 @@ static struct sk_buff *link_insert_deferred_queue(struct link *l_ptr,  	if (l_ptr->oldest_deferred_in == NULL)  		return buf; -	seq_no = msg_seqno(buf_msg(l_ptr->oldest_deferred_in)); +	seq_no = buf_seqno(l_ptr->oldest_deferred_in);  	if (seq_no == mod(l_ptr->next_in_no)) {  		l_ptr->newest_deferred_in->next = buf;  		buf = l_ptr->oldest_deferred_in; @@ -1653,7 +1656,7 @@ void tipc_recv_msg(struct sk_buff *head, struct tipc_bearer *b_ptr)  	read_lock_bh(&tipc_net_lock);  	while (head) {  		struct tipc_node *n_ptr; -		struct link *l_ptr; +		struct tipc_link *l_ptr;  		struct sk_buff *crs;  		struct sk_buff *buf = head;  		struct tipc_msg *msg; @@ -1733,14 +1736,12 @@ void tipc_recv_msg(struct sk_buff *head, struct tipc_bearer *b_ptr)  		/* Release acked messages */ -		if (less(n_ptr->bclink.acked, msg_bcast_ack(msg))) { -			if (tipc_node_is_up(n_ptr) && n_ptr->bclink.supported) -				tipc_bclink_acknowledge(n_ptr, msg_bcast_ack(msg)); -		} +		if (tipc_node_is_up(n_ptr) && n_ptr->bclink.supported) +			tipc_bclink_acknowledge(n_ptr, msg_bcast_ack(msg));  		crs = l_ptr->first_out;  		while ((crs != l_ptr->next_out) && -		       less_eq(msg_seqno(buf_msg(crs)), ackd)) { +		       less_eq(buf_seqno(crs), ackd)) {  			struct sk_buff *next = crs->next;  			buf_discard(crs); @@ -1863,7 +1864,7 @@ u32 tipc_link_defer_pkt(struct sk_buff **head,  {  	struct sk_buff *prev = NULL;  	struct sk_buff *crs = *head; -	u32 seq_no = msg_seqno(buf_msg(buf)); +	u32 seq_no = buf_seqno(buf);  	buf->next = NULL; @@ -1874,7 +1875,7 @@ u32 tipc_link_defer_pkt(struct sk_buff **head,  	}  	/* Last ? */ -	if (less(msg_seqno(buf_msg(*tail)), seq_no)) { +	if (less(buf_seqno(*tail), seq_no)) {  		(*tail)->next = buf;  		*tail = buf;  		return 1; @@ -1908,10 +1909,10 @@ u32 tipc_link_defer_pkt(struct sk_buff **head,   * link_handle_out_of_seq_msg - handle arrival of out-of-sequence packet   */ -static void link_handle_out_of_seq_msg(struct link *l_ptr, +static void link_handle_out_of_seq_msg(struct tipc_link *l_ptr,  				       struct sk_buff *buf)  { -	u32 seq_no = msg_seqno(buf_msg(buf)); +	u32 seq_no = buf_seqno(buf);  	if (likely(msg_user(buf_msg(buf)) == LINK_PROTOCOL)) {  		link_recv_proto_msg(l_ptr, buf); @@ -1946,8 +1947,9 @@ static void link_handle_out_of_seq_msg(struct link *l_ptr,  /*   * Send protocol message to the other endpoint.   */ -void tipc_link_send_proto_msg(struct link *l_ptr, u32 msg_typ, int probe_msg, -			      u32 gap, u32 tolerance, u32 priority, u32 ack_mtu) +void tipc_link_send_proto_msg(struct tipc_link *l_ptr, u32 msg_typ, +				int probe_msg, u32 gap, u32 tolerance, +				u32 priority, u32 ack_mtu)  {  	struct sk_buff *buf = NULL;  	struct tipc_msg *msg = l_ptr->pmsg; @@ -1973,10 +1975,10 @@ void tipc_link_send_proto_msg(struct link *l_ptr, u32 msg_typ, int probe_msg,  		if (!tipc_link_is_up(l_ptr))  			return;  		if (l_ptr->next_out) -			next_sent = msg_seqno(buf_msg(l_ptr->next_out)); +			next_sent = buf_seqno(l_ptr->next_out);  		msg_set_next_sent(msg, next_sent);  		if (l_ptr->oldest_deferred_in) { -			u32 rec = msg_seqno(buf_msg(l_ptr->oldest_deferred_in)); +			u32 rec = buf_seqno(l_ptr->oldest_deferred_in);  			gap = mod(rec - mod(l_ptr->next_in_no));  		}  		msg_set_seq_gap(msg, gap); @@ -2064,7 +2066,7 @@ void tipc_link_send_proto_msg(struct link *l_ptr, u32 msg_typ, int probe_msg,   * change at any time. The node with lowest address rules   */ -static void link_recv_proto_msg(struct link *l_ptr, struct sk_buff *buf) +static void link_recv_proto_msg(struct tipc_link *l_ptr, struct sk_buff *buf)  {  	u32 rec_gap = 0;  	u32 max_pkt_info; @@ -2197,12 +2199,12 @@ exit:   * tipc_link_tunnel(): Send one message via a link belonging to   * another bearer. Owner node is locked.   */ -static void tipc_link_tunnel(struct link *l_ptr, +static void tipc_link_tunnel(struct tipc_link *l_ptr,  			     struct tipc_msg *tunnel_hdr,  			     struct tipc_msg  *msg,  			     u32 selector)  { -	struct link *tunnel; +	struct tipc_link *tunnel;  	struct sk_buff *buf;  	u32 length = msg_size(msg); @@ -2231,11 +2233,11 @@ static void tipc_link_tunnel(struct link *l_ptr,   *               Owner node is locked.   */ -void tipc_link_changeover(struct link *l_ptr) +void tipc_link_changeover(struct tipc_link *l_ptr)  {  	u32 msgcount = l_ptr->out_queue_size;  	struct sk_buff *crs = l_ptr->first_out; -	struct link *tunnel = l_ptr->owner->active_links[0]; +	struct tipc_link *tunnel = l_ptr->owner->active_links[0];  	struct tipc_msg tunnel_hdr;  	int split_bundles; @@ -2294,7 +2296,7 @@ void tipc_link_changeover(struct link *l_ptr)  	}  } -void tipc_link_send_duplicate(struct link *l_ptr, struct link *tunnel) +void tipc_link_send_duplicate(struct tipc_link *l_ptr, struct tipc_link *tunnel)  {  	struct sk_buff *iter;  	struct tipc_msg tunnel_hdr; @@ -2358,11 +2360,11 @@ static struct sk_buff *buf_extract(struct sk_buff *skb, u32 from_pos)   *  via other link. Node is locked. Return extracted buffer.   */ -static int link_recv_changeover_msg(struct link **l_ptr, +static int link_recv_changeover_msg(struct tipc_link **l_ptr,  				    struct sk_buff **buf)  {  	struct sk_buff *tunnel_buf = *buf; -	struct link *dest_link; +	struct tipc_link *dest_link;  	struct tipc_msg *msg;  	struct tipc_msg *tunnel_msg = buf_msg(tunnel_buf);  	u32 msg_typ = msg_type(tunnel_msg); @@ -2462,7 +2464,7 @@ void tipc_link_recv_bundle(struct sk_buff *buf)   * The buffer is complete, inclusive total message length.   * Returns user data length.   */ -static int link_send_long_buf(struct link *l_ptr, struct sk_buff *buf) +static int link_send_long_buf(struct tipc_link *l_ptr, struct sk_buff *buf)  {  	struct sk_buff *buf_chain = NULL;  	struct sk_buff *buf_chain_tail = (struct sk_buff *)&buf_chain; @@ -2591,7 +2593,7 @@ int tipc_link_recv_fragment(struct sk_buff **pending, struct sk_buff **fb,  	/* Is there an incomplete message waiting for this fragment? */ -	while (pbuf && ((msg_seqno(buf_msg(pbuf)) != long_msg_seq_no) || +	while (pbuf && ((buf_seqno(pbuf) != long_msg_seq_no) ||  			(msg_orignode(fragm) != msg_orignode(buf_msg(pbuf))))) {  		prev = pbuf;  		pbuf = pbuf->next; @@ -2658,7 +2660,7 @@ int tipc_link_recv_fragment(struct sk_buff **pending, struct sk_buff **fb,   * @l_ptr: pointer to link   */ -static void link_check_defragm_bufs(struct link *l_ptr) +static void link_check_defragm_bufs(struct tipc_link *l_ptr)  {  	struct sk_buff *prev = NULL;  	struct sk_buff *next = NULL; @@ -2688,7 +2690,7 @@ static void link_check_defragm_bufs(struct link *l_ptr) -static void link_set_supervision_props(struct link *l_ptr, u32 tolerance) +static void link_set_supervision_props(struct tipc_link *l_ptr, u32 tolerance)  {  	if ((tolerance < TIPC_MIN_LINK_TOL) || (tolerance > TIPC_MAX_LINK_TOL))  		return; @@ -2700,7 +2702,7 @@ static void link_set_supervision_props(struct link *l_ptr, u32 tolerance)  } -void tipc_link_set_queue_limits(struct link *l_ptr, u32 window) +void tipc_link_set_queue_limits(struct tipc_link *l_ptr, u32 window)  {  	/* Data messages from this node, inclusive FIRST_FRAGM */  	l_ptr->queue_limit[TIPC_LOW_IMPORTANCE] = window; @@ -2730,11 +2732,12 @@ void tipc_link_set_queue_limits(struct link *l_ptr, u32 window)   * Returns pointer to link (or 0 if invalid link name).   */ -static struct link *link_find_link(const char *name, struct tipc_node **node) +static struct tipc_link *link_find_link(const char *name, +					struct tipc_node **node)  { -	struct link_name link_name_parts; +	struct tipc_link_name link_name_parts;  	struct tipc_bearer *b_ptr; -	struct link *l_ptr; +	struct tipc_link *l_ptr;  	if (!link_name_validate(name, &link_name_parts))  		return NULL; @@ -2754,13 +2757,113 @@ static struct link *link_find_link(const char *name, struct tipc_node **node)  	return l_ptr;  } +/** + * link_value_is_valid -- validate proposed link tolerance/priority/window + * + * @cmd - value type (TIPC_CMD_SET_LINK_*) + * @new_value - the new value + * + * Returns 1 if value is within range, 0 if not. + */ + +static int link_value_is_valid(u16 cmd, u32 new_value) +{ +	switch (cmd) { +	case TIPC_CMD_SET_LINK_TOL: +		return (new_value >= TIPC_MIN_LINK_TOL) && +			(new_value <= TIPC_MAX_LINK_TOL); +	case TIPC_CMD_SET_LINK_PRI: +		return (new_value <= TIPC_MAX_LINK_PRI); +	case TIPC_CMD_SET_LINK_WINDOW: +		return (new_value >= TIPC_MIN_LINK_WIN) && +			(new_value <= TIPC_MAX_LINK_WIN); +	} +	return 0; +} + + +/** + * link_cmd_set_value - change priority/tolerance/window for link/bearer/media + * @name - ptr to link, bearer, or media name + * @new_value - new value of link, bearer, or media setting + * @cmd - which link, bearer, or media attribute to set (TIPC_CMD_SET_LINK_*) + * + * Caller must hold 'tipc_net_lock' to ensure link/bearer/media is not deleted. + * + * Returns 0 if value updated and negative value on error. + */ + +static int link_cmd_set_value(const char *name, u32 new_value, u16 cmd) +{ +	struct tipc_node *node; +	struct tipc_link *l_ptr; +	struct tipc_bearer *b_ptr; +	struct tipc_media *m_ptr; + +	l_ptr = link_find_link(name, &node); +	if (l_ptr) { +		/* +		 * acquire node lock for tipc_link_send_proto_msg(). +		 * see "TIPC locking policy" in net.c. +		 */ +		tipc_node_lock(node); +		switch (cmd) { +		case TIPC_CMD_SET_LINK_TOL: +			link_set_supervision_props(l_ptr, new_value); +			tipc_link_send_proto_msg(l_ptr, +				STATE_MSG, 0, 0, new_value, 0, 0); +			break; +		case TIPC_CMD_SET_LINK_PRI: +			l_ptr->priority = new_value; +			tipc_link_send_proto_msg(l_ptr, +				STATE_MSG, 0, 0, 0, new_value, 0); +			break; +		case TIPC_CMD_SET_LINK_WINDOW: +			tipc_link_set_queue_limits(l_ptr, new_value); +			break; +		} +		tipc_node_unlock(node); +		return 0; +	} + +	b_ptr = tipc_bearer_find(name); +	if (b_ptr) { +		switch (cmd) { +		case TIPC_CMD_SET_LINK_TOL: +			b_ptr->tolerance = new_value; +			return 0; +		case TIPC_CMD_SET_LINK_PRI: +			b_ptr->priority = new_value; +			return 0; +		case TIPC_CMD_SET_LINK_WINDOW: +			b_ptr->window = new_value; +			return 0; +		} +		return -EINVAL; +	} + +	m_ptr = tipc_media_find(name); +	if (!m_ptr) +		return -ENODEV; +	switch (cmd) { +	case TIPC_CMD_SET_LINK_TOL: +		m_ptr->tolerance = new_value; +		return 0; +	case TIPC_CMD_SET_LINK_PRI: +		m_ptr->priority = new_value; +		return 0; +	case TIPC_CMD_SET_LINK_WINDOW: +		m_ptr->window = new_value; +		return 0; +	} +	return -EINVAL; +} +  struct sk_buff *tipc_link_cmd_config(const void *req_tlv_area, int req_tlv_space,  				     u16 cmd)  {  	struct tipc_link_config *args;  	u32 new_value; -	struct link *l_ptr; -	struct tipc_node *node;  	int res;  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_LINK_CONFIG)) @@ -2769,6 +2872,10 @@ struct sk_buff *tipc_link_cmd_config(const void *req_tlv_area, int req_tlv_space  	args = (struct tipc_link_config *)TLV_DATA(req_tlv_area);  	new_value = ntohl(args->value); +	if (!link_value_is_valid(cmd, new_value)) +		return tipc_cfg_reply_error_string( +			"cannot change, value invalid"); +  	if (!strcmp(args->name, tipc_bclink_name)) {  		if ((cmd == TIPC_CMD_SET_LINK_WINDOW) &&  		    (tipc_bclink_set_queue_limits(new_value) == 0)) @@ -2778,43 +2885,7 @@ struct sk_buff *tipc_link_cmd_config(const void *req_tlv_area, int req_tlv_space  	}  	read_lock_bh(&tipc_net_lock); -	l_ptr = link_find_link(args->name, &node); -	if (!l_ptr) { -		read_unlock_bh(&tipc_net_lock); -		return tipc_cfg_reply_error_string("link not found"); -	} - -	tipc_node_lock(node); -	res = -EINVAL; -	switch (cmd) { -	case TIPC_CMD_SET_LINK_TOL: -		if ((new_value >= TIPC_MIN_LINK_TOL) && -		    (new_value <= TIPC_MAX_LINK_TOL)) { -			link_set_supervision_props(l_ptr, new_value); -			tipc_link_send_proto_msg(l_ptr, STATE_MSG, -						 0, 0, new_value, 0, 0); -			res = 0; -		} -		break; -	case TIPC_CMD_SET_LINK_PRI: -		if ((new_value >= TIPC_MIN_LINK_PRI) && -		    (new_value <= TIPC_MAX_LINK_PRI)) { -			l_ptr->priority = new_value; -			tipc_link_send_proto_msg(l_ptr, STATE_MSG, -						 0, 0, 0, new_value, 0); -			res = 0; -		} -		break; -	case TIPC_CMD_SET_LINK_WINDOW: -		if ((new_value >= TIPC_MIN_LINK_WIN) && -		    (new_value <= TIPC_MAX_LINK_WIN)) { -			tipc_link_set_queue_limits(l_ptr, new_value); -			res = 0; -		} -		break; -	} -	tipc_node_unlock(node); - +	res = link_cmd_set_value(args->name, new_value, cmd);  	read_unlock_bh(&tipc_net_lock);  	if (res)  		return tipc_cfg_reply_error_string("cannot change link setting"); @@ -2827,7 +2898,7 @@ struct sk_buff *tipc_link_cmd_config(const void *req_tlv_area, int req_tlv_space   * @l_ptr: pointer to link   */ -static void link_reset_statistics(struct link *l_ptr) +static void link_reset_statistics(struct tipc_link *l_ptr)  {  	memset(&l_ptr->stats, 0, sizeof(l_ptr->stats));  	l_ptr->stats.sent_info = l_ptr->next_out_no; @@ -2837,7 +2908,7 @@ static void link_reset_statistics(struct link *l_ptr)  struct sk_buff *tipc_link_cmd_reset_stats(const void *req_tlv_area, int req_tlv_space)  {  	char *link_name; -	struct link *l_ptr; +	struct tipc_link *l_ptr;  	struct tipc_node *node;  	if (!TLV_CHECK(req_tlv_area, req_tlv_space, TIPC_TLV_LINK_NAME)) @@ -2885,7 +2956,7 @@ static u32 percent(u32 count, u32 total)  static int tipc_link_stats(const char *name, char *buf, const u32 buf_size)  {  	struct print_buf pb; -	struct link *l_ptr; +	struct tipc_link *l_ptr;  	struct tipc_node *node;  	char *status;  	u32 profile_total = 0; @@ -3007,7 +3078,7 @@ struct sk_buff *tipc_link_cmd_show_stats(const void *req_tlv_area, int req_tlv_s  u32 tipc_link_get_max_pkt(u32 dest, u32 selector)  {  	struct tipc_node *n_ptr; -	struct link *l_ptr; +	struct tipc_link *l_ptr;  	u32 res = MAX_PKT_DEFAULT;  	if (dest == tipc_own_addr) @@ -3026,7 +3097,7 @@ u32 tipc_link_get_max_pkt(u32 dest, u32 selector)  	return res;  } -static void link_print(struct link *l_ptr, const char *str) +static void link_print(struct tipc_link *l_ptr, const char *str)  {  	char print_area[256];  	struct print_buf pb; @@ -3046,13 +3117,12 @@ static void link_print(struct link *l_ptr, const char *str)  	tipc_printf(buf, "NXI(%u):", mod(l_ptr->next_in_no));  	tipc_printf(buf, "SQUE");  	if (l_ptr->first_out) { -		tipc_printf(buf, "[%u..", msg_seqno(buf_msg(l_ptr->first_out))); +		tipc_printf(buf, "[%u..", buf_seqno(l_ptr->first_out));  		if (l_ptr->next_out) -			tipc_printf(buf, "%u..", -				    msg_seqno(buf_msg(l_ptr->next_out))); -		tipc_printf(buf, "%u]", msg_seqno(buf_msg(l_ptr->last_out))); -		if ((mod(msg_seqno(buf_msg(l_ptr->last_out)) - -			 msg_seqno(buf_msg(l_ptr->first_out))) +			tipc_printf(buf, "%u..", buf_seqno(l_ptr->next_out)); +		tipc_printf(buf, "%u]", buf_seqno(l_ptr->last_out)); +		if ((mod(buf_seqno(l_ptr->last_out) - +			 buf_seqno(l_ptr->first_out))  		     != (l_ptr->out_queue_size - 1)) ||  		    (l_ptr->last_out->next != NULL)) {  			tipc_printf(buf, "\nSend queue inconsistency\n"); @@ -3064,8 +3134,8 @@ static void link_print(struct link *l_ptr, const char *str)  		tipc_printf(buf, "[]");  	tipc_printf(buf, "SQSIZ(%u)", l_ptr->out_queue_size);  	if (l_ptr->oldest_deferred_in) { -		u32 o = msg_seqno(buf_msg(l_ptr->oldest_deferred_in)); -		u32 n = msg_seqno(buf_msg(l_ptr->newest_deferred_in)); +		u32 o = buf_seqno(l_ptr->oldest_deferred_in); +		u32 n = buf_seqno(l_ptr->newest_deferred_in);  		tipc_printf(buf, ":RQUE[%u..%u]", o, n);  		if (l_ptr->deferred_inqueue_sz != mod((n + 1) - o)) {  			tipc_printf(buf, ":RQSIZ(%u)", diff --git a/net/tipc/link.h b/net/tipc/link.h index e56cb532913..73c18c140e1 100644 --- a/net/tipc/link.h +++ b/net/tipc/link.h @@ -45,6 +45,12 @@  #define PUSH_FINISHED 2  /* + * Out-of-range value for link sequence numbers + */ + +#define INVALID_LINK_SEQ 0x10000 + +/*   * Link states   */ @@ -61,7 +67,7 @@  #define MAX_PKT_DEFAULT 1500  /** - * struct link - TIPC link data structure + * struct tipc_link - TIPC link data structure   * @addr: network address of link's peer node   * @name: link name character string   * @media_addr: media address to use when sending messages over link @@ -109,7 +115,7 @@   * @stats: collects statistics regarding link activity   */ -struct link { +struct tipc_link {  	u32 addr;  	char name[TIPC_MAX_LINK_NAME];  	struct tipc_media_addr media_addr; @@ -207,24 +213,24 @@ struct link {  struct tipc_port; -struct link *tipc_link_create(struct tipc_node *n_ptr, +struct tipc_link *tipc_link_create(struct tipc_node *n_ptr,  			      struct tipc_bearer *b_ptr,  			      const struct tipc_media_addr *media_addr); -void tipc_link_delete(struct link *l_ptr); -void tipc_link_changeover(struct link *l_ptr); -void tipc_link_send_duplicate(struct link *l_ptr, struct link *dest); -void tipc_link_reset_fragments(struct link *l_ptr); -int tipc_link_is_up(struct link *l_ptr); -int tipc_link_is_active(struct link *l_ptr); -u32 tipc_link_push_packet(struct link *l_ptr); -void tipc_link_stop(struct link *l_ptr); +void tipc_link_delete(struct tipc_link *l_ptr); +void tipc_link_changeover(struct tipc_link *l_ptr); +void tipc_link_send_duplicate(struct tipc_link *l_ptr, struct tipc_link *dest); +void tipc_link_reset_fragments(struct tipc_link *l_ptr); +int tipc_link_is_up(struct tipc_link *l_ptr); +int tipc_link_is_active(struct tipc_link *l_ptr); +u32 tipc_link_push_packet(struct tipc_link *l_ptr); +void tipc_link_stop(struct tipc_link *l_ptr);  struct sk_buff *tipc_link_cmd_config(const void *req_tlv_area, int req_tlv_space, u16 cmd);  struct sk_buff *tipc_link_cmd_show_stats(const void *req_tlv_area, int req_tlv_space);  struct sk_buff *tipc_link_cmd_reset_stats(const void *req_tlv_area, int req_tlv_space); -void tipc_link_reset(struct link *l_ptr); +void tipc_link_reset(struct tipc_link *l_ptr);  int tipc_link_send(struct sk_buff *buf, u32 dest, u32 selector);  void tipc_link_send_names(struct list_head *message_list, u32 dest); -int tipc_link_send_buf(struct link *l_ptr, struct sk_buff *buf); +int tipc_link_send_buf(struct tipc_link *l_ptr, struct sk_buff *buf);  u32 tipc_link_get_max_pkt(u32 dest, u32 selector);  int tipc_link_send_sections_fast(struct tipc_port *sender,  				 struct iovec const *msg_sect, @@ -235,19 +241,26 @@ void tipc_link_recv_bundle(struct sk_buff *buf);  int  tipc_link_recv_fragment(struct sk_buff **pending,  			     struct sk_buff **fb,  			     struct tipc_msg **msg); -void tipc_link_send_proto_msg(struct link *l_ptr, u32 msg_typ, int prob, u32 gap, -			      u32 tolerance, u32 priority, u32 acked_mtu); -void tipc_link_push_queue(struct link *l_ptr); +void tipc_link_send_proto_msg(struct tipc_link *l_ptr, u32 msg_typ, int prob, +			      u32 gap, u32 tolerance, u32 priority, +			      u32 acked_mtu); +void tipc_link_push_queue(struct tipc_link *l_ptr);  u32 tipc_link_defer_pkt(struct sk_buff **head, struct sk_buff **tail,  		   struct sk_buff *buf); -void tipc_link_wakeup_ports(struct link *l_ptr, int all); -void tipc_link_set_queue_limits(struct link *l_ptr, u32 window); -void tipc_link_retransmit(struct link *l_ptr, struct sk_buff *start, u32 retransmits); +void tipc_link_wakeup_ports(struct tipc_link *l_ptr, int all); +void tipc_link_set_queue_limits(struct tipc_link *l_ptr, u32 window); +void tipc_link_retransmit(struct tipc_link *l_ptr, +			  struct sk_buff *start, u32 retransmits);  /*   * Link sequence number manipulation routines (uses modulo 2**16 arithmetic)   */ +static inline u32 buf_seqno(struct sk_buff *buf) +{ +	return msg_seqno(buf_msg(buf)); +} +  static inline u32 mod(u32 x)  {  	return x & 0xffffu; @@ -282,32 +295,32 @@ static inline u32 lesser(u32 left, u32 right)   * Link status checking routines   */ -static inline int link_working_working(struct link *l_ptr) +static inline int link_working_working(struct tipc_link *l_ptr)  {  	return l_ptr->state == WORKING_WORKING;  } -static inline int link_working_unknown(struct link *l_ptr) +static inline int link_working_unknown(struct tipc_link *l_ptr)  {  	return l_ptr->state == WORKING_UNKNOWN;  } -static inline int link_reset_unknown(struct link *l_ptr) +static inline int link_reset_unknown(struct tipc_link *l_ptr)  {  	return l_ptr->state == RESET_UNKNOWN;  } -static inline int link_reset_reset(struct link *l_ptr) +static inline int link_reset_reset(struct tipc_link *l_ptr)  {  	return l_ptr->state == RESET_RESET;  } -static inline int link_blocked(struct link *l_ptr) +static inline int link_blocked(struct tipc_link *l_ptr)  {  	return l_ptr->exp_msg_count || l_ptr->blocked;  } -static inline int link_congested(struct link *l_ptr) +static inline int link_congested(struct tipc_link *l_ptr)  {  	return l_ptr->out_queue_size >= l_ptr->queue_limit[0];  } diff --git a/net/tipc/msg.c b/net/tipc/msg.c index 83d50967910..3e4d3e29be6 100644 --- a/net/tipc/msg.c +++ b/net/tipc/msg.c @@ -333,11 +333,14 @@ void tipc_msg_dbg(struct print_buf *buf, struct tipc_msg *msg, const char *str)  	}  	if (msg_user(msg) ==  LINK_CONFIG) { -		u32 *raw = (u32 *)msg; -		struct tipc_media_addr *orig = (struct tipc_media_addr *)&raw[5]; +		struct tipc_media_addr orig; +  		tipc_printf(buf, ":DDOM(%x):", msg_dest_domain(msg));  		tipc_printf(buf, ":NETID(%u):", msg_bc_netid(msg)); -		tipc_media_addr_printf(buf, orig); +		memcpy(orig.value, msg_media_addr(msg), sizeof(orig.value)); +		orig.media_id = 0; +		orig.broadcast = 0; +		tipc_media_addr_printf(buf, &orig);  	}  	if (msg_user(msg) == BCAST_PROTOCOL) {  		tipc_printf(buf, "BCNACK:AFTER(%u):", msg_bcgap_after(msg)); diff --git a/net/tipc/msg.h b/net/tipc/msg.h index d93178f2e85..7b0cda16710 100644 --- a/net/tipc/msg.h +++ b/net/tipc/msg.h @@ -78,6 +78,8 @@  #define MAX_MSG_SIZE (MAX_H_SIZE + TIPC_MAX_USER_MSG_SIZE) +#define TIPC_MEDIA_ADDR_OFFSET	5 +  struct tipc_msg {  	__be32 hdr[15]; @@ -682,6 +684,10 @@ static inline void msg_set_redundant_link(struct tipc_msg *m, u32 r)  	msg_set_bits(m, 5, 12, 0x1, r);  } +static inline char *msg_media_addr(struct tipc_msg *m) +{ +	return (char *)&m->hdr[TIPC_MEDIA_ADDR_OFFSET]; +}  /*   * Word 9 @@ -734,14 +740,4 @@ int tipc_msg_build(struct tipc_msg *hdr, struct iovec const *msg_sect,  		   u32 num_sect, unsigned int total_len,  			    int max_size, int usrmem, struct sk_buff **buf); -static inline void msg_set_media_addr(struct tipc_msg *m, struct tipc_media_addr *a) -{ -	memcpy(&((int *)m)[5], a, sizeof(*a)); -} - -static inline void msg_get_media_addr(struct tipc_msg *m, struct tipc_media_addr *a) -{ -	memcpy(a, &((int *)m)[5], sizeof(*a)); -} -  #endif diff --git a/net/tipc/name_distr.c b/net/tipc/name_distr.c index b7ca1bd7b15..98ebb37f180 100644 --- a/net/tipc/name_distr.c +++ b/net/tipc/name_distr.c @@ -176,7 +176,7 @@ void tipc_named_withdraw(struct publication *publ)  void tipc_named_node_up(unsigned long nodearg)  {  	struct tipc_node *n_ptr; -	struct link *l_ptr; +	struct tipc_link *l_ptr;  	struct publication *publ;  	struct distr_item *item = NULL;  	struct sk_buff *buf = NULL; @@ -322,10 +322,9 @@ void tipc_named_recv(struct sk_buff *buf)  /**   * tipc_named_reinit - re-initialize local publication list   * - * This routine is called whenever TIPC networking is (re)enabled. + * This routine is called whenever TIPC networking is enabled.   * All existing publications by this node that have "cluster" or "zone" scope - * are updated to reflect the node's current network address. - * (If the node's address is unchanged, the update loop terminates immediately.) + * are updated to reflect the node's new network address.   */  void tipc_named_reinit(void) @@ -333,10 +332,9 @@ void tipc_named_reinit(void)  	struct publication *publ;  	write_lock_bh(&tipc_nametbl_lock); -	list_for_each_entry(publ, &publ_root, local_list) { -		if (publ->node == tipc_own_addr) -			break; + +	list_for_each_entry(publ, &publ_root, local_list)  		publ->node = tipc_own_addr; -	} +  	write_unlock_bh(&tipc_nametbl_lock);  } diff --git a/net/tipc/name_table.c b/net/tipc/name_table.c index 46e6b6c2ecc..89eb5621ebb 100644 --- a/net/tipc/name_table.c +++ b/net/tipc/name_table.c @@ -251,8 +251,8 @@ static struct publication *tipc_nameseq_insert_publ(struct name_seq *nseq,  						    u32 type, u32 lower, u32 upper,  						    u32 scope, u32 node, u32 port, u32 key)  { -	struct subscription *s; -	struct subscription *st; +	struct tipc_subscription *s; +	struct tipc_subscription *st;  	struct publication *publ;  	struct sub_seq *sseq;  	struct name_info *info; @@ -381,7 +381,7 @@ static struct publication *tipc_nameseq_remove_publ(struct name_seq *nseq, u32 i  	struct sub_seq *sseq = nameseq_find_subseq(nseq, inst);  	struct name_info *info;  	struct sub_seq *free; -	struct subscription *s, *st; +	struct tipc_subscription *s, *st;  	int removed_subseq = 0;  	if (!sseq) @@ -448,7 +448,8 @@ found:   * sequence overlapping with the requested sequence   */ -static void tipc_nameseq_subscribe(struct name_seq *nseq, struct subscription *s) +static void tipc_nameseq_subscribe(struct name_seq *nseq, +					struct tipc_subscription *s)  {  	struct sub_seq *sseq = nseq->sseqs; @@ -625,7 +626,7 @@ not_found:   */  int tipc_nametbl_mc_translate(u32 type, u32 lower, u32 upper, u32 limit, -			      struct port_list *dports) +			      struct tipc_port_list *dports)  {  	struct name_seq *seq;  	struct sub_seq *sseq; @@ -739,7 +740,7 @@ int tipc_nametbl_withdraw(u32 type, u32 lower, u32 ref, u32 key)   * tipc_nametbl_subscribe - add a subscription object to the name table   */ -void tipc_nametbl_subscribe(struct subscription *s) +void tipc_nametbl_subscribe(struct tipc_subscription *s)  {  	u32 type = s->seq.type;  	struct name_seq *seq; @@ -763,7 +764,7 @@ void tipc_nametbl_subscribe(struct subscription *s)   * tipc_nametbl_unsubscribe - remove a subscription object from name table   */ -void tipc_nametbl_unsubscribe(struct subscription *s) +void tipc_nametbl_unsubscribe(struct tipc_subscription *s)  {  	struct name_seq *seq; diff --git a/net/tipc/name_table.h b/net/tipc/name_table.h index 62d77e5e902..8086b42f92a 100644 --- a/net/tipc/name_table.h +++ b/net/tipc/name_table.h @@ -39,8 +39,8 @@  #include "node_subscr.h" -struct subscription; -struct port_list; +struct tipc_subscription; +struct tipc_port_list;  /*   * TIPC name types reserved for internal TIPC use (both current and planned) @@ -90,7 +90,7 @@ extern rwlock_t tipc_nametbl_lock;  struct sk_buff *tipc_nametbl_get(const void *req_tlv_area, int req_tlv_space);  u32 tipc_nametbl_translate(u32 type, u32 instance, u32 *node);  int tipc_nametbl_mc_translate(u32 type, u32 lower, u32 upper, u32 limit, -			 struct port_list *dports); +			 struct tipc_port_list *dports);  int tipc_nametbl_publish_rsv(u32 ref, unsigned int scope,  			struct tipc_name_seq const *seq);  struct publication *tipc_nametbl_publish(u32 type, u32 lower, u32 upper, @@ -100,8 +100,8 @@ struct publication *tipc_nametbl_insert_publ(u32 type, u32 lower, u32 upper,  					u32 scope, u32 node, u32 ref, u32 key);  struct publication *tipc_nametbl_remove_publ(u32 type, u32 lower,  					u32 node, u32 ref, u32 key); -void tipc_nametbl_subscribe(struct subscription *s); -void tipc_nametbl_unsubscribe(struct subscription *s); +void tipc_nametbl_subscribe(struct tipc_subscription *s); +void tipc_nametbl_unsubscribe(struct tipc_subscription *s);  int tipc_nametbl_init(void);  void tipc_nametbl_stop(void); diff --git a/net/tipc/net.c b/net/tipc/net.c index fafef6c3c0f..61afee7e829 100644 --- a/net/tipc/net.c +++ b/net/tipc/net.c @@ -174,7 +174,6 @@ void tipc_net_route_msg(struct sk_buff *buf)  int tipc_net_start(u32 addr)  {  	char addr_string[16]; -	int res;  	if (tipc_mode != TIPC_NODE_MODE)  		return -ENOPROTOOPT; @@ -187,9 +186,7 @@ int tipc_net_start(u32 addr)  	tipc_named_reinit();  	tipc_port_reinit(); -	res = tipc_bclink_init(); -	if (res) -		return res; +	tipc_bclink_init();  	tipc_k_signal((Handler)tipc_subscr_start, 0);  	tipc_k_signal((Handler)tipc_cfg_init, 0); @@ -207,8 +204,8 @@ void tipc_net_stop(void)  	if (tipc_mode != TIPC_NET_MODE)  		return;  	write_lock_bh(&tipc_net_lock); -	tipc_bearer_stop();  	tipc_mode = TIPC_NODE_MODE; +	tipc_bearer_stop();  	tipc_bclink_stop();  	list_for_each_entry_safe(node, t_node, &tipc_node_list, list)  		tipc_node_delete(node); diff --git a/net/tipc/node.c b/net/tipc/node.c index 27b4bb0cca6..6b226faad89 100644 --- a/net/tipc/node.c +++ b/net/tipc/node.c @@ -136,9 +136,9 @@ void tipc_node_delete(struct tipc_node *n_ptr)   * Link becomes active (alone or shared) or standby, depending on its priority.   */ -void tipc_node_link_up(struct tipc_node *n_ptr, struct link *l_ptr) +void tipc_node_link_up(struct tipc_node *n_ptr, struct tipc_link *l_ptr)  { -	struct link **active = &n_ptr->active_links[0]; +	struct tipc_link **active = &n_ptr->active_links[0];  	n_ptr->working_links++; @@ -171,14 +171,14 @@ void tipc_node_link_up(struct tipc_node *n_ptr, struct link *l_ptr)  static void node_select_active_links(struct tipc_node *n_ptr)  { -	struct link **active = &n_ptr->active_links[0]; +	struct tipc_link **active = &n_ptr->active_links[0];  	u32 i;  	u32 highest_prio = 0;  	active[0] = active[1] = NULL;  	for (i = 0; i < MAX_BEARERS; i++) { -		struct link *l_ptr = n_ptr->links[i]; +		struct tipc_link *l_ptr = n_ptr->links[i];  		if (!l_ptr || !tipc_link_is_up(l_ptr) ||  		    (l_ptr->priority < highest_prio)) @@ -197,9 +197,9 @@ static void node_select_active_links(struct tipc_node *n_ptr)   * tipc_node_link_down - handle loss of link   */ -void tipc_node_link_down(struct tipc_node *n_ptr, struct link *l_ptr) +void tipc_node_link_down(struct tipc_node *n_ptr, struct tipc_link *l_ptr)  { -	struct link **active; +	struct tipc_link **active;  	n_ptr->working_links--; @@ -239,14 +239,14 @@ int tipc_node_is_up(struct tipc_node *n_ptr)  	return tipc_node_active_links(n_ptr);  } -void tipc_node_attach_link(struct tipc_node *n_ptr, struct link *l_ptr) +void tipc_node_attach_link(struct tipc_node *n_ptr, struct tipc_link *l_ptr)  {  	n_ptr->links[l_ptr->b_ptr->identity] = l_ptr;  	atomic_inc(&tipc_num_links);  	n_ptr->link_cnt++;  } -void tipc_node_detach_link(struct tipc_node *n_ptr, struct link *l_ptr) +void tipc_node_detach_link(struct tipc_node *n_ptr, struct tipc_link *l_ptr)  {  	n_ptr->links[l_ptr->b_ptr->identity] = NULL;  	atomic_dec(&tipc_num_links); @@ -307,7 +307,7 @@ static void node_established_contact(struct tipc_node *n_ptr)  	n_ptr->bclink.acked = tipc_bclink_get_last_sent();  	if (n_ptr->bclink.supported) { -		tipc_nmap_add(&tipc_bcast_nmap, n_ptr->addr); +		tipc_bclink_add_node(n_ptr->addr);  		if (n_ptr->addr < tipc_own_addr)  			tipc_own_tag++;  	} @@ -350,9 +350,8 @@ static void node_lost_contact(struct tipc_node *n_ptr)  			n_ptr->bclink.defragm = NULL;  		} -		tipc_nmap_remove(&tipc_bcast_nmap, n_ptr->addr); -		tipc_bclink_acknowledge(n_ptr, -					mod(n_ptr->bclink.acked + 10000)); +		tipc_bclink_remove_node(n_ptr->addr); +		tipc_bclink_acknowledge(n_ptr, INVALID_LINK_SEQ);  		if (n_ptr->addr < tipc_own_addr)  			tipc_own_tag--; @@ -361,7 +360,7 @@ static void node_lost_contact(struct tipc_node *n_ptr)  	/* Abort link changeover */  	for (i = 0; i < MAX_BEARERS; i++) { -		struct link *l_ptr = n_ptr->links[i]; +		struct tipc_link *l_ptr = n_ptr->links[i];  		if (!l_ptr)  			continue;  		l_ptr->reset_checkpoint = l_ptr->next_in_no; diff --git a/net/tipc/node.h b/net/tipc/node.h index 4f15cb40aaa..0b1c5f8b699 100644 --- a/net/tipc/node.h +++ b/net/tipc/node.h @@ -79,8 +79,8 @@ struct tipc_node {  	struct hlist_node hash;  	struct list_head list;  	struct list_head nsub; -	struct link *active_links[2]; -	struct link *links[MAX_BEARERS]; +	struct tipc_link *active_links[2]; +	struct tipc_link *links[MAX_BEARERS];  	int link_cnt;  	int working_links;  	int block_setup; @@ -117,10 +117,10 @@ extern u32 tipc_own_tag;  struct tipc_node *tipc_node_find(u32 addr);  struct tipc_node *tipc_node_create(u32 addr);  void tipc_node_delete(struct tipc_node *n_ptr); -void tipc_node_attach_link(struct tipc_node *n_ptr, struct link *l_ptr); -void tipc_node_detach_link(struct tipc_node *n_ptr, struct link *l_ptr); -void tipc_node_link_down(struct tipc_node *n_ptr, struct link *l_ptr); -void tipc_node_link_up(struct tipc_node *n_ptr, struct link *l_ptr); +void tipc_node_attach_link(struct tipc_node *n_ptr, struct tipc_link *l_ptr); +void tipc_node_detach_link(struct tipc_node *n_ptr, struct tipc_link *l_ptr); +void tipc_node_link_down(struct tipc_node *n_ptr, struct tipc_link *l_ptr); +void tipc_node_link_up(struct tipc_node *n_ptr, struct tipc_link *l_ptr);  int tipc_node_active_links(struct tipc_node *n_ptr);  int tipc_node_redundant_links(struct tipc_node *n_ptr);  int tipc_node_is_up(struct tipc_node *n_ptr); diff --git a/net/tipc/port.c b/net/tipc/port.c index 54d812a5a4d..d91efc69e6f 100644 --- a/net/tipc/port.c +++ b/net/tipc/port.c @@ -80,7 +80,7 @@ int tipc_multicast(u32 ref, struct tipc_name_seq const *seq,  	struct tipc_msg *hdr;  	struct sk_buff *buf;  	struct sk_buff *ibuf = NULL; -	struct port_list dports = {0, NULL, }; +	struct tipc_port_list dports = {0, NULL, };  	struct tipc_port *oport = tipc_port_deref(ref);  	int ext_targets;  	int res; @@ -142,11 +142,11 @@ int tipc_multicast(u32 ref, struct tipc_name_seq const *seq,   * If there is no port list, perform a lookup to create one   */ -void tipc_port_recv_mcast(struct sk_buff *buf, struct port_list *dp) +void tipc_port_recv_mcast(struct sk_buff *buf, struct tipc_port_list *dp)  {  	struct tipc_msg *msg; -	struct port_list dports = {0, NULL, }; -	struct port_list *item = dp; +	struct tipc_port_list dports = {0, NULL, }; +	struct tipc_port_list *item = dp;  	int cnt = 0;  	msg = buf_msg(buf); diff --git a/net/tipc/port.h b/net/tipc/port.h index b9aa34195ae..f751807e2a9 100644 --- a/net/tipc/port.h +++ b/net/tipc/port.h @@ -151,7 +151,7 @@ struct tipc_port {  };  extern spinlock_t tipc_port_list_lock; -struct port_list; +struct tipc_port_list;  /*   * TIPC port manipulation routines @@ -228,7 +228,7 @@ int tipc_port_reject_sections(struct tipc_port *p_ptr, struct tipc_msg *hdr,  			      unsigned int total_len, int err);  struct sk_buff *tipc_port_get_ports(void);  void tipc_port_recv_proto_msg(struct sk_buff *buf); -void tipc_port_recv_mcast(struct sk_buff *buf, struct port_list *dp); +void tipc_port_recv_mcast(struct sk_buff *buf, struct tipc_port_list *dp);  void tipc_port_reinit(void);  /** diff --git a/net/tipc/ref.c b/net/tipc/ref.c index 83116892528..9e37b7812c3 100644 --- a/net/tipc/ref.c +++ b/net/tipc/ref.c @@ -110,8 +110,7 @@ int tipc_ref_table_init(u32 requested_size, u32 start)  	/* allocate table & mark all entries as uninitialized */ -	table = __vmalloc(actual_size * sizeof(struct reference), -			  GFP_KERNEL | __GFP_HIGHMEM | __GFP_ZERO, PAGE_KERNEL); +	table = vzalloc(actual_size * sizeof(struct reference));  	if (table == NULL)  		return -ENOMEM; diff --git a/net/tipc/socket.c b/net/tipc/socket.c index 42b8324ff2e..e2f7c5d370b 100644 --- a/net/tipc/socket.c +++ b/net/tipc/socket.c @@ -185,9 +185,6 @@ static int tipc_create(struct net *net, struct socket *sock, int protocol,  	/* Validate arguments */ -	if (!net_eq(net, &init_net)) -		return -EAFNOSUPPORT; -  	if (unlikely(protocol != 0))  		return -EPROTONOSUPPORT; diff --git a/net/tipc/subscr.c b/net/tipc/subscr.c index 198371723b4..8c49566da8f 100644 --- a/net/tipc/subscr.c +++ b/net/tipc/subscr.c @@ -40,14 +40,14 @@  #include "subscr.h"  /** - * struct subscriber - TIPC network topology subscriber + * struct tipc_subscriber - TIPC network topology subscriber   * @port_ref: object reference to server port connecting to subscriber   * @lock: pointer to spinlock controlling access to subscriber's server port   * @subscriber_list: adjacent subscribers in top. server's list of subscribers   * @subscription_list: list of subscription objects for this subscriber   */ -struct subscriber { +struct tipc_subscriber {  	u32 port_ref;  	spinlock_t *lock;  	struct list_head subscriber_list; @@ -92,7 +92,7 @@ static u32 htohl(u32 in, int swap)   *       try to take the lock if the message is rejected and returned!   */ -static void subscr_send_event(struct subscription *sub, +static void subscr_send_event(struct tipc_subscription *sub,  			      u32 found_lower,  			      u32 found_upper,  			      u32 event, @@ -118,7 +118,7 @@ static void subscr_send_event(struct subscription *sub,   * Returns 1 if there is overlap, otherwise 0.   */ -int tipc_subscr_overlap(struct subscription *sub, +int tipc_subscr_overlap(struct tipc_subscription *sub,  			u32 found_lower,  			u32 found_upper) @@ -138,7 +138,7 @@ int tipc_subscr_overlap(struct subscription *sub,   * Protected by nameseq.lock in name_table.c   */ -void tipc_subscr_report_overlap(struct subscription *sub, +void tipc_subscr_report_overlap(struct tipc_subscription *sub,  				u32 found_lower,  				u32 found_upper,  				u32 event, @@ -158,7 +158,7 @@ void tipc_subscr_report_overlap(struct subscription *sub,   * subscr_timeout - subscription timeout has occurred   */ -static void subscr_timeout(struct subscription *sub) +static void subscr_timeout(struct tipc_subscription *sub)  {  	struct tipc_port *server_port; @@ -205,7 +205,7 @@ static void subscr_timeout(struct subscription *sub)   * Called with subscriber port locked.   */ -static void subscr_del(struct subscription *sub) +static void subscr_del(struct tipc_subscription *sub)  {  	tipc_nametbl_unsubscribe(sub);  	list_del(&sub->subscription_list); @@ -224,11 +224,11 @@ static void subscr_del(struct subscription *sub)   * simply wait for it to be released, then claim it.)   */ -static void subscr_terminate(struct subscriber *subscriber) +static void subscr_terminate(struct tipc_subscriber *subscriber)  {  	u32 port_ref; -	struct subscription *sub; -	struct subscription *sub_temp; +	struct tipc_subscription *sub; +	struct tipc_subscription *sub_temp;  	/* Invalidate subscriber reference */ @@ -278,10 +278,10 @@ static void subscr_terminate(struct subscriber *subscriber)   */  static void subscr_cancel(struct tipc_subscr *s, -			  struct subscriber *subscriber) +			  struct tipc_subscriber *subscriber)  { -	struct subscription *sub; -	struct subscription *sub_temp; +	struct tipc_subscription *sub; +	struct tipc_subscription *sub_temp;  	int found = 0;  	/* Find first matching subscription, exit if not found */ @@ -314,10 +314,10 @@ static void subscr_cancel(struct tipc_subscr *s,   * Called with subscriber port locked.   */ -static struct subscription *subscr_subscribe(struct tipc_subscr *s, -					     struct subscriber *subscriber) +static struct tipc_subscription *subscr_subscribe(struct tipc_subscr *s, +					     struct tipc_subscriber *subscriber)  { -	struct subscription *sub; +	struct tipc_subscription *sub;  	int swap;  	/* Determine subscriber's endianness */ @@ -393,7 +393,7 @@ static void subscr_conn_shutdown_event(void *usr_handle,  				       unsigned int size,  				       int reason)  { -	struct subscriber *subscriber = usr_handle; +	struct tipc_subscriber *subscriber = usr_handle;  	spinlock_t *subscriber_lock;  	if (tipc_port_lock(port_ref) == NULL) @@ -416,9 +416,9 @@ static void subscr_conn_msg_event(void *usr_handle,  				  const unchar *data,  				  u32 size)  { -	struct subscriber *subscriber = usr_handle; +	struct tipc_subscriber *subscriber = usr_handle;  	spinlock_t *subscriber_lock; -	struct subscription *sub; +	struct tipc_subscription *sub;  	/*  	 * Lock subscriber's server port (& make a local copy of lock pointer, @@ -471,12 +471,12 @@ static void subscr_named_msg_event(void *usr_handle,  				   struct tipc_portid const *orig,  				   struct tipc_name_seq const *dest)  { -	struct subscriber *subscriber; +	struct tipc_subscriber *subscriber;  	u32 server_port_ref;  	/* Create subscriber object */ -	subscriber = kzalloc(sizeof(struct subscriber), GFP_ATOMIC); +	subscriber = kzalloc(sizeof(struct tipc_subscriber), GFP_ATOMIC);  	if (subscriber == NULL) {  		warn("Subscriber rejected, no memory\n");  		return; @@ -568,8 +568,8 @@ failed:  void tipc_subscr_stop(void)  { -	struct subscriber *subscriber; -	struct subscriber *subscriber_temp; +	struct tipc_subscriber *subscriber; +	struct tipc_subscriber *subscriber_temp;  	spinlock_t *subscriber_lock;  	if (topsrv.setup_port) { diff --git a/net/tipc/subscr.h b/net/tipc/subscr.h index 4b06ef6f840..ef6529c8456 100644 --- a/net/tipc/subscr.h +++ b/net/tipc/subscr.h @@ -37,10 +37,10 @@  #ifndef _TIPC_SUBSCR_H  #define _TIPC_SUBSCR_H -struct subscription; +struct tipc_subscription;  /** - * struct subscription - TIPC network topology subscription object + * struct tipc_subscription - TIPC network topology subscription object   * @seq: name sequence associated with subscription   * @timeout: duration of subscription (in ms)   * @filter: event filtering to be done for subscription @@ -52,7 +52,7 @@ struct subscription;   * @evt: template for events generated by subscription   */ -struct subscription { +struct tipc_subscription {  	struct tipc_name_seq seq;  	u32 timeout;  	u32 filter; @@ -64,11 +64,11 @@ struct subscription {  	struct tipc_event evt;  }; -int tipc_subscr_overlap(struct subscription *sub, +int tipc_subscr_overlap(struct tipc_subscription *sub,  			u32 found_lower,  			u32 found_upper); -void tipc_subscr_report_overlap(struct subscription *sub, +void tipc_subscr_report_overlap(struct tipc_subscription *sub,  				u32 found_lower,  				u32 found_upper,  				u32 event,  |