Power Policy Manager

Add error logging to CPU plugin

Bug #210553 reported by Tony Espy
4
Affects Status Importance Assigned to Milestone
Power Policy Manager
Triaged
Low
mabbas

Bug Description

While doing ppm testing on Hardy, I noticed that the CPU plugin silently continues if the sysfs cpufreq 'scaling_governor' interface doesn't exist. As I was testing on a Crown Beach with a C0 board, this interface didn't exist. This type of logging should be standard practice.

Tags: intel pending ume
Zhao Fan (fan-zhao)
Changed in ppm:
assignee: nobody → mohamed-abbas
importance: Undecided → Wishlist
Revision history for this message
mabbas (mohamed-abbas) wrote :

we can add some error log but we dont want to flood the system with error log especially ppm is genaric utility that can run on different platform that some wont support all ppm plugins

Revision history for this message
Tony Espy (awe) wrote :

Typically the way to do this is check the first time the plugin is called...if the interface to be used doesn't exist, then log the error once, set a flag and remain silent thereafter...

mabbas (mohamed-abbas)
Changed in ppm:
importance: Wishlist → Low
milestone: none → m11
status: New → Triaged
Revision history for this message
May Xie (may-xie) wrote :

ppm is not used, remove this bug from m11

Changed in ppm:
milestone: m11 → none
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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