Re: [PATCH 0/3] update bidirectional series to sit on top of sg_table
On Tue, Jan 22 2008 at 18:31 +0200, FUJITA Tomonori <[EMAIL PROTECTED]> wrote: > From: James Bottomley <[EMAIL PROTECTED]> > Subject: [PATCH 0/3] update bidirectional series to sit on top of sg_table > Date: Fri, 11 Jan 2008 21:09:00 -0600 > >> OK, I suppose in the scheme of things, it's my turn to bear some of the >> pain. the SCSI bidirectional series rejects pretty badly with sg_table, >> and since sg_table has to go in, I rebased the series on top of it. >> >> Additionally, I tidied up the patches to take advantages of some of the >> features of sg_table. I killed both use_sg and sg_count in favour of >> using sg_table.nseg for the count. >> >> Just so you can test all of this to make sure I got it right, you can >> pull the patch series from >> >> master.kernel.org:/pub/scm/linux/kernel/git/jejb/scsi-bidi-2.6.git > > I've just started to look at the bidi tree. > > I suppose that only panasas guys have tested the bidi tree with their > OSD target devices. To test the bidi tree, OSD Yes, but not only Panasas guys. Pete Wyckoff is doing bidi threw bsg to an OSD scsi_tgt target. And Seagate guys have an OSD device they use also threw bsg. In Panasas we used IBM's OSD-Initiator and now our own osd_lib, an in kernel only library that will be part of the Objects-layout drivers for pNFS. > I added XDWRITEREAD_10 > support to scsi_debug and sgv4_xdwriteread tool to my makeshift bsg > tool collections: > > git://git.kernel.org/pub/scm/linux/kernel/git/tomo/sgv4-tools.git > > It just sends XDWRITEREAD_10 commands like this: > > tulip:~# sgv4_xdwriteread --length 16384 /sys/class/bsg/1:0:0:0 > driver:0, transport:0, device:0, din_resid: 0, dout_resid: 0 > > No errors. > > I'll send the patchset (over the bidi tree) to add XDWRITEREAD_10 > support to scsi_debug though I'm not sure whether it's worth adding it > to mainline. > - Thanks TOMO, I think this is most valuable contribution, because now programmers have a most simple setup they can put up, to make sure they did not break, also the bidi path. And not need to setup the complete OSD stack for that. Also it is a very good reference implementation for drivers that want to support XDWRITEREAD_10. I bet the raid guys will love it. So Thanks Boaz - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 0/3] update bidirectional series to sit on top of sg_table
From: James Bottomley <[EMAIL PROTECTED]> Subject: [PATCH 0/3] update bidirectional series to sit on top of sg_table Date: Fri, 11 Jan 2008 21:09:00 -0600 > OK, I suppose in the scheme of things, it's my turn to bear some of the > pain. the SCSI bidirectional series rejects pretty badly with sg_table, > and since sg_table has to go in, I rebased the series on top of it. > > Additionally, I tidied up the patches to take advantages of some of the > features of sg_table. I killed both use_sg and sg_count in favour of > using sg_table.nseg for the count. > > Just so you can test all of this to make sure I got it right, you can > pull the patch series from > > master.kernel.org:/pub/scm/linux/kernel/git/jejb/scsi-bidi-2.6.git I've just started to look at the bidi tree. I suppose that only panasas guys have tested the bidi tree with their OSD target devices. To test the bidi tree, I added XDWRITEREAD_10 support to scsi_debug and sgv4_xdwriteread tool to my makeshift bsg tool collections: git://git.kernel.org/pub/scm/linux/kernel/git/tomo/sgv4-tools.git It just sends XDWRITEREAD_10 commands like this: tulip:~# sgv4_xdwriteread --length 16384 /sys/class/bsg/1:0:0:0 driver:0, transport:0, device:0, din_resid: 0, dout_resid: 0 No errors. I'll send the patchset (over the bidi tree) to add XDWRITEREAD_10 support to scsi_debug though I'm not sure whether it's worth adding it to mainline. - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 0/3] update bidirectional series to sit on top of sg_table
OK, I suppose in the scheme of things, it's my turn to bear some of the pain. the SCSI bidirectional series rejects pretty badly with sg_table, and since sg_table has to go in, I rebased the series on top of it. Additionally, I tidied up the patches to take advantages of some of the features of sg_table. I killed both use_sg and sg_count in favour of using sg_table.nseg for the count. Just so you can test all of this to make sure I got it right, you can pull the patch series from master.kernel.org:/pub/scm/linux/kernel/git/jejb/scsi-bidi-2.6.git James - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
