[lkml]   [2005]   [Sep]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH][BUG] fix memory leak on reading cpuset files after seeking beyond eof
    Takahiro-san wrote:
    > I fixed the bug. Sorry for my previous incomplete patch.
    > The following patch is against linux-2.6.14-rc2.

    The patch looks perfect. But it needs a good opening comment and

    The header comment of the patch needs to be just the words that should
    go in the source control (git) log comment, for someone to read to
    understand this patch, years from now, when they have long since
    forgotten our little email thread here. Andrew Morton's and Linus
    Torvalds's automatic patch handling tools just pull that comment out,
    unedited, and put it into the log of changes for the kernel/cpuset.c
    file, when they accept a patch.

    The Subject needs to be a good name for the patch. I always use a
    short phrase starting with the word "cpuset" if it is a cpuset patch.
    If it is a fix, Andrew seems to enjoy having the word 'fix' as the
    last word of the Subject. The tools of Andrew and Linus will take that
    exact Subject line, trim off whatever is inside the '[...]', replace
    spaces with hyphens '-', and use that for the name of the patch file.

    Directly include Andrew or Linus in the Cc list, if you want them
    to apply the patch. They might notice the patch even if you don't Cc
    them, but it is less certain in that case.

    Ah - one cute trick - I hand edited the pathname of the file that shows
    in the first two lines of the actual patch, to show the release of Linux
    "linux-2.6.14-rc2" you worked against. I hope I didn't break the patch
    by hand editing it. If I were good, I would send the patch just to
    myself first, and make sure it still applied. But I am lazy and over

    See also the following three documents, on how to submit patches
    to the Linux kernel:

    1) Documentation/SubmittingPatches, a file in the kernel source
    2) Andrew Morton's "The Perfect Patch", available at:
    3) Jeff Garzik's "Linux kernel patch submission format", at:

    I will resend this patch with a suitable header comment and Subject
    in a few minutes. You will easily see what I mean when you see my next

    The header comment on what I will resend now will be quite short,
    as this is an easy patch to explain. The header comments on most of
    the patches I send are much longer -- probably too long, but Andrew
    seems reluctant to complain about comments that are too long. He
    is much more likely to complain if they are too short.

    Excellent work. Thank-you.

    I won't rest till it's the best ...
    Programmer, Linux Scalability
    Paul Jackson <> 1.925.600.0401
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-09-28 15:45    [W:0.023 / U:28.172 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site