Commit 6eada011 authored by Eric Dumazet's avatar Eric Dumazet Committed by David S. Miller

netns: constify net_hash_mix() and various callers

const qualifiers ease code review by making clear
which objects are not written in a function.
Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 8f6320de
...@@ -34,7 +34,7 @@ static inline struct udphdr *inner_udp_hdr(const struct sk_buff *skb) ...@@ -34,7 +34,7 @@ static inline struct udphdr *inner_udp_hdr(const struct sk_buff *skb)
#define UDP_HTABLE_SIZE_MIN (CONFIG_BASE_SMALL ? 128 : 256) #define UDP_HTABLE_SIZE_MIN (CONFIG_BASE_SMALL ? 128 : 256)
static inline int udp_hashfn(struct net *net, unsigned num, unsigned mask) static inline u32 udp_hashfn(const struct net *net, u32 num, u32 mask)
{ {
return (num + net_hash_mix(net)) & mask; return (num + net_hash_mix(net)) & mask;
} }
......
...@@ -221,8 +221,8 @@ inet_bind_bucket_create(struct kmem_cache *cachep, struct net *net, ...@@ -221,8 +221,8 @@ inet_bind_bucket_create(struct kmem_cache *cachep, struct net *net,
void inet_bind_bucket_destroy(struct kmem_cache *cachep, void inet_bind_bucket_destroy(struct kmem_cache *cachep,
struct inet_bind_bucket *tb); struct inet_bind_bucket *tb);
static inline int inet_bhashfn(struct net *net, const __u16 lport, static inline u32 inet_bhashfn(const struct net *net, const __u16 lport,
const int bhash_size) const u32 bhash_size)
{ {
return (lport + net_hash_mix(net)) & (bhash_size - 1); return (lport + net_hash_mix(net)) & (bhash_size - 1);
} }
...@@ -231,7 +231,7 @@ void inet_bind_hash(struct sock *sk, struct inet_bind_bucket *tb, ...@@ -231,7 +231,7 @@ void inet_bind_hash(struct sock *sk, struct inet_bind_bucket *tb,
const unsigned short snum); const unsigned short snum);
/* These can have wildcards, don't try too hard. */ /* These can have wildcards, don't try too hard. */
static inline int inet_lhashfn(struct net *net, const unsigned short num) static inline u32 inet_lhashfn(const struct net *net, const unsigned short num)
{ {
return (num + net_hash_mix(net)) & (INET_LHTABLE_SIZE - 1); return (num + net_hash_mix(net)) & (INET_LHTABLE_SIZE - 1);
} }
......
...@@ -5,7 +5,7 @@ ...@@ -5,7 +5,7 @@
struct net; struct net;
static inline unsigned int net_hash_mix(struct net *net) static inline u32 net_hash_mix(const struct net *net)
{ {
#ifdef CONFIG_NET_NS #ifdef CONFIG_NET_NS
/* /*
...@@ -13,7 +13,7 @@ static inline unsigned int net_hash_mix(struct net *net) ...@@ -13,7 +13,7 @@ static inline unsigned int net_hash_mix(struct net *net)
* always zeroed * always zeroed
*/ */
return (unsigned)(((unsigned long)net) >> L1_CACHE_SHIFT); return (u32)(((unsigned long)net) >> L1_CACHE_SHIFT);
#else #else
return 0; return 0;
#endif #endif
......
...@@ -107,7 +107,7 @@ static const struct nla_policy ifa_ipv4_policy[IFA_MAX+1] = { ...@@ -107,7 +107,7 @@ static const struct nla_policy ifa_ipv4_policy[IFA_MAX+1] = {
static struct hlist_head inet_addr_lst[IN4_ADDR_HSIZE]; static struct hlist_head inet_addr_lst[IN4_ADDR_HSIZE];
static u32 inet_addr_hash(struct net *net, __be32 addr) static u32 inet_addr_hash(const struct net *net, __be32 addr)
{ {
u32 val = (__force u32) addr ^ net_hash_mix(net); u32 val = (__force u32) addr ^ net_hash_mix(net);
......
...@@ -24,7 +24,7 @@ ...@@ -24,7 +24,7 @@
#include <net/secure_seq.h> #include <net/secure_seq.h>
#include <net/ip.h> #include <net/ip.h>
static unsigned int inet_ehashfn(struct net *net, const __be32 laddr, static u32 inet_ehashfn(const struct net *net, const __be32 laddr,
const __u16 lport, const __be32 faddr, const __u16 lport, const __be32 faddr,
const __be16 fport) const __be16 fport)
{ {
......
...@@ -64,11 +64,11 @@ EXPORT_SYMBOL_GPL(pingv6_ops); ...@@ -64,11 +64,11 @@ EXPORT_SYMBOL_GPL(pingv6_ops);
static u16 ping_port_rover; static u16 ping_port_rover;
static inline int ping_hashfn(struct net *net, unsigned int num, unsigned int mask) static inline u32 ping_hashfn(const struct net *net, u32 num, u32 mask)
{ {
int res = (num + net_hash_mix(net)) & mask; u32 res = (num + net_hash_mix(net)) & mask;
pr_debug("hash(%d) = %d\n", num, res); pr_debug("hash(%u) = %u\n", num, res);
return res; return res;
} }
EXPORT_SYMBOL_GPL(ping_hash); EXPORT_SYMBOL_GPL(ping_hash);
......
...@@ -318,7 +318,7 @@ static int ipv4_rcv_saddr_equal(const struct sock *sk1, const struct sock *sk2) ...@@ -318,7 +318,7 @@ static int ipv4_rcv_saddr_equal(const struct sock *sk1, const struct sock *sk2)
inet1->inet_rcv_saddr == inet2->inet_rcv_saddr)); inet1->inet_rcv_saddr == inet2->inet_rcv_saddr));
} }
static unsigned int udp4_portaddr_hash(struct net *net, __be32 saddr, static u32 udp4_portaddr_hash(const struct net *net, __be32 saddr,
unsigned int port) unsigned int port)
{ {
return jhash_1word((__force u32)saddr, net_hash_mix(net)) ^ port; return jhash_1word((__force u32)saddr, net_hash_mix(net)) ^ port;
...@@ -421,7 +421,7 @@ static inline int compute_score2(struct sock *sk, struct net *net, ...@@ -421,7 +421,7 @@ static inline int compute_score2(struct sock *sk, struct net *net,
return score; return score;
} }
static unsigned int udp_ehashfn(struct net *net, const __be32 laddr, static u32 udp_ehashfn(const struct net *net, const __be32 laddr,
const __u16 lport, const __be32 faddr, const __u16 lport, const __be32 faddr,
const __be16 fport) const __be16 fport)
{ {
......
...@@ -23,7 +23,7 @@ ...@@ -23,7 +23,7 @@
#include <net/secure_seq.h> #include <net/secure_seq.h>
#include <net/ip.h> #include <net/ip.h>
static unsigned int inet6_ehashfn(struct net *net, static u32 inet6_ehashfn(const struct net *net,
const struct in6_addr *laddr, const struct in6_addr *laddr,
const u16 lport, const u16 lport,
const struct in6_addr *faddr, const struct in6_addr *faddr,
......
...@@ -53,7 +53,7 @@ ...@@ -53,7 +53,7 @@
#include <trace/events/skb.h> #include <trace/events/skb.h>
#include "udp_impl.h" #include "udp_impl.h"
static unsigned int udp6_ehashfn(struct net *net, static u32 udp6_ehashfn(const struct net *net,
const struct in6_addr *laddr, const struct in6_addr *laddr,
const u16 lport, const u16 lport,
const struct in6_addr *faddr, const struct in6_addr *faddr,
...@@ -104,7 +104,7 @@ int ipv6_rcv_saddr_equal(const struct sock *sk, const struct sock *sk2) ...@@ -104,7 +104,7 @@ int ipv6_rcv_saddr_equal(const struct sock *sk, const struct sock *sk2)
return 0; return 0;
} }
static unsigned int udp6_portaddr_hash(struct net *net, static u32 udp6_portaddr_hash(const struct net *net,
const struct in6_addr *addr6, const struct in6_addr *addr6,
unsigned int port) unsigned int port)
{ {
......
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