[lkml]   [2008]   [Apr]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] scsi: fix sense_slab/bio swapping livelock
    On Tue, 8 Apr 2008, Pekka Enberg wrote:
    > Pekka Enberg wrote:
    > > Although you weren't convinced by my arguments, I still have difficulties
    > > understanding why this kind of bad behavior would be acceptable in an
    > > embedded environment and why we don't need to fix it for the SLOB case as
    > > well.

    If we do decide that it's seriously bad behaviour, then of course
    it should be fixed in SLOB also. I just meant that the fact that
    SLOB's been merging slabs too doesn't give me great confidence that
    that's the right and safe thing to do: SLOB isn't our gold standard.

    > >
    > > But you do bring up a good point of SLUB changing the behavior on OOM
    > > situations for which SLAB_NOMERGE sounds like a good-enough stop-gap measure
    > > for the short term. I would prefer some other fix even if it means getting
    > > rid of slab merging competely (which would suck as it's very nice for making
    > > memory footprint smaller).
    > I wonder if we can get away with a SLAB_IO flag that you can use to annotate
    > caches that participate in writeout and the allocator could keep some spare
    > pages around that can be handed out for them in case of OOM...

    Well, the work of keeping spares around to be handed out in case of
    OOM was done in 2.5 with mempools. They seem to be working well
    (until you try swapping over network as Peter is pursuing),
    I don't think we need to duplicate that.

    I've the uncomfortable feeling that I'm making a big fuss over this
    behaviour on the one hand, but utterly failing to justify that fuss
    on the other. It seems to come down to me wanting fewer backtraces
    in my logs: I'd better find stronger reasons than that if we're to
    pursue this.


     \ /
      Last update: 2008-04-07 23:29    [W:0.020 / U:0.448 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site