Messages in this thread | | | Date | Thu, 30 Apr 2015 09:28:04 +0200 | From | Michael Wang <> | Subject | Re: [PATCH v7 04/23] IB/Verbs: Reform IB-core cm |
| |
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. >
| |