diff options
| author | Eric W. Biederman <ebiederm@aristanetworks.com> | 2009-02-22 00:11:09 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-03-03 01:14:27 -0800 |
| commit | 17edde520927070a6bf14a6a75027c0b843443e5 (patch) | |
| tree | dcd940213c93bc943c670f48aa52ec8571351912 /net/core/net_namespace.c | |
| parent | 2f20d2e667ab1ca44cde5fb361386dff5bb6081d (diff) | |
netns: Remove net_alive
It turns out that net_alive is unnecessary, and the original problem
that led to it being added was simply that the icmp code thought
it was a network device and wound up being unable to handle packets
while there were still packets in the network namespace.
Now that icmp and tcp have been fixed to properly register themselves
this problem is no longer present and we have a stronger guarantee
that packets will not arrive in a network namespace then that provided
by net_alive in netif_receive_skb. So remove net_alive allowing
packet reception run a little faster.
Additionally document the strong reason why network namespace cleanup
is safe so that if something happens again someone else will have
a chance of figuring it out.
Signed-off-by: Eric W. Biederman <ebiederm@aristanetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/net_namespace.c')
| -rw-r--r-- | net/core/net_namespace.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c index 2adb1a7d361..e3bebd36f05 100644 --- a/net/core/net_namespace.c +++ b/net/core/net_namespace.c @@ -157,9 +157,6 @@ static void cleanup_net(struct work_struct *work) struct pernet_operations *ops; struct net *net; - /* Be very certain incoming network packets will not find us */ - rcu_barrier(); - net = container_of(work, struct net, work); mutex_lock(&net_mutex); |