ms: Drop setting (egprs_)ms_class during bts_alloc_ms()
That information is not required during allocation of the object, and
most times it is not known.
Defer setting it only to meaningul values in paths obtaining the
information from peers.
Change-Id: I36f07dc389f7abe205fc4bcddbde93735f5d5cfc
diff --git a/tests/tbf/TbfTest.cpp b/tests/tbf/TbfTest.cpp
index cc62bc2..63074d0 100644
--- a/tests/tbf/TbfTest.cpp
+++ b/tests/tbf/TbfTest.cpp
@@ -123,7 +123,7 @@
/*
* Make a uplink and downlink allocation
*/
- ms = bts_alloc_ms(bts, 0, 0);
+ ms = bts_alloc_ms(bts);
gprs_rlcmac_tbf *dl_tbf = dl_tbf_alloc(bts,
ms, 0, false);
OSMO_ASSERT(dl_tbf != NULL);
@@ -206,7 +206,9 @@
GprsMs *ms;
gprs_rlcmac_dl_tbf *dl_tbf;
- ms = bts_alloc_ms(bts, ms_class, egprs_ms_class);
+ ms = bts_alloc_ms(bts);
+ ms_set_ms_class(ms, ms_class);
+ ms_set_egprs_ms_class(ms, egprs_ms_class);
tfi = bts_tfi_find_free(bts, GPRS_RLCMAC_DL_TBF, &trx_no, -1);
OSMO_ASSERT(tfi >= 0);
@@ -2345,7 +2347,8 @@
gprs_bssgp_init(bts, 4234, 4234, 1, 1, false, 0, 0, 0);
/* Does no support EGPRS */
- ms = bts_alloc_ms(bts, ms_class, 0);
+ ms = bts_alloc_ms(bts);
+ ms_set_ms_class(ms, ms_class);
dl_tbf = dl_tbf_alloc(bts, ms, 0, false);
ws_check(dl_tbf, __func__, 4, 64, true);
@@ -2353,7 +2356,9 @@
/* EGPRS-only */
/* Does support EGPRS */
- ms = bts_alloc_ms(bts, ms_class, ms_class);
+ ms = bts_alloc_ms(bts);
+ ms_set_ms_class(ms, ms_class);
+ ms_set_egprs_ms_class(ms, ms_class);
dl_tbf = dl_tbf_alloc(bts, ms, 0, false);
ws_check(dl_tbf, __func__, 4, 128 + 4 * 64, true);
@@ -2393,7 +2398,9 @@
/* EGPRS-only */
/* Does support EGPRS */
- ms = bts_alloc_ms(bts, ms_class, ms_class);
+ ms = bts_alloc_ms(bts);
+ ms_set_ms_class(ms, ms_class);
+ ms_set_egprs_ms_class(ms, ms_class);
dl_tbf = dl_tbf_alloc(bts, ms, 0, true);
ws_check(dl_tbf, __func__, 1, 128 + 1 * 64, false);
@@ -2470,7 +2477,7 @@
trx0->pdch[2].enable();
trx0->pdch[3].enable();
- second_ms = bts_alloc_ms(bts, 0, 0);
+ second_ms = bts_alloc_ms(bts);
ms_set_tlli(second_ms, new_tlli);
ul_tbf = ul_tbf_alloc(bts, second_ms, 0, true);
OSMO_ASSERT(ul_tbf != NULL);
@@ -3329,7 +3336,7 @@
int rc = 0;
- ms = bts_alloc_ms(bts, 0, 0);
+ ms = bts_alloc_ms(bts);
ms_set_tlli(ms, tlli);
ul_tbf = ms_new_ul_tbf_rejected_pacch(ms, pdch);