[MERGED] osmo-pcu[master]: introduce new counter rlc_sent_control

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/OpenBSC@lists.osmocom.org/.

Holger Freyther gerrit-no-reply at lists.osmocom.org
Sun May 22 11:11:53 UTC 2016


Holger Freyther has submitted this change and it was merged.

Change subject: introduce new counter rlc_sent_control
......................................................................


introduce new counter rlc_sent_control

Counts control messages (UL/DL assignment, UL ACKs or page requests)

Change-Id: Ib41031d430beddfb48d54470e632436f2c99c360
Reviewed-on: https://gerrit.osmocom.org/99
Reviewed-by: Holger Freyther <holger at freyther.de>
Tested-by: Jenkins Builder
---
M src/bts.cpp
M src/bts.h
M src/gprs_rlcmac_sched.cpp
3 files changed, 6 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  Holger Freyther: Looks good to me, approved



diff --git a/src/bts.cpp b/src/bts.cpp
index d1636b6..fdae1f0 100644
--- a/src/bts.cpp
+++ b/src/bts.cpp
@@ -72,6 +72,7 @@
 	{ "rlc.rel.timedout",		"RLC Release Timeout  "},
 	{ "rlc.late-block",		"RLC Late Block       "},
 	{ "rlc.sent_dummy",		"RLC Sent Dummy       "},
+	{ "rlc.sent_control",		"RLC Sent Control     "},
 	{ "decode.errors",		"Decode Errors        "},
 	{ "sba.allocated",		"SBA Allocated        "},
 	{ "sba.freed",			"SBA Freed            "},
diff --git a/src/bts.h b/src/bts.h
index 829e3c2..35f24d1 100644
--- a/src/bts.h
+++ b/src/bts.h
@@ -234,6 +234,7 @@
 		CTR_RLC_REL_TIMEDOUT,
 		CTR_RLC_LATE_BLOCK,
 		CTR_RLC_SENT_DUMMY,
+		CTR_RLC_SENT_CONTROL,
 		CTR_DECODE_ERRORS,
 		CTR_SBA_ALLOCATED,
 		CTR_SBA_FREED,
@@ -311,6 +312,7 @@
 	void rlc_rel_timedout();
 	void rlc_late_block();
 	void rlc_sent_dummy();
+	void rlc_sent_control();
 	void decode_error();
 	void sba_allocated();
 	void sba_freed();
@@ -452,6 +454,7 @@
 CREATE_COUNT_INLINE(rlc_rel_timedout, CTR_RLC_REL_TIMEDOUT);
 CREATE_COUNT_INLINE(rlc_late_block, CTR_RLC_LATE_BLOCK);
 CREATE_COUNT_INLINE(rlc_sent_dummy, CTR_RLC_SENT_DUMMY);
+CREATE_COUNT_INLINE(rlc_sent_control, CTR_RLC_SENT_CONTROL);
 CREATE_COUNT_INLINE(decode_error, CTR_DECODE_ERRORS)
 CREATE_COUNT_INLINE(sba_allocated, CTR_SBA_ALLOCATED)
 CREATE_COUNT_INLINE(sba_freed, CTR_SBA_FREED)
diff --git a/src/gprs_rlcmac_sched.cpp b/src/gprs_rlcmac_sched.cpp
index cd5f683..fce3aaf 100644
--- a/src/gprs_rlcmac_sched.cpp
+++ b/src/gprs_rlcmac_sched.cpp
@@ -339,6 +339,8 @@
 	/* Prio 1: select control message */
 	msg = sched_select_ctrl_msg(trx, ts, fn, block_nr, pdch, ul_ass_tbf,
 		dl_ass_tbf, ul_ack_tbf);
+	if (msg)
+		bts->bts->rlc_sent_control();
 
 	/* Prio 2: select data message for downlink */
 	if (!msg) {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib41031d430beddfb48d54470e632436f2c99c360
Gerrit-PatchSet: 3
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus <lynxis at fe80.eu>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Jenkins Builder



More information about the OpenBSC mailing list