Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966218AbXBPI0d (ORCPT ); Fri, 16 Feb 2007 03:26:33 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S966220AbXBPI0c (ORCPT ); Fri, 16 Feb 2007 03:26:32 -0500 Received: from pasmtpb.tele.dk ([80.160.77.98]:38333 "EHLO pasmtpB.tele.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966216AbXBPI0c (ORCPT ); Fri, 16 Feb 2007 03:26:32 -0500 Date: Fri, 16 Feb 2007 09:26:48 +0100 From: Sam Ravnborg To: David Brownell Cc: Linux Kernel list , Andi Kleen , Andrew Morton , ebiederm@xmission.com, kai@germaschewski.name, Russell King , rusty@rustcorp.com.au Subject: Re: [patch 2.6.20-git] remove modpost false warnings on ARM Message-ID: <20070216082648.GA11771@uranus.ravnborg.org> References: <200701120831.37513.david-b@pacbell.net> <200702151910.46397.david-b@pacbell.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200702151910.46397.david-b@pacbell.net> User-Agent: Mutt/1.4.2.1i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1557 Lines: 33 On Thu, Feb 15, 2007 at 07:10:45PM -0800, David Brownell wrote: > This patch stops "modpost" from issuing erroneous modpost warnings on ARM > builds, which it's been doing since since maybe last summer. A canonical > example would be driver method table entries: > > WARNING: - Section mismatch: reference to .exit.text:_remove > from .data after '$d' (at offset 0x4) > > That "$d" symbol is generated by tools conformant with ARM ABI specs; in > this case it's a symbol **in the middle of** a "_driver" struct. > > The erroneous warnings appear to be issued because "modpost" whitelists > references from "_driver" data into init and exit sections ... but > doesn't know should also include those "$d" mapping symbols, which are > not otherwise associated with "_driver" symbols. > > This patch prevents the modpost symbol lookup code from ever returning > those mapping symbols, so it will return a whitelisted symbol instead. > Then things work as expected. > > Now to revert various code-bloating "fixes" that got merged because of > this modpost bug.... > > Signed-off-by: David Brownell Acked-by: Sam Ravnborg And if I get my dev machine operational before akpm merges this I will take care of having it merged. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/