ixgbe: Move configuration of set_real_num_rx/tx_queues into open
authorAlexander Duyck <alexander.h.duyck@intel.com>
Thu, 12 Jul 2012 05:52:53 +0000 (05:52 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Wed, 18 Jul 2012 01:55:13 +0000 (18:55 -0700)
It makes much more sense for us to configure the real number of Tx and Rx
queues in the ixgbe_open call than it does in ixgbe_set_num_queues.  By
setting the number in ixgbe_open we can avoid a number of unecessary
updates and only have to make the calls once.

Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Tested-by: Phil Schmitt <phillip.j.schmitt@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/ixgbe/ixgbe_lib.c
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c

index d308e714017104e14107168a98c259dd21d5346d..c03d771c5eb92c91ed688b865d9c8a2b83df6dd7 100644 (file)
@@ -349,7 +349,7 @@ static bool ixgbe_set_rss_queues(struct ixgbe_adapter *adapter)
  * fallthrough conditions.
  *
  **/
-static int ixgbe_set_num_queues(struct ixgbe_adapter *adapter)
+static void ixgbe_set_num_queues(struct ixgbe_adapter *adapter)
 {
        /* Start with base case */
        adapter->num_rx_queues = 1;
@@ -358,29 +358,14 @@ static int ixgbe_set_num_queues(struct ixgbe_adapter *adapter)
        adapter->num_rx_queues_per_pool = 1;
 
        if (ixgbe_set_sriov_queues(adapter))
-               goto done;
+               return;
 
 #ifdef CONFIG_IXGBE_DCB
        if (ixgbe_set_dcb_queues(adapter))
-               goto done;
+               return;
 
 #endif
-       if (ixgbe_set_rss_queues(adapter))
-               goto done;
-
-       /* fallback to base case */
-       adapter->num_rx_queues = 1;
-       adapter->num_tx_queues = 1;
-
-done:
-       if ((adapter->netdev->reg_state == NETREG_UNREGISTERED) ||
-           (adapter->netdev->reg_state == NETREG_UNREGISTERING))
-               return 0;
-
-       /* Notify the stack of the (possibly) reduced queue counts. */
-       netif_set_real_num_tx_queues(adapter->netdev, adapter->num_tx_queues);
-       return netif_set_real_num_rx_queues(adapter->netdev,
-                                           adapter->num_rx_queues);
+       ixgbe_set_rss_queues(adapter);
 }
 
 static void ixgbe_acquire_msix_vectors(struct ixgbe_adapter *adapter,
@@ -710,11 +695,10 @@ static void ixgbe_reset_interrupt_capability(struct ixgbe_adapter *adapter)
  * Attempt to configure the interrupts using the best available
  * capabilities of the hardware and the kernel.
  **/
-static int ixgbe_set_interrupt_capability(struct ixgbe_adapter *adapter)
+static void ixgbe_set_interrupt_capability(struct ixgbe_adapter *adapter)
 {
        struct ixgbe_hw *hw = &adapter->hw;
-       int err = 0;
-       int vector, v_budget;
+       int vector, v_budget, err;
 
        /*
         * It's easy to be greedy for MSI-X vectors, but it really
@@ -747,7 +731,7 @@ static int ixgbe_set_interrupt_capability(struct ixgbe_adapter *adapter)
                ixgbe_acquire_msix_vectors(adapter, v_budget);
 
                if (adapter->flags & IXGBE_FLAG_MSIX_ENABLED)
-                       goto out;
+                       return;
        }
 
        adapter->flags &= ~IXGBE_FLAG_DCB_ENABLED;
@@ -762,25 +746,17 @@ static int ixgbe_set_interrupt_capability(struct ixgbe_adapter *adapter)
        if (adapter->flags & IXGBE_FLAG_SRIOV_ENABLED)
                ixgbe_disable_sriov(adapter);
 
-       err = ixgbe_set_num_queues(adapter);
-       if (err)
-               return err;
-
+       ixgbe_set_num_queues(adapter);
        adapter->num_q_vectors = 1;
 
        err = pci_enable_msi(adapter->pdev);
-       if (!err) {
-               adapter->flags |= IXGBE_FLAG_MSI_ENABLED;
-       } else {
+       if (err) {
                netif_printk(adapter, hw, KERN_DEBUG, adapter->netdev,
                             "Unable to allocate MSI interrupt, "
                             "falling back to legacy.  Error: %d\n", err);
-               /* reset err */
-               err = 0;
+               return;
        }
-
-out:
-       return err;
+       adapter->flags |= IXGBE_FLAG_MSI_ENABLED;
 }
 
 /**
@@ -798,15 +774,10 @@ int ixgbe_init_interrupt_scheme(struct ixgbe_adapter *adapter)
        int err;
 
        /* Number of supported queues */
-       err = ixgbe_set_num_queues(adapter);
-       if (err)
-               return err;
+       ixgbe_set_num_queues(adapter);
 
-       err = ixgbe_set_interrupt_capability(adapter);
-       if (err) {
-               e_dev_err("Unable to setup interrupt capabilities\n");
-               goto err_set_interrupt;
-       }
+       /* Set interrupt mode */
+       ixgbe_set_interrupt_capability(adapter);
 
        err = ixgbe_alloc_q_vectors(adapter);
        if (err) {
@@ -826,7 +797,6 @@ int ixgbe_init_interrupt_scheme(struct ixgbe_adapter *adapter)
 
 err_alloc_q_vectors:
        ixgbe_reset_interrupt_capability(adapter);
-err_set_interrupt:
        return err;
 }
 
index 373342f0b807b1b9e30d42b7611e9b2f7e1b969f..7f2aa220501e05cb1039c6173fae55a0c923d1ba 100644 (file)
@@ -4798,10 +4798,26 @@ static int ixgbe_open(struct net_device *netdev)
        if (err)
                goto err_req_irq;
 
+       /* Notify the stack of the actual queue counts. */
+       err = netif_set_real_num_tx_queues(netdev,
+                                          adapter->num_rx_pools > 1 ? 1 :
+                                          adapter->num_tx_queues);
+       if (err)
+               goto err_set_queues;
+
+
+       err = netif_set_real_num_rx_queues(netdev,
+                                          adapter->num_rx_pools > 1 ? 1 :
+                                          adapter->num_rx_queues);
+       if (err)
+               goto err_set_queues;
+
        ixgbe_up_complete(adapter);
 
        return 0;
 
+err_set_queues:
+       ixgbe_free_irq(adapter);
 err_req_irq:
        ixgbe_free_all_rx_resources(adapter);
 err_setup_rx:
@@ -4864,23 +4880,19 @@ static int ixgbe_resume(struct pci_dev *pdev)
 
        pci_wake_from_d3(pdev, false);
 
-       rtnl_lock();
-       err = ixgbe_init_interrupt_scheme(adapter);
-       rtnl_unlock();
-       if (err) {
-               e_dev_err("Cannot initialize interrupts for device\n");
-               return err;
-       }
-
        ixgbe_reset(adapter);
 
        IXGBE_WRITE_REG(&adapter->hw, IXGBE_WUS, ~0);
 
-       if (netif_running(netdev)) {
+       rtnl_lock();
+       err = ixgbe_init_interrupt_scheme(adapter);
+       if (!err && netif_running(netdev))
                err = ixgbe_open(netdev);
-               if (err)
-                       return err;
-       }
+
+       rtnl_unlock();
+
+       if (err)
+               return err;
 
        netif_device_attach(netdev);