From: Sebastian Basierski <[email protected]> When multiple VFs are brought up simultaneously, a VF can receive a CTS (Clear To Send) mailbox message instead of the expected RESET | (N)ACK response, because the watchdog is sending a gratuitous ping right as the VF mailbox transaction is in progress.
Remove the ixgbe_ping_all_vfs() call from ixgbe_watchdog_link_is_up(). Link-state changes are already communicated to VFs through the normal mailbox protocol; the extra ping here is redundant and races with VF initialization. Signed-off-by: Sebastian Basierski <[email protected]> Signed-off-by: Aleksandr Loktionov <[email protected]> --- drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c index 1885fe8..52f0cdc 100644 --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c @@ -8163,9 +8163,6 @@ static void ixgbe_watchdog_link_is_up(struct ixgbe_adapter *adapter) /* update the default user priority for VFs */ ixgbe_update_default_up(adapter); - - /* ping all the active vfs to let them know link has changed */ - ixgbe_ping_all_vfs(adapter); } /** -- 2.52.0
