Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759370AbYGBBiZ (ORCPT ); Tue, 1 Jul 2008 21:38:25 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755201AbYGBBiR (ORCPT ); Tue, 1 Jul 2008 21:38:17 -0400 Received: from [194.117.236.238] ([194.117.236.238]:39819 "EHLO heracles.linux360.ro" rhost-flags-FAIL-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1756179AbYGBBiQ (ORCPT ); Tue, 1 Jul 2008 21:38:16 -0400 Date: Wed, 2 Jul 2008 04:28:06 +0300 From: Eduard - Gabriel Munteanu To: Denys Vlasenko Cc: linux-arch@vger.kernel.org, Andrew Morton , Russell King , David Howells , Ralf Baechle , Lennert Buytenhek , Josh Boyer , Paul Mackerras , David Woodhouse , Andi Kleen , torvalds@linux-foundation.org, Paul Gortmaker , linux-embedded@vger.kernel.org, linux-kernel@vger.kernel.org, Tim Bird , Martin Schwidefsky , Dave Miller Subject: Re: [PATCH 1/23] make section names compatible with -ffunction-sections -fdata-sections Message-ID: <20080702042806.4078bb09@linux360.ro> In-Reply-To: <200807020233.48646.vda.linux@googlemail.com> References: <200807020233.48646.vda.linux@googlemail.com> X-Mailer: Claws Mail 3.4.0 (GTK+ 2.12.1; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1433 Lines: 41 On Wed, 2 Jul 2008 02:33:48 +0200 Denys Vlasenko wrote: > Hi Andrew, folks, > > I am unsure how to synchronize propagation of these patches > across all architectures. > > Andrew, how this can be done without causing lots of pain > for arch maintainers? Please advise. Hi, AFAICS, there is a lot of code in .lds.S files which really is arch-independent, but still is duplicated in every arch. Kinda messy to change anything in there. I noticed this while writing another patch, namely early (pre-SMP) initcall support. Fortunately, there was a generic header included by all .lds.S files and I could fit my modification in there. My suggestion is (for both you and arch maintainers)... why not make an effort to reduce code duplication in these files? Life would be so much easier. The idea is: - Write a macro to define all generic sections, possibly taking in alignment as an argument. - Have each arch's .lds.S file define arch-dependent stuff and use that macro for generic sections. This would surely be immediately useful, more readily accepted by maintainers and would open up the way for a lighter version of your patch, IMO. Cheers, Eduard -- 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/