Commit graph

90,654 commits

Message SHA1
Merge "[remoteauth] Clean up unused deps" into main am: 10f59a248c
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2757186

Change-Id: I92aabeedf970c655a2598ff4ded65e1b15dc3257
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Jin Chen   authored and   Automerger Merge Worker   commited 2023-09-22 02:49:59 +00:00
3ec94c211e
Merge "[remoteauth] Clean up unused deps" into main
Jin Chen   authored and   Gerrit Code Review   commited 2023-09-22 02:18:36 +00:00
10f59a248c
Merge "[Refactor] Make uid owner match comparison logic into a function" into main am: 3897aaf7e9
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2754265

Change-Id: I75a3080ca4a815deb4d38a35c4ce5c1d2d22ca18
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Maciej Żenczykowski   authored and   Automerger Merge Worker   commited 2023-09-21 11:29:12 +00:00
d2229688a9
Merge "[Refactor] Make uid owner match comparison logic into a function" into main
Maciej Żenczykowski   authored and   Gerrit Code Review   commited 2023-09-21 10:21:52 +00:00
3897aaf7e9
Merge "[remoteauth] Implement UwbRangingSession" into main am: 462572b24b
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2726504

Change-Id: Icb66b94c50190ce89c09072bed3fc958ef9aacb6
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Jin Chen   authored and   Automerger Merge Worker   commited 2023-09-21 02:33:48 +00:00
5be6652302
Merge "[remoteauth] Implement UwbRangingSession" into main
Jin Chen   authored and   Gerrit Code Review   commited 2023-09-21 01:18:17 +00:00
462572b24b
[remoteauth] Implement UwbRangingSession
Test: atest RemoteAuthUnitTests
Bug: 290675814
Change-Id: I9a38298092607fe87bc7a8b157ff9d44d84fa55a

Jin Chen   commited 2023-09-20 15:33:48 -07:00
c54521ba85
Merge "bump min_sdk_version from 29 (Q) to 30 (R)" into main am: a828357fe1
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2755673

Change-Id: Iad32c375b9a29ce14726d9d52ea06f1c200e6c3f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Maciej Żenczykowski   authored and   Automerger Merge Worker   commited 2023-09-20 17:37:18 +00:00
997ff58786
Merge "bump min_sdk_version from 29 (Q) to 30 (R)" into main
Maciej Żenczykowski   authored and   Gerrit Code Review   commited 2023-09-20 17:01:40 +00:00
a828357fe1
Merge "Add test for partial responses" into main am: d292cff70e
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2437173

Change-Id: I5f68885adf0083c3cea1de67fd5ce95749246915
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Treehugger Robot   authored and   Automerger Merge Worker   commited 2023-09-20 09:36:54 +00:00
4741c6eba8
Merge "Add test for partial responses" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-09-20 09:07:15 +00:00
d292cff70e
Merge "SyncSM04: implement processMessage function" into main am: 35f19e74a9
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2301482

Change-Id: Ib8468e83da956ee4a6c191baad8be13db51813b9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Mark Chien   authored and   Automerger Merge Worker   commited 2023-09-20 03:23:52 +00:00
6ab29917b4
Merge "SyncSM04: implement processMessage function" into main
Mark Chien   authored and   Gerrit Code Review   commited 2023-09-20 02:36:05 +00:00
35f19e74a9
[remoteauth] Clean up unused deps
Fix: 301051105
Test: build
Change-Id: I23dfab79503db2fe34ffd06bf48d54961c0bc424

Jin Chen   commited 2023-09-19 18:04:24 -07:00
f86048224e
bump min_sdk_version from 29 (Q) to 30 (R)
Mainline no longer supports Q.
These files were recently moved into p/m/C as part of a refactor.

Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: I0f06ace921db6c79d63c0048bdb73d167ff606cf

Maciej Żenczykowski   commited 2023-09-19 23:22:58 +00:00
127a1441c3
Merge "[Thread] initial Thread network service" into main am: 01e6dadd1f
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2721333

Change-Id: I5b1dad9c40dcade78c352265164da0992a3fcb0d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Kangping Dong   authored and   Automerger Merge Worker   commited 2023-09-19 11:29:17 +00:00
e5f1122ec9
Merge "SyncSM03: implement state transition logic" into main am: 6551cc5620
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2325064

Change-Id: I7b39cdf76b3faafa8001562122d90fa2ab5f2714
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Treehugger Robot   authored and   Automerger Merge Worker   commited 2023-09-19 11:29:01 +00:00
65b43d4e34
Merge "[Thread] initial Thread network service" into main
Kangping Dong   authored and   Gerrit Code Review   commited 2023-09-19 11:25:37 +00:00
01e6dadd1f
Merge "SyncSM03: implement state transition logic" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-09-19 10:54:24 +00:00
6551cc5620
Merge "Check that wifi, cell validates before tests" into main am: 1bc9f5c66d
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2756047

Change-Id: I3009e6a1de603203d457646381094f1abcc51ad0
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Treehugger Robot   authored and   Automerger Merge Worker   commited 2023-09-19 10:37:38 +00:00
1ec550b7a3
Merge "Add back partial connectivity test to presubmit" into main am: bc884540b3
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2739413

Change-Id: I790521debb64ce88c1f232b42028ab87735390f3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Treehugger Robot   authored and   Automerger Merge Worker   commited 2023-09-19 10:37:29 +00:00
638397b1ee
Merge "Check that wifi, cell validates before tests" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-09-19 10:20:38 +00:00
1bc9f5c66d
Merge "Add back partial connectivity test to presubmit" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-09-19 09:51:17 +00:00
bc884540b3
Check that wifi, cell validates before tests
Tests are sometimes run on hardware devices with wrongly configured wifi
or cell data. Ensure that this is reported as an infra error, and not a
test error, so the root cause is easier to identify.

Bug: 264170054
Test: atest
Change-Id: I4f964fbd4ee497e8ac92f7729375b75b6c4594a3

Remi NGUYEN VAN   commited 2023-09-19 17:26:20 +09:00
f1846ac13b
SyncSM04: implement processMessage function
After this change, all synchronous state machine functions are
implemented.
There are some intentional limiations:
1. Allow setup states before machine thread started or inside machine
   thread but states can only be setup once before calling #start.
2. The behavior of transitionTo is defined so that the destination
   state can never change during state transitions. Once a destination
   has been specified, it cannot be changed until the previous state
   transition is complete. In other words, SyncStateMachine does not
   support calling transitionTo in State enter or exit.
3. Support sendSelfMessage in State#processMessage(), #enter() and
   #exit(). This allow automaton to decide the follow up action by
   itself. For example: if something wrong happens during state
   transition, the implementation can enqueue an error message then
   process the error message after finishing the current state
   transition.

Test: atest SynStateMachineTest
Change-Id: I0790df4eeab2dccdb9f407d9131a62c3c12d123b

Mark   commited 2023-09-19 08:20:56 +00:00
21cdb40e59
Merge "SyncSM02: implement addState function" into main am: c5ebf2a7e6
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2325063

Change-Id: I8e0920f123afe43b2a0e8174afa654ef9afb76e8
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Treehugger Robot   authored and   Automerger Merge Worker   commited 2023-09-19 05:20:33 +00:00
bf562951db
Merge "SyncSM02: implement addState function" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-09-19 04:46:22 +00:00
c5ebf2a7e6
Merge "SyncSM01: define SyncStateMachine user interface" into main am: 85c1247b3c
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2325062

Change-Id: Ia7202e652e36829882b58e1f6fc4a6f65c53c1c8
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Treehugger Robot   authored and   Automerger Merge Worker   commited 2023-09-19 04:28:20 +00:00
60b5e666a8
Merge "SyncSM01: define SyncStateMachine user interface" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-09-19 03:44:58 +00:00
85c1247b3c
[Refactor] Make uid owner match comparison logic into a function
As an inline function, the logic can be reused by others.

Bug: Bug: 288340533
Test: build; presubmit
Change-Id: I8e57829e304e829eed72cc165b051cd22088260d

Ken Chen   commited 2023-09-17 00:11:48 +08:00
f7d23e1a60
SyncSM03: implement state transition logic
Implement performTransitions which does state transitions.
1. Determine the common ancestor state of current/destination states
2. Invoke state exit list from current state to common ancestor state.
3. Invoke state enter list from common ancestor state to destState by
going through mEnterStateStack.

The state transition is always defined so that the target can never be
changed mid-way of a state transition. In other words, calling
transitionTo in State enter and exit is not allowed.

Test: atest SyncStateMachineTest
Change-Id: I51e8c5440a8b9ac25715c3d030717421f68c15b3

Mark   commited 2023-09-15 14:38:12 +00:00
9e572d10f3
SyncSM02: implement addState function
Duplicated state is not allowed.

Test: atest SynStateMachineTest
Change-Id: I0d5c73f666f90aebcfbf535cf0f824c5050941a2

Mark   commited 2023-09-15 14:23:52 +00:00
1d460b568e
Merge "SyncSM10: explicitly keep TetheringUtils jni methods" into main am: 19b785954a
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2353243

Change-Id: Iab609a6277b41f376e8335518b24a0163fd1e0bb
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Treehugger Robot   authored and   Automerger Merge Worker   commited 2023-09-15 09:16:00 +00:00
59b4a7ebcc
Merge "SyncSM10: explicitly keep TetheringUtils jni methods" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-09-15 08:32:37 +00:00
19b785954a
SyncSM01: define SyncStateMachine user interface
This change adds SyncStateMachine class and provides the user
interface but doesn't have the details implementation.

Test: m
Change-Id: Ic2fa69b9e6424601fc2df2d07a052f48a6a46370

Mark   commited 2023-09-15 08:07:22 +00:00
b48caa22f3
Merge "Fix EthernetTetheringTest#testLocalOnlyTethering flaky" into main am: 47158c98b5
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2676841

Change-Id: Iffe892336e49b68f69b14aca140345983abeb3bf
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Treehugger Robot   authored and   Automerger Merge Worker   commited 2023-09-14 18:07:58 +00:00
b41a17a4f4
Merge "Fix EthernetTetheringTest#testLocalOnlyTethering flaky" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-09-14 17:40:06 +00:00
47158c98b5
SyncSM10: explicitly keep TetheringUtils jni methods
Found R8 accidentally remove TetheringUtils jni methods while doing
some refactor. If any of jni java methods is removed by R8, devices
would fail to boot because of jni registering failure. Explicitly
adding progurad rule to make sure R8 never remove it.

Test: boot, atest CtsTetheringTest

Change-Id: I091a2f094d01f683fb454ed4624be35555940393

Mark   authored and   Mark Chien   commited 2023-09-14 14:39:25 +00:00
87800837dd
Add test for partial responses
Test that when a responder only responds with the exact records that
were queried, so only reply for PTR in discovery, only send SRV, TXT,
A, AAAA when asked explicitly, service resolve succeeds.

This ensures that the querier sends followup queries for each record.
See RFC6763 12., especially the last paragraph.

Bug: 267570781
Bug: 267371243
Test: atest NsdManagerTest
Change-Id: Ia392e80c1e27b479c6177d19f6b4be6032dcb1cd

Remi NGUYEN VAN   commited 2023-09-14 18:34:58 +09:00
0852131520
Merge "Return for getBlockedPorts NonNull" into main am: c3a188d7dc
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2301890

Change-Id: I3329bb4e043a088719c2db5d6b362d4c83ece9ee
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Treehugger Robot   authored and   Automerger Merge Worker   commited 2023-09-14 06:39:04 +00:00
09a3570aa0
Merge "Return for getBlockedPorts NonNull" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-09-14 06:12:36 +00:00
c3a188d7dc
netd.c - nettrace - add 'wakeup' boolean am: b9cf34715c
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2751707

Change-Id: Ib2354cf50a18d1536cfb7461a96ffb6851a05d35
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Maciej Żenczykowski   authored and   Automerger Merge Worker   commited 2023-09-14 03:15:07 +00:00
7b38eabba1
netd.c - nettrace - add 'wakeup' boolean
This is based on network driver populated skb->mark magic bit.
This is the bit used by netd's WakeupController.

We mandated the location of this bit in U, though we haven't
(yet??) mandated it being supported by all network drivers.

If the driver doesn't support it, it could always
be false (skb->mark should default to 0),
or potentially (this is very very unlikely) be garbage.

IFIRC nettrace isn't enabled on pre-U devices anyway.

Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: I2b3b91315d77c08c022396253b26954593dd3f5a

Maciej Żenczykowski   commited 2023-09-13 22:05:30 +00:00
b9cf34715c
nettrace - skip over a single hopopts or dstops ipv6 extension header am: 73896a7b70
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2750043

Change-Id: I0ab8d41b7c5a63e8eaf83f89849b84a8fd499f13
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Maciej Żenczykowski   authored and   Automerger Merge Worker   commited 2023-09-13 19:39:03 +00:00
acb4a0e1d1
Merge "Add newApi linter check for mDNS library" into main am: 365cf0a393
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2718538

Change-Id: I6e392853943db3ff509bc25a4055367774e65cae
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Yuyang Huang   authored and   Automerger Merge Worker   commited 2023-09-13 06:17:54 +00:00
8992911f24
nettrace - skip over a single hopopts or dstops ipv6 extension header
Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: Iad7800387e1b84587794be93d037f712ac95fa07

Maciej Żenczykowski   commited 2023-09-13 06:11:18 +00:00
73896a7b70
Merge "Add newApi linter check for mDNS library" into main
Yuyang Huang   authored and   Gerrit Code Review   commited 2023-09-13 05:55:51 +00:00
365cf0a393
Merge "Fix flaky test: testOnNetworkConnectivityReportedFalse" into main am: bda7d35533
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2738817

Change-Id: Iea72367eeabd0ab1e794240eda730a52ef389ae0
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Paul Hu   authored and   Automerger Merge Worker   commited 2023-09-13 03:49:33 +00:00
6002f71400
Merge "Fix flaky test: testOnNetworkConnectivityReportedFalse" into main
Paul Hu   authored and   Gerrit Code Review   commited 2023-09-13 02:56:17 +00:00
bda7d35533
Merge "Add test for downstream tethering" into main am: dba0dbc7ec
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2606668

Change-Id: I251696b855e7f971ac740eb70fa86a4158f15c58
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

Treehugger Robot   authored and   Automerger Merge Worker   commited 2023-09-13 02:09:58 +00:00
ac88cd6df9