Messages in this thread | | | Date | Sat, 3 Nov 2007 11:31:23 -0700 (PDT) | From | Christoph Lameter <> | Subject | Re: [PATCH 1/2] slub: fix leakage |
| |
On Sat, 3 Nov 2007, Hugh Dickins wrote:
> The culprit turns out to be __slab_alloc(), where it copes with the race > that another task has assigned the cpu slab while we were allocating one. > Don't rush off to load_freelist there: that assumes c->freelist is empty, > and will lose all of its free slots when c->page->freelist is not empty. > Instead just do a local allocation from c->freelist when it has one.
Hmmm.. Right. This will require some fixes to the optimizations in mm.
Acked-by: Christoph Lameter <clameter@sgi.com> - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |