Re: LLDD: scatterlists only?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Douglas Gilbert wrote:
James Bottomley wrote:

On Sun, 2005-08-14 at 16:24 -0700, Luben Tuikov wrote:


Did someone have a patch or was there a talk
that SCSI Core is moving towards sending _only_ scatterlists
down to LLDDs?  (effectively BUG_ON(!cmd->use_sg))


Yes, you can already see the beginnings in the -mm tree.

Jens is maintaining a block tree with this in here:

http://www.kernel.org/git/?p=linux/kernel/git/axboe/linux-2.6-block.git;a=shortlog;h=rq-map

And I have a scsi-block-2.6 tree based off this here:

http://www.kernel.org/git/?p=linux/kernel/git/jejb/scsi-block-2.6.git;a=summary

The missing (and frankly really nasty) pieces are pulling all the sg
handling out of sg and st, but fortunately Mike Christie is working on
that.


James,

I haven't looked at what would be involved with the sg
driver but it is good to hear Mike C. is working on it.


I sent a patch to begin this the other day
http://marc.theaimsgroup.com/?l=linux-scsi&m=112356952007369&w=2

The patch is not complete. I have to add a bunch of stuff I ripped out like the security checks and mmap code. I also have to move some code to a scsi command like James did for the sync code. I am looking over st to try and figure out what to share right now.

One question though. For SG_DXFER_TO_FROM_DEV it looks like for the block layer SG_IO code only the write part of the command could get run. This is becuase in sg_io() the reading variable is never used so a copy_to_user (that is if we end up not being able to do direct IO) may never happen. What commands use SG_DXFER_TO_FROM_DEV, and do we need the read part (or did I misread the block layer code or have we just been getting lucky and been doing zero copy).

Also for the new interface the write and read will be the same size becuase we use dxfer_len, but for the old interface is the same true? From looking at the sg driver it looks like there could be different values for the read and write part, but is that ever used or is it just extra tests. And again what commands do this, so I can test them?

Thanks

Mike
-
: 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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux