diff options
author | Jon Paul Maloy <jon.maloy@ericsson.com> | 2015-02-05 08:36:36 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-02-05 19:00:01 -0500 |
commit | c5898636c440da91d58f10beac00f073e68378df (patch) | |
tree | 67e7148b03414171b1d2012bd8f27ea63433c3ed /net/tipc/bcast.h | |
parent | 4134069f3ea6cd96903e426bd3dfb9bb44165357 (diff) |
tipc: reduce usage of context info in socket and link
The most common usage of namespace information is when we fetch the
own node addess from the net structure. This leads to a lot of
passing around of a parameter of type 'struct net *' between
functions just to make them able to obtain this address.
However, in many cases this is unnecessary. The own node address
is readily available as a member of both struct tipc_sock and
tipc_link, and can be fetched from there instead.
The fact that the vast majority of functions in socket.c and link.c
anyway are maintaining a pointer to their respective base structures
makes this option even more compelling.
In this commit, we introduce the inline functions tsk_own_node()
and link_own_node() to make it easy for functions to fetch the node
address from those structs instead of having to pass along and
dereference the namespace struct.
In particular, we make calls to the msg_xx() functions in msg.{h,c}
context independent by directly passing them the own node address
as parameter when needed. Those functions should be regarded as
leaves in the code dependency tree, and it is hence desirable to
keep them namspace unaware.
Apart from a potential positive effect on cache behavior, these
changes make it easier to introduce the changes that will follow
later in this series.
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/bcast.h')
-rw-r--r-- | net/tipc/bcast.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/tipc/bcast.h b/net/tipc/bcast.h index a4583a109486..6ea190dccfe1 100644 --- a/net/tipc/bcast.h +++ b/net/tipc/bcast.h | |||
@@ -139,7 +139,7 @@ void tipc_bclink_acknowledge(struct tipc_node *n_ptr, u32 acked); | |||
139 | void tipc_bclink_rcv(struct net *net, struct sk_buff *buf); | 139 | void tipc_bclink_rcv(struct net *net, struct sk_buff *buf); |
140 | u32 tipc_bclink_get_last_sent(struct net *net); | 140 | u32 tipc_bclink_get_last_sent(struct net *net); |
141 | u32 tipc_bclink_acks_missing(struct tipc_node *n_ptr); | 141 | u32 tipc_bclink_acks_missing(struct tipc_node *n_ptr); |
142 | void tipc_bclink_update_link_state(struct net *net, struct tipc_node *n_ptr, | 142 | void tipc_bclink_update_link_state(struct tipc_node *node, |
143 | u32 last_sent); | 143 | u32 last_sent); |
144 | int tipc_bclink_stats(struct net *net, char *stats_buf, const u32 buf_size); | 144 | int tipc_bclink_stats(struct net *net, char *stats_buf, const u32 buf_size); |
145 | int tipc_bclink_reset_stats(struct net *net); | 145 | int tipc_bclink_reset_stats(struct net *net); |