[lkml]   [2006]   [Mar]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [patch] hugetlb strict commit accounting
    On Thu, Mar 09, 2006 at 04:02:06AM -0800, Chen, Kenneth W wrote:
    > David Gibson wrote on Thursday, March 09, 2006 3:27 AM
    > > Again, there are no changes to the fault handler. Including the
    > > promised changes which would mean my instantiation serialization path
    > > isn't necessary ;-).
    > This is the major portion that I omitted in the first patch and is the
    > real kicker that fulfills the promise of guaranteed available hugetlb
    > page for shared mapping.
    > You can shower me all over on the lock protection :-) yes, this is not
    > perfect and was the reason I did not post it earlier, but I want to give
    > you the concept on how I envision this route would work.
    > Again PRIVATE mapping is busted, you can't count them from inode. You
    > would have to count them via mm_struct (I think).

    I don't think there's any sane way to reserve for PRIVATE mappings.
    To do it strictly you'd have to reaccount the whole block on every
    fork(), and that would mean that any process using >0.5 of the
    system's hugepages could never fork(), even if the child was just
    going to exec().

    Given that, it's simplest just to allow free overcommit for PRIVATE
    mappings. *But* you can ensure that PRIVATE allocations (i.e. COW
    faults) don't mess with any previously reserved SHARED mappings.

    > Note: definition of "reservation" in earlier patch is total hugetlb pages
    > needed for that file, including the one that is already faulted in. Maybe
    > that throw you off a bit because I'm guessing your definition is "needed
    > in the future" and probably you are looking for a decrement of the counter
    > in the fault path?

    No, I realised that distinction.

    David Gibson | I'll have my music baroque, and my code
    david AT | minimalist, thank you. NOT _the_ _other_
    | _way_ _around_!
    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: 2006-03-09 13:18    [W:0.025 / U:3.364 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site