Carl-Daniel Hailfinger ade1cd1f11 The current K8 stack preservation code in disable_car() works by chance, ... but that's not something we should rely on. The new code is entirely rewritten, fixes a few missing constraints in the asm and should be a lot more readable. However, the generated code is NOT identical. The old code was broken because of the missing ecx clobber constraint and it did not copy the stack back (ecx was zero at the beginning of the copy-back loop and so the loop executed exactly zero times). So this is a genuine bug fix. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> Ron writes: wow! nice catch! Acked-by: Ronald G. Minnich <rminnich@gmail.com> We also need disable_car_and_halt, which only disables car and halts, for the APs (i.e. no need to copy stack back) git-svn-id: svn://coreboot.org/repository/coreboot-v3@858 f3766cd6-281f-0410-b1cd-43a5c92072e9		2008-09-05 10:29:33 +00:00
..
x86	The current K8 stack preservation code in disable_car() works by chance,	2008-09-05 10:29:33 +00:00