Re: Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

On Tue, Dec 01, 2015 at 10:33:18AM -0500, Chuck Lever wrote: > Oracle recently announced Sonoma, which is a SPARC CPU with > an on-chip IB HCA. Oracle plans to publish an open-source > GPL device driver that enables this HCA in Linux for SPARC. > We???d eventually like to contribute it to the

Re: Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

> On Nov 24, 2015, at 2:12 AM, Jason Gunthorpe > wrote: > > On Mon, Nov 23, 2015 at 10:52:26PM -0800, Christoph Hellwig wrote: >> >> 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

Re: Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

On 25/11/2015 21:28, Jason Gunthorpe wrote: On Wed, Nov 25, 2015 at 09:09:20AM -0800, santosh shilimkar wrote: I'd say drop the current iWarp transport if it's not testable. The only real difference between IB and iWarp is the needed to create a MR for the RDMA READ sink, and we're much

Re: Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

On 11/25/2015 1:00 AM, Christoph Hellwig wrote: On Tue, Nov 24, 2015 at 01:54:02PM -0800, santosh shilimkar wrote: As already indicated to Sagi [1], RDS IB FR support is work in progress and I was hoping to get it ready for 4.5. There are few issues we found with one of the HCA and hence the

Re: Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

On Wed, Nov 25, 2015 at 7:09 PM, santosh shilimkar wrote: >>> As already indicated to Sagi [1], RDS IB FR support is work in >>> progress and I was hoping to get it ready for 4.5. These are really good news! can you please elaborate a bit on the design changes this

Re: Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

On 11/25/2015 10:22 AM, Or Gerlitz wrote: On Wed, Nov 25, 2015 at 7:09 PM, santosh shilimkar wrote: As already indicated to Sagi [1], RDS IB FR support is work in progress and I was hoping to get it ready for 4.5. These are really good news! can you please

Re: Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

On Wed, Nov 25, 2015 at 09:09:20AM -0800, santosh shilimkar wrote: > >I'd say drop the current iWarp transport if it's not testable. The > >only real difference between IB and iWarp is the needed to create > >a MR for the RDMA READ sink, and we're much better of adding that into > >the current IB

Re: Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

On Tue, Nov 24, 2015 at 01:54:02PM -0800, santosh shilimkar wrote: > As already indicated to Sagi [1], RDS IB FR support is work in > progress and I was hoping to get it ready for 4.5. There are few > issues we found with one of the HCA and hence the progress > slowed down. Looking at where we

Re: Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

Hi Christoph, On 11/23/2015 10:52 PM, Christoph Hellwig wrote: 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

Re: Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

On 11/24/2015 1:52 AM, Christoph Hellwig wrote: 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

Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

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

Re: Future of FMR support, was: Re: [PATCH v1 5/9] xprtrdma: Add ro_unmap_sync method for FMR

On Mon, Nov 23, 2015 at 10:52:26PM -0800, Christoph Hellwig wrote: > > 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. mthca is unfortunately very popular in