diff options
| author | Swen Schillig | 2008-05-19 12:17:37 +0200 |
|---|---|---|
| committer | James Bottomley | 2008-06-05 16:27:13 +0200 |
| commit | d26ab06ede83287f99067fee3034c5455a75faf9 (patch) | |
| tree | 6070ae50cded8086ceab61092348790b39876146 /drivers/s390/scsi/zfcp_dbf.c | |
| parent | [SCSI] mpt fusion: make struct mpt_proc_root_dir static (diff) | |
| download | kernel-qcow2-linux-d26ab06ede83287f99067fee3034c5455a75faf9.tar.gz kernel-qcow2-linux-d26ab06ede83287f99067fee3034c5455a75faf9.tar.xz kernel-qcow2-linux-d26ab06ede83287f99067fee3034c5455a75faf9.zip | |
[SCSI] zfcp: receiving an unsolicted status can lead to I/O stall
Processing of an unsolicted status request can lead to a locking race
of the request_queue's queue_lock during the recreation of the
used up status read request while still in interrupt context
of the response handler.
Detaching the 'refill' of the long running status read requests from
the handler to a scheduled work is solving this issue.
In addition, each refill-run is trying to re-establish the full amount
of status read requests, which might have failed in earlier runs.
Signed-off-by: Swen Schillig <swen@vnet.ibm.com>
Signed-off-by: Christof Schmitt <christof.schmitt@de.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers/s390/scsi/zfcp_dbf.c')
| -rw-r--r-- | drivers/s390/scsi/zfcp_dbf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/s390/scsi/zfcp_dbf.c b/drivers/s390/scsi/zfcp_dbf.c index c8bad675dbd1..1710c12a32c4 100644 --- a/drivers/s390/scsi/zfcp_dbf.c +++ b/drivers/s390/scsi/zfcp_dbf.c @@ -268,7 +268,7 @@ void zfcp_hba_dbf_event_fsf_unsol(const char *tag, struct zfcp_adapter *adapter, strncpy(rec->tag, "stat", ZFCP_DBF_TAG_SIZE); strncpy(rec->tag2, tag, ZFCP_DBF_TAG_SIZE); - rec->u.status.failed = adapter->status_read_failed; + rec->u.status.failed = atomic_read(&adapter->stat_miss); if (status_buffer != NULL) { rec->u.status.status_type = status_buffer->status_type; rec->u.status.status_subtype = status_buffer->status_subtype; |