lkml.org 
[lkml]   [2005]   [Sep]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRE: [PATCH 2.6.13] scsi: sd fails to copy cmd_len on SG_IO
Date
From


> -----Original Message-----
> From: Mike Christie [mailto:michaelc@cs.wisc.edu]
> Sent: Tuesday, September 13, 2005 10:49 AM
> To: Timothy Thelin
> Cc: James Bottomley; SCSI Mailing List; Linux Kernel; Andrew Morton
> Subject: Re: [PATCH 2.6.13] scsi: sd fails to copy cmd_len on SG_IO
>
>
> Timothy Thelin wrote:
> > This fixes an issue when doing SG_IO on an sd device: the
> > sd driver fails to copy the request's cmd_len to the scsi
> > command's cmd_len when initializing the command.
> >
>
> Do you need the same fix to st, sr, and scsi_lib (in the
> scsi_generic_done path)?
>

I just looked, and st and sr look like they need the same fix, but i'm
unaware of where scsi_lib might need it (I'm new to the Linux scsi stack).
Mind elaborating on your thoughts of the scsi_generic_done path? (I cant
find
the symbol in drivers/scsi/*)

> > Signed-off-by: Timothy Thelin <timothy.thelin@wdc.com>
> >
> > --- linux-2.6.13.orig/drivers/scsi/sd.c 2005-08-28
> 16:41:01.000000000 -0700
> > +++ linux-2.6.13/drivers/scsi/sd.c 2005-09-13
> 09:39:06.000000000 -0700
> > @@ -236,6 +236,7 @@ static int sd_init_command(struct scsi_c
> > return 0;
> >
> > memcpy(SCpnt->cmnd, rq->cmd, sizeof(SCpnt->cmnd));
> > + SCpnt->cmd_len = rq->cmd_len;
> > if (rq_data_dir(rq) == WRITE)
> > SCpnt->sc_data_direction = DMA_TO_DEVICE;
> > else if (rq->data_len)
> > -
> > To unsubscribe from this list: send the line "unsubscribe
> linux-scsi" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-09-13 20:28    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site