[PATCH] osmo-pcu[master]: introduce new counter rlc_sent_dummy

[lynxis lazus]

Hello Jenkins Builder,

I'd like you to reexamine a change.  Please visit

    https://gerrit.osmocom.org/85

to look at the new patch set (#3).

introduce new counter rlc_sent_dummy

rlc_sent_dummy count the amount of dummy package which are
sent in case no data packet is in the queue.

Change-Id: Ia60eab853d9145980f30d63e4ce4b520b8c51381
---
M src/bts.cpp
M src/bts.h
M src/gprs_rlcmac_sched.cpp
3 files changed, 9 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/85/85/3

diff --git a/src/bts.cpp b/src/bts.cpp
index 715fb51..a182317 100644
--- a/src/bts.cpp
+++ b/src/bts.cpp
@@ -71,6 +71,7 @@
 	{ "rlc.ack.failed",		"RLC Ack Failed       "},
 	{ "rlc.rel.timedout",		"RLC Release Timeout  "},
 	{ "rlc.late-block",		"RLC Late Block       "},
+	{ "rlc.sent_dummy",             "RLC Sent Dummy       "},
 	{ "decode.errors",		"Decode Errors        "},
 	{ "sba.allocated",		"SBA Allocated        "},
 	{ "sba.freed",			"SBA Freed            "},
diff --git a/src/bts.h b/src/bts.h
index c975304..829e3c2 100644
--- a/src/bts.h
+++ b/src/bts.h
@@ -233,6 +233,7 @@
 		CTR_RLC_ACK_FAILED,
 		CTR_RLC_REL_TIMEDOUT,
 		CTR_RLC_LATE_BLOCK,
+		CTR_RLC_SENT_DUMMY,
 		CTR_DECODE_ERRORS,
 		CTR_SBA_ALLOCATED,
 		CTR_SBA_FREED,
@@ -309,6 +310,7 @@
 	void rlc_ack_failed();
 	void rlc_rel_timedout();
 	void rlc_late_block();
+	void rlc_sent_dummy();
 	void decode_error();
 	void sba_allocated();
 	void sba_freed();
@@ -449,6 +451,7 @@
 CREATE_COUNT_INLINE(rlc_ack_failed, CTR_RLC_ACK_FAILED);
 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(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 8e05623..087c03c 100644
--- a/src/gprs_rlcmac_sched.cpp
+++ b/src/gprs_rlcmac_sched.cpp
@@ -348,8 +348,12 @@
 	}
 
 	/* Prio 3: send dummy contol message */
-	if (!msg)
+	if (!msg) {
+		/* increase counter */
 		msg = sched_dummy();
+		if (msg)
+			bts->bts->rlc_sent_dummy();
+	}
 
 	if (!msg)
 		return -ENOMEM;

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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia60eab853d9145980f30d63e4ce4b520b8c51381
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