rocker: fix a sleeping in atomic bug
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 6 Jan 2022 11:57:54 +0000 (14:57 +0300)
committerDavid S. Miller <davem@davemloft.net>
Thu, 6 Jan 2022 13:45:02 +0000 (13:45 +0000)
This code is holding the &ofdpa->flow_tbl_lock spinlock so it is not
allowed to sleep.  That means we have to pass the OFDPA_OP_FLAG_NOWAIT
flag to ofdpa_flow_tbl_del().

Fixes: 936bd486564a ("rocker: use FIB notifications instead of switchdev calls")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/rocker/rocker_ofdpa.c

index 3e1ca7a..bc70c6a 100644 (file)
@@ -2783,7 +2783,8 @@ static void ofdpa_fib4_abort(struct rocker *rocker)
                if (!ofdpa_port)
                        continue;
                nh->fib_nh_flags &= ~RTNH_F_OFFLOAD;
-               ofdpa_flow_tbl_del(ofdpa_port, OFDPA_OP_FLAG_REMOVE,
+               ofdpa_flow_tbl_del(ofdpa_port,
+                                  OFDPA_OP_FLAG_REMOVE | OFDPA_OP_FLAG_NOWAIT,
                                   flow_entry);
        }
        spin_unlock_irqrestore(&ofdpa->flow_tbl_lock, flags);