[NET] NETNS: Omit neigh_parms->net and pneigh_entry->net without CONFIG_NET_NS.

Introduce neigh_parms/pneigh_entry inlines: neigh_parms_net(), pneigh_net().
Without CONFIG_NET_NS, no namespace other than &init_net exists.
Let's explicitly define them to help compiler optimizations.

Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
This commit is contained in:
YOSHIFUJI Hideaki 2008-03-26 03:49:59 +09:00
parent 1218854afa
commit 57da52c1e6
2 changed files with 41 additions and 10 deletions

View File

@ -38,7 +38,9 @@ struct neighbour;
struct neigh_parms struct neigh_parms
{ {
#ifdef CONFIG_NET_NS
struct net *net; struct net *net;
#endif
struct net_device *dev; struct net_device *dev;
struct neigh_parms *next; struct neigh_parms *next;
int (*neigh_setup)(struct neighbour *); int (*neigh_setup)(struct neighbour *);
@ -131,7 +133,9 @@ struct neigh_ops
struct pneigh_entry struct pneigh_entry
{ {
struct pneigh_entry *next; struct pneigh_entry *next;
#ifdef CONFIG_NET_NS
struct net *net; struct net *net;
#endif
struct net_device *dev; struct net_device *dev;
u8 flags; u8 flags;
u8 key[0]; u8 key[0];
@ -213,6 +217,17 @@ extern struct neighbour *neigh_event_ns(struct neigh_table *tbl,
extern struct neigh_parms *neigh_parms_alloc(struct net_device *dev, struct neigh_table *tbl); extern struct neigh_parms *neigh_parms_alloc(struct net_device *dev, struct neigh_table *tbl);
extern void neigh_parms_release(struct neigh_table *tbl, struct neigh_parms *parms); extern void neigh_parms_release(struct neigh_table *tbl, struct neigh_parms *parms);
static inline
struct net *neigh_parms_net(const struct neigh_parms *parms)
{
#ifdef CONFIG_NET_NS
return parms->net;
#else
return &init_net;
#endif
}
extern unsigned long neigh_rand_reach_time(unsigned long base); extern unsigned long neigh_rand_reach_time(unsigned long base);
extern void pneigh_enqueue(struct neigh_table *tbl, struct neigh_parms *p, extern void pneigh_enqueue(struct neigh_table *tbl, struct neigh_parms *p,
@ -220,6 +235,16 @@ extern void pneigh_enqueue(struct neigh_table *tbl, struct neigh_parms *p,
extern struct pneigh_entry *pneigh_lookup(struct neigh_table *tbl, struct net *net, const void *key, struct net_device *dev, int creat); extern struct pneigh_entry *pneigh_lookup(struct neigh_table *tbl, struct net *net, const void *key, struct net_device *dev, int creat);
extern int pneigh_delete(struct neigh_table *tbl, struct net *net, const void *key, struct net_device *dev); extern int pneigh_delete(struct neigh_table *tbl, struct net *net, const void *key, struct net_device *dev);
static inline
struct net *pneigh_net(const struct pneigh_entry *pneigh)
{
#ifdef CONFIG_NET_NS
return pneigh->net;
#else
return &init_net;
#endif
}
extern void neigh_app_ns(struct neighbour *n); extern void neigh_app_ns(struct neighbour *n);
extern void neigh_for_each(struct neigh_table *tbl, void (*cb)(struct neighbour *, void *), void *cookie); extern void neigh_for_each(struct neigh_table *tbl, void (*cb)(struct neighbour *, void *), void *cookie);
extern void __neigh_for_each_release(struct neigh_table *tbl, int (*cb)(struct neighbour *)); extern void __neigh_for_each_release(struct neigh_table *tbl, int (*cb)(struct neighbour *));

View File

@ -483,7 +483,7 @@ struct pneigh_entry * pneigh_lookup(struct neigh_table *tbl,
for (n = tbl->phash_buckets[hash_val]; n; n = n->next) { for (n = tbl->phash_buckets[hash_val]; n; n = n->next) {
if (!memcmp(n->key, pkey, key_len) && if (!memcmp(n->key, pkey, key_len) &&
(n->net == net) && (pneigh_net(n) == net) &&
(n->dev == dev || !n->dev)) { (n->dev == dev || !n->dev)) {
read_unlock_bh(&tbl->lock); read_unlock_bh(&tbl->lock);
goto out; goto out;
@ -500,7 +500,9 @@ struct pneigh_entry * pneigh_lookup(struct neigh_table *tbl,
if (!n) if (!n)
goto out; goto out;
#ifdef CONFIG_NET_NS
n->net = hold_net(net); n->net = hold_net(net);
#endif
memcpy(n->key, pkey, key_len); memcpy(n->key, pkey, key_len);
n->dev = dev; n->dev = dev;
if (dev) if (dev)
@ -540,14 +542,14 @@ int pneigh_delete(struct neigh_table *tbl, struct net *net, const void *pkey,
for (np = &tbl->phash_buckets[hash_val]; (n = *np) != NULL; for (np = &tbl->phash_buckets[hash_val]; (n = *np) != NULL;
np = &n->next) { np = &n->next) {
if (!memcmp(n->key, pkey, key_len) && n->dev == dev && if (!memcmp(n->key, pkey, key_len) && n->dev == dev &&
(n->net == net)) { (pneigh_net(n) == net)) {
*np = n->next; *np = n->next;
write_unlock_bh(&tbl->lock); write_unlock_bh(&tbl->lock);
if (tbl->pdestructor) if (tbl->pdestructor)
tbl->pdestructor(n); tbl->pdestructor(n);
if (n->dev) if (n->dev)
dev_put(n->dev); dev_put(n->dev);
release_net(n->net); release_net(pneigh_net(n));
kfree(n); kfree(n);
return 0; return 0;
} }
@ -570,7 +572,7 @@ static int pneigh_ifdown(struct neigh_table *tbl, struct net_device *dev)
tbl->pdestructor(n); tbl->pdestructor(n);
if (n->dev) if (n->dev)
dev_put(n->dev); dev_put(n->dev);
release_net(n->net); release_net(pneigh_net(n));
kfree(n); kfree(n);
continue; continue;
} }
@ -1284,7 +1286,7 @@ static inline struct neigh_parms *lookup_neigh_params(struct neigh_table *tbl,
struct neigh_parms *p; struct neigh_parms *p;
for (p = &tbl->parms; p; p = p->next) { for (p = &tbl->parms; p; p = p->next) {
if ((p->dev && p->dev->ifindex == ifindex && p->net == net) || if ((p->dev && p->dev->ifindex == ifindex && neigh_parms_net(p) == net) ||
(!p->dev && !ifindex)) (!p->dev && !ifindex))
return p; return p;
} }
@ -1318,7 +1320,9 @@ struct neigh_parms *neigh_parms_alloc(struct net_device *dev,
dev_hold(dev); dev_hold(dev);
p->dev = dev; p->dev = dev;
#ifdef CONFIG_NET_NS
p->net = hold_net(net); p->net = hold_net(net);
#endif
p->sysctl_table = NULL; p->sysctl_table = NULL;
write_lock_bh(&tbl->lock); write_lock_bh(&tbl->lock);
p->next = tbl->parms.next; p->next = tbl->parms.next;
@ -1360,7 +1364,7 @@ void neigh_parms_release(struct neigh_table *tbl, struct neigh_parms *parms)
static void neigh_parms_destroy(struct neigh_parms *parms) static void neigh_parms_destroy(struct neigh_parms *parms)
{ {
release_net(parms->net); release_net(neigh_parms_net(parms));
kfree(parms); kfree(parms);
} }
@ -1371,7 +1375,9 @@ void neigh_table_init_no_netlink(struct neigh_table *tbl)
unsigned long now = jiffies; unsigned long now = jiffies;
unsigned long phsize; unsigned long phsize;
#ifdef CONFIG_NET_NS
tbl->parms.net = &init_net; tbl->parms.net = &init_net;
#endif
atomic_set(&tbl->parms.refcnt, 1); atomic_set(&tbl->parms.refcnt, 1);
INIT_RCU_HEAD(&tbl->parms.rcu_head); INIT_RCU_HEAD(&tbl->parms.rcu_head);
tbl->parms.reachable_time = tbl->parms.reachable_time =
@ -1958,7 +1964,7 @@ static int neightbl_dump_info(struct sk_buff *skb, struct netlink_callback *cb)
break; break;
for (nidx = 0, p = tbl->parms.next; p; p = p->next) { for (nidx = 0, p = tbl->parms.next; p; p = p->next) {
if (net != p->net) if (net != neigh_parms_net(p))
continue; continue;
if (nidx++ < neigh_skip) if (nidx++ < neigh_skip)
@ -2254,7 +2260,7 @@ static struct pneigh_entry *pneigh_get_first(struct seq_file *seq)
state->flags |= NEIGH_SEQ_IS_PNEIGH; state->flags |= NEIGH_SEQ_IS_PNEIGH;
for (bucket = 0; bucket <= PNEIGH_HASHMASK; bucket++) { for (bucket = 0; bucket <= PNEIGH_HASHMASK; bucket++) {
pn = tbl->phash_buckets[bucket]; pn = tbl->phash_buckets[bucket];
while (pn && (pn->net != net)) while (pn && (pneigh_net(pn) != net))
pn = pn->next; pn = pn->next;
if (pn) if (pn)
break; break;
@ -2277,7 +2283,7 @@ static struct pneigh_entry *pneigh_get_next(struct seq_file *seq,
if (++state->bucket > PNEIGH_HASHMASK) if (++state->bucket > PNEIGH_HASHMASK)
break; break;
pn = tbl->phash_buckets[state->bucket]; pn = tbl->phash_buckets[state->bucket];
while (pn && (pn->net != net)) while (pn && (pneigh_net(pn) != net))
pn = pn->next; pn = pn->next;
if (pn) if (pn)
break; break;
@ -2740,7 +2746,7 @@ int neigh_sysctl_register(struct net_device *dev, struct neigh_parms *p,
neigh_path[NEIGH_CTL_PATH_PROTO].ctl_name = p_id; neigh_path[NEIGH_CTL_PATH_PROTO].ctl_name = p_id;
t->sysctl_header = t->sysctl_header =
register_net_sysctl_table(p->net, neigh_path, t->neigh_vars); register_net_sysctl_table(neigh_parms_net(p), neigh_path, t->neigh_vars);
if (!t->sysctl_header) if (!t->sysctl_header)
goto free_procname; goto free_procname;