lkml.org 
[lkml]   [2008]   [Nov]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/7] mm: introduce simple_malloc()/simple_free()
    Arjan van de Ven wrote:
    > On Mon, 17 Nov 2008 07:39:55 +1000
    >
    > I would go further than this.
    >
    > Make the code just use vmalloc(). Period.
    >
    > But then make vmalloc() smart and try do a direct mapping allocation
    > first, before falling back to a virtual mapping. (and based on size it
    > wouldn't even try it for just big things)
    >
    >
    >

    Hi, Arjan van de Ven

    (I'll rename simple_malloc/simple_free to kvmalloc/kvfree)

    I think vmalloc() should only do one thing(virtual mapping). Your idea
    can be implemented in helper function kvmalloc() if there is a good
    algorithm provided for it.

    kvmalloc() is for cleanup mostly. It will remove existed duplicate code.
    As David and I pointed out, vmalloc() is need for some good reason.

    If we do not introduce a helper function, these duplicate code
    are still spread everywhere.


    kvfree() is not only for free the memory allocated by kvmalloc().

    kvfree() frees the memory that we don't know whether it was allocated
    by kmalloc() nor vmalloc().

    Someone use a flag for it, and other guys calculate size of memory again
    before kfree() or vfree(). these two ways increase complexity, and
    it is hard to prove the re-calculated size is reliable.

    kvfree() will remove these needless complexity!



    Thanx, Lai



    \
     
     \ /
      Last update: 2008-11-17 03:15    [W:3.179 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site