16359 2026-07-15 20:06:11 +0000 mpg123 recipe uses wrong --with-cpu in EXTRA_OECONF on aarch64 2026-08-20 13:02:36 +0000 1 1 7 Build System, Metadata & Runtime OE-Core multimedia 6.1 ARM Multiple RESOLVED FIXED Medium normal 6.1 1 ivan.nestlerode ivan.nestlerode mathieu.dubois-briand meta.mr.watcher meta.watcher randy.macleod ross.burton --- --- Don't know oldest_to_newest 106174 0 ivan.nestlerode 2026-07-15 20:06:11 +0000 On aarch64 the mpg123 recipe should use this option in EXTRA_OECONF: --with-cpu=neon64 but today it does not do so. Today, it does not pass any --with-cpu option at all in EXTRA_CONF on aarch64. This is probably due to confusion about TUNE_FEATURES and when it contains "neon". Even though the aarch64 CPU supports the NEON instructions, the Yocto aarch64 configurations do not set "neon" in TUNE_FEATURES. Recipes with configure steps that involve NEON basically have to special case aarch64 due to this. The libpng recipe had a very similar problem that was eventually fixed in commit 12e68d5824849fa20f0e3fe8fc1921da111bb6fb. The fix is a one-line change to the mpg123 recipe to look for aarch64 in TUNE_FEATURES to possibly set --with-cpu=neon64. I will attach the patch after this bug is opened. 106175 1 5234 ivan.nestlerode 2026-07-15 20:15:27 +0000 Created attachment 5234 patch to fix lack of NEON usage on aarch64 in mpg123 recipe 106184 2 ross.burton 2026-07-16 14:39:25 +0000 We don't take patches via the bugzilla, would you be able to post it to the mailing list? https://docs.yoctoproject.org/contributor-guide/submit-changes.html I don't think the new logic needs to be nested, instead just have an extra check: ${@bb.utils.contains('TUNE_FEATURES', 'neon', '--with-cpu=neon', '', d)} \ ${@bb.utils.contains('TUNE_FEATURES', 'aarch64', '--with-cpu=neon64', '', d)} \ That said I'm also wondering if aarch64 should just add neon to the tune features, because it _does_ have it. 106187 3 randy.macleod 2026-07-16 14:46:09 +0000 Ivan putting in needinfo and assigning to you. 106189 4 ross.burton 2026-07-16 14:58:46 +0000 Also looking at the configure script there are two relevant options: neon64 Use code optimized for AArch64 NEON SIMD engine aarch64 Pack neon64 and generic[[_dither]] decoders, for 64bit ARM processors Also a bit annoying that mpg123 makes this an option considering aarch64 mandates neon... 106199 5 ivan.nestlerode 2026-07-16 15:44:53 +0000 > That said I'm also wondering if aarch64 should just add neon to the tune features, > because it _does_ have it. I think that some people were opposed to doing this during the initial creation of aarch64 machine definitions: https://www.mail-archive.com/openembedded-core@lists.openembedded.org/msg78377.html Maybe what is needed is a more comprehensive recipe audit in openembedded-core and/or meta-openembedded for recipes that look for "neon" in TUNE_FEATURES to see if they correctly account for aarch64 or not. It is certainly a bit of a foot gun here for recipes that require custom configure options to turn on hand-written NEON assembly. I will dust off my b4 setup and email this patch to the mailing list (or a modified version thereof from the feedback here). Thanks. 106310 6 ivan.nestlerode 2026-07-23 15:19:35 +0000 I sent the patch to the mailing list: https://lists.openembedded.org/g/openembedded-core/message/241576 106583 7 mathieu.dubois-briand 2026-08-20 13:02:36 +0000 Patch merged: https://git.openembedded.org/openembedded-core/commit/?id=cec486b0f4c1853a97570b71baf69d575c68abc1 5234 2026-07-15 20:15:27 +0000 2026-07-15 20:15:27 +0000 patch to fix lack of NEON usage on aarch64 in mpg123 recipe 0001-mpg123-fix-lack-of-NEON-usage-on-aarch64.patch text/plain 1468 ivan.nestlerode RnJvbSBlYjMxNTU4MGY0M2IxYTc2NTdlYmNjMDQ4Mjk2YTAzMjVjNTVjYWNmIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBJdmFuIE5lc3RsZXJvZGUgPGl2YW4ubmVzdGxlcm9kZUBzb25v cy5jb20+CkRhdGU6IFdlZCwgMTUgSnVsIDIwMjYgMTY6MDc6NTggLTA0MDAKU3ViamVjdDogW1BB VENIXSBtcGcxMjM6IGZpeCBsYWNrIG9mIE5FT04gdXNhZ2Ugb24gYWFyY2g2NAoKRml4ZXMgW1lP Q1RPICMxNjM1OV0KCm1wZzEyMyBzdXBwb3J0cyBzcGVjaWZpYyAtLXdpdGgtY3B1PSBjb25maWd1 cmUgb3B0aW9ucyB0byB1c2UgTkVPTgppbnN0cnVjdGlvbnMsIGJ1dCB0aGlzIHJlY2lwZSB3YXMg bm90IHVzaW5nIGFueSBvZiB0aGVzZSBvbiBhYXJjaDY0LgpUaGUgcHJvYmxlbSBpcyB0aGF0IGl0 IHdhcyBvbmx5IGtleWluZyBvZmYgb2Ygd2hldGhlciBUVU5FX0ZFQVRVUkVTCmNvbnRhaW5zICJu ZW9uIi4gYWFyY2g2NCBuZXZlciBwdXRzICJuZW9uIiBpbnRvIFRVTkVfRkVBVFVSRVMgc28gaXQK aGFzIHRvIGJlIGNoZWNrZWQgc2VwYXJhdGVseS4KClVzZSAtLXdpdGgtY3B1PW5lb242NCBvbiBh YXJjaDY0LgotLS0KIG1ldGEvcmVjaXBlcy1tdWx0aW1lZGlhL21wZzEyMy9tcGcxMjNfMS4zMy42 LmJiIHwgMiArLQogMSBmaWxlIGNoYW5nZWQsIDEgaW5zZXJ0aW9uKCspLCAxIGRlbGV0aW9uKC0p CgpkaWZmIC0tZ2l0IGEvbWV0YS9yZWNpcGVzLW11bHRpbWVkaWEvbXBnMTIzL21wZzEyM18xLjMz LjYuYmIgYi9tZXRhL3JlY2lwZXMtbXVsdGltZWRpYS9tcGcxMjMvbXBnMTIzXzEuMzMuNi5iYgpp bmRleCBmMDdlNmRiY2QzLi43NGQ4YTIwMjczIDEwMDY0NAotLS0gYS9tZXRhL3JlY2lwZXMtbXVs dGltZWRpYS9tcGcxMjMvbXBnMTIzXzEuMzMuNi5iYgorKysgYi9tZXRhL3JlY2lwZXMtbXVsdGlt ZWRpYS9tcGcxMjMvbXBnMTIzXzEuMzMuNi5iYgpAQCAtNDEsNyArNDEsNyBAQCBFWFRSQV9PRUNP TkYgPSAiIFwKICAgICAtLWVuYWJsZS1zaGFyZWQgXAogICAgIC0tZW5hYmxlLWxhcmdlZmlsZSBc CiAgICAgLS13aXRoLWF1ZGlvPScke0FVRElPTU9EU30nIFwKLSAgICAke0BiYi51dGlscy5jb250 YWlucygnVFVORV9GRUFUVVJFUycsICduZW9uJywgJy0td2l0aC1jcHU9bmVvbicsICcnLCBkKX0g XAorICAgICR7QGJiLnV0aWxzLmNvbnRhaW5zKCdUVU5FX0ZFQVRVUkVTJywgJ2FhcmNoNjQnLCAn LS13aXRoLWNwdT1uZW9uNjQnLCBiYi51dGlscy5jb250YWlucygnVFVORV9GRUFUVVJFUycsICdu ZW9uJywgJy0td2l0aC1jcHU9bmVvbicsICcnLCBkKSwgZCl9IFwKICAgICAke0BiYi51dGlscy5j b250YWlucygnVFVORV9GRUFUVVJFUycsICdhbHRpdmVjJywgJy0td2l0aC1jcHU9YWx0aXZlYycs ICcnLCBkKX0gXAogICAgICR7QGJiLnV0aWxzLmNvbnRhaW5zKCdUQVJHRVRfRlBVJywgJ3NvZnQn LCAnLS13aXRoLWNwdT1nZW5lcmljX25vZnB1JywgJycsIGQpfSBcCiAiCg==