diff options
authorMatan Azrad <>2017-10-22 05:51:08 +0000
committerFerruh Yigit <>2017-10-27 01:05:55 +0200
commit295f7ed2dd452aba090080f380c2981b366aa2da (patch)
parent1f148d8eaf3baaf123fd8bf692947fe9437a5ab6 (diff)
net/failsafe: fix Rx clean race
When removing a device, the fail-safe checks that it is not within its datapath before cleaning it. When checking whether an Rx burst should be performed on a device, the remove flag is not checked. Thus the port could still enter its datapath and miss a removal round. Furthermore, there is a race between the thread removing the device and the polling thread. Check the remove flag before entering a sub-device Rx burst when in safe mode. This check mitigates the aforementioned race condition. Fixes: 72a57bfd9a0e ("net/failsafe: add fast burst functions") Cc: Signed-off-by: Matan Azrad <> Acked-by: Gaetan Rivet <>
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/failsafe/failsafe_rxtx.c b/drivers/net/failsafe/failsafe_rxtx.c
index 7311421..70157c8 100644
--- a/drivers/net/failsafe/failsafe_rxtx.c
+++ b/drivers/net/failsafe/failsafe_rxtx.c
@@ -43,7 +43,8 @@ fs_rx_unsafe(struct sub_device *sdev)
return (ETH(sdev) == NULL) ||
(ETH(sdev)->rx_pkt_burst == NULL) ||
- (sdev->state != DEV_STARTED);
+ (sdev->state != DEV_STARTED) ||
+ (sdev->remove != 0);
static inline int