fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41619?usp=email )
Change subject: s1ap_proxy: signal GTP-U addr and eNB info to sctp_proxy
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41619?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: I87e7d22a4bc5cb7816a167ac7dd29ea917594ce8
Gerrit-Change-Number: 41619
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 18 Dec 2025 21:23:00 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: pespin.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41616?usp=email )
Change subject: sctp_proxy: use lists:filter/2 in sctp_send_pending/2
......................................................................
Patch Set 2: Code-Review+2
(1 comment)
File src/sctp_proxy.erl:
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41616/comment/6f703c84_85db… :
PS1, Line 307: lists:reverse(Pending)),
> Yes, it's a normal practice in Erlang: you prepend items to a list and then do the reverse. […]
Done
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41616?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Ic666fd77bc6d04341f3c57140dea5d4a5e616e80
Gerrit-Change-Number: 41616
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 18 Dec 2025 21:22:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41615?usp=email )
Change subject: sctp_proxy: use a record for storing state data
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/41615?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: Ice9c255c0cf14db0a216bb078198b9b9c76d22a7
Gerrit-Change-Number: 41615
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 18 Dec 2025 21:22:20 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
falconia has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-bsc/+/41726?usp=email )
Change subject: vty: rm misleading warnings about DTX on E1 BTS
......................................................................
vty: rm misleading warnings about DTX on E1 BTS
Since the days of pre-split OpenBSC, whenever an operator would
enable DTX (either UL or DL) in BSC vty on an E1 BTS, there was
a warning message emitted, saying that this configuration is
"neither supported nor tested". While those statements may have
been correct in 2016 when that code was added (commit 6079528b),
they are no longer correct in the present: both DTXu and DTXd
work correctly now on E1 BTS that follow TS 48.060 or 48.061,
with all 4 supported speech codecs: FR, HR, EFR and AMR.
Full-featured, production-quality operation with E1 BTS requires
use of tw-e1abis-mgw in the place of osmo-mgw - but even when
legacy OsmoMGW-E1 is used, it is misleading to say that DTX is
"neither supported nor tested", as the architectural shortcomings
of OsmoMGW-E1 (lack of TFO transform for non-AMR codecs and lack
of any support for AMR) are not specific to DTX per se.
Change-Id: I63626b554d38b1ab094838374968e64193da1c11
---
M src/osmo-bsc/bts_vty.c
1 file changed, 0 insertions(+), 6 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/26/41726/1
diff --git a/src/osmo-bsc/bts_vty.c b/src/osmo-bsc/bts_vty.c
index bbee866..d69fb1f 100644
--- a/src/osmo-bsc/bts_vty.c
+++ b/src/osmo-bsc/bts_vty.c
@@ -210,9 +210,6 @@
struct gsm_bts *bts = vty->index;
bts->dtxu = (argc > 0) ? GSM48_DTX_SHALL_BE_USED : GSM48_DTX_MAY_BE_USED;
- if (!is_ipa_abisip_bts(bts))
- vty_out(vty, "%% DTX enabled on non-IP BTS: this configuration "
- "neither supported nor tested!%s", VTY_NEWLINE);
return CMD_SUCCESS;
}
@@ -240,9 +237,6 @@
struct gsm_bts *bts = vty->index;
bts->dtxd = true;
- if (!is_ipa_abisip_bts(bts))
- vty_out(vty, "%% DTX enabled on non-IP BTS: this configuration "
- "neither supported nor tested!%s", VTY_NEWLINE);
return CMD_SUCCESS;
}
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/41726?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I63626b554d38b1ab094838374968e64193da1c11
Gerrit-Change-Number: 41726
Gerrit-PatchSet: 1
Gerrit-Owner: falconia <falcon(a)freecalypso.org>