Re: [Powertop] Multiple Batteries

[Sergey Senozhatsky <sergey.senozhatsky at gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2119 bytes --]

On (11/23/12 22:55), Justin Brown wrote:
>    Hello,
>    I'm using Powertop 2.1, and I've noticed a somewhat minor bug. (I couldn't
>    find a public bug tracker, but would be more than happy to move this
>    discussion to one if someone could send me a URL.)
>    I have two batteries in my laptop, but Powertop does not recognize both
>    when calculating battery life remaining. For example Gnome-Shell reports
>    that my batteries are at 66% and 100% (85% combined) giving me 10 hours
>    and 11 minutes of estimated time remaining. On the other hand, Powertop
>    reports just 3 hours 17 �minutes of time. Both programs are reporting the
>    proper discharge rate. It just seems like Powertop is only using my
>    smaller battery (the currently active one).
>    I'm not sure if this issue has been reported, or if the developers even
>    have equipment with which to test. I'm more than willing to provide any
>    sort of debugging output or testing as needed. I could also probably
>    create a patch, but that's likely to take a lot longer for me to get setup
>    with development on this project.�
>    It may be as simple as iterating over a directory in /sys/ or /proc to
>    gather stats on multiple batteries. An experienced dev might be able to
>    fix it quickly.
>    Cheers,
>    Justin


can you please test the following patch?


----

power_supply rate could be 0.00, so we need to take care in power_meter::time_left() 
while perform dev_capacity()/joules_consumed() == inf operation.


Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky(a)gmail.com>

---

diff --git a/src/measurement/measurement.cpp b/src/measurement/measurement.cpp
index 4a33db9..f65cb0c 100644
--- a/src/measurement/measurement.cpp
+++ b/src/measurement/measurement.cpp
@@ -65,12 +65,13 @@ double power_meter::time_left(void)
 	if (cap < 0.001)
 		return 0.0;
 
-	left = cap / rate;
+	if (rate < 0.001)
+		rate = 1.0;
 
+	left = cap / rate;
 	return left;
 }
 
-
 vector<class power_meter *> power_meters;
 
 void start_power_measurement(void)