From: David Rientjes on
On Wed, 30 Jun 2010, Xiaotian Feng wrote:

> In slab, all __xxx_track_caller is defined on CONFIG_DEBUG_SLAB || CONFIG_TRACING,
> thus caller tracking function should be worked for CONFIG_TRACING. But if
> CONFIG_DEBUG_SLAB is not set, include/linux/slab.h will define xxx_track_caller to
> __xxx() without consideration of CONFIG_TRACING. This will break the caller tracking
> behaviour then.
>
> Signed-off-by: Xiaotian Feng <dfeng(a)redhat.com>
> Cc: Christoph Lameter <cl(a)linux-foundation.org>
> Cc: Pekka Enberg <penberg(a)cs.helsinki.fi>
> Cc: Matt Mackall <mpm(a)selenic.com>
> Cc: Vegard Nossum <vegard.nossum(a)gmail.com>
> Cc: Dmitry Monakhov <dmonakhov(a)openvz.org>
> Cc: Catalin Marinas <catalin.marinas(a)arm.com>
> Cc: David Rientjes <rientjes(a)google.com>

Acked-by: David Rientjes <rientjes(a)google.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Pekka Enberg on
David Rientjes wrote:
> On Wed, 30 Jun 2010, Xiaotian Feng wrote:
>
>> In slab, all __xxx_track_caller is defined on CONFIG_DEBUG_SLAB || CONFIG_TRACING,
>> thus caller tracking function should be worked for CONFIG_TRACING. But if
>> CONFIG_DEBUG_SLAB is not set, include/linux/slab.h will define xxx_track_caller to
>> __xxx() without consideration of CONFIG_TRACING. This will break the caller tracking
>> behaviour then.
>>
>> Signed-off-by: Xiaotian Feng <dfeng(a)redhat.com>
>> Cc: Christoph Lameter <cl(a)linux-foundation.org>
>> Cc: Pekka Enberg <penberg(a)cs.helsinki.fi>
>> Cc: Matt Mackall <mpm(a)selenic.com>
>> Cc: Vegard Nossum <vegard.nossum(a)gmail.com>
>> Cc: Dmitry Monakhov <dmonakhov(a)openvz.org>
>> Cc: Catalin Marinas <catalin.marinas(a)arm.com>
>> Cc: David Rientjes <rientjes(a)google.com>
>
> Acked-by: David Rientjes <rientjes(a)google.com>

Applied, thanks!
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/