Commit be339aee authored by Dhananjay Phadke's avatar Dhananjay Phadke Committed by David S. Miller

netxen: fix irq tear down and msix leak.

o Fix the order of irq and hardware context teardown.
  Also synchronize the interrupt in dev close() before
  releasing tx buffers.

o Fix possible msi-x vector leak if available vectors are
  less than requested.

o Request multiple msix vectors only if hardware supports
  multiple rx queues.
Signed-off-by: default avatarDhananjay Phadke <dhananjay@netxen.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 7a2469ce
...@@ -175,12 +175,6 @@ netxen_napi_add(struct netxen_adapter *adapter, struct net_device *netdev) ...@@ -175,12 +175,6 @@ netxen_napi_add(struct netxen_adapter *adapter, struct net_device *netdev)
struct nx_host_sds_ring *sds_ring; struct nx_host_sds_ring *sds_ring;
struct netxen_recv_context *recv_ctx = &adapter->recv_ctx; struct netxen_recv_context *recv_ctx = &adapter->recv_ctx;
if ((adapter->flags & NETXEN_NIC_MSIX_ENABLED) &&
adapter->rss_supported)
adapter->max_sds_rings = (num_online_cpus() >= 4) ? 4 : 2;
else
adapter->max_sds_rings = 1;
if (netxen_alloc_sds_rings(recv_ctx, adapter->max_sds_rings)) if (netxen_alloc_sds_rings(recv_ctx, adapter->max_sds_rings))
return 1; return 1;
...@@ -216,8 +210,9 @@ netxen_napi_disable(struct netxen_adapter *adapter) ...@@ -216,8 +210,9 @@ netxen_napi_disable(struct netxen_adapter *adapter)
for (ring = 0; ring < adapter->max_sds_rings; ring++) { for (ring = 0; ring < adapter->max_sds_rings; ring++) {
sds_ring = &recv_ctx->sds_rings[ring]; sds_ring = &recv_ctx->sds_rings[ring];
netxen_nic_disable_int(sds_ring);
napi_disable(&sds_ring->napi); napi_disable(&sds_ring->napi);
netxen_nic_disable_int(sds_ring);
synchronize_irq(sds_ring->irq);
} }
} }
...@@ -407,11 +402,11 @@ static void netxen_set_msix_bit(struct pci_dev *pdev, int enable) ...@@ -407,11 +402,11 @@ static void netxen_set_msix_bit(struct pci_dev *pdev, int enable)
} }
} }
static void netxen_init_msix_entries(struct netxen_adapter *adapter) static void netxen_init_msix_entries(struct netxen_adapter *adapter, int count)
{ {
int i; int i;
for (i = 0; i < MSIX_ENTRIES_PER_ADAPTER; i++) for (i = 0; i < count; i++)
adapter->msix_entries[i].entry = i; adapter->msix_entries[i].entry = i;
} }
...@@ -496,6 +491,15 @@ netxen_setup_intr(struct netxen_adapter *adapter) ...@@ -496,6 +491,15 @@ netxen_setup_intr(struct netxen_adapter *adapter)
{ {
struct netxen_legacy_intr_set *legacy_intrp; struct netxen_legacy_intr_set *legacy_intrp;
struct pci_dev *pdev = adapter->pdev; struct pci_dev *pdev = adapter->pdev;
int err, num_msix;
if (adapter->rss_supported) {
num_msix = (num_online_cpus() >= MSIX_ENTRIES_PER_ADAPTER) ?
MSIX_ENTRIES_PER_ADAPTER : 2;
} else
num_msix = 1;
adapter->max_sds_rings = 1;
adapter->flags &= ~(NETXEN_NIC_MSI_ENABLED | NETXEN_NIC_MSIX_ENABLED); adapter->flags &= ~(NETXEN_NIC_MSI_ENABLED | NETXEN_NIC_MSIX_ENABLED);
...@@ -512,26 +516,35 @@ netxen_setup_intr(struct netxen_adapter *adapter) ...@@ -512,26 +516,35 @@ netxen_setup_intr(struct netxen_adapter *adapter)
if (adapter->msix_supported) { if (adapter->msix_supported) {
netxen_init_msix_entries(adapter); netxen_init_msix_entries(adapter, num_msix);
if (pci_enable_msix(pdev, adapter->msix_entries, err = pci_enable_msix(pdev, adapter->msix_entries, num_msix);
MSIX_ENTRIES_PER_ADAPTER)) if (err == 0) {
goto request_msi;
adapter->flags |= NETXEN_NIC_MSIX_ENABLED; adapter->flags |= NETXEN_NIC_MSIX_ENABLED;
netxen_set_msix_bit(pdev, 1); netxen_set_msix_bit(pdev, 1);
if (adapter->rss_supported)
adapter->max_sds_rings = num_msix;
dev_info(&pdev->dev, "using msi-x interrupts\n"); dev_info(&pdev->dev, "using msi-x interrupts\n");
return;
}
if (err > 0)
pci_disable_msix(pdev);
/* fall through for msi */
}
} else {
request_msi:
if (use_msi && !pci_enable_msi(pdev)) { if (use_msi && !pci_enable_msi(pdev)) {
adapter->flags |= NETXEN_NIC_MSI_ENABLED; adapter->flags |= NETXEN_NIC_MSI_ENABLED;
adapter->msi_tgt_status = adapter->msi_tgt_status =
msi_tgt_status[adapter->ahw.pci_func]; msi_tgt_status[adapter->ahw.pci_func];
dev_info(&pdev->dev, "using msi interrupts\n"); dev_info(&pdev->dev, "using msi interrupts\n");
} else return;
}
dev_info(&pdev->dev, "using legacy interrupts\n"); dev_info(&pdev->dev, "using legacy interrupts\n");
adapter->msix_entries[0].vector = pdev->irq; adapter->msix_entries[0].vector = pdev->irq;
}
} }
static void static void
...@@ -767,7 +780,7 @@ netxen_nic_request_irq(struct netxen_adapter *adapter) ...@@ -767,7 +780,7 @@ netxen_nic_request_irq(struct netxen_adapter *adapter)
for (ring = 0; ring < adapter->max_sds_rings; ring++) { for (ring = 0; ring < adapter->max_sds_rings; ring++) {
sds_ring = &recv_ctx->sds_rings[ring]; sds_ring = &recv_ctx->sds_rings[ring];
sprintf(sds_ring->name, "%16s[%d]", netdev->name, ring); sprintf(sds_ring->name, "%s[%d]", netdev->name, ring);
err = request_irq(sds_ring->irq, handler, err = request_irq(sds_ring->irq, handler,
flags, sds_ring->name, sds_ring); flags, sds_ring->name, sds_ring);
if (err) if (err)
...@@ -830,8 +843,6 @@ netxen_nic_down(struct netxen_adapter *adapter, struct net_device *netdev) ...@@ -830,8 +843,6 @@ netxen_nic_down(struct netxen_adapter *adapter, struct net_device *netdev)
{ {
netif_carrier_off(netdev); netif_carrier_off(netdev);
netif_stop_queue(netdev); netif_stop_queue(netdev);
smp_mb();
netxen_napi_disable(adapter);
if (adapter->stop_port) if (adapter->stop_port)
adapter->stop_port(adapter); adapter->stop_port(adapter);
...@@ -839,6 +850,8 @@ netxen_nic_down(struct netxen_adapter *adapter, struct net_device *netdev) ...@@ -839,6 +850,8 @@ netxen_nic_down(struct netxen_adapter *adapter, struct net_device *netdev)
if (NX_IS_REVISION_P3(adapter->ahw.revision_id)) if (NX_IS_REVISION_P3(adapter->ahw.revision_id))
netxen_p3_free_mac_list(adapter); netxen_p3_free_mac_list(adapter);
netxen_napi_disable(adapter);
netxen_release_tx_buffers(adapter); netxen_release_tx_buffers(adapter);
FLUSH_SCHEDULED_WORK(); FLUSH_SCHEDULED_WORK();
...@@ -917,10 +930,9 @@ netxen_nic_attach(struct netxen_adapter *adapter) ...@@ -917,10 +930,9 @@ netxen_nic_attach(struct netxen_adapter *adapter)
static void static void
netxen_nic_detach(struct netxen_adapter *adapter) netxen_nic_detach(struct netxen_adapter *adapter)
{ {
netxen_nic_free_irq(adapter);
netxen_release_rx_buffers(adapter); netxen_release_rx_buffers(adapter);
netxen_free_hw_resources(adapter); netxen_free_hw_resources(adapter);
netxen_nic_free_irq(adapter);
netxen_free_sw_resources(adapter); netxen_free_sw_resources(adapter);
adapter->is_up = 0; adapter->is_up = 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment