tipc: Minor optimization of broadcast link transmit queue statistic

The two broadcast link statistics fields that are used to derive the
average length of that link's transmit queue are now updated only after
a successful attempt to send a broadcast message, since there is no need
to update these values when an unsuccessful send attempt leaves the
queue unchanged.

Signed-off-by: Allan Stephens <allan.stephens@windriver.com>
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>

diff --git a/net/tipc/bcast.c b/net/tipc/bcast.c
index 8f58df2..dd990b0 100644 (file)
--- a/net/tipc/bcast.c
+++ b/net/tipc/bcast.c
@@ -424,11 +424,11 @@ int tipc_bclink_send_msg(struct sk_buff *buf)
        }
 
        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;