On Wed, Oct 19, 2022 at 03:53:22PM +0200, Ahmad Fatoum wrote: > The aarch64 cross-compiler built with Yocto's kirkstone release > configures -mbranch-protection=standard as default. > > The resulting PACIASP/AUTIASP instructions are interpreted as NOPs by > existing CPUs, but they increase image size and misalign the barebox > header in ENTRY_FUNCTION(). Also on ARMv8.3+, where they serve a > purpose, we wouldn't benefit from them as we don't setup any pointer > authentication keys. Thus disable it. > > The misaligned barebox header caused is_barebox_arm_head to fail, > breaking all code that verified barebox' header, like i.MX8MM > imx-usb-loader barebox-side support and barebox_update for > some ARM64 SoCs without a SoC-specific header. > > Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> > --- > This is a workaround. ENTRY_FUNCTION() should be rewritten for ARM64, > so we don't expect function prologue to be exactly two bytes long. > Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> > > v1 -> v2: > - revised commit message after understanding ramification better. > --- > Makefile | 5 +++++ > 1 file changed, 5 insertions(+) Applied, thanks Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
