[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20090428215050.67b7b4db.akpm@linux-foundation.org>
Date: Tue, 28 Apr 2009 21:50:50 -0700
From: Andrew Morton <akpm@...ux-foundation.org>
To: Nathan Lynch <ntl@...ox.com>
Cc: Wu Fengguang <fengguang.wu@...el.com>,
"linux-kernel\@vger.kernel.org" <linux-kernel@...r.kernel.org>,
"kosaki.motohiro\@jp.fujitsu.com" <kosaki.motohiro@...fujitsu.com>,
"andi\@firstfloor.org" <andi@...stfloor.org>,
"mpm\@selenic.com" <mpm@...enic.com>,
"adobriyan\@gmail.com" <adobriyan@...il.com>,
"linux-mm\@kvack.org" <linux-mm@...ck.org>,
Stephen Rothwell <sfr@...b.auug.org.au>,
Chandra Seetharaman <sekharan@...ibm.com>,
Olof Johansson <olof@...om.net>,
Helge Deller <deller@...isc-linux.org>, linuxppc-dev@...abs.org
Subject: Re: [PATCH 5/5] proc: export more page flags in /proc/kpageflags
On Tue, 28 Apr 2009 23:41:52 -0500 Nathan Lynch <ntl@...ox.com> wrote:
> > CONFIG_DEBUG_KERNEL being enabled in distro kernels effectively means
> >
> > #ifdef CONFIG_DEBUG_KERNEL == #if 1
> >
> > as the following patch demos. Now it becomes obviously silly.
>
> Sure, #if 1 is usually silly. But if the point is that DEBUG_KERNEL is
> not supposed to directly affect code generation, then I see two options
> for powerpc:
>
> - remove the #ifdef CONFIG_DEBUG_KERNEL guards from
> arch/powerpc/kernel/sysfs.c, unconditionally enabling the hid/ima
> sysfs attributes, or
>
> - define a new config symbol which governs whether those attributes are
> enabled, and make it depend on DEBUG_KERNEL
yup.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists