On Mon, Nov 23, 2015 at 07:57:42PM -0500, Tom Talpey wrote: > On 11/23/2015 5:14 PM, Chuck Lever wrote: > >FMR's ro_unmap method is already synchronous because ib_unmap_fmr() > >is a synchronous verb. However, some improvements can be made here. > > I thought FMR support was about to be removed in the core. Seems like everyone would love to kill it, but no one dares to do it just yet. Reasons to keep FMRs: - mthca doesn't support FRs but haven't been staged out - RDS only supports FRMs for the IB transports (it does support FRs for an entirely separate iWarp transports. I'd love to know why we can't just use that iWarp transport for IB as well). - mlx4 hardware might be slower with FRs than FRMs (Or mentioned this in reply to the iSER remote invalidation series). So at lest for 4.5 we're unlikely to be able to get rid of it alone due to the RDS issue. We'll then need performance numbers for mlx4, and figure out how much we care about mthca. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
