diff options
| author | Cornelia Huck <cornelia.huck@de.ibm.com> | 2006-06-29 09:03:35 -0400 |
|---|---|---|
| committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2006-06-29 09:03:35 -0400 |
| commit | b4f7b1ee577857aca467e6825122a3b29d6c3c0e (patch) | |
| tree | bb17f4d6bea22b1d5a4c0a927c41f1ba782ef2ae | |
| parent | 3ee526841ba409f7b23f825f120772a79e7d7bd2 (diff) | |
[S390] cio chpid offline.
After setting a path to a dasd offline at the SE, I/O hangs on that
dasd for 5 minutes, then continues.
I/O for which an interrupt will not be reported after the channel
path has been disabled was not terminated by the common I/O layer,
causing the dasd MIH to hit after 5 minutes.
Be more aggressive in terminating I/O after setting a channel path
offline. Also make sure to generate a fake irb if the device
driver issues an I/O request after being notified of the killed
I/O and clear residual information from the irb before trying to
start the delayed verification.
Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
| -rw-r--r-- | drivers/s390/cio/chsc.c | 3 | ||||
| -rw-r--r-- | drivers/s390/cio/device_fsm.c | 2 | ||||
| -rw-r--r-- | drivers/s390/cio/device_ops.c | 3 |
3 files changed, 5 insertions, 3 deletions
diff --git a/drivers/s390/cio/chsc.c b/drivers/s390/cio/chsc.c index 4a9da5bdd535..b00f3ed051a0 100644 --- a/drivers/s390/cio/chsc.c +++ b/drivers/s390/cio/chsc.c | |||
| @@ -244,8 +244,7 @@ s390_subchannel_remove_chpid(struct device *dev, void *data) | |||
| 244 | 244 | ||
| 245 | if ((sch->schib.scsw.actl & SCSW_ACTL_DEVACT) && | 245 | if ((sch->schib.scsw.actl & SCSW_ACTL_DEVACT) && |
| 246 | (sch->schib.scsw.actl & SCSW_ACTL_SCHACT) && | 246 | (sch->schib.scsw.actl & SCSW_ACTL_SCHACT) && |
| 247 | (sch->schib.pmcw.lpum == mask) && | 247 | (sch->schib.pmcw.lpum == mask)) { |
| 248 | (sch->vpm == 0)) { | ||
| 249 | int cc; | 248 | int cc; |
| 250 | 249 | ||
| 251 | cc = cio_clear(sch); | 250 | cc = cio_clear(sch); |
diff --git a/drivers/s390/cio/device_fsm.c b/drivers/s390/cio/device_fsm.c index 49ec562d7f60..5ec8ef1df9fa 100644 --- a/drivers/s390/cio/device_fsm.c +++ b/drivers/s390/cio/device_fsm.c | |||
| @@ -861,6 +861,8 @@ ccw_device_clear_verify(struct ccw_device *cdev, enum dev_event dev_event) | |||
| 861 | irb = (struct irb *) __LC_IRB; | 861 | irb = (struct irb *) __LC_IRB; |
| 862 | /* Accumulate status. We don't do basic sense. */ | 862 | /* Accumulate status. We don't do basic sense. */ |
| 863 | ccw_device_accumulate_irb(cdev, irb); | 863 | ccw_device_accumulate_irb(cdev, irb); |
| 864 | /* Remember to clear irb to avoid residuals. */ | ||
| 865 | memset(&cdev->private->irb, 0, sizeof(struct irb)); | ||
| 864 | /* Try to start delayed device verification. */ | 866 | /* Try to start delayed device verification. */ |
| 865 | ccw_device_online_verify(cdev, 0); | 867 | ccw_device_online_verify(cdev, 0); |
| 866 | /* Note: Don't call handler for cio initiated clear! */ | 868 | /* Note: Don't call handler for cio initiated clear! */ |
diff --git a/drivers/s390/cio/device_ops.c b/drivers/s390/cio/device_ops.c index 5c9904b418e1..b266ad8e14ff 100644 --- a/drivers/s390/cio/device_ops.c +++ b/drivers/s390/cio/device_ops.c | |||
| @@ -78,7 +78,8 @@ ccw_device_start_key(struct ccw_device *cdev, struct ccw1 *cpa, | |||
| 78 | return -ENODEV; | 78 | return -ENODEV; |
| 79 | if (cdev->private->state == DEV_STATE_NOT_OPER) | 79 | if (cdev->private->state == DEV_STATE_NOT_OPER) |
| 80 | return -ENODEV; | 80 | return -ENODEV; |
| 81 | if (cdev->private->state == DEV_STATE_VERIFY) { | 81 | if (cdev->private->state == DEV_STATE_VERIFY || |
| 82 | cdev->private->state == DEV_STATE_CLEAR_VERIFY) { | ||
| 82 | /* Remember to fake irb when finished. */ | 83 | /* Remember to fake irb when finished. */ |
| 83 | if (!cdev->private->flags.fake_irb) { | 84 | if (!cdev->private->flags.fake_irb) { |
| 84 | cdev->private->flags.fake_irb = 1; | 85 | cdev->private->flags.fake_irb = 1; |
