Re: + page-owner-tracking.patch added to -mm tree

From: Pekka Enberg
Date: Fri Apr 03 2009 - 10:41:20 EST


Ingo Molnar wrote:
* Eduard - Gabriel Munteanu <eduard.munteanu@xxxxxxxxxxx> wrote:

One thing I'm not sure about this patch is whether it manages to record an allocation only once, i.e. does it log a single event when/if the slab allocator requests pages? Some time ago I sent a patch adding GFP_NOTRACE to gfp.h, but was rejected. Maybe this could be a way out of the mess.

(GFP_NOTRACE would also allow us to log "backend" allocations easily and treat them separately, for the record, or simply filter them out.)

makes a lot of sense IMO to annotate these via a GFP flag.

Yup, make sense. I think I rejected the patch (did I?) because I wanted to fix the slub/slab mess differently but here it makes perfect sense.

Pekka
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/