Closed Bug 863453 Opened 11 years ago Closed 6 years ago

[meta] Support c++ unwinding in fennec nightly

Categories

(Core :: Gecko Profiler, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: BenWa, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: meta)

This bug tracks the work required to be able to install a fennec nightly APK in a locked phone and dump profiles that include full native C++ stack via native unwinding that reach back to XRE_Main the majority of the time.
Depends on: 856331
We can also use this to include C++ stacks in ANR reports.
I'd rather not make this increase the scope further then it already is. I hope to expose our unwinding features via bug 809317 which would provide the useful unwind possible. Can you file another bug to discuss that?
(In reply to Benoit Girard (:BenWa) from comment #2)
> I'd rather not make this increase the scope further then it already is. I
> hope to expose our unwinding features via bug 809317 which would provide the
> useful unwind possible. Can you file another bug to discuss that?

Yeah of course. Comment 1 was just a general comment about the usefulness of having unwinding in Fennec.
Depends on: 867999
Blocks: 1307215
Markus, is there anything left to be done in this bug? The dependencies are all fixed.
Flags: needinfo?(mstange)
I think this is working. Makoto, can you confirm?
Flags: needinfo?(mstange) → needinfo?(m_kato)
(In reply to Markus Stange [:mstange] from comment #5)
> I think this is working. Makoto, can you confirm?

Yes, arm's EHABI stack walker works well. so this has already fixed.
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(m_kato)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.