OSDN Git Service

Don't time out when fetching tether offload stats.
authorLorenzo Colitti <lorenzo@google.com>
Mon, 14 Aug 2017 04:07:42 +0000 (13:07 +0900)
committerLorenzo Colitti <lorenzo@google.com>
Tue, 15 Aug 2017 03:02:33 +0000 (12:02 +0900)
commitdca784af3c30bd077483d37001b8528b9d9ee490
treef903f89bd49c3314aa34ed0c70c5a80c989c4fcd
parent27ad3f1e48c615a9e47f6ce4ef4d1f5b79ac9e98
Don't time out when fetching tether offload stats.

Currently, fetching tethering offload stats has a 1000ms timeout,
after which we return stats of zero. However, returning zero is
invalid and will cause various parts of the network stats
accounting code to complain that statistics are moving backwards,
causing at least a log.wtf, and possibly a crash.

This CL removes the timeout entirely.

An alternative would have been to keep the timeout and return
null if the stats fetch failed. However, this complicates the
code, and if the HAL is persistently slow, could cause no stats
to be counted, ever. Given the impact of such behaviour on users'
data plans it is likely better to block until the stats are
collected.

(cherry picked from commit fef69a126793c83aa0d99a8c597037c3a29929c0)

Bug: 29337859
Bug: 32163131
Test: builds
Test: OffloadControllerTest passes
Change-Id: I1e017b6fef2d8a02a7edbf35bed3e4402f447ab9
services/core/java/com/android/server/connectivity/tethering/OffloadController.java