Harald Welte has submitted this change and it was merged.

Change subject: osmo-bsc: Initialize logging before initializing rate_ctr
......................................................................


osmo-bsc: Initialize logging before initializing rate_ctr

The library code for rate counter initialization, which is called
from the descendants of bsc_network_alloc() might already want to
log something (particularly after Change-Id
Ifc6ac824f5dae9a848bb4a5d067c64a69eb40b56 in libosmocore), so the
logging framework must be initialized before.

Change-Id: I1e893c97e023e63489fe8c46539b5e507d3cec8f
---
M openbsc/src/osmo-bsc/osmo_bsc_main.c
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved; Verified



diff --git a/openbsc/src/osmo-bsc/osmo_bsc_main.c 
b/openbsc/src/osmo-bsc/osmo_bsc_main.c
index 90651b9..6081460 100644
--- a/openbsc/src/osmo-bsc/osmo_bsc_main.c
+++ b/openbsc/src/osmo-bsc/osmo_bsc_main.c
@@ -195,15 +195,15 @@
        tall_bsc_ctx = talloc_named_const(NULL, 1, "openbsc");
        msgb_talloc_ctx_init(tall_bsc_ctx, 0);
 
+       osmo_init_logging(&log_info);
+       osmo_stats_init(tall_bsc_ctx);
+
        /* Allocate global gsm_network struct */
        rc = bsc_network_alloc(NULL);
        if (rc) {
                fprintf(stderr, "Allocation failed. exiting.\n");
                exit(1);
        }
-
-       osmo_init_logging(&log_info);
-       osmo_stats_init(tall_bsc_ctx);
 
        bts_init();
        libosmo_abis_init(tall_bsc_ctx);

-- 
To view, visit https://gerrit.osmocom.org/4420
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1e893c97e023e63489fe8c46539b5e507d3cec8f
Gerrit-PatchSet: 1
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>

Reply via email to