diff options
author | Ying Xue <ying.xue@windriver.com> | 2013-12-10 23:45:38 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-12-11 00:17:42 -0500 |
commit | d77b3831f7d59d69aa49d5d1df10bbe56671dc5d (patch) | |
tree | 68ee50f8e762bd9e524502cd6173ee7c50f5949b /net/tipc/node.c | |
parent | 4ba3f99a3624187f47f160234fc49a3b8054a0c1 (diff) |
tipc: eliminate redundant code with kfree_skb_list routine
sk_buff lists are currently relased by looping over the list and
explicitly releasing each buffer.
We replace all occurrences of this loop with a call to kfree_skb_list().
Signed-off-by: Ying Xue <ying.xue@windriver.com>
Reviewed-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Signed-off-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/node.c')
-rw-r--r-- | net/tipc/node.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/net/tipc/node.c b/net/tipc/node.c index 25100c0a6fe8..bf1ac89b4806 100644 --- a/net/tipc/node.c +++ b/net/tipc/node.c | |||
@@ -291,11 +291,7 @@ static void node_lost_contact(struct tipc_node *n_ptr) | |||
291 | 291 | ||
292 | /* Flush broadcast link info associated with lost node */ | 292 | /* Flush broadcast link info associated with lost node */ |
293 | if (n_ptr->bclink.recv_permitted) { | 293 | if (n_ptr->bclink.recv_permitted) { |
294 | while (n_ptr->bclink.deferred_head) { | 294 | kfree_skb_list(n_ptr->bclink.deferred_head); |
295 | struct sk_buff *buf = n_ptr->bclink.deferred_head; | ||
296 | n_ptr->bclink.deferred_head = buf->next; | ||
297 | kfree_skb(buf); | ||
298 | } | ||
299 | n_ptr->bclink.deferred_size = 0; | 295 | n_ptr->bclink.deferred_size = 0; |
300 | 296 | ||
301 | if (n_ptr->bclink.reasm_head) { | 297 | if (n_ptr->bclink.reasm_head) { |