Attention is currently required from: dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/31214 )
Change subject: mgcp_sdp: add fmtp string to define HR GSM RTP format
......................................................................
Patch Set 6:
(3 comments)
File src/libosmo-mgcp/mgcp_network.c:
https://gerrit.osmocom.org/c/osmo-mgw/+/31214/comment/c06f4857_858a89db
PS6, Line 704: if (msgb_length(msg) == GSM_HR_BYTES + sizeof(struct rtp_hdr)) {
Not criticial, but you could even have a switch case here: switch(msgb_length(msg))
https://gerrit.osmocom.org/c/osmo-mgw/+/31214/comment/354e9fd5_7e1a58ed
PS6, Line 705: /* Abort early in case the input format (TS 101318) already
Abort sounds like failing here. "Do nothing in case" would be far more informative imho.
https://gerrit.osmocom.org/c/osmo-mgw/+/31214/comment/b6c2fe05_b254f68a
PS6, Line 1250: } else if ((rtp_end->rfc5993_hr_convert ||
Sounds like we want to have some function and put all this conditions together? It starts to be difficult to read it, and this is quite a main data code path.
--
To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/31214
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: Idde8da27fd335dc03b8fbd9e0fedc1491b77e9e4
Gerrit-Change-Number: 31214
Gerrit-PatchSet: 6
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 20 Feb 2023 15:32:09 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: fixeria.
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/docker-playground/+/31413 )
Change subject: open5gs-latest: add osmocom user for ttcn3-pgw-test
......................................................................
Patch Set 1:
(2 comments)
File open5gs-latest/Dockerfile:
https://gerrit.osmocom.org/c/docker-playground/+/31413/comment/9cc13e78_9b9…
PS1, Line 34: crate
create
https://gerrit.osmocom.org/c/docker-playground/+/31413/comment/a3105bad_748…
PS1, Line 35: ARG username=osmocom
what's the advantage of making this an argument, instead of just hardcoding a username?
--
To view, visit https://gerrit.osmocom.org/c/docker-playground/+/31413
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I9139fc52fa6973cf495559d3c4202d9f804a2079
Gerrit-Change-Number: 31413
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 20 Feb 2023 15:26:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: neels.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/31410 )
Change subject: hnbgw: f_iuh2iu_disconnect: allow more time
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/31410
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: Ibf6eaeb1b82d43e4f208f64a71f2f6e889883a11
Gerrit-Change-Number: 31410
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 20 Feb 2023 15:24:37 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: osmith.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31408 )
Change subject: gsm0808_dec_channel_type: fix dec of ch_rate_type
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31408
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ib5fba18eb82736c4f52f315ae1197159b7090e69
Gerrit-Change-Number: 31408
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 20 Feb 2023 15:12:00 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment