[PATCH net-next] ipv4: remove the useless argument from ip_tunnel_hash()

Subsystems: networking [general], networking [ipv4/ipv6], the rest

STALE4598d

2 messages, 2 authors, 2014-01-13 · open the first message on its own page

[PATCH net-next] ipv4: remove the useless argument from ip_tunnel_hash()

From: Duan Jiong <hidden>
Date: 2014-01-12 10:04:40


Since commit c544193214 introduced function ip_tunnel_hash(), the argument
itn is no longer in use, so remove it.

Signed-off-by: Duan Jiong <redacted>
---
 net/ipv4/ip_tunnel.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/net/ipv4/ip_tunnel.c b/net/ipv4/ip_tunnel.c
index d3929a6..6f5a347 100644
--- a/net/ipv4/ip_tunnel.c
+++ b/net/ipv4/ip_tunnel.c
@@ -61,8 +61,7 @@
 #include <net/ip6_route.h>
 #endif
 
-static unsigned int ip_tunnel_hash(struct ip_tunnel_net *itn,
-				   __be32 key, __be32 remote)
+static unsigned int ip_tunnel_hash(__be32 key, __be32 remote)
 {
 	return hash_32((__force u32)key ^ (__force u32)remote,
 			 IP_TNL_HASH_BITS);
@@ -204,7 +203,7 @@ struct ip_tunnel *ip_tunnel_lookup(struct ip_tunnel_net *itn,
 	struct ip_tunnel *t, *cand = NULL;
 	struct hlist_head *head;
 
-	hash = ip_tunnel_hash(itn, key, remote);
+	hash = ip_tunnel_hash(key, remote);
 	head = &itn->tunnels[hash];
 
 	hlist_for_each_entry_rcu(t, head, hash_node) {
@@ -236,7 +235,7 @@ struct ip_tunnel *ip_tunnel_lookup(struct ip_tunnel_net *itn,
 			cand = t;
 	}
 
-	hash = ip_tunnel_hash(itn, key, 0);
+	hash = ip_tunnel_hash(key, 0);
 	head = &itn->tunnels[hash];
 
 	hlist_for_each_entry_rcu(t, head, hash_node) {
@@ -292,7 +291,7 @@ static struct hlist_head *ip_bucket(struct ip_tunnel_net *itn,
 	else
 		remote = 0;
 
-	h = ip_tunnel_hash(itn, parms->i_key, remote);
+	h = ip_tunnel_hash(parms->i_key, remote);
 	return &itn->tunnels[h];
 }
 
-- 
1.8.3.1

Re: [PATCH net-next] ipv4: remove the useless argument from ip_tunnel_hash()

From: Sergei Shtylyov <hidden>
Date: 2014-01-13 13:55:17

Hello.

On 12-01-2014 14:04, Duan Jiong wrote:
Since commit c544193214 introduced function ip_tunnel_hash(), the argument
   Please also specify that commit's summary line in parens.
itn is no longer in use, so remove it.
Signed-off-by: Duan Jiong <redacted>
WBR, Sergei
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help