Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f1d860dd authored by Bookatz's avatar Bookatz Committed by Adam Bookatz
Browse files

BatteryStats: Fix BleScan apportioning 

The current BleScan apportioning formula has an error.
leftOverRxTimeMs is supposed to be constant at this point,
representing all the transfer time not already accounted
for by the scanning time. That figure is independent of which uids have
already been processed.
Note that updateWifiState has the same basic
formula (for Wifi accounting) but without this bug.

Bug: 113269219
Test: none
Change-Id: I600ac5e238580d951baa60ba02ed6d99ff48b6b1
parent bda1e07c

core/java/com/android/internal/os/BatteryStatsImpl.java

+0 −2
Original line number Diff line number Diff line
@@ -11725,7 +11725,6 @@ public class BatteryStatsImpl extends BatteryStats { 
                        Slog.d(TAG, "UID=" + uid + " rx_bytes=" + rxBytes + " rx_time=" + timeRxMs);

                    }

                    counter.getRxTimeCounter().addCountLocked(timeRxMs);

                    leftOverRxTimeMs -= timeRxMs;

                }















                if (totalTxBytes > 0 && txBytes > 0) {









@@ -11734,7 +11733,6 @@ public class BatteryStatsImpl extends BatteryStats {













                        Slog.d(TAG, "UID=" + uid + " tx_bytes=" + txBytes + " tx_time=" + timeTxMs);















                    }















                    counter.getTxTimeCounters()[0].addCountLocked(timeTxMs);













                    leftOverTxTimeMs -= timeTxMs;















                }















            }















        }