Closed Bug 604527 Opened 15 years ago Closed 6 years ago

Automatic test for wakeups

Categories

(Firefox for Android Graveyard :: General, defect)

All
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: azakai, Unassigned)

References

Details

Attachments

(1 file)

Attached patch first stabSplinter Review
Attached is a first stab at an automatic test for wakeups. Instructions for running it are in the patch. Basically, it runs Fennec, waits, then runs powertop and parses the # of wakeups, then checks that is in the right range. Test only works on Linux. Might also work on Android/Meego devices but not sure. However, testing on desktop Linux is very useful in itself (the # of wakeups is basically the same as on devices). Need to think how to integrate such a test into our normal automatic test frameworks, etc.
Flags: in-testsuite?
Blocks: 446418
Forgive my ignorance, but what are you trying to measure and what is going to need to be installed on each device to make this work? Is this something that needs to be measured during every test or only during a specific "wakeup measurement" - perhaps a Talos style test?
The test measures CPU wakeups - how many times the CPU goes from idle to doing something. Ideally we should have 0, when the browser is idle. But at least we should not regress on this. The way to measure those is with 'powertop'. On ubuntu, |apt-get install powertop| gets it, it should be easily available elsewhere as well. (Note that powertop needs to be run with root privileges - 'sudo powertop' or such, which the test mentions.) I think this should run with every commit, and not like Talos.
We need this part of the automated tests ran against the nightlies. in the meantime, we can probably add a litmus test and have the tester run the python script on their device each time a nightly build is installed.
Flags: in-litmus?(martijn.martijn)
Flags: in-litmus?(martijn.martijn)
Closing all opened bug in a graveyard component
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: