[PATCH] osmo-bts[master]: CTRL: Allow chainging of the CTRL-Interface IP address

dexter gerrit-no-reply at lists.osmocom.org
Tue Jan 10 14:16:20 UTC 2017

Review at  https://gerrit.osmocom.org/1558

CTRL: Allow chainging of the CTRL-Interface IP address

Currently the IP address where the control interface is bound
to is hardcoded to This leads to problems with
multiple instances on one and the same machine. This commit
integrates the ctrl interface bind option into the VTY, so
that we can bind the ctrl interface to any IP address, just
like we do it with the VTY already.

Change-Id: If51e0c645c0789a4f4a8c51737fb81fb12f80829
M include/osmo-bts/control_if.h
M src/common/bts_ctrl_lookup.c
M src/common/main.c
3 files changed, 12 insertions(+), 4 deletions(-)

  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/58/1558/1

diff --git a/include/osmo-bts/control_if.h b/include/osmo-bts/control_if.h
index 750f6bd..490c87a 100644
--- a/include/osmo-bts/control_if.h
+++ b/include/osmo-bts/control_if.h
@@ -1,4 +1,5 @@
 #pragma once
 int bts_ctrl_cmds_install(struct gsm_bts *bts);
-struct ctrl_handle *bts_controlif_setup(struct gsm_bts *bts);
+struct ctrl_handle *bts_controlif_setup(struct gsm_bts *bts,
+					const char *bind_addr, uint16_t port);
diff --git a/src/common/bts_ctrl_lookup.c b/src/common/bts_ctrl_lookup.c
index 3ee9b4a..f0157e9 100644
--- a/src/common/bts_ctrl_lookup.c
+++ b/src/common/bts_ctrl_lookup.c
@@ -87,12 +87,14 @@
 	return -ERANGE;
-struct ctrl_handle *bts_controlif_setup(struct gsm_bts *bts)
+struct ctrl_handle *bts_controlif_setup(struct gsm_bts *bts,
+					const char *bind_addr, uint16_t port)
 	struct ctrl_handle *hdl;
 	int rc = 0;
-	hdl = ctrl_interface_setup(bts, OSMO_CTRL_PORT_BTS, bts_ctrl_node_lookup);
+	hdl = ctrl_interface_setup_dynip(bts, bind_addr, port,
+					 bts_ctrl_node_lookup);
 	if (!hdl)
 		return NULL;
diff --git a/src/common/main.c b/src/common/main.c
index f99f196..00c8b23 100644
--- a/src/common/main.c
+++ b/src/common/main.c
@@ -50,6 +50,10 @@
 #include <osmo-bts/bts_model.h>
 #include <osmo-bts/pcu_if.h>
 #include <osmo-bts/control_if.h>
+#include <osmocom/ctrl/control_if.h>
+#include <osmocom/ctrl/ports.h>
+#include <osmocom/ctrl/control_vty.h>
+#include <openbsc/ctrl.h>
 #include <osmo-bts/oml.h>
 int quit = 0;
@@ -237,6 +241,7 @@
+	ctrl_vty_init(tall_bts_ctx);
 	handle_options(argc, argv);
@@ -308,7 +313,7 @@
-	bts_controlif_setup(bts);
+	bts_controlif_setup(bts, ctrl_vty_get_bind_addr(), OSMO_CTRL_PORT_BTS);
 	rc = telnet_init_dynif(tall_bts_ctx, NULL, vty_get_bind_addr(),

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If51e0c645c0789a4f4a8c51737fb81fb12f80829
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: dexter <pmaier at sysmocom.de>

More information about the gerrit-log mailing list