grpc 1.51.0 → 1.52.0.pre2

data/src/core/ext/transport/chttp2/transport/context_list.cc

@@ -1,20 +1,20 @@
-/*
- *
- * Copyright 2018 gRPC authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- *
- */
+//
+//
+// Copyright 2018 gRPC authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+//
+//
 
 #include <grpc/support/port_platform.h>
 
@@ -36,7 +36,7 @@ void ContextList::Append(ContextList** head, grpc_chttp2_stream* s) {
       write_timestamps_callback_g == nullptr) {
     return;
   }
-  /* Create a new element in the list and add it at the front */
+  // Create a new element in the list and add it at the front
   ContextList* elem = new ContextList();
   elem->trace_context_ = get_copied_context_fn_g(s->context);
   elem->byte_offset_ = s->byte_counter;
@@ -68,4 +68,4 @@ void grpc_http2_set_write_timestamps_callback(
 void grpc_http2_set_fn_get_copied_context(void* (*fn)(void*)) {
   get_copied_context_fn_g = fn;
 }
-} /* namespace grpc_core */
+}  // namespace grpc_core

data/src/core/ext/transport/chttp2/transport/context_list.h

@@ -1,20 +1,20 @@
-/*
- *
- * Copyright 2018 gRPC authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- *
- */
+//
+//
+// Copyright 2018 gRPC authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+//
+//
 
 #ifndef GRPC_CORE_EXT_TRANSPORT_CHTTP2_TRANSPORT_CONTEXT_LIST_H
 #define GRPC_CORE_EXT_TRANSPORT_CHTTP2_TRANSPORT_CONTEXT_LIST_H
@@ -28,16 +28,16 @@
 #include "src/core/lib/iomgr/error.h"
 
 namespace grpc_core {
-/** A list of RPC Contexts */
+/// A list of RPC Contexts
 class ContextList {
  public:
-  /* Creates a new element with \a context as the value and appends it to the
-   * list. */
+  // Creates a new element with \a context as the value and appends it to the
+  // list.
   static void Append(ContextList** head, grpc_chttp2_stream* s);
 
-  /* Executes a function \a fn with each context in the list and \a ts. It also
-   * frees up the entire list after this operation. It is intended as a callback
-   * and hence does not take a ref on \a error */
+  // Executes a function \a fn with each context in the list and \a ts. It also
+  // frees up the entire list after this operation. It is intended as a callback
+  // and hence does not take a ref on \a error
   static void Execute(void* arg, Timestamps* ts, grpc_error_handle error);
 
  private:
@@ -49,6 +49,6 @@ class ContextList {
 void grpc_http2_set_write_timestamps_callback(
     void (*fn)(void*, Timestamps*, grpc_error_handle error));
 void grpc_http2_set_fn_get_copied_context(void* (*fn)(void*));
-} /* namespace grpc_core */
+}  // namespace grpc_core
 
-#endif /* GRPC_CORE_EXT_TRANSPORT_CHTTP2_TRANSPORT_CONTEXT_LIST_H */
+#endif  // GRPC_CORE_EXT_TRANSPORT_CHTTP2_TRANSPORT_CONTEXT_LIST_H

data/src/core/ext/transport/chttp2/transport/decode_huff.cc

@@ -100,13 +100,6 @@ const uint8_t HuffDecoderCommon::table1_0_outer_[256] = {
     48, 49, 49, 50, 50, 51, 51, 52, 52, 53, 53, 54, 54, 55, 55, 56, 56, 57, 57,
     58, 58, 59, 59, 60, 60, 61, 61, 62, 62, 63, 63, 64, 64, 65, 65, 66, 66, 67,
     67, 68, 69, 70, 71, 72, 73, 74, 75};
-const uint8_t HuffDecoderCommon::table5_0_emit_[4] = {0x21, 0x22, 0x28, 0x29};
-const uint8_t HuffDecoderCommon::table5_0_inner_[4] = {0x02, 0x06, 0x0a, 0x0e};
-const uint8_t HuffDecoderCommon::table5_0_outer_[4] = {0, 1, 2, 3};
-const uint8_t HuffDecoderCommon::table7_0_emit_[1] = {0x3f};
-const uint8_t HuffDecoderCommon::table7_0_inner_[3] = {0x02, 0x01, 0x00};
-const uint8_t HuffDecoderCommon::table7_0_outer_[4] = {0, 1, 1, 2};
-const uint8_t HuffDecoderCommon::table8_0_emit_[4] = {0x3f, 0x27, 0x2b, 0x7c};
 const uint8_t HuffDecoderCommon::table8_0_inner_[6] = {0x01, 0x02, 0x06,
                                                        0x0a, 0x0e, 0x00};
 const uint8_t HuffDecoderCommon::table8_0_outer_[8] = {0, 1, 2, 3, 4, 0, 0, 5};
@@ -158,11 +151,8 @@ const uint8_t* const HuffDecoderCommon::table6_ops_[2] = {
     table6_0_ops_,
     table6_1_ops_,
 };
-const uint8_t HuffDecoderCommon::table13_0_emit_[3] = {0x5c, 0xc3, 0xd0};
 const uint8_t HuffDecoderCommon::table13_0_inner_[5] = {0x02, 0x06, 0x0a, 0x01,
                                                         0x00};
-const uint8_t HuffDecoderCommon::table13_0_outer_[16] = {
-    0, 1, 2, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 4};
 const uint8_t HuffDecoderCommon::table14_0_emit_[11] = {
     0x5c, 0xc3, 0xd0, 0x80, 0x82, 0x83, 0xa2, 0xb8, 0xc2, 0xe0, 0xe2};
 const uint8_t HuffDecoderCommon::table14_0_ops_[32] = {
@@ -231,35 +221,16 @@ const uint8_t HuffDecoderCommon::table12_0_outer_[256] = {
     46, 47, 47, 48, 48, 49, 49, 50, 51, 52, 53, 54, 55, 56, 57, 58, 59, 60, 61,
     62, 63, 64, 65, 66, 67, 68, 69, 70, 71, 72, 73, 74, 75, 76, 77, 78, 79, 80,
     81, 82, 83, 84, 85, 86, 87, 88, 89};
-const uint8_t HuffDecoderCommon::table17_0_emit_[2] = {0x09, 0x8e};
-const uint8_t HuffDecoderCommon::table17_0_inner_[2] = {0x01, 0x03};
-const uint8_t HuffDecoderCommon::table17_0_outer_[2] = {0, 1};
-const uint8_t HuffDecoderCommon::table18_0_emit_[2] = {0x90, 0x91};
-const uint8_t HuffDecoderCommon::table18_0_inner_[2] = {0x01, 0x03};
-const uint8_t HuffDecoderCommon::table18_0_outer_[2] = {0, 1};
-const uint8_t HuffDecoderCommon::table19_0_emit_[2] = {0x94, 0x9f};
-const uint8_t HuffDecoderCommon::table20_0_emit_[2] = {0xab, 0xce};
-const uint8_t HuffDecoderCommon::table21_0_emit_[2] = {0xd7, 0xe1};
-const uint8_t HuffDecoderCommon::table22_0_emit_[2] = {0xec, 0xed};
-const uint8_t HuffDecoderCommon::table23_0_emit_[4] = {0xc7, 0xcf, 0xea, 0xeb};
-const uint8_t HuffDecoderCommon::table23_0_outer_[4] = {0, 1, 2, 3};
 const uint8_t HuffDecoderCommon::table24_0_emit_[8] = {0xc0, 0xc1, 0xc8, 0xc9,
                                                        0xca, 0xcd, 0xd2, 0xd5};
 const uint8_t HuffDecoderCommon::table24_0_inner_[8] = {0x03, 0x07, 0x0b, 0x0f,
                                                         0x13, 0x17, 0x1b, 0x1f};
-const uint8_t HuffDecoderCommon::table24_0_outer_[8] = {0, 1, 2, 3, 4, 5, 6, 7};
 const uint8_t HuffDecoderCommon::table25_0_emit_[16] = {
     0xd3, 0xd4, 0xd6, 0xdd, 0xde, 0xdf, 0xf1, 0xf4,
     0xf5, 0xf6, 0xf7, 0xf8, 0xfa, 0xfb, 0xfc, 0xfd};
 const uint8_t HuffDecoderCommon::table25_0_inner_[16] = {
     0x04, 0x0c, 0x14, 0x1c, 0x24, 0x2c, 0x34, 0x3c,
     0x44, 0x4c, 0x54, 0x5c, 0x64, 0x6c, 0x74, 0x7c};
-const uint8_t HuffDecoderCommon::table25_0_outer_[16] = {
-    0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15};
-const uint8_t HuffDecoderCommon::table27_0_emit_[1] = {0xfe};
-const uint8_t HuffDecoderCommon::table27_0_inner_[3] = {0x02, 0x01, 0x00};
-const uint8_t HuffDecoderCommon::table27_0_outer_[16] = {
-    0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 2};
 const uint8_t HuffDecoderCommon::table26_0_emit_[30] = {
     0xfe, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08, 0x0b, 0x0c,
     0x0e, 0x0f, 0x10, 0x11, 0x12, 0x13, 0x14, 0x15, 0x17, 0x18,
@@ -269,11 +240,6 @@ const uint16_t HuffDecoderCommon::table26_0_inner_[31] = {
     0x0085, 0x0095, 0x00a5, 0x00b5, 0x00c5, 0x00d5, 0x00e5, 0x00f5,
     0x0105, 0x0115, 0x0125, 0x0135, 0x0145, 0x0155, 0x0165, 0x0175,
     0x0185, 0x0195, 0x01a5, 0x01b5, 0x01c5, 0x01d5, 0x000d};
-const uint8_t HuffDecoderCommon::table26_0_outer_[32] = {
-    0,  0,  1,  2,  3,  4,  5,  6,  7,  8,  9,  10, 11, 12, 13, 14,
-    15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30};
-const uint8_t HuffDecoderCommon::table28_0_emit_[3] = {0x0a, 0x0d, 0x16};
-const uint8_t HuffDecoderCommon::table28_0_inner_[4] = {0x02, 0x0a, 0x12, 0x06};
 const uint8_t HuffDecoderCommon::table30_0_emit_[7] = {0xda, 0xdb, 0xee, 0xf0,
                                                        0xf2, 0xf3, 0xff};
 const uint8_t HuffDecoderCommon::table30_0_inner_[8] = {0x02, 0x06, 0x0a, 0x0e,
@@ -282,6 +248,4 @@ const uint8_t HuffDecoderCommon::table29_0_emit_[9] = {
     0xda, 0xdb, 0xee, 0xf0, 0xf2, 0xf3, 0xff, 0xcb, 0xcc};
 const uint8_t HuffDecoderCommon::table29_0_inner_[9] = {
     0x03, 0x0b, 0x13, 0x1b, 0x23, 0x2b, 0x33, 0x3c, 0x44};
-const uint8_t HuffDecoderCommon::table29_0_outer_[16] = {
-    0, 0, 1, 1, 2, 2, 3, 3, 4, 4, 5, 5, 6, 6, 7, 8};
 }  // namespace grpc_core

data/src/core/ext/transport/chttp2/transport/decode_huff.h

@@ -47,22 +47,24 @@ class HuffDecoderCommon {
     return table1_0_emit_[emit];
   }
   static inline uint64_t GetOp5(size_t i) {
-    return table5_0_inner_[table5_0_outer_[i]];
+    return (i < 2 ? (i ? 6 : 2) : ((i - 2) ? 14 : 10));
   }
   static inline uint64_t GetEmit5(size_t, size_t emit) {
-    return table5_0_emit_[emit];
+    return (emit < 2 ? (emit + 33) : ((emit - 2) + 40));
   }
   static inline uint64_t GetOp7(size_t i) {
-    return table7_0_inner_[table7_0_outer_[i]];
+    return ((i < 2 ? (i) : ((i - 2) + 1)) < 1
+                ? (((void)(i < 2 ? (i) : ((i - 2) + 1)), 2))
+                : (((i < 2 ? (i) : ((i - 2) + 1)) - 1) ? 0 : 1));
   }
   static inline uint64_t GetEmit7(size_t, size_t emit) {
-    return table7_0_emit_[emit];
+    return ((void)emit, 63);
   }
   static inline uint64_t GetOp8(size_t i) {
     return table8_0_inner_[table8_0_outer_[i]];
   }
   static inline uint64_t GetEmit8(size_t, size_t emit) {
-    return table8_0_emit_[emit];
+    return (emit < 2 ? (emit ? 39 : 63) : ((emit - 2) ? 124 : 43));
   }
   static inline uint64_t GetOp9(size_t i) {
     return table9_0_inner_[table9_0_outer_[i]];
@@ -85,10 +87,10 @@ class HuffDecoderCommon {
     return table6_emit_[i >> 6][emit];
   }
   static inline uint64_t GetOp13(size_t i) {
-    return table13_0_inner_[table13_0_outer_[i]];
+    return table13_0_inner_[(i < 3 ? (i) : ((i - 3) / 12 + 3))];
   }
   static inline uint64_t GetEmit13(size_t, size_t emit) {
-    return table13_0_emit_[emit];
+    return (emit < 1 ? (((void)emit, 92)) : ((emit - 1) ? 208 : 195));
   }
   static inline uint64_t GetOp14(size_t i) { return table14_0_ops_[i]; }
   static inline uint64_t GetEmit14(size_t, size_t emit) {
@@ -108,86 +110,67 @@ class HuffDecoderCommon {
   static inline uint64_t GetEmit12(size_t, size_t emit) {
     return table12_0_emit_[emit];
   }
-  static inline uint64_t GetOp17(size_t i) {
-    return table17_0_inner_[table17_0_outer_[i]];
-  }
+  static inline uint64_t GetOp17(size_t i) { return i ? 3 : 1; }
   static inline uint64_t GetEmit17(size_t, size_t emit) {
-    return table17_0_emit_[emit];
-  }
-  static inline uint64_t GetOp18(size_t i) {
-    return table18_0_inner_[table18_0_outer_[i]];
-  }
-  static inline uint64_t GetEmit18(size_t, size_t emit) {
-    return table18_0_emit_[emit];
-  }
-  static inline uint64_t GetOp19(size_t i) {
-    return table17_0_inner_[table18_0_outer_[i]];
+    return emit ? 142 : 9;
   }
+  static inline uint64_t GetOp18(size_t i) { return i ? 3 : 1; }
+  static inline uint64_t GetEmit18(size_t, size_t emit) { return emit + 144; }
+  static inline uint64_t GetOp19(size_t i) { return i ? 3 : 1; }
   static inline uint64_t GetEmit19(size_t, size_t emit) {
-    return table19_0_emit_[emit];
-  }
-  static inline uint64_t GetOp20(size_t i) {
-    return table17_0_inner_[table18_0_outer_[i]];
+    return emit ? 159 : 148;
   }
+  static inline uint64_t GetOp20(size_t i) { return i ? 3 : 1; }
   static inline uint64_t GetEmit20(size_t, size_t emit) {
-    return table20_0_emit_[emit];
-  }
-  static inline uint64_t GetOp21(size_t i) {
-    return table17_0_inner_[table18_0_outer_[i]];
+    return emit ? 206 : 171;
   }
+  static inline uint64_t GetOp21(size_t i) { return i ? 3 : 1; }
   static inline uint64_t GetEmit21(size_t, size_t emit) {
-    return table21_0_emit_[emit];
-  }
-  static inline uint64_t GetOp22(size_t i) {
-    return table17_0_inner_[table18_0_outer_[i]];
-  }
-  static inline uint64_t GetEmit22(size_t, size_t emit) {
-    return table22_0_emit_[emit];
+    return emit ? 225 : 215;
   }
+  static inline uint64_t GetOp22(size_t i) { return i ? 3 : 1; }
+  static inline uint64_t GetEmit22(size_t, size_t emit) { return emit + 236; }
   static inline uint64_t GetOp23(size_t i) {
-    return table5_0_inner_[table23_0_outer_[i]];
+    return (i < 2 ? (i ? 6 : 2) : ((i - 2) ? 14 : 10));
   }
   static inline uint64_t GetEmit23(size_t, size_t emit) {
-    return table23_0_emit_[emit];
-  }
-  static inline uint64_t GetOp24(size_t i) {
-    return table24_0_inner_[table24_0_outer_[i]];
+    return (emit < 2 ? (emit ? 207 : 199) : ((emit - 2) + 234));
   }
+  static inline uint64_t GetOp24(size_t i) { return table24_0_inner_[i]; }
   static inline uint64_t GetEmit24(size_t, size_t emit) {
     return table24_0_emit_[emit];
   }
-  static inline uint64_t GetOp25(size_t i) {
-    return table25_0_inner_[table25_0_outer_[i]];
-  }
+  static inline uint64_t GetOp25(size_t i) { return table25_0_inner_[i]; }
   static inline uint64_t GetEmit25(size_t, size_t emit) {
     return table25_0_emit_[emit];
   }
   static inline uint64_t GetOp27(size_t i) {
-    return table27_0_inner_[table27_0_outer_[i]];
+    return (
+        (i < 1 ? (((void)i, 0)) : ((i - 1) / 14 + 1)) < 1
+            ? (((void)(i < 1 ? (((void)i, 0)) : ((i - 1) / 14 + 1)), 2))
+            : (((i < 1 ? (((void)i, 0)) : ((i - 1) / 14 + 1)) - 1) ? 0 : 1));
   }
   static inline uint64_t GetEmit27(size_t, size_t emit) {
-    return table27_0_emit_[emit];
+    return ((void)emit, 254);
   }
   static inline uint64_t GetOp26(size_t i) {
-    return table26_0_inner_[table26_0_outer_[i]];
+    return table26_0_inner_[(i < 1 ? (((void)i, 0)) : ((i - 1)))];
   }
   static inline uint64_t GetEmit26(size_t, size_t emit) {
     return table26_0_emit_[emit];
   }
   static inline uint64_t GetOp28(size_t i) {
-    return table28_0_inner_[table5_0_outer_[i]];
+    return (i < 2 ? (i ? 10 : 2) : ((i - 2) ? 6 : 18));
   }
   static inline uint64_t GetEmit28(size_t, size_t emit) {
-    return table28_0_emit_[emit];
-  }
-  static inline uint64_t GetOp30(size_t i) {
-    return table30_0_inner_[table24_0_outer_[i]];
+    return (emit < 1 ? (((void)emit, 10)) : ((emit - 1) ? 22 : 13));
   }
+  static inline uint64_t GetOp30(size_t i) { return table30_0_inner_[i]; }
   static inline uint64_t GetEmit30(size_t, size_t emit) {
     return table30_0_emit_[emit];
   }
   static inline uint64_t GetOp29(size_t i) {
-    return table29_0_inner_[table29_0_outer_[i]];
+    return table29_0_inner_[(i < 13 ? (i / 2 + 0) : ((i - 13) + 6))];
   }
   static inline uint64_t GetEmit29(size_t, size_t emit) {
     return table29_0_emit_[emit];
@@ -207,13 +190,6 @@ class HuffDecoderCommon {
   static const uint8_t table1_0_emit_[74];
   static const uint16_t table1_0_inner_[76];
   static const uint8_t table1_0_outer_[256];
-  static const uint8_t table5_0_emit_[4];
-  static const uint8_t table5_0_inner_[4];
-  static const uint8_t table5_0_outer_[4];
-  static const uint8_t table7_0_emit_[1];
-  static const uint8_t table7_0_inner_[3];
-  static const uint8_t table7_0_outer_[4];
-  static const uint8_t table8_0_emit_[4];
   static const uint8_t table8_0_inner_[6];
   static const uint8_t table8_0_outer_[8];
   static const uint8_t table9_0_emit_[6];
@@ -229,9 +205,7 @@ class HuffDecoderCommon {
   static const uint8_t table6_1_ops_[64];
   static const uint8_t* const table6_emit_[2];
   static const uint8_t* const table6_ops_[2];
-  static const uint8_t table13_0_emit_[3];
   static const uint8_t table13_0_inner_[5];
-  static const uint8_t table13_0_outer_[16];
   static const uint8_t table14_0_emit_[11];
   static const uint8_t table14_0_ops_[32];
   static const uint8_t table15_0_emit_[24];
@@ -241,37 +215,16 @@ class HuffDecoderCommon {
   static const uint8_t table12_0_emit_[79];
   static const uint16_t table12_0_inner_[90];
   static const uint8_t table12_0_outer_[256];
-  static const uint8_t table17_0_emit_[2];
-  static const uint8_t table17_0_inner_[2];
-  static const uint8_t table17_0_outer_[2];
-  static const uint8_t table18_0_emit_[2];
-  static const uint8_t table18_0_inner_[2];
-  static const uint8_t table18_0_outer_[2];
-  static const uint8_t table19_0_emit_[2];
-  static const uint8_t table20_0_emit_[2];
-  static const uint8_t table21_0_emit_[2];
-  static const uint8_t table22_0_emit_[2];
-  static const uint8_t table23_0_emit_[4];
-  static const uint8_t table23_0_outer_[4];
   static const uint8_t table24_0_emit_[8];
   static const uint8_t table24_0_inner_[8];
-  static const uint8_t table24_0_outer_[8];
   static const uint8_t table25_0_emit_[16];
   static const uint8_t table25_0_inner_[16];
-  static const uint8_t table25_0_outer_[16];
-  static const uint8_t table27_0_emit_[1];
-  static const uint8_t table27_0_inner_[3];
-  static const uint8_t table27_0_outer_[16];
   static const uint8_t table26_0_emit_[30];
   static const uint16_t table26_0_inner_[31];
-  static const uint8_t table26_0_outer_[32];
-  static const uint8_t table28_0_emit_[3];
-  static const uint8_t table28_0_inner_[4];
   static const uint8_t table30_0_emit_[7];
   static const uint8_t table30_0_inner_[8];
   static const uint8_t table29_0_emit_[9];
   static const uint8_t table29_0_inner_[9];
-  static const uint8_t table29_0_outer_[16];
 };
 template <typename F>
 class HuffDecoder : public HuffDecoderCommon {

data/src/core/ext/transport/chttp2/transport/flow_control.cc

@@ -1,20 +1,20 @@
-/*
- *
- * Copyright 2017 gRPC authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- *
- */
+//
+//
+// Copyright 2017 gRPC authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+//
+//
 
 #include <grpc/support/port_platform.h>
 
@@ -27,6 +27,7 @@
 #include <cmath>
 #include <ostream>
 #include <string>
+#include <tuple>
 #include <vector>
 
 #include "absl/strings/str_cat.h"
@@ -122,7 +123,7 @@ uint32_t TransportFlowControl::MaybeSendUpdate(bool writing_anyway) {
       announced_window_ != target_announced_window) {
     const uint32_t announce =
         static_cast<uint32_t>(Clamp(target_announced_window - announced_window_,
-                                    int64_t(0), kMaxWindowUpdateSize));
+                                    int64_t{0}, kMaxWindowUpdateSize));
     announced_window_ += announce;
     return announce;
   }
@@ -255,7 +256,7 @@ TransportFlowControl::TargetInitialWindowSizeBasedOnMemoryPressureAndBdp()
   //                                                                pressure
   const double kAnythingGoesPressure = 0.2;
   const double kAdjustedToBdpPressure = 0.5;
-  const double kAnythingGoesWindow = std::max(double(1 << 24), bdp);
+  const double kAnythingGoesWindow = std::max(double{1 << 24}, bdp);
   if (memory_pressure < kAnythingGoesPressure) {
     return kAnythingGoesWindow;
   } else if (memory_pressure < kAdjustedToBdpPressure) {
@@ -278,16 +279,21 @@ void TransportFlowControl::UpdateSetting(
         Clamp(new_desired_value, grpc_chttp2_settings_parameters[id].min_value,
               grpc_chttp2_settings_parameters[id].max_value);
     if (new_desired_value != *desired_value) {
-      *desired_value = new_desired_value;
+      if (grpc_flowctl_trace.enabled()) {
+        gpr_log(GPR_INFO, "[flowctl] UPDATE SETTING %s from %" PRId64 " to %d",
+                grpc_chttp2_settings_parameters[id].name, *desired_value,
+                new_desired_value);
+      }
       // Reaching zero can only happen for initial window size, and if it occurs
       // we really want to wake up writes and ensure all the queued stream
       // window updates are flushed, since stream flow control operates
       // differently at zero window size.
       FlowControlAction::Urgency urgency =
           FlowControlAction::Urgency::QUEUE_UPDATE;
-      if (new_desired_value == 0) {
+      if (*desired_value == 0 || new_desired_value == 0) {
         urgency = FlowControlAction::Urgency::UPDATE_IMMEDIATELY;
       }
+      *desired_value = new_desired_value;
       (action->*set)(urgency, *desired_value);
     }
   } else {
@@ -302,6 +308,21 @@ void TransportFlowControl::UpdateSetting(
   }
 }
 
+FlowControlAction TransportFlowControl::SetAckedInitialWindow(uint32_t value) {
+  acked_init_window_ = value;
+  FlowControlAction action;
+  if (IsFlowControlFixesEnabled() &&
+      acked_init_window_ != target_initial_window_size_) {
+    FlowControlAction::Urgency urgency =
+        FlowControlAction::Urgency::QUEUE_UPDATE;
+    if (acked_init_window_ == 0 || target_initial_window_size_ == 0) {
+      urgency = FlowControlAction::Urgency::UPDATE_IMMEDIATELY;
+    }
+    action.set_send_initial_window_update(urgency, target_initial_window_size_);
+  }
+  return action;
+}
+
 FlowControlAction TransportFlowControl::PeriodicUpdate() {
   FlowControlAction action;
   if (enable_bdp_probe_) {
@@ -347,25 +368,42 @@ FlowControlAction TransportFlowControl::PeriodicUpdate() {
       }
       // Though initial window 'could' drop to 0, we keep the floor at
       // kMinInitialWindowSize
-      UpdateSetting(
-          GRPC_CHTTP2_SETTINGS_INITIAL_WINDOW_SIZE,
-          &target_initial_window_size_,
-          static_cast<int32_t>(Clamp(target, double(kMinInitialWindowSize),
-                                     double(kMaxInitialWindowSize))),
-          &action, &FlowControlAction::set_send_initial_window_update);
+      UpdateSetting(GRPC_CHTTP2_SETTINGS_INITIAL_WINDOW_SIZE,
+                    &target_initial_window_size_,
+                    static_cast<int32_t>(Clamp(
+                        target, static_cast<double>(kMinInitialWindowSize),
+                        static_cast<double>(kMaxInitialWindowSize))),
+                    &action,
+                    &FlowControlAction::set_send_initial_window_update);
       // get bandwidth estimate and update max_frame accordingly.
       double bw_dbl = bdp_estimator_.EstimateBandwidth();
       // we target the max of BDP or bandwidth in microseconds.
       UpdateSetting(
           GRPC_CHTTP2_SETTINGS_MAX_FRAME_SIZE, &target_frame_size_,
-          static_cast<int32_t>(Clamp(
-              std::max(
-                  static_cast<int32_t>(Clamp(bw_dbl, 0.0, double(INT_MAX))) /
-                      1000,
-                  static_cast<int32_t>(target_initial_window_size_)),
-              16384, 16777215)),
+          static_cast<int32_t>(
+              Clamp(std::max(static_cast<int32_t>(Clamp(
+                                 bw_dbl, 0.0, static_cast<double>(INT_MAX))) /
+                                 1000,
+                             static_cast<int32_t>(target_initial_window_size_)),
+                    16384, 16777215)),
           &action, &FlowControlAction::set_send_max_frame_size_update);
     }
+
+    if (IsTcpFrameSizeTuningEnabled()) {
+      // Advertise PREFERRED_RECEIVE_CRYPTO_FRAME_SIZE to peer. By advertising
+      // PREFERRED_RECEIVE_CRYPTO_FRAME_SIZE to the peer, we are informing the
+      // peer that we have tcp frame size tuning enabled and we inform it of our
+      // prefered rx frame sizes. The prefered rx frame size is determined as:
+      // Clamp(target_frame_size_ * 2, 16384, 0x7fffffff). In the future, this
+      // maybe updated to a different function of the memory pressure.
+      UpdateSetting(
+          GRPC_CHTTP2_SETTINGS_GRPC_PREFERRED_RECEIVE_CRYPTO_FRAME_SIZE,
+          &target_preferred_rx_crypto_frame_size_,
+          Clamp(static_cast<unsigned int>(target_frame_size_ * 2), 16384u,
+                0x7ffffffu),
+          &action,
+          &FlowControlAction::set_preferred_rx_crypto_frame_size_update);
+    }
   }
   return UpdateAction(action);
 }
@@ -376,7 +414,7 @@ uint32_t StreamFlowControl::MaybeSendUpdate() {
   pending_size_ = absl::nullopt;
   tfc_upd.UpdateAnnouncedWindowDelta(&announced_window_delta_, announce);
   GPR_ASSERT(DesiredAnnounceSize() == 0);
-  tfc_upd.MakeAction();
+  std::ignore = tfc_upd.MakeAction();
   return static_cast<uint32_t>(announce);
 }
 
@@ -393,7 +431,7 @@ int64_t StreamFlowControl::DesiredAnnounceSize() const {
       return std::min(min_progress_size_, kMaxWindowDelta);
     }
   }();
-  return Clamp(desired_window_delta - announced_window_delta_, int64_t(0),
+  return Clamp(desired_window_delta - announced_window_delta_, int64_t{0},
                kMaxWindowUpdateSize);
 }
 
@@ -407,9 +445,8 @@ FlowControlAction StreamFlowControl::UpdateAction(FlowControlAction action) {
     // Currently set at half the initial window size or 8kb (whichever is
     // greater). 8kb means we don't send rapidly unnecessarily when the initial
     // window size is small.
-    const int64_t hurry_up_size =
-        std::max(static_cast<int64_t>(tfc_->sent_init_window()) / 2,
-                 static_cast<int64_t>(8192));
+    const int64_t hurry_up_size = std::max(
+        static_cast<int64_t>(tfc_->sent_init_window()) / 2, int64_t{8192});
     if (desired_announce_size > hurry_up_size) {
       urgency = FlowControlAction::Urgency::UPDATE_IMMEDIATELY;
     }