[PATCH v2 9/9] x86: Add cleanup_before_linux()
Tom Rini
trini at konsulko.com
Tue Feb 24 17:04:37 CET 2026
On Tue, Feb 24, 2026 at 07:25:51AM -0700, Simon Glass wrote:
> This function is not present on x86, but there is an x86-specific
> version which is never called. This is a bug, meaning that APs are not
> parked before booting.
>
> The original weak function was added in 2012 by:
> commit f30fc4de4160 ("x86: Add a default implementation for cleanup_before_linux()")
>
> At later change made use of the weak function but was actually dead
> code. The Fixes tag reflects that.
>
> Add cleanup_before_linux(), making it call x86_cleanup_before_linux()
>
> Fixes: 99a573fb326f ("x86: mp: Park CPUs before running the OS")
> Signed-off-by: Simon Glass <simon.glass at canonical.com>
We're at over 5 years since 99a573fb326f and over a decade since
f30fc4de4160 and I don't think it was ever actually called. We should
just be dropping this, not suddenly calling it.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20260224/30b02d84/attachment.sig>
More information about the U-Boot
mailing list