diff options
author | Eric Dumazet <eric.dumazet@gmail.com> | 2010-04-28 15:14:43 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-04-30 19:29:42 -0400 |
commit | 767dd03369ac18af58efdef0383d6eb986eab426 (patch) | |
tree | 5af8a861110676a084078a168c27fd3935d41f13 | |
parent | 21851264120b0bd1f953328cb131abcfa9305bc3 (diff) |
net: speedup sock_recv_ts_and_drops()
sock_recv_ts_and_drops() is fat and slow (~ 4% of cpu time on some
profiles)
We can test all socket flags at once to make fast path fast again.
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | include/net/sock.h | 19 | ||||
-rw-r--r-- | net/socket.c | 4 |
2 files changed, 20 insertions, 3 deletions
diff --git a/include/net/sock.h b/include/net/sock.h index d361c7769fe0..e1777db5b9ab 100644 --- a/include/net/sock.h +++ b/include/net/sock.h | |||
@@ -1635,7 +1635,24 @@ sock_recv_timestamp(struct msghdr *msg, struct sock *sk, struct sk_buff *skb) | |||
1635 | sk->sk_stamp = kt; | 1635 | sk->sk_stamp = kt; |
1636 | } | 1636 | } |
1637 | 1637 | ||
1638 | extern void sock_recv_ts_and_drops(struct msghdr *msg, struct sock *sk, struct sk_buff *skb); | 1638 | extern void __sock_recv_ts_and_drops(struct msghdr *msg, struct sock *sk, |
1639 | struct sk_buff *skb); | ||
1640 | |||
1641 | static inline void sock_recv_ts_and_drops(struct msghdr *msg, struct sock *sk, | ||
1642 | struct sk_buff *skb) | ||
1643 | { | ||
1644 | #define FLAGS_TS_OR_DROPS ((1UL << SOCK_RXQ_OVFL) | \ | ||
1645 | (1UL << SOCK_RCVTSTAMP) | \ | ||
1646 | (1UL << SOCK_TIMESTAMPING_RX_SOFTWARE) | \ | ||
1647 | (1UL << SOCK_TIMESTAMPING_SOFTWARE) | \ | ||
1648 | (1UL << SOCK_TIMESTAMPING_RAW_HARDWARE) | \ | ||
1649 | (1UL << SOCK_TIMESTAMPING_SYS_HARDWARE)) | ||
1650 | |||
1651 | if (sk->sk_flags & FLAGS_TS_OR_DROPS) | ||
1652 | __sock_recv_ts_and_drops(msg, sk, skb); | ||
1653 | else | ||
1654 | sk->sk_stamp = skb->tstamp; | ||
1655 | } | ||
1639 | 1656 | ||
1640 | /** | 1657 | /** |
1641 | * sock_tx_timestamp - checks whether the outgoing packet is to be time stamped | 1658 | * sock_tx_timestamp - checks whether the outgoing packet is to be time stamped |
diff --git a/net/socket.c b/net/socket.c index 9822081eab38..cb7c1f6c0d6e 100644 --- a/net/socket.c +++ b/net/socket.c | |||
@@ -655,13 +655,13 @@ inline void sock_recv_drops(struct msghdr *msg, struct sock *sk, struct sk_buff | |||
655 | sizeof(__u32), &skb->dropcount); | 655 | sizeof(__u32), &skb->dropcount); |
656 | } | 656 | } |
657 | 657 | ||
658 | void sock_recv_ts_and_drops(struct msghdr *msg, struct sock *sk, | 658 | void __sock_recv_ts_and_drops(struct msghdr *msg, struct sock *sk, |
659 | struct sk_buff *skb) | 659 | struct sk_buff *skb) |
660 | { | 660 | { |
661 | sock_recv_timestamp(msg, sk, skb); | 661 | sock_recv_timestamp(msg, sk, skb); |
662 | sock_recv_drops(msg, sk, skb); | 662 | sock_recv_drops(msg, sk, skb); |
663 | } | 663 | } |
664 | EXPORT_SYMBOL_GPL(sock_recv_ts_and_drops); | 664 | EXPORT_SYMBOL_GPL(__sock_recv_ts_and_drops); |
665 | 665 | ||
666 | static inline int __sock_recvmsg_nosec(struct kiocb *iocb, struct socket *sock, | 666 | static inline int __sock_recvmsg_nosec(struct kiocb *iocb, struct socket *sock, |
667 | struct msghdr *msg, size_t size, int flags) | 667 | struct msghdr *msg, size_t size, int flags) |