diff options
author | Jens Axboe <jens.axboe@oracle.com> | 2009-02-23 08:53:35 +0100 |
---|---|---|
committer | Jens Axboe <jens.axboe@oracle.com> | 2009-02-26 10:45:48 +0100 |
commit | 5e4c91c84b194b26cf592779e451f4b5be777cba (patch) | |
tree | 334b9e2725eba9aebc18fefe7c52eb1523569245 | |
parent | 9e8c0bccdc944bd09361672d47660810c027bcaa (diff) | |
download | op-kernel-dev-5e4c91c84b194b26cf592779e451f4b5be777cba.zip op-kernel-dev-5e4c91c84b194b26cf592779e451f4b5be777cba.tar.gz |
cciss: shorten 30s timeout on controller reset
If reset_devices is set for kexec, then cciss will delay 30 seconds
since the old 5i controller _may_ need that long to recover. Replace
the long sleep with incremental sleep and tests to reduce the 30 seconds
to worst case for 5i, so that other controllers will proceed quickly.
Reviewed-by: Mike Miller <mike.miller@hp.com>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
-rw-r--r-- | drivers/block/cciss.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/drivers/block/cciss.c b/drivers/block/cciss.c index d2cb67b..b5a0611 100644 --- a/drivers/block/cciss.c +++ b/drivers/block/cciss.c @@ -3611,11 +3611,15 @@ static int __devinit cciss_init_one(struct pci_dev *pdev, schedule_timeout_uninterruptible(30*HZ); /* Now try to get the controller to respond to a no-op */ - for (i=0; i<12; i++) { + for (i=0; i<30; i++) { if (cciss_noop(pdev) == 0) break; - else - printk("cciss: no-op failed%s\n", (i < 11 ? "; re-trying" : "")); + + schedule_timeout_uninterruptible(HZ); + } + if (i == 30) { + printk(KERN_ERR "cciss: controller seems dead\n"); + return -EBUSY; } } |