diff options
| author | Andrew Vasquez <andrew.vasquez@qlogic.com> | 2005-12-06 13:58:06 -0500 |
|---|---|---|
| committer | James Bottomley <jejb@mulgrave.(none)> | 2005-12-07 09:33:17 -0500 |
| commit | 0da69df1e54146eece38e0a144051f6dd3526821 (patch) | |
| tree | 80efaa4061980dea9638719923cd98ee7ab750b7 | |
| parent | 23443b1d6130eff8e1335e4f84eaf0577a331dcf (diff) | |
[SCSI] qla2xxx: Correct short-WRITE status handling.
Properly check FC_RESID for any non-transfered bytes
regardless of firmware completion status.
Signed-off-by: Andrew Vasquez <andrew.vasquez@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
| -rw-r--r-- | drivers/scsi/qla2xxx/qla_isr.c | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/drivers/scsi/qla2xxx/qla_isr.c b/drivers/scsi/qla2xxx/qla_isr.c index 09afc0f06bd4..5181d966fecb 100644 --- a/drivers/scsi/qla2xxx/qla_isr.c +++ b/drivers/scsi/qla2xxx/qla_isr.c | |||
| @@ -909,6 +909,21 @@ qla2x00_status_entry(scsi_qla_host_t *ha, void *pkt) | |||
| 909 | resid = resid_len; | 909 | resid = resid_len; |
| 910 | cp->resid = resid; | 910 | cp->resid = resid; |
| 911 | CMD_RESID_LEN(cp) = resid; | 911 | CMD_RESID_LEN(cp) = resid; |
| 912 | |||
| 913 | if (!lscsi_status && | ||
| 914 | ((unsigned)(cp->request_bufflen - resid) < | ||
| 915 | cp->underflow)) { | ||
| 916 | qla_printk(KERN_INFO, ha, | ||
| 917 | "scsi(%ld:%d:%d:%d): Mid-layer underflow " | ||
| 918 | "detected (%x of %x bytes)...returning " | ||
| 919 | "error status.\n", ha->host_no, | ||
| 920 | cp->device->channel, cp->device->id, | ||
| 921 | cp->device->lun, resid, | ||
| 922 | cp->request_bufflen); | ||
| 923 | |||
| 924 | cp->result = DID_ERROR << 16; | ||
| 925 | break; | ||
| 926 | } | ||
| 912 | } | 927 | } |
| 913 | cp->result = DID_OK << 16 | lscsi_status; | 928 | cp->result = DID_OK << 16 | lscsi_status; |
| 914 | 929 | ||
