From b809866be5ce2265a074e5615e23a73f4c4764b4 Mon Sep 17 00:00:00 2001 From: Holger Hans Peter Freyther Date: Wed, 30 Oct 2013 14:50:17 +0100 Subject: tbf: Learn and propagate the TLLI changes due a new P-TMSI During a routing area update a new P-TMSI was assigned. During the PACKET CONTROL ACK on the DL we notice the change of TLLI but didn't propagate this. This means that a Routing Area Update Complete was only sent after a new RACH request. Addresses: <0007> gprs_rlcmac_meas.cpp:103 UL RSSI of TLLI=0x88661bc6: -67 dBm <0002> bts.cpp:945 Got ACK, but UL TBF is gone TLLI=0xe512eba3 <0007> gprs_rlcmac_meas.cpp:158 DL packet loss of IMSI=274080000004765 / TLLI=0xe512eba3: 0% <0002> tbf.cpp:668 TBF TFI=0 TLLI=0x88661bc6 T3169 timeout during transsmission <0002> tbf.cpp:690 - Assignment was on PACCH <0002> tbf.cpp:694 - No uplink data received yet --- tests/tbf/TbfTest.cpp | 102 ++++++++++++++++++++++++++++++++++++++++++++++++++ tests/tbf/TbfTest.err | 3 ++ tests/tbf/TbfTest.ok | 0 3 files changed, 105 insertions(+) create mode 100644 tests/tbf/TbfTest.cpp create mode 100644 tests/tbf/TbfTest.err create mode 100644 tests/tbf/TbfTest.ok (limited to 'tests/tbf') diff --git a/tests/tbf/TbfTest.cpp b/tests/tbf/TbfTest.cpp new file mode 100644 index 00000000..03c2324b --- /dev/null +++ b/tests/tbf/TbfTest.cpp @@ -0,0 +1,102 @@ +/* + * TbfTest.cpp + * + * Copyright (C) 2013 by Holger Hans Peter Freyther + * + * All Rights Reserved + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License as published by + * the Free Software Foundation; either version 3 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU Affero General Public License + * along with this program. If not, see . + * + */ + +#include "bts.h" +#include "tbf.h" +#include "gprs_debug.h" + +extern "C" { +#include +#include +#include +#include +} + +void *tall_pcu_ctx; +int16_t spoof_mnc = 0, spoof_mcc = 0; + +static void test_tbf_tlli_update() +{ + BTS the_bts; + 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(); + + /* + * Make a uplink and downlink allocation + */ + gprs_rlcmac_tbf *dl_tbf = tbf_alloc(the_bts.bts_data(), + NULL, GPRS_RLCMAC_DL_TBF, 0, + 0, 0, 0); + dl_tbf->update_tlli(0x2342); + dl_tbf->tlli_mark_valid(); + dl_tbf->ta = 4; + the_bts.timing_advance()->remember(0x2342, dl_tbf->ta); + + gprs_rlcmac_tbf *ul_tbf = tbf_alloc(the_bts.bts_data(), + ul_tbf, GPRS_RLCMAC_UL_TBF, 0, + 0, 0, 0); + ul_tbf->update_tlli(0x2342); + ul_tbf->tlli_mark_valid(); + + + OSMO_ASSERT(the_bts.tbf_by_tlli(0x2342, GPRS_RLCMAC_DL_TBF) == dl_tbf); + OSMO_ASSERT(the_bts.tbf_by_tlli(0x2342, GPRS_RLCMAC_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.tbf_by_tlli(0x2342, GPRS_RLCMAC_DL_TBF)); + OSMO_ASSERT(!the_bts.tbf_by_tlli(0x2342, GPRS_RLCMAC_UL_TBF)); + + + OSMO_ASSERT(the_bts.tbf_by_tlli(0x4232, GPRS_RLCMAC_DL_TBF) == dl_tbf); + OSMO_ASSERT(the_bts.tbf_by_tlli(0x4232, GPRS_RLCMAC_UL_TBF) == ul_tbf); +} + +int main(int argc, char **argv) +{ + tall_pcu_ctx = talloc_named_const(NULL, 1, "moiji-mobile TbfTest context"); + if (!tall_pcu_ctx) + abort(); + + msgb_set_talloc_ctx(tall_pcu_ctx); + osmo_init_logging(&gprs_log_info); + log_set_use_color(osmo_stderr_target, 0); + log_set_print_filename(osmo_stderr_target, 0); + + test_tbf_tlli_update(); + return EXIT_SUCCESS; +} + +/* + * stubs that should not be reached + */ +extern "C" { +void l1if_pdch_req() { abort(); } +void l1if_connect_pdch() { abort(); } +void l1if_close_pdch() { abort(); } +void l1if_open_pdch() { abort(); } +} diff --git a/tests/tbf/TbfTest.err b/tests/tbf/TbfTest.err new file mode 100644 index 00000000..9e2f1830 --- /dev/null +++ b/tests/tbf/TbfTest.err @@ -0,0 +1,3 @@ +TBF(TFI=0 TLLI=0x00000000 DIR=DL) changing tlli from TLLI=0x00000000 TLLI=0x00002342 ul_changed=0 +TBF(TFI=0 TLLI=0x00000000 DIR=UL) changing tlli from TLLI=0x00000000 TLLI=0x00002342 ul_changed=0 +TBF(TFI=0 TLLI=0x00002342 DIR=DL) changing tlli from TLLI=0x00002342 TLLI=0x00004232 ul_changed=1 diff --git a/tests/tbf/TbfTest.ok b/tests/tbf/TbfTest.ok new file mode 100644 index 00000000..e69de29b -- cgit v1.2.3