diff options
author | Eric Dumazet <edumazet@google.com> | 2012-07-18 04:11:12 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-07-18 14:28:46 -0400 |
commit | ddbe503203855939946430e39bae58de11b70b69 (patch) | |
tree | 1605a8d3b14a92819eb8ed47ae5a84c1b66e12f8 /net/ipv4/tcp_metrics.c | |
parent | dc9059512c09d09b99de6cd3a8bc842507934cbb (diff) |
ipv6: add ipv6_addr_hash() helper
Introduce ipv6_addr_hash() helper doing a XOR on all bits
of an IPv6 address, with an optimized x86_64 version.
Use it in flow dissector, as suggested by Andrew McGregor,
to reduce hash collision probabilities in fq_codel (and other
users of flow dissector)
Use it in ip6_tunnel.c and use more bit shuffling, as suggested
by David Laight, as existing hash was ignoring most of them.
Use it in sunrpc and use more bit shuffling, using hash_32().
Use it in net/ipv6/addrconf.c, using hash_32() as well.
As a cleanup, use it in net/ipv4/tcp_metrics.c
Signed-off-by: Eric Dumazet <edumazet@google.com>
Reported-by: Andrew McGregor <andrewmcgr@gmail.com>
Cc: Dave Taht <dave.taht@gmail.com>
Cc: Tom Herbert <therbert@google.com>
Cc: David Laight <David.Laight@ACULAB.COM>
Cc: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/tcp_metrics.c')
-rw-r--r-- | net/ipv4/tcp_metrics.c | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/net/ipv4/tcp_metrics.c b/net/ipv4/tcp_metrics.c index 5a38a2d5a95b..1a115b665792 100644 --- a/net/ipv4/tcp_metrics.c +++ b/net/ipv4/tcp_metrics.c | |||
@@ -211,10 +211,7 @@ static struct tcp_metrics_block *__tcp_get_metrics_req(struct request_sock *req, | |||
211 | break; | 211 | break; |
212 | case AF_INET6: | 212 | case AF_INET6: |
213 | *(struct in6_addr *)addr.addr.a6 = inet6_rsk(req)->rmt_addr; | 213 | *(struct in6_addr *)addr.addr.a6 = inet6_rsk(req)->rmt_addr; |
214 | hash = ((__force unsigned int) addr.addr.a6[0] ^ | 214 | hash = ipv6_addr_hash(&inet6_rsk(req)->rmt_addr); |
215 | (__force unsigned int) addr.addr.a6[1] ^ | ||
216 | (__force unsigned int) addr.addr.a6[2] ^ | ||
217 | (__force unsigned int) addr.addr.a6[3]); | ||
218 | break; | 215 | break; |
219 | default: | 216 | default: |
220 | return NULL; | 217 | return NULL; |
@@ -251,10 +248,7 @@ static struct tcp_metrics_block *__tcp_get_metrics_tw(struct inet_timewait_sock | |||
251 | case AF_INET6: | 248 | case AF_INET6: |
252 | tw6 = inet6_twsk((struct sock *)tw); | 249 | tw6 = inet6_twsk((struct sock *)tw); |
253 | *(struct in6_addr *)addr.addr.a6 = tw6->tw_v6_daddr; | 250 | *(struct in6_addr *)addr.addr.a6 = tw6->tw_v6_daddr; |
254 | hash = ((__force unsigned int) addr.addr.a6[0] ^ | 251 | hash = ipv6_addr_hash(&tw6->tw_v6_daddr); |
255 | (__force unsigned int) addr.addr.a6[1] ^ | ||
256 | (__force unsigned int) addr.addr.a6[2] ^ | ||
257 | (__force unsigned int) addr.addr.a6[3]); | ||
258 | break; | 252 | break; |
259 | default: | 253 | default: |
260 | return NULL; | 254 | return NULL; |
@@ -291,10 +285,7 @@ static struct tcp_metrics_block *tcp_get_metrics(struct sock *sk, | |||
291 | break; | 285 | break; |
292 | case AF_INET6: | 286 | case AF_INET6: |
293 | *(struct in6_addr *)addr.addr.a6 = inet6_sk(sk)->daddr; | 287 | *(struct in6_addr *)addr.addr.a6 = inet6_sk(sk)->daddr; |
294 | hash = ((__force unsigned int) addr.addr.a6[0] ^ | 288 | hash = ipv6_addr_hash(&inet6_sk(sk)->daddr); |
295 | (__force unsigned int) addr.addr.a6[1] ^ | ||
296 | (__force unsigned int) addr.addr.a6[2] ^ | ||
297 | (__force unsigned int) addr.addr.a6[3]); | ||
298 | break; | 289 | break; |
299 | default: | 290 | default: |
300 | return NULL; | 291 | return NULL; |