Re: [PATCH v2 0/7] Add support for OPA classport info

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

 





On 4/28/2017 11:47 AM, Chandramouli, Dasaratharaman wrote:


On 4/28/2017 11:10 AM, Doug Ledford wrote:
On Mon, 2017-03-20 at 19:38 -0400, 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.

I took patches 1-6 of this series.  However, I need you to rebase patch
7 against my current k.o/for-4.12-rdma-netdevice branch as there are
significant conflicts between this and the VNIC patches I've already
taken.


Hi Doug -- I pulled your for-4.12-rdma-netdevice and tried to apply
patch 7. I see no conflicts. It compiled cleanly as well. May be i am
missing something here. Just want to make sure i have the same branch as
yours.
Is this your commit at the HEAD currently?

commit 94d595c56077fd8b0f61701e03fd4b3dc8c62038
Author: Dasaratharaman Chandramouli <dasaratharaman.chandramouli@xxxxxxxxx>
Date:   Mon Mar 20 19:38:09 2017 -0400

    IB/core: Add rdma_cap_opa_ah to expose opa address handles


Thanks,
Dasa

I take that back. It does fail to compile when OPA_VNIC is enabled since struct opa_class_port_info is defined at multiple locations. I will re-spin this series and in lieu of patch 7, i will submit two patches, the first one would cleanup some of the re-defined structure definitions and the next one would add SA support for OPA class port info.

Once this series is accepted, as cleanup, we can look into fixing opa_vnic and hfi1 to use the SA to query for OPA classport info.


Thanks,
Dasa
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[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