aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMathiasRaoul <mathias.raoul@gmail.com>2020-02-07 16:29:05 +0000
committerFlorin Coras <florin.coras@gmail.com>2020-03-27 17:43:33 +0000
commitffdc72da4f086e9a62b946970778495bba400e69 (patch)
tree3633cea2b6aa5f862eccc85a7e80d9db66d6aa96
parent4e149776890a5ac91bb14957d57def3c73325061 (diff)
quic: Check quicly version tag at compile time
- updates the quicly version to 0.1.0-vpp - adds workaround for quicly_send()/assert_consistency() failure Type: feature Change-Id: I4c7e0ffc720ad9a685b89046a83646d59febd6cd Signed-off-by: MathiasRaoul <mathias.raoul@gmail.com> Signed-off-by: Nathan Skrzypczak <nathan.skrzypczak@gmail.com> Signed-off-by: Dave Wallace <dwallacelf@gmail.com>
-rw-r--r--build/external/packages/quicly.mk4
-rw-r--r--build/external/patches/quicly_0.0.10-vpp/0001-cmake-install.patch60
-rw-r--r--build/external/patches/quicly_0.0.3-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch33
-rw-r--r--build/external/patches/quicly_0.0.3-vpp/0001-cmake-install.patch57
-rw-r--r--build/external/patches/quicly_0.0.5-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch33
-rw-r--r--build/external/patches/quicly_0.0.5-vpp/0001-cmake-install.patch60
-rw-r--r--build/external/patches/quicly_0.0.5-vpp/0002-quicly-rtt-time-skew.patch16
-rw-r--r--build/external/patches/quicly_0.0.6-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch33
-rw-r--r--build/external/patches/quicly_0.0.6-vpp/0001-cmake-install.patch60
-rw-r--r--build/external/patches/quicly_0.0.7-vpp/0001-cmake-install.patch60
-rw-r--r--build/external/patches/quicly_0.0.8-vpp/0001-cmake-install.patch60
-rw-r--r--build/external/patches/quicly_0.0.9-vpp/0001-cmake-install.patch60
-rw-r--r--build/external/patches/quicly_0.1.0-vpp/0001-quicly-send-assert-consistency.patch21
-rw-r--r--src/plugins/quic/CMakeLists.txt42
-rw-r--r--src/plugins/quic/quic.c18
15 files changed, 56 insertions, 561 deletions
diff --git a/build/external/packages/quicly.mk b/build/external/packages/quicly.mk
index 9d6a4281c5b..29abad8b56c 100644
--- a/build/external/packages/quicly.mk
+++ b/build/external/packages/quicly.mk
@@ -11,9 +11,9 @@
# See the License for the specific language governing permissions and
# limitations under the License.
-quicly_version := 0.0.10-vpp
+quicly_version := 0.1.0-vpp
quicly_tarball := quicly_$(quicly_version).tar.gz
-quicly_tarball_md5sum := 771ba05c1724ce0b56deaaaa62eb48f1
+quicly_tarball_md5sum := 223f62c4dda7cbb1d907956cafcfd3f0
quicly_tarball_strip_dirs := 1
quicly_url := https://github.com/vpp-quic/quicly/releases/download/v$(quicly_version)/quicly_$(quicly_version).tar.gz
diff --git a/build/external/patches/quicly_0.0.10-vpp/0001-cmake-install.patch b/build/external/patches/quicly_0.0.10-vpp/0001-cmake-install.patch
deleted file mode 100644
index 0ca7d2986d4..00000000000
--- a/build/external/patches/quicly_0.0.10-vpp/0001-cmake-install.patch
+++ /dev/null
@@ -1,60 +0,0 @@
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index f5478d3..156df4c 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -65,6 +65,8 @@ IF (WITH_DTRACE)
- ENDIF ()
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
-+
- ADD_LIBRARY(quicly ${QUICLY_LIBRARY_FILES})
-
- ADD_CUSTOM_COMMAND(
-@@ -84,6 +86,16 @@ TARGET_LINK_LIBRARIES(examples-echo quicly ${OPENSSL_LIBRARIES} ${CMAKE_DL_LIBS}
-
- ADD_EXECUTABLE(udpfw t/udpfw.c)
-
-+INSTALL (
-+ DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS quicly
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} WITH_DTRACE=${WITH_DTRACE} prove --exec "sh -c" -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
- DEPENDS cli test.t)
-Submodule deps/picotls contains untracked content
-Submodule deps/picotls contains modified content
-diff --git a/deps/picotls/CMakeLists.txt b/deps/picotls/CMakeLists.txt
-index 874b9be..14411e2 100644
---- a/deps/picotls/CMakeLists.txt
-+++ b/deps/picotls/CMakeLists.txt
-@@ -12,6 +12,7 @@ IF (WITH_DTRACE)
- MESSAGE(STATUS "Enabling USDT support")
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
- SET(CMAKE_C_FLAGS "-std=c99 -Wall -O2 -g ${CC_WARNING_FLAGS} ${CMAKE_C_FLAGS}")
- INCLUDE_DIRECTORIES(
- deps/cifra/src/ext
-@@ -131,6 +132,15 @@ ELSE ()
- MESSAGE(WARNING "Disabling OpenSSL support (requires 1.0.1 or newer)")
- ENDIF ()
-
-+INSTALL (DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS picotls-core picotls-openssl
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} prove --exec '' -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} DEPENDS ${TEST_EXES} cli)
-
- IF (CMAKE_SYSTEM_NAME STREQUAL "Linux")
diff --git a/build/external/patches/quicly_0.0.3-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch b/build/external/patches/quicly_0.0.3-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch
deleted file mode 100644
index 80f3bcb684f..00000000000
--- a/build/external/patches/quicly_0.0.3-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch
+++ /dev/null
@@ -1,33 +0,0 @@
-diff --git a/deps/picotls/include/picotls.h b/deps/picotls/include/picotls.h
-index 06d62f9..de33b86 100644
---- a/deps/picotls/include/picotls.h
-+++ b/deps/picotls/include/picotls.h
-@@ -276,6 +276,8 @@ typedef struct st_ptls_aead_context_t {
- size_t (*do_encrypt_final)(struct st_ptls_aead_context_t *ctx, void *output);
- size_t (*do_decrypt)(struct st_ptls_aead_context_t *ctx, void *output, const void *input, size_t inlen, const void *iv,
- const void *aad, size_t aadlen);
-+ size_t (*do_encrypt)(struct st_ptls_aead_context_t *ctx, void *output, const void *input, size_t inlen, uint64_t seq,
-+ const void *iv, const void *aad, size_t aadlen);
- } ptls_aead_context_t;
-
- /**
-diff --git a/deps/picotls/lib/picotls.c b/deps/picotls/lib/picotls.c
-index 70d2fef..f98f7b4 100644
---- a/deps/picotls/lib/picotls.c
-+++ b/deps/picotls/lib/picotls.c
-@@ -4890,6 +4890,13 @@ size_t ptls_aead_encrypt(ptls_aead_context_t *ctx, void *output, const void *inp
- {
- size_t off = 0;
-
-+ if(ctx->do_encrypt)
-+ {
-+ uint8_t iv[PTLS_MAX_IV_SIZE];
-+ ptls_aead__build_iv(ctx, iv, seq);
-+ return ctx->do_encrypt(ctx, output, input, inlen, seq, iv, aad, aadlen);
-+ }
-+
- ptls_aead_encrypt_init(ctx, seq, aad, aadlen);
- off += ptls_aead_encrypt_update(ctx, ((uint8_t *)output) + off, input, inlen);
- off += ptls_aead_encrypt_final(ctx, ((uint8_t *)output) + off);
-
-
diff --git a/build/external/patches/quicly_0.0.3-vpp/0001-cmake-install.patch b/build/external/patches/quicly_0.0.3-vpp/0001-cmake-install.patch
deleted file mode 100644
index 3ac90c7d0aa..00000000000
--- a/build/external/patches/quicly_0.0.3-vpp/0001-cmake-install.patch
+++ /dev/null
@@ -1,57 +0,0 @@
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index 202cc52..b5c2bee 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -40,6 +40,8 @@ SET(UNITTEST_SOURCE_FILES
- t/stream-concurrency.c
- t/test.c)
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
-+
- ADD_LIBRARY(quicly ${QUICLY_LIBRARY_FILES})
-
- ADD_EXECUTABLE(cli ${PICOTLS_OPENSSL_FILES} src/cli.c)
-@@ -50,6 +52,16 @@ TARGET_LINK_LIBRARIES(test.t quicly ${OPENSSL_LIBRARIES} ${CMAKE_DL_LIBS})
-
- ADD_EXECUTABLE(udpfw t/udpfw.c)
-
-+INSTALL (
-+ DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS quicly
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} prove --exec "sh -c" -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
- DEPENDS cli test.t)
-Submodule deps/picotls contains untracked content
-Submodule deps/picotls contains modified content
-diff --git a/deps/picotls/CMakeLists.txt b/deps/picotls/CMakeLists.txt
-index cda6aad..62b23b7 100644
---- a/deps/picotls/CMakeLists.txt
-+++ b/deps/picotls/CMakeLists.txt
-@@ -5,6 +5,7 @@ PROJECT(picotls)
-
- FIND_PACKAGE(PkgConfig REQUIRED)
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
- SET(CMAKE_C_FLAGS "-std=c99 -Wall -O2 -g ${CC_WARNING_FLAGS} ${CMAKE_C_FLAGS}")
- INCLUDE_DIRECTORIES(${OPENSSL_INCLUDE_DIR} deps/cifra/src/ext deps/cifra/src deps/micro-ecc deps/picotest include)
- SET(MINICRYPTO_LIBRARY_FILES
-@@ -97,3 +98,12 @@ IF (BUILD_FUZZER)
-
-
- ENDIF()
-+
-+INSTALL (DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS picotls-core picotls-openssl
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
diff --git a/build/external/patches/quicly_0.0.5-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch b/build/external/patches/quicly_0.0.5-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch
deleted file mode 100644
index 80f3bcb684f..00000000000
--- a/build/external/patches/quicly_0.0.5-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch
+++ /dev/null
@@ -1,33 +0,0 @@
-diff --git a/deps/picotls/include/picotls.h b/deps/picotls/include/picotls.h
-index 06d62f9..de33b86 100644
---- a/deps/picotls/include/picotls.h
-+++ b/deps/picotls/include/picotls.h
-@@ -276,6 +276,8 @@ typedef struct st_ptls_aead_context_t {
- size_t (*do_encrypt_final)(struct st_ptls_aead_context_t *ctx, void *output);
- size_t (*do_decrypt)(struct st_ptls_aead_context_t *ctx, void *output, const void *input, size_t inlen, const void *iv,
- const void *aad, size_t aadlen);
-+ size_t (*do_encrypt)(struct st_ptls_aead_context_t *ctx, void *output, const void *input, size_t inlen, uint64_t seq,
-+ const void *iv, const void *aad, size_t aadlen);
- } ptls_aead_context_t;
-
- /**
-diff --git a/deps/picotls/lib/picotls.c b/deps/picotls/lib/picotls.c
-index 70d2fef..f98f7b4 100644
---- a/deps/picotls/lib/picotls.c
-+++ b/deps/picotls/lib/picotls.c
-@@ -4890,6 +4890,13 @@ size_t ptls_aead_encrypt(ptls_aead_context_t *ctx, void *output, const void *inp
- {
- size_t off = 0;
-
-+ if(ctx->do_encrypt)
-+ {
-+ uint8_t iv[PTLS_MAX_IV_SIZE];
-+ ptls_aead__build_iv(ctx, iv, seq);
-+ return ctx->do_encrypt(ctx, output, input, inlen, seq, iv, aad, aadlen);
-+ }
-+
- ptls_aead_encrypt_init(ctx, seq, aad, aadlen);
- off += ptls_aead_encrypt_update(ctx, ((uint8_t *)output) + off, input, inlen);
- off += ptls_aead_encrypt_final(ctx, ((uint8_t *)output) + off);
-
-
diff --git a/build/external/patches/quicly_0.0.5-vpp/0001-cmake-install.patch b/build/external/patches/quicly_0.0.5-vpp/0001-cmake-install.patch
deleted file mode 100644
index 0ca7d2986d4..00000000000
--- a/build/external/patches/quicly_0.0.5-vpp/0001-cmake-install.patch
+++ /dev/null
@@ -1,60 +0,0 @@
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index f5478d3..156df4c 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -65,6 +65,8 @@ IF (WITH_DTRACE)
- ENDIF ()
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
-+
- ADD_LIBRARY(quicly ${QUICLY_LIBRARY_FILES})
-
- ADD_CUSTOM_COMMAND(
-@@ -84,6 +86,16 @@ TARGET_LINK_LIBRARIES(examples-echo quicly ${OPENSSL_LIBRARIES} ${CMAKE_DL_LIBS}
-
- ADD_EXECUTABLE(udpfw t/udpfw.c)
-
-+INSTALL (
-+ DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS quicly
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} WITH_DTRACE=${WITH_DTRACE} prove --exec "sh -c" -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
- DEPENDS cli test.t)
-Submodule deps/picotls contains untracked content
-Submodule deps/picotls contains modified content
-diff --git a/deps/picotls/CMakeLists.txt b/deps/picotls/CMakeLists.txt
-index 874b9be..14411e2 100644
---- a/deps/picotls/CMakeLists.txt
-+++ b/deps/picotls/CMakeLists.txt
-@@ -12,6 +12,7 @@ IF (WITH_DTRACE)
- MESSAGE(STATUS "Enabling USDT support")
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
- SET(CMAKE_C_FLAGS "-std=c99 -Wall -O2 -g ${CC_WARNING_FLAGS} ${CMAKE_C_FLAGS}")
- INCLUDE_DIRECTORIES(
- deps/cifra/src/ext
-@@ -131,6 +132,15 @@ ELSE ()
- MESSAGE(WARNING "Disabling OpenSSL support (requires 1.0.1 or newer)")
- ENDIF ()
-
-+INSTALL (DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS picotls-core picotls-openssl
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} prove --exec '' -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} DEPENDS ${TEST_EXES} cli)
-
- IF (CMAKE_SYSTEM_NAME STREQUAL "Linux")
diff --git a/build/external/patches/quicly_0.0.5-vpp/0002-quicly-rtt-time-skew.patch b/build/external/patches/quicly_0.0.5-vpp/0002-quicly-rtt-time-skew.patch
deleted file mode 100644
index 589d8cde125..00000000000
--- a/build/external/patches/quicly_0.0.5-vpp/0002-quicly-rtt-time-skew.patch
+++ /dev/null
@@ -1,16 +0,0 @@
-diff --git a/lib/quicly.c b/lib/quicly.c
-index 95d5f13..3f4fd18 100644
---- a/lib/quicly.c
-+++ b/lib/quicly.c
-@@ -351,7 +351,10 @@ static __thread int64_t now;
-
- static void update_now(quicly_context_t *ctx)
- {
-- now = ctx->now->cb(ctx->now);
-+ int64_t newval = ctx->now->cb(ctx->now);
-+
-+ if (now < newval)
-+ now = newval;
- }
-
- /**
diff --git a/build/external/patches/quicly_0.0.6-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch b/build/external/patches/quicly_0.0.6-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch
deleted file mode 100644
index 80f3bcb684f..00000000000
--- a/build/external/patches/quicly_0.0.6-vpp/0001-Add-function-ptr-in-ptls-ctx-for-aead_encrypt.patch
+++ /dev/null
@@ -1,33 +0,0 @@
-diff --git a/deps/picotls/include/picotls.h b/deps/picotls/include/picotls.h
-index 06d62f9..de33b86 100644
---- a/deps/picotls/include/picotls.h
-+++ b/deps/picotls/include/picotls.h
-@@ -276,6 +276,8 @@ typedef struct st_ptls_aead_context_t {
- size_t (*do_encrypt_final)(struct st_ptls_aead_context_t *ctx, void *output);
- size_t (*do_decrypt)(struct st_ptls_aead_context_t *ctx, void *output, const void *input, size_t inlen, const void *iv,
- const void *aad, size_t aadlen);
-+ size_t (*do_encrypt)(struct st_ptls_aead_context_t *ctx, void *output, const void *input, size_t inlen, uint64_t seq,
-+ const void *iv, const void *aad, size_t aadlen);
- } ptls_aead_context_t;
-
- /**
-diff --git a/deps/picotls/lib/picotls.c b/deps/picotls/lib/picotls.c
-index 70d2fef..f98f7b4 100644
---- a/deps/picotls/lib/picotls.c
-+++ b/deps/picotls/lib/picotls.c
-@@ -4890,6 +4890,13 @@ size_t ptls_aead_encrypt(ptls_aead_context_t *ctx, void *output, const void *inp
- {
- size_t off = 0;
-
-+ if(ctx->do_encrypt)
-+ {
-+ uint8_t iv[PTLS_MAX_IV_SIZE];
-+ ptls_aead__build_iv(ctx, iv, seq);
-+ return ctx->do_encrypt(ctx, output, input, inlen, seq, iv, aad, aadlen);
-+ }
-+
- ptls_aead_encrypt_init(ctx, seq, aad, aadlen);
- off += ptls_aead_encrypt_update(ctx, ((uint8_t *)output) + off, input, inlen);
- off += ptls_aead_encrypt_final(ctx, ((uint8_t *)output) + off);
-
-
diff --git a/build/external/patches/quicly_0.0.6-vpp/0001-cmake-install.patch b/build/external/patches/quicly_0.0.6-vpp/0001-cmake-install.patch
deleted file mode 100644
index 0ca7d2986d4..00000000000
--- a/build/external/patches/quicly_0.0.6-vpp/0001-cmake-install.patch
+++ /dev/null
@@ -1,60 +0,0 @@
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index f5478d3..156df4c 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -65,6 +65,8 @@ IF (WITH_DTRACE)
- ENDIF ()
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
-+
- ADD_LIBRARY(quicly ${QUICLY_LIBRARY_FILES})
-
- ADD_CUSTOM_COMMAND(
-@@ -84,6 +86,16 @@ TARGET_LINK_LIBRARIES(examples-echo quicly ${OPENSSL_LIBRARIES} ${CMAKE_DL_LIBS}
-
- ADD_EXECUTABLE(udpfw t/udpfw.c)
-
-+INSTALL (
-+ DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS quicly
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} WITH_DTRACE=${WITH_DTRACE} prove --exec "sh -c" -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
- DEPENDS cli test.t)
-Submodule deps/picotls contains untracked content
-Submodule deps/picotls contains modified content
-diff --git a/deps/picotls/CMakeLists.txt b/deps/picotls/CMakeLists.txt
-index 874b9be..14411e2 100644
---- a/deps/picotls/CMakeLists.txt
-+++ b/deps/picotls/CMakeLists.txt
-@@ -12,6 +12,7 @@ IF (WITH_DTRACE)
- MESSAGE(STATUS "Enabling USDT support")
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
- SET(CMAKE_C_FLAGS "-std=c99 -Wall -O2 -g ${CC_WARNING_FLAGS} ${CMAKE_C_FLAGS}")
- INCLUDE_DIRECTORIES(
- deps/cifra/src/ext
-@@ -131,6 +132,15 @@ ELSE ()
- MESSAGE(WARNING "Disabling OpenSSL support (requires 1.0.1 or newer)")
- ENDIF ()
-
-+INSTALL (DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS picotls-core picotls-openssl
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} prove --exec '' -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} DEPENDS ${TEST_EXES} cli)
-
- IF (CMAKE_SYSTEM_NAME STREQUAL "Linux")
diff --git a/build/external/patches/quicly_0.0.7-vpp/0001-cmake-install.patch b/build/external/patches/quicly_0.0.7-vpp/0001-cmake-install.patch
deleted file mode 100644
index 0ca7d2986d4..00000000000
--- a/build/external/patches/quicly_0.0.7-vpp/0001-cmake-install.patch
+++ /dev/null
@@ -1,60 +0,0 @@
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index f5478d3..156df4c 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -65,6 +65,8 @@ IF (WITH_DTRACE)
- ENDIF ()
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
-+
- ADD_LIBRARY(quicly ${QUICLY_LIBRARY_FILES})
-
- ADD_CUSTOM_COMMAND(
-@@ -84,6 +86,16 @@ TARGET_LINK_LIBRARIES(examples-echo quicly ${OPENSSL_LIBRARIES} ${CMAKE_DL_LIBS}
-
- ADD_EXECUTABLE(udpfw t/udpfw.c)
-
-+INSTALL (
-+ DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS quicly
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} WITH_DTRACE=${WITH_DTRACE} prove --exec "sh -c" -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
- DEPENDS cli test.t)
-Submodule deps/picotls contains untracked content
-Submodule deps/picotls contains modified content
-diff --git a/deps/picotls/CMakeLists.txt b/deps/picotls/CMakeLists.txt
-index 874b9be..14411e2 100644
---- a/deps/picotls/CMakeLists.txt
-+++ b/deps/picotls/CMakeLists.txt
-@@ -12,6 +12,7 @@ IF (WITH_DTRACE)
- MESSAGE(STATUS "Enabling USDT support")
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
- SET(CMAKE_C_FLAGS "-std=c99 -Wall -O2 -g ${CC_WARNING_FLAGS} ${CMAKE_C_FLAGS}")
- INCLUDE_DIRECTORIES(
- deps/cifra/src/ext
-@@ -131,6 +132,15 @@ ELSE ()
- MESSAGE(WARNING "Disabling OpenSSL support (requires 1.0.1 or newer)")
- ENDIF ()
-
-+INSTALL (DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS picotls-core picotls-openssl
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} prove --exec '' -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} DEPENDS ${TEST_EXES} cli)
-
- IF (CMAKE_SYSTEM_NAME STREQUAL "Linux")
diff --git a/build/external/patches/quicly_0.0.8-vpp/0001-cmake-install.patch b/build/external/patches/quicly_0.0.8-vpp/0001-cmake-install.patch
deleted file mode 100644
index 0ca7d2986d4..00000000000
--- a/build/external/patches/quicly_0.0.8-vpp/0001-cmake-install.patch
+++ /dev/null
@@ -1,60 +0,0 @@
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index f5478d3..156df4c 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -65,6 +65,8 @@ IF (WITH_DTRACE)
- ENDIF ()
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
-+
- ADD_LIBRARY(quicly ${QUICLY_LIBRARY_FILES})
-
- ADD_CUSTOM_COMMAND(
-@@ -84,6 +86,16 @@ TARGET_LINK_LIBRARIES(examples-echo quicly ${OPENSSL_LIBRARIES} ${CMAKE_DL_LIBS}
-
- ADD_EXECUTABLE(udpfw t/udpfw.c)
-
-+INSTALL (
-+ DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS quicly
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} WITH_DTRACE=${WITH_DTRACE} prove --exec "sh -c" -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
- DEPENDS cli test.t)
-Submodule deps/picotls contains untracked content
-Submodule deps/picotls contains modified content
-diff --git a/deps/picotls/CMakeLists.txt b/deps/picotls/CMakeLists.txt
-index 874b9be..14411e2 100644
---- a/deps/picotls/CMakeLists.txt
-+++ b/deps/picotls/CMakeLists.txt
-@@ -12,6 +12,7 @@ IF (WITH_DTRACE)
- MESSAGE(STATUS "Enabling USDT support")
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
- SET(CMAKE_C_FLAGS "-std=c99 -Wall -O2 -g ${CC_WARNING_FLAGS} ${CMAKE_C_FLAGS}")
- INCLUDE_DIRECTORIES(
- deps/cifra/src/ext
-@@ -131,6 +132,15 @@ ELSE ()
- MESSAGE(WARNING "Disabling OpenSSL support (requires 1.0.1 or newer)")
- ENDIF ()
-
-+INSTALL (DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS picotls-core picotls-openssl
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} prove --exec '' -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} DEPENDS ${TEST_EXES} cli)
-
- IF (CMAKE_SYSTEM_NAME STREQUAL "Linux")
diff --git a/build/external/patches/quicly_0.0.9-vpp/0001-cmake-install.patch b/build/external/patches/quicly_0.0.9-vpp/0001-cmake-install.patch
deleted file mode 100644
index 0ca7d2986d4..00000000000
--- a/build/external/patches/quicly_0.0.9-vpp/0001-cmake-install.patch
+++ /dev/null
@@ -1,60 +0,0 @@
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index f5478d3..156df4c 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -65,6 +65,8 @@ IF (WITH_DTRACE)
- ENDIF ()
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
-+
- ADD_LIBRARY(quicly ${QUICLY_LIBRARY_FILES})
-
- ADD_CUSTOM_COMMAND(
-@@ -84,6 +86,16 @@ TARGET_LINK_LIBRARIES(examples-echo quicly ${OPENSSL_LIBRARIES} ${CMAKE_DL_LIBS}
-
- ADD_EXECUTABLE(udpfw t/udpfw.c)
-
-+INSTALL (
-+ DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS quicly
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} WITH_DTRACE=${WITH_DTRACE} prove --exec "sh -c" -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t
- WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR}
- DEPENDS cli test.t)
-Submodule deps/picotls contains untracked content
-Submodule deps/picotls contains modified content
-diff --git a/deps/picotls/CMakeLists.txt b/deps/picotls/CMakeLists.txt
-index 874b9be..14411e2 100644
---- a/deps/picotls/CMakeLists.txt
-+++ b/deps/picotls/CMakeLists.txt
-@@ -12,6 +12,7 @@ IF (WITH_DTRACE)
- MESSAGE(STATUS "Enabling USDT support")
- ENDIF ()
-
-+SET(CMAKE_POSITION_INDEPENDENT_CODE ON)
- SET(CMAKE_C_FLAGS "-std=c99 -Wall -O2 -g ${CC_WARNING_FLAGS} ${CMAKE_C_FLAGS}")
- INCLUDE_DIRECTORIES(
- deps/cifra/src/ext
-@@ -131,6 +132,15 @@ ELSE ()
- MESSAGE(WARNING "Disabling OpenSSL support (requires 1.0.1 or newer)")
- ENDIF ()
-
-+INSTALL (DIRECTORY ${CMAKE_SOURCE_DIR}/include/
-+ DESTINATION include
-+ FILES_MATCHING PATTERN "*.h")
-+
-+INSTALL(TARGETS picotls-core picotls-openssl
-+ RUNTIME DESTINATION bin
-+ LIBRARY DESTINATION lib
-+ ARCHIVE DESTINATION lib)
-+
- ADD_CUSTOM_TARGET(check env BINARY_DIR=${CMAKE_CURRENT_BINARY_DIR} prove --exec '' -v ${CMAKE_CURRENT_BINARY_DIR}/*.t t/*.t WORKING_DIRECTORY ${CMAKE_CURRENT_SOURCE_DIR} DEPENDS ${TEST_EXES} cli)
-
- IF (CMAKE_SYSTEM_NAME STREQUAL "Linux")
diff --git a/build/external/patches/quicly_0.1.0-vpp/0001-quicly-send-assert-consistency.patch b/build/external/patches/quicly_0.1.0-vpp/0001-quicly-send-assert-consistency.patch
new file mode 100644
index 00000000000..2e0caaf1c9b
--- /dev/null
+++ b/build/external/patches/quicly_0.1.0-vpp/0001-quicly-send-assert-consistency.patch
@@ -0,0 +1,21 @@
+diff --git a/lib/quicly.c b/lib/quicly.c
+index 71fa149..668677f 100644
+--- a/lib/quicly.c
++++ b/lib/quicly.c
+@@ -3631,10 +3631,14 @@ int quicly_send(quicly_conn_t *conn, quicly_datagram_t **packets, size_t *num_pa
+ * progress (i.e. due to the payload of lost packet being cancelled), then PTO for the previously sent packet. To accomodate
+ * that, we allow to rerun the do_send function just once.
+ */
+- if (s.num_packets == 0 && conn->egress.loss.alarm_at <= now) {
++ if (conn->egress.loss.alarm_at <= now) {
+ assert(conn->egress.loss.alarm_at == now);
+- if ((ret = do_send(conn, &s)) != 0)
+- return ret;
++ if (s.num_packets == 0) {
++ if ((ret = do_send(conn, &s)) != 0)
++ return ret;
++ } else {
++ conn->egress.loss.alarm_at = now + 1;
++ }
+ }
+ assert_consistency(conn, 1);
diff --git a/src/plugins/quic/CMakeLists.txt b/src/plugins/quic/CMakeLists.txt
index 43fb483a7bc..859b6d98c4c 100644
--- a/src/plugins/quic/CMakeLists.txt
+++ b/src/plugins/quic/CMakeLists.txt
@@ -12,8 +12,8 @@
# See the License for the specific language governing permissions and
# limitations under the License.
-message(STATUS "Looking for quicly")
unset(QUIC_LINK_LIBRARIES)
+set(EXPECTED_QUICLY_VERSION "0.1.0-vpp")
find_path(QUICLY_INCLUDE_DIR NAMES quicly.h)
find_path(PICOTLS_INCLUDE_DIR NAMES picotls.h)
@@ -28,22 +28,32 @@ list(APPEND QUIC_LINK_LIBRARIES
)
if(QUICLY_INCLUDE_DIR AND QUIC_LINK_LIBRARIES)
- include_directories (${QUICLY_INCLUDE_DIR})
-
- if(PICOTLS_INCLUDE_DIR)
- include_directories (${PICOTLS_INCLUDE_DIR})
+ if(EXISTS "${QUICLY_INCLUDE_DIR}/quicly/version.h")
+ file(STRINGS "${QUICLY_INCLUDE_DIR}/quicly/version.h" quicly_version_str REGEX "^#define[\t ]+LIBQUICLY_VERSION[\t ]+\".*\"")
+ string(REGEX REPLACE "^#define[\t ]+LIBQUICLY_VERSION[\t ]+\"([^\"]*)\".*" "\\1" QUICLY_VERSION_STRING "${quicly_version_str}")
+ unset(quicly_version_str)
endif()
- add_vpp_plugin(quic
- SOURCES
- certs.c
- error.c
- quic.c
- quic_crypto.c
-
- LINK_LIBRARIES ${QUIC_LINK_LIBRARIES}
- )
- message(STATUS "Found quicly in ${QUICLY_INCLUDE_DIR}")
+ if (${QUICLY_VERSION_STRING} MATCHES "${EXPECTED_QUICLY_VERSION}")
+ include_directories (${QUICLY_INCLUDE_DIR})
+
+ if(PICOTLS_INCLUDE_DIR)
+ include_directories (${PICOTLS_INCLUDE_DIR})
+ endif()
+
+ add_vpp_plugin(quic
+ SOURCES
+ certs.c
+ error.c
+ quic.c
+ quic_crypto.c
+
+ LINK_LIBRARIES ${QUIC_LINK_LIBRARIES}
+ )
+ message(STATUS "Found quicly ${EXPECTED_QUICLY_VERSION} in ${QUICLY_INCLUDE_DIR}")
+ else()
+ message(STATUS "-- quicly ${EXPECTED_QUICLY_VERSION} not found - QUIC plugin disabled")
+ endif()
else()
- message(WARNING "-- quicly not found - quic_plugin disabled")
+ message(WARNING "-- quicly headers not found - QUIC plugin disabled")
endif()
diff --git a/src/plugins/quic/quic.c b/src/plugins/quic/quic.c
index 03e46c60616..3991656cefa 100644
--- a/src/plugins/quic/quic.c
+++ b/src/plugins/quic/quic.c
@@ -784,7 +784,7 @@ quic_on_stream_destroy (quicly_stream_t * stream, int err)
clib_mem_free (stream->data);
}
-static int
+static void
quic_on_stop_sending (quicly_stream_t * stream, int err)
{
#if QUIC_DEBUG >= 2
@@ -797,10 +797,9 @@ quic_on_stop_sending (quicly_stream_t * stream, int err)
session_handle (stream_session), quic_format_err, err);
#endif
/* TODO : handle STOP_SENDING */
- return 0;
}
-static int
+static void
quic_on_receive_reset (quicly_stream_t * stream, int err)
{
quic_stream_data_t *stream_data = (quic_stream_data_t *) stream->data;
@@ -813,10 +812,9 @@ quic_on_receive_reset (quicly_stream_t * stream, int err)
session_handle (stream_session), quic_format_err, err);
#endif
session_transport_closing_notify (&sctx->connection);
- return 0;
}
-static int
+static void
quic_on_receive (quicly_stream_t * stream, size_t off, const void *src,
size_t len)
{
@@ -846,7 +844,7 @@ quic_on_receive (quicly_stream_t * stream, size_t off, const void *src,
stream_session->thread_index, f,
max_enq, len, stream_data->app_rx_data_len, off,
off - stream_data->app_rx_data_len + len);
- return 0;
+ return;
}
if (PREDICT_FALSE ((off - stream_data->app_rx_data_len + len) > max_enq))
{
@@ -858,7 +856,7 @@ quic_on_receive (quicly_stream_t * stream, size_t off, const void *src,
stream_session->thread_index, f,
max_enq, len, stream_data->app_rx_data_len, off,
off - stream_data->app_rx_data_len + len);
- return 1;
+ return; /* This shouldn't happen */
}
if (off == stream_data->app_rx_data_len)
{
@@ -888,7 +886,7 @@ quic_on_receive (quicly_stream_t * stream, size_t off, const void *src,
len, (u8 *) src);
QUIC_ASSERT (rlen == 0);
}
- return 0;
+ return;
}
void
@@ -912,7 +910,7 @@ quic_fifo_egress_shift (quicly_stream_t * stream, size_t delta)
QUIC_ASSERT (!rv);
}
-int
+void
quic_fifo_egress_emit (quicly_stream_t * stream, size_t off, void *dst,
size_t * len, int *wrote_all)
{
@@ -944,8 +942,6 @@ quic_fifo_egress_emit (quicly_stream_t * stream, size_t off, void *dst,
stream_data->app_tx_data_len = off + *len;
svm_fifo_peek (f, off, *len, dst);
-
- return 0;
}
static const quicly_stream_callbacks_t quic_stream_callbacks = {