AArch64 support for toolkit

RESOLVED INVALID

Status

()

Toolkit
Crash Reporting
RESOLVED INVALID
4 years ago
4 years ago

People

(Reporter: Marcin Juszkiewicz, Assigned: Marcin Juszkiewicz)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Created attachment 8364282 [details] [diff] [review]
aarch64-toolkit.patch

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:29.0) Gecko/20100101 Firefox/29.0 (Beta/Release)
Build ID: 20140122030521
(Assignee)

Updated

4 years ago
Blocks: 962534
(Assignee)

Updated

4 years ago
Attachment #8364282 - Flags: review?(ted)
Comment on attachment 8364282 [details] [diff] [review]
aarch64-toolkit.patch

Review of attachment 8364282 [details] [diff] [review]:
-----------------------------------------------------------------

So first things first, the Breakpad source is imported wholesale from an upstream Google code project, so fixes really need to go there first:
https://code.google.com/p/google-breakpad/

Second, this really isn't going to do anything. We shouldn't be building this code at all unless we have a full Breakpad port for Linux/AArch64. If we're building this code then you should be patching configure instead to stop building it in this configuration:
http://mxr.mozilla.org/mozilla-central/source/configure.in#5900

A Breakpad port would be useful if you're intending on shipping Firefox in this configuration, but it's probably not critical at this point. Breakpad does already have code in place to support arm64 on the backend (for newer iPhones):
https://code.google.com/p/google-breakpad/source/browse/trunk/src/google_breakpad/common/minidump_cpu_arm64.h

so I assume a Linux port wouldn't be a lot of work, mostly just fixing the client side to shuffle register state around as appropriate.
Attachment #8364282 - Flags: review?(ted) → review-
Assignee: nobody → marcin
(Assignee)

Comment 2

4 years ago
LG_QUANTUM part is in google-breakpad for quite long time:

commit daf6d0446ce64fb563b7d96fda077e6406c602be
Author: Riku Voipio <riku.voipio@linaro.org>
Date:   Mon Mar 18 16:40:20 2013 +0200

    Add aarch64 LG_QUANTUM size definition
    
    Signed-off-by: Riku Voipio <riku.voipio@linaro.org>

But you are right. I will drop that patch from my patchset.
I can't find that commit you're referencing, are you sure that's Breakpad?
(Assignee)

Comment 4

4 years ago
Oops, wrong tab. sorry - it was jemalloc.
(Assignee)

Comment 5

4 years ago
Closing for now - when Breakpad will be ported to aarch64-gnu-linux then proper request will be done.
Status: UNCONFIRMED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.