From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933702AbYBNVAZ (ORCPT ); Thu, 14 Feb 2008 16:00:25 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932460AbYBNU7z (ORCPT ); Thu, 14 Feb 2008 15:59:55 -0500 Received: from pasmtpa.tele.dk ([80.160.77.114]:48043 "EHLO pasmtpA.tele.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932351AbYBNU7y (ORCPT ); Thu, 14 Feb 2008 15:59:54 -0500 Date: Thu, 14 Feb 2008 22:00:03 +0100 From: Sam Ravnborg To: Erez Zadok Cc: Andrew Morton , linux-kernel@vger.kernel.org Subject: Re: [PATCH] unhide CONFIG_DEBUG_SECTION_MISMATCH Message-ID: <20080214210003.GF21894@uranus.ravnborg.org> References: <200802142054.m1EKs49x023091@agora.fsl.cs.sunysb.edu> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200802142054.m1EKs49x023091@agora.fsl.cs.sunysb.edu> User-Agent: Mutt/1.4.2.1i Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Feb 14, 2008 at 03:54:04PM -0500, Erez Zadok wrote: > Using: v2.6.25-rc1-120-ge760e71 > > In a normal compilation, I might this message: > > ... > MODPOST vmlinux.o > WARNING: modpost: Found 4 section mismatch(es). > To see full details build your kernel with: > 'make CONFIG_DEBUG_SECTION_MISMATCH=y' > ... > > I can indeed try to re-make, passing CONFIG_DEBUG_SECTION_MISMATCH=y on the > command line, but I can't turn on the option in my .config. That's because > the option depends on "UNDEFINED". (Was that an attempt to "hide" the > option? Why?) The following small patch allows me to set the option in my > .config. It was done so on purpose. The rationale here is that we yet have too many section mismatch for a typical build to pass my "good stomach" test. So until we get down on an acceptable level we should not be too noisy. And I wanted it to be turned off also for allyesconfig builds. I hope to spend time on the reaming warnings soon but the option will not be enabled until latest next mergewindow. You could argue that I could have doen this in better ways The present version was just simple. Sam