diff options
| author | Allan Stephens <allan.stephens@windriver.com> | 2011-10-24 13:27:31 -0400 |
|---|---|---|
| committer | Paul Gortmaker <paul.gortmaker@windriver.com> | 2011-12-27 11:33:51 -0500 |
| commit | 9157bafb44637a2cfefc222d6551100ead40e79e (patch) | |
| tree | 03daf741edd6dc848ec38eff1aead23c31c5c83a /net/tipc | |
| parent | 2b78f9a002dccc587912af4da3bf1db86909de91 (diff) | |
| download | op-kernel-dev-9157bafb44637a2cfefc222d6551100ead40e79e.zip op-kernel-dev-9157bafb44637a2cfefc222d6551100ead40e79e.tar.gz | |
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>
Diffstat (limited to 'net/tipc')
| -rw-r--r-- | net/tipc/bcast.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/net/tipc/bcast.c b/net/tipc/bcast.c index 8f58df2..dd990b0 100644 --- 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; |
