Change in osmo-bsc[master]: stats: Add counters for received BSSMAP messages.

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

ipse gerrit-no-reply at lists.osmocom.org
Thu May 7 23:44:02 UTC 2020 

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


Change subject: stats: Add counters for received BSSMAP messages.
......................................................................

stats: Add counters for received BSSMAP messages.

Change-Id: I3f08d71b58b4e8d6f61376d85c2051e194aa8e43
---
M include/osmocom/bsc/bsc_msc_data.h
M src/osmo-bsc/osmo_bsc_bssap.c
M src/osmo-bsc/osmo_bsc_msc.c
3 files changed, 44 insertions(+), 0 deletions(-)



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

diff --git a/include/osmocom/bsc/bsc_msc_data.h b/include/osmocom/bsc/bsc_msc_data.h
index f8fb0f0..e241e59 100644
--- a/include/osmocom/bsc/bsc_msc_data.h
+++ b/include/osmocom/bsc/bsc_msc_data.h
@@ -56,6 +56,23 @@
 	MSC_CON_TYPE_LOCAL,
 };
 
+/* Constants for the MSC rate counters */
+enum {
+	MSC_CTR_BSSMAP_NUM_RX_UDT_RESET_ACKNOWLEDGE,
+	MSC_CTR_BSSMAP_NUM_RX_UDT_RESET,
+	MSC_CTR_BSSMAP_NUM_RX_UDT_PAGING,
+	MSC_CTR_BSSMAP_NUM_RX_UDT_UNKNOWN,
+	MSC_CTR_BSSMAP_NUM_RX_DT1_CLEAR_CMD,
+	MSC_CTR_BSSMAP_NUM_RX_DT1_CIPHER_MODE_CMD,
+	MSC_CTR_BSSMAP_NUM_RX_DT1_ASSIGMENT_RQST,
+	MSC_CTR_BSSMAP_NUM_RX_DT1_LCLS_CONNECT_CTRL,
+	MSC_CTR_BSSMAP_NUM_RX_DT1_HANDOVER_CMD,
+	MSC_CTR_BSSMAP_NUM_RX_DT1_CLASSMARK_RQST,
+	MSC_CTR_BSSMAP_NUM_RX_DT1_UNKNOWN,
+	MSC_CTR_BSSMAP_NUM_RX_DTAP_MSG,
+	MSC_CTR_BSSMAP_NUM_RX_DTAP_ERROR,
+};
+
 /* Constants for the MSC stats */
 enum {
 	MSC_STAT_NUM_MSC_LINKS_ACTIVE,
diff --git a/src/osmo-bsc/osmo_bsc_bssap.c b/src/osmo-bsc/osmo_bsc_bssap.c
index e79ce9d..b1db851 100644
--- a/src/osmo-bsc/osmo_bsc_bssap.c
+++ b/src/osmo-bsc/osmo_bsc_bssap.c
@@ -1018,15 +1018,19 @@
 
 	switch (msg->l4h[0]) {
 	case BSS_MAP_MSG_RESET_ACKNOWLEDGE:
+		rate_ctr_inc(&msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_UDT_RESET_ACKNOWLEDGE]);
 		ret = bssmap_handle_reset_ack(msc, msg, length);
 		break;
 	case BSS_MAP_MSG_RESET:
+		rate_ctr_inc(&msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_UDT_RESET]);
 		ret = bssmap_handle_reset(msc, msg, length);
 		break;
 	case BSS_MAP_MSG_PAGING:
+		rate_ctr_inc(&msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_UDT_PAGING]);
 		ret = bssmap_handle_paging(msc, msg, length);
 		break;
 	default:
+		rate_ctr_inc(&msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_UDT_UNKNOWN]);
 		LOGP(DMSC, LOGL_NOTICE, "Received unimplemented BSSMAP UDT %s\n",
 			gsm0808_bssmap_name(msg->l4h[0]));
 		break;
@@ -1050,24 +1054,31 @@
 
 	switch (msg->l4h[0]) {
 	case BSS_MAP_MSG_CLEAR_CMD:
+		rate_ctr_inc(&conn->sccp.msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_DT1_CLEAR_CMD]);
 		ret = bssmap_handle_clear_cmd(conn, msg, length);
 		break;
 	case BSS_MAP_MSG_CIPHER_MODE_CMD:
+		rate_ctr_inc(&conn->sccp.msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_DT1_CIPHER_MODE_CMD]);
 		ret = bssmap_handle_cipher_mode(conn, msg, length);
 		break;
 	case BSS_MAP_MSG_ASSIGMENT_RQST:
+		rate_ctr_inc(&conn->sccp.msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_DT1_ASSIGMENT_RQST]);
 		ret = bssmap_handle_assignm_req(conn, msg, length);
 		break;
 	case BSS_MAP_MSG_LCLS_CONNECT_CTRL:
+		rate_ctr_inc(&conn->sccp.msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_DT1_LCLS_CONNECT_CTRL]);
 		ret = bssmap_handle_lcls_connect_ctrl(conn, msg, length);
 		break;
 	case BSS_MAP_MSG_HANDOVER_CMD:
+		rate_ctr_inc(&conn->sccp.msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_DT1_HANDOVER_CMD]);
 		ret = bssmap_handle_handover_cmd(conn, msg, length);
 		break;
 	case BSS_MAP_MSG_CLASSMARK_RQST:
+		rate_ctr_inc(&conn->sccp.msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_DT1_CLASSMARK_RQST]);
 		ret = gsm48_send_rr_classmark_enquiry(conn->lchan);
 		break;
 	default:
+		rate_ctr_inc(&conn->sccp.msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_DT1_UNKNOWN]);
 		LOGP(DMSC, LOGL_NOTICE, "Unimplemented msg type: %s\n",
 			gsm0808_bssmap_name(msg->l4h[0]));
 		break;
@@ -1102,17 +1113,20 @@
 
 	header = (struct dtap_header *) msg->l3h;
 	if (sizeof(*header) >= length) {
+		rate_ctr_inc(&conn->sccp.msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_DTAP_ERROR]);
 		LOGP(DMSC, LOGL_ERROR, "The DTAP header does not fit. Wanted: %zu got: %u, hex: %s\n",
 		     sizeof(*header), length, osmo_hexdump(msg->l3h, length));
 		return -1;
 	}
 
 	if (header->length > length - sizeof(*header)) {
+		rate_ctr_inc(&conn->sccp.msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_DTAP_ERROR]);
 		LOGP(DMSC, LOGL_ERROR, "The DTAP l4 information does not fit. Wanted: %u got: %zu, hex: %s\n",
 		     header->length, length - sizeof(*header), osmo_hexdump(msg->l3h, length));
 		return -1;
 	}
 
+	rate_ctr_inc(&conn->sccp.msc->msc_ctrs->ctr[MSC_CTR_BSSMAP_NUM_RX_DTAP_MSG]);
 	LOGP(DMSC, LOGL_INFO, "Rx MSC DTAP, SAPI: %u CHAN: %u\n", header->link_id & 0x07, header->link_id & 0xC0);
 
 	/* forward the data */
diff --git a/src/osmo-bsc/osmo_bsc_msc.c b/src/osmo-bsc/osmo_bsc_msc.c
index c9052cb..9588a1a 100644
--- a/src/osmo-bsc/osmo_bsc_msc.c
+++ b/src/osmo-bsc/osmo_bsc_msc.c
@@ -44,6 +44,19 @@
 #include <unistd.h>
 
 static const struct rate_ctr_desc msc_ctr_description[] = {
+	[MSC_CTR_BSSMAP_NUM_RX_UDT_RESET_ACKNOWLEDGE] = {"bssmap:num_rx_udt_reset_acknowledge", "Number of received BSSMAP UDT RESET ACKNOWLEDGE messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_UDT_RESET] = {"bssmap:num_rx_udt_reset", "Number of received BSSMAP UDT RESET messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_UDT_PAGING] = {"bssmap:num_rx_udt_paging", "Number of received BSSMAP UDT PAGING messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_UDT_UNKNOWN] = {"bssmap:num_rx_udt_unknown", "Number of received BSSMAP unknown UDT messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_DT1_CLEAR_CMD] = {"bssmap:num_rx_dt1_clear_cmd", "Number of received BSSMAP DT1 CLEAR CMD messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_DT1_CIPHER_MODE_CMD] = {"bssmap:num_rx_dt1_cipher_mode_cmd", "Number of received BSSMAP DT1 CIPHER MODE CMD messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_DT1_ASSIGMENT_RQST] = {"bssmap:num_rx_dt1_assignment_rqst", "Number of received BSSMAP DT1 ASSIGMENT RQST messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_DT1_LCLS_CONNECT_CTRL] = {"bssmap:num_rx_dt1_lcls_connect_ctrl", "Number of received BSSMAP DT1 LCLS CONNECT CTRL messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_DT1_HANDOVER_CMD] = {"bssmap:num_rx_dt1_handover_cmd", "Number of received BSSMAP DT1 HANDOVER CMD messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_DT1_CLASSMARK_RQST] = {"bssmap:num_rx_dt1_classmark_rqst", "Number of received BSSMAP DT1 CLASSMARK RQST messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_DT1_UNKNOWN] = {"bssmap:num_rx_dt1_unknown", "Number of received BSSMAP unknown DT1 messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_DTAP_MSG] = {"bssmap:num_rx_dtap_msg", "Number of received BSSMAP DTAP messages"},
+	[MSC_CTR_BSSMAP_NUM_RX_DTAP_ERROR] = {"bssmap:num_rx_dtap_error", "Number of received BSSMAP DATP messages with errors"},
 };
 
 static const struct rate_ctr_group_desc msc_ctrg_desc = {

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I3f08d71b58b4e8d6f61376d85c2051e194aa8e43
Gerrit-Change-Number: 18112
Gerrit-PatchSet: 1
Gerrit-Owner: ipse <Alexander.Chemeris at gmail.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200507/70066475/attachment.htm>

More information about the gerrit-log mailing list