From b056953a9467fb22fd83e24aa5bd396dd3e8b8b5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Kjell=20Hedstr=C3=B6m=20-=20seeking=20Senior=20Engineering?= =?UTF-8?q?=20roles=20as=20well=20as=20contract=20opportunities?= Date: Mon, 11 Dec 2023 19:00:35 -0700 Subject: [PATCH 01/17] Create cmake-multi-platform.yml --- .github/workflows/cmake-multi-platform.yml | 75 ++++++++++++++++++++++ 1 file changed, 75 insertions(+) create mode 100644 .github/workflows/cmake-multi-platform.yml diff --git a/.github/workflows/cmake-multi-platform.yml b/.github/workflows/cmake-multi-platform.yml new file mode 100644 index 0000000..979dc21 --- /dev/null +++ b/.github/workflows/cmake-multi-platform.yml @@ -0,0 +1,75 @@ +# This starter workflow is for a CMake project running on multiple platforms. There is a different starter workflow if you just want a single platform. +# See: https://github.com/actions/starter-workflows/blob/main/ci/cmake-single-platform.yml +name: CMake on multiple platforms + +on: + push: + branches: [ "master" ] + pull_request: + branches: [ "master" ] + +jobs: + build: + runs-on: ${{ matrix.os }} + + strategy: + # Set fail-fast to false to ensure that feedback is delivered for all matrix combinations. Consider changing this to true when your workflow is stable. + fail-fast: false + + # Set up a matrix to run the following 3 configurations: + # 1. + # 2. + # 3. + # + # To add more build types (Release, Debug, RelWithDebInfo, etc.) customize the build_type list. + matrix: + os: [ubuntu-latest, macos-latest, windows-latest] + build_type: [Release] + c_compiler: [gcc, clang, cl] + include: + - os: windows-latest + c_compiler: cl + cpp_compiler: cl + - os: ubuntu-latest + c_compiler: gcc + cpp_compiler: g++ + - os: ubuntu-latest + c_compiler: clang + cpp_compiler: clang++ + exclude: + - os: windows-latest + c_compiler: gcc + - os: windows-latest + c_compiler: clang + - os: ubuntu-latest + c_compiler: cl + + steps: + - uses: actions/checkout@v3 + + - name: Set reusable strings + # Turn repeated input strings (such as the build output directory) into step outputs. These step outputs can be used throughout the workflow file. + id: strings + shell: bash + run: | + echo "build-output-dir=${{ github.workspace }}/build" >> "$GITHUB_OUTPUT" + + - name: Configure CMake + # Configure CMake in a 'build' subdirectory. `CMAKE_BUILD_TYPE` is only required if you are using a single-configuration generator such as make. + # See https://cmake.org/cmake/help/latest/variable/CMAKE_BUILD_TYPE.html?highlight=cmake_build_type + run: > + cmake -B ${{ steps.strings.outputs.build-output-dir }} + -DCMAKE_CXX_COMPILER=${{ matrix.cpp_compiler }} + -DCMAKE_C_COMPILER=${{ matrix.c_compiler }} + -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} + -S ${{ github.workspace }} + + - name: Build + # Build your program with the given configuration. Note that --config is needed because the default Windows generator is a multi-config generator (Visual Studio generator). + run: cmake --build ${{ steps.strings.outputs.build-output-dir }} --config ${{ matrix.build_type }} + + - name: Test + working-directory: ${{ steps.strings.outputs.build-output-dir }} + # Execute tests defined by the CMake configuration. Note that --build-config is needed because the default Windows generator is a multi-config generator (Visual Studio generator). + # See https://cmake.org/cmake/help/latest/manual/ctest.1.html for more detail + run: ctest --build-config ${{ matrix.build_type }} From 6de6062516ba07e65a3ff8434752a5c3706b3711 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Kjell=20Hedstr=C3=B6m=20-=20seeking=20Senior=20Engineering?= =?UTF-8?q?=20roles=20as=20well=20as=20contract=20opportunities?= Date: Mon, 11 Dec 2023 19:12:15 -0700 Subject: [PATCH 02/17] Update cmake-multi-platform.yml --- .github/workflows/cmake-multi-platform.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/cmake-multi-platform.yml b/.github/workflows/cmake-multi-platform.yml index 979dc21..eab3462 100644 --- a/.github/workflows/cmake-multi-platform.yml +++ b/.github/workflows/cmake-multi-platform.yml @@ -1,4 +1,5 @@ -# This starter workflow is for a CMake project running on multiple platforms. There is a different starter workflow if you just want a single platform. +# This starter workflow is for a CMake project running on multiple platforms. +# There is a different starter workflow if you just want a single platform. # See: https://github.com/actions/starter-workflows/blob/main/ci/cmake-single-platform.yml name: CMake on multiple platforms From 8b5a810d545f8d090eaed583e8358e18b72dc805 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Kjell=20Hedstr=C3=B6m=20-=20seeking=20Senior=20Engineering?= =?UTF-8?q?=20roles=20as=20well=20as=20contract=20opportunities?= Date: Mon, 11 Dec 2023 20:16:38 -0700 Subject: [PATCH 03/17] Update CMakeLists.txt --- CMakeLists.txt | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index e9278c1..7aecbb4 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -41,10 +41,12 @@ endif() # change to c++20 in version 2.0 if (APPLE) - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -std=c++14 -pthread -D_GLIBCXX_USE_NANOSLEEP") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -std=c++20 -pthread -D_GLIBCXX_USE_NANOSLEEP") +elseif(MSVC) + #? else() set(PLATFORM_LINK_LIBRIES rt) - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -std=c++14 -pthread -lrt -D_GLIBCXX_USE_NANOSLEEP -D_GLIBCXX_USE_SCHED_YIELD") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -std=c++20 -pthread -lrt -D_GLIBCXX_USE_NANOSLEEP -D_GLIBCXX_USE_SCHED_YIELD") endif() From 409c1e15b9b2f07e6e6620756fb8dd6d5de7db99 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Kjell=20Hedstr=C3=B6m=20-=20seeking=20Senior=20Engineering?= =?UTF-8?q?=20roles=20as=20well=20as=20contract=20opportunities?= Date: Mon, 11 Dec 2023 20:22:19 -0700 Subject: [PATCH 04/17] Update CMakeLists.txt --- CMakeLists.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 7aecbb4..e96c88d 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -41,12 +41,12 @@ endif() # change to c++20 in version 2.0 if (APPLE) - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -std=c++20 -pthread -D_GLIBCXX_USE_NANOSLEEP") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -stdlib=libc++ -std=c++20 -pthread -D_GLIBCXX_USE_NANOSLEEP") elseif(MSVC) #? else() set(PLATFORM_LINK_LIBRIES rt) - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -std=c++20 -pthread -lrt -D_GLIBCXX_USE_NANOSLEEP -D_GLIBCXX_USE_SCHED_YIELD") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -stdlib=libc++ -std=c++20 -pthread -lrt -D_GLIBCXX_USE_NANOSLEEP -D_GLIBCXX_USE_SCHED_YIELD") endif() From bfff89e1f85c1c9161434097baa1ebcb28fd042e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Kjell=20Hedstr=C3=B6m=20-=20seeking=20Senior=20Engineering?= =?UTF-8?q?=20roles=20as=20well=20as=20contract=20opportunities?= Date: Mon, 11 Dec 2023 21:23:39 -0700 Subject: [PATCH 05/17] Update CMakeLists.txt --- CMakeLists.txt | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index e96c88d..9da4305 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -40,15 +40,16 @@ if(SYSTEM_INCLUDE) endif() # change to c++20 in version 2.0 -if (APPLE) - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -stdlib=libc++ -std=c++20 -pthread -D_GLIBCXX_USE_NANOSLEEP") -elseif(MSVC) - #? +if (MSVC) else() - set(PLATFORM_LINK_LIBRIES rt) - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -stdlib=libc++ -std=c++20 -pthread -lrt -D_GLIBCXX_USE_NANOSLEEP -D_GLIBCXX_USE_SCHED_YIELD") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -stdlib=libc++ -std=c++20 -pthread -D_GLIBCXX_USE_NANOSLEEP") endif() +if (CMAKE_CXX_COMPILER_ID MATCHES "Clang") + add_compile_options("-stdlib=libc++") +endif() + + file(GLOB SRC_FILES ${PROJECT_SRC}/g3log/*.h ${PROJECT_SRC}/q/*.hpp ${PROJECT_SRC}/*.cpp) file(GLOB HEADER_FILES ${PROJECT_SRC}/q/*.hpp) From e9a0470ceb58a573647904dad2919f4ccaeb0dc2 Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 21:32:01 -0700 Subject: [PATCH 06/17] updated for clang/gcc --- CMakeLists.txt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 6455990..10d8961 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -16,6 +16,8 @@ if ( NOT VERSION ) execute_process(COMMAND bash "-c" "git rev-list --branches HEAD | wc -l | tr -d ' ' | tr -d '\n'" OUTPUT_VARIABLE GIT_VERSION WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}) endif() +message( STATUS "Compiler: ${CMAKE_C_COMPILER}") + set(MINOR_VERSION 1) math(EXPR VERSION-BASE ${GIT_VERSION}/255) math(EXPR VERSION-REMAINDER ${GIT_VERSION}%255) @@ -42,7 +44,7 @@ endif() # change to c++20 in version 2.0 if (MSVC) else() - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -stdlib=libc++ -std=c++20 -pthread -D_GLIBCXX_USE_NANOSLEEP") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -std=c++2a -pthread -D_GLIBCXX_USE_NANOSLEEP") endif() if (CMAKE_CXX_COMPILER_ID MATCHES "Clang") From 39281d92a8c07cdf4b0c5b0b846b4eecf8738e6f Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 21:41:11 -0700 Subject: [PATCH 07/17] in progress --- .github/workflows/cmake-multi-platform.yml | 17 ++++++++++++++++- 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/.github/workflows/cmake-multi-platform.yml b/.github/workflows/cmake-multi-platform.yml index eab3462..7e725e2 100644 --- a/.github/workflows/cmake-multi-platform.yml +++ b/.github/workflows/cmake-multi-platform.yml @@ -5,10 +5,19 @@ name: CMake on multiple platforms on: push: + paths-ignore: + - docs/** + - '**.md' branches: [ "master" ] pull_request: + paths-ignore: + - docs/** + - '**.md' branches: [ "master" ] +env: + BUILD_TYPE: Release + jobs: build: runs-on: ${{ matrix.os }} @@ -21,7 +30,7 @@ jobs: # 1. # 2. # 3. - # + # 4. # To add more build types (Release, Debug, RelWithDebInfo, etc.) customize the build_type list. matrix: os: [ubuntu-latest, macos-latest, windows-latest] @@ -64,6 +73,12 @@ jobs: -DCMAKE_C_COMPILER=${{ matrix.c_compiler }} -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} -S ${{ github.workspace }} + + # - name: Create Build Environment + # # Some projects don't allow in-source building, so create a separate build directory + # # We'll use this as our working directory for all subsequent commands + # run: cmake -E make_directory ${{github.workspace}}/build + - name: Build # Build your program with the given configuration. Note that --config is needed because the default Windows generator is a multi-config generator (Visual Studio generator). From bd307e3bda384be6bd4d4cfda80db612c1ba80b5 Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 21:43:34 -0700 Subject: [PATCH 08/17] ... --- CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 10d8961..0e72914 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -44,7 +44,7 @@ endif() # change to c++20 in version 2.0 if (MSVC) else() - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -std=c++2a -pthread -D_GLIBCXX_USE_NANOSLEEP") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -std=c++17 -pthread -D_GLIBCXX_USE_NANOSLEEP") endif() if (CMAKE_CXX_COMPILER_ID MATCHES "Clang") From 2d20068e333df3f97244251da05069ce473906ab Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 21:46:24 -0700 Subject: [PATCH 09/17] ... --- .github/workflows/cmake-multi-platform.yml | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/.github/workflows/cmake-multi-platform.yml b/.github/workflows/cmake-multi-platform.yml index 7e725e2..9421ad3 100644 --- a/.github/workflows/cmake-multi-platform.yml +++ b/.github/workflows/cmake-multi-platform.yml @@ -5,14 +5,14 @@ name: CMake on multiple platforms on: push: - paths-ignore: - - docs/** - - '**.md' + # paths-ignore: + # - docs/** + # - '**.md' branches: [ "master" ] pull_request: - paths-ignore: - - docs/** - - '**.md' + # paths-ignore: + # - docs/** + # - '**.md' branches: [ "master" ] env: @@ -73,7 +73,7 @@ jobs: -DCMAKE_C_COMPILER=${{ matrix.c_compiler }} -DCMAKE_BUILD_TYPE=${{ matrix.build_type }} -S ${{ github.workspace }} - + # - name: Create Build Environment # # Some projects don't allow in-source building, so create a separate build directory # # We'll use this as our working directory for all subsequent commands From 3d7bb0f2d445dfabfae427beaff83b0d4b8f4c52 Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 21:55:35 -0700 Subject: [PATCH 10/17] loosing track --- CMakeLists.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 0e72914..be69256 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -43,12 +43,13 @@ endif() # change to c++20 in version 2.0 if (MSVC) -else() +else () set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wunused -std=c++17 -pthread -D_GLIBCXX_USE_NANOSLEEP") endif() if (CMAKE_CXX_COMPILER_ID MATCHES "Clang") add_compile_options("-stdlib=libc++") + #add_compile_options("-std=c++20") endif() From f8a10769e7c52b12ce01cc8312e2b535b453a78b Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 22:08:45 -0700 Subject: [PATCH 11/17] push --- .github/workflows/cmake-multi-platform.yml | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/.github/workflows/cmake-multi-platform.yml b/.github/workflows/cmake-multi-platform.yml index 9421ad3..64e860e 100644 --- a/.github/workflows/cmake-multi-platform.yml +++ b/.github/workflows/cmake-multi-platform.yml @@ -14,7 +14,7 @@ on: # - docs/** # - '**.md' branches: [ "master" ] - + env: BUILD_TYPE: Release @@ -43,7 +43,7 @@ jobs: - os: ubuntu-latest c_compiler: gcc cpp_compiler: g++ - - os: ubuntu-latest + - os: macos-latest c_compiler: clang cpp_compiler: clang++ exclude: @@ -53,6 +53,8 @@ jobs: c_compiler: clang - os: ubuntu-latest c_compiler: cl + - os: macos-latest + c_compiler: clang steps: - uses: actions/checkout@v3 From e5bdef35082657e01397fddd3d5ba30c992cc824 Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 22:18:38 -0700 Subject: [PATCH 12/17] reduce complexity --- .github/workflows/cmake-multi-platform.yml | 17 ++++------------- 1 file changed, 4 insertions(+), 13 deletions(-) diff --git a/.github/workflows/cmake-multi-platform.yml b/.github/workflows/cmake-multi-platform.yml index 64e860e..c18f0d4 100644 --- a/.github/workflows/cmake-multi-platform.yml +++ b/.github/workflows/cmake-multi-platform.yml @@ -33,28 +33,19 @@ jobs: # 4. # To add more build types (Release, Debug, RelWithDebInfo, etc.) customize the build_type list. matrix: - os: [ubuntu-latest, macos-latest, windows-latest] + os: [ubuntu-latest, macos-latest] build_type: [Release] - c_compiler: [gcc, clang, cl] + c_compiler: [gcc, clang] include: - - os: windows-latest - c_compiler: cl - cpp_compiler: cl - os: ubuntu-latest - c_compiler: gcc cpp_compiler: g++ - os: macos-latest - c_compiler: clang cpp_compiler: clang++ exclude: - - os: windows-latest - c_compiler: gcc - - os: windows-latest - c_compiler: clang - os: ubuntu-latest - c_compiler: cl + cpp_compiler: clang++ - os: macos-latest - c_compiler: clang + cpp_compiler: g++ steps: - uses: actions/checkout@v3 From 19a8c4a3e14fe233f7f0bd7f27674af7a83aec01 Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 22:24:43 -0700 Subject: [PATCH 13/17] cimp --- .github/workflows/cmake-multi-platform.yml | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/.github/workflows/cmake-multi-platform.yml b/.github/workflows/cmake-multi-platform.yml index c18f0d4..261a589 100644 --- a/.github/workflows/cmake-multi-platform.yml +++ b/.github/workflows/cmake-multi-platform.yml @@ -38,14 +38,16 @@ jobs: c_compiler: [gcc, clang] include: - os: ubuntu-latest + c_compiler: gcc cpp_compiler: g++ - os: macos-latest + c_compiler: clang cpp_compiler: clang++ exclude: - os: ubuntu-latest - cpp_compiler: clang++ + c_compiler: clang - os: macos-latest - cpp_compiler: g++ + c_compiler: gcc steps: - uses: actions/checkout@v3 From 1302482551077a4f19ce50532202e240f1a1579f Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 22:33:00 -0700 Subject: [PATCH 14/17] disable super slow tests --- test/test_performance.cpp | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/test/test_performance.cpp b/test/test_performance.cpp index 2d6d3c7..7e731c9 100644 --- a/test/test_performance.cpp +++ b/test/test_performance.cpp @@ -57,7 +57,7 @@ TEST(Performance, MPMC_1_to_1_Smaller) { RunSPSC(queue, kAmount); } -TEST(Performance, SPSC_Flexible_20secRun_LargeData) { +TEST(Performance, DISABLED_SPSC_Flexible_20secRun_LargeData) { using namespace std; auto queue = queue_api::CreateQueue>(kSmallQueueSize); const size_t large = 65000; @@ -69,7 +69,7 @@ TEST(Performance, SPSC_Flexible_20secRun_LargeData) { RunMPMC(queue, payload, numberOfProducers, numberOfConsumers, kTimeToRunSec); } -TEST(Performance, SPSC_Fixed_20secRun_LargeData) { +TEST(Performance, DISABLED_SPSC_Fixed_20secRun_LargeData) { using namespace std; auto queue = queue_api::CreateQueue>(); const size_t large = 65000; @@ -81,7 +81,7 @@ TEST(Performance, SPSC_Fixed_20secRun_LargeData) { RunMPMC(queue, payload, numberOfProducers, numberOfConsumers, kTimeToRunSec); } -TEST(Performance, MPMC_1_to_4_20secRun_LargeData) { +TEST(Performance, DISABLED_MPMC_1_to_4_20secRun_LargeData) { using namespace std; auto queue = queue_api::CreateQueue>(kSmallQueueSize); const size_t large = 65000; @@ -93,7 +93,7 @@ TEST(Performance, MPMC_1_to_4_20secRun_LargeData) { RunMPMC(queue, payload, numberOfProducers, numberOfConsumers, kTimeToRunSec); } -TEST(Performance, MPMC_1_to_4_20secRun_SmallData) { +TEST(Performance, DISABLED_MPMC_1_to_4_20secRun_SmallData) { using namespace std; auto queue = queue_api::CreateQueue>(kSmallQueueSize); const size_t small = 10; @@ -105,7 +105,7 @@ TEST(Performance, MPMC_1_to_4_20secRun_SmallData) { RunMPMC(queue, payload, numberOfProducers, numberOfConsumers, kTimeToRunSec); } -TEST(Performance, lock_free__SPMC_1_to_4_20secRun_LargeData) { +TEST(Performance, DISABLED_lock_free__SPMC_1_to_4_20secRun_LargeData) { using namespace std; using element = std::string; using qtype = spsc::flexible::circular_fifo; @@ -122,7 +122,7 @@ TEST(Performance, lock_free__SPMC_1_to_4_20secRun_LargeData) { RunSPMC(queues, payload, kTimeToRunSec); } -TEST(Performance, lock_free__SPMC_1_to_4_20secRun_SmallData) { +TEST(Performance, DISABLED_lock_free__SPMC_1_to_4_20secRun_SmallData) { using namespace std; using element = std::string; using qtype = spsc::flexible::circular_fifo; @@ -139,7 +139,7 @@ TEST(Performance, lock_free__SPMC_1_to_4_20secRun_SmallData) { RunSPMC(queues, payload, kTimeToRunSec); } -TEST(Performance, MPMC_4_to_1_20secRun_LargeData) { +TEST(Performance, DISABLED_MPMC_4_to_1_20secRun_LargeData) { using namespace std; auto queue = queue_api::CreateQueue>(kSmallQueueSize); const size_t large = 65000; @@ -151,7 +151,7 @@ TEST(Performance, MPMC_4_to_1_20secRun_LargeData) { RunMPMC(queue, payload, numberOfProducers, numberOfConsumers, kTimeToRunSec); } -TEST(Performance, MPMC_4_to_1_20secRun_SmallData) { +TEST(Performance, DISABLED_MPMC_4_to_1_20secRun_SmallData) { using namespace std; auto queue = queue_api::CreateQueue>(kSmallQueueSize); const size_t small = 10; @@ -163,7 +163,7 @@ TEST(Performance, MPMC_4_to_1_20secRun_SmallData) { RunMPMC(queue, payload, numberOfProducers, numberOfConsumers, kTimeToRunSec); } -TEST(Performance, lock_free__MPSC_4_to_1_20secRun_LargeData) { +TEST(Performance, DISABLED_lock_free__MPSC_4_to_1_20secRun_LargeData) { using namespace std; using element = std::string; using qtype = spsc::flexible::circular_fifo; @@ -180,7 +180,7 @@ TEST(Performance, lock_free__MPSC_4_to_1_20secRun_LargeData) { RunMPSC(queues, payload, kTimeToRunSec); } -TEST(Performance, lock_free__MPSC_4_to_1_20secRun_SmallData) { +TEST(Performance, DISABLED_lock_free__MPSC_4_to_1_20secRun_SmallData) { using namespace std; using element = std::string; using qtype = spsc::flexible::circular_fifo; @@ -197,7 +197,7 @@ TEST(Performance, lock_free__MPSC_4_to_1_20secRun_SmallData) { RunMPSC(queues, payload, kTimeToRunSec); } -TEST(Performance, MPMC_4_to_4_20secRun_LargeData) { +TEST(Performance, DISABLED_MPMC_4_to_4_20secRun_LargeData) { using namespace std; auto queue = queue_api::CreateQueue>(kSmallQueueSize); const size_t large = 65000; From 78f76a1c132b3f378e394a497bd67c5b05836639 Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 22:36:33 -0700 Subject: [PATCH 15/17] md --- .github/workflows/cmake-multi-platform.yml | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/.github/workflows/cmake-multi-platform.yml b/.github/workflows/cmake-multi-platform.yml index 261a589..2f1d450 100644 --- a/.github/workflows/cmake-multi-platform.yml +++ b/.github/workflows/cmake-multi-platform.yml @@ -5,14 +5,12 @@ name: CMake on multiple platforms on: push: - # paths-ignore: - # - docs/** - # - '**.md' + paths-ignore: + - '**.md' branches: [ "master" ] pull_request: - # paths-ignore: - # - docs/** - # - '**.md' + paths-ignore: + - '**.md' branches: [ "master" ] env: From 154626876af58ceba591f8d8ca13982941d93ec5 Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 22:37:06 -0700 Subject: [PATCH 16/17] . --- examples/spsc_main.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/examples/spsc_main.cpp b/examples/spsc_main.cpp index b8e78e8..f660886 100644 --- a/examples/spsc_main.cpp +++ b/examples/spsc_main.cpp @@ -44,7 +44,7 @@ void consumeMessages(ReceiverQ& receiver, std::atomic& keep_working) { int main() { - // Create a flexible SPSC queue with a dynamic size, determined at runtime initialization + // Create a flexible SPSC queue with a dynamic size, determined at runtime initialization. auto queue = queue_api::CreateQueue>(10); // Get the sender and receiver endpoints of the queue From 61f24280a0ee8699f0a27ba4499596b0e4639c72 Mon Sep 17 00:00:00 2001 From: Kjell Hedstrom Date: Mon, 11 Dec 2023 22:38:12 -0700 Subject: [PATCH 17/17] fixed space --- .github/workflows/cmake-multi-platform.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/cmake-multi-platform.yml b/.github/workflows/cmake-multi-platform.yml index 2f1d450..5198b77 100644 --- a/.github/workflows/cmake-multi-platform.yml +++ b/.github/workflows/cmake-multi-platform.yml @@ -5,11 +5,11 @@ name: CMake on multiple platforms on: push: - paths-ignore: + paths-ignore: - '**.md' branches: [ "master" ] pull_request: - paths-ignore: + paths-ignore: - '**.md' branches: [ "master" ]