Dear LinuxBIOS readers!
This is the automated build check service of LinuxBIOS.
The developer "mjones" checked in revision 3017 to the LinuxBIOS source repository and caused the following changes:
Change Log: Changed the stop_this_cpu() to just hlt. Removed local APIC INIT (don't worry the APIC and AP are still initialized).
The local APIC INIT seemed to be the incorrect thing to do to stop an AP. The Intel Multiprocessor specification indicated that a vector should be set and a START should happen following an INIT. Then AP will execute the instructions pointed to by the vector. There is no vector or start in stop_this_cpu(). This seems to put the AP in an in-between state. In the case of Barcelona the AP's MSRs and PCI register are not accessible by the hardware debugger.
The better solution seems to be to just put the AP in a hlt and allow the AP to go into C1. Then APIC managing software running on the BSP can program the AP as needed.
Signed-off-by: Marc Jones marc.jones@amd.com Reviewed-by: Jordan Crouse jordan.crouse@amd.com Acked-by: Ronald G. Minnich rminnich@gmail.com
Build Log: Configuration of amd:serengeti_cheetah_fam10 is still broken
If something broke during this checkin please be a pain in mjones's neck until the issue is fixed.
If this issue is not fixed within 24h the revision should be backed out.
Best regards, LinuxBIOS automatic build system