Convert the old marker related code in Rust to use new profiler rust marker API
Categories
(Core :: Gecko Profiler, task, P3)
Tracking
()
People
(Reporter: canova, Assigned: canova)
References
Details
Attachments
(5 files)
We have some hacks around rust code to add some markers for a while. Since Bug 1654413, we have a canonical marker API for this instead. We should remove the old code and replace it with our new API instead.
Assignee | ||
Comment 1•4 years ago
|
||
Clone will make the ProfilerTime easier to use. And Add/Sub implementations are
helpful when you need to subtract/add some duration before adding a marker.
There is a similar code in the Webrender marker code, and this will allow them
to use the new API instead of some custom code.
Assignee | ||
Comment 2•4 years ago
|
||
Depends on D127111
Assignee | ||
Comment 3•4 years ago
|
||
Depends on D127112
Assignee | ||
Comment 4•4 years ago
|
||
Depends on D127113
Updated•4 years ago
|
Comment 7•4 years ago
|
||
Backed out 4 changesets (Bug 1733335) for causing build bustages.
Backout link
Push with failures - wrench
Failure Log
Assignee | ||
Comment 8•4 years ago
|
||
I delayed landing this for one day because this was touching the same places with Bug 1690619. It's merged to central now. I will land it after fixing the issues and rebasing on top of the central.
Assignee | ||
Comment 11•4 years ago
|
||
This was causing asan and dt failures because bindgen was computing the size of
MarkerSchema as 122 bytes instead of 144 bytes. This was causing a heap buffer
overflow. After making the std::vector an opaque type, it computes the size
properly as 144 bytes.
Depends on D127114
Assignee | ||
Comment 12•4 years ago
|
||
There was a problem with the MarkerSchema and bindgen wasn't computing its size because it has an std::vector
in it. After making the std::vector
an opaque type in the Rust side, the problem is resolved. This new patch should fix this.
Comment 13•4 years ago
|
||
Comment 14•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/be8872815105
https://hg.mozilla.org/mozilla-central/rev/93abc883209f
https://hg.mozilla.org/mozilla-central/rev/299a058089e7
https://hg.mozilla.org/mozilla-central/rev/8f6e6f2c2cbf
https://hg.mozilla.org/mozilla-central/rev/5640485486b0
Updated•4 years ago
|
Description
•