Hi David,
On Wed, Sep 5, 2012 at 9:57 PM, David Rientjes <rientjes@xxxxxxxxxx> wrote:
> On Wed, 5 Sep 2012, Ezequiel Garcia wrote:
>
>> @@ -454,15 +455,35 @@ void *__kmalloc_node(size_t size, gfp_t gfp, int node)
>> gfp |= __GFP_COMP;
>> ret = slob_new_pages(gfp, order, node);
>>
>> - trace_kmalloc_node(_RET_IP_, ret,
>> + trace_kmalloc_node(caller, ret,
>> size, PAGE_SIZE << order, gfp, node);
>> }
>>
>> kmemleak_alloc(ret, size, 1, gfp);
>> return ret;
>> }
>> +
>> +void *__kmalloc_node(size_t size, gfp_t gfp, int node)
>> +{
>> + return __do_kmalloc_node(size, gfp, node, _RET_IP_);
>> +}
>> EXPORT_SYMBOL(__kmalloc_node);
>>
>> +#ifdef CONFIG_TRACING
>> +void *__kmalloc_track_caller(size_t size, gfp_t gfp, unsigned long caller)
>> +{
>> + return __do_kmalloc_node(size, gfp, -1, caller);
>
> NUMA_NO_NODE.
>
Mmm, you bring an interesting issue. If you look at mm/slob.c and
include/linux/slob_def.h
there are lots of places with -1 instead of NUMA_NO_NODE.
Do you think it's worth to prepare a patch fixing all of those?
Thanks,
Ezequiel.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>