diff options
author | Eric Dumazet <dada1@cosmosbay.com> | 2007-12-21 06:07:41 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 17:59:59 -0500 |
commit | 8df09ea3b8ccfe0c94844102d33fa46f57c08d9e (patch) | |
tree | a7e91a1e2c9eea21b1e1a6a67bd855899627aafe /include | |
parent | ce55dd3610f7ac29bf8d159c2e2ace9aaf2c3038 (diff) |
[SOCK] Avoid integer divides where not necessary in include/net/sock.h
Because sk_wmem_queued, sk_sndbuf are signed, a divide per two
may force compiler to use an integer divide.
We can instead use a right shift.
Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include')
-rw-r--r-- | include/net/sock.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/net/sock.h b/include/net/sock.h index 416bc994adad..e178b49eefbb 100644 --- a/include/net/sock.h +++ b/include/net/sock.h | |||
@@ -445,7 +445,7 @@ static inline int sk_acceptq_is_full(struct sock *sk) | |||
445 | */ | 445 | */ |
446 | static inline int sk_stream_min_wspace(struct sock *sk) | 446 | static inline int sk_stream_min_wspace(struct sock *sk) |
447 | { | 447 | { |
448 | return sk->sk_wmem_queued / 2; | 448 | return sk->sk_wmem_queued >> 1; |
449 | } | 449 | } |
450 | 450 | ||
451 | static inline int sk_stream_wspace(struct sock *sk) | 451 | static inline int sk_stream_wspace(struct sock *sk) |
@@ -1187,7 +1187,7 @@ static inline void sk_wake_async(struct sock *sk, int how, int band) | |||
1187 | static inline void sk_stream_moderate_sndbuf(struct sock *sk) | 1187 | static inline void sk_stream_moderate_sndbuf(struct sock *sk) |
1188 | { | 1188 | { |
1189 | if (!(sk->sk_userlocks & SOCK_SNDBUF_LOCK)) { | 1189 | if (!(sk->sk_userlocks & SOCK_SNDBUF_LOCK)) { |
1190 | sk->sk_sndbuf = min(sk->sk_sndbuf, sk->sk_wmem_queued / 2); | 1190 | sk->sk_sndbuf = min(sk->sk_sndbuf, sk->sk_wmem_queued >> 1); |
1191 | sk->sk_sndbuf = max(sk->sk_sndbuf, SOCK_MIN_SNDBUF); | 1191 | sk->sk_sndbuf = max(sk->sk_sndbuf, SOCK_MIN_SNDBUF); |
1192 | } | 1192 | } |
1193 | } | 1193 | } |
@@ -1211,7 +1211,7 @@ static inline struct page *sk_stream_alloc_page(struct sock *sk) | |||
1211 | */ | 1211 | */ |
1212 | static inline int sock_writeable(const struct sock *sk) | 1212 | static inline int sock_writeable(const struct sock *sk) |
1213 | { | 1213 | { |
1214 | return atomic_read(&sk->sk_wmem_alloc) < (sk->sk_sndbuf / 2); | 1214 | return atomic_read(&sk->sk_wmem_alloc) < (sk->sk_sndbuf >> 1); |
1215 | } | 1215 | } |
1216 | 1216 | ||
1217 | static inline gfp_t gfp_any(void) | 1217 | static inline gfp_t gfp_any(void) |