diff options
| author |   Jon Paul Maloy <jon.maloy@ericsson.com> | 2015-07-16 16:54:24 -0400 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2015-07-20 20:41:15 -0700 |
| commit | af9b028e270fda6fb812d70d17d902297df1ceb5 (patch) | |
| tree | 1a204c6d10d597d5db18908dc2066e980a78120d /net/tipc/bearer.h | |
| parent | tipc: change sk_buffer handling in tipc_link_xmit() (diff) | |
| download | linux-dev-af9b028e270fda6fb812d70d17d902297df1ceb5.tar.xz linux-dev-af9b028e270fda6fb812d70d17d902297df1ceb5.zip | |
tipc: make media xmit call outside node spinlock context
Currently, message sending is performed through a deep call chain,
where the node spinlock is grabbed and held during a significant
part of the transmission time. This is clearly detrimental to
overall throughput performance; it would be better if we could send
the message after the spinlock has been released.
In this commit, we do instead let the call revert on the stack after
the buffer chain has been added to the transmission queue, whereafter
clones of the buffers are transmitted to the device layer outside the
spinlock scope.
As a further step in our effort to separate the roles of the node
and link entities we also move the function tipc_link_xmit() to
node.c, and rename it to tipc_node_xmit().
Reviewed-by: Ying Xue <ying.xue@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/bearer.h')
| -rw-r--r-- | net/tipc/bearer.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/net/tipc/bearer.h b/net/tipc/bearer.h index dc714d977768..6426f242f626 100644 --- a/net/tipc/bearer.h +++ b/net/tipc/bearer.h @@ -217,5 +217,8 @@ void tipc_bearer_cleanup(void); void tipc_bearer_stop(struct net *net); void tipc_bearer_send(struct net *net, u32 bearer_id, struct sk_buff *buf, struct tipc_media_addr *dest); +void tipc_bearer_xmit(struct net *net, u32 bearer_id, + struct sk_buff_head *xmitq, + struct tipc_media_addr *dst); #endif /* _TIPC_BEARER_H */ |