DO NOT MERGE: don't test BPF tethering offload in CTS
testTetherUdpV4AfterR verifies BPF map content that is going to break on Android 13 devices with newer mainline module versions which are going to support CLAT tethering offload. The CLAT tethering offload requires BPF map format modification which will break CTS for T. This is for Android 13 only because testTetherUdpV4AfterR is added since T and the BPF offload test has been moved from CTS to MTS. Changes: Before: - testTetherUdpV4UpToR: test tethering basic function on R. - testTetherUdpV4AfterR: test tethering basic function and BPF tethering offload on S+. After: - testTetherUdpV4: test tethering basic function on R+. (renamed from testTetherUdpV4UpToR) - testTetherUdpV4AfterR: (suppressed) Bug: 258637850 Test: atest EthernetTetheringTest Change-Id: Ia75347094fd7819c5dc68e05fcaac5c698c7729e
This commit is contained in:
committed by
Nucca Chen
parent
26d3723f64
commit
7af74d7095
@@ -85,7 +85,6 @@ import com.android.net.module.util.structs.Ipv4Header;
|
||||
import com.android.net.module.util.structs.Ipv6Header;
|
||||
import com.android.net.module.util.structs.UdpHeader;
|
||||
import com.android.testutils.DevSdkIgnoreRule;
|
||||
import com.android.testutils.DevSdkIgnoreRule.IgnoreAfter;
|
||||
import com.android.testutils.DevSdkIgnoreRule.IgnoreUpTo;
|
||||
import com.android.testutils.DeviceInfoUtils;
|
||||
import com.android.testutils.DumpTestUtils;
|
||||
@@ -95,6 +94,7 @@ import com.android.testutils.TestNetworkTracker;
|
||||
|
||||
import org.junit.After;
|
||||
import org.junit.Before;
|
||||
import org.junit.Ignore;
|
||||
import org.junit.Rule;
|
||||
import org.junit.Test;
|
||||
import org.junit.runner.RunWith;
|
||||
@@ -1062,8 +1062,7 @@ public class EthernetTetheringTest {
|
||||
}
|
||||
|
||||
@Test
|
||||
@IgnoreAfter(Build.VERSION_CODES.R)
|
||||
public void testTetherUdpV4UpToR() throws Exception {
|
||||
public void testTetherUdpV4() throws Exception {
|
||||
initializeTethering();
|
||||
runUdp4Test(new TetheringTester(mDownstreamReader), new RemoteResponder(mUpstreamReader),
|
||||
false /* usingBpf */);
|
||||
@@ -1095,10 +1094,12 @@ public class EthernetTetheringTest {
|
||||
assertTrue(isUdpOffloadSupportedByKernel("5.10.0"));
|
||||
}
|
||||
|
||||
// b/258637850 - this test will break with newer mainline modules on T due to support CLAT
|
||||
// tethering offload which changes BPF map format, so it has to be suppressed in CTS for T.
|
||||
@Ignore
|
||||
// TODO: refactor test testTetherUdpV4* into IPv4 UDP non-offload and offload tests.
|
||||
// That can be easier to know which feature is verified from test results.
|
||||
@Test
|
||||
@IgnoreUpTo(Build.VERSION_CODES.R)
|
||||
public void testTetherUdpV4AfterR() throws Exception {
|
||||
initializeTethering();
|
||||
final String kernelVersion = VintfRuntimeInfo.getKernelVersion();
|
||||
|
||||
Reference in New Issue
Block a user