Re: [PATCH 14/17] scsi: replace custom rq mapping withblk_rq_map_kern_sgl()

From: James Bottomley
Date: Wed Apr 01 2009 - 13:05:54 EST


On Wed, 2009-04-01 at 20:00 +0300, Boaz Harrosh wrote:
> On 04/01/2009 04:44 PM, Tejun Heo wrote:
> > Impact: hack removal
> >
> > SCSI needs to map sgl into rq for kernel PC requests; however, block
> > API didn't have such feature so it used its own rq mapping function
> > which hooked into block/bio internals and is generally considered an
> > ugly hack. The private function may also produce requests which are
> > bigger than queue per-rq limits.
> >
> > Block blk_rq_map_kern_sgl(). Kill the private implementation and use
> > it.
> >
> > Signed-off-by: Tejun Heo <tj@xxxxxxxxxx>
>
> James, TOMO
>
> what happened to Tomo's patches that removes all this after fixing up
> all users (sg.c)?
>
> I thought that was agreed and done? What is left to do for that to go
> in.

They couldn't go in because they would break libosd. You were going to
send patches to fix libosd so it no longer relied on the exported
function ... did that happen and I missed it?

James


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/