Attention is currently required from: fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/31455 )
Change subject: library/DIAMETER_Emulation: add API for generating Session-Id AVP
......................................................................
Patch Set 1:
(3 comments)
File library/DIAMETER_Emulation.ttcn:
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/31455/comment/da10e8cd_95d6…
PS1, Line 90: var uint32_t g_sess_id_seed_h32;
I suspect that's because uint64_t is not supported in TTCN3?
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/31455/comment/6b385550_0ce0…
PS1, Line 393: [] DIAMETER_PROC.getcall(DIAMETEREM_gen_sess_id:{?,?}) -> param(identity) {
ah so they are encoded visibly as 2 parts, weird.
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/31455/comment/4697c5d3_fd44…
PS1, Line 397: { g_sess_id_seed_h32 := g_sess_id_seed_h32 + 1; }
totally strange formatting here.
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/31455
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: I0fbd271ad123e9663aab5163915e1edbf5915bb6
Gerrit-Change-Number: 31455
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 22 Feb 2023 13:38:27 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31456 )
Change subject: i460_mux: add define constant for maximum number of subchannels
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31456
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Id3a3782927c7dcbc873223d56129f291c04fee26
Gerrit-Change-Number: 31456
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 22 Feb 2023 13:34:38 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-abis/+/31454 )
Change subject: trau_pcu_ericsson: add testvectors for MCS1 and MCS2
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/31454
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: I7e7e35930a373c9db74faef24f6c404eb5516278
Gerrit-Change-Number: 31454
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 22 Feb 2023 13:31:03 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: laforge.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31452 )
Change subject: New unit test for XOR-2G authentication
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31452
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I7014258751624ff18c51912b6348c3cd876bb23f
Gerrit-Change-Number: 31452
Gerrit-PatchSet: 3
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Wed, 22 Feb 2023 13:29:00 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: msuraev.
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/31362 )
Change subject: pcu_l1_if: get rid of l1if checks
......................................................................
Patch Set 2:
(1 comment)
File src/pcu_l1_if.cpp:
https://gerrit.osmocom.org/c/osmo-pcu/+/31362/comment/4e67878e_943266d8
PS2, Line 219: msg->data, msg->len);
This patch changes the code behavior a bit: Even when fl1h is NULL we still go ahead. l1if_pdch_req checks fl1h so we won't crash, the msg buffer is freed and we return without doing anything. In the old code we would continue with the pcu_tx_data_req and behave like a PCU with no direct PHY support at all. I am not sure if this is good.
However, I think the new behavior is better since in case a PHY is selected and for some reason couldn't open properly so that fl1h is NULL it shouldn't fall back to the behavior of a PCU without direct PHY access. Its not even clear if the other side supports the indirect way via pcu_sock at all.
--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/31362
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I54c5de04382e0bd0dbbf233eaffb403fc492d070
Gerrit-Change-Number: 31362
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: msuraev <msuraev(a)sysmocom.de>
Gerrit-Attention: msuraev <msuraev(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 22 Feb 2023 12:38:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: pespin.
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/31438 )
Change subject: pcu_l1_if_phy: add new API call l1if_disconnect_pdch
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> So you are adding a new API but I see it used nowhere?
I am using it in the follow up patch that integrates the Ericsson support. Should we merge it there?
--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/31438
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I4cc72f032f0abdd5833cdd3b1fe68c69394d89a4
Gerrit-Change-Number: 31438
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 22 Feb 2023 12:30:11 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment