- Jul 13, 2020
-
-
Treehugger Robot authored
Do not run ethernet tethering tests if tethering is not supported because Tethering APIs would fail. Also skip the test if EthernetManager is not avaliable. EthernetManager would be null if there are no FEATURE_ETHERNET and FEATURE_USB_HOST. Bug: 159869957 Test: atest CtsTetheringTest Merged-In: I2b9be6799c6edeefc8cd74897a8704dbe80dd061 Change-Id: I2b9be6799c6edeefc8cd74897a8704dbe80dd061
-
- Jul 08, 2020
-
-
Treehugger Robot authored
If the test run adb over network and ethernet is available, it is likely that adb may run over ethernet. Then the test would fail because adb would break when ethernet is switching from client mode to server mode. Bug: 160389275 Test: atest CtsTetheringTest Merged-In: I57d365d33316881c50c3bf7fd1c98926d10842d3 Change-Id: I57d365d33316881c50c3bf7fd1c98926d10842d3
-
- Jul 01, 2020
-
-
markchien authored
LocalLog is in android.util* instead of android.net*. No crash happened because the methods used by tethering are @UnsupportedAppUsage. Bug: 160113128 Test: atest TetheringTests Change-Id: I5f070b96f96aaabf7ec8da52a3d3444ed940fb56
-
- Jun 24, 2020
-
-
TreeHugger Robot authored
-
Lorenzo Colitti authored
Send netlink request over fd for offload config before completing init sequence. Provides existing conntrack entries to IPA. Resolves issue where there are NAT misses in IPA due to IPA only having the conntrack entries added after tethering starts. Bug: 149109043 Test: OffloadHardwareInterfaceTest Original-Change: https://android-review.googlesource.com/1290954 Merged-In: Iaf3e847e92f205b55f10fa85c17b9f3995d52099 Change-Id: Iaf3e847e92f205b55f10fa85c17b9f3995d52099
-
- Jun 22, 2020
-
-
Dedy Lansky authored
This change is a combination of following changes: 1) Tethering: add TETHERING_WIGIG type Currently both WIFI and WIGIG use the same tethering type, TETHERING_WIFI. This causes conflicts between the frameworks, when both WIFI and WIGIG SoftAPs are started, one or both will not work. Fix this by using a seperate tethering type for WIGIG. 2) Tethering: remove TETHERING_WIGIG state machine on interface down The wigig state machine relies on a TETHERING_STATE_CHANGED broadcast that is sent when the tethering state machine is first created, during interface up. Currently the tethering state machine is not removed on interface down except for TETHERING_BLUETOOTH, and as a result wigig tethering only works the first time SoftAP is started. In order to fix this, remove the tethering state machine on interface down for TETHERING_WIGIG as well. Bug: 143356416 Test: TetheringCoverageTests Change-Id: Ic4d3aca0ed69234093af7f0206dab3335938c52a Merged-In: Ic4d3aca0ed69234093af7f0206dab3335938c52a
-
waynema authored
Tethering package. Tethering needs StatsEvent and StatsLog which are in framework-statsd.stubs.module_lib to write metrics. Bug: 153942334 Test: make Change-Id: I8ba142a9b3678de36e6c4880a64e992e00680f38
-
- Jun 19, 2020
-
-
Mark Chien authored
When LAN link is changed for using ethernet interface from Ethernet tethering to Ethernet, there is confiict as Ethernet Tethering is trying to make ethernet link down and Ethernet is trying to make ethernet link up. So, this would make Ethernet only manage ethernet interface link state. Test: manual Bug: 130840861 Merged-In: I1cd40ae764bdeecbb59e3889e9399b7f4b05f9cc Change-Id: I1cd40ae764bdeecbb59e3889e9399b7f4b05f9cc
-
TreeHugger Robot authored
* changes: Test that IpNeighborMonitor is stopped when IpServer stops. Memory leak due to no stop for IpNeighborMonitor.
-
Treehugger Robot authored
Bug: 158961959 Test: build/make/tools/checkowners.py packages/NetworkStack/OWNERS Test: ./build/make/tools/checkowners.py \ frameworks/base/packages/Tethering/OWNERS Merged-In: I13b291e5db6a8a8c9f2bfa477acabaea250aa48c Change-Id: I13b291e5db6a8a8c9f2bfa477acabaea250aa48c
-
Treehugger Robot authored
Bug: 159097215 Test: test-only change Original-Change: https://android-review.googlesource.com/1343441 Merged-In: I2292c1cbff06a304f70191b88d833b19af2b8b92 Change-Id: I2292c1cbff06a304f70191b88d833b19af2b8b92
-
Lorenzo Colitti authored
1. When Wi-Fi enabled, ap0 interface is added and IpNeighborMonitor's start() is invoked in IpServer's constructor. 2. There's no stop for IpNeighborMonitor when IpServer stop. 3. During overnight test for Wi-Fi, networkstack is too slow to process request due to memory leak in IpNeighborMonitor and binder buffer is blocked causing exception. Solution: Invoke IpNeighborMonitor.stop() in UnavailableState.enter(). Bug: 159097215 Test: automatic, turn on and off Wi-Fi every 6 seconds overnight Original-Change: https://android-review.googlesource.com/1343440 Merged-In: I8f60c13706f05306e8f25a15f7861d7ecabbc10e Change-Id: I8f60c13706f05306e8f25a15f7861d7ecabbc10e
-
Remi NGUYEN VAN authored
-
- Jun 18, 2020
-
-
Lorenzo Colitti authored
-
Luke Huang authored
This class might be used by some mainline modules. Bug: 151052811 Test: atest DnsPacketTest Test: atest DnsResolverTest Change-Id: I8841d91456952ded5efbf8ea221289aecc7746ad
-
- Jun 17, 2020
-
-
Nucca Chen authored
Bug: 150736748 Test: BpfCoordinatorTest Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1315576 Merged-In: I7f8dde3b57ee14eb33edbe2fd383df33cccc231c Change-Id: I70a6e5c8e765daf40cb738feb7fd70cf3c8f052d (cherry picked from commit bb618934)
-
Nucca Chen authored
Bug: 150736748 Test: BpfCoordinatorTest Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1313802 Merged-In: Iedb936b7592b6be773d1b84a2498bfc5a440a198 Change-Id: I298ae39a1fa61b2cf97752aa908aa2d7d0f9783d (cherry picked from commit 3aba923c)
-
Nucca Chen authored
Bug: 150736748 Test: BpfCoordinatorTest Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1313955 Merged-In: Id413b7f2f7edb2e5c3e02d5677fe536ed52fbbcb Change-Id: I48a8de478c9200b8c9f88e785340fc7973e4a13f (cherry picked from commit fe737bc2)
-
Nucca Chen authored
$ adb shell dumpsys tethering BPF offload: Polling started Stats provider registered Upstream quota: {rmnet_data2=9223372036854775807} Forwarding stats: 12(rmnet_data2) - ForwardedStats(rxb: 1065, rxp: 5, txb: 0, txp: 0) Forwarding rules: [wlan1]: iif(iface) oif(iface) v6addr srcmac dstmac 12(rmnet_data2) 31(wlan1) /2401:e180:8831:77ae:a900:a03b:41fb.. Bug: 150736748 Test: Enable tethering on mobile data and check dumpsys tethering Original-Change: https://android-review.googlesource.com/1302438 Merged-In: I95ea3050d92f3ba8136a63cd399d3450d183c8dc Change-Id: I95ea3050d92f3ba8136a63cd399d3450d183c8dc
-
Nucca Chen authored
The applying data limit is based on the forwarding rule changes. Add the tests for verifying their interactions with netd. Bug: 150736748 Test: BpfCoordinatorTest Original-Change: https://android-review.googlesource.com/1311659 Merged-In: I5a98c4cd74e2de6005ee05defa761f6af3fd4e75 Change-Id: I5a98c4cd74e2de6005ee05defa761f6af3fd4e75
-
Mark Chien authored
Merge "Revert "Revert "Pass entitlement configuration to Settings for entitlement check""" into rvc-dev
-
- Jun 16, 2020
-
-
markchien authored
There is a protection in Settings that Settings would gray out if tethering requires entitlement check but the entitlement app is invalid. Tethering resource is moved from framework to tethering module, so Settings can not fetch entitlement app name anymore. In this change, tethering module would check whether entitltement app package name is exsited if entitlement check is needed. Tethering would be not supported (Settings tethering option would be hidded) if entitlement app is not installed. After moving the protection into tethering module, TetherUtil#isProvisioningNeeded is no longer needed. Because The only use case is Settings wants to gray out tethering setting when entitltement check is needed but entitlement app is invalid. Bug: 146918263 Test: atest TetheringCoverageTests Change-Id: I9a5ff5dbc1db3f3be7fcd7146862a16b373507e6
-
- Jun 15, 2020
-
-
Mark Chien authored
Add commit message here for reference: Tethering resource configuration is move from framework to tethering module. The resource would not be accessible from outside of tethering module. List the replacements of framework resources usage and intent extra: 1. R.string.config_mobile_hotspot_provision_response --> android.net.extra.TETHER_PROVISIONING_RESPONSE. 2. R.string.config_mobile_hotspot_provision_app_no_ui --> android.net.extra.TETHER_UI_PROVISIONING_APP_NAME 3. R.array.config_mobile_hotspot_provision_app --> android.net.extra.TETHER_SILENT_PROVISIONING_ACTION Besides, the current active subId would put in android.net.extra.TETHER_SUBID Note: They are not APIs because of API freeze. Now both tethering module and Settings define these strings independently. Bug: 146918263 Test: atest TetherServiceTest atest TetherProvisioningActivityTest This reverts commit 99889031. Reason for revert: Resume the CL and put this CL with settings part in the same topic to avoid break. Change-Id: I114b4c258743661df51e5a969e150047a292e035
-
- Jun 12, 2020
-
-
Mark Chien authored
-
Mark Chien authored
Original CL has dependencies with unmerged settings change: https://googleplex-android-review.git.corp.google.com/c/platform/packages/apps/Settings/+/11524847 They should be in the same topic, revert it first. Will resume it and put the same with settings part CL. This reverts commit 217d7b01. Reason for revert: This break hotspot because it should merged with settings part together. Bug: 158836492 Change-Id: I94d3ee25168cfb3d125030654c4bb8ddd670abfc
-
Remi NGUYEN VAN authored
The classes should not be picked up from frameworks/base, as they are part of several mainline modules. Also refine comments in DhcpResults following feedback in previous change. Bug: 151052811 Test: m; manual: flashed, wifi and telephony working Test: atest NetworkStackCoverageTests Change-Id: I7074651c6a2a7a6b11bcf13cc4bb03833d7d655f
-
- Jun 11, 2020
-
-
TreeHugger Robot authored
-
markchien authored
Tethering resource configuration is move from framwork to tethering module. Since tethering resource would not be accessible from outside of tethering module, EntitlementManager would tell Settings the entitlement configuration via intent extra when run entitlement check. Bug: 146918263 Test: atest TetheringTests Change-Id: I6f23553bb1da5f0b767f920b32a86fafb9e00b9e
-
Mark Chien authored
-
- Jun 10, 2020
-
-
TreeHugger Robot authored
-
paulhu authored
Currently tethering notification is sending a pending intent for redirecting user to tether settings page. However, this intent is implicit that only create with Settings.ACTION_TETHER_SETTINGS. For security reasons, this intetnt should specify an explicit component to be delivered to. Thus, specify the settings package name to this intent. Test: atest TetheringTests Bug: 156353008 Change-Id: I49187aee8a004caa890e2a73c0a28d280215c7d4 Merged-In: I49187aee8a004caa890e2a73c0a28d280215c7d4
-
Treehugger Robot authored
The BPF tethering coordinator listens to the forwarding rule changes for updating data limit which is applied in the following conditions. - After adding the first rule on a given upstream, add data limit. - After removing the last rule on a given upstream, clear data limit. - The service applies a new data limit on current upstream. The reason for relying on rule changes is because the Tethering and IpServer objects have multi-internal state machines. It is hard to synchronize all of their states. Note that the data limit cleanup for stopping or switching upstream relies on offload rules are all removed as well. Bug: 150736748 Test: manual Original-Change: https://android-review.googlesource.com/1302436 Merged-In: I829d36339973f9473fe6b616c48aa288f18d1c46 Change-Id: I829d36339973f9473fe6b616c48aa288f18d1c46
-
Lorenzo Colitti authored
Bug: 150736748 Test: atest BpfCoordinatorTest Original-Change: https://android-review.googlesource.com/1311658 Merged-In: Ic1f37de75b064d7c8717e1b496e13174bb8693ec Change-Id: Ic1f37de75b064d7c8717e1b496e13174bb8693ec
-
Lorenzo Colitti authored
Invoke the existing polling thread to update data alert statistics. Once the data alert limit has reached, trigger the notification. Bug: 150736748 Test: BpfCoordinatorTest Original-Change: https://android-review.googlesource.com/1302435 Merged-In: Ibf25560ca2e9f003d8eba01361dc7d35ec1b1627 Change-Id: Ibf25560ca2e9f003d8eba01361dc7d35ec1b1627
-
Hungming Chen authored
This is a preparation for moving adding/removing forwarding rules from IpServer to BpfCoordinator. Bug: 150736748 Test: atest IpServerTest Original-Change: https://android-review.googlesource.com/1317234 Merged-In: I85316ef09ff3c9389ded11dcc384493d699da48e Change-Id: I85316ef09ff3c9389ded11dcc384493d699da48e
-
Hungming Chen authored
Verify that the coordinator could fetch tether stats from BPF maps and report the network stats to the service. Bug: 150736748 Test: atest BpfCoordinatorTest Original-Change: https://android-review.googlesource.com/1305574 Merged-In: Ib1756159a2047c5db7d31359b0f288f840bd1bb1 Change-Id: Ib1756159a2047c5db7d31359b0f288f840bd1bb1
-
Hungming Chen authored
Make BPF tethering offload coordinator, BpfCoordinator, registers a network stats provider, BpfTetherStatsProvider, and provide the tethering stats from the BPF map. Bug: 150736748 Test: new test BpfCoordinatorTest Original-Change: https://android-review.googlesource.com/1256189 Merged-In: I22e71f87b67668f7e733e4f215d93bf5b2c9380d Change-Id: I22e71f87b67668f7e733e4f215d93bf5b2c9380d
-
Hungming Chen authored
Used to record offload transmitted/received forwarded bytes/packets. Bug: 150736748 Test: new test BpfTetheringCoordinatorTest Original-Change: https://android-review.googlesource.com/1306257 Merged-In: Ie8725f95c3ddd5fb3811d479de32d2c1f7dcb493 Change-Id: Ie8725f95c3ddd5fb3811d479de32d2c1f7dcb493
-
Mark Chien authored
The TetheringTests are green in postsubmit for a while. Also add TetheringIntegrationTests to postsubmit. Bug: 148998835 Test: atest --test-mapping frameworks/base/packages/Tethering:postsubmit atest --test-mapping frameworks/base/packages/Tethering:presubmit Merged-In: I80d29d06fde5a718c31a0a5ad2e2562be5bf419f Change-Id: I80d29d06fde5a718c31a0a5ad2e2562be5bf419f
-
- Jun 09, 2020
-
-
Lorenzo Colitti authored
This is required to use the new BPF tethering APIs that set data usage limits. AOSP and master are already using -unstable instead of -V3. In order not to change master, the Merged-In tag is set to the CL that switched AOSP (and master) to -unstable. Test: m Bug: 150736748 Merged-In: I2d2cedf560319653f67f6b06f7abb0bf66eba91a Change-Id: Idba29b0fe428ac6552a025dcbe15729739088e9e
-