lkml.org 
[lkml]   [2015]   [Apr]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [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.
    >


    \
     
     \ /
      Last update: 2015-04-30 10:01    [W:5.265 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site