aboutsummaryrefslogtreecommitdiffstats
path: root/src/bts.cpp
diff options
context:
space:
mode:
authorHarald Welte <laforge@gnumonks.org>2017-10-29 10:57:27 +0100
committerHarald Welte <laforge@gnumonks.org>2017-10-29 11:02:34 +0100
commit3e51d3e5bd6d3c2dff79e47c7ad22c073b0dba93 (patch)
tree25f90b833fcf3823d6b28990cb64b26a07465d2b /src/bts.cpp
parentac0490ad2a086a4e2664bed19bfe2787cbf4d450 (diff)
Call osmo_init_logging() before static BTS constructor
The BTS constructor uses functions of libosmocore that could in turn want to log something. This requires the logging to be initialized before. The only way to achieve this is to add an __attribute__((constructor)) function *before* the BTS constructor is being run. This solution might not be elegant, but I guess it's the only way to initialize a C library before calling C++ constructors of global static instance of a class. In case anyone comes up with a better / cleaner approach, we can always change later. This change requires libosmocore >= 0.10.1, as only that permits multiple calls to osmo_init_logging() which may now occur. Change-Id: I28dc4f0db229518348c92413959fed5ae85d753d
Diffstat (limited to 'src/bts.cpp')
-rw-r--r--src/bts.cpp11
1 files changed, 11 insertions, 0 deletions
diff --git a/src/bts.cpp b/src/bts.cpp
index b768569..e41b1fa 100644
--- a/src/bts.cpp
+++ b/src/bts.cpp
@@ -36,6 +36,7 @@ extern "C" {
#include <osmocom/gsm/protocol/gsm_04_08.h>
#include <osmocom/gsm/gsm_utils.h>
#include <osmocom/core/gsmtap_util.h>
+ #include <osmocom/core/application.h>
}
#include <arpa/inet.h>
@@ -48,6 +49,16 @@ extern "C" {
extern void *tall_pcu_ctx;
+extern "C" {
+ /* e must make sure to initialize logging before the BTS static
+ * constructors are executed below, as those call libosmocore APIs that
+ * require logging already to be initialized. */
+ __attribute__((constructor)) static void early_init(void)
+ {
+ osmo_init_logging(&gprs_log_info);
+ }
+}
+
static BTS s_bts;
/**