On Sun, Mar 09 2008 at 6:44 +0200, FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx> wrote: > This patchset adds new two helper functions to copy data between an SG > table and liner buffer, and converts severl LLDs to use them. > > The new APIs are used mainly in the code to spoof SCSI commands > (INQUIRY, READ CAPACITY, MODE SENSE etc), that is, LLDs build a fake > reposense and copy it to the sg list in scsi_cmnd struct. Several LLDs > have similar functions for such code. This patchset removes such > duplication. > > Another reason to do this work is that because we relaxed the default > alignment requirements (from 511 to 3) post 2.6.24, the above commands > might come with multiple scatter gather entries but several LLDs make > the assumption that such commands come with only one sg entries and > can't handle multiple sg entries. The new APIs can handle multiple sg > entries so LLDs don't need to care about anything. > > The first patch adds the new APIs to lib/scatterlist.c and the rest > are for SCSI. I like to push the whole patchset via scsi-misc (since > it's easier). > > This is against scsi-misc (a6680f71ca27ea78c4c4e577076aecb9ace476f1). > > arch/ia64/hp/sim/simscsi.c | 23 ++-------- > drivers/scsi/3w-9xxx.c | 21 ++++----- > drivers/scsi/3w-xxxx.c | 12 +---- > drivers/scsi/aacraid/aachba.c | 49 ++++++++-------------- > drivers/scsi/ips.c | 50 ++++------------------- > drivers/scsi/ps3rom.c | 92 ++++------------------------------------ > drivers/scsi/scsi_debug.c | 79 ++++++----------------------------- > drivers/scsi/stex.c | 66 ++++------------------------- > include/linux/scatterlist.h | 5 ++ > include/scsi/scsi_cmnd.h | 14 ++++++ > lib/scatterlist.c | 90 ++++++++++++++++++++++++++++++++++++++++ > 11 files changed, 182 insertions(+), 319 deletions(-) > > Hi Tomo. Nice cleanup, Cheers. Have you had a look at drivers/usb/storage/protocol.c usb_stor_access_xfer_buf() ? It looks like it could also use these, but there they have a twist where they want to do it in parts. Do you think that the code there could also use the helpers presented here somehow? (I know that one of the USB guys was asking about it) Boaz -- 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
