- Jan 06, 2023
-
-
Patrick Rohr authored
A similar option already exists in packages/modules/Connectivity/Cronet/apex/Android.bp. This is in preparation of adding cronet to framework-tethering. It is possible that we only end up using one of the variables but better safe than sorry. This change will be blocked by the blocked automerger into tm-mainline-prod. If this continues to be a problem, we might have to play some tricks with automerger. Test: TH Change-Id: I40ba22aba041ec12ec2bf2881df14e6c0ee53b44
-
Patrick Rohr authored
* changes: gn2bp: Use startswith instead of regex gn2bp: Remove unused condition for src path filtering gn2bp: Use "out" prefix for src path filtering
-
Jean Chalard authored
-
Motomu Utsumi authored
Test: ./update_results.sh Change-Id: Ic85de603d6edb3565552731bac60a57961c8aecc
-
Motomu Utsumi authored
Test: ./update_results.sh Change-Id: I5731123496490759e5ec3e8f53dd19d8fe6554b4
-
Motomu Utsumi authored
instead of "out/test". Test: ./update_results.sh Change-Id: I28fd863742f2531bb942a15c636ca311ef45c307
-
- Jan 05, 2023
-
-
Patrick Rohr authored
-
Patrick Rohr authored
It's very basic for now. Test: run the script Change-Id: Ie848f8da6936f4300ba5ecdc041523440be25e8a
-
- Jan 04, 2023
-
-
Patrick Rohr authored
-
Maciej Żenczykowski authored
-
Maciej Żenczykowski authored
This is a repeat of: https://android-review.git.corp.google.com/c/platform/packages/modules/Connectivity/+/2266447 which was reverted in: https://android-review.git.corp.google.com/c/platform/packages/modules/Connectivity/+/2372509 This time with kver >= 4.14 protections of the bpf_skb_adjust_room() bpf helper which isn't present on 4.9 T devices. Original change comments: Tested manually on a flame device connected to an ipv6-only wifi network (GoogleGuest). On server: nc -4 -l -u -p 443 On client (phone): adb shell nc -4 -u my.server 443 On client (phone): adb shell tcpdump -l -ee -vv -s 1600 -i v4-wlan0 On client send something to server "Hi." On server send something to client "Hey!" You should see normal unfragmented IP packets. Then on server send something really long (I used 57 copies of the 26 letter English alphabet). This should be long enough that fragmentation is required. You should see tcpdump show 2 ipv4 fragments, and netcat show the packet being delivered correctly. (and previous versions of the code were buggy and were resulting in corrupt packets and things not working) Test: TreeHugger Signed-off-by:
Maciej Żenczykowski <maze@google.com> Change-Id: I6758e63d8133215edd26b4cd2d73a5b5f261ffd1
-
Maciej Żenczykowski authored
-
- Jan 03, 2023
-
-
Chidera Olibie authored
Instant apps cannot create sockets and the CTSTestServer used in all of cronet cts need to create sockets Test: m cts && cts-tradefed run cts -m CtsCronetTestCases Bug: 263472960 Change-Id: I4b929c2c339e23f01351d96930c5185b0b25d964
-
Hungming Chen authored
Respect the upstream interface mtu instead of always setting mtu 1500. Using upstream interface mtu is probably not the best solution for tether offload pmtu but it at least respects the upstream interface mtu which may be less than 1500. Test: manual test 1. Connect to carrier Taiwan Star. 2. Enable tethering and check the mtu 1434 in dumpsys. IPv4 Upstream: proto [inDstMac] iif(iface) src -> nat -> dst [outDstMac] pmtu age udp [02:10:45:32:ef:35] 54(54) 192.168.72.125:39034 -> 15(rmnet1) 100.83.189.11:39034 -> 142.251.43.14:443 [00:00:00:00:00:00] 1434 69657ms IPv4 Downstream: proto [inDstMac] iif(iface) src -> nat -> dst [outDstMac] pmtu age udp [00:00:00:00:00:00] 15(rmnet1) 142.251.43.14:443 -> 54(54) 100.83.189.11:39034 -> 192.168.72.125:39034 [9a:8a:4d:ec:a4:7c] 1434 69633ms Bug: 262860312 Test: atest BpfCoordinatorTest Change-Id: Ic6dcb95ed76d5306053e4645b6baebc67ff082cf
-
Maciej Żenczykowski authored
-
Patrick Rohr authored
Test: cd external/cronet && mm Change-Id: Idec19f1364c9dca4d278a33eebf500a4db87f389
-
Treehugger Robot authored
-
Patrick Rohr authored
-
Maciej Żenczykowski authored
This reverts commit be9685c3. Reason for revert: fails on 4.9 due to bpf_skb_adjust_room requiring a later kernel, will need an alternative approach Bug: 261818177 Test: TreeHugger Signed-off-by:
Maciej Żenczykowski <maze@google.com> Change-Id: I26535a96de80febc2fd54dcb564cde4f9ed7b3c9
-
Aaron Huang authored
-
- Dec 30, 2022
-
-
Maciej Żenczykowski authored
-
Maciej Żenczykowski authored
-
Maciej Żenczykowski authored
will make it easier to extend this for 5.4+ behaviour as well without having to introduce another is_5_4 boolean Bug: 263884894 Test: TreeHugger Signed-off-by:
Maciej Żenczykowski <maze@google.com> Change-Id: Id4f6512d813dd460cb2b9a7ccb6a5f7b7e937575
-
- Dec 29, 2022
-
-
Maciej Żenczykowski authored
easier on bpf verifier with no third case Bug: 263884894 Test: TreeHugger Signed-off-by:
Maciej Żenczykowski <maze@google.com> Change-Id: I5076de6f83ba522ed4783bca0a9d7fca4024986a
-
Mohannad Farrag authored
* This was failing the pre-submits are cc_objects are compilable modules but they failed to compile due to empty srcs. However, this didn't affect the compilation of Cronet as the empty cc_objects are never in the path of compilation of Cronet because their attributes are propagated up the tree. Test: m Change-Id: I7cc87d53d4f460ef4958d5ecfb23ac28662fdbdc
-
- Dec 28, 2022
-
-
Hungming Chen authored
This is a preparation to add new interface index. Test: atest BpfCoordinatorTest Change-Id: I758c2e7142099d74099fb8203f41ffec9b1182ee
-
Aaron Huang authored
Currently, mSubscriberIdMatchRule is only used in NetworkTemplate and it depends on if mMatchSubscriberIds is empty or not. Thus, remove it since is not really necessary and replace it with checking matchSubscriberIds if needed. Bug: 238843364 Test: build, FrameworksNetTests CtsNetTestCases Change-Id: Ic66d2ff2826846778b004bb15a4718a62fa1f470
-
- Dec 27, 2022
-
-
Maciej Żenczykowski authored
Test: TreeHugger Signed-off-by:
Maciej Żenczykowski <maze@google.com> Change-Id: I12e147e61febd96324cdbb6642e0894e5c399526
-
- Dec 23, 2022
-
-
Motomu Utsumi authored
media_ndk_headers is not available to tethering apex. But header files are available via libmediandk ndk_library Test: m Change-Id: I319d9f3a68c64d7b73af68db40d4a509eba59cd3
-
Motomu Utsumi authored
Test: ./update_results.sh Change-Id: I02fb762d2a4af0fe5b200a9aafc6dee33d3eaed0
-
Motomu Utsumi authored
Test: ./update_results.sh Change-Id: I4f02494448ede2eeefadbbb9c7c4fdf38ede645c
-
Motomu Utsumi authored
Test: ./update_results.sh Change-Id: I2d1df64fe312806fc834d58847ba7f041e202fc3
-
Motomu Utsumi authored
Test: ./update_results.sh Change-Id: Ibc2b3152ab68c5b46acff7d19d2c853889fa887c
-
Motomu Utsumi authored
This helps to know java_sources/java_actions are in which java group Test: ./update_results.sh Change-Id: Iec3fad1aded935a8d17184a2315d7f21b2a803f2
-
Motomu Utsumi authored
* changes: gn2bp: remove hardcoded jni_registration denylist gn2bp: add javacflag to skip GEN_JNI stub generation gn2bp: use Android libgtest_prod_headers gn2bp: depend on :current_android_jar for JNI generation
-
Patrick Rohr authored
Test: m cronet_aml_java Change-Id: I1470e1faa64f17729c63082228423f8177798d88
-
Patrick Rohr authored
This conflicts as the fully-featured GEN_JNI is already generated by another genrule. Test: none Change-Id: I522501444b02d2a304e81d86d86b0c8f3589035c
-
Patrick Rohr authored
This requires aosp/2366787 to work. Unfortunately, Chromium does not add a dependency on gtest, so builtin_module cannot be used. Using the gtest prod headers from Android is fine for //base. The only thing that's used is gtest_prod.h, which includes one friend macro that hasn't changed in 14 years. Test: build cronet Change-Id: I03b12ae698699527da7e69c9d4d50da1822fd5bc
-
- Dec 22, 2022
-
-
Maciej Żenczykowski authored
-
Patrick Rohr authored
Test: update_results.sh Change-Id: Id08cb64c65cc2f2b2136d6e437ad8af1876da180
-