>Trivial patch: align rse_clear_invalid to 32-byte boundary. Good for >a 9 cycle speed up. > > --david > >Signed-off-by: David Mosberger-Tang <[EMAIL PROTECTED]> > ># This is a BitKeeper generated diff -Nru style patch. ># ># ChangeSet ># 2005/01/18 17:38:56-08:00 [EMAIL PROTECTED] ># ia64: Align rse_clear_invalid to double-bundle boundary. >Improves (normal) ># getpid() from 284 to 275 cycles. ># ># arch/ia64/kernel/entry.S ># 2005/01/18 17:38:40-08:00 [EMAIL PROTECTED] +1 -0 ># (rse_clear_invalid): Align it to 32 bytes. ># >diff -Nru a/arch/ia64/kernel/entry.S b/arch/ia64/kernel/entry.S >--- a/arch/ia64/kernel/entry.S 2005-01-18 20:57:18 -08:00 >+++ b/arch/ia64/kernel/entry.S 2005-01-18 20:57:18 -08:00 >@@ -971,6 +971,7 @@ > shladd in0=loc1,3,r17 > mov in1=0 > ;; >+ .align 32 > rse_clear_invalid: > #ifdef CONFIG_ITANIUM > // cycle 0 >
I've had reports from Ken Chen and Bjorn Helgaas of illegal instruction traps in the kernel while booting. Backing out this patch makes the problem go away. Bjorn was using "defconfig". Not sure what Ken used. I haven't seen any such problems using tiger_defconfig, nor with defconfig on my tiger. -Tony - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
