[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20070510124926.GB18534@linux-sh.org>
Date: Thu, 10 May 2007 21:49:26 +0900
From: Paul Mundt <lethal@...ux-sh.org>
To: Roland Dreier <rdreier@...co.com>
Cc: Roland Dreier <rolandd@...co.com>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] net: trivial MLX4_DEBUG dependency fix.
On Thu, May 10, 2007 at 04:42:37AM -0700, Roland Dreier wrote:
> > CONFIG_MLX4_DEBUG works out to a def_bool y for those that
> > have CONFIG_EMBEDDED set. Make it depend on MLX4_CORE..
>
> Thanks, applied... (by the way, this bug just results in an
> extraneous CONFIG variable being defined, right? There's no further
> breakage -- or am I misunderstanding the situation?)
>
Correct, there was no real breakage, just the extraneous variable.
SLUB_DEBUG had the same symptoms, which is why I referenced it ;-)
> > I'll let someone else wonder why debugging output is default enabled,
> > this seems to be a worrying trend as of late.
>
> Actually this option just controls whether to build the debugging
> output at all. The output is controlled at runtime with a module
> parameter (which can be changed via sysfs after the module is loaded),
> and the output defaults to off.
>
> The reason why I want building the debugging stuff to default to y is
> so that distros automatically build modules with debugging enabled.
> Otherwise it's a pain to try and gather info from someone who has
> problems with a kernel they didn't build.
>
That's more reasonable at least. Thanks for the clarification.
-
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