From mboxrd@z Thu Jan  1 00:00:00 1970
From: James Bottomley <James.Bottomley@HansenPartnership.com>
Subject: Re: [PATCH 14/23] make section names compatible with
	-ffunction-sections -fdata-sections: parisc
Date: Sun, 06 Jul 2008 11:19:32 -0500
Message-ID: <1215361172.4197.21.camel@localhost.localdomain>
References: <200807020239.11410.vda.linux@googlemail.com>
	 <200807061746.02517.vda.linux@googlemail.com>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Return-path: <linux-embedded-owner@vger.kernel.org>
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=hansenpartnership.com;
	s=2007; t=1215361175; bh=W5ddtkB1T4Pgscbdrl0Ws2weWu/nn4d7SMqqr3QYwg
	w=; l=1277; h=Subject:From:To:Cc:In-Reply-To:References:
	 Content-Type:Date:Message-Id:Mime-Version:
	 Content-Transfer-Encoding; b=Qh3RpIiFiQKjTpmCCGhRyMe3+emGyCoHwOofT
	T32r6w1n/oE9o/LECPaKahlSZLCkLOQW374j6K+hAuBPkXJJyU3ajrnjfQnoxi9R5Aq
	9hraIrFwaJyyLNSn5ZbH1YgCAX/oqNyI3FhTrvoX8D41JTINVp7/RxPzlr0YTD4HGqI
	=
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=hansenpartnership.com;
	s=2007; t=1215361173; bh=W5ddtkB1T4Pgscbdrl0Ws2weWu/nn4d7SMqqr3QYwg
	w=; l=1277; h=Subject:From:To:Cc:In-Reply-To:References:
	 Content-Type:Date:Message-Id:Mime-Version:
	 Content-Transfer-Encoding; b=JZCz+EiB4L8uEad78pxDJT+RUx/tuTvn5olXs
	bvmRuTupTs7dgv0Lmjxd7gaSnxXV9iYj8gPLBlCVbC9akvdLOqlea8HK3BT9+ESSXVx
	k+LTQHSWBP5Y5Z2+UzDAdMV6GHU1Am6DdTtWPe6Tkm1RJ6nDqdu4162j5qYysoJls1I
	=
In-Reply-To: <200807061746.02517.vda.linux@googlemail.com>
Sender: linux-embedded-owner@vger.kernel.org
List-ID: <linux-embedded.vger.kernel.org>
Content-Type: text/plain; charset="us-ascii"
To: Denys Vlasenko <vda.linux@googlemail.com>
Cc: linux-arch@vger.kernel.org, Russell King <rmk@arm.linux.org.uk>, David Howells <dhowells@redhat.com>, Ralf Baechle <ralf@linux-mips.org>, Lennert Buytenhek <kernel@wantstofly.org>, Josh Boyer <jwboyer@linux.vnet.ibm.com>, Paul Mackerras <paulus@samba.org>, David Woodhouse <dwmw2@infradead.org>, Andi Kleen <andi@firstfloor.org>, torvalds@linux-foundation.org, akpm@linux-foundation.org, Paul Gortmaker <paul.gortmaker@windriver.com>, linux-embedded@vger.kernel.org, linux-kernel@vger.kernel.org, Tim Bird <tim.bird@am.sony.com>, Martin Schwidefsky <schwidefsky@de.ibm.com>, Dave Miller <davem@davemloft.net>

On Sun, 2008-07-06 at 17:46 +0200, Denys Vlasenko wrote:
> The purpose of this patch is to make kernel buildable
> with "gcc -ffunction-sections -fdata-sections".
> This patch fixes parisc architecture.
> 
> Updated version - does not try to rename sections
> which really are expected to be produced by gcc - .text.sys_exit etc.

Actually, I've thought about it some more and I don't think your
approach is quite right.  The way gcc works with -ffunction-sections
makes it clear that it expects the section namespace to be scoped left
to right. However, by using *.text and *.data, your patch is following a
reverse convention (putting your most global name to the right).  This
is setting us up for a clash with future global section names that our
toolchain might emit in future.

Since we have to have special handling for your reversed sections in
every vmlinux.lds, what about introducing our own namespace instead?  So
we reserve the global section specifier linux and then make everything a
namespace under this.  So our constructed text and data sections become

linux.text.*

and

linux.data.*

That way we could only ever get a clash if the toolchain decided to use
a section global name linux (which we can beat them up over).

James