[lkml]   [2011]   [Oct]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 2/9] mm: alloc_contig_freed_pages() added
    On Sat, 15 Oct 2011 01:29:33 +0200, Andrew Morton <> wrote:

    > On Thu, 06 Oct 2011 15:54:42 +0200
    > Marek Szyprowski <> wrote:
    >> From: KAMEZAWA Hiroyuki <>
    >> This commit introduces alloc_contig_freed_pages() function
    > The "freed" seems redundant to me. Wouldn't "alloc_contig_pages" be a
    > better name?

    The “freed” is there because the function operates on pages that are in
    buddy system, ie. it is given a range of PFNs that are to be removed
    from buddy system.

    There's also a alloc_contig_range() function (added by next patch)
    which frees pages in given range and then calls
    alloc_contig_free_pages() to allocate them.

    IMO, if there was an alloc_contig_pages() function, it would have to
    be one level up (ie. it would figure out where to allocate memory and
    then call alloc_contig_range()). (That's really what CMA is doing).

    Still, as I think of it now, maybe alloc_contig_free_range() would be
    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: 2011-10-16 10:03    [W:0.023 / U:0.284 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site