lkml.org 
[lkml]   [2010]   [Mar]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 0/7] tcp: bugs and cleanup for 2.6.34-rc1
    On 3/11/10 10:01 AM, Eric Dumazet wrote:
    > It would be nice if you could update your knowledge of how linux
    > development works these days.
    >
    Perhaps you could supply pointers to the relevant documentation?


    > You throw too many changes at once to let them being reviewed,
    > understood, and accepted.
    >
    These were originally submitted in groups of 1, 2, and 3 patches for
    review. For example, as TCPCT parts 1h and 1i (and had been part of
    earlier patch series, too), as of 2009-12-03:

    http://patchwork.ozlabs.org/patch/40284/

    Resubmitted again in even finer grained patches, as of 2009-12-31 and
    again 2010-01-06:

    http://lkml.org/lkml/2010/1/6/299

    Since February, I've grouped them all together, as they've been reviewed,
    and re-reviewed -- yet only deprecated by the netdev maintainer.


    > For your information, we had to correct a fatal bug introduced by your
    > last commits, and as far as I know, you didnt help that much.
    >
    > http://git2.kernel.org/?p=linux/kernel/git/davem/net-2.6.git;a=commit;h=28b2774a0d5852236dab77a4147b8b88548110f1
    >
    Splendid! Thank you for the heads up! Unfortunately, that 3 day old
    email wasn't CC'd to me. I'm glad to hear that Mika is testing.

    And thank you for testing the patch. (You did test, didn't you?)
    Testing is always good!

    That code always worked for me, and presumably for Adam (who wrote it).
    We've always used small amounts of data -- only 64 bytes, as originally
    specified. The latest API document allows up to 1220. Folks just keep
    wanting more!

    (The latest API also drops the subscript, so that patch would have been
    changed eventually....)

    This code (PATCH v3 5/7) handles the data on the receiving side of the
    same transaction, a patch that was first submitted over 18 months ago!


    > <at this moment of time>
    >
    > We are post linux-2.6.34-rc1, so only bug fixes are wanted by Linus and
    > David, to be integrated in 2.6.34 (and previous versions if needed)
    >
    > We are _not_ interested by new stuff at *this* moment, especially if it
    > takes lot of time to review.
    >
    > </at this moment of time>
    >
    Good. Because this isn't new stuff. It's bug fixes and related cleanup.
    Generally, the cleanup was needed to find and test the bugs and patches.

    They're already "split up" from the main set of patches, as Ilpo asked
    over 4 months ago.

    I've not been making *any* new submissions around here, until *existing*
    submissions have been applied.


    > Last time I made some comments on your patches, you just ignored them or
    > loaned, because obviously who is Eric Dumazet to tell William Allen
    > Simpson how things should be done ? Silly me !
    >
    Last time you made any comments at all, it was trivial argument about
    parenthesis and casts. I asked directly for more *substantive* review:

    http://lkml.org/lkml/2010/1/13/162

    Thankfully, we've had substantive review from Andi over a period of
    months, on parts 3 and 4 of the current patch series....

    And a short attaboy of part 2 a couple of weeks ago.


    > Speaking for myself, but as your previous mails were ignored, I felt it
    > was time to clarify the points.
    >
    Thank you.


    \
     
     \ /
      Last update: 2010-03-11 18:41    [W:0.025 / U:30.024 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site