On Thursday 20 December 2007 18:58:07 David Miller wrote: > From: Rusty Russell <rusty@xxxxxxxxxxxxxxx> > Date: Thu, 20 Dec 2007 18:53:48 +1100 > > > Manipulating the magic chains is horrible; it looks simple to the > > places which simply want to iterate through it, but it's awful for > > code which wants to create them. > > I'm not saying complexity is inherent in this stuff, but > assuming that it is the complexity should live as far away > from the minions (the iterators in this case). Therefore, > the creators is the right spot for the hard stuff. In this case, the main benefit of the sg chaining was that the conversion of most scsi drivers was easy (basically sg++ -> sg = sg_next(sg)). The conversion to sg_ring is more complex, but the end result is not significantly more complex. However, the cost to code which manipulates sg chains was significant: I tried using them in virtio and it was too ugly to live (so that doesn't support sg chaining). If this was the best we could do, that'd be fine. But, as demonstrated, there are real benefits of having an explicit header: 1) It removes the chain-end/explicit count ambiguity (see http://lkml.org/lkml/2007/10/25/209 & thread) 2) It allows others to manipulate sg chains, which couldn't be done before (eg. the ATA code which wants to append a padding element). 3) I can now hand you an sg ring for you to fill: sg chains can't do that. In short, sg_ring is generally useful primitive. sg chains are a clever hack for scsi_lib to create, and everyone else to read. Hope that clarifies, Rusty. - 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
