LineageOS/android_packages_modules_Connectivity
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge "Fix and test usb function configuration change" into sc-dev

Browse Source
This commit is contained in:
TreeHugger Robot
2021-07-01 07:04:48 +00:00
committed by Android (Google) Code Review
parent 2e0cdad198 66fa6f24f9
commit 424ef2043c
2 changed files with 61 additions and 30 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
Tethering/src/com/android/networkstack/tethering/Tethering.java
View File

@@ -1039,7 +1039,7 @@ public class Tethering {
final boolean rndisEnabled = intent.getBooleanExtra(USB_FUNCTION_RNDIS, false); final boolean rndisEnabled = intent.getBooleanExtra(USB_FUNCTION_RNDIS, false);
final boolean ncmEnabled = intent.getBooleanExtra(USB_FUNCTION_NCM, false); final boolean ncmEnabled = intent.getBooleanExtra(USB_FUNCTION_NCM, false);
mLog.log(String.format("USB bcast connected:%s configured:%s rndis:%s ncm:%s", mLog.i(String.format("USB bcast connected:%s configured:%s rndis:%s ncm:%s",
usbConnected, usbConfigured, rndisEnabled, ncmEnabled)); usbConnected, usbConfigured, rndisEnabled, ncmEnabled));
// There are three types of ACTION_USB_STATE: // There are three types of ACTION_USB_STATE:
@@ -1416,7 +1416,7 @@ public class Tethering {
// If TETHERING_USB is forced to use ncm function, TETHERING_NCM would no longer be // If TETHERING_USB is forced to use ncm function, TETHERING_NCM would no longer be
// available. // available.
if (mConfig.isUsingNcm()) return TETHER_ERROR_SERVICE_UNAVAIL; if (mConfig.isUsingNcm() && enable) return TETHER_ERROR_SERVICE_UNAVAIL;
UsbManager usbManager = (UsbManager) mContext.getSystemService(Context.USB_SERVICE); UsbManager usbManager = (UsbManager) mContext.getSystemService(Context.USB_SERVICE);
usbManager.setCurrentFunctions(enable ? UsbManager.FUNCTION_NCM : UsbManager.FUNCTION_NONE); usbManager.setCurrentFunctions(enable ? UsbManager.FUNCTION_NCM : UsbManager.FUNCTION_NONE);
@@ -2552,7 +2552,7 @@ public class Tethering {
return; return;
} }
mLog.log("adding IpServer for: " + iface); mLog.i("adding IpServer for: " + iface);
final TetherState tetherState = new TetherState( final TetherState tetherState = new TetherState(
new IpServer(iface, mLooper, interfaceType, mLog, mNetd, mBpfCoordinator, new IpServer(iface, mLooper, interfaceType, mLog, mNetd, mBpfCoordinator,
makeControlCallback(), mConfig.enableLegacyDhcpServer, makeControlCallback(), mConfig.enableLegacyDhcpServer,
@@ -2567,7 +2567,7 @@ public class Tethering {
if (tetherState == null) return; if (tetherState == null) return;
tetherState.ipServer.stop(); tetherState.ipServer.stop();
mLog.log("removing IpServer for: " + iface); mLog.i("removing IpServer for: " + iface);
mTetherStates.remove(iface); mTetherStates.remove(iface);
} }

83
Tethering/tests/unit/src/com/android/networkstack/tethering/TetheringTest.java
View File

@@ -2610,12 +2610,57 @@ public class TetheringTest {
reset(mBluetoothAdapter, mBluetoothPan); reset(mBluetoothAdapter, mBluetoothPan);
} }
private void runDualStackUsbTethering(final String expectedIface) throws Exception {
when(mNetd.interfaceGetList()).thenReturn(new String[] {expectedIface});
when(mRouterAdvertisementDaemon.start())
.thenReturn(true);
final UpstreamNetworkState upstreamState = buildMobileDualStackUpstreamState();
runUsbTethering(upstreamState);
verify(mNetd).interfaceGetList();
verify(mNetd).tetherAddForward(expectedIface, TEST_MOBILE_IFNAME);
verify(mNetd).ipfwdAddInterfaceForward(expectedIface, TEST_MOBILE_IFNAME);
verify(mRouterAdvertisementDaemon).start();
verify(mDhcpServer, timeout(DHCPSERVER_START_TIMEOUT_MS)).startWithCallbacks(
any(), any());
sendIPv6TetherUpdates(upstreamState);
assertSetIfaceToDadProxy(1 /* numOfCalls */, TEST_MOBILE_IFNAME /* ifaceName */);
verify(mRouterAdvertisementDaemon).buildNewRa(any(), notNull());
verify(mNetd).tetherApplyDnsInterfaces();
}
private void forceUsbTetheringUse(final int function) {
Settings.Global.putInt(mContentResolver, TETHER_FORCE_USB_FUNCTIONS, function);
final ContentObserver observer = mTethering.getSettingsObserverForTest();
observer.onChange(false /* selfChange */);
mLooper.dispatchAll();
}
private void verifyUsbTetheringStopDueToSettingChange(final String iface) {
verify(mUsbManager, times(2)).setCurrentFunctions(UsbManager.FUNCTION_NONE);
mTethering.interfaceRemoved(iface);
sendUsbBroadcast(true, true, -1 /* no functions enabled */);
reset(mUsbManager, mNetd, mDhcpServer, mRouterAdvertisementDaemon,
mIPv6TetheringCoordinator, mDadProxy);
}
@Test @Test
public void testUsbTetheringWithNcmFunction() throws Exception { public void testUsbFunctionConfigurationChange() throws Exception {
when(mResources.getInteger(R.integer.config_tether_usb_functions)).thenReturn( // Run TETHERING_NCM.
TetheringConfiguration.TETHER_USB_NCM_FUNCTION); runNcmTethering();
verify(mDhcpServer, timeout(DHCPSERVER_START_TIMEOUT_MS).times(1)).startWithCallbacks(
any(), any());
// Change the USB tethering function to NCM. Because the USB tethering function was set to
// RNDIS (the default), tethering is stopped.
forceUsbTetheringUse(TETHER_USB_NCM_FUNCTION);
verifyUsbTetheringStopDueToSettingChange(TEST_NCM_IFNAME);
// TODO: move this into setup after allowing configure TEST_NCM_REGEX into
// config_tether_usb_regexs and config_tether_ncm_regexs at the same time.
when(mResources.getStringArray(R.array.config_tether_usb_regexs)) when(mResources.getStringArray(R.array.config_tether_usb_regexs))
.thenReturn(new String[] {TEST_NCM_REGEX}); .thenReturn(new String[] {TEST_RNDIS_REGEX, TEST_NCM_REGEX});
sendConfigurationChanged(); sendConfigurationChanged();
// If TETHERING_USB is forced to use ncm function, TETHERING_NCM would no longer be // If TETHERING_USB is forced to use ncm function, TETHERING_NCM would no longer be
@@ -2625,30 +2670,16 @@ public class TetheringTest {
mLooper.dispatchAll(); mLooper.dispatchAll();
ncmResult.assertHasResult(); ncmResult.assertHasResult();
final UpstreamNetworkState upstreamState = buildMobileDualStackUpstreamState(); // Run TETHERING_USB with ncm configuration.
runUsbTethering(upstreamState); runDualStackUsbTethering(TEST_NCM_IFNAME);
verify(mNetd).interfaceGetList(); // Change configuration to rndis.
verify(mNetd).tetherAddForward(TEST_NCM_IFNAME, TEST_MOBILE_IFNAME); forceUsbTetheringUse(TETHER_USB_RNDIS_FUNCTION);
verify(mNetd).ipfwdAddInterfaceForward(TEST_NCM_IFNAME, TEST_MOBILE_IFNAME); verifyUsbTetheringStopDueToSettingChange(TEST_NCM_IFNAME);
verify(mRouterAdvertisementDaemon).start(); // Run TETHERING_USB with rndis configuration.
verify(mDhcpServer, timeout(DHCPSERVER_START_TIMEOUT_MS)).startWithCallbacks( runDualStackUsbTethering(TEST_RNDIS_IFNAME);
any(), any()); runStopUSBTethering();
sendIPv6TetherUpdates(upstreamState);
assertSetIfaceToDadProxy(1 /* numOfCalls */, TEST_MOBILE_IFNAME /* ifaceName */);
verify(mRouterAdvertisementDaemon).buildNewRa(any(), notNull());
verify(mNetd).tetherApplyDnsInterfaces();
Settings.Global.putInt(mContentResolver, TETHER_FORCE_USB_FUNCTIONS,
TETHER_USB_RNDIS_FUNCTION);
final ContentObserver observer = mTethering.getSettingsObserverForTest();
observer.onChange(false /* selfChange */);
mLooper.dispatchAll();
// stop TETHERING_USB and TETHERING_NCM
verify(mUsbManager, times(2)).setCurrentFunctions(UsbManager.FUNCTION_NONE);
mTethering.interfaceRemoved(TEST_NCM_IFNAME);
sendUsbBroadcast(true, true, -1 /* function */);
} }
// TODO: Test that a request for hotspot mode doesn't interfere with an // TODO: Test that a request for hotspot mode doesn't interfere with an
// already operating tethering mode interface. // already operating tethering mode interface.