1959478 - Investigate crashes on 16KB page size emulators

Status: NEW

(Firefox for Android :: General, defect)

Description

[Ryan VanderMeulen [:RyanVM]]

As far as I can tell, we've done all of the required work to support Android 16KB page sizes. We're building with NDK r28 (mozilla-central, app-services, and Glean) and AGP 8.9.1. We recently picked up the JNA 5.17.0 update which should have fully-operational 16KB page size support. I've manually looked at the .so files in the 64-bit APKs and they all show the expected page sizes. AGP 8.10 (currently in RC) also specifically added a new lint check for this and we pass (where we didn't prior to updating everything to NDK r28).

However, when I download the latest x86-64 nightly APK from archive.mozilla.org and try to install and run it in a 16KB page size test emulator in Android Studio, I'm greeted with an instant startup crash. So clearly something isn't working as expected yet.

This isn't actively blocking anything at the moment, but it seems very likely that Google Play will eventually require 16KB page size support for app submissions. It would be nice if we could sort out what's blocking this so we don't need a fire drill for it later.

Comment 1

[Ryan VanderMeulen [:RyanVM]]

I can sometimes get it to launch successfully, but it still crashes pretty quickly afterwards.

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Ted, this may be of interest to you too. I wonder if the issue lives somewhere in the JIT or something where maybe there's a baked-in assumption somewhere that Android == 4KB?

Comment 3

[Ted Campbell [:tcampbell]]

We went through the JIT work to support 16KB pages for Apple Silicon, but it is possible there is some missing configuration plumbing needed to also support Android. I'll take a look next week

Comment 4

[Ted Campbell [:tcampbell]]

Likely just need to change this code to also apply to Android aarch64, but I should double check a few things first.

Comment 5

[Jon Coppeard (:jonco)]

(In reply to Ted Campbell [:tcampbell] from comment #4)
It shouldn't crash if PageSize is wrong though, just disable decommit.

Comment 6

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

The Bugbug bot thinks this bug should belong to the 'Firefox for Android::Crash Reporting' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.