nfp: remove nfp_net_irqs_wanted()
authorJakub Kicinski <jakub.kicinski@netronome.com>
Mon, 31 Oct 2016 20:43:20 +0000 (20:43 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 1 Nov 2016 15:05:00 +0000 (11:05 -0400)
nfp_net_irqs_wanted() doesn't really encapsulate much logic,
remove it and inline the calculations.

Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/netronome/nfp/nfp_net_common.c

index 49a1f54e07874983c957fa9d1b828389850f2615..f26c3927746fdf1ad26ecc4344afe91321949eb1 100644 (file)
@@ -307,28 +307,6 @@ static int nfp_net_msix_alloc(struct nfp_net *nn, int nr_vecs)
        return nvecs;
 }
 
-/**
- * nfp_net_irqs_wanted() - Work out how many interrupt vectors we want
- * @nn:       NFP Network structure
- *
- * We want a vector per CPU (or ring), whatever is smaller plus
- * NFP_NET_NON_Q_VECTORS for LSC etc.
- *
- * Return: Number of interrupts wanted
- */
-static int nfp_net_irqs_wanted(struct nfp_net *nn)
-{
-       unsigned int vecs;
-       int ncpus;
-
-       ncpus = num_online_cpus();
-
-       vecs = max_t(int, nn->num_tx_rings, nn->num_rx_rings);
-       vecs = min_t(int, vecs, ncpus);
-
-       return vecs + NFP_NET_NON_Q_VECTORS;
-}
-
 /**
  * nfp_net_irqs_alloc() - allocates MSI-X irqs
  * @nn:       NFP Network structure
@@ -339,7 +317,7 @@ int nfp_net_irqs_alloc(struct nfp_net *nn)
 {
        int wanted_irqs;
 
-       wanted_irqs = nfp_net_irqs_wanted(nn);
+       wanted_irqs = nn->num_r_vecs + NFP_NET_NON_Q_VECTORS;
 
        nn->num_irqs = nfp_net_msix_alloc(nn, wanted_irqs);
        if (nn->num_irqs == 0) {
@@ -2726,6 +2704,9 @@ struct nfp_net *nfp_net_netdev_alloc(struct pci_dev *pdev,
        nn->num_tx_rings = min_t(int, nqs, max_tx_rings);
        nn->num_rx_rings = min_t(int, nqs, max_rx_rings);
 
+       nn->num_r_vecs = max(nn->num_tx_rings, nn->num_rx_rings);
+       nn->num_r_vecs = min_t(unsigned int, nn->num_r_vecs, num_online_cpus());
+
        nn->txd_cnt = NFP_NET_TX_DESCS_DEFAULT;
        nn->rxd_cnt = NFP_NET_RX_DESCS_DEFAULT;