Commit 2ff9f924 authored by Allan Stephens's avatar Allan Stephens Committed by Paul Gortmaker

tipc: Cosmetic changes to broadcast bearer send routine

Updates the comments in the broadcast bearer send routine to more
accurately describe the processing done by the routine. Also replaces
the improper use of a TIPC payload message error status symbol (in a place
that has nothing to do with such errors) with its numeric equivalent.
Signed-off-by: default avatarAllan Stephens <allan.stephens@windriver.com>
Signed-off-by: default avatarPaul Gortmaker <paul.gortmaker@windriver.com>
parent 2e2d9be8
...@@ -535,10 +535,11 @@ u32 tipc_bclink_acks_missing(struct tipc_node *n_ptr) ...@@ -535,10 +535,11 @@ u32 tipc_bclink_acks_missing(struct tipc_node *n_ptr)
/** /**
* tipc_bcbearer_send - send a packet through the broadcast pseudo-bearer * tipc_bcbearer_send - send a packet through the broadcast pseudo-bearer
* *
* Send through as many bearers as necessary to reach all nodes * Send packet over as many bearers as necessary to reach all nodes
* that support TIPC multicasting. * that have joined the broadcast link.
* *
* Returns 0 if packet sent successfully, non-zero if not * Returns 0 (packet sent successfully) under all circumstances,
* since the broadcast link's pseudo-bearer never blocks
*/ */
static int tipc_bcbearer_send(struct sk_buff *buf, static int tipc_bcbearer_send(struct sk_buff *buf,
...@@ -547,7 +548,12 @@ static int tipc_bcbearer_send(struct sk_buff *buf, ...@@ -547,7 +548,12 @@ static int tipc_bcbearer_send(struct sk_buff *buf,
{ {
int bp_index; int bp_index;
/* Prepare buffer for broadcasting (if first time trying to send it) */ /*
* Prepare broadcast link message for reliable transmission,
* if first time trying to send it;
* preparation is skipped for broadcast link protocol messages
* since they are sent in an unreliable manner and don't need it
*/
if (likely(!msg_non_seq(buf_msg(buf)))) { if (likely(!msg_non_seq(buf_msg(buf)))) {
struct tipc_msg *msg; struct tipc_msg *msg;
...@@ -596,18 +602,12 @@ static int tipc_bcbearer_send(struct sk_buff *buf, ...@@ -596,18 +602,12 @@ static int tipc_bcbearer_send(struct sk_buff *buf,
} }
if (bcbearer->remains_new.count == 0) if (bcbearer->remains_new.count == 0)
return 0; break; /* all targets reached */
bcbearer->remains = bcbearer->remains_new; bcbearer->remains = bcbearer->remains_new;
} }
/* return 0;
* Unable to reach all targets (indicate success, since currently
* there isn't code in place to properly block & unblock the
* pseudo-bearer used by the broadcast link)
*/
return TIPC_OK;
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment