1204787 - Add |mach power|

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[Nicholas Nethercote [inactive]]

I have a shell script that runs tools/power/rapl and powermetrics and extracts the most useful stat from them for Firefox, Safari and Chrome. It has become my starting point for evaluating power consumption on any new website. Example output can be seen in bug 1190719 comment 17.

I want to get this in the tree so other people can use it.

Comment 1

[Nicholas Nethercote [inactive]]

Attachment: Add |mach power|

There is a "njn" comment in there for something I didn't know how to do --
check if we're running on Mac OS X 10.10 or later.

Comment 2

[Mike Hommey [:glandium]]

Comment on attachment 8661088 [details] [diff] [review]
Add |mach power|

Review of attachment 8661088 [details] [diff] [review]:
-----------------------------------------------------------------

::: tools/power/mach_commands.py
@@ +23,5 @@
> +        MachCommandBase.__init__(self, context)
> +
> +    @Command('power', category='misc',
> +        # njn: want to check for Mac 10.10 or above here...
> +        #conditions=[...],

You want to give a function that checks StrictVersion(platform.mac_ver()[0]) >= StrictVersion('10.10'), where StrictVersion is imported from distutils.version.

@@ +102,5 @@
> +        # could do this by properly parsing powermetrics output, but it's
> +        # simpler and more robust to just grep for a handful of identifying
> +        # strings.
> +
> +        print()

Why the empty line?

Comment 3

[Nicholas Nethercote [inactive]]

> > +        print()
> 
> Why the empty line?

So there's a blank line between the rapl output and the powermetrics output. I'll add a comment.

Comment 4

[Nicholas Nethercote [inactive]]

Attachment: Add |mach power|

Now with a proper "OS X 10.10 or greater" condition.

Comment 5

[Mike Hommey [:glandium]]

Comment on attachment 8662153 [details] [diff] [review]
Add |mach power|

Review of attachment 8662153 [details] [diff] [review]:
-----------------------------------------------------------------

::: tools/power/mach_commands.py
@@ +18,5 @@
> +
> +
> +def is_osx_10_10_or_greater(cls):
> +    import platform
> +    mac_ver0 = platform.mac_ver()[0]

release would be a more explicit name than mac_ver0, and matches the definition of what mac_ver() returns.

@@ +110,5 @@
> +        # could do this by properly parsing powermetrics output, but it's
> +        # simpler and more robust to just grep for a handful of identifying
> +        # strings.
> +
> +        print()

You forgot comment 3.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c7cc8e32b62f

Comment 7

[Nicholas Nethercote [inactive]]

I documented this at https://developer.mozilla.org/en-US/docs/Mozilla/Performance/powermetrics#mach_power and added links to it from https://developer.mozilla.org/en-US/docs/Mozilla/Performance/Power_profiling_overview and https://developer.mozilla.org/en-US/docs/Mozilla/Performance/tools_power_rapl.

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c7cc8e32b62f