diff options
author | Eric Dumazet <dada1@cosmosbay.com> | 2008-01-03 23:46:48 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 18:00:36 -0500 |
commit | 65f7651788e18fadb2fbb7276af935d7871e1803 (patch) | |
tree | dcad32d4344d6d11d80061773d9d1dbc9ae92223 /net/ipv6 | |
parent | 571e7682026fd0e25833d103a3eeb74be29bf199 (diff) |
[NET]: prot_inuse cleanups and optimizations
1) Cleanups (all functions are prefixed by sock_prot_inuse)
sock_prot_inc_use(prot) -> sock_prot_inuse_add(prot,-1)
sock_prot_dec_use(prot) -> sock_prot_inuse_add(prot,-1)
sock_prot_inuse() -> sock_prot_inuse_get()
New functions :
sock_prot_inuse_init() and sock_prot_inuse_free() to abstract pcounter use.
2) if CONFIG_PROC_FS=n, we can zap 'inuse' member from "struct proto",
since nobody wants to read the inuse value.
This saves 1372 bytes on i386/SMP and some cpu cycles.
Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6')
-rw-r--r-- | net/ipv6/inet6_hashtables.c | 4 | ||||
-rw-r--r-- | net/ipv6/ipv6_sockglue.c | 8 | ||||
-rw-r--r-- | net/ipv6/proc.c | 8 |
3 files changed, 10 insertions, 10 deletions
diff --git a/net/ipv6/inet6_hashtables.c b/net/ipv6/inet6_hashtables.c index 0765d8bd380f..a66a7d8e2811 100644 --- a/net/ipv6/inet6_hashtables.c +++ b/net/ipv6/inet6_hashtables.c | |||
@@ -43,7 +43,7 @@ void __inet6_hash(struct inet_hashinfo *hashinfo, | |||
43 | } | 43 | } |
44 | 44 | ||
45 | __sk_add_node(sk, list); | 45 | __sk_add_node(sk, list); |
46 | sock_prot_inc_use(sk->sk_prot); | 46 | sock_prot_inuse_add(sk->sk_prot, 1); |
47 | write_unlock(lock); | 47 | write_unlock(lock); |
48 | } | 48 | } |
49 | EXPORT_SYMBOL(__inet6_hash); | 49 | EXPORT_SYMBOL(__inet6_hash); |
@@ -216,7 +216,7 @@ unique: | |||
216 | BUG_TRAP(sk_unhashed(sk)); | 216 | BUG_TRAP(sk_unhashed(sk)); |
217 | __sk_add_node(sk, &head->chain); | 217 | __sk_add_node(sk, &head->chain); |
218 | sk->sk_hash = hash; | 218 | sk->sk_hash = hash; |
219 | sock_prot_inc_use(sk->sk_prot); | 219 | sock_prot_inuse_add(sk->sk_prot, 1); |
220 | write_unlock(lock); | 220 | write_unlock(lock); |
221 | 221 | ||
222 | if (twp != NULL) { | 222 | if (twp != NULL) { |
diff --git a/net/ipv6/ipv6_sockglue.c b/net/ipv6/ipv6_sockglue.c index 20fece4ad3d8..bf2a686aa13d 100644 --- a/net/ipv6/ipv6_sockglue.c +++ b/net/ipv6/ipv6_sockglue.c | |||
@@ -268,8 +268,8 @@ static int do_ipv6_setsockopt(struct sock *sk, int level, int optname, | |||
268 | struct inet_connection_sock *icsk = inet_csk(sk); | 268 | struct inet_connection_sock *icsk = inet_csk(sk); |
269 | 269 | ||
270 | local_bh_disable(); | 270 | local_bh_disable(); |
271 | sock_prot_dec_use(sk->sk_prot); | 271 | sock_prot_inuse_add(sk->sk_prot, -1); |
272 | sock_prot_inc_use(&tcp_prot); | 272 | sock_prot_inuse_add(&tcp_prot, 1); |
273 | local_bh_enable(); | 273 | local_bh_enable(); |
274 | sk->sk_prot = &tcp_prot; | 274 | sk->sk_prot = &tcp_prot; |
275 | icsk->icsk_af_ops = &ipv4_specific; | 275 | icsk->icsk_af_ops = &ipv4_specific; |
@@ -282,8 +282,8 @@ static int do_ipv6_setsockopt(struct sock *sk, int level, int optname, | |||
282 | if (sk->sk_protocol == IPPROTO_UDPLITE) | 282 | if (sk->sk_protocol == IPPROTO_UDPLITE) |
283 | prot = &udplite_prot; | 283 | prot = &udplite_prot; |
284 | local_bh_disable(); | 284 | local_bh_disable(); |
285 | sock_prot_dec_use(sk->sk_prot); | 285 | sock_prot_inuse_add(sk->sk_prot, -1); |
286 | sock_prot_inc_use(prot); | 286 | sock_prot_inuse_add(prot, 1); |
287 | local_bh_enable(); | 287 | local_bh_enable(); |
288 | sk->sk_prot = prot; | 288 | sk->sk_prot = prot; |
289 | sk->sk_socket->ops = &inet_dgram_ops; | 289 | sk->sk_socket->ops = &inet_dgram_ops; |
diff --git a/net/ipv6/proc.c b/net/ipv6/proc.c index 41e9980b3e0e..571d95a21c15 100644 --- a/net/ipv6/proc.c +++ b/net/ipv6/proc.c | |||
@@ -36,13 +36,13 @@ static struct proc_dir_entry *proc_net_devsnmp6; | |||
36 | static int sockstat6_seq_show(struct seq_file *seq, void *v) | 36 | static int sockstat6_seq_show(struct seq_file *seq, void *v) |
37 | { | 37 | { |
38 | seq_printf(seq, "TCP6: inuse %d\n", | 38 | seq_printf(seq, "TCP6: inuse %d\n", |
39 | sock_prot_inuse(&tcpv6_prot)); | 39 | sock_prot_inuse_get(&tcpv6_prot)); |
40 | seq_printf(seq, "UDP6: inuse %d\n", | 40 | seq_printf(seq, "UDP6: inuse %d\n", |
41 | sock_prot_inuse(&udpv6_prot)); | 41 | sock_prot_inuse_get(&udpv6_prot)); |
42 | seq_printf(seq, "UDPLITE6: inuse %d\n", | 42 | seq_printf(seq, "UDPLITE6: inuse %d\n", |
43 | sock_prot_inuse(&udplitev6_prot)); | 43 | sock_prot_inuse_get(&udplitev6_prot)); |
44 | seq_printf(seq, "RAW6: inuse %d\n", | 44 | seq_printf(seq, "RAW6: inuse %d\n", |
45 | sock_prot_inuse(&rawv6_prot)); | 45 | sock_prot_inuse_get(&rawv6_prot)); |
46 | seq_printf(seq, "FRAG6: inuse %d memory %d\n", | 46 | seq_printf(seq, "FRAG6: inuse %d memory %d\n", |
47 | ip6_frag_nqueues(), ip6_frag_mem()); | 47 | ip6_frag_nqueues(), ip6_frag_mem()); |
48 | return 0; | 48 | return 0; |