[FFE]: powerstat, sync to 0.02.08

Bug #1550366 reported by Colin Ian King
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
powerstat (Ubuntu)
Fix Released
Medium
Colin Ian King

Bug Description

I was late in getting the powerstat features into Xenial and I'm requesting a Feature Freeze Exception. Apart from some minor code clean ups there are a few fixes and one new feature (GPU power consumption -g option) that this new version incorporates.

Changes (from change log)

Makefile: bump version
Move N/A message for GPU power right one char
   - fix to clean up output formatting
Add some more per function comments
Constify a few more func args
Make all non-main functions static
Minor fix up on GPU Watts field
  - formatting alignment fixes
Add GPU average stats
  - GPU power consumption statistics information at end of run
Clean up column formatting
  - fix formatting on various wide fields
Add -g GPU stats
  - new feature to gather i915 power consumption statistics, low impact change, does not regress other features
Update and correct copyright years
  - minor date changes

The code has been sanity checked through static analysis tools (cppcheck, CoverityScan, clang scan-build) as well as valgrind for memory leaks. It has been also tested on other architectures that don't have GPU statistic kernel interfaces to sanity check no regressions have occurred with the new GPU feature.

As the author and maintainer of this utility, so I'm confident this code is sane.

Changed in powerstat (Ubuntu):
status: New → In Progress
importance: Undecided → Medium
Revision history for this message
Colin Ian King (colin-king) wrote :

changelog diff

Revision history for this message
Colin Ian King (colin-king) wrote :

Attached is a pbuild build log file.

Revision history for this message
Colin Ian King (colin-king) wrote :

Example of package being installed:

king@lenovo:~/pbuilder/unstable_result$ sudo dpkg -i powerstat_0.02.08-1_amd64.deb
[sudo] password for king:
(Reading database ... 318477 files and directories currently installed.)
Preparing to unpack powerstat_0.02.08-1_amd64.deb ...
Unpacking powerstat (0.02.08-1) over (0.02.07-1) ...
Setting up powerstat (0.02.08-1) ...
Processing triggers for man-db (2.7.5-1) ...
king@lenovo:~/pbuilder/unstable_result$ echo $?
0

Revision history for this message
Colin Ian King (colin-king) wrote :

Examples of new feature (see attached log file)

?field.comment=Examples of new feature (see attached log file)

Changed in powerstat (Ubuntu):
assignee: nobody → Colin Ian King (colin-king)
Changed in powerstat (Ubuntu):
milestone: none → ubuntu-16.03
Changed in powerstat (Ubuntu):
status: In Progress → New
Revision history for this message
Iain Lane (laney) wrote :

Thanks, this seems fine - do it.

Changed in powerstat (Ubuntu):
status: New → Confirmed
Changed in powerstat (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.