1822827 - Startup failure: "Illegal instruction" on non-NEON arm-linux-gnueabihf

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Emanuele Rocca]

Steps to reproduce:

Firefox 110.0.1, as well as 102.9.0esr, fails to start on arm-linux-gnueabihf systems without NEON instructions.

To reproduce the issue, setup an armhf chroot as described on https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=982794#55 and try running:

$ QEMU_CPU=cortex-r5f firefox --version

Actual results:

(armhf)root@ariel:/home/ema# QEMU_CPU=cortex-r5f firefox --version
qemu: uncaught target signal 4 (Illegal instruction) - core dumped
Illegal instruction

Expected results:

(sid-armhf)root@ariel:/home/ema# QEMU_CPU=cortex-r5f firefox --version
Mozilla Firefox 110.0.1

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The Bugbug bot thinks this bug should belong to the 'Core::Widget: Gtk' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 2

[Robert Longson [:longsonr]]

NEON is now always used throughout the build. The groundwork for this was taken care of when we migrated all non-NEON ARM users to 52 indefinitely and not letting them progress to 53+. So that's why NEON is mandatory now.

Comment 3

[Emanuele Rocca]

Attachment: Bug 1822827 - Remove explicit NEON flags from skia build

Not all 32bit ARM CPUs have NEON instructions. Stop passing NEON_FLAGS.

See https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=982794

Comment 4

[Darkspirit]

Support for

• non-NEON ARM
• CPUs older than Pentium 4/AMD Opteron
• Windows XP
• Windows Vista
• 32 bit MacOS

ended with Firefox 52 and was removed with Firefox 53 (March/April 2017).

Comment 5

[Mike Hommey [:glandium]]

(In reply to Robert Longson [:longsonr] from comment #2)

NEON is now always used throughout the build. The groundwork for this was taken care of when we migrated all non-NEON ARM users to 52 indefinitely and not letting them progress to 53+. So that's why NEON is mandatory now.

That's not actually true.

(In reply to Darkspirit from comment #4)

Support for

• non-NEON ARM
• CPUs older than Pentium 4/AMD Opteron
• Windows XP
• Windows Vista
• 32 bit MacOS

ended with Firefox 52 and was removed with Firefox 53 (March/April 2017).

That's for the builds mozilla ships. We don't prevent downstreams from supporting those.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2e0f02d3955d
Remove explicit NEON flags from skia build r=glandium

Comment 7

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/2e0f02d3955d