diff options
| author |   Eric Dumazet <edumazet@google.com> | 2016-11-15 10:15:12 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2016-11-16 13:40:58 -0500 |
| commit | 21cb84c48ca0619181106f0f44f3802a989de024 (patch) | |
| tree | 1d3feac3e20855761e2e0b99105908980b4c18ed /include/net/busy_poll.h | |
| parent | net: busy-poll: allow preemption in sk_busy_loop() (diff) | |
| download | linux-dev-21cb84c48ca0619181106f0f44f3802a989de024.tar.xz linux-dev-21cb84c48ca0619181106f0f44f3802a989de024.zip | |
net: busy-poll: remove need_resched() from sk_can_busy_loop()
Now sk_busy_loop() can schedule by itself, we can remove
need_resched() check from sk_can_busy_loop()
Also add a const to its struct sock parameter.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Willem de Bruijn <willemb@google.com>
Cc: Adam Belay <abelay@google.com>
Cc: Tariq Toukan <tariqt@mellanox.com>
Cc: Yuval Mintz <Yuval.Mintz@cavium.com>
Cc: Ariel Elior <ariel.elior@cavium.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/busy_poll.h')
| -rw-r--r-- | include/net/busy_poll.h | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/include/net/busy_poll.h b/include/net/busy_poll.h index 2fbeb1313c0f..965e52b9b5a3 100644 --- a/include/net/busy_poll.h +++ b/include/net/busy_poll.h @@ -58,10 +58,9 @@ static inline unsigned long busy_loop_end_time(void) return busy_loop_us_clock() + ACCESS_ONCE(sysctl_net_busy_poll); } -static inline bool sk_can_busy_loop(struct sock *sk) +static inline bool sk_can_busy_loop(const struct sock *sk) { - return sk->sk_ll_usec && sk->sk_napi_id && - !need_resched() && !signal_pending(current); + return sk->sk_ll_usec && sk->sk_napi_id && !signal_pending(current); } |