hpsa: break hpsa_free_irqs_and_disable_msix into two functions
replace calls to hpsa_free_irqs_and_disable_msix with hpsa_free_irqs and hpsa_disable_interrupt_mode Reviewed-by: Scott Teel <scott.teel@pmcs.com> Reviewed-by: Kevin Barnett <kevin.barnett@pmcs.com> Reviewed-by: Tomas Henzl <thenzl@redhat.com> Reviewed-by: Hannes Reinecke <hare@Suse.de> Signed-off-by: Robert Elliott <elliott@hp.com> Signed-off-by: Don Brace <don.brace@pmcs.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: James Bottomley <JBottomley@Odin.com>
This commit is contained in:

committed by
James Bottomley

parent
ecf418d14f
commit
cc64c817a8
@@ -6351,10 +6351,20 @@ static int find_PCI_BAR_index(struct pci_dev *pdev, unsigned long pci_bar_addr)
|
|||||||
return -1;
|
return -1;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static void hpsa_disable_interrupt_mode(struct ctlr_info *h)
|
||||||
|
{
|
||||||
|
if (h->msix_vector) {
|
||||||
|
if (h->pdev->msix_enabled)
|
||||||
|
pci_disable_msix(h->pdev);
|
||||||
|
} else if (h->msi_vector) {
|
||||||
|
if (h->pdev->msi_enabled)
|
||||||
|
pci_disable_msi(h->pdev);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
/* If MSI/MSI-X is supported by the kernel we will try to enable it on
|
/* If MSI/MSI-X is supported by the kernel we will try to enable it on
|
||||||
* controllers that are capable. If not, we use legacy INTx mode.
|
* controllers that are capable. If not, we use legacy INTx mode.
|
||||||
*/
|
*/
|
||||||
|
|
||||||
static void hpsa_interrupt_mode(struct ctlr_info *h)
|
static void hpsa_interrupt_mode(struct ctlr_info *h)
|
||||||
{
|
{
|
||||||
#ifdef CONFIG_PCI_MSI
|
#ifdef CONFIG_PCI_MSI
|
||||||
@@ -6995,20 +7005,6 @@ static int hpsa_kdump_soft_reset(struct ctlr_info *h)
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static void hpsa_free_irqs_and_disable_msix(struct ctlr_info *h)
|
|
||||||
{
|
|
||||||
hpsa_free_irqs(h);
|
|
||||||
#ifdef CONFIG_PCI_MSI
|
|
||||||
if (h->msix_vector) {
|
|
||||||
if (h->pdev->msix_enabled)
|
|
||||||
pci_disable_msix(h->pdev);
|
|
||||||
} else if (h->msi_vector) {
|
|
||||||
if (h->pdev->msi_enabled)
|
|
||||||
pci_disable_msi(h->pdev);
|
|
||||||
}
|
|
||||||
#endif /* CONFIG_PCI_MSI */
|
|
||||||
}
|
|
||||||
|
|
||||||
static void hpsa_free_reply_queues(struct ctlr_info *h)
|
static void hpsa_free_reply_queues(struct ctlr_info *h)
|
||||||
{
|
{
|
||||||
int i;
|
int i;
|
||||||
@@ -7025,7 +7021,7 @@ static void hpsa_free_reply_queues(struct ctlr_info *h)
|
|||||||
|
|
||||||
static void hpsa_undo_allocations_after_kdump_soft_reset(struct ctlr_info *h)
|
static void hpsa_undo_allocations_after_kdump_soft_reset(struct ctlr_info *h)
|
||||||
{
|
{
|
||||||
hpsa_free_irqs_and_disable_msix(h);
|
hpsa_free_irqs(h);
|
||||||
hpsa_free_sg_chain_blocks(h);
|
hpsa_free_sg_chain_blocks(h);
|
||||||
hpsa_free_cmd_pool(h);
|
hpsa_free_cmd_pool(h);
|
||||||
kfree(h->ioaccel1_blockFetchTable);
|
kfree(h->ioaccel1_blockFetchTable);
|
||||||
@@ -7037,6 +7033,7 @@ static void hpsa_undo_allocations_after_kdump_soft_reset(struct ctlr_info *h)
|
|||||||
iounmap(h->transtable);
|
iounmap(h->transtable);
|
||||||
if (h->cfgtable)
|
if (h->cfgtable)
|
||||||
iounmap(h->cfgtable);
|
iounmap(h->cfgtable);
|
||||||
|
hpsa_disable_interrupt_mode(h);
|
||||||
pci_disable_device(h->pdev);
|
pci_disable_device(h->pdev);
|
||||||
pci_release_regions(h->pdev);
|
pci_release_regions(h->pdev);
|
||||||
kfree(h);
|
kfree(h);
|
||||||
@@ -7531,7 +7528,8 @@ static void hpsa_shutdown(struct pci_dev *pdev)
|
|||||||
*/
|
*/
|
||||||
hpsa_flush_cache(h);
|
hpsa_flush_cache(h);
|
||||||
h->access.set_intr_mask(h, HPSA_INTR_OFF);
|
h->access.set_intr_mask(h, HPSA_INTR_OFF);
|
||||||
hpsa_free_irqs_and_disable_msix(h);
|
hpsa_free_irqs(h);
|
||||||
|
hpsa_disable_interrupt_mode(h); /* pci_init 2 */
|
||||||
}
|
}
|
||||||
|
|
||||||
static void hpsa_free_device_info(struct ctlr_info *h)
|
static void hpsa_free_device_info(struct ctlr_info *h)
|
||||||
@@ -7562,7 +7560,10 @@ static void hpsa_remove_one(struct pci_dev *pdev)
|
|||||||
destroy_workqueue(h->rescan_ctlr_wq);
|
destroy_workqueue(h->rescan_ctlr_wq);
|
||||||
destroy_workqueue(h->resubmit_wq);
|
destroy_workqueue(h->resubmit_wq);
|
||||||
hpsa_unregister_scsi(h); /* unhook from SCSI subsystem */
|
hpsa_unregister_scsi(h); /* unhook from SCSI subsystem */
|
||||||
|
|
||||||
|
/* includes hpsa_free_irqs and hpsa_disable_interrupt_mode */
|
||||||
hpsa_shutdown(pdev);
|
hpsa_shutdown(pdev);
|
||||||
|
|
||||||
iounmap(h->vaddr);
|
iounmap(h->vaddr);
|
||||||
iounmap(h->transtable);
|
iounmap(h->transtable);
|
||||||
iounmap(h->cfgtable);
|
iounmap(h->cfgtable);
|
||||||
|
Reference in New Issue
Block a user