Re: [PATCH 07/32] arm: delete __cpuinit/__CPUINIT usage from all ARM users

2013-07-02 Thread Paul Gortmaker
[Re: [PATCH 07/32] arm: delete __cpuinit/__CPUINIT usage from all ARM users] On 02/07/2013 (Tue 09:58) Russell King - ARM Linux wrote: > On Tue, Jul 02, 2013 at 04:52:00PM +0800, Joseph Lo wrote: > > I just tested this series on Tegra platform. It looks broken CPU hotplug > > fun

Re: [PATCH 07/32] arm: delete __cpuinit/__CPUINIT usage from all ARM users

2013-07-02 Thread Russell King - ARM Linux
On Tue, Jul 02, 2013 at 04:52:00PM +0800, Joseph Lo wrote: > I just tested this series on Tegra platform. It looks broken CPU hotplug > function for Tegra at least. The CPU can't plug-in after unplugging. And > the system resume function also not working when "enable_nonboot_cpus". > > Both of

Re: [PATCH 07/32] arm: delete __cpuinit/__CPUINIT usage from all ARM users

2013-07-02 Thread Joseph Lo
Adding linux-tegra in Cc. On Tue, 2013-06-25 at 03:30 +0800, Paul Gortmaker wrote: > The __cpuinit type of throwaway sections might have made sense > some time ago when RAM was more constrained, but now the savings > do not offset the cost and complications. For example, the fix in > commit

Re: [PATCH 07/32] arm: delete __cpuinit/__CPUINIT usage from all ARM users

2013-07-02 Thread Paul Gortmaker
[Re: [PATCH 07/32] arm: delete __cpuinit/__CPUINIT usage from all ARM users] On 02/07/2013 (Tue 09:58) Russell King - ARM Linux wrote: On Tue, Jul 02, 2013 at 04:52:00PM +0800, Joseph Lo wrote: I just tested this series on Tegra platform. It looks broken CPU hotplug function for Tegra

Re: [PATCH 07/32] arm: delete __cpuinit/__CPUINIT usage from all ARM users

2013-07-02 Thread Joseph Lo
Adding linux-tegra in Cc. On Tue, 2013-06-25 at 03:30 +0800, Paul Gortmaker wrote: The __cpuinit type of throwaway sections might have made sense some time ago when RAM was more constrained, but now the savings do not offset the cost and complications. For example, the fix in commit

Re: [PATCH 07/32] arm: delete __cpuinit/__CPUINIT usage from all ARM users

2013-07-02 Thread Russell King - ARM Linux
On Tue, Jul 02, 2013 at 04:52:00PM +0800, Joseph Lo wrote: I just tested this series on Tegra platform. It looks broken CPU hotplug function for Tegra at least. The CPU can't plug-in after unplugging. And the system resume function also not working when enable_nonboot_cpus. Both of the issue

[PATCH 07/32] arm: delete __cpuinit/__CPUINIT usage from all ARM users

2013-06-24 Thread Paul Gortmaker
The __cpuinit type of throwaway sections might have made sense some time ago when RAM was more constrained, but now the savings do not offset the cost and complications. For example, the fix in commit 5e427ec2d0 ("x86: Fix bit corruption at CPU resume time") is a good example of the nasty type of

[PATCH 07/32] arm: delete __cpuinit/__CPUINIT usage from all ARM users

2013-06-24 Thread Paul Gortmaker
The __cpuinit type of throwaway sections might have made sense some time ago when RAM was more constrained, but now the savings do not offset the cost and complications. For example, the fix in commit 5e427ec2d0 (x86: Fix bit corruption at CPU resume time) is a good example of the nasty type of