linux-kernel - Re: [GIT] kbuild core updates for 2.6.39-rc1

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20110321101749.GA5228@sepie.suse.cz>
Date:	Mon, 21 Mar 2011 11:17:49 +0100
From:	Michal Marek <mmarek@...e.cz>
To:	Uwe Kleine-König 
	<u.kleine-koenig@...gutronix.de>
Cc:	Linus Torvalds <torvalds@...ux-foundation.org>, bp@...en8.de,
	dot@...at.at, mikew@...gle.com, randy.dunlap@...cle.com,
	linux-kbuild@...r.kernel.org, linux-kernel@...r.kernel.org,
	Mike Frysinger <vapier@...too.org>,
	Randy Dunlap <rdunlap@...otime.net>
Subject: Re: [GIT] kbuild core updates for 2.6.39-rc1

On Mon, Mar 21, 2011 at 08:47:48AM +0100, Uwe Kleine-König wrote:
> On Sun, Mar 20, 2011 at 05:53:24PM -0700, Linus Torvalds wrote:
> > On Sat, Mar 19, 2011 at 1:58 AM, Michal Marek <mmarek@...e.cz> wrote:
> > >
> > > * Section mismatch analysis enabled by default
> > >  - there are only a few warnings for defconfig builds,
> > >    all{yes,mod}config still has a lot of them. Depending on the general
> > >    perception of this change, we might disable it later in the rc phase
> > >    again.
> > 
> > No.
> > 
> > We don't enable stuff like this by default. Not when it actually
> > changes some really fundamental compiler flags, and thus the whole "we
> > can disable it late in -rc again" is a totally broken option, since if
> > we have any compiler issues, we'd want to know it early.

OK, point taken.


> Michal, do you want just remove the default y and squash it into the
> commit?

I'm about to add this to the kbuild branch. Linus, would you pull such
version?

From: Michal Marek <mmarek@...e.cz>
Subject: [PATCH] kbuild: Make DEBUG_SECTION_MISMATCH selectable, but not on by default

CONFIG_DEBUG_SECTION_MISMATCH has also runtime effects due to the
-fno-inline-functions-called-once compiler flag, so forcing it on
everyone is not a good idea.

Signed-off-by: Michal Marek <mmarek@...e.cz>

diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index 8b6a4f1..5a24805 100644
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -102,7 +102,6 @@ config HEADERS_CHECK
 
 config DEBUG_SECTION_MISMATCH
 	bool "Enable full Section mismatch analysis"
-	default y
 	help
 	  The section mismatch analysis checks if there are illegal
 	  references from one section to another section.

Michal
--
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/