- Jun 02, 2020
-
-
Paul Duffin authored
Removes use of the special framework-modules naming scheme. Bug: 155164730 Test: m java Exempt-From-Owner-Approval: Build cleanup. Change-Id: I3896d7c91e937f503b49f5df011c21ee51219652 Merged-In: I0c31e2183353dfb5bd49f04f3455cb7b10be6866 Merged-In: I3b78fcbcacc3df787e171d6eedeef1e51b087615 (cherry picked from adac7d2c)
-
Paul Duffin authored
Switching from java_library to java_sdk_library switched the meaning of the module name from referring to the implementation library to referring to the stubs. This change updates the visibility rules to reflect that new meaning. Visibility rules that were previously set for the java_library have been moved to the impl_library_visibility property and the special //visibility:override value has been prepended to prevent it from inheriting the values from the visibility property. Visibility rules set for the stubs (via stubs_library_visibility) property have been moved to the visibility property. Bug: 155164730 Test: m nothing Exempt-From-Owner-Approval: Build cleanup Change-Id: Icc9bc5a9ef86cf7ba0f15c2b2a4abd596ec9f640 Merged-In: Icc9bc5a9ef86cf7ba0f15c2b2a4abd596ec9f640 (cherry picked from a17cf677)
-
Treehugger Robot authored
-
- Jun 01, 2020
-
-
Nucca Chen authored
* changes: Make tethering module to use netd_aidl_interface-unstable-java [BOT.5] Move class Ipv6ForwardingRule from IpServer to the coordinator [BOT.3] Add unit test for polling network stats in the coordinator [BOT.2] Create a coordinator and stats provider to provide tether stats [BOT.1] Add a class ForwardedStats in TetheringUtils
-
Paul Hu authored
-
- May 29, 2020
-
-
Hungming Chen authored
Use unstable aidl for new api tetherOffload* and modified parcel TetherStatsParcel in BpfCoordinator{, Test}. Bug: 150736748 Test: m Change-Id: I2d2cedf560319653f67f6b06f7abb0bf66eba91a
-
Hungming Chen authored
This is a preparation for moving adding/removing forwarding rules from IpServer to BpfCoordinator. Bug: 150736748 Test: atest IpServerTest 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 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 Change-Id: I22e71f87b67668f7e733e4f215d93bf5b2c9380d
-
Hungming Chen authored
Used to record offload transmitted/received forwarded bytes/packets. Bug: 150736748 Test: new test BpfTetheringCoordinatorTest Change-Id: Ie8725f95c3ddd5fb3811d479de32d2c1f7dcb493
-
markchien authored
Shell has TETHER_PRIVILEGED permission. To avoid any service to adopt shell identity by lunching service with Shell process, gate exemptFromEentitlementCheck by NETWORK_STACK. Bug: 157702014 Test: atest TetheringCoverageTests Change-Id: I6ddfda23d36ea9981e3e1eb5a87767f452a65852
-
Mark Chien authored
-
- May 28, 2020
-
-
paulhu authored
KClass#getSimpleName need refer to kotlin-reflect.jar which need include it in Andorid.bp. However, it's not necessary to use KClass#getSimpleName but use Class#getSimpleName instead. Test: atest TetheringTests Bug: 157527499 Change-Id: I49bc336a276d30152402eba926cc583bc81e8e5c Merged-In: I49bc336a276d30152402eba926cc583bc81e8e5c
-
Paul Duffin authored
-
Paul Duffin authored
The names of the individual modules do not quite follow the pattern that java_sdk_library uses so this temporarily sets the following: naming_scheme: "frameworks-modules" That causes java_sdk_library to use a naming scheme that matches the one used by the individual modules of this. It will be cleaned up later. Part of the purpose of the java_sdk_library is to hide the implementation code and force users of the library to depend on stubs for a well defined API. Ideally, it would allow access to the implementation in those cases where it is safe, e.g. from within the same APEX, or from tests for the implementation. Unfortunately, due to limitations in the build it does not yet have enough information to make that decision correctly which means that any code that needs to compile against the implementation is broken which would prevent us from converting the module to java_sdk_library. However, the only way to provide the additional information to allow the implementation to be correctly exposed is to convert the modules to java_sdk_library; a cycle. In order to break that cycle the java_sdk_library creates a special <module>.impl target which is used directly by tests and any other code that needs it. Once all the modules have been converted to a java_sdk_library then we can resolve the limitations in the build and remove the direct references to <module>.impl. Test: m update-api Bug: 155164730 Merged-In: If5c115f482751f9f4b5f047e9e401a18e36799ef Merged-In: Id1c2e848430c49a2da7402244814cd084f5da77c Change-Id: Id1c2e848430c49a2da7402244814cd084f5da77c
-
Treehugger Robot authored
-
markchien authored
Extend the timeout to lower the EthernetTetheringTest flaky. Besides, also explicitly exempt entitlement check in EtetherntTetheringTest because it do not test tethering upstream currently. Thus, the tests would not be interrupted by entitlement check if test SIM is entitlement required. Bug: 156713866 Test: TetheringCoverageTests, CtsTehteringTest Change-Id: I45e8e8d737486def9d0de8943ec7f09ca0942a0b
-
- May 27, 2020
-
-
markchien authored
- Add New class PrivateAddressCoordinator to coordinate the private address conflict problem. - Downstream prefix would be random in 192.168.0.0/24 ~ 192.168.255.0/24. - If new upstream prefix is conflict with existing downstream prefix, downstream would be kicked out and it would request a new one. - The last conflict upstream prefixes would be blacklist. Avoid to select downstream prefix which is conflict with prefixes in blacklist. Bug: 130879722 Test: -build, flash, boot -atest TetheringTests Change-Id: Ib45b87bcd9eeb5da03fb7ec90b1af9ca53998cf5
-
- May 26, 2020
-
-
markchien authored
Bug: 156866746 Test: atest TetheringTests, TetheringCoverageTests Change-Id: I7c539f1f4a447b5913164b222601c6113c6fe645
-
- May 25, 2020
-
-
Mark Chien authored
-
- May 24, 2020
-
-
markchien authored
If upstream is cellular, set the TTL in Router Advertisements to "network-set TTL - 1" for carrier requirement. For other non-cellular upstream, set TTL as "network-set TTL + 1" to preventing arbitrary distinction between tethered and untethered traffic. Bug: 154776299 Test: atest TetheringTests Change-Id: I7f2696a642f96c6aafb5613b980bf5bcdd08bbda
-
- May 20, 2020
-
-
markchien authored
AdoptShellPermissionIdentity can not pass permission check by Settings#checkAndNoteWriteSettingsOperation. It would compare the caller uid and its package name. See error below: 1. java.lang.SecurityException: Specified package com.android.shell under uid 10239 but it is really 2000 2. java.lang.SecurityException: uid 10245 does not have android.permission.UPDATE_APP_OPS_STATS. Override the method and test if caller hold WRITE_SETTINGS directly. Bug: 154869719 Test: TetheringTests, TetheringCoverageTests, NetworkStackNextTests, NetworkStackCoverageTests Change-Id: I2a60c4d66ef30028f9663159f85464ea815248e2
-
- May 19, 2020
-
-
Treehugger Robot authored
-
markchien authored
Bug: 154869719 Test: atest TetheringTests Change-Id: I7beea3f011d930e433443ed62d772a3f8cce5d78
-
- May 14, 2020
-
-
Treehugger Robot authored
-
markchien authored
Bug: 156557122 Test: atest TetheringTests Change-Id: Id9c0397306f3872fc23520d1354f338035a96dc9
-
Xiao Ma authored
-
- May 13, 2020
-
-
Treehugger Robot authored
-
Xiao Ma authored
Add the specific implementation of onNewPrefixRequest callback on IpServer side, also refactor some common code. Bug: 130741856 Test: atest TetheringTests Change-Id: If2871bf899cb5890bbfee18063a194c92b6f474e
-
markchien authored
1. Move isTetheringSupport logic from TetheringService to Tethering. 2. Small readability improvement in TetheringTest. Also change config_tether_upstream_automatic from false to true in TetheringTest. So TetheringTests would default run automatic select upstream flow instead of selecting by legacy perferred network type list. Bug: 153609486 Test: atest TetheringTest Change-Id: I5a82a6347f62d3a7031db5c56e8e0c8530dafd8f Merged-In: I5a82a6347f62d3a7031db5c56e8e0c8530dafd8f
-
Paul Duffin authored
-
Paul Duffin authored
Merge the otherwise unused tethering-aidl-interfaces into framework-tethering. This is in preparation for converting to use java_sdk_library. Bug: 155164730 Test: m droid Change-Id: I4583539d11ba69320aa5a0dfcfee072c81affac2
-
Mark Chien authored
-
markchien authored
Bug: 145490751 Test: atest TetheringTests Change-Id: Ia402a6caaa0dfaa05d25a02101c515bbd884d33f
-
- May 12, 2020
-
-
Hungming Chen authored
- Correct description and spelling in the code and xml files. - Add a TODO for refactoring the IpServer constructor. - Refine the if-statement for starting IP neighbor monitor. Test: atest IpServerTest Change-Id: If9c8bc6f785fa80575db56de4e223292e9807ace
-
Nucca Chen authored
* changes: Use device option to control BPF offload features Add tether BPF offload config to device config and resource
-
- May 11, 2020
-
-
Jooyung Han authored
Modules contributing mainline modules (APK/APEX) should set min_sdk_version as well as apex_available. InProcessTethering is a part of com.android.tethering.inprocess, therefore min_sdk_version is set. Bug: 145796956 Bug: 150999716 Test: m Change-Id: I655211e0b9aa4bd7cb3718b6f567f0aa1a0176a5
-
- May 08, 2020
-
-
Mark Chien authored
-
Treehugger Robot authored
-
markchien authored
To exempt from entitlement check, caller need to hold TETHER_PRIVILEGED permission. Bug: 141256482 Test: atest TetheringTests Change-Id: I2eb37f5e92f5f5150a7fb7c25b945e28704d27a0
-