Date: Sat, 20 Sep 2003 18:16:56 -0600 From: Scott Long <scottl@freebsd.org> To: Thomas Quinot <thomas@FreeBSD.org> Cc: ken@FreeBSD.org Subject: Re: Missing length specification in scsi_request_sense Message-ID: <3F6CEDF8.5070803@freebsd.org> In-Reply-To: <20030920110131.GA41062@melusine.cuivre.fr.eu.org> References: <20030920110131.GA41062@melusine.cuivre.fr.eu.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Thomas Quinot wrote: > The CDB built by scsi_request_sense always has a zero length indication, > preventing correct transfer of sense data during recovery action by > cam_periph. The following change resolves the problem, please review. > > Thomas. > > Index: scsi/scsi_all.c > =================================================================== > RCS file: /home/ncvs/src/sys/cam/scsi/scsi_all.c,v > retrieving revision 1.42 > diff -u -r1.42 scsi_all.c > --- scsi/scsi_all.c 10 Jun 2003 18:14:04 -0000 1.42 > +++ scsi/scsi_all.c 20 Sep 2003 10:53:38 -0000 > @@ -2408,6 +2408,7 @@ > scsi_cmd = (struct scsi_request_sense *)&csio->cdb_io.cdb_bytes; > bzero(scsi_cmd, sizeof(*scsi_cmd)); > scsi_cmd->opcode = REQUEST_SENSE; > + scsi_cmd->length = dxfer_len; > } > > void > Since just about every driver in existance does auto-sense, it doesn't surprise me that you're finding these bugs. This patch looks reasonable to me. Scott
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3F6CEDF8.5070803>