Get rid of volatile in favor of ATOMIC_T

Author: syyyr

src/log.c

@@ -29,15 +29,15 @@
 /**
  * @brief libnetconf verbose level variable
  */
-volatile uint8_t verbose_level = 0;
+ATOMIC_T verbose_level = 0;
 
 void (*depr_print_clb)(NC_VERB_LEVEL level, const char *msg);
 void (*print_clb)(const struct nc_session *session, NC_VERB_LEVEL level, const char *msg);
 
 API void
 nc_verbosity(NC_VERB_LEVEL level)
 {
-    verbose_level = level;
+    ATOMIC_STORE_RELAXED(verbose_level, level);
     ly_log_level((LY_LOG_LEVEL)level);
 }
 

src/log_p.h

@@ -18,6 +18,7 @@
 
 #include <stdint.h>
 
+#include "compat.h"
 #include "log.h"
 
 /*
@@ -36,16 +37,16 @@ void prv_printf(const struct nc_session *session, NC_VERB_LEVEL level, const cha
 /**
  * @brief Verbose level variable
  */
-extern volatile uint8_t verbose_level;
+extern ATOMIC_T verbose_level;
 
 /*
  * Verbose printing macros
  */
 #define ERR(session, format, args ...) prv_printf(session,NC_VERB_ERROR,format,##args)
-#define WRN(session, format, args ...) if(verbose_level>=NC_VERB_WARNING){prv_printf(session,NC_VERB_WARNING,format,##args);}
-#define VRB(session, format, args ...) if(verbose_level>=NC_VERB_VERBOSE){prv_printf(session,NC_VERB_VERBOSE,format,##args);}
-#define DBG(session, format, args ...) if(verbose_level>=NC_VERB_DEBUG){prv_printf(session,NC_VERB_DEBUG,format,##args);}
-#define DBL(session, format, args ...) if(verbose_level>=NC_VERB_DEBUG_LOWLVL){prv_printf(session,NC_VERB_DEBUG_LOWLVL,format,##args);}
+#define WRN(session, format, args ...) if(ATOMIC_LOAD_RELAXED(verbose_level)>=NC_VERB_WARNING){prv_printf(session,NC_VERB_WARNING,format,##args);}
+#define VRB(session, format, args ...) if(ATOMIC_LOAD_RELAXED(verbose_level)>=NC_VERB_VERBOSE){prv_printf(session,NC_VERB_VERBOSE,format,##args);}
+#define DBG(session, format, args ...) if(ATOMIC_LOAD_RELAXED(verbose_level)>=NC_VERB_DEBUG){prv_printf(session,NC_VERB_DEBUG,format,##args);}
+#define DBL(session, format, args ...) if(ATOMIC_LOAD_RELAXED(verbose_level)>=NC_VERB_DEBUG_LOWLVL){prv_printf(session,NC_VERB_DEBUG_LOWLVL,format,##args);}
 
 #define ERRMEM ERR(NULL, "%s: memory reallocation failed (%s:%d).", __func__, __FILE__, __LINE__)
 #define ERRARG(arg) ERR(NULL, "%s: invalid argument (%s).", __func__, arg)