lkml.org 
[lkml]   [2009]   [Jun]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH v2] slab,slub: ignore __GFP_WAIT if we're booting or suspending
    From
    Date

    > We emit a debug warning but dont crash, so all should be fine and
    > the culprits can then be fixed, right?

    ... rewind ... :-)

    Ok so, no, the culprit cannot be all fixed in a satifactory way.

    The main reason is that I believe it's not "right" to have every caller
    of slab around know whether GFP_KERNEL is good to go or it should get
    into GFP_NOWAIT. This depends on many factors (among others us moving
    things around more), and is not actually a good solution for thing that
    can be called both at boot and later, such as get_vm_area().

    I really think we are looking for trouble (and a lot of hidden bugs) by
    trying to "fix" all callers, in addition to making some code like
    vmalloc() more failure prone because it's unconditionally changed from
    GFP_KERNEL to GFP_NOWAIT.

    It seems a lot more reasonably to me to have sl*b naturally degrade to
    NOWAIT when it's too early to enable interrupts.

    In addition, my proposal of having bits to mask off gfp will also be
    useful in fixing similar issues with suspend/resume vs. GFP_NOIO which
    should really become implicit when devices start becoming suspended.

    Cheers,
    Ben.




    \
     
     \ /
      Last update: 2009-06-12 11:25    [W:0.035 / U:0.816 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site