commit cdd70f224858d639489eae5e801152c606eadb43
author yawensu <yawensu@google.com> Wed Sep 28 08:46:08 2022 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Wed Sep 28 08:46:08 2022 +0000
tree 145e384f71bb63ba2f3b151154fafed58afa76e6
parent 30f08b2db23178eb1c8340357ee7f77e488abd67
parent 780e2cb944b98d74e62b69229aaa6060e0ebb429

Set MTU to 1200 for CST because we reserve 80 bytes for VCN. am: db501eb217 am: 780e2cb944

Original change: https://android-review.googlesource.com/c/platform/packages/services/Iwlan/+/2233786

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

src/com/google/android/iwlan/IwlanDataService.java

diff --git a/src/com/google/android/iwlan/IwlanDataService.java b/src/com/google/android/iwlan/IwlanDataService.java
index 4811bdd..89604aa 100644
--- a/src/com/google/android/iwlan/IwlanDataService.java
+++ b/src/com/google/android/iwlan/IwlanDataService.java
@@ -209,6 +209,7 @@
             // ideally it should be 1280 - tunnelling overhead ?
             private static final int LINK_MTU =
                     1280; // TODO: need to substract tunnelling overhead?
+            private static final int LINK_MTU_CST = 1200; // Reserve 80 bytes for VCN.
             static final int TUNNEL_DOWN = 1;
             static final int TUNNEL_IN_BRINGUP = 2;
             static final int TUNNEL_UP = 3;
@@ -235,7 +236,11 @@
             }
 
             public int getLinkMtu() {
-                return LINK_MTU; // TODO: need to substract tunnelling overhead
+                if ((sDefaultDataTransport == Transport.MOBILE) && sNetworkConnected) {
+                    return LINK_MTU_CST;
+                } else {
+                    return LINK_MTU; // TODO: need to substract tunnelling overhead
+                }
             }
 
             public void setProtocolType(int protocolType) {