x

Author: bratpiorka

.github/workflows/reusable_proxy_lib.yml

@@ -54,6 +54,10 @@ jobs:
       - name: Build UMF
         run: cmake --build ${{env.BUILD_DIR}} -j $(nproc)
 
+      - name: Run "ctest --output-on-failure" without proxy library
+        working-directory: ${{env.BUILD_DIR}}
+        run: ctest --output-on-failure
+        
       - name: Run "ctest --output-on-failure" with proxy library
         working-directory: ${{env.BUILD_DIR}}
         run: LD_PRELOAD=./lib/libumf_proxy.so ctest --output-on-failure

src/coarse/coarse.c

@@ -22,13 +22,6 @@
 #include "utils_concurrency.h"
 #include "utils_log.h"
 
-#ifdef _WIN32
-UTIL_ONCE_FLAG Log_initialized = UTIL_ONCE_FLAG_INIT;
-#else
-void __attribute__((constructor)) coarse_init(void) { utils_log_init(); }
-void __attribute__((destructor)) coarse_destroy(void) {}
-#endif /* _WIN32 */
-
 typedef struct coarse_t {
     // handle of the memory provider
     void *provider;
@@ -883,9 +876,7 @@ static umf_result_t coarse_get_stats_no_lock(coarse_t *coarse,
 // PUBLIC API
 
 umf_result_t coarse_new(coarse_params_t *coarse_params, coarse_t **pcoarse) {
-#ifdef _WIN32
-    utils_init_once(&Log_initialized, utils_log_init);
-#endif /* _WIN32 */
+    utils_log_init();
 
     if (coarse_params == NULL || pcoarse == NULL) {
         LOG_ERR("coarse parameters or handle is missing");

src/libumf_linux.c

@@ -9,9 +9,9 @@
 
 #include <umf.h>
 
-void __attribute__((constructor)) umfCreate(void) { (void)umfInit(); }
+void __attribute__((constructor(102))) umfCreate(void) { (void)umfInit(); }
 
-void __attribute__((destructor)) umfDestroy(void) { (void)umfTearDown(); }
+void __attribute__((destructor(102))) umfDestroy(void) { (void)umfTearDown(); }
 
 void libumfInit(void) {
     // do nothing, additional initialization not needed

src/proxy_lib/proxy_lib_linux.c

@@ -1,6 +1,6 @@
 /*
  *
- * Copyright (C) 2024 Intel Corporation
+ * Copyright (C) 2024-2025 Intel Corporation
  *
  * Under the Apache License v2.0 with LLVM Exceptions. See LICENSE.TXT.
  * SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
@@ -9,15 +9,10 @@
 
 #include "proxy_lib.h"
 
-// The priority 102 is used, because the constructor should be called as the second one
-// (just after the first constructor of the base allocator with priority 101)
-// and the destructor as the last but one (just before the last destructor
-// of the base allocator with priority 101), because this library
-// provides the memory allocation API.
-void __attribute__((constructor(102))) proxy_lib_create(void) {
+void __attribute__((constructor(101))) proxy_lib_create(void) {
     proxy_lib_create_common();
 }
 
-void __attribute__((destructor(102))) proxy_lib_destroy(void) {
+void __attribute__((destructor(101))) proxy_lib_destroy(void) {
     proxy_lib_destroy_common();
 }

src/utils/utils_load_library.c

@@ -79,10 +79,16 @@ void *utils_open_library(const char *filename, int userFlags) {
         LOG_FATAL("dlopen(%s) failed with error: %s", filename, dlerror());
     }
 
+    LOG_DEBUG("Opened library %s with handle %p", filename, handle);
+
     return handle;
 }
 
-int utils_close_library(void *handle) { return dlclose(handle); }
+int utils_close_library(void *handle) {
+    LOG_DEBUG("Closing library handle %p", handle);
+
+    return dlclose(handle);
+}
 
 void *utils_get_symbol_addr(void *handle, const char *symbol,
                             const char *libname) {

src/utils/utils_log.c

@@ -30,6 +30,7 @@
 #include "ctl/ctl_internal.h"
 #include "utils_assert.h"
 #include "utils_common.h"
+#include "utils_concurrency.h"
 #include "utils_log.h"
 
 #define UMF_MAGIC_STR "\x00@(#) "
@@ -60,6 +61,8 @@ char const __umf_str_1__all_cmake_vars[] =
 #define MAX_FILE_PATH 256
 #define MAX_ENV_LEN 2048
 
+static UTIL_ONCE_FLAG initOnce = UTIL_ONCE_FLAG_INIT;
+
 typedef struct {
     bool enableTimestamp;
     bool enablePid;
@@ -247,7 +250,7 @@ void utils_plog(utils_log_level_t level, const char *func, const char *format,
 
 static const char *bool_to_str(int b) { return b ? "yes" : "no"; }
 
-void utils_log_init(void) {
+static void utils_log_init_once(void) {
     const char *envVar = getenv("UMF_LOG");
 
     if (!envVar) {
@@ -342,6 +345,8 @@ void utils_log_init(void) {
              bool_to_str(loggerConfig.enableTimestamp));
 }
 
+void utils_log_init(void) { utils_init_once(&initOnce, utils_log_init_once); }
+
 // this is needed for logger unit test
 #ifndef DISABLE_CTL_LOGGER
 static umf_result_t