Re: [PATCH] kbuild: reenable section mismatch analysis

From: Sam Ravnborg
Date: Fri Mar 11 2011 - 00:47:31 EST


On Thu, Mar 10, 2011 at 04:41:00PM -0800, Andrew Morton wrote:
> On Fri, 18 Feb 2011 21:47:20 +0100
> Uwe Kleine-K__nig <u.kleine-koenig@xxxxxxxxxxxxxx> wrote:
>
> > This was disabled in commit
> >
> > e5f95c8 (kbuild: print only total number of section mismatces found)
> >
> > because there were too many warnings. Now we're down to a reasonable
> > number again, so we start scaring people with the details.
> >
> > Signed-off-by: Uwe Kleine-K__nig <u.kleine-koenig@xxxxxxxxxxxxxx>
> > ---
> > lib/Kconfig.debug | 4 ----
> > 1 files changed, 0 insertions(+), 4 deletions(-)
> >
> > diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> > index 2b97418..0f77ed1 100644
> > --- a/lib/Kconfig.debug
> > +++ b/lib/Kconfig.debug
> > @@ -102,11 +102,7 @@ config HEADERS_CHECK
> >
> > config DEBUG_SECTION_MISMATCH
> > bool "Enable full Section mismatch analysis"
> > - depends on UNDEFINED || (BLACKFIN)
> > default y
> > - # This option is on purpose disabled for now.
> > - # It will be enabled when we are down to a reasonable number
> > - # of section mismatch warnings (< 10 for an allyesconfig build)
> > help
> > The section mismatch analysis checks if there are illegal
> > references from one section to another section.
>
> eaaaargh! 650 lines of crap added to my i386 allmodconfig build output.
>
> I'll be locally opting out of this one, thanks.

How about that we enable this in the merge window.
And if it continue to be unreasonable then drop it again
around rc5 time?

The likelyhood that warnings are fixed is bigger if people
see them and get annoyed.

Sam
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/