Closed Bug 1774844 Opened 2 years ago Closed 2 years ago

Use the Windows Energy Meter Interface to record power use data in profiles

Categories

(Core :: Gecko Profiler, task, P3)

Product:
Core
Component:
Gecko Profiler
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
104 Branch
Tracking Flags:
Tracking Status
relnote-firefox --- 104+
firefox104 --- fixed

People

(Reporter: florian, Assigned: florian)

References

Details

Attachments

(5 files)

Bug 1774844 - record EMI data as profiler counters, r=gerald.
(deleted), text/x-phabricator-request
Details
Bug 1774844 - Add a new 'Power Use' (ie. 'power') profiler feature, r=gerald.
(deleted), text/x-phabricator-request
Details
Bug 1774844 - Record per process power use on Apple Silicon, r=gerald.
(deleted), text/x-phabricator-request
Details
Bug 1774844 - avoid failures of the test_ext_geckoProfiler_schema.js test when running with a conditioned profile,
(deleted), text/x-phabricator-request
Details
image.png
(deleted), image/png
Details

https://docs.microsoft.com/en-us/windows-hardware/drivers/powermeter/energy-meter-interface

On Windows 11 this reports RAPL data from the CPU (similar to what we were getting previously with Intel Power Gadget).

On Windows 10, this will only report something if the machine has hardware power meters (eg. the Surface Book 1 device).

Example profile: https://share.firefox.dev/3N0tcT5
There's no way to display this data yet, but profile.counters from the console shows that the counters exist, and have data.

Severity: -- → N/A
Priority: -- → P3
Attachment #9281851 - Attachment description: Bug 1774844 - WIP record EMI data as profiler counters, r=gerald. → Bug 1774844 - record EMI data as profiler counters, r=gerald.
Pushed by fqueze@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d98352922a5c Add a new 'Power Use' (ie. 'power') profiler feature, r=gerald. https://hg.mozilla.org/integration/autoland/rev/b4b82b6892b3 record EMI data as profiler counters, r=gerald. https://hg.mozilla.org/integration/autoland/rev/f33bef1f7d56 Record per process power use on Apple Silicon, r=gerald.
Regressions: 1777265
Attachment #9283457 - Attachment description: WIP: Bug 1774844 - avoid failures of the test_ext_geckoProfiler_schema.js test when running with a conditioned profile, → Bug 1774844 - avoid failures of the test_ext_geckoProfiler_schema.js test when running with a conditioned profile,
Pushed by fqueze@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/31f0ab2e90f8 Add a new 'Power Use' (ie. 'power') profiler feature, r=gerald. https://hg.mozilla.org/integration/autoland/rev/4c9f15465211 avoid failures of the test_ext_geckoProfiler_schema.js test when running with a conditioned profile, r=extension-reviewers,mixedpuppy https://hg.mozilla.org/integration/autoland/rev/185ab81e5f51 record EMI data as profiler counters, r=gerald. https://hg.mozilla.org/integration/autoland/rev/0839b2f7ae02 Record per process power use on Apple Silicon, r=gerald.
Flags: needinfo?(florian)
Blocks: 1778282
Depends on: 1778315

Release Note Request (optional, but appreciated)
[Why is this notable]: New power saving feature
[Affects Firefox for Android]: Not yet
[Suggested wording]: The Firefox profiler can analyze power usage of a website (Apple M1 and Windows 11 only)
[Links (documentation, blog post, etc)]: example: https://share.firefox.dev/3nNc2xST

I will add a screenshot in the next comment

relnote-firefox: --- → ?
Attached image image.png (deleted) —

thank you! This has been added to 104 relnotes. The image is pending PR https://github.com/mozilla/bedrock/pull/12042, but the above link (https://share.firefox.dev/3nNc2xST) throws a 404.

relnote-firefox: ? → 104+
Flags: needinfo?(sledru)

If you have the screenshot, this is good enough :)

Flags: needinfo?(sledru)
Depends on: 1841837
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: