From: Jens Axboe <jens.axboe@xxxxxxxxxx> Subject: Re: [PATCH 0/3] SG v4 support Date: Fri, 15 Dec 2006 19:57:05 +0100 > On Sat, Dec 16 2006, FUJITA Tomonori wrote: > > The following patches add SG v4 support to bsg. They are over Jens' > > bsg branch. > > > > I just replaced bsg's SG v3 support with SG v4 and haven't added > > anything new yet. So it can only handle SCSI commands. > > Good start! Just one comment before I look over this and merge it - I'd > prefer keeping this out of sg.c. One of the problems we have right now > are dual pieces of code for sg v3, and I think it would be silly to > continue down this path. Lets keep sg.c as a legacy sg v3 interface > (it'll be the only one except SG_IO in the block layer), and let bsg > take sg v4 and forward. Yeah, that's what I want to do. > There's really zero gain in having it in two places. > > IOW, put the sg v4 structure in bsg.h. That's fine by me. I put the sg v4 structure in sg.h simply I thought bsg.h is for kernel space and some user-space programs need to include sg.h for some old ioctl stuff even if they use only sg v4. > > I've uploaded simple programs to just read/write by using SG v4: > > > > http://www.kernel.org/pub/linux/kernel/people/tomo/sg4/ > > Great! > > I'll get this reviewed and merged as soon as I can, probably monday. Thanks. The patchset is on the top of the two minor patches: http://marc.theaimsgroup.com/?l=linux-scsi&m=116621265732083&w=2 http://marc.theaimsgroup.com/?l=linux-scsi&m=116621265731963&w=2 The latter needs small modifications if you will change blk_rq_unmap_user interface. - 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
