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
next prev 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