Commit 2946e82b authored by Robert Elliott's avatar Robert Elliott Committed by James Bottomley

hpsa: use scsi host_no as hpsa controller number

Rather than numbering the hpsa controllers with an
incrementing 0..n value (e.g., that shows up in
/proc/interrupts), use the scsi midlayer
host_no (e.g. matching /sys/class/scsi_host/hostNN).
Reviewed-by: default avatarScott Teel <scott.teel@pmcs.com>
Reviewed-by: default avatarKevin Barnett <kevin.barnett@pmcs.com>
Reviewed-by: default avatarTomas Henzl <thenzl@redhat.com>
Reviewed-by: default avatarHannes Reinecke <hare@Suse.de>
Signed-off-by: default avatarRobert Elliott <elliott@hp.com>
Signed-off-by: default avatarDon Brace <don.brace@pmcs.com>
Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Signed-off-by: default avatarJames Bottomley <JBottomley@Odin.com>
parent 73153fe5
...@@ -324,32 +324,35 @@ static int check_for_unit_attention(struct ctlr_info *h, ...@@ -324,32 +324,35 @@ static int check_for_unit_attention(struct ctlr_info *h,
switch (asc) { switch (asc) {
case STATE_CHANGED: case STATE_CHANGED:
dev_warn(&h->pdev->dev, dev_warn(&h->pdev->dev,
HPSA "%d: a state change detected, command retried\n", "%s: a state change detected, command retried\n",
h->ctlr); h->devname);
break; break;
case LUN_FAILED: case LUN_FAILED:
dev_warn(&h->pdev->dev, dev_warn(&h->pdev->dev,
HPSA "%d: LUN failure detected\n", h->ctlr); "%s: LUN failure detected\n", h->devname);
break; break;
case REPORT_LUNS_CHANGED: case REPORT_LUNS_CHANGED:
dev_warn(&h->pdev->dev, dev_warn(&h->pdev->dev,
HPSA "%d: report LUN data changed\n", h->ctlr); "%s: report LUN data changed\n", h->devname);
/* /*
* Note: this REPORT_LUNS_CHANGED condition only occurs on the external * Note: this REPORT_LUNS_CHANGED condition only occurs on the external
* target (array) devices. * target (array) devices.
*/ */
break; break;
case POWER_OR_RESET: case POWER_OR_RESET:
dev_warn(&h->pdev->dev, HPSA "%d: a power on " dev_warn(&h->pdev->dev,
"or device reset detected\n", h->ctlr); "%s: a power on or device reset detected\n",
h->devname);
break; break;
case UNIT_ATTENTION_CLEARED: case UNIT_ATTENTION_CLEARED:
dev_warn(&h->pdev->dev, HPSA "%d: unit attention " dev_warn(&h->pdev->dev,
"cleared by another initiator\n", h->ctlr); "%s: unit attention cleared by another initiator\n",
h->devname);
break; break;
default: default:
dev_warn(&h->pdev->dev, HPSA "%d: unknown " dev_warn(&h->pdev->dev,
"unit attention detected\n", h->ctlr); "%s: unknown unit attention detected\n",
h->devname);
break; break;
} }
return 1; return 1;
...@@ -4764,22 +4767,16 @@ static int hpsa_scan_finished(struct Scsi_Host *sh, ...@@ -4764,22 +4767,16 @@ static int hpsa_scan_finished(struct Scsi_Host *sh,
return finished; return finished;
} }
static void hpsa_unregister_scsi(struct ctlr_info *h) static int hpsa_scsi_host_alloc(struct ctlr_info *h)
{
/* we are being forcibly unloaded, and may not refuse. */
scsi_remove_host(h->scsi_host);
scsi_host_put(h->scsi_host);
h->scsi_host = NULL;
}
static int hpsa_register_scsi(struct ctlr_info *h)
{ {
struct Scsi_Host *sh; struct Scsi_Host *sh;
int error; int error;
sh = scsi_host_alloc(&hpsa_driver_template, sizeof(h)); sh = scsi_host_alloc(&hpsa_driver_template, sizeof(h));
if (sh == NULL) if (sh == NULL) {
goto fail; dev_err(&h->pdev->dev, "scsi_host_alloc failed\n");
return -ENOMEM;
}
sh->io_port = 0; sh->io_port = 0;
sh->n_io_port = 0; sh->n_io_port = 0;
...@@ -4791,7 +4788,6 @@ static int hpsa_register_scsi(struct ctlr_info *h) ...@@ -4791,7 +4788,6 @@ static int hpsa_register_scsi(struct ctlr_info *h)
sh->can_queue = h->nr_cmds - HPSA_NRESERVED_CMDS; sh->can_queue = h->nr_cmds - HPSA_NRESERVED_CMDS;
sh->cmd_per_lun = sh->can_queue; sh->cmd_per_lun = sh->can_queue;
sh->sg_tablesize = h->maxsgentries; sh->sg_tablesize = h->maxsgentries;
h->scsi_host = sh;
sh->hostdata[0] = (unsigned long) h; sh->hostdata[0] = (unsigned long) h;
sh->irq = h->intr[h->intr_mode]; sh->irq = h->intr[h->intr_mode];
sh->unique_id = sh->irq; sh->unique_id = sh->irq;
...@@ -4800,24 +4796,24 @@ static int hpsa_register_scsi(struct ctlr_info *h) ...@@ -4800,24 +4796,24 @@ static int hpsa_register_scsi(struct ctlr_info *h)
dev_err(&h->pdev->dev, dev_err(&h->pdev->dev,
"%s: scsi_init_shared_tag_map failed for controller %d\n", "%s: scsi_init_shared_tag_map failed for controller %d\n",
__func__, h->ctlr); __func__, h->ctlr);
goto fail_host_put; scsi_host_put(sh);
} return error;
error = scsi_add_host(sh, &h->pdev->dev);
if (error) {
dev_err(&h->pdev->dev, "%s: scsi_add_host failed for controller %d\n",
__func__, h->ctlr);
goto fail_host_put;
} }
scsi_scan_host(sh); h->scsi_host = sh;
return 0; return 0;
}
fail_host_put: static int hpsa_scsi_add_host(struct ctlr_info *h)
scsi_host_put(sh); {
return error; int rv;
fail:
dev_err(&h->pdev->dev, "%s: scsi_host_alloc" rv = scsi_add_host(h->scsi_host, &h->pdev->dev);
" failed for controller %d\n", __func__, h->ctlr); if (rv) {
return -ENOMEM; dev_err(&h->pdev->dev, "scsi_add_host failed\n");
return rv;
}
scsi_scan_host(h->scsi_host);
return 0;
} }
/* /*
...@@ -7483,7 +7479,9 @@ static void hpsa_undo_allocations_after_kdump_soft_reset(struct ctlr_info *h) ...@@ -7483,7 +7479,9 @@ static void hpsa_undo_allocations_after_kdump_soft_reset(struct ctlr_info *h)
hpsa_free_sg_chain_blocks(h); /* init_one 6 */ hpsa_free_sg_chain_blocks(h); /* init_one 6 */
hpsa_free_cmd_pool(h); /* init_one 5 */ hpsa_free_cmd_pool(h); /* init_one 5 */
hpsa_free_irqs(h); /* init_one 4 */ hpsa_free_irqs(h); /* init_one 4 */
hpsa_free_pci_init(h); /* init_one 3 */ scsi_host_put(h->scsi_host); /* init_one 3 */
h->scsi_host = NULL; /* init_one 3 */
hpsa_free_pci_init(h); /* init_one 2_5 */
free_percpu(h->lockup_detected); /* init_one 2 */ free_percpu(h->lockup_detected); /* init_one 2 */
h->lockup_detected = NULL; /* init_one 2 */ h->lockup_detected = NULL; /* init_one 2 */
if (h->resubmit_wq) { if (h->resubmit_wq) {
...@@ -7793,9 +7791,15 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) ...@@ -7793,9 +7791,15 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
rc = hpsa_pci_init(h); rc = hpsa_pci_init(h);
if (rc) if (rc)
goto clean2; /* lockup, aer/h */ goto clean2; /* lu, aer/h */
/* relies on h-> settings made by hpsa_pci_init, including
* interrupt_mode h->intr */
rc = hpsa_scsi_host_alloc(h);
if (rc)
goto clean2_5; /* pci, lu, aer/h */
sprintf(h->devname, HPSA "%d", number_of_controllers); sprintf(h->devname, HPSA "%d", h->scsi_host->host_no);
h->ctlr = number_of_controllers; h->ctlr = number_of_controllers;
number_of_controllers++; number_of_controllers++;
...@@ -7809,7 +7813,7 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) ...@@ -7809,7 +7813,7 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
dac = 0; dac = 0;
} else { } else {
dev_err(&pdev->dev, "no suitable DMA available\n"); dev_err(&pdev->dev, "no suitable DMA available\n");
goto clean3; /* pci, lockup, aer/h */ goto clean3; /* shost, pci, lu, aer/h */
} }
} }
...@@ -7818,13 +7822,13 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) ...@@ -7818,13 +7822,13 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
rc = hpsa_request_irqs(h, do_hpsa_intr_msi, do_hpsa_intr_intx); rc = hpsa_request_irqs(h, do_hpsa_intr_msi, do_hpsa_intr_intx);
if (rc) if (rc)
goto clean3; /* pci, lockup, aer/h */ goto clean3; /* shost, pci, lu, aer/h */
rc = hpsa_alloc_cmd_pool(h); rc = hpsa_alloc_cmd_pool(h);
if (rc) if (rc)
goto clean4; /* irq, pci, lockup, aer/h */ goto clean4; /* irq, shost, pci, lu, aer/h */
rc = hpsa_alloc_sg_chain_blocks(h); rc = hpsa_alloc_sg_chain_blocks(h);
if (rc) if (rc)
goto clean5; /* cmd, irq, pci, lockup, aer/h */ goto clean5; /* cmd, irq, shost, pci, lu, aer/h */
init_waitqueue_head(&h->scan_wait_queue); init_waitqueue_head(&h->scan_wait_queue);
init_waitqueue_head(&h->abort_cmd_wait_queue); init_waitqueue_head(&h->abort_cmd_wait_queue);
init_waitqueue_head(&h->abort_sync_wait_queue); init_waitqueue_head(&h->abort_sync_wait_queue);
...@@ -7833,11 +7837,16 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) ...@@ -7833,11 +7837,16 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
pci_set_drvdata(pdev, h); pci_set_drvdata(pdev, h);
h->ndevices = 0; h->ndevices = 0;
h->hba_mode_enabled = 0; h->hba_mode_enabled = 0;
h->scsi_host = NULL;
spin_lock_init(&h->devlock); spin_lock_init(&h->devlock);
rc = hpsa_put_ctlr_into_performant_mode(h); rc = hpsa_put_ctlr_into_performant_mode(h);
if (rc) if (rc)
goto clean6; /* sg, cmd, irq, pci, lockup, aer/h */ goto clean6; /* sg, cmd, irq, shost, pci, lu, aer/h */
/* hook into SCSI subsystem */
rc = hpsa_scsi_add_host(h);
if (rc)
goto clean7; /* perf, sg, cmd, irq, shost, pci, lu, aer/h */
/* create the resubmit workqueue */ /* create the resubmit workqueue */
h->rescan_ctlr_wq = hpsa_create_controller_wq(h, "rescan"); h->rescan_ctlr_wq = hpsa_create_controller_wq(h, "rescan");
...@@ -7892,7 +7901,7 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) ...@@ -7892,7 +7901,7 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
rc = hpsa_kdump_soft_reset(h); rc = hpsa_kdump_soft_reset(h);
if (rc) if (rc)
/* Neither hard nor soft reset worked, we're hosed. */ /* Neither hard nor soft reset worked, we're hosed. */
goto clean8; goto clean9;
dev_info(&h->pdev->dev, "Board READY.\n"); dev_info(&h->pdev->dev, "Board READY.\n");
dev_info(&h->pdev->dev, dev_info(&h->pdev->dev,
...@@ -7927,9 +7936,6 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) ...@@ -7927,9 +7936,6 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
h->access.set_intr_mask(h, HPSA_INTR_ON); h->access.set_intr_mask(h, HPSA_INTR_ON);
hpsa_hba_inquiry(h); hpsa_hba_inquiry(h);
rc = hpsa_register_scsi(h); /* hook ourselves into SCSI subsystem */
if (rc)
goto clean8; /* wq, perf, sg, cmd, irq, pci, lockup, aer/h */
/* Monitor the controller for firmware lockups */ /* Monitor the controller for firmware lockups */
h->heartbeat_sample_interval = HEARTBEAT_SAMPLE_INTERVAL; h->heartbeat_sample_interval = HEARTBEAT_SAMPLE_INTERVAL;
...@@ -7941,20 +7947,23 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent) ...@@ -7941,20 +7947,23 @@ static int hpsa_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
h->heartbeat_sample_interval); h->heartbeat_sample_interval);
return 0; return 0;
clean8: /* perf, sg, cmd, irq, pci, lockup, aer/h */ clean9: /* wq, sh, perf, sg, cmd, irq, shost, pci, lu, aer/h */
kfree(h->hba_inquiry_data); kfree(h->hba_inquiry_data);
clean7: /* perf, sg, cmd, irq, pci, lockup, aer/h */ clean7: /* perf, sg, cmd, irq, shost, pci, lu, aer/h */
hpsa_free_performant_mode(h); hpsa_free_performant_mode(h);
h->access.set_intr_mask(h, HPSA_INTR_OFF); h->access.set_intr_mask(h, HPSA_INTR_OFF);
clean6: /* sg, cmd, irq, pci, lockup, wq/aer/h */ clean6: /* sg, cmd, irq, pci, lockup, wq/aer/h */
hpsa_free_sg_chain_blocks(h); hpsa_free_sg_chain_blocks(h);
clean5: /* cmd, irq, pci, lockup, aer/h */ clean5: /* cmd, irq, shost, pci, lu, aer/h */
hpsa_free_cmd_pool(h); hpsa_free_cmd_pool(h);
clean4: /* irq, pci, lockup, aer/h */ clean4: /* irq, shost, pci, lu, aer/h */
hpsa_free_irqs(h); hpsa_free_irqs(h);
clean3: /* pci, lockup, aer/h */ clean3: /* shost, pci, lu, aer/h */
scsi_host_put(h->scsi_host);
h->scsi_host = NULL;
clean2_5: /* pci, lu, aer/h */
hpsa_free_pci_init(h); hpsa_free_pci_init(h);
clean2: /* lockup, aer/h */ clean2: /* lu, aer/h */
if (h->lockup_detected) { if (h->lockup_detected) {
free_percpu(h->lockup_detected); free_percpu(h->lockup_detected);
h->lockup_detected = NULL; h->lockup_detected = NULL;
...@@ -8053,17 +8062,22 @@ static void hpsa_remove_one(struct pci_dev *pdev) ...@@ -8053,17 +8062,22 @@ static void hpsa_remove_one(struct pci_dev *pdev)
hpsa_free_device_info(h); /* scan */ hpsa_free_device_info(h); /* scan */
hpsa_unregister_scsi(h); /* init_one 9 */ kfree(h->hba_inquiry_data); /* init_one 10 */
kfree(h->hba_inquiry_data); /* init_one 9 */ h->hba_inquiry_data = NULL; /* init_one 10 */
h->hba_inquiry_data = NULL; /* init_one 9 */ if (h->scsi_host)
scsi_remove_host(h->scsi_host); /* init_one 8 */
hpsa_free_ioaccel2_sg_chain_blocks(h);
hpsa_free_performant_mode(h); /* init_one 7 */ hpsa_free_performant_mode(h); /* init_one 7 */
hpsa_free_sg_chain_blocks(h); /* init_one 6 */ hpsa_free_sg_chain_blocks(h); /* init_one 6 */
hpsa_free_cmd_pool(h); /* init_one 5 */ hpsa_free_cmd_pool(h); /* init_one 5 */
/* hpsa_free_irqs already called via hpsa_shutdown init_one 4 */ /* hpsa_free_irqs already called via hpsa_shutdown init_one 4 */
scsi_host_put(h->scsi_host); /* init_one 3 */
h->scsi_host = NULL; /* init_one 3 */
/* includes hpsa_disable_interrupt_mode - pci_init 2 */ /* includes hpsa_disable_interrupt_mode - pci_init 2 */
hpsa_free_pci_init(h); /* init_one 3 */ hpsa_free_pci_init(h); /* init_one 2.5 */
free_percpu(h->lockup_detected); /* init_one 2 */ free_percpu(h->lockup_detected); /* init_one 2 */
h->lockup_detected = NULL; /* init_one 2 */ h->lockup_detected = NULL; /* init_one 2 */
......
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