Messages in this thread | | | Date | Tue, 22 Sep 2009 21:54:33 +0300 | Subject | Re: [PATCH 2/4] slqb: Record what node is local to a kmem_cache_cpu | From | Pekka Enberg <> |
| |
Hi Mel,
On Tue, Sep 22, 2009 at 4:54 PM, Mel Gorman <mel@csn.ul.ie> wrote: >> I don't understand how the memory leak happens from the above >> description (or reading the code). page_to_nid() returns some crazy >> value at free time? > > Nope, it isn't a leak as such, the allocator knows where the memory is. > The problem is that is always frees remote but on allocation, it sees > the per-cpu list is empty and calls the page allocator again. The remote > lists just grow. > >> The remote list isn't drained properly? > > That is another way of looking at it. When the remote lists get to a > watermark, they should drain. However, it's worth pointing out if it's > repaired in this fashion, the performance of SLQB will suffer as it'll > never reuse the local list of pages and instead always get cold pages > from the allocator.
I worry about setting c->local_nid to the node of the allocated struct kmem_cache_cpu. It seems like an arbitrary policy decision that's not necessarily the best option and I'm not totally convinced it's correct when cpusets are configured. SLUB seems to do the sane thing here by using page allocator fallback (which respects cpusets AFAICT) and recycling one slab slab at a time.
Can I persuade you into sending me a patch that fixes remote list draining to get things working on PPC? I'd much rather wait for Nick's input on the allocation policy and performance.
Pekka
| |