Change in osmo-bsc[master]: RES IND: allow empty Resource Information IE

[neels]

neels has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/25002 )


Change subject: RES IND: allow empty Resource Information IE
......................................................................

RES IND: allow empty Resource Information IE

If all channels of a BTS are in use and there are no interference
ratings to be reported, the Resource Information IE may be empty. Do not
log this as an error, it is not something that needs operator attention.

Related: SYS#5313
Change-Id: I75b851ef1269674f43db3fb3a48518e76182d7f0
---
M src/osmo-bsc/abis_rsl.c
1 file changed, 4 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/02/25002/1

diff --git a/src/osmo-bsc/abis_rsl.c b/src/osmo-bsc/abis_rsl.c
index 5bb0efe..b491285 100644
--- a/src/osmo-bsc/abis_rsl.c
+++ b/src/osmo-bsc/abis_rsl.c
@@ -1492,7 +1492,7 @@
 		}
 	}
 
-	res_info_ie = TLVP_GET_MINLEN(&tp, RSL_IE_RESOURCE_INFO, 2);
+	res_info_ie = TLVP_GET(&tp, RSL_IE_RESOURCE_INFO);
 	if (!res_info_ie) {
 		LOGP(DRSL, LOGL_ERROR, "Rx Resource Indication: missing Resource Info IE\n");
 		return -ENOENT;
@@ -1510,7 +1510,9 @@
 		return -EINVAL;
 	}
 
-	/* Now iterate the reported levels and update corresponding lchans */
+	/* Now iterate the reported levels and update corresponding lchans.
+	 * Note that an empty res_info_ie can also make sense, if no lchans are idle and no interference ratings are
+	 * present. The practical effect of the message then is to invalidate previous interference ratings. */
 	for (i = 0; i < res_info_ie->len; i += 2) {
 		struct gsm_bts *bts = trx->bts;
 		uint8_t chan_nr = res_info_ie->val[i];

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/25002
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I75b851ef1269674f43db3fb3a48518e76182d7f0
Gerrit-Change-Number: 25002
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210721/e27f38f5/attachment.htm>