On Mon, 2011-01-24 at 10:18 -0500, Or Gerlitz wrote: > David Dillow wrote: > > Perhaps, except for the need for different devices to call different map > > commands, such as ib_dma_*() and sbus_dma_*() etc. There may be other > > reasons as well.
> BTW - ib_dma_map_xxx and friends are considered by me as bug and not a > feature, I'm not a fan of them either, but I've not gone looking to see if I could find a better solution either. > >> I'm still trying to understand the bigger picture with your patch set and > >> what > >> role the mlx4/mthca patch has in it > > > This patch is orthogonal to the SRP mapping changes and was not used for > > the performance numbers listed -- the 50% improvement was achieved with > > _no_ coalescing in the SG lists. Each 4 KB page was its own SG entry. > > I see, thanks for clarifying this out load and clear, so maybe you'll > first get the seven srp patches reviewed and merged, and only then see > if/what benefit this patch brings, I'm a little bit worried of > changing something below everyone's (srp, iser, rds, p9, nfs-rdma, > lustre, etc) legs which doesn't have any notable benefit, I would be > happy to hear others/opinions Well, I believe it only affects the block merging -- I have to catch a flight, so I cannot recheck right this moment -- that limits the effects to SRP and iSER. The others don't use it at all. We know SRP is fine, and a quick glance suggests that iSER is as well -- you limit the maximum IO size, so you don't get too large of request lists anyways. If you were to expand that, the SCSI midlayer provides hooks that could be used to clamp down the DMA size for devices hanging off iSER. SRP cannot do that, as while it is safe to go to a smaller value than the hardware supports, it is not safe to try and increase it from the default. I'll try to recheck once I get a few moments someplace quiet. -- Dave Dillow National Center for Computational Science Oak Ridge National Laboratory (865) 241-6602 office -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
