#2861 BLOC First D: Battery indicator not changing

[Zarro Boogs per Child]

#2861: Battery indicator not changing
------------------------+---------------------------------------------------
  Reporter:  crschmidt  |       Owner:  dilinger              
      Type:  defect     |      Status:  new                   
  Priority:  blocker    |   Milestone:  First Deployment, V1.0
 Component:  kernel     |     Version:  Build 542             
Resolution:             |    Keywords:                        
  Verified:  0          |  
------------------------+---------------------------------------------------

Comment(by dilinger):

 Replying to [comment:10 rsmith]:
 > Replying to [comment:9 kimquirk]:
 > > I have seen this as well. Richard said it is likely that the polling
 of battery life was turned off during suspend/resume testing so the
 inidicator on the home page doesn't get updated.
 > >
 >
 > I think the problem is that to prevent the laptop from waking up from
 resume every on 1% change notify we set the battery event mask so that
 battery % events are suppressed.  If sugar is depending on getting SCIs
 for battery % change then it would never update.
 > This is supported by the indicator updating properly when you plug up
 external power which will generate an event that is not masked.
 >
 > The fix is to set the 1% mask only when the system goes into suspend and
 then re-enable when we come back out of suspend.


 I don't think we want that yet; not until userspace is actually suspending
 the machine when idle.   Otherwise, we suspend manually and then unsuspend
 and never resuspend (wasting power).  We have similar problems currently
 with wireless packets waking the machine up as well.

 I don't think sugar should *require* battery events before updating the
 battery display.  It should be able to handle either case; updating via
 SCI, or setting a timer (say for every 10 mins) to poll battery status.
 Obviously, it needs to be done carefully so as to not waste power
 unnecessarily (ie, set the timer for 10 mins, and if a battery event
 happens, reset the timer so that it the timer only ever triggers when the
 machine is busy anyways).

-- 
Ticket URL: <https://dev.laptop.org/ticket/2861#comment:11>
One Laptop Per Child <https://dev.laptop.org>
OLPC bug tracking system