Re: Please revert edada399 and 9203fc9c

2009-04-28 Thread Sam Ravnborg
On Tue, Apr 28, 2009 at 02:56:33PM +1000, Paul Mackerras wrote: Linus, Please revert commits edada399 (powerpc: Use TEXT_TEXT macro in linker script.) and 9203fc9c (powerpc: Use __REF macro instead of old .text.init.refok.), which depends on edada399. Commit edada399 breaks the build

Re: Please revert edada399 and 9203fc9c

2009-04-28 Thread Tim Abbott
Please revert commits edada399 (powerpc: Use TEXT_TEXT macro in linker script.) and 9203fc9c (powerpc: Use __REF macro instead of old .text.init.refok.), which depends on edada399. Paul, Sorry for breaking the build. I will be sure to CC you and linuxppc-dev@ozlabs.org on future powerpc

Re: Please revert edada399 and 9203fc9c

2009-04-28 Thread Benjamin Herrenschmidt
On Tue, 2009-04-28 at 10:35 +0200, Sam Ravnborg wrote: This is partly my fault :-( I acked them as I did not see the issue with conditional branches. Could I ask you to add a comment to the lds file about this. I'm back :-) Not a huge deal per-se, it's just that the patches were merged in

Re: Please revert edada399 and 9203fc9c

2009-04-28 Thread Linus Torvalds
On Tue, 28 Apr 2009, Paul Mackerras wrote: And they are clearly not fixes for regressions or serious bugs, so why are these patches going in after -rc3 anyway? Yeah, my bad. They looked so obviously trivial, but obviously weren't. I'm not taking any more of them. Linus

Please revert edada399 and 9203fc9c

2009-04-27 Thread Paul Mackerras
Linus, Please revert commits edada399 (powerpc: Use TEXT_TEXT macro in linker script.) and 9203fc9c (powerpc: Use __REF macro instead of old .text.init.refok.), which depends on edada399. Commit edada399 breaks the build because it moves the __ftr_alt_* sections of a file away from the .text