net: rps: fix cpu unplug
authorEric Dumazet <edumazet@google.com>
Fri, 16 Jan 2015 01:04:22 +0000 (17:04 -0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 16 Jan 2015 06:02:42 +0000 (01:02 -0500)
commitac64da0b83d82abe62f78b3d0e21cca31aea24fa
tree1cacda83c251cfdc161757317dca62fd6fb16a50
parent57d737c5e21cd26f63272aa4b0e34680788e12ed
net: rps: fix cpu unplug

softnet_data.input_pkt_queue is protected by a spinlock that
we must hold when transferring packets from victim queue to an active
one. This is because other cpus could still be trying to enqueue packets
into victim queue.

A second problem is that when we transfert the NAPI poll_list from
victim to current cpu, we absolutely need to special case the percpu
backlog, because we do not want to add complex locking to protect
process_queue : Only owner cpu is allowed to manipulate it, unless cpu
is offline.

Based on initial patch from Prasad Sodagudi & Subash Abhinov
Kasiviswanathan.

This version is better because we do not slow down packet processing,
only make migration safer.

Reported-by: Prasad Sodagudi <psodagud@codeaurora.org>
Reported-by: Subash Abhinov Kasiviswanathan <subashab@codeaurora.org>
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Tom Herbert <therbert@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c