Re: [PATCH v4 0/5] IB/srp: Add RDMA/CM support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2018-01-17 at 18:11 -0500, Doug Ledford wrote:
> On Tue, 2018-01-16 at 10:39 -0800, Bart Van Assche wrote:
> > Hello Jason and Doug,
> > 
> > The five patches in this series add RDMA/CM support to the SRP
> > initiator. Please consider these patches for kernel v4.16.
> > 
> > Thanks,
> > 
> > Bart.
> > 
> > Changes compared to v3:
> > - Moved a kobj_ns_drop() call inside an if-statement to keep the grab and put
> >   ns calls balanced.
> > 
> > Changes compared to v2:
> > - Added a patch that exports kobj_ns_grab_current() and kobj_ns_drop().
> > - Made the rdma_create_id() call also use the namespace of the process that
> >   triggered SRP login. Added kobj_ns_grab_current() and kobj_ns_drop() calls
> >   to the SRP initiator to ensure that that network namespace exists at least
> >   as long as the SRP channel that is using it.
> > 
> > Changes compared to v1:
> > - Use the network namespace of the process that writes into the SRP sysfs login
> >   attribute in rdma_create_id() and inet_pton_with_scope() calls.
> > - Removed an irrelevant part from the RDMA/CM patch description.
> > 
> > Bart Van Assche (5):
> >   kobject: Export kobj_ns_grab_current() and kobj_ns_drop()
> >   IB/srp: Use kstrtoull() instead of simple_strtoull()
> >   IB/srp: Make the path record query error message more informative
> >   IB/srp: Refactor srp_send_req()
> >   IB/srp: Add RDMA/CM support
> > 
> >  drivers/infiniband/ulp/srp/ib_srp.c | 762 ++++++++++++++++++++++++++++--------
> >  drivers/infiniband/ulp/srp/ib_srp.h |  42 +-
> >  include/scsi/srp.h                  |  17 +
> >  lib/kobject.c                       |   2 +
> >  4 files changed, 654 insertions(+), 169 deletions(-)
> > 
> 
> The patchset looks pretty good now, but obviously we need an ack from
> Greg before patch 1 can be applied.
> 

Since I took the majority of the srpt patchset, I took patches 2, 3, and
4 of this set.  We're waiting on Greg's ack on patch 1, and patch 5 is
the only one that needs patch 1, so that at least get's the backlog
widdled down.  Thanks.

-- 
Doug Ledford <dledford@xxxxxxxxxx>
    GPG KeyID: B826A3330E572FDD
    Key fingerprint = AE6B 1BDA 122B 23B4 265B  1274 B826 A333 0E57 2FDD

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux