On Mon, Jan 07 2008, Rusty Russell wrote: > I realize that sg chaining is a ploy to make the rest of the kernel > devs feel the pain of the SCSI subsystem. But this was a little > unsubtle. > > Signed-off-by: Rusty Russell <rusty@xxxxxxxxxxxxxxx> > > diff -r b3aec596b841 include/linux/scatterlist.h > --- a/include/linux/scatterlist.h Mon Jan 07 12:43:56 2008 +1100 > +++ b/include/linux/scatterlist.h Mon Jan 07 15:01:51 2008 +1100 > @@ -188,8 +188,8 @@ static inline void sg_chain(struct scatt > /* > * offset and length are unused for chain entry. Clear them. > */ > - prv->offset = 0; > - prv->length = 0; > + prv[prv_nents - 1].offset = 0; > + prv[prv_nents - 1].length = 0; > > /* > * Set lowest bit to indicate a link pointer, and make sure to clear Dang, I should have caught that. Thanks Rusty... -- Jens Axboe - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
