diff options
author | brking@us.ibm.com <brking@us.ibm.com> | 2005-11-17 10:35:12 -0500 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.(none)> | 2005-12-13 20:10:58 -0500 |
commit | ce155ccecd4094e7b5e68058d26db691713240fc (patch) | |
tree | 1bc690d3f0555b05c3be587267c01135fa829563 | |
parent | ed7e8ef7f12f5c3c8bbb85eeb0a1ded91c7c5dbf (diff) |
[SCSI] ipr: Driver initialization fix for kexec/kdump
When kexec booting a kernel when the previous kernel did not
call ipr's shutdown method, the ipr adapter does not get
properly initialized, which can result in the ipr adapter
completing commands issued by the previous kernel. Fix ipr
to detect this scenario by reading the adapter's interrupt
mask register and the microprocessor interrupt register.
If the interrupt mask register indicates that interrupts
are enabled or the reset alert bit is set when the card is
probed, this means the card is in an unknown state and we
hard reset the card.
Signed-off-by: Brian King <brking@us.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
-rw-r--r-- | drivers/scsi/ipr.c | 17 | ||||
-rw-r--r-- | drivers/scsi/ipr.h | 5 |
2 files changed, 19 insertions, 3 deletions
diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c index fa2cb3582cfa..b6714da4d6e2 100644 --- a/drivers/scsi/ipr.c +++ b/drivers/scsi/ipr.c | |||
@@ -5887,7 +5887,12 @@ static int __devinit ipr_probe_ioa_part2(struct ipr_ioa_cfg *ioa_cfg) | |||
5887 | ENTER; | 5887 | ENTER; |
5888 | spin_lock_irqsave(ioa_cfg->host->host_lock, host_lock_flags); | 5888 | spin_lock_irqsave(ioa_cfg->host->host_lock, host_lock_flags); |
5889 | dev_dbg(&ioa_cfg->pdev->dev, "ioa_cfg adx: 0x%p\n", ioa_cfg); | 5889 | dev_dbg(&ioa_cfg->pdev->dev, "ioa_cfg adx: 0x%p\n", ioa_cfg); |
5890 | _ipr_initiate_ioa_reset(ioa_cfg, ipr_reset_enable_ioa, IPR_SHUTDOWN_NONE); | 5890 | if (ioa_cfg->needs_hard_reset) { |
5891 | ioa_cfg->needs_hard_reset = 0; | ||
5892 | ipr_initiate_ioa_reset(ioa_cfg, IPR_SHUTDOWN_NONE); | ||
5893 | } else | ||
5894 | _ipr_initiate_ioa_reset(ioa_cfg, ipr_reset_enable_ioa, | ||
5895 | IPR_SHUTDOWN_NONE); | ||
5891 | 5896 | ||
5892 | spin_unlock_irqrestore(ioa_cfg->host->host_lock, host_lock_flags); | 5897 | spin_unlock_irqrestore(ioa_cfg->host->host_lock, host_lock_flags); |
5893 | wait_event(ioa_cfg->reset_wait_q, !ioa_cfg->in_reset_reload); | 5898 | wait_event(ioa_cfg->reset_wait_q, !ioa_cfg->in_reset_reload); |
@@ -6264,6 +6269,7 @@ static int __devinit ipr_probe_ioa(struct pci_dev *pdev, | |||
6264 | unsigned long ipr_regs_pci; | 6269 | unsigned long ipr_regs_pci; |
6265 | void __iomem *ipr_regs; | 6270 | void __iomem *ipr_regs; |
6266 | u32 rc = PCIBIOS_SUCCESSFUL; | 6271 | u32 rc = PCIBIOS_SUCCESSFUL; |
6272 | volatile u32 mask, uproc; | ||
6267 | 6273 | ||
6268 | ENTER; | 6274 | ENTER; |
6269 | 6275 | ||
@@ -6356,6 +6362,15 @@ static int __devinit ipr_probe_ioa(struct pci_dev *pdev, | |||
6356 | goto cleanup_nomem; | 6362 | goto cleanup_nomem; |
6357 | } | 6363 | } |
6358 | 6364 | ||
6365 | /* | ||
6366 | * If HRRQ updated interrupt is not masked, or reset alert is set, | ||
6367 | * the card is in an unknown state and needs a hard reset | ||
6368 | */ | ||
6369 | mask = readl(ioa_cfg->regs.sense_interrupt_mask_reg); | ||
6370 | uproc = readl(ioa_cfg->regs.sense_uproc_interrupt_reg); | ||
6371 | if ((mask & IPR_PCII_HRRQ_UPDATED) == 0 || (uproc & IPR_UPROCI_RESET_ALERT)) | ||
6372 | ioa_cfg->needs_hard_reset = 1; | ||
6373 | |||
6359 | ipr_mask_and_clear_interrupts(ioa_cfg, ~IPR_PCII_IOA_TRANS_TO_OPER); | 6374 | ipr_mask_and_clear_interrupts(ioa_cfg, ~IPR_PCII_IOA_TRANS_TO_OPER); |
6360 | rc = request_irq(pdev->irq, ipr_isr, SA_SHIRQ, IPR_NAME, ioa_cfg); | 6375 | rc = request_irq(pdev->irq, ipr_isr, SA_SHIRQ, IPR_NAME, ioa_cfg); |
6361 | 6376 | ||
diff --git a/drivers/scsi/ipr.h b/drivers/scsi/ipr.h index 6bec673c925c..b639332131f1 100644 --- a/drivers/scsi/ipr.h +++ b/drivers/scsi/ipr.h | |||
@@ -36,8 +36,8 @@ | |||
36 | /* | 36 | /* |
37 | * Literals | 37 | * Literals |
38 | */ | 38 | */ |
39 | #define IPR_DRIVER_VERSION "2.1.0" | 39 | #define IPR_DRIVER_VERSION "2.1.1" |
40 | #define IPR_DRIVER_DATE "(October 31, 2005)" | 40 | #define IPR_DRIVER_DATE "(November 15, 2005)" |
41 | 41 | ||
42 | /* | 42 | /* |
43 | * IPR_MAX_CMD_PER_LUN: This defines the maximum number of outstanding | 43 | * IPR_MAX_CMD_PER_LUN: This defines the maximum number of outstanding |
@@ -922,6 +922,7 @@ struct ipr_ioa_cfg { | |||
922 | u8 dump_taken:1; | 922 | u8 dump_taken:1; |
923 | u8 allow_cmds:1; | 923 | u8 allow_cmds:1; |
924 | u8 allow_ml_add_del:1; | 924 | u8 allow_ml_add_del:1; |
925 | u8 needs_hard_reset:1; | ||
925 | 926 | ||
926 | enum ipr_cache_state cache_state; | 927 | enum ipr_cache_state cache_state; |
927 | u16 type; /* CCIN of the card */ | 928 | u16 type; /* CCIN of the card */ |