Re: [PATCH] ARM: move body of head-common.S back to text section

2013-07-05 Thread Dave P Martin
On Wed, Jul 03, 2013 at 08:22:35PM -0400, Paul Gortmaker wrote: > [Re: [PATCH] ARM: move body of head-common.S back to text section] On > 03/07/2013 (Wed 18:20) Russell King - ARM Linux wrote: > > > On Wed, Jul 03, 2013 at 11:30:12AM -0400, Paul Gortmaker wrote: > > > [Re

Re: [PATCH] ARM: move body of head-common.S back to text section

2013-07-03 Thread Paul Gortmaker
[Re: [PATCH] ARM: move body of head-common.S back to text section] On 03/07/2013 (Wed 18:20) Russell King - ARM Linux wrote: > On Wed, Jul 03, 2013 at 11:30:12AM -0400, Paul Gortmaker wrote: > > [Re: [PATCH] ARM: move body of head-common.S back to text section] On > > 03/07/

Re: [PATCH] ARM: move body of head-common.S back to text section

2013-07-03 Thread Russell King - ARM Linux
On Wed, Jul 03, 2013 at 11:30:12AM -0400, Paul Gortmaker wrote: > [Re: [PATCH] ARM: move body of head-common.S back to text section] On > 03/07/2013 (Wed 11:00) Russell King - ARM Linux wrote: > > > On Wed, Jul 03, 2013 at 01:19:07AM -0400, Paul Gortmaker wrote: > > &

Re: [PATCH] ARM: move body of head-common.S back to text section

2013-07-03 Thread Paul Gortmaker
[Re: [PATCH] ARM: move body of head-common.S back to text section] On 03/07/2013 (Wed 11:00) Russell King - ARM Linux wrote: > On Wed, Jul 03, 2013 at 01:19:07AM -0400, Paul Gortmaker wrote: > > As an aside, I'm now thinking any __INIT that implicitly rely on EOF for > > clo

Re: [PATCH] ARM: move body of head-common.S back to text section

2013-07-03 Thread Russell King - ARM Linux
On Wed, Jul 03, 2013 at 01:19:07AM -0400, Paul Gortmaker wrote: > As an aside, I'm now thinking any __INIT that implicitly rely on EOF for > closure are nasty traps waiting to happen and it might be worthwhile to > audit and explicitly __FINIT them before someone appends to the file... That hides

Re: [PATCH] ARM: move body of head-common.S back to text section

2013-07-02 Thread Paul Gortmaker
[Re: [PATCH] ARM: move body of head-common.S back to text section] On 02/07/2013 (Tue 20:44) Stephen Warren wrote: > On 07/02/2013 05:22 PM, Stephen Boyd wrote: > > On 07/02, Stephen Warren wrote: > >> From: Stephen Warren > >> > >> Commit 281ecb7 "arm:

Re: [PATCH] ARM: move body of head-common.S back to text section

2013-07-02 Thread Stephen Warren
On 07/02/2013 05:22 PM, Stephen Boyd wrote: > On 07/02, Stephen Warren wrote: >> From: Stephen Warren >> >> Commit 281ecb7 "arm: delete __cpuinit/__CPUINIT usage from all ARM >> users" removed a __CPUINIT from head-common.S. However, the code >> immediately before the removed tag was marked __INIT

Re: [PATCH] ARM: move body of head-common.S back to text section

2013-07-02 Thread Stephen Boyd
On 07/02, Stephen Warren wrote: > From: Stephen Warren > > Commit 281ecb7 "arm: delete __cpuinit/__CPUINIT usage from all ARM > users" removed a __CPUINIT from head-common.S. However, the code > immediately before the removed tag was marked __INIT, and was missing > a match __FINIT. This caused c

[PATCH] ARM: move body of head-common.S back to text section

2013-07-02 Thread Stephen Warren
From: Stephen Warren Commit 281ecb7 "arm: delete __cpuinit/__CPUINIT usage from all ARM users" removed a __CPUINIT from head-common.S. However, the code immediately before the removed tag was marked __INIT, and was missing a match __FINIT. This caused code after the removed __CPUINIT to end up in