On Mon, 2008-09-01 at 07:23 -0700, Arjan van de Ven wrote: > > well it's a pretty bad bug since there are many places that depend on > the kernel text stuff to work. > (backtracing, lockdep, some of the security checks etc). PPC needs to > be fixed rather urgently. > but yeah the system kept plodding on. > David also sent a patch to fix it fwiw. My patch broke allyesconfig though -- we need .text and .fixup to be close together, otherwise the branches back from fixup sections are too long to fit in a relative jump. Because TEXT_TEXT doesn't include .fixup, there was a _reason_ for not using TEXT_TEXT, and it's to ensure that the branches back to .text sections within the .fixup code are short enough. If they're too far away, then the final link fails. It might be sufficient to put the .fixup section immediately _before_ .text rather than a long way after it as my patch did. > I'm a bit sad to see a debug patch get reverted just because it caught > bugs ;-) > at least I assume this is very temporary. Yeah, I'm sure it will be. http://moss.amr.ith.intel.com/sites/Purchasing/CPC/EMEA_ApplyUpdateCard/EMEA_Apply_Update_Card.aspx -- David Woodhouse Open Source Technology Centre David.Woodhouse@xxxxxxxxx Intel Corporation -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
