LineageOS/android_packages_modules_Connectivity
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge "[SP19] Reduce log when set limit and request stats update" into rvc-dev

Browse Source
This commit is contained in:
Lorenzo Colitti
2020-03-10 03:06:21 +00:00
committed by Android (Google) Code Review
parent 47f84e53b0 cee6cd88ee
commit 407e9eb150
1 changed files with 9 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
Tethering/src/com/android/server/connectivity/tethering/OffloadController.java
View File

@@ -288,10 +288,18 @@ public class OffloadController {
@Override @Override
public void setLimit(String iface, long quotaBytes) { public void setLimit(String iface, long quotaBytes) {
mLog.i("setLimit: " + iface + "," + quotaBytes);
// Listen for all iface is necessary since upstream might be changed after limit // Listen for all iface is necessary since upstream might be changed after limit
// is set. // is set.
mHandler.post(() -> { mHandler.post(() -> {
final Long curIfaceQuota = mInterfaceQuotas.get(iface);
// If the quota is set to unlimited, the value set to HAL is Long.MAX_VALUE,
// which is ~8.4 x 10^6 TiB, no one can actually reach it. Thus, it is not
// useful to set it multiple times.
// Otherwise, the quota needs to be updated to tell HAL to re-count from now even
// if the quota is the same as the existing one.
if (null == curIfaceQuota && QUOTA_UNLIMITED == quotaBytes) return;
if (quotaBytes == QUOTA_UNLIMITED) { if (quotaBytes == QUOTA_UNLIMITED) {
mInterfaceQuotas.remove(iface); mInterfaceQuotas.remove(iface);
} else { } else {
@@ -323,7 +331,6 @@ public class OffloadController {
@Override @Override
public void requestStatsUpdate(int token) { public void requestStatsUpdate(int token) {
mLog.i("requestStatsUpdate: " + token);
// Do not attempt to update stats by querying the offload HAL // Do not attempt to update stats by querying the offload HAL
// synchronously from a different thread than the Handler thread. http://b/64771555. // synchronously from a different thread than the Handler thread. http://b/64771555.
mHandler.post(() -> { mHandler.post(() -> {