cellular-infrastructure
/
osmo-bsc
mirror of https://gerrit.osmocom.org/osmo-bsc
9
0
Fork 0
Code Issues Releases Wiki Activity

Don't drop OML links for Get Attributes NACK 

Previously any OML NACK message will result in BSC dropping OML link to
BTS which makes it impossible to use optional OML messages which might
be unsupported by BTS. Fix this for 3GPP TS 52.021 §8.11.1 Get
Attributes message. Also, log human-readable NACK name to see what
exactly causing OML link drop.

Change-Id: Ib8af2872c27abb793172ec59bdc145b8d54f83da
Related: OS#1614
This commit is contained in:
Max 2017-03-16 13:57:47 +01:00
parent 6dd0fc685b
commit 88d4fc70d5
1 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
openbsc/src/libbsc/bsc_init.c
View File

@ -44,19 +44,19 @@ extern struct gsm_network *bsc_gsmnet;
/* Callback function for NACK on the OML NM */
static int oml_msg_nack(struct nm_nack_signal_data *nack)
{
if (nack->mt == NM_MT_SET_BTS_ATTR_NACK) {
LOGP(DNM, LOGL_ERROR, "Failed to set BTS attributes. That is fatal. "
"Was the bts type and frequency properly specified?\n");
goto drop_bts;
} else {
LOGP(DNM, LOGL_ERROR, "Got a NACK going to drop the OML links.\n");
goto drop_bts;
if (nack->mt == NM_MT_GET_ATTR_NACK) {
LOGP(DNM, LOGL_ERROR, "BTS%u does not support Get Attributes "
"OML message.\n", nack->bts->nr);
return 0;
}
return 0;
if (nack->mt == NM_MT_SET_BTS_ATTR_NACK)
LOGP(DNM, LOGL_ERROR, "Failed to set BTS attributes. That is fatal. "
"Was the bts type and frequency properly specified?\n");
else
LOGP(DNM, LOGL_ERROR, "Got %s NACK going to drop the OML links.\n",
abis_nm_nack_name(nack->mt));
drop_bts:
if (!nack->bts) {
LOGP(DNM, LOGL_ERROR, "Unknown bts. Can not drop it.\n");
return 0;