diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2009-12-02 21:29:09 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-12-03 15:23:47 -0500 |
commit | b099ce2602d806deb41caaa578731848995cdb2a (patch) | |
tree | 7541d0d59c73e6b6274d420b8a3ec35d85ca5f2d /include/net/inet_timewait_sock.h | |
parent | 575f4cd5a5b639457747434dbe18d175fa767db4 (diff) |
net: Batch inet_twsk_purge
This function walks the whole hashtable so there is no point in
passing it a network namespace. Instead I purge all timewait
sockets from dead network namespaces that I find. If the namespace
is one of the once I am trying to purge I am guaranteed no new timewait
sockets can be formed so this will get them all. If the namespace
is one I am not acting for it might form a few more but I will
call inet_twsk_purge again and shortly to get rid of them. In
any even if the network namespace is dead timewait sockets are
useless.
Move the calls of inet_twsk_purge into batch_exit routines so
that if I am killing a bunch of namespaces at once I will just
call inet_twsk_purge once and save a lot of redundant unnecessary
work.
My simple 4k network namespace exit test the cleanup time dropped from
roughly 8.2s to 1.6s. While the time spent running inet_twsk_purge fell
to about 2ms. 1ms for ipv4 and 1ms for ipv6.
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/inet_timewait_sock.h')
-rw-r--r-- | include/net/inet_timewait_sock.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/net/inet_timewait_sock.h b/include/net/inet_timewait_sock.h index 773b10fa38e4..4fd007f34dd5 100644 --- a/include/net/inet_timewait_sock.h +++ b/include/net/inet_timewait_sock.h | |||
@@ -212,14 +212,14 @@ extern void inet_twsk_schedule(struct inet_timewait_sock *tw, | |||
212 | extern void inet_twsk_deschedule(struct inet_timewait_sock *tw, | 212 | extern void inet_twsk_deschedule(struct inet_timewait_sock *tw, |
213 | struct inet_timewait_death_row *twdr); | 213 | struct inet_timewait_death_row *twdr); |
214 | 214 | ||
215 | extern void inet_twsk_purge(struct net *net, struct inet_hashinfo *hashinfo, | 215 | extern void inet_twsk_purge(struct inet_hashinfo *hashinfo, |
216 | struct inet_timewait_death_row *twdr, int family); | 216 | struct inet_timewait_death_row *twdr, int family); |
217 | 217 | ||
218 | static inline | 218 | static inline |
219 | struct net *twsk_net(const struct inet_timewait_sock *twsk) | 219 | struct net *twsk_net(const struct inet_timewait_sock *twsk) |
220 | { | 220 | { |
221 | #ifdef CONFIG_NET_NS | 221 | #ifdef CONFIG_NET_NS |
222 | return twsk->tw_net; | 222 | return rcu_dereference(twsk->tw_net); |
223 | #else | 223 | #else |
224 | return &init_net; | 224 | return &init_net; |
225 | #endif | 225 | #endif |
@@ -229,7 +229,7 @@ static inline | |||
229 | void twsk_net_set(struct inet_timewait_sock *twsk, struct net *net) | 229 | void twsk_net_set(struct inet_timewait_sock *twsk, struct net *net) |
230 | { | 230 | { |
231 | #ifdef CONFIG_NET_NS | 231 | #ifdef CONFIG_NET_NS |
232 | twsk->tw_net = net; | 232 | rcu_assign_pointer(twsk->tw_net, net); |
233 | #endif | 233 | #endif |
234 | } | 234 | } |
235 | #endif /* _INET_TIMEWAIT_SOCK_ */ | 235 | #endif /* _INET_TIMEWAIT_SOCK_ */ |