Commit 21cb84c4 authored by Eric Dumazet's avatar Eric Dumazet Committed by David S. Miller

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: default avatarEric 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: default avatarDavid S. Miller <davem@davemloft.net>
parent 217f6974
...@@ -58,10 +58,9 @@ static inline unsigned long busy_loop_end_time(void) ...@@ -58,10 +58,9 @@ static inline unsigned long busy_loop_end_time(void)
return busy_loop_us_clock() + ACCESS_ONCE(sysctl_net_busy_poll); 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 && return sk->sk_ll_usec && sk->sk_napi_id && !signal_pending(current);
!need_resched() && !signal_pending(current);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment