Closed
Bug 1898057
Opened 1 year ago
Closed 1 year ago
Record how much energy was used per process type
Categories
(Toolkit :: Performance Monitoring, task)
Tracking
()
RESOLVED
FIXED
128 Branch
People
(Reporter: florian, Assigned: florian)
References
Details
Attachments
(3 files)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
Bug 1898057 - actually record how much energy was used per process type on Apple Silicon, r=chutten.
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-release+
|
Details | Review |
On Apple Silicon, we can ask the kernel how much energy has been used by the current process. I think it would be interesting to send this data over Telemetry to be able to check how effective bug 1895985 is in the wild (we should see a significant decrease of power used by content.background processes after shipping that feature).
| Assignee | ||
Comment 1•1 year ago
|
||
Pushed by fqueze@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/268ac502e3b7
Record how much energy was used per process type on Apple Silicon, r=chutten.
Comment 3•1 year ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox128:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 128 Branch
| Assignee | ||
Comment 4•1 year ago
|
||
Pushed by fqueze@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/149f95e3b549
actually record how much energy was used per process type on Apple Silicon, r=chutten.
Comment 6•1 year ago
|
||
A patch has been attached on this bug, which was already closed. Filing a separate bug will ensure better tracking. If this was not by mistake and further action is needed, please alert the appropriate party. (Or: if the patch doesn't change behavior -- e.g. landing a test case, or fixing a typo -- then feel free to disregard this message)
Comment 7•1 year ago
|
||
| bugherder | ||
| Assignee | ||
Comment 8•1 year ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D211098
Updated•1 year ago
|
Attachment #9405256 -
Flags: approval-mozilla-beta?
Comment 9•1 year ago
|
||
beta Uplift Approval Request
- User impact if declined: None
- Code covered by automated testing: no
- Fix verified in Nightly: no
- Needs manual QE test: no
- Steps to reproduce for manual QE testing: N/A
- Risk associated with taking this patch: Low
- Explanation of risk level: The new code only exists on Apple Silicon, and only collects power use data to send it through Glean
- String changes made/needed: none
- Is Android affected?: no
Updated•1 year ago
|
Attachment #9405256 -
Flags: approval-mozilla-beta? → approval-mozilla-release?
Updated•1 year ago
|
Attachment #9405256 -
Flags: approval-mozilla-release? → approval-mozilla-release+
Updated•1 year ago
|
status-firefox127:
--- → fixed
Comment 10•1 year ago
|
||
| uplift | ||
You need to log in
before you can comment on or make changes to this bug.
Description
•