From mboxrd@z Thu Jan 1 00:00:00 1970 From: richard -rw- weinberger Date: Sat, 07 May 2011 09:34:13 +0000 Subject: Re: [PATCH] lib: Consolidate DEBUG_STACK_USAGE option Message-Id: List-Id: References: <1304747831-2098-1-git-send-email-sboyd@codeaurora.org> In-Reply-To: <1304747831-2098-1-git-send-email-sboyd@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Stephen Boyd Cc: Andrew Morton , linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, linux-arm-kernel@lists.infradead.org, uclinux-dist-devel@blackfin.uclinux.org, linux-m32r@ml.linux-m32r.org, linux-mips@linux-mips.org, linuxppc-dev@lists.ozlabs.org, linux-sh@vger.kernel.org, sparclinux@vger.kernel.org, Chris Metcalf , user-mode-linux-devel@lists.sourceforge.net, x86@kernel.org On Sat, May 7, 2011 at 7:57 AM, Stephen Boyd wrote: > Most arches define CONFIG_DEBUG_STACK_USAGE exactly the same way. > Move it to lib/Kconfig.debug so each arch doesn't have to define > it. This obviously makes the option generic, but that's fine > because the config is already used in generic code. > > It's not obvious to me that sysrq-P actually does anything > different with this option enabled, but I erred on the side of > caution by keeping the most inclusive wording. > > Cc: linux-arch@vger.kernel.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: uclinux-dist-devel@blackfin.uclinux.org > Cc: linux-m32r@ml.linux-m32r.org > Cc: linux-mips@linux-mips.org > Cc: linuxppc-dev@lists.ozlabs.org > Cc: linux-sh@vger.kernel.org > Cc: sparclinux@vger.kernel.org > Cc: Chris Metcalf > Cc: user-mode-linux-devel@lists.sourceforge.net > Cc: x86@kernel.org > Signed-off-by: Stephen Boyd > --- Acked-by: Richard Weinberger -- Thanks, //richard From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754135Ab1EGJeR (ORCPT ); Sat, 7 May 2011 05:34:17 -0400 Received: from mail-qw0-f46.google.com ([209.85.216.46]:61431 "EHLO mail-qw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753737Ab1EGJeP (ORCPT ); Sat, 7 May 2011 05:34:15 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=mgV2bLMx3VUAdB+8XUTU0H9RJ/WP+1TEAnuSBJ1vFVpKOTcIDroyKjWIMjqrc7nflU 0Y/UqcBlwgdKW0qhCcmexDYGUxopDjq2pM6PXuFMiLLogwXE56geGoe4Qbauz23ExPEX qWap9e9El189yStq2x45uRgQ2aOiQTjJq9s+w= MIME-Version: 1.0 In-Reply-To: <1304747831-2098-1-git-send-email-sboyd@codeaurora.org> References: <1304747831-2098-1-git-send-email-sboyd@codeaurora.org> Date: Sat, 7 May 2011 11:34:13 +0200 Message-ID: Subject: Re: [PATCH] lib: Consolidate DEBUG_STACK_USAGE option From: richard -rw- weinberger To: Stephen Boyd Cc: Andrew Morton , linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, linux-arm-kernel@lists.infradead.org, uclinux-dist-devel@blackfin.uclinux.org, linux-m32r@ml.linux-m32r.org, linux-mips@linux-mips.org, linuxppc-dev@lists.ozlabs.org, linux-sh@vger.kernel.org, sparclinux@vger.kernel.org, Chris Metcalf , user-mode-linux-devel@lists.sourceforge.net, x86@kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, May 7, 2011 at 7:57 AM, Stephen Boyd wrote: > Most arches define CONFIG_DEBUG_STACK_USAGE exactly the same way. > Move it to lib/Kconfig.debug so each arch doesn't have to define > it. This obviously makes the option generic, but that's fine > because the config is already used in generic code. > > It's not obvious to me that sysrq-P actually does anything > different with this option enabled, but I erred on the side of > caution by keeping the most inclusive wording. > > Cc: linux-arch@vger.kernel.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: uclinux-dist-devel@blackfin.uclinux.org > Cc: linux-m32r@ml.linux-m32r.org > Cc: linux-mips@linux-mips.org > Cc: linuxppc-dev@lists.ozlabs.org > Cc: linux-sh@vger.kernel.org > Cc: sparclinux@vger.kernel.org > Cc: Chris Metcalf > Cc: user-mode-linux-devel@lists.sourceforge.net > Cc: x86@kernel.org > Signed-off-by: Stephen Boyd > --- Acked-by: Richard Weinberger -- Thanks, //richard From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qw0-f51.google.com (mail-qw0-f51.google.com [209.85.216.51]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority" (verified OK)) by ozlabs.org (Postfix) with ESMTPS id 6D7C8100811 for ; Sat, 7 May 2011 19:34:17 +1000 (EST) Received: by qwf7 with SMTP id 7so2564793qwf.38 for ; Sat, 07 May 2011 02:34:13 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <1304747831-2098-1-git-send-email-sboyd@codeaurora.org> References: <1304747831-2098-1-git-send-email-sboyd@codeaurora.org> Date: Sat, 7 May 2011 11:34:13 +0200 Message-ID: Subject: Re: [PATCH] lib: Consolidate DEBUG_STACK_USAGE option From: richard -rw- weinberger To: Stephen Boyd Content-Type: text/plain; charset=ISO-8859-1 Cc: linux-arch@vger.kernel.org, linux-mips@linux-mips.org, linux-m32r@ml.linux-m32r.org, user-mode-linux-devel@lists.sourceforge.net, linux-sh@vger.kernel.org, x86@kernel.org, linux-kernel@vger.kernel.org, Chris Metcalf , sparclinux@vger.kernel.org, uclinux-dist-devel@blackfin.uclinux.org, Andrew Morton , linuxppc-dev@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Sat, May 7, 2011 at 7:57 AM, Stephen Boyd wrote: > Most arches define CONFIG_DEBUG_STACK_USAGE exactly the same way. > Move it to lib/Kconfig.debug so each arch doesn't have to define > it. This obviously makes the option generic, but that's fine > because the config is already used in generic code. > > It's not obvious to me that sysrq-P actually does anything > different with this option enabled, but I erred on the side of > caution by keeping the most inclusive wording. > > Cc: linux-arch@vger.kernel.org > Cc: linux-arm-kernel@lists.infradead.org > Cc: uclinux-dist-devel@blackfin.uclinux.org > Cc: linux-m32r@ml.linux-m32r.org > Cc: linux-mips@linux-mips.org > Cc: linuxppc-dev@lists.ozlabs.org > Cc: linux-sh@vger.kernel.org > Cc: sparclinux@vger.kernel.org > Cc: Chris Metcalf > Cc: user-mode-linux-devel@lists.sourceforge.net > Cc: x86@kernel.org > Signed-off-by: Stephen Boyd > --- Acked-by: Richard Weinberger -- Thanks, //richard From mboxrd@z Thu Jan 1 00:00:00 1970 From: richard.weinberger@gmail.com (richard -rw- weinberger) Date: Sat, 7 May 2011 11:34:13 +0200 Subject: [PATCH] lib: Consolidate DEBUG_STACK_USAGE option In-Reply-To: <1304747831-2098-1-git-send-email-sboyd@codeaurora.org> References: <1304747831-2098-1-git-send-email-sboyd@codeaurora.org> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Sat, May 7, 2011 at 7:57 AM, Stephen Boyd wrote: > Most arches define CONFIG_DEBUG_STACK_USAGE exactly the same way. > Move it to lib/Kconfig.debug so each arch doesn't have to define > it. This obviously makes the option generic, but that's fine > because the config is already used in generic code. > > It's not obvious to me that sysrq-P actually does anything > different with this option enabled, but I erred on the side of > caution by keeping the most inclusive wording. > > Cc: linux-arch at vger.kernel.org > Cc: linux-arm-kernel at lists.infradead.org > Cc: uclinux-dist-devel at blackfin.uclinux.org > Cc: linux-m32r at ml.linux-m32r.org > Cc: linux-mips at linux-mips.org > Cc: linuxppc-dev at lists.ozlabs.org > Cc: linux-sh at vger.kernel.org > Cc: sparclinux at vger.kernel.org > Cc: Chris Metcalf > Cc: user-mode-linux-devel at lists.sourceforge.net > Cc: x86 at kernel.org > Signed-off-by: Stephen Boyd > --- Acked-by: Richard Weinberger -- Thanks, //richard