aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAnton Blanchard <anton@samba.org>2011-08-01 05:43:45 -0400
committerJames Bottomley <JBottomley@Parallels.com>2011-08-27 10:35:55 -0400
commit5d7c20b7fa5c6ca19e871b4050e321c99d32bd43 (patch)
tree75f267ab9a8bbb6549d75d00a01bfce1975906f6
parent98cbe371fd373f13806595835b79da07f3a2f934 (diff)
[SCSI] ipr: Always initiate hard reset in kdump kernel
During kdump testing I noticed timeouts when initialising each IPR adapter. While the driver has logic to detect an adapter in an indeterminate state, it wasn't triggering and each adapter went through a 5 minute timeout before finally going operational. Some analysis showed the needs_hard_reset flag wasn't getting set. We can check the reset_devices kernel parameter which is set by kdump and force a full reset. This fixes the problem. Signed-off-by: Anton Blanchard <anton@samba.org> Cc: <stable@kernel.org> Acked-by: Brian King <brking@linux.vnet.ibm.com> Signed-off-by: James Bottomley <JBottomley@Parallels.com>
-rw-r--r--drivers/scsi/ipr.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/ipr.c b/drivers/scsi/ipr.c
index 8d636301e32c..acbb9241262d 100644
--- a/drivers/scsi/ipr.c
+++ b/drivers/scsi/ipr.c
@@ -8812,7 +8812,7 @@ static int __devinit ipr_probe_ioa(struct pci_dev *pdev,
8812 uproc = readl(ioa_cfg->regs.sense_uproc_interrupt_reg32); 8812 uproc = readl(ioa_cfg->regs.sense_uproc_interrupt_reg32);
8813 if ((mask & IPR_PCII_HRRQ_UPDATED) == 0 || (uproc & IPR_UPROCI_RESET_ALERT)) 8813 if ((mask & IPR_PCII_HRRQ_UPDATED) == 0 || (uproc & IPR_UPROCI_RESET_ALERT))
8814 ioa_cfg->needs_hard_reset = 1; 8814 ioa_cfg->needs_hard_reset = 1;
8815 if (interrupts & IPR_PCII_ERROR_INTERRUPTS) 8815 if ((interrupts & IPR_PCII_ERROR_INTERRUPTS) || reset_devices)
8816 ioa_cfg->needs_hard_reset = 1; 8816 ioa_cfg->needs_hard_reset = 1;
8817 if (interrupts & IPR_PCII_IOA_UNIT_CHECKED) 8817 if (interrupts & IPR_PCII_IOA_UNIT_CHECKED)
8818 ioa_cfg->ioa_unit_checked = 1; 8818 ioa_cfg->ioa_unit_checked = 1;