From mboxrd@z Thu Jan 1 00:00:00 1970 From: Brian King Subject: Re: ipr SATA problems in 2.6.20 Date: Tue, 16 Jan 2007 17:17:59 -0600 Message-ID: <45AD5D27.8020004@linux.vnet.ibm.com> References: <45AD50AD.2040700@linux.vnet.ibm.com> <20070116224115.50d29cf7@localhost.localdomain> <45AD55A4.50208@pobox.com> <1168989094.2789.20.camel@mulgrave.il.steeleye.com> Reply-To: brking@linux.vnet.ibm.com Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1168989094.2789.20.camel@mulgrave.il.steeleye.com> Sender: linux-ide-owner@vger.kernel.org To: James Bottomley Cc: Jeff Garzik , Alan , linux-scsi@vger.kernel.org, "linux-ide@vger.kernel.org" , Tejun Heo List-Id: linux-scsi@vger.kernel.org James Bottomley wrote: > On Tue, 2007-01-16 at 17:45 -0500, Jeff Garzik wrote: >> Tejun recently updated the CDB length areas of the code. I bet it's >> either a bug somewhere in there, or the SCSI layer isn't passing us >> proper command lengths in a case or two. >> >> Additional traces (starting with SCSI command, before it hits libata) >> would be helpful. > > Actually, this looks like a potential bug in atapi_xlat(): it doesn't > set qc->dma_dir. Shouldn't it be setting it from > qc->scsicmd->sc_data_direction like ata_scsi_translate() does? I think we are OK here since atapi_xlate is only ever called by ata_scsi_translate. Brian -- Brian King eServer Storage I/O IBM Linux Technology Center