Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

sym53c8xx_2: Set DID_REQUEUE return code when aborting squeue

This patch fixes I/O errors with the sym53c8xx_2 driver when the disk
returns QUEUE FULL status.

When the controller encounters an error (including QUEUE FULL or BUSY
status), it aborts all not yet submitted requests in the function
sym_dequeue_from_squeue.

This function aborts them with DID_SOFT_ERROR.

If the disk has full tag queue, the request that caused the overflow is
aborted with QUEUE FULL status (and the scsi midlayer properly retries
it until it is accepted by the disk), but the sym53c8xx_2 driver aborts
the following requests with DID_SOFT_ERROR --- for them, the midlayer
does just a few retries and then signals the error up to sd.

The result is that disk returning QUEUE FULL causes request failures.

The error was reproduced on 53c895 with COMPAQ BD03685A24 disk
(rebranded ST336607LC) with command queue 48 or 64 tags. The disk has
64 tags, but under some access patterns it return QUEUE FULL when there
are less than 64 pending tags. The SCSI specification allows returning
QUEUE FULL anytime and it is up to the host to retry.

Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Cc: Matthew Wilcox <matthew@wil.cx>
Cc: James Bottomley <JBottomley@Parallels.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

authored by

Mikulas Patocka and committed by
Linus Torvalds
fd1232b2 18aa0da3

+4
+4
drivers/scsi/sym53c8xx_2/sym_hipd.c
··· 3000 3000 if ((target == -1 || cp->target == target) && 3001 3001 (lun == -1 || cp->lun == lun) && 3002 3002 (task == -1 || cp->tag == task)) { 3003 + #ifdef SYM_OPT_HANDLE_DEVICE_QUEUEING 3003 3004 sym_set_cam_status(cp->cmd, DID_SOFT_ERROR); 3005 + #else 3006 + sym_set_cam_status(cp->cmd, DID_REQUEUE); 3007 + #endif 3004 3008 sym_remque(&cp->link_ccbq); 3005 3009 sym_insque_tail(&cp->link_ccbq, &np->comp_ccbq); 3006 3010 }