- gerrit-log - lists.osmocom.org

Change in osmo-bsc[master]: paging: start/stop credit_timer based on C0 running
by pespin 09 May '22

09 May '22

Attention is currently required from: pespin. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/28047 ) Change subject: paging: start/stop credit_timer based on C0 running ...................................................................... Set Ready For Review -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/28047 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I1b5b1a98115b4e9d821eb3330fc5b970a0e78a44 Gerrit-Change-Number: 28047 Gerrit-PatchSet: 5 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Mon, 09 May 2022 18:07:38 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in osmo-bsc[master]: Introduce new signal S_NM_RUNNING_CHG and implement it for rcarrier,b...
by pespin 09 May '22

09 May '22

Attention is currently required from: laforge. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/28046 ) Change subject: Introduce new signal S_NM_RUNNING_CHG and implement it for rcarrier,bbtransc ...................................................................... Set Ready For Review -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/28046 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I206d4c7863a77fbab6a600126742a6a6b8fc3614 Gerrit-Change-Number: 28046 Gerrit-PatchSet: 4 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Mon, 09 May 2022 18:07:35 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in osmo-pcu[master]: TbfTest: Reset MS timeout to 0 in test_tbf_dl_llc_loss()
by pespin 09 May '22

09 May '22

Attention is currently required from: pespin. Hello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/osmo-pcu/+/28077 to look at the new patch set (#2). Change subject: TbfTest: Reset MS timeout to 0 in test_tbf_dl_llc_loss() ...................................................................... TbfTest: Reset MS timeout to 0 in test_tbf_dl_llc_loss() The timeout is set to a high value to avoid freeing the MS during main loop iteration. Let's set it back to 0 to have a common free path in test infrastructure. Change-Id: I6dc4765163fde1a46574b49f3185aea65391e0d0 --- M tests/tbf/TbfTest.cpp 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/77/28077/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/28077 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: I6dc4765163fde1a46574b49f3185aea65391e0d0 Gerrit-Change-Number: 28077 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newpatchset

1 0

Change in osmo-bsc[master]: paging: Avoid queueing more than 60 second estimated requests
by osmith 09 May '22

09 May '22

Attention is currently required from: pespin, fixeria. osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/28062 ) Change subject: paging: Avoid queueing more than 60 second estimated requests ...................................................................... Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/28062 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I6523c6816a4d16b71084d004e979be40cf0aeeb0 Gerrit-Change-Number: 28062 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Mon, 09 May 2022 15:43:12 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in osmo-pcu[master]: gprs_ms_stoage: Release all MS in ms_storage cleanup()
by pespin 09 May '22

09 May '22

pespin has uploaded a new patch set (#3). ( https://gerrit.osmocom.org/c/osmo-pcu/+/28078 ) Change subject: gprs_ms_stoage: Release all MS in ms_storage cleanup() ...................................................................... gprs_ms_stoage: Release all MS in ms_storage cleanup() This makes sure all MS and TBF objects under a bts are freed when its underlaying storage is freed (cleanup() is called). Related: OS#5555 Change-Id: Idf3ad8b70c97313dec38a49d48cf36d6c9b404c7 --- M src/gprs_ms_storage.cpp M tests/tbf/TbfTest.err 2 files changed, 139 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/78/28078/3 -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/28078 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: Idf3ad8b70c97313dec38a49d48cf36d6c9b404c7 Gerrit-Change-Number: 28078 Gerrit-PatchSet: 3 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-CC: Jenkins Builder Gerrit-MessageType: newpatchset

1 0

Change in osmo-pcu[master]: gprs_ms_stoage: Release all MS in ms_storage cleanup()
by pespin 09 May '22

09 May '22

pespin has uploaded a new patch set (#2). ( https://gerrit.osmocom.org/c/osmo-pcu/+/28078 ) Change subject: gprs_ms_stoage: Release all MS in ms_storage cleanup() ...................................................................... gprs_ms_stoage: Release all MS in ms_storage cleanup() This makes sure all MS and TBF objects under a bts are freed when its underlaying storage is freed (cleanup() is called). Change-Id: Idf3ad8b70c97313dec38a49d48cf36d6c9b404c7 --- M src/gprs_ms_storage.cpp M tests/tbf/TbfTest.err 2 files changed, 139 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/78/28078/2 -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/28078 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: Idf3ad8b70c97313dec38a49d48cf36d6c9b404c7 Gerrit-Change-Number: 28078 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-CC: Jenkins Builder Gerrit-MessageType: newpatchset

1 0

Change in osmo-pcu[master]: WIP: Release all MS in ms_storage cleanup()
by pespin 09 May '22

09 May '22

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-pcu/+/28078 ) Change subject: WIP: Release all MS in ms_storage cleanup() ...................................................................... WIP: Release all MS in ms_storage cleanup() This makes sure all MS and TBF objects under a bts are freed when its underlaying storage is freed (cleanup() is called). Change-Id: Idf3ad8b70c97313dec38a49d48cf36d6c9b404c7 --- M src/gprs_ms_storage.cpp M tests/tbf/TbfTest.err 2 files changed, 139 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/78/28078/1 diff --git a/src/gprs_ms_storage.cpp b/src/gprs_ms_storage.cpp index f980824..c9a41c9 100644 --- a/src/gprs_ms_storage.cpp +++ b/src/gprs_ms_storage.cpp @@ -61,7 +61,10 @@ llist_for_each_safe(pos, tmp, &m_list) { struct GprsMs *ms = llist_entry(pos, typeof(*ms), list); ms_set_callback(ms, NULL); - ms_storage_ms_idle_cb(ms); + ms_set_timeout(ms, 0); + llist_del(&ms->list); + bts_stat_item_dec(ms->bts, STAT_MS_PRESENT); + talloc_free(ms); } } diff --git a/tests/tbf/TbfTest.err b/tests/tbf/TbfTest.err index 892b31e..51df12a 100644 --- a/tests/tbf/TbfTest.err +++ b/tests/tbf/TbfTest.err @@ -48,6 +48,9 @@ Modifying MS object, TLLI: 0x00002342 -> 0x00004232, already confirmed partly Modifying MS object, TLLI = 0x00004232, TA 4 -> 6 === end test_tbf_tlli_update === +MS(TLLI=0x00004232, IMSI=, TA=6, 0/0, UL DL) Destroying MS object +MS(TLLI=0x00004232, IMSI=, TA=6, 0/0, DL) Detaching TBF: TBF(TFI=0 TLLI=0x00004232 DIR=UL STATE=NEW) +MS(TLLI=0x00004232, IMSI=, TA=6, 0/0,) Detaching TBF: TBF(TFI=0 TLLI=0x00004232 DIR=DL STATE=NEW) === start test_tbf_final_ack === PDCH(bts=0,trx=0,ts=4) PDCH state: disabled => enabled Creating MS object, TLLI = 0xffffffff @@ -1612,6 +1615,70 @@ MS(TLLI=0xc0000020, IMSI=001001000000032, TA=220, 45/0,) No PDCH resource === end test_tbf_exhaustion === MS(TLLI=0xc0000020, IMSI=001001000000032, TA=220, 45/0,) Destroying MS object +MS(TLLI=0xc000001f, IMSI=001001000000031, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000001f, IMSI=001001000000031, TA=220, 45/0,) Detaching TBF: TBF(TFI=31 TLLI=0xc000001f DIR=DL STATE=ASSIGN) +MS(TLLI=0xc000001e, IMSI=001001000000030, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000001e, IMSI=001001000000030, TA=220, 45/0,) Detaching TBF: TBF(TFI=30 TLLI=0xc000001e DIR=DL STATE=ASSIGN) +MS(TLLI=0xc000001d, IMSI=001001000000029, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000001d, IMSI=001001000000029, TA=220, 45/0,) Detaching TBF: TBF(TFI=29 TLLI=0xc000001d DIR=DL STATE=ASSIGN) +MS(TLLI=0xc000001c, IMSI=001001000000028, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000001c, IMSI=001001000000028, TA=220, 45/0,) Detaching TBF: TBF(TFI=28 TLLI=0xc000001c DIR=DL STATE=ASSIGN) +MS(TLLI=0xc000001b, IMSI=001001000000027, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000001b, IMSI=001001000000027, TA=220, 45/0,) Detaching TBF: TBF(TFI=27 TLLI=0xc000001b DIR=DL STATE=ASSIGN) +MS(TLLI=0xc000001a, IMSI=001001000000026, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000001a, IMSI=001001000000026, TA=220, 45/0,) Detaching TBF: TBF(TFI=26 TLLI=0xc000001a DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000019, IMSI=001001000000025, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000019, IMSI=001001000000025, TA=220, 45/0,) Detaching TBF: TBF(TFI=25 TLLI=0xc0000019 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000018, IMSI=001001000000024, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000018, IMSI=001001000000024, TA=220, 45/0,) Detaching TBF: TBF(TFI=24 TLLI=0xc0000018 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000017, IMSI=001001000000023, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000017, IMSI=001001000000023, TA=220, 45/0,) Detaching TBF: TBF(TFI=23 TLLI=0xc0000017 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000016, IMSI=001001000000022, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000016, IMSI=001001000000022, TA=220, 45/0,) Detaching TBF: TBF(TFI=22 TLLI=0xc0000016 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000015, IMSI=001001000000021, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000015, IMSI=001001000000021, TA=220, 45/0,) Detaching TBF: TBF(TFI=21 TLLI=0xc0000015 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000014, IMSI=001001000000020, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000014, IMSI=001001000000020, TA=220, 45/0,) Detaching TBF: TBF(TFI=20 TLLI=0xc0000014 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000013, IMSI=001001000000019, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000013, IMSI=001001000000019, TA=220, 45/0,) Detaching TBF: TBF(TFI=19 TLLI=0xc0000013 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000012, IMSI=001001000000018, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000012, IMSI=001001000000018, TA=220, 45/0,) Detaching TBF: TBF(TFI=18 TLLI=0xc0000012 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000011, IMSI=001001000000017, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000011, IMSI=001001000000017, TA=220, 45/0,) Detaching TBF: TBF(TFI=17 TLLI=0xc0000011 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000010, IMSI=001001000000016, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000010, IMSI=001001000000016, TA=220, 45/0,) Detaching TBF: TBF(TFI=16 TLLI=0xc0000010 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc000000f, IMSI=001001000000015, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000000f, IMSI=001001000000015, TA=220, 45/0,) Detaching TBF: TBF(TFI=15 TLLI=0xc000000f DIR=DL STATE=ASSIGN) +MS(TLLI=0xc000000e, IMSI=001001000000014, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000000e, IMSI=001001000000014, TA=220, 45/0,) Detaching TBF: TBF(TFI=14 TLLI=0xc000000e DIR=DL STATE=ASSIGN) +MS(TLLI=0xc000000d, IMSI=001001000000013, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000000d, IMSI=001001000000013, TA=220, 45/0,) Detaching TBF: TBF(TFI=13 TLLI=0xc000000d DIR=DL STATE=ASSIGN) +MS(TLLI=0xc000000c, IMSI=001001000000012, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000000c, IMSI=001001000000012, TA=220, 45/0,) Detaching TBF: TBF(TFI=12 TLLI=0xc000000c DIR=DL STATE=ASSIGN) +MS(TLLI=0xc000000b, IMSI=001001000000011, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000000b, IMSI=001001000000011, TA=220, 45/0,) Detaching TBF: TBF(TFI=11 TLLI=0xc000000b DIR=DL STATE=ASSIGN) +MS(TLLI=0xc000000a, IMSI=001001000000010, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc000000a, IMSI=001001000000010, TA=220, 45/0,) Detaching TBF: TBF(TFI=10 TLLI=0xc000000a DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000009, IMSI=001001000000009, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000009, IMSI=001001000000009, TA=220, 45/0,) Detaching TBF: TBF(TFI=9 TLLI=0xc0000009 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000008, IMSI=001001000000008, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000008, IMSI=001001000000008, TA=220, 45/0,) Detaching TBF: TBF(TFI=8 TLLI=0xc0000008 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000007, IMSI=001001000000007, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000007, IMSI=001001000000007, TA=220, 45/0,) Detaching TBF: TBF(TFI=7 TLLI=0xc0000007 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000006, IMSI=001001000000006, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000006, IMSI=001001000000006, TA=220, 45/0,) Detaching TBF: TBF(TFI=6 TLLI=0xc0000006 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000005, IMSI=001001000000005, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000005, IMSI=001001000000005, TA=220, 45/0,) Detaching TBF: TBF(TFI=5 TLLI=0xc0000005 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000004, IMSI=001001000000004, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000004, IMSI=001001000000004, TA=220, 45/0,) Detaching TBF: TBF(TFI=4 TLLI=0xc0000004 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000003, IMSI=001001000000003, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000003, IMSI=001001000000003, TA=220, 45/0,) Detaching TBF: TBF(TFI=3 TLLI=0xc0000003 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000002, IMSI=001001000000002, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000002, IMSI=001001000000002, TA=220, 45/0,) Detaching TBF: TBF(TFI=2 TLLI=0xc0000002 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000001, IMSI=001001000000001, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000001, IMSI=001001000000001, TA=220, 45/0,) Detaching TBF: TBF(TFI=1 TLLI=0xc0000001 DIR=DL STATE=ASSIGN) +MS(TLLI=0xc0000000, IMSI=001001000000000, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0000000, IMSI=001001000000000, TA=220, 45/0,) Detaching TBF: TBF(TFI=0 TLLI=0xc0000000 DIR=DL STATE=ASSIGN) === start test_tbf_dl_llc_loss === PDCH(bts=0,trx=0,ts=4) PDCH state: disabled => enabled Creating MS object, TLLI = 0xffffffff @@ -1721,6 +1788,8 @@ TBF(TFI=0 TLLI=0xc0123456 DIR=DL STATE=FINISHED) msg block (BSN 2, CS-1): 07 01 04 4d 03 03 03 03 03 03 03 03 03 03 03 03 03 03 03 03 03 03 03 MSG = 07 01 04 4d 03 03 03 03 03 03 03 03 03 03 03 03 03 03 03 03 03 03 03 === end test_tbf_dl_llc_loss === +MS(TLLI=0xc0123456, IMSI=001001000123456, TA=0, 45/0, DL) Destroying MS object +MS(TLLI=0xc0123456, IMSI=001001000123456, TA=0, 45/0,) Detaching TBF: TBF(TFI=0 TLLI=0xc0123456 DIR=DL STATE=FINISHED) === start test_tbf_single_phase === PDCH(bts=0,trx=0,ts=7) PDCH state: disabled => enabled Found first unallocated TRX=0 TFI=0 @@ -1805,6 +1874,9 @@ - TRX=0 (0) TS=7 TA=7 TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN) appending 4 bytes === end test_tbf_single_phase === +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 0/0, UL DL) Destroying MS object +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 0/0, DL) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FINISHED) +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 0/0,) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN) === start test_tbf_two_phase === PDCH(bts=0,trx=0,ts=7) PDCH state: disabled => enabled MS requests Uplink resource on CCCH/RACH: ra=0x73 (8 bit) Fn=2654167 qta=31 @@ -1904,6 +1976,9 @@ TBF(DL-TFI_0){NEW}: state_chg to ASSIGN TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN) Starting timer X2001 [assignment (PACCH)] with 2 sec. 0 microsec TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN) appending 4 bytes +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/0, UL DL) Destroying MS object +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/0, DL) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FLOW) +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/0,) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN) === end test_tbf_two_phase === === start test_tbf_ra_update_rach === PDCH(bts=0,trx=0,ts=7) PDCH state: disabled => enabled @@ -2169,6 +2244,9 @@ Modifying MS object, TLLI: 0xf5667788 confirmed TBF(TFI=0 TLLI=0xf5667788 DIR=DL STATE=FINISHED) appending 4 bytes New MS: TLLI = 0xf5667788, TA = 7, IMSI = 0011223344, LLC = 1 +MS(TLLI=0xf5667788, IMSI=0011223344, TA=7, 1/0, UL DL) Destroying MS object +MS(TLLI=0xf5667788, IMSI=0011223344, TA=7, 1/0, DL) Detaching TBF: TBF(TFI=1 TLLI=0xf5667788 DIR=UL STATE=FLOW) +MS(TLLI=0xf5667788, IMSI=0011223344, TA=7, 1/0,) Detaching TBF: TBF(TFI=0 TLLI=0xf5667788 DIR=DL STATE=FINISHED) === end test_tbf_ra_update_rach === === start test_tbf_dl_flow_and_rach_two_phase === PDCH(bts=0,trx=0,ts=7) PDCH state: disabled => enabled @@ -2343,6 +2421,9 @@ TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FLOW) Frame 1 starts at offset 0, length=20, is_complete=0 TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FLOW) No gaps in received block, last block: BSN=0 CV=15 New MS: TLLI = 0xf1223344, TA = 7, IMSI = 0011223344, LLC = 2 +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/0, UL DL) Destroying MS object +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/0, DL) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FLOW) +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/0,) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN) === end test_tbf_dl_flow_and_rach_two_phase === === start test_tbf_dl_flow_and_rach_single_phase === PDCH(bts=0,trx=0,ts=7) PDCH state: disabled => enabled @@ -2516,6 +2597,8 @@ UL_ACK_TBF(UL-TFI_0){NONE}: Received Event SCHED_ACK UL_ACK_TBF(UL-TFI_0){NONE}: state_chg to SCHED_UL_ACK New MS: TLLI = 0xf1223344, TA = 7, IMSI = 0011223344, LLC = 2 +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/0, UL) Destroying MS object +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/0,) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FINISHED) === end test_tbf_dl_flow_and_rach_single_phase === === start test_tbf_dl_reuse === PDCH(bts=0,trx=0,ts=7) PDCH state: disabled => enabled @@ -3572,6 +3655,9 @@ TBF(TFI=1 TLLI=0xf1223344 DIR=DL STATE=FINISHED) starting timer T3191 [final block (DL-TBF)] with 5 sec. 0 microsec, cur_fn=2654413 TBF(TFI=1 TLLI=0xf1223344 DIR=DL STATE=FINISHED) Scheduled Ack/Nack polling on FN=2654461, TS=7 TBF(TFI=1 TLLI=0xf1223344 DIR=DL STATE=FINISHED) msg block (BSN 10, CS-1): 0f 03 14 4d 43 20 50 41 43 4b 45 54 20 30 39 20 28 54 42 46 20 32 29 +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/0, UL DL) Destroying MS object +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/0, DL) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=RELEASING) +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/0,) Detaching TBF: TBF(TFI=1 TLLI=0xf1223344 DIR=DL STATE=FINISHED) === end test_tbf_dl_reuse === === start test_tbf_gprs_egprs === PDCH(bts=0,trx=0,ts=4) PDCH state: disabled => enabled @@ -3606,6 +3692,8 @@ - TRX=0 (0) TS=4 TA=220 TBF(TFI=0 TLLI=0xc0006789 DIR=DL STATE=ASSIGN) appending 256 bytes === end test_tbf_gprs_egprs === +MS(TLLI=0xc0006789, IMSI=001001123456789, TA=220, 45/0, DL) Destroying MS object +MS(TLLI=0xc0006789, IMSI=001001123456789, TA=220, 45/0,) Detaching TBF: TBF(TFI=0 TLLI=0xc0006789 DIR=DL STATE=ASSIGN) === start test_tbf_ws === PDCH(bts=0,trx=0,ts=4) PDCH state: disabled => enabled PDCH(bts=0,trx=0,ts=2) PDCH state: disabled => enabled @@ -3807,6 +3895,9 @@ TBF(DL-TFI_0){NEW}: state_chg to ASSIGN TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) Starting timer X2001 [assignment (PACCH)] with 2 sec. 0 microsec TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) appending 256 bytes +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1, UL DL) Destroying MS object +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1, DL) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FLOW EGPRS) +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1,) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) === end test_tbf_egprs_two_phase === === start test_tbf_egprs_two_phase_spb === PDCH(bts=0,trx=0,ts=7) PDCH state: disabled => enabled @@ -4038,6 +4129,9 @@ TBF(DL-TFI_0){NEW}: state_chg to ASSIGN TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) Starting timer X2001 [assignment (PACCH)] with 2 sec. 0 microsec TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) appending 256 bytes +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1, UL DL) Destroying MS object +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1, DL) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FLOW EGPRS) +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1,) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) === end test_tbf_egprs_two_phase_spb === === start test_tbf_egprs_dl === PDCH(bts=0,trx=0,ts=4) PDCH state: disabled => enabled @@ -6886,6 +6980,9 @@ TBF(DL-TFI_0){NEW}: state_chg to ASSIGN TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) Starting timer X2001 [assignment (PACCH)] with 2 sec. 0 microsec TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) appending 256 bytes +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1, UL DL) Destroying MS object +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1, DL) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FLOW EGPRS) +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1,) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) === end test_tbf_puan_urbb_len === === start test_tbf_update_ws === PDCH(bts=0,trx=0,ts=4) PDCH state: disabled => enabled @@ -7069,6 +7166,9 @@ TBF(DL-TFI_0){NEW}: state_chg to ASSIGN TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) Starting timer X2001 [assignment (PACCH)] with 2 sec. 0 microsec TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) appending 256 bytes +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1, UL DL) Destroying MS object +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1, DL) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FLOW EGPRS) +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1,) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) === end test_tbf_li_decoding === === start test_tbf_epdan_out_of_rx_window === PDCH(bts=0,trx=0,ts=4) PDCH state: disabled => enabled @@ -7348,6 +7448,20 @@ No PDCH resource for Uplink TBF Tx Immediate Assignment Reject on AGCH MS(TLLI=0xffffffff, IMSI=, TA=220, 0/0,) Destroying MS object +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0, UL) Destroying MS object +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0,) Detaching TBF: TBF(TFI=6 TLLI=0xffffffff DIR=UL STATE=FLOW) +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0, UL) Destroying MS object +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0,) Detaching TBF: TBF(TFI=5 TLLI=0xffffffff DIR=UL STATE=FLOW) +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0, UL) Destroying MS object +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0,) Detaching TBF: TBF(TFI=4 TLLI=0xffffffff DIR=UL STATE=FLOW) +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0, UL) Destroying MS object +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0,) Detaching TBF: TBF(TFI=3 TLLI=0xffffffff DIR=UL STATE=FLOW) +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0, UL) Destroying MS object +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0,) Detaching TBF: TBF(TFI=2 TLLI=0xffffffff DIR=UL STATE=FLOW) +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0, UL) Destroying MS object +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0,) Detaching TBF: TBF(TFI=1 TLLI=0xffffffff DIR=UL STATE=FLOW) +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0, UL) Destroying MS object +MS(TLLI=0xffffffff, IMSI=, TA=7, 0/0,) Detaching TBF: TBF(TFI=0 TLLI=0xffffffff DIR=UL STATE=FLOW) === end test_immediate_assign_rej_multi_block === === start test_immediate_assign_rej_single_block === PDCH(bts=0,trx=0,ts=7) PDCH state: disabled => enabled @@ -8620,6 +8734,9 @@ Got 'TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FLOW EGPRS)', TA=7 Got MS: TLLI = 0xf1223344, TA = 7 TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) appending 256 bytes +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1, UL DL) Destroying MS object +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1, DL) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=UL STATE=FLOW EGPRS) +MS(TLLI=0xf1223344, IMSI=0011223344, TA=7, 1/1,) Detaching TBF: TBF(TFI=0 TLLI=0xf1223344 DIR=DL STATE=ASSIGN EGPRS) === end test_tbf_egprs_two_phase_puan === === start test_packet_access_rej_epdan === PDCH(bts=0,trx=0,ts=4) PDCH state: disabled => enabled @@ -8666,6 +8783,8 @@ UL_ASS_TBF(DL-TFI_0){SEND_ASS_REJ}: Received Event CREATE_RLCMAC_MSG UL_ASS_TBF(DL-TFI_0){SEND_ASS_REJ}: state_chg to NONE packet reject: 40 84 7f f7 6e e6 7e ab 2b 2b 2b 2b 2b 2b 2b 2b 2b 2b 2b 2b 2b 2b 2b +MS(TLLI=0xffeeddcc, IMSI=, TA=0, 11/11, DL) Destroying MS object +MS(TLLI=0xffeeddcc, IMSI=, TA=0, 11/11,) Detaching TBF: TBF(TFI=0 TLLI=0xffeeddcc DIR=DL STATE=FLOW EGPRS) === end test_packet_access_rej_epdan === === start test_packet_access_rej_prr === PDCH(bts=0,trx=0,ts=7) PDCH state: disabled => enabled @@ -9124,6 +9243,22 @@ TBF(TFI=0 TLLI=0xffeeddcc DIR=UL STATE=ASSIGN EGPRS) Scheduled UL Assignment polling on PACCH (FN=2654231, TS=7) UL_ASS_TBF(UL-TFI_0){SEND_ASS}: state_chg to WAIT_ACK PDCH(bts=0,trx=0,ts=7) FN=2654218 Scheduling control message at RTS for TBF(TFI=0 TLLI=0xffeeddcc DIR=UL STATE=ASSIGN EGPRS) +MS(TLLI=0xffeeddd3, IMSI=, TA=7, 11/11, UL) Destroying MS object +MS(TLLI=0xffeeddd3, IMSI=, TA=7, 11/11,) Detaching TBF: TBF(TFI=-1 TLLI=0xffeeddd3 DIR=UL STATE=ASSIGN) +MS(TLLI=0xffeeddd2, IMSI=, TA=7, 11/11, UL) Destroying MS object +MS(TLLI=0xffeeddd2, IMSI=, TA=7, 11/11,) Detaching TBF: TBF(TFI=6 TLLI=0xffeeddd2 DIR=UL STATE=ASSIGN EGPRS) +MS(TLLI=0xffeeddd1, IMSI=, TA=7, 11/11, UL) Destroying MS object +MS(TLLI=0xffeeddd1, IMSI=, TA=7, 11/11,) Detaching TBF: TBF(TFI=5 TLLI=0xffeeddd1 DIR=UL STATE=ASSIGN EGPRS) +MS(TLLI=0xffeeddd0, IMSI=, TA=7, 11/11, UL) Destroying MS object +MS(TLLI=0xffeeddd0, IMSI=, TA=7, 11/11,) Detaching TBF: TBF(TFI=4 TLLI=0xffeeddd0 DIR=UL STATE=ASSIGN EGPRS) +MS(TLLI=0xffeeddcf, IMSI=, TA=7, 11/11, UL) Destroying MS object +MS(TLLI=0xffeeddcf, IMSI=, TA=7, 11/11,) Detaching TBF: TBF(TFI=3 TLLI=0xffeeddcf DIR=UL STATE=ASSIGN EGPRS) +MS(TLLI=0xffeeddce, IMSI=, TA=7, 11/11, UL) Destroying MS object +MS(TLLI=0xffeeddce, IMSI=, TA=7, 11/11,) Detaching TBF: TBF(TFI=2 TLLI=0xffeeddce DIR=UL STATE=ASSIGN EGPRS) +MS(TLLI=0xffeeddcd, IMSI=, TA=7, 11/11, UL) Destroying MS object +MS(TLLI=0xffeeddcd, IMSI=, TA=7, 11/11,) Detaching TBF: TBF(TFI=1 TLLI=0xffeeddcd DIR=UL STATE=ASSIGN EGPRS) +MS(TLLI=0xffeeddcc, IMSI=, TA=7, 11/11, UL) Destroying MS object +MS(TLLI=0xffeeddcc, IMSI=, TA=7, 11/11,) Detaching TBF: TBF(TFI=0 TLLI=0xffeeddcc DIR=UL STATE=ASSIGN EGPRS) === end test_packet_access_rej_prr === === start test_packet_access_rej_prr_no_other_tbfs === PDCH(bts=0,trx=0,ts=7) PDCH state: disabled => enabled -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/28078 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: Idf3ad8b70c97313dec38a49d48cf36d6c9b404c7 Gerrit-Change-Number: 28078 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

Change in osmo-pcu[master]: TbfTest: Reset MS timeout to 0 in test_tbf_dl_llc_loss()
by pespin 09 May '22

09 May '22

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-pcu/+/28077 ) Change subject: TbfTest: Reset MS timeout to 0 in test_tbf_dl_llc_loss() ...................................................................... TbfTest: Reset MS timeout to 0 in test_tbf_dl_llc_loss() The timeout is set to a high value to avoid freeing the MS during main loop iteration. Let's set it back to 0 to have a common free path in test infrastructure. Change-Id: I6dc4765163fde1a46574b49f3185aea65391e0d0 --- M tests/tbf/TbfTest.cpp 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/77/28077/1 diff --git a/tests/tbf/TbfTest.cpp b/tests/tbf/TbfTest.cpp index 585496a..eb85f60 100644 --- a/tests/tbf/TbfTest.cpp +++ b/tests/tbf/TbfTest.cpp @@ -622,6 +622,8 @@ fprintf(stderr, "=== end %s ===\n", __func__); + /* Restore MS release timeout to 0 to make sure it is freed immediatelly: */ + ms_set_timeout(ms, 0); TALLOC_FREE(the_pcu); } -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/28077 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: I6dc4765163fde1a46574b49f3185aea65391e0d0 Gerrit-Change-Number: 28077 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

Change in osmo-pcu[master]: tests/tbf: Set up pcu timers in prepare_pcu()
by pespin 09 May '22

09 May '22

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-pcu/+/28074 ) Change subject: tests/tbf: Set up pcu timers in prepare_pcu() ...................................................................... tests/tbf: Set up pcu timers in prepare_pcu() Change-Id: I30a491f82cced558ad72108c3c5ed7015d1d4d3b --- M tests/tbf/TbfTest.cpp 1 file changed, 2 insertions(+), 3 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/74/28074/1 diff --git a/tests/tbf/TbfTest.cpp b/tests/tbf/TbfTest.cpp index d077852..4651269 100644 --- a/tests/tbf/TbfTest.cpp +++ b/tests/tbf/TbfTest.cpp @@ -67,6 +67,8 @@ { struct gprs_pcu *pcu = gprs_pcu_alloc(tall_pcu_ctx); bssgp_set_bssgp_callback(gprs_gp_send_test_cb, NULL); + osmo_tdef_set(pcu->T_defs, -2030, 0, OSMO_TDEF_S); + osmo_tdef_set(pcu->T_defs, -2031, 0, OSMO_TDEF_S); return pcu; } @@ -190,10 +192,7 @@ the_pcu->alloc_algorithm = alloc_algorithm_a; bts->initial_cs_dl = cs; bts->initial_cs_ul = cs; - osmo_tdef_set(the_pcu->T_defs, -2030, 0, OSMO_TDEF_S); - osmo_tdef_set(the_pcu->T_defs, -2031, 0, OSMO_TDEF_S); trx = &bts->trx[0]; - trx->pdch[ts_no].enable(); bts_set_current_frame_number(bts, DUMMY_FN); } -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/28074 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: I30a491f82cced558ad72108c3c5ed7015d1d4d3b Gerrit-Change-Number: 28074 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

Change in osmo-pcu[master]: gprs_pcu: Explicitly free all bts objects in list before freeing pcu
by pespin 09 May '22

09 May '22

pespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-pcu/+/28075 ) Change subject: gprs_pcu: Explicitly free all bts objects in list before freeing pcu ...................................................................... gprs_pcu: Explicitly free all bts objects in list before freeing pcu This is mostly important to unit tests, where the gprs_pcu object is recreated several times (it is not recreated in normal operation). The BTS objects are already being freed through talloc dependency tree. However, since they may trigger counter updates, access to the pcu list, etc, let's better do it explicitly before erasing gprs_pcu object fields. Related: OS#5555 Change-Id: If2a8360e214d993a8a89993532ff1099b30bdbb1 --- M src/gprs_pcu.c 1 file changed, 4 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/75/28075/1 diff --git a/src/gprs_pcu.c b/src/gprs_pcu.c index e8dba06..ecb7f29 100644 --- a/src/gprs_pcu.c +++ b/src/gprs_pcu.c @@ -61,6 +61,10 @@ static int gprs_pcu_talloc_destructor(struct gprs_pcu *pcu) { + struct gprs_rlcmac_bts *bts; + while ((bts = llist_first_entry_or_null(&pcu->bts_list, struct gprs_rlcmac_bts, list))) + talloc_free(bts); + if (osmo_timer_pending(&pcu->update_stats_timer)) osmo_timer_del(&pcu->update_stats_timer); neigh_cache_free(pcu->neigh_cache); -- To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/28075 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-pcu Gerrit-Branch: master Gerrit-Change-Id: If2a8360e214d993a8a89993532ff1099b30bdbb1 Gerrit-Change-Number: 28075 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

2025

2024

2023

2022

gerrit-log