Re: [PATCH v7 04/23] IB/Verbs: Reform IB-core cm

Michael Wang
2015-04-30T07:28:04Z

On 04/29/2015 05:48 PM, Or Gerlitz wrote:
[snip]
>
>>
>> I think the CC list is not that big for a patch set covered such a wide
>> range, isn't it :-P
>
> Maybe it's a matter of taste, but for me it look way way too big. If
> you really want to have such
> a huge listing, do it in the early patches of the series where you
> introduce the new concepts, and later,on downstream patches, when you
> use it, put one person if they happen to be the author or maintainthat
> area (e.g Sean <-- CM/CMA, Doug/Erez <-- IPoIB Ira, Hal <-- MAD, Steve
> <-- IW_CM, etc)

Thanks for the suggestion, I can't callback correctly who participated
on which part of the review accurately... my bad, will take care next
time :-) and will stop add new CC from now on.

BTW, as now folks already familiar with the cap_XX stuff, may be
the last version to be applied could merge all the cap_XX into one,
after all, it's more focus on the description rather than logical,
separate cap_XX won't help easier the review anyway.

Regards,
Michael Wang

>
> Or.
>