Commit 5056a1ef authored by YOSHIFUJI Hideaki's avatar YOSHIFUJI Hideaki Committed by David S. Miller

[IPV4] IP_GRE: Unify code path to get hash array index.

Signed-off-by: default avatarYOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
parent 87d1a164
...@@ -191,11 +191,11 @@ static struct ip_tunnel * ipgre_tunnel_lookup(__be32 remote, __be32 local, __be3 ...@@ -191,11 +191,11 @@ static struct ip_tunnel * ipgre_tunnel_lookup(__be32 remote, __be32 local, __be3
return NULL; return NULL;
} }
static struct ip_tunnel **ipgre_bucket(struct ip_tunnel *t) static struct ip_tunnel **__ipgre_bucket(struct ip_tunnel_parm *parms)
{ {
__be32 remote = t->parms.iph.daddr; __be32 remote = parms->iph.daddr;
__be32 local = t->parms.iph.saddr; __be32 local = parms->iph.saddr;
__be32 key = t->parms.i_key; __be32 key = parms->i_key;
unsigned h = HASH(key); unsigned h = HASH(key);
int prio = 0; int prio = 0;
...@@ -209,6 +209,11 @@ static struct ip_tunnel **ipgre_bucket(struct ip_tunnel *t) ...@@ -209,6 +209,11 @@ static struct ip_tunnel **ipgre_bucket(struct ip_tunnel *t)
return &tunnels[prio][h]; return &tunnels[prio][h];
} }
static inline struct ip_tunnel **ipgre_bucket(struct ip_tunnel *t)
{
return __ipgre_bucket(&t->parms);
}
static void ipgre_tunnel_link(struct ip_tunnel *t) static void ipgre_tunnel_link(struct ip_tunnel *t)
{ {
struct ip_tunnel **tp = ipgre_bucket(t); struct ip_tunnel **tp = ipgre_bucket(t);
...@@ -240,17 +245,9 @@ static struct ip_tunnel * ipgre_tunnel_locate(struct ip_tunnel_parm *parms, int ...@@ -240,17 +245,9 @@ static struct ip_tunnel * ipgre_tunnel_locate(struct ip_tunnel_parm *parms, int
__be32 key = parms->i_key; __be32 key = parms->i_key;
struct ip_tunnel *t, **tp, *nt; struct ip_tunnel *t, **tp, *nt;
struct net_device *dev; struct net_device *dev;
unsigned h = HASH(key);
int prio = 0;
char name[IFNAMSIZ]; char name[IFNAMSIZ];
if (local) for (tp = __ipgre_bucket(parms); (t = *tp) != NULL; tp = &t->next) {
prio |= 1;
if (remote && !MULTICAST(remote)) {
prio |= 2;
h ^= HASH(remote);
}
for (tp = &tunnels[prio][h]; (t = *tp) != NULL; tp = &t->next) {
if (local == t->parms.iph.saddr && remote == t->parms.iph.daddr) { if (local == t->parms.iph.saddr && remote == t->parms.iph.daddr) {
if (key == t->parms.i_key) if (key == t->parms.i_key)
return t; return t;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment