[lkml]   [2007]   [Aug]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] NOMMU: Separate out VMAs
    David Howells wrote:
    > Bernd Schmidt <> wrote:
    >> In do_mmap_private, I've commented out the logic to free excess pages, as it
    >> fragments terribly
    > I wonder if there's a good heuristic for this. The problem is that whilst
    > not releasing excess pages _may_ seem like a good idea, if your system is
    > something like a single persistent app, then it really is not.
    > For instance, if such an app allocates a byte over 16MB (perhaps implicitly in
    > the binfmt driver), then you'd completely waste a large chunk of RAM. In the
    > 16MB+1 case, the wastage would be a byte less than 16MB.

    I think it would be good to have a mechanism to group free pages by
    purpose - so that if we break up a high-order page in order to allocate
    memory for process A, then the remaining pages remain in a special pool
    that the allocator will prefer to hand out only to process A.

    >> Also, I think you're freeing high-order pages unaligned to
    >> their order?
    > Yeah, but some of the pages might still be in use when we want to release
    > them.

    Not following you here. Is it valid to free an order-2 page that's not
    aligned at order-2?

    >> In do_munmap, we can deal with freeing more than one vma. I've not touched
    >> the rb-tree logic in the shared file case, as I have no idea what it's trying
    >> to do given that only exact matches are allowed.
    > I'd generally rather not do this. You can't use MAP_FIXED to request adjacent
    > regions, so why should you anticipate there would be any?

    Adjacent regions can happen by accident, and the uClibc malloc will try
    to merge free areas when they are adjacent - there's a lot of
    special-case code in there to prevent this on uClinux systems by
    essentially duplicating VMA tracking. That's something we want to
    avoid, because it eats performance (especially in threaded apps due to
    additional locking).

    This footer brought to you by insane German lawmakers.
    Analog Devices GmbH Wilhelm-Wagenfeld-Str. 6 80807 Muenchen
    Sitz der Gesellschaft Muenchen, Registergericht Muenchen HRB 40368
    Geschaeftsfuehrer Thomas Wessel, William A. Martin, Margaret Seif
    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: 2007-08-20 18:07    [W:0.022 / U:4.784 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site