Commit graph

75,063 commits

Message SHA1
[remoteauth] Add RangingSession interfaces
* Initial commit to define interfaces and data structures for ranging
  session.

Test: atest RemoteAuthUnitTests
Bug: 290675597
Change-Id: I1eb54c079a212e8d3599b156899d0c46c6b6d636

Jin Chen   commited 2023-08-24 13:28:50 -07:00
2ff4b6b403
Merge "Add RemoteAuthService" into main
Igor Zaslavsky   authored and   Gerrit Code Review   commited 2023-08-24 20:02:42 +00:00
a92d8cab5f
Add RemoteAuthService
Add RemoteAuthService APIs and Stubs (hide) for Settings of D2DA

This CL introduces new SystemApi allow user to discover remote devices
compatible to be registered as remote authenticators via RemoteAuthManager

Design doc: go/remote-auth-manager-fishfood-design

Test: built successfully.
Bug: 290092977
API-Coverage-Bug: 294934095
Change-Id: Iaaae1126065fdc3db469eeb8d85ac654b8199a12

Igor Zaslavsky   commited 2023-08-24 17:50:11 +00:00
ec344f4c18
Merge "Execute netd callbacks on CS handler thread" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-08-24 08:33:03 +00:00
f180b14d9a
Merge "Move ArpPacket.java to frameworks/libs/net" into main
Yuyang Huang   authored and   Gerrit Code Review   commited 2023-08-24 07:12:27 +00:00
136bf7c8e3
Execute netd callbacks on CS handler thread
Test: TH
Bug: 293965195
Change-Id: I178a9904d2ee090ed1fb487c061226909cbde1d6

Chalard Jean   commited 2023-08-24 15:44:15 +09:00
5d05c4b9d1
Merge "Add some tests back which were removed for fixing kotlin nullable errors" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-08-24 04:14:28 +00:00
760bcac429
Add some tests back which were removed for fixing kotlin nullable errors
Followup CL for aosp/2700076

Bug: 296972712
Test: build with aosp/2688146
Change-Id: I617b330c42a99e4c839adac2ea614756b5955ce9

Motomu Utsumi   commited 2023-08-24 11:25:21 +09:00
5324475f6e
Merge changes I2399960f,I7b6e1a3b into main
* changes:
  ethernet: clean up EthernetManagerTest
  ethernet: test adding listener after removing server mode iface

Patrick Rohr   authored and   Gerrit Code Review   commited 2023-08-23 16:23:39 +00:00
0466125e93
Move ArpPacket.java to frameworks/libs/net
Bug: 281639507
Test: TH
Change-Id: I754c3581438d716d2156b80e4ce29b3f2adf76d4

Yuyang Huang   commited 2023-08-23 18:24:35 +09:00
5c2126ed38
Merge "Also bundle Cronet telemetry jar" into main
Stefano Duo   authored and   Gerrit Code Review   commited 2023-08-23 08:49:55 +00:00
5e542536cd
Merge "[Thread] move Thread into com.android.tethering" into main
Kangping Dong   authored and   Gerrit Code Review   commited 2023-08-23 08:37:38 +00:00
0dc49df9a4
[Thread] move Thread into com.android.tethering
Bug: 296211911
Change-Id: Ifd4f7cd06f1aeaba739d27d2328e89031ca73235

Kangping Dong   commited 2023-08-23 14:28:50 +08:00
e7fef8938a
ethernet: clean up EthernetManagerTest
createChangeEvent is not expected to be called from a test, so make it
private. Also delete pollOrThrow as polling inline creates a better log
output.

Test: atest EthernetManagerTest
Change-Id: I2399960fee2c51fdf340186189ae83a20c4ac6f3

Patrick Rohr   commited 2023-08-22 12:39:36 -07:00
f454f89354
ethernet: test adding listener after removing server mode iface
Tests that registering a EthernetStateListener functions as expected
after requesting a tethering interface and subsequently removing that
interface.

Test: atest testCallbacks_afterRemovingServerModeInterface
Bug: 290971780
Change-Id: I7b6e1a3b3ac23f4fbdd2d20f63f83d59c4d4999d

Patrick Rohr   commited 2023-08-22 12:37:58 -07:00
f848c6c689
Grant prohr ownership of ethernet tests
Test: N/A
Change-Id: I50101f9e5dc4dede5732a31da4ecd53d027adedc

Patrick Rohr   commited 2023-08-22 12:37:43 -07:00
88a88f1304
Merge "Ignore EXCLUDED_ROUTES LinkPropertiesTest up to S_V2" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-08-22 08:19:25 +00:00
b5208bad3e
Ignore EXCLUDED_ROUTES LinkPropertiesTest up to S_V2
The platform doesn't allow overwriting compat flags that have target SDK
lower than the platform SDK in release build. Ignoring the test cases
that need compat flag overwrite up to S_V2 makes the MTS pass on S
device.

Bug: 296248982
Test: TH
Change-Id: Iab102725004c39b4dc3c0200c9bc673dd6be1302

Yuyang Huang   commited 2023-08-22 15:17:38 +09:00
7fe61f4cbd
Merge "Fix kotlin nullable errors in Connectivity tests" into main
Colin Cross   authored and   Gerrit Code Review   commited 2023-08-22 04:13:01 +00:00
d5038a042c
Initial commit for connectivity.
Bug: 290265761
Bug: 291146480
Bug: 290275468
Test: None
Change-Id: Iaba64415cf81454b6e999f10bad54810884fc3da

Salil Rajadhyaksha   commited 2023-08-21 18:49:51 +00:00
67b9284de0
Merge "TrafficController - trivialize dump()" into main
Maciej Ĺ»enczykowski   authored and   Gerrit Code Review   commited 2023-08-21 17:16:11 +00:00
a5af3e3e5e
Merge "TrafficController - remove dead code" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-08-21 17:14:34 +00:00
02c6061cf8
Merge "BpfHandler - remove use of getMap()" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-08-21 17:10:25 +00:00
d5ebd0b013
Also bundle Cronet telemetry jar
Bug: 284972285
Test: m & atest NetHttpTests
Change-Id: Id24fa826f617187722b329821dae9d9548bf218b

Stefano Duo   commited 2023-08-21 16:51:08 +01:00
2a90733b08
Merge "Adjust owners for incremental NsdManager changes" into main
Remi NGUYEN VAN   authored and   Gerrit Code Review   commited 2023-08-21 05:50:14 +00:00
d9218c52b8
Merge "Rename to isTetheringFeatureEnabled" into main
Motomu Utsumi   authored and   Gerrit Code Review   commited 2023-08-21 03:17:38 +00:00
cad1226991
TrafficController - trivialize dump()
The goal is to be able to remove getMap() from BpfMap.h,
as it effectively leaks (what should be an) internal fd.

In practice the dump() operation isn't useful,
as initMaps() and thus native_init() already
guarantee all these maps have valid file descriptors.

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

Maciej Ĺ»enczykowski   commited 2023-08-20 19:25:07 +00:00
7d2a4e9d47
TrafficController - remove dead code
Test: TreeHugger
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: Id3a128cd72f467fa60213f57c6dd942a0629ba1d

Maciej Ĺ»enczykowski   commited 2023-08-20 18:48:48 +00:00
29c8830417
BpfHandler - remove use of getMap()
The FD number isn't actually useful debugging information,
and I want to get rid of this fd leak out of BpfMap class.

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

Maciej Ĺ»enczykowski   commited 2023-08-20 16:42:09 +00:00
85d4a5e198
Merge "remove unused TcpTx/RxPackets java & jni constants" into main
Maciej Ĺ»enczykowski   authored and   Gerrit Code Review   commited 2023-08-18 14:49:40 +00:00
359dacfd26
Merge "simplify tcpTx/RxPackets" into main
Maciej Ĺ»enczykowski   authored and   Gerrit Code Review   commited 2023-08-18 14:45:56 +00:00
b4870c4ce2
Merge changes from topic "trackrecord-backtrace" into main
* changes:
  ethernet: use TrackRecord backtrace for better failure logs
  ethernet: correctly report tethering interface state
  ethernet: add missing assert in eventuallyExpect helper

Patrick Rohr   authored and   Gerrit Code Review   commited 2023-08-18 14:10:21 +00:00
c45928b35b
Merge "Report resolution stop" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-08-18 09:30:26 +00:00
aa4db9753b
Adjust owners for incremental NsdManager changes
Allow adding tests or adjusting them to increase coverage on behavior
that is implemented in NsdService.

Test: m
Change-Id: I1b0851bdd220caba68c47d5d50d86a543227817b

Remi NGUYEN VAN   commited 2023-08-18 14:49:34 +09:00
7ac1b9bc33
Report resolution stop
Report the nsd metrics event of resolution stop.

Bug: 287546772
Test: atest FrameworksNetTestCases
Change-Id: I2e6de22c946d58a3410f16b2f2689fd3ffc635a4

Paul Hu   commited 2023-08-18 11:59:06 +08:00
6014905672
Merge "Stop the created handler threads on the tests" into main
Paul Hu   authored and   Gerrit Code Review   commited 2023-08-18 02:40:45 +00:00
a436faf92d
Add tests in Ikev2VpnTest back to presubmit
Those tests are no longer flaky after fix being merged, so add
them back to presubmit.

Fix: 293239828
Test: atest CtsNetTestCases:android.net.cts.Ikev2VpnTest \
      --rerun-until-failure 100
Change-Id: I3e5b0f8053404b8840e8ce67ea82344ab53ce421

chiachangwang   authored and   Chiachang Wang   commited 2023-08-18 00:41:55 +00:00
739f3c7ae9
Merge changes I26022db4,Ib45a130e,Ie39de2b3 into main
* changes:
  StatsValue - define operator+=
  eliminate 'struct Stats' in favour of StatsValue
  eliminate InitStats

Treehugger Robot   authored and   Gerrit Code Review   commited 2023-08-17 21:16:39 +00:00
2238dc2ce3
Merge "[Cronet] NetHttpCoverageTests presubmit SLO fixes" into main
Treehugger Robot   authored and   Gerrit Code Review   commited 2023-08-17 16:33:20 +00:00
6cabb41329
[Cronet] NetHttpCoverageTests presubmit SLO fixes
- Adds NetHttpCoverageTests to postsubmit. This would allow us
to run tests with @SkipPresubmit in post submit.
- Don't jarjar com.android.testutil so that the annotation can
still be found by the testrunner.

Bug: b/293843426
Test: th
Change-Id: I9e792d338515acba798092bbc4c615aef2cac47a

Chidera Olibie   commited 2023-08-17 14:52:31 +00:00
be9f351f36
Merge "Stop the created handler threads" into main
Chiachang Wang   authored and   Gerrit Code Review   commited 2023-08-17 09:40:05 +00:00
033ca4c8cf
Rename to isTetheringFeatureEnabled
DeviceConfigUtils has isFeatureEnabled for NetworkStack and Tethering
which were confusing since the difference was only the arguments.
This CL renames isFeatureEnabled for Tethering module to
isTetheringFeatureEnabled to avoid confusion.

Bug: 279108992
Test: m
Change-Id: I05255bb38619afbe621d29524d7910d206a149f1

Motomu Utsumi   commited 2023-08-17 17:47:19 +09:00
624aeb4274
Stop the created handler threads on the tests
The created handler thread will stay there for a while until
all the tests are finished. This is considered as a leakage
which will take the resource from the devices. In low end
devices, this may possibly impact the performance. The created
threads should be closed explicitly.

Test: atest FrameworksNetTests
Change-Id: Id5faceec341578f1c9757194c91a433fbb3f657f

Paul Hu   commited 2023-08-17 06:53:16 +00:00
f6e1867963
Stop the created handler threads
The created handler thread will stay there for a while until
all the tests are finished. This is considered as a leakage
which will take the resource from the devices. In low end
devices, this may possibly impact the performance. The created
threads should be closed explicitly.

Test: atest FrameworksNetTests
Change-Id: I332d55d8effdb975fd9a8953b19910a7fb25e426

chiachangwang   commited 2023-08-17 06:18:46 +00:00
8d336eb1a3
Merge "Use isTetheringFeatureNotChickenedOut for kill switch" into main
Motomu Utsumi   authored and   Gerrit Code Review   commited 2023-08-17 06:13:21 +00:00
23b0bab6b1
Merge changes from topic "isTetheringFeatureNotChickenedOut" into main
* changes:
  Use Java code path for bpf map operations in BpfNetMaps
  Rename to isTetheringFeatureNotChickenedOut

Motomu Utsumi   authored and   Gerrit Code Review   commited 2023-08-17 06:12:36 +00:00
006020b83a
Merge "Report resolution metrics data" into main
Paul Hu   authored and   Gerrit Code Review   commited 2023-08-17 02:57:45 +00:00
69a18c6f90
Use isTetheringFeatureNotChickenedOut for kill switch
Following CLs remove the default value from isFeatureEnabled since the
default value was difficult to use correctly.
For example, there was a code that use isAtLeastT() as default value and
expect feature is always enabled on T+ devices.
However, default value is used only when the device does not have the
flag value or flag value is 0.
So the expectation that the feature is always enabled on T+ devices is
not correct if flag is pushed to the device by mistake or the device is
upgraded from S to T but the flag value is not cleared.
If the feature should be always enabled on T+ devices, `isAtleastT() ||
isFeatureEnabled()` should be used instead of `isFeatureEnabled(
isAtleastT() /* defaultValue*/ )`

After the default value argument is removed from isFeatureEnabled,
isFeatureEnabled is used for default false flag and
isTetheringFeatureNotChickenedOut is used for default true kill switch flag.

automatic_on_off_keepalive_version flag is not configured and devices
don't have this flag value.
So updating to use isTetheringFeatureNotChickenedOut should not have any
behavior change.

Test: m
Bug: 279108992
Change-Id: I43ceab6cc5234bd5a43af3acbd7bd7344844c5a6

Motomu Utsumi   commited 2023-08-16 20:15:52 +09:00
0a50c65f75
Use Java code path for bpf map operations in BpfNetMaps
Experiment to verify bpf map operations refactoring showed no
change of map size metrics and no system health regressions.
So this CL update BpfNetMaps to always use new code path with kill
switch.

Test: m
Bug: 217624062
Change-Id: Ic17470ac5c075884871ea2df2c13c054f5ac2929

Motomu Utsumi   commited 2023-08-16 20:08:20 +09:00
7628ea30b0
Rename to isTetheringFeatureNotChickenedOut
Original name was isTetheringFeatureForceDisabled but it's expected that
the caller of this method will use as `enabled =
!isTetheringFeatureForceDisabled` which is error prune.
So aosp/2711138 renames to isTetheringFeatureNotChickenedOut and updates to
return true if the feature is not disabled.

Bug: 279108992
Test: m
Change-Id: Ia992113e9fb1aee09fcfe48e2b39ca03876063ee

Motomu Utsumi   commited 2023-08-16 20:07:53 +09:00
dafb9f257b