linux/drivers/scsi/aacraid
Salyzyn, Mark 8e31e607ea [SCSI] aacraid: do not set valid bit in sense information
Luben Tuikov [mailto:ltuikov@yahoo.com] sez:
> Just as in your case and Tony's case, which I presume
> uses the same RAID firmware vendor, it would've
> probably been better if the RAID firmware vendor
> fixed the firmware to not set the VALID bit if the
> INFORMATION field is not valid.

Point taken regarding the aacraid driver. Dropped the VALID bit, and
then did some cleanup/simplification of the set_sense procedure and
the associated parameters. Mike did some preliminary tests when the
VALID bit was dropped before the 'Re: [PATCH] [SCSI] sd: make error
handling more robust' patches came on the scene. The change in the
SCSI subsystem does make this enclosed aacraid patch unnecessary, so
this aacraid patch is merely post battle ground cleanup. If the
simplification is an issue, repugnant, too much for a back-port to the
stable trees or clouds the point, this patch could be happily
distilled down to:

diff -ru a/drivers/scsi/aacraid/aachba.c b/drivers/scsi/aacraid/aachba.c
--- a/drivers/scsi/aacraid/aachba.c     2008-02-06 16:26:45.834938955 -0500
+++ b/drivers/scsi/aacraid/aachba.c     2008-02-06 16:32:01.109035329 -0500
@@ -865,7 +865,7 @@
                         u32 residue)
 {
-        sense_buf[0] = 0xF0;    /* Sense data valid, err code 70h (current error) */
+        sense_buf[0] = 0x70;    /* Sense data invalid, err code 70h (current error) */
         sense_buf[1] = 0;       /* Segment number, always zero */

         if (incorrect_length) {

Signed-off-by: Mark Salyzyn <aacraid@adaptec.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
2008-02-07 18:31:51 -06:00
..
aachba.c [SCSI] aacraid: do not set valid bit in sense information 2008-02-07 18:31:51 -06:00
aacraid.h [SCSI] aacraid: add Voodoo Lite class of cards. 2008-01-23 11:29:34 -06:00
commctrl.c [SCSI] aacraid: fib context lock for management ioctls (take 2) 2008-02-07 18:02:33 -06:00
comminit.c [SCSI] aacraid: fix big endian issues 2008-01-23 11:29:19 -06:00
commsup.c [SCSI] aacraid: add Voodoo Lite class of cards. 2008-01-23 11:29:34 -06:00
dpcsup.c [SCSI] aacraid: OS panic after Adapter panic (hardening). 2008-01-23 11:29:21 -06:00
linit.c [SCSI] aacraid: pci_set_dma_max_seg_size opened up for late model controllers 2008-02-07 18:02:44 -06:00
Makefile [SCSI] aacraid: Begin adding support for new adapter type 2007-01-27 09:27:31 -06:00
nark.c [SCSI] aacraid: draw line in sand, sundry cleanup and version update 2007-07-27 09:13:28 -04:00
rkt.c [SCSI] aacraid: draw line in sand, sundry cleanup and version update 2007-07-27 09:13:28 -04:00
rx.c [SCSI] aacraid: fix big endian issues 2008-01-23 11:29:19 -06:00
sa.c [SCSI] aacraid: Correct sa platform support. (Was: [Bug 8469] Bad EIP value on pentium3 SMP kernel-2.6.21.1) 2007-05-16 13:06:26 -04:00
TODO [SCSI] aacraid: Newer adapter communication iterface support 2005-10-28 11:41:53 -05:00