From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail144.messagelabs.com (mail144.messagelabs.com [216.82.254.51]) by kanga.kvack.org (Postfix) with ESMTP id C3DFA6B01AC for ; Sun, 4 Jul 2010 12:51:03 -0400 (EDT) Message-ID: <4C30BBF0.1030801@cs.helsinki.fi> Date: Sun, 04 Jul 2010 19:50:56 +0300 From: Pekka Enberg MIME-Version: 1.0 Subject: Re: [PATCH V2] slab: fix caller tracking on !CONFIG_DEBUG_SLAB && CONFIG_TRACING References: <1277891842-18898-1-git-send-email-dfeng@redhat.com> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org To: David Rientjes Cc: Xiaotian Feng , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Christoph Lameter , Matt Mackall , Vegard Nossum , Dmitry Monakhov , Catalin Marinas List-ID: 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 >> Cc: Christoph Lameter >> Cc: Pekka Enberg >> Cc: Matt Mackall >> Cc: Vegard Nossum >> Cc: Dmitry Monakhov >> Cc: Catalin Marinas >> Cc: David Rientjes > > Acked-by: David Rientjes Applied, thanks! -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org