- gerrit-log - lists.osmocom.org

Change in osmo-cbc[master]: CBSP: Append/store results in KILL COMPLETE + KILL FAIL
by pespin 29 Apr '22

29 Apr '22

Attention is currently required from: laforge. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-cbc/+/27984 ) Change subject: CBSP: Append/store results in KILL COMPLETE + KILL FAIL ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/27984 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-cbc Gerrit-Branch: master Gerrit-Change-Id: Iabe5b4e6a238597455294bf91759d4dc6e90d660 Gerrit-Change-Number: 27984 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Fri, 29 Apr 2022 12:27:20 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in osmo-ttcn3-hacks[master]: cbc: Expect CBC to include Channel Indicator IE in CBSP KILL
by pespin 29 Apr '22

29 Apr '22

Attention is currently required from: laforge. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27983 ) Change subject: cbc: Expect CBC to include Channel Indicator IE in CBSP KILL ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27983 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Id9f280158a0370134ec4a4ca9a977d884670b27e Gerrit-Change-Number: 27983 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Fri, 29 Apr 2022 12:26:33 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in osmo-ttcn3-hacks[master]: bsc: Avoid sending CCCH Load Ind (paging) when RSL C0 becomes up
by pespin 29 Apr '22

29 Apr '22

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27957 ) Change subject: bsc: Avoid sending CCCH Load Ind (paging) when RSL C0 becomes up ...................................................................... bsc: Avoid sending CCCH Load Ind (paging) when RSL C0 becomes up The paging CCCH Load Ind should only be sent when the BTS is over the threshold, which is not the case when it starts up. The BSC should underststand that the BTS under threshold and be able to send paging requests. Change-Id: I19a97e97ec96ba8ea2e28b1be8ac3b706b43e1b7 --- M bsc/BSC_Tests.ttcn 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn index 2858cdf..26d47b6 100644 --- a/bsc/BSC_Tests.ttcn +++ b/bsc/BSC_Tests.ttcn @@ -961,7 +961,6 @@ alt { [] IPA_RSL[i].receive(tr_ASP_IPA_EV(ASP_IPA_EVENT_ID_ACK)) { T.stop; - IPA_RSL[i].send(ts_ASP_RSL_UD(ts_RSL_PAGING_LOAD_IND(23))); } [] IPA_RSL[i].receive(ASP_IPA_Event:?) { repeat } [] IPA_RSL[i].receive { repeat } -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27957 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: I19a97e97ec96ba8ea2e28b1be8ac3b706b43e1b7 Gerrit-Change-Number: 27957 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

1 0

Change in osmo-cbc[master]: CBSP: Set Channel Indication IE in KILL for CBS
by laforge 29 Apr '22

29 Apr '22

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-cbc/+/27982 ) Change subject: CBSP: Set Channel Indication IE in KILL for CBS ...................................................................... CBSP: Set Channel Indication IE in KILL for CBS The channel Indicator IE shall be present if the KILL relates to a CBS message. Its absence indicates KILL for ETWS/PWS. Change-Id: I11bfac3236d7cf1e30ae5bae0abcbd72531ab5cd Related: SYS#5906, OS#5540, OS#5541 --- M src/smscb_peer_fsm.c 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/src/smscb_peer_fsm.c b/src/smscb_peer_fsm.c index 4e431ff..49b3b1d 100644 --- a/src/smscb_peer_fsm.c +++ b/src/smscb_peer_fsm.c @@ -479,6 +479,12 @@ cbsp->u.kill.old_serial_nr = mp->cbcmsg->msg.serial_nr; /* TODO: we assume that the delete will always affect all original cells */ cbsp_append_cell_list(&cbsp->u.kill.cell_list, cbsp, mp); + if (!mp->cbcmsg->msg.is_etws) { + /* Channel Indication IE is only present in CBS, not in ETWS! */ + cbsp->u.kill.channel_ind = talloc_zero(cbsp, enum cbsp_channel_ind); + OSMO_ASSERT(cbsp->u.kill.channel_ind); + *(cbsp->u.kill.channel_ind) = CBSP_CHAN_IND_BASIC; + } cbsp_cbc_client_tx(mp->peer->client.cbsp, cbsp); osmo_fsm_inst_state_chg(fi, SMSCB_S_WAIT_DELETE_ACK, 10, T_WAIT_DELETE_ACK); break; -- To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/27982 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-cbc Gerrit-Branch: master Gerrit-Change-Id: I11bfac3236d7cf1e30ae5bae0abcbd72531ab5cd Gerrit-Change-Number: 27982 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged

1 0

Change in osmo-cbc[master]: CBSP: Set Channel Indication IE in KILL for CBS
by laforge 29 Apr '22

29 Apr '22

laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-cbc/+/27982 ) Change subject: CBSP: Set Channel Indication IE in KILL for CBS ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/27982 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-cbc Gerrit-Branch: master Gerrit-Change-Id: I11bfac3236d7cf1e30ae5bae0abcbd72531ab5cd Gerrit-Change-Number: 27982 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Comment-Date: Fri, 29 Apr 2022 11:43:13 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in osmo-cbc[master]: CBSP: Append/store results in KILL COMPLETE + KILL FAIL
by laforge 29 Apr '22

29 Apr '22

laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-cbc/+/27984 ) Change subject: CBSP: Append/store results in KILL COMPLETE + KILL FAIL ...................................................................... CBSP: Append/store results in KILL COMPLETE + KILL FAIL When the BSC returns a KILL COMPLETE or KILL FAIL, append the contained lists (number of broadcasts completed list, fail list, ...) to update the state of the CBC. Change-Id: Iabe5b4e6a238597455294bf91759d4dc6e90d660 --- M src/smscb_peer_fsm.c 1 file changed, 10 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-cbc refs/changes/84/27984/1 diff --git a/src/smscb_peer_fsm.c b/src/smscb_peer_fsm.c index 49b3b1d..08b0f3f 100644 --- a/src/smscb_peer_fsm.c +++ b/src/smscb_peer_fsm.c @@ -411,16 +411,24 @@ static void smscb_p_fsm_wait_delete_ack(struct osmo_fsm_inst *fi, uint32_t event, void *data) { struct cbc_message_peer *mp = (struct cbc_message_peer *) fi->priv; + struct osmo_cbsp_decoded *dec = NULL; switch (event) { case SMSCB_E_CBSP_DELETE_ACK: - /* TODO: append results */ + dec = data; + /* append results */ + msg_peer_append_cbsp_compl(mp, &dec->u.kill_compl.num_compl_list); + msg_peer_append_cbsp_cell(mp, &dec->u.kill_compl.cell_list); /* Signal parent fsm about completion */ osmo_fsm_inst_dispatch(fi->proc.parent, SMSCB_E_CBSP_DELETE_ACK, mp); osmo_fsm_inst_state_chg(fi, SMSCB_S_DELETED, 0, 0); break; case SMSCB_E_CBSP_DELETE_NACK: - /* TODO: append results */ + dec = data; + /* append results */ + msg_peer_append_cbsp_compl(mp, &dec->u.kill_fail.num_compl_list); + msg_peer_append_cbsp_cell(mp, &dec->u.kill_fail.cell_list); + msg_peer_append_cbsp_fail(mp, &dec->u.kill_fail.fail_list); /* Signal parent fsm about completion */ osmo_fsm_inst_dispatch(fi->proc.parent, SMSCB_E_CBSP_DELETE_NACK, mp); osmo_fsm_inst_state_chg(fi, SMSCB_S_DELETED, 0, 0); -- To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/27984 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-cbc Gerrit-Branch: master Gerrit-Change-Id: Iabe5b4e6a238597455294bf91759d4dc6e90d660 Gerrit-Change-Number: 27984 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-MessageType: newchange

1 0

Change in osmo-ttcn3-hacks[master]: cbc: Expect CBC to include Channel Indicator IE in CBSP KILL
by laforge 29 Apr '22

29 Apr '22

laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27983 ) Change subject: cbc: Expect CBC to include Channel Indicator IE in CBSP KILL ...................................................................... cbc: Expect CBC to include Channel Indicator IE in CBSP KILL This should be present for all KILL relating to CBS. Change-Id: Id9f280158a0370134ec4a4ca9a977d884670b27e Related: SYS#5906, OS#5540, OS#5541 --- M cbc/CBC_Tests.ttcn 1 file changed, 2 insertions(+), 1 deletion(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks refs/changes/83/27983/1 diff --git a/cbc/CBC_Tests.ttcn b/cbc/CBC_Tests.ttcn index 36fa55c..e76e61e 100644 --- a/cbc/CBC_Tests.ttcn +++ b/cbc/CBC_Tests.ttcn @@ -377,7 +377,8 @@ f_ecbe_tx_delete_cbs(msg.msg_id); /* FIXME: cbc segfaults if we terminate here (if we don't wait for Connect_result? */ - f_cbsp_handle_kill(0, msg.msg_id, msg.ser_nr, cell_list_success, cell_list_success); + f_cbsp_handle_kill(0, msg.msg_id, msg.ser_nr, exp_list:=cell_list_success, tx_list:=cell_list_success, + tx_fail_list:=omit, tx_compl_list:=omit, channel_ind:=msg.channel_ind); f_ecbe_rx_resp(200); } -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/27983 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Id9f280158a0370134ec4a4ca9a977d884670b27e Gerrit-Change-Number: 27983 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-MessageType: newchange

1 0

Change in osmo-cbc[master]: CBSP: Set Channel Indication IE in KILL for CBS
by laforge 29 Apr '22

29 Apr '22

laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-cbc/+/27982 ) Change subject: CBSP: Set Channel Indication IE in KILL for CBS ...................................................................... CBSP: Set Channel Indication IE in KILL for CBS The channel Indicator IE shall be present if the KILL relates to a CBS message. Its absence indicates KILL for ETWS/PWS. Change-Id: I11bfac3236d7cf1e30ae5bae0abcbd72531ab5cd Related: SYS#5906, OS#5540, OS#5541 --- M src/smscb_peer_fsm.c 1 file changed, 6 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-cbc refs/changes/82/27982/1 diff --git a/src/smscb_peer_fsm.c b/src/smscb_peer_fsm.c index 4e431ff..49b3b1d 100644 --- a/src/smscb_peer_fsm.c +++ b/src/smscb_peer_fsm.c @@ -479,6 +479,12 @@ cbsp->u.kill.old_serial_nr = mp->cbcmsg->msg.serial_nr; /* TODO: we assume that the delete will always affect all original cells */ cbsp_append_cell_list(&cbsp->u.kill.cell_list, cbsp, mp); + if (!mp->cbcmsg->msg.is_etws) { + /* Channel Indication IE is only present in CBS, not in ETWS! */ + cbsp->u.kill.channel_ind = talloc_zero(cbsp, enum cbsp_channel_ind); + OSMO_ASSERT(cbsp->u.kill.channel_ind); + *(cbsp->u.kill.channel_ind) = CBSP_CHAN_IND_BASIC; + } cbsp_cbc_client_tx(mp->peer->client.cbsp, cbsp); osmo_fsm_inst_state_chg(fi, SMSCB_S_WAIT_DELETE_ACK, 10, T_WAIT_DELETE_ACK); break; -- To view, visit https://gerrit.osmocom.org/c/osmo-cbc/+/27982 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-cbc Gerrit-Branch: master Gerrit-Change-Id: I11bfac3236d7cf1e30ae5bae0abcbd72531ab5cd Gerrit-Change-Number: 27982 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-MessageType: newchange

1 0

Change in osmo-bts[master]: pcu_sock: comment out {dl,ul}_tbf_ext related warnings
by fixeria 29 Apr '22

29 Apr '22

Attention is currently required from: pespin. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/23918 ) Change subject: pcu_sock: comment out {dl,ul}_tbf_ext related warnings ...................................................................... Patch Set 4: This change is ready for review. -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/23918 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: Iebf9961cffbd7aa20b263f7dc0016a44782dec60 Gerrit-Change-Number: 23918 Gerrit-PatchSet: 4 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Fri, 29 Apr 2022 10:37:19 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in osmo-bsc[master]: paging: Early stop work_timer when paging queue becomes empty
by fixeria 29 Apr '22

29 Apr '22

Attention is currently required from: pespin. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/27973 ) Change subject: paging: Early stop work_timer when paging queue becomes empty ...................................................................... Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/27973 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I096ab7231f52c741c5fded37acd5b309e1de06e3 Gerrit-Change-Number: 27973 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Fri, 29 Apr 2022 08:05:38 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

2025

2024

2023

2022

gerrit-log