Attention is currently required from: fixeria.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/35661?usp=email )
Change subject: mobile: VTY: use 'enum gsm48_bcap_transp' from libosmogsm
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/35661?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Iebc0692f17a5a5d04e57d74a720db52775937bdd
Gerrit-Change-Number: 35661
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 24 Jan 2024 08:39:15 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: fixeria.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/35640?usp=email )
Change subject: mobile: handle V.24 line status updates from V.110 TA
......................................................................
Patch Set 4:
(1 comment)
File src/host/layer23/src/mobile/tch_data.c:
https://gerrit.osmocom.org/c/osmocom-bb/+/35640/comment/d18acff1_e7e7b1ea
PS2, Line 210: LOGP(DL1C, LOGL_NOTICE, "V.110 TA circuit %s (%s) is %s\n",
> I agree in genral, but I don't expect the circuit state to change too often, unless the flow control […]
RTS/CTS flow control will happen *all the time* in any realistic scenario where you have a backlog of to be transmitted data. This is nothing a user ever wants to read about.
If you need higher log priority for things like DTR, then you have to differentiate between those two. But for normal flow control it should definitely not be anything > DEBUG.
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/35640?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I05ba2e10fefe3cae687831b5ced971aa244ad336
Gerrit-Change-Number: 35640
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 24 Jan 2024 08:38:48 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: neels.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-upf/+/35668?usp=email )
Change subject: manual: explain IP forwarding
......................................................................
Patch Set 1:
(1 comment)
File doc/manuals/chapters/running.adoc:
https://gerrit.osmocom.org/c/osmo-upf/+/35668/comment/0ad670f6_cbd6fc67
PS1, Line 259: and to two specific network interfaces eth0 and eth23.
> but where to put this configuration??
as I wrote in the ticket: This would be part of `/etc/nftables.conf` on a debian system. Might be useful to add that there are many distribution-specific ways to configure this, and there might also be higher-level firewall rule management software like 'ufw'. The user should configure the firewall rules just like any other forwarding firewall rules they usually configure on their system, using whatever system they have deployed.
--
To view, visit https://gerrit.osmocom.org/c/osmo-upf/+/35668?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-upf
Gerrit-Branch: master
Gerrit-Change-Id: I7b54f9203c1a77efd43f90b9a1c0105bc5c3efde
Gerrit-Change-Number: 35668
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 24 Jan 2024 08:36:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: neels.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-upf/+/35667?usp=email )
Change subject: manual: explain GTP Echo workaround for tunmap
......................................................................
Patch Set 1: Code-Review-1
(1 comment)
Patchset:
PS1:
I would remove the entire "tunend can be removed for forwarding-only" section. It's misleading. Everyone will always need to answert to GTP ECHO REQ in any spec-compliant deployment.
I'd argue the user is not interested in reading about a hypothetical scenario without echo-req/resp that never occurs in the real-world. They want to know what to do for a normal configuration.
--
To view, visit https://gerrit.osmocom.org/c/osmo-upf/+/35667?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-upf
Gerrit-Branch: master
Gerrit-Change-Id: Ic824fc876d1fad181254cb6894e51464c443b53c
Gerrit-Change-Number: 35667
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 24 Jan 2024 08:33:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: neels.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-upf/+/35666?usp=email )
Change subject: manual: 'Running': tweak, mention 'tunmap' and 'tunend'
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File doc/manuals/chapters/running.adoc:
https://gerrit.osmocom.org/c/osmo-upf/+/35666/comment/6ca9bed6_ba140fa9
PS1, Line 91: The netfilter module
Unrelated, FYI, netfilter is not actually a module. Try lsmod | grep netfilter and you won't find anything, unlike lsmod | grep gtp. Should probably best be phrased: "The netfilter framework is used" or if you want to list amodule"The netfilter framework and the nf_tables module is used".
--
To view, visit https://gerrit.osmocom.org/c/osmo-upf/+/35666?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-upf
Gerrit-Branch: master
Gerrit-Change-Id: I9760ca214933d0b05080a3e70807b0cd06380a27
Gerrit-Change-Number: 35666
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 24 Jan 2024 08:31:30 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: neels.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-upf/+/35665?usp=email )
Change subject: manual: 'Running': flatten section depths a bit
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://gerrit.osmocom.org/c/osmo-upf/+/35665?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-upf
Gerrit-Branch: master
Gerrit-Change-Id: I0bd43300aa4b45315ea58ab35c77da005d1a4fa4
Gerrit-Change-Number: 35665
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 24 Jan 2024 08:29:23 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment