Closed
Bug 963034
Opened 10 years ago
Closed 10 years ago
AArch64 support for toolkit
Categories
(Toolkit :: Crash Reporting, defect)
Toolkit
Crash Reporting
Tracking
()
RESOLVED
INVALID
People
(Reporter: marcin, Assigned: marcin)
References
Details
Attachments
(1 file)
721 bytes,
patch
|
ted
:
review-
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:29.0) Gecko/20100101 Firefox/29.0 (Beta/Release) Build ID: 20140122030521
Assignee | ||
Updated•10 years ago
|
Attachment #8364282 -
Flags: review?(ted)
Comment 1•10 years ago
|
||
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-
Updated•10 years ago
|
Assignee: nobody → marcin
Assignee | ||
Comment 2•10 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.
Comment 3•10 years ago
|
||
I can't find that commit you're referencing, are you sure that's Breakpad?
Assignee | ||
Comment 4•10 years ago
|
||
Oops, wrong tab. sorry - it was jemalloc.
Assignee | ||
Comment 5•10 years ago
|
||
Closing for now - when Breakpad will be ported to aarch64-gnu-linux then proper request will be done.
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
Comment 6•10 years ago
|
||
FYI upstream is working on some AArch64 support: https://code.google.com/p/google-breakpad/source/detail?r=1301
You need to log in
before you can comment on or make changes to this bug.
Description
•