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,14 @@ void utils_log_init(void) {
              bool_to_str(loggerConfig.enableTimestamp));
 }
 
+void utils_log_init(void) { utils_init_once(&initOnce, utils_log_init_once); }
+
+void utils_log_init_test(void) {
+    // this function is used in tests to initialize the logging system
+    // without using the utils_init_once to test various logging scenarios
+    utils_log_init_once();
+}
+
 // this is needed for logger unit test
 #ifndef DISABLE_CTL_LOGGER
 static umf_result_t

src/utils/utils_log.h

@@ -37,6 +37,10 @@ typedef enum {
 #define LOG_PFATAL(...) utils_plog(LOG_FATAL, __func__, __VA_ARGS__);
 
 void utils_log_init(void);
+
+// this function is used in tests to initialize the logging system
+void utils_log_init_test(void);
+
 #ifdef _WIN32
 void utils_log(utils_log_level_t level, const char *func, const char *format,
                ...);

test/utils/utils_log.cpp

@@ -128,7 +128,7 @@ void helper_log_init(const char *var) {
     env_variable = var;
     fopen_count = 0;
     fput_count = 0;
-    utils_log_init();
+    utils_log_init_test();
     env_variable = NULL;
     EXPECT_EQ(fopen_count, expect_fopen_count);
     EXPECT_EQ(fput_count, expect_fput_count);
@@ -161,8 +161,8 @@ TEST_F(test, parseEnv_errors) {
     helper_checkConfig(&b, &loggerConfig);
     helper_log_init("_level:debug");
     helper_checkConfig(&b, &loggerConfig);
-    expected_message =
-        "[ERROR UMF] utils_log_init: Cannot open output file - path too long\n";
+    expected_message = "[ERROR UMF] utils_log_init_once: Cannot open output "
+                       "file - path too long\n";
     std::string test_env = "output:file," + std::string(300, 'x');
     helper_log_init(test_env.c_str());
 }
@@ -247,7 +247,8 @@ TEST_F(test, parseEnv) {
                             expect_fput_count = 1;
                             if (expected_filename.size() > MAX_FILE_PATH) {
                                 expected_message =
-                                    "[ERROR UMF] utils_log_init: Cannot open "
+                                    "[ERROR UMF] utils_log_init_once: Cannot "
+                                    "open "
                                     "output file - path too long\n";
                             }
                         }