1 files changed, 29 insertions, 23 deletions

diff --git a/drivers/net/ixgbe/ixgbe_dcb_nl.c b/drivers/net/ixgbe/ixgbe_dcb_nl.c
index 615c2803202a..7d158a5c545b 100644
--- a/drivers/net/ixgbe/ixgbe_dcb_nl.c
+++ b/drivers/net/ixgbe/ixgbe_dcb_nl.c
@@ -124,39 +124,45 @@ static u16 ixgbe_dcb_select_queue(struct net_device *dev, struct sk_buff *skb)
         return 0;
 }
 
-static void ixgbe_dcbnl_set_state(struct net_device *netdev, u8 state)
+static u8 ixgbe_dcbnl_set_state(struct net_device *netdev, u8 state)
 {
+        u8 err = 0;
         struct ixgbe_adapter *adapter = netdev_priv(netdev);
 
         DPRINTK(DRV, INFO, "Set DCB Admin Mode.\n");
 
         if (state > 0) {
                 /* Turn on DCB */
-                if (adapter->flags & IXGBE_FLAG_DCB_ENABLED) {
-                        return;
-                } else {
-                        if (netif_running(netdev))
-                                netdev->stop(netdev);
-                        ixgbe_reset_interrupt_capability(adapter);
-                        ixgbe_napi_del_all(adapter);
-                        kfree(adapter->tx_ring);
-                        kfree(adapter->rx_ring);
-                        adapter->tx_ring = NULL;
-                        adapter->rx_ring = NULL;
-                        netdev->select_queue = &ixgbe_dcb_select_queue;
+                if (adapter->flags & IXGBE_FLAG_DCB_ENABLED)
+                        goto out;
 
-                        adapter->flags &= ~IXGBE_FLAG_RSS_ENABLED;
-                        adapter->flags |= IXGBE_FLAG_DCB_ENABLED;
-                        ixgbe_init_interrupt_scheme(adapter);
-                        ixgbe_napi_add_all(adapter);
-                        if (netif_running(netdev))
-                                netdev->open(netdev);
+                if (!(adapter->flags & IXGBE_FLAG_MSIX_ENABLED)) {
+                        DPRINTK(DRV, ERR, "Enable failed, needs MSI-X\n");
+                        err = 1;
+                        goto out;
                 }
+
+                if (netif_running(netdev))
+                        netdev->netdev_ops->ndo_stop(netdev);
+                ixgbe_reset_interrupt_capability(adapter);
+                ixgbe_napi_del_all(adapter);
+                kfree(adapter->tx_ring);
+                kfree(adapter->rx_ring);
+                adapter->tx_ring = NULL;
+                adapter->rx_ring = NULL;
+                netdev->select_queue = &ixgbe_dcb_select_queue;
+
+                adapter->flags &= ~IXGBE_FLAG_RSS_ENABLED;
+                adapter->flags |= IXGBE_FLAG_DCB_ENABLED;
+                ixgbe_init_interrupt_scheme(adapter);
+                ixgbe_napi_add_all(adapter);
+                if (netif_running(netdev))
+                        netdev->netdev_ops->ndo_open(netdev);
         } else {
                 /* Turn off DCB */
                 if (adapter->flags & IXGBE_FLAG_DCB_ENABLED) {
                         if (netif_running(netdev))
-                                netdev->stop(netdev);
+                                netdev->netdev_ops->ndo_stop(netdev);
                         ixgbe_reset_interrupt_capability(adapter);
                         ixgbe_napi_del_all(adapter);
                         kfree(adapter->tx_ring);
@@ -170,11 +176,11 @@ static void ixgbe_dcbnl_set_state(struct net_device *netdev, u8 state)
                         ixgbe_init_interrupt_scheme(adapter);
                         ixgbe_napi_add_all(adapter);
                         if (netif_running(netdev))
-                                netdev->open(netdev);
-                } else {
-                        return;
+                                netdev->netdev_ops->ndo_open(netdev);
                 }
         }
+out:
+        return err;
 }
 
 static void ixgbe_dcbnl_get_perm_hw_addr(struct net_device *netdev,