[PATCH] osmo-pcu[master]: TBF-UL: add simpler test helper

[Max]

Hello Jenkins Builder,

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

    https://gerrit.osmocom.org/5337

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

TBF-UL: add simpler test helper

Add function to set both V_R and V_Q values to 0 which is useful for TBF
test.

Change-Id: I719abfbd5b88c694cbbd69d5c4dcb42baaca91b2
---
M src/rlc.h
M tests/tbf/TbfTest.cpp
2 files changed, 9 insertions(+), 4 deletions(-)


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

diff --git a/src/rlc.h b/src/rlc.h
index 14d2082..5f0284c 100644
--- a/src/rlc.h
+++ b/src/rlc.h
@@ -343,6 +343,7 @@
 
 	const void set_v_r(int);
 	const void set_v_q(int);
+	void reset_v_x();
 
 	const uint16_t ssn() const;
 
@@ -574,6 +575,12 @@
 	return is_in_window(bsn) && m_v_n.is_received(bsn) && offset_v_r < ws();
 }
 
+inline void gprs_rlc_ul_window::reset_v_x()
+{
+	m_v_r = 0;
+	m_v_q = 0;
+}
+
 inline const void gprs_rlc_ul_window::set_v_r(int v_r)
 {
 	m_v_r = v_r;
diff --git a/tests/tbf/TbfTest.cpp b/tests/tbf/TbfTest.cpp
index 5b24775..1370867 100644
--- a/tests/tbf/TbfTest.cpp
+++ b/tests/tbf/TbfTest.cpp
@@ -1722,8 +1722,7 @@
 		"Got MS: TLLI = 0x%08x, TA = %d\n", ms->tlli(), ms->ta());
 	send_dl_data(&the_bts, tlli, imsi, test_data, sizeof(test_data));
 
-	ul_tbf->m_window.set_v_r(0);
-	ul_tbf->m_window.set_v_q(0);
+	ul_tbf->m_window.reset_v_x();
 	/* Function to generate URBB with length */
 	ul_tbf = establish_ul_tbf_two_phase_puan_URBB_with_length(&the_bts, ts_no, tlli, &fn,
 		qta, ms_class, egprs_ms_class, ul_tbf);
@@ -1735,8 +1734,7 @@
 
 	send_dl_data(&the_bts, tlli, imsi, test_data, sizeof(test_data));
 
-	ul_tbf->m_window.set_v_r(0);
-	ul_tbf->m_window.set_v_q(0);
+	ul_tbf->m_window.reset_v_x();
 	/* Function to generate CRBB */
 	bts->ws_base = 128;
 	bts->ws_pdch = 64;

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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I719abfbd5b88c694cbbd69d5c4dcb42baaca91b2
Gerrit-PatchSet: 3
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder