Re: [PATCH v7 00/23] IB/Verbs: IB Management Helpers

Or Gerlitz
2015-04-30T05:40:20Z

On 4/29/2015 7:28 PM, Doug Ledford wrote:
> On Tue, 2015-04-28 at 17:10 +0200, Michael Wang wrote:
>> >Since v6:
>> > * Thanks to Ira, Devesh for the review and testing:-)
>> > * Thanks for the comments from Sean, Tom, Jason, Doug, Devesh, Ira,
>> > Liran:-) Please remind me if anything missed:-P
>> > * Use query_protocol() and enum protocol type in 1#
>> > * Use rdma_protocol_XX() in 2#
>> > * Drop cma_set_legacy_transport()
>> > * Reserve rdma_ib_or_iboe() and rdma_node_get_transport()
>> > * Updated github repository to v7
> I've taken your patchset and threw it into a for-4.2 branch in my repo.
> This will get it 0day testing. I've also pulled it into my test cluster
> and done minimal testing (bootup, finds devices, gets IP address via dhcp).

Doug,

As I wrote to Michael, this series carries way (way) over Cc: pointers
in each it's commits. We have never had something like this since this
stack was formed 10y ago and accepted into 2.6.12 --- and I didn't hear
any good reason for why doing so (see the thread for "[PATCH v7 04/23]
IB/Verbs: Reform IB-core cm") -- can you please provide your opinion on
the matter?

Or.