public inbox for linux-rdma@vger.kernel.org
 help / color / mirror / Atom feed
From: "Chandramouli, Dasaratharaman" <dasaratharaman.chandramouli-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
To: Don Hiatt <don.hiatt-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
	Ira Weiny <ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
	Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	linux-rdma <linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH v2 0/7] Add support for OPA classport info
Date: Fri, 14 Apr 2017 16:32:48 -0700	[thread overview]
Message-ID: <c3adeff8-d196-bb22-18f9-dfbd48b4149d@intel.com> (raw)
In-Reply-To: <1490053090-32840-1-git-send-email-dasaratharaman.chandramouli-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>

Looks like there hasn't been any comments on this patchset.
Doug, if you have no concerns, would you be able to pick it up.

Thanks,
Dasa

On 3/20/2017 4:38 PM, Dasaratharaman Chandramouli wrote:
> This series moves the classport info query initiation and update
> from callers such as ipoib to the ib_sa module itself. The classport
> info cache is updated whenever ib_sa receives an appropriate state
> change event.
>
> Since classport info is only used to check if sendonly full member support
> is enabled by the SM, we expose a function ib_sa_sendonly_fullmem_support
> that can be called to check if the support is enabled.
>
> Additionally, we introduce support for opa classport info. These are
> defined specifically for OPA devices and expose additional features in the
> capability mask bits along with longer LID sizes in some of the other
> fields.
>
> Patch 1 to 3 fix checkpatch issues (1 issue type per patch) on two
> functions that patch 4 then moves around. Patch 5 makes changes
> to implicitly query and cache classport info. Patch 6 adds
> verbs capability API for core layers to query and find out if they
> are running on an OPA device. Finally, patch 7 adds OPA classport info
> query support.
>
> Changes from v0:
> ---------------
> 1. Initiate Classport Info refresh also on a LID change and SM change
> events.
>
> 2. Make proper use of the 'flags' field in struct ib_sa_query that carries
> the flag IB_SA_QUERY_OPA. This makes changes to static functions such as
> alloc_mad and init_mad unnecessary.
>
> 3. Remove the unwanted rename of CLASSPORTINFO_REC_FIELD to
> IB_CLASSPORTINFO_REC_FIELD.
>
> 4. Fix the subject prefix. Remove 'rdma-core' from the subject.
>
> Dasaratharaman Chandramouli (7):
>   IB/SA: Fix lines longer than 80 columns
>   IB/SA: Add braces when using sizeof
>   IB/SA: Remove unwanted braces
>   IB/SA: Move functions update_sm_ah() and ib_sa_event()
>   IB/SA: Modify SA to implicitly cache Class Port info
>   IB/core: Add rdma_cap_opa_ah to expose opa address handles
>   IB/SA: Add support to query opa classport info.
>
>  drivers/infiniband/core/cma.c                  |  76 +---
>  drivers/infiniband/core/sa_query.c             | 521 +++++++++++++++++--------
>  drivers/infiniband/hw/hfi1/mad.c               |  25 --
>  drivers/infiniband/ulp/ipoib/ipoib.h           |   1 -
>  drivers/infiniband/ulp/ipoib/ipoib_main.c      |  71 ----
>  drivers/infiniband/ulp/ipoib/ipoib_multicast.c |   9 +-
>  include/rdma/ib_mad.h                          |  25 ++
>  include/rdma/ib_sa.h                           |  13 +-
>  include/rdma/ib_verbs.h                        |  16 +
>  9 files changed, 418 insertions(+), 339 deletions(-)
>
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2017-04-14 23:32 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-20 23:38 [PATCH v2 0/7] Add support for OPA classport info Dasaratharaman Chandramouli
     [not found] ` <1490053090-32840-1-git-send-email-dasaratharaman.chandramouli-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2017-03-20 23:38   ` [PATCH v2 1/7] IB/SA: Fix lines longer than 80 columns Dasaratharaman Chandramouli
2017-03-20 23:38   ` [PATCH v2 2/7] IB/SA: Add braces when using sizeof Dasaratharaman Chandramouli
2017-03-20 23:38   ` [PATCH v2 3/7] IB/SA: Remove unwanted braces Dasaratharaman Chandramouli
2017-03-20 23:38   ` [PATCH v2 4/7] IB/SA: Move functions update_sm_ah() and ib_sa_event() Dasaratharaman Chandramouli
2017-03-20 23:38   ` [PATCH v2 5/7] IB/SA: Modify SA to implicitly cache Class Port info Dasaratharaman Chandramouli
2017-03-20 23:38   ` [PATCH v2 6/7] IB/core: Add rdma_cap_opa_ah to expose opa address handles Dasaratharaman Chandramouli
2017-03-20 23:38   ` [PATCH v2 7/7] IB/SA: Add support to query opa classport info Dasaratharaman Chandramouli
2017-04-14 23:32   ` Chandramouli, Dasaratharaman [this message]
2017-04-28 18:10   ` [PATCH v2 0/7] Add support for OPA " Doug Ledford
     [not found]     ` <1493403017.3041.166.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-04-28 18:47       ` Chandramouli, Dasaratharaman
     [not found]         ` <9a061845-2a2f-3c19-08b3-00a9ddab8292-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2017-04-28 19:23           ` Chandramouli, Dasaratharaman
     [not found]             ` <b6f3caf1-7558-1291-b361-ac58932d10ae-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2017-04-28 22:15               ` Doug Ledford
     [not found]                 ` <1493417741.3041.171.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2017-04-28 23:35                   ` Chandramouli, Dasaratharaman
     [not found]                     ` <07556307-eea5-5571-66d7-90be9abebba0-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2017-04-28 23:46                       ` Chandramouli, Dasaratharaman
     [not found]                         ` <05c8bb91-7291-4cda-6299-5f6061ef635e-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2017-04-29 18:49                           ` Chandramouli, Dasaratharaman

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=c3adeff8-d196-bb22-18f9-dfbd48b4149d@intel.com \
    --to=dasaratharaman.chandramouli-ral2jqcrhueavxtiumwx3w@public.gmane.org \
    --cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=don.hiatt-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=ira.weiny-ral2JQCrhuEAvxtiuMwx3w@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox