Enable ubench benchmark with OS memory provider on Windows

Signed-off-by: Lukasz Dorau <[email protected]>

Author: ldorau

.github/workflows/benchmarks.yml

@@ -39,6 +39,7 @@ jobs:
           -DUMF_BUILD_TESTS=OFF
           -DUMF_FORMAT_CODE_STYLE=OFF
           -DUMF_DEVELOPER_MODE=OFF
+          -DUMF_BUILD_OS_MEMORY_PROVIDER=ON
           -DUMF_BUILD_LIBUMF_POOL_JEMALLOC=ON
           -DUMF_BUILD_LIBUMF_POOL_DISJOINT=ON
           -DUMF_BUILD_LIBUMF_POOL_SCALABLE=ON
@@ -67,3 +68,63 @@ jobs:
       - name: Run MT benchmarks
         working-directory: ${{github.workspace}}/build
         run: ./benchmark/multithread_bench
+
+  benchmarks-windows:
+    name: Windows
+    env:
+      BUILD_DIR : "${{github.workspace}}/build/"
+      HWLOC_PATH: "${{github.workspace}}/build/vcpkg/packages/hwloc_x64-windows"
+    strategy:
+      fail-fast: false
+      matrix:
+        build_type: [Release]
+        compiler: [{c: cl, cxx: cl}]
+        shared_library: ['ON', 'OFF']
+    runs-on: 'windows-2022'
+
+    steps:
+    - name: Checkout
+      uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
+
+    - name: Initialize vcpkg
+      uses: lukka/run-vcpkg@5e0cab206a5ea620130caf672fce3e4a6b5666a1 # v11.5
+      with:
+        vcpkgGitCommitId: 3dd44b931481d7a8e9ba412621fa810232b66289
+        vcpkgDirectory: ${{github.workspace}}/build/vcpkg
+        vcpkgJsonGlob: '**/vcpkg.json'
+
+    - name: Install dependencies
+      run: vcpkg install
+      shell: pwsh # Specifies PowerShell as the shell for running the script.
+
+    - name: Configure build
+      run: >
+        cmake
+        -B ${{env.BUILD_DIR}}
+        -DCMAKE_C_COMPILER=${{matrix.compiler.c}}
+        -DCMAKE_CXX_COMPILER=${{matrix.compiler.cxx}}
+        -DCMAKE_PREFIX_PATH=${{env.HWLOC_PATH}}
+        -DUMF_BUILD_SHARED_LIBRARY=${{matrix.shared_library}}
+        -DUMF_BUILD_BENCHMARKS=ON
+        -DUMF_BUILD_BENCHMARKS_MT=OFF
+        -DUMF_BUILD_TESTS=OFF
+        -DUMF_FORMAT_CODE_STYLE=OFF
+        -DUMF_DEVELOPER_MODE=OFF
+        -DUMF_BUILD_OS_MEMORY_PROVIDER=ON
+        -DUMF_BUILD_LIBUMF_POOL_DISJOINT=ON
+        -DUMF_BUILD_LIBUMF_POOL_JEMALLOC=OFF
+        -DUMF_BUILD_LIBUMF_POOL_SCALABLE=OFF
+        -DUMF_ENABLE_POOL_TRACKING=OFF
+
+    - name: Build UMF
+      run: cmake --build ${{env.BUILD_DIR}} --config ${{matrix.build_type}} -j $Env:NUMBER_OF_PROCESSORS
+
+    - name: Run benchmarks
+      working-directory: ${{env.BUILD_DIR}}
+      shell: pwsh # Specifies PowerShell as the shell for running the script.
+      run: |
+        # add path to umf.dll
+        $env:PATH += ";${{env.BUILD_DIR}}\bin\${{matrix.build_type}}"
+        # add path to hwloc-15.dll
+        $env:PATH += ";${{env.HWLOC_PATH}}\bin"
+        .\benchmark\${{matrix.build_type}}\ubench.exe

CMakeLists.txt

@@ -123,10 +123,10 @@ if(UMF_BUILD_TESTS)
 endif()
 
 if(UMF_BUILD_BENCHMARKS)
-    if(LINUX)
+    if(LINUX OR WINDOWS)
         add_subdirectory(benchmark)
     else()
-        message(FATAL_ERROR "benchmarks are supported on Linux only")
+        message(FATAL_ERROR "benchmarks are supported on Linux and Windows only")
     endif()
 endif()
 

benchmark/CMakeLists.txt

@@ -22,25 +22,29 @@ endif()
 
 add_executable(ubench ubench.c)
 
+if(LINUX)
+	set(LIBS_LINUX m)
+endif()
+
 add_dependencies(ubench
 	umf
 	${LIBS_OPTIONAL})
 
-target_include_directories(ubench PRIVATE ${UMF_CMAKE_SOURCE_DIR}/include/)
+target_include_directories(ubench PRIVATE ${UMF_CMAKE_SOURCE_DIR}/include/ ${UMF_CMAKE_SOURCE_DIR}/src/utils/)
 
 target_link_libraries(ubench
 	umf
 	${LIBS_OPTIONAL}
 	${CMAKE_THREAD_LIBS_INIT}
-	m)
+	${LIBS_LINUX})
 
 if (UMF_BUILD_BENCHMARKS_MT)
 	add_executable(multithread_bench multithread.cpp)
 	target_link_libraries(multithread_bench
 		umf
 		${LIBS_OPTIONAL}
 		pthread
-		m)
+		${LIBS_LINUX})
 	target_include_directories(multithread_bench PRIVATE ${UMF_CMAKE_SOURCE_DIR}/include/)
 endif()
 

benchmark/ubench.c

@@ -24,13 +24,17 @@
 #endif
 
 #include <stdbool.h>
+
+#ifndef _WIN32
 #include <unistd.h>
+#endif
 
 #include "ubench.h"
+#include "utils_common.h"
 
 // BENCHMARK CONFIG
 #define N_ITERATIONS 1000
-#define ALLOC_SIZE (getpagesize())
+#define ALLOC_SIZE (util_get_page_size())
 
 // OS MEMORY PROVIDER CONFIG
 #define OS_MEMORY_PROVIDER_TRACE (0)
@@ -65,7 +69,7 @@ static void do_benchmark(alloc_t *array, size_t iters, malloc_t malloc_f,
 }
 
 static alloc_t *alloc_array(size_t iters) {
-    Alloc_size = ALLOC_SIZE;
+    Alloc_size = (int)ALLOC_SIZE;
     alloc_t *array = malloc(iters * sizeof(alloc_t));
     if (array == NULL) {
         perror("malloc() failed");
@@ -223,7 +227,7 @@ UBENCH_EX(simple, disjoint_pool_with_os_memory_provider) {
         exit(-1);
     }
 
-    umf_disjoint_pool_params_t disjoint_memory_pool_params = {};
+    umf_disjoint_pool_params_t disjoint_memory_pool_params = {0};
     disjoint_memory_pool_params.SlabMinSize = DISJOINT_POOL_SLAB_MIN_SIZE;
     disjoint_memory_pool_params.MaxPoolableSize =
         DISJOINT_POOL_MAX_POOLABLE_SIZE;