ms: Support new and old TLLIs
According to the specification (GSM 04.08/24.008, 4.7.1.5) after a
new P-TMSI has been assigned, the old P-TMSI must be kept basically
until it has been used by both sides. Since the TLLI will be derived
from the P-TMSI, the old TLLI must also be kept until the new TLLI
has been used by both MS and SGSN.
This commit modifies the TLLI handling of GprsMs accordingly.
set_tlli() is only used with TLLIs derived from MS messages,
confirm_tlli() is used with TLLIs derived from messages received from
the SGSN. tlli() returns the value set by the MS. check_tlli()
matches each of the TLLI used by either MS or SGSN as well as the old
TLLI until it has been confirmed.
Sponsored-by: On-Waves ehf
diff --git a/tests/tbf/TbfTest.cpp b/tests/tbf/TbfTest.cpp
index d5bcb67..2ec715b 100644
--- a/tests/tbf/TbfTest.cpp
+++ b/tests/tbf/TbfTest.cpp
@@ -51,6 +51,8 @@
static void test_tbf_tlli_update()
{
BTS the_bts;
+ GprsMs *ms, *ms_new;
+
the_bts.bts_data()->alloc_algorithm = alloc_algorithm_a;
the_bts.bts_data()->trx[0].pdch[2].enable();
the_bts.bts_data()->trx[0].pdch[3].enable();
@@ -63,6 +65,7 @@
0, 0, 0);
dl_tbf->update_tlli(0x2342);
dl_tbf->tlli_mark_valid();
+ dl_tbf->update_ms(0x2342);
dl_tbf->ta = 4;
the_bts.timing_advance()->remember(0x2342, dl_tbf->ta);
@@ -71,23 +74,33 @@
0, 0, 0);
ul_tbf->update_tlli(0x2342);
ul_tbf->tlli_mark_valid();
-
- OSMO_ASSERT(the_bts.dl_tbf_by_tlli(0x2342) == dl_tbf);
- OSMO_ASSERT(the_bts.ul_tbf_by_tlli(0x2342) == ul_tbf);
+ ms = the_bts.ms_by_tlli(0x2342);
+ OSMO_ASSERT(ms != NULL);
+ OSMO_ASSERT(ms->dl_tbf() == dl_tbf);
+ OSMO_ASSERT(ms->ul_tbf() == ul_tbf);
/*
* Now check.. that DL changes and that the timing advance
* has changed.
*/
dl_tbf->update_tlli(0x4232);
- OSMO_ASSERT(!the_bts.dl_tbf_by_tlli(0x2342));
- OSMO_ASSERT(!the_bts.ul_tbf_by_tlli(0x2342));
+ ms->confirm_tlli(0x4232);
-
- OSMO_ASSERT(the_bts.dl_tbf_by_tlli(0x4232) == dl_tbf);
- OSMO_ASSERT(the_bts.ul_tbf_by_tlli(0x4232) == ul_tbf);
+ /* It is still there, since the new TLLI has not been used for UL yet */
+ ms_new = the_bts.ms_by_tlli(0x2342);
+ OSMO_ASSERT(ms == ms_new);
+
+ ms_new = the_bts.ms_by_tlli(0x4232);
+ OSMO_ASSERT(ms == ms_new);
+ OSMO_ASSERT(ms->dl_tbf() == dl_tbf);
+ OSMO_ASSERT(ms->ul_tbf() == ul_tbf);
+
+ /* Now use the new TLLI for UL */
+ ms->set_tlli(0x4232);
+ ms_new = the_bts.ms_by_tlli(0x2342);
+ OSMO_ASSERT(ms_new == NULL);
OSMO_ASSERT(the_bts.timing_advance()->recall(0x4232) == 4);
}
@@ -318,7 +331,7 @@
snprintf(imsi, sizeof(imsi)-1, "001001%9d", i);
- rc = gprs_rlcmac_dl_tbf::handle(bts, tlli, imsi, ms_class,
+ rc = gprs_rlcmac_dl_tbf::handle(bts, tlli, 0, imsi, ms_class,
delay_csec, buf, sizeof(buf));
if (rc < 0)