All of lore.kernel.org
 help / color / mirror / Atom feed
From: Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: Bart Van Assche <bart.vanassche-Sjgp3cTcYWE@public.gmane.org>,
	Jason Gunthorpe <jgg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Cc: Greg Kroah-Hartman
	<gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>,
	linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v4 0/5] IB/srp: Add RDMA/CM support
Date: Wed, 17 Jan 2018 20:35:57 -0500	[thread overview]
Message-ID: <1516239357.3403.299.camel@redhat.com> (raw)
In-Reply-To: <1516230684.3403.290.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

[-- Attachment #1: Type: text/plain, Size: 2281 bytes --]

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-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
    GPG KeyID: B826A3330E572FDD
    Key fingerprint = AE6B 1BDA 122B 23B4 265B  1274 B826 A333 0E57 2FDD

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

      parent reply	other threads:[~2018-01-18  1:35 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-16 18:39 [PATCH v4 0/5] IB/srp: Add RDMA/CM support Bart Van Assche
     [not found] ` <20180116183945.30629-1-bart.vanassche-Sjgp3cTcYWE@public.gmane.org>
2018-01-16 18:39   ` [PATCH v4 1/5] kobject: Export kobj_ns_grab_current() and kobj_ns_drop() Bart Van Assche
     [not found]     ` <20180116183945.30629-2-bart.vanassche-Sjgp3cTcYWE@public.gmane.org>
2018-01-22 15:02       ` Greg Kroah-Hartman
2018-01-16 18:39   ` [PATCH v4 2/5] IB/srp: Use kstrtoull() instead of simple_strtoull() Bart Van Assche
2018-01-16 18:39   ` [PATCH v4 3/5] IB/srp: Make the path record query error message more informative Bart Van Assche
2018-01-16 18:39   ` [PATCH v4 4/5] IB/srp: Refactor srp_send_req() Bart Van Assche
2018-01-16 18:39   ` [PATCH v4 5/5] IB/srp: Add RDMA/CM support Bart Van Assche
2018-01-17 23:11   ` [PATCH v4 0/5] " Doug Ledford
     [not found]     ` <1516230684.3403.290.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2018-01-18  1:35       ` Doug Ledford [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1516239357.3403.299.camel@redhat.com \
    --to=dledford-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=bart.vanassche-Sjgp3cTcYWE@public.gmane.org \
    --cc=gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org \
    --cc=jgg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.