On Tue, 2012-03-06 at 03:59 +0100, Peter Stuge wrote: > Kyösti Mälkki wrote: > > commit bdab48d3f13d117bd1100be616837e6d1dbb55fc > > Author: Kyösti Mälkki <[email protected]> > > Date: Mon Mar 5 09:25:12 2012 +0200 > > > > Fix address of IDT in real-mode entry > > Was this tested not to cause regression? >
1. Tested as a fix to boot AP CPU on a hyper-threaded Intel P4 models f29 and f49. 2. Tested to cause no regression to boot a non-hyper-threaded Intel P4 CPU model f27 on the same platform as #1 testing. 3. Tested to cause no regression to boot AP CPUs on an Intel P4 Xeon platform with dual socket604 and hyper-threaded model f25 CPUs. The patch was not required for this platform to boot. I have no possibility to test other platforms, so any feedback with AMDs and VIAs is much appreciated. For these tests, I considered it as a "fix" or "no regression" if romstage executed with proper console output. Idwer Vollering proceeded with the tests #1 and #2 for me on a platform with no supported memory controller. Test #3 is the A-Open board [1]. Gerrit version is outdated, I have postponed updates for that until all the dependencies are merged. Thanks, KM [1] http://review.coreboot.org/#/c/303/ -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
