浏览代码

[PATCH 2/2] cciss: remove calls to pci_disable_device

This patch removes calls to pci_disable_device except in fail_all_cmds. The
pci_disable_device function does something nasty to Smart Array controllers
that pci_enable_device does not undo. So if the driver is unloaded it cannot be
reloaded.

Also, customers can disable any pci device via the ROM Based Setup Utility
(RBSU). If the customer has disabled the controller we should not try to
blindly enable the card from the driver.  Please consider this for inclusion.

Signed-off-by: Mike Miller <mike.miller@hp.com>
Acked-by: Alan Cox <alan@redhat.com>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
mike.miller@hp.com 18 年之前
父节点
当前提交
c4977f596c
共有 1 个文件被更改,包括 1 次插入5 次删除
  1. 1 5
      drivers/block/cciss.c

+ 1 - 5
drivers/block/cciss.c

@@ -3006,10 +3006,8 @@ static int cciss_pci_init(ctlr_info_t *c, struct pci_dev *pdev)
 
 
       err_out_free_res:
       err_out_free_res:
 	pci_release_regions(pdev);
 	pci_release_regions(pdev);
-
-      err_out_disable_pdev:
-	pci_disable_device(pdev);
 	return err;
 	return err;
+
 }
 }
 
 
 /*
 /*
@@ -3383,7 +3381,6 @@ static int __devinit cciss_init_one(struct pci_dev *pdev,
 			blk_cleanup_queue(drv->queue);
 			blk_cleanup_queue(drv->queue);
 	}
 	}
 	pci_release_regions(pdev);
 	pci_release_regions(pdev);
-	pci_disable_device(pdev);
 	pci_set_drvdata(pdev, NULL);
 	pci_set_drvdata(pdev, NULL);
 	free_hba(i);
 	free_hba(i);
 	return -1;
 	return -1;
@@ -3453,7 +3450,6 @@ static void __devexit cciss_remove_one(struct pci_dev *pdev)
 	kfree(hba[i]->scsi_rejects.complete);
 	kfree(hba[i]->scsi_rejects.complete);
 #endif
 #endif
 	pci_release_regions(pdev);
 	pci_release_regions(pdev);
-	pci_disable_device(pdev);
 	pci_set_drvdata(pdev, NULL);
 	pci_set_drvdata(pdev, NULL);
 	free_hba(i);
 	free_hba(i);
 }
 }