net/mlx4_core: Check device state before unregistering it
authorAlex Vesker <valex@mellanox.com>
Mon, 25 Jul 2016 12:42:13 +0000 (15:42 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 10 May 2020 08:26:19 +0000 (10:26 +0200)
commit 9b022a6e0f26af108b9105b16b310393c898d9bd upstream.

Verify that the device state is registered before un-registering it.
This check is required to prevent an OOPS on flows that do
re-registration of the device and its previous state was
unregistered.

Fixes: 225c7b1feef1 ("IB/mlx4: Add a driver Mellanox ConnectX InfiniBand adapters")
Signed-off-by: Alex Vesker <valex@mellanox.com>
Signed-off-by: Tariq Toukan <tariqt@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/net/ethernet/mellanox/mlx4/intf.c

index 1a134e08f010a994c414521d0b611f0bc9529f68..cfc2a7632201195ed7fcddbaa813f0aba5d8b60f 100644 (file)
@@ -217,6 +217,9 @@ void mlx4_unregister_device(struct mlx4_dev *dev)
        struct mlx4_priv *priv = mlx4_priv(dev);
        struct mlx4_interface *intf;
 
+       if (!(dev->persist->interface_state & MLX4_INTERFACE_STATE_UP))
+               return;
+
        mlx4_stop_catas_poll(dev);
        if (dev->persist->interface_state & MLX4_INTERFACE_STATE_DELETION &&
            mlx4_is_slave(dev)) {