Re: [PATCH rdma-next 00/10] Relaxed ordering memory regions

Jason Gunthorpe
2020-01-16T20:11:06Z

On Thu, Jan 16, 2020 at 11:20:08AM +0200, Leon Romanovsky wrote:
> On Wed, Jan 15, 2020 at 02:08:48PM -0400, Jason Gunthorpe wrote:
> > On Wed, Jan 08, 2020 at 08:05:30PM +0200, Yishai Hadas wrote:
> > > This series adds an ioctl command to allocate an async event file followed by a
> > > new ioctl command to get a device context.
> > >
> > > The get device context command enables reading some core generic capabilities
> > > such as supporting an optional MR access flags by IB core and its related
> > > drivers.
> > >
> > > Once the above is enabled, a new optional MR access flag named
> > > IB_UVERBS_ACCESS_RELAXED_ORDERING is added and is used by mlx5 driver.
> > >
> > > This optional flag allows creation of relaxed ordering memory regions. Access
> > > through such MRs can improve performance by allowing the system to reorder
> > > certain accesses.
> > >
> > > As relaxed ordering is an optimization, drivers that do not support it can
> > > simply ignore it.
> > >
> > > Note: This series relies on the 'Refactoring FD usage' series [1] that was sent
> > > to rdma-next.
> > > [1] https://patchwork.kernel.org/project/linux-rdma/list/?series=225541
> > >
> > > Yishai
> > >
> > > Jason Gunthorpe (3):
> > > RDMA/core: Add UVERBS_METHOD_ASYNC_EVENT_ALLOC
> > > RDMA/core: Remove ucontext_lock from the uverbs_destry_ufile_hw() path
> > > RDMA/uverbs: Add ioctl command to get a device context
> > >
> > > Michael Guralnik (7):
> > > net/mlx5: Expose relaxed ordering bits
> > > RDMA/uverbs: Verify MR access flags
> > > RDMA/core: Add optional access flags range
> > > RDMA/efa: Allow passing of optional access flags for MR registration
> > > RDMA/uverbs: Add new relaxed ordering memory region access flag
> > > RDMA/core: Add the core support field to METHOD_GET_CONTEXT
> > > RDMA/mlx5: Set relaxed ordering when requested
> >
> > This looks OK, can you update the shared branch please
>
> Thanks, applied
> f4db8e8b0dc3 net/mlx5: Expose relaxed ordering bits

Okay, applied to for-next

Thanks,
Jason