Attention is currently required from: fixeria.
Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-trx/+/43611?usp=email
to look at the new patch set (#5).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: proxy: implement freq. hopping
......................................................................
proxy: implement freq. hopping
Handle TRXC SETFH ("<HSN> <MAIO> <RXF1> <TXF1> [... <RXFN> <TXFN>]"),
storing a per-channel struct proxy_trx_fh (HSN, MAIO, and the parsed
Mobile Allocation of Rx/Tx frequency pairs, struct proxy_trx_fh_freq).
burst_fwd.c resolves both the source channel's Tx frequency and each
candidate destination channel's Rx frequency per TDMA frame number via
proxy_trx_fh_resolve() whenever a channel has hopping configured,
falling back to the static RXTUNE/TXTUNE frequency otherwise.
Change-Id: I574b4d4dc337f771b044bd868088b29db9902de9
Related: OS#6672
---
M proxy/include/osmocom/proxy/trx.h
M proxy/src/burst_fwd.c
M proxy/src/ctrl_cmd.c
M proxy/src/trx.c
4 files changed, 163 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/11/43611/5
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43611?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I574b4d4dc337f771b044bd868088b29db9902de9
Gerrit-Change-Number: 43611
Gerrit-PatchSet: 5
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-trx/+/43630?usp=email
to look at the new patch set (#4).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: proxy: reset all per-chan parameters on POWEROFF
......................................................................
proxy: reset all per-chan parameters on POWEROFF
Change-Id: Ife14ed20614d39c385531ba0fe4fd728cebc6675
Related: OS#6672
---
M proxy/src/trx.c
1 file changed, 24 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/30/43630/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43630?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Ife14ed20614d39c385531ba0fe4fd728cebc6675
Gerrit-Change-Number: 43630
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy(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>
Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-trx/+/43631?usp=email
to look at the new patch set (#4).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: proxy: make max TRXD PDU version VTY-configurable
......................................................................
proxy: make max TRXD PDU version VTY-configurable
Change-Id: Ibd384ee57a728b709b1e43726b46c8c3840b5535
Related: OS#6672
---
M proxy/include/osmocom/proxy/trx.h
M proxy/src/ctrl_cmd.c
M proxy/src/trx.c
M proxy/src/vty.c
4 files changed, 21 insertions(+), 2 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/31/43631/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43631?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Ibd384ee57a728b709b1e43726b46c8c3840b5535
Gerrit-Change-Number: 43631
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy(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>
Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-trx/+/43632?usp=email
to look at the new patch set (#4).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: proxy: make TRXDv2+ PDU batching VTY-configurable
......................................................................
proxy: make TRXDv2+ PDU batching VTY-configurable
Change-Id: I56c27db81d03bfbe58b4960e031a2f805df6e21b
Related: OS#6672
---
M proxy/src/vty.c
1 file changed, 30 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/32/43632/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43632?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I56c27db81d03bfbe58b4960e031a2f805df6e21b
Gerrit-Change-Number: 43632
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy(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>
Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-trx/+/43633?usp=email
to look at the new patch set (#4).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: proxy: check for PC clock skew
......................................................................
proxy: check for PC clock skew
This replicates the clock skew detection logic that exists in
osmo-bts-trx (see trx_fn_timer_cb()).
Change-Id: I256ff6a4f4a05ea5507b5d9395a946672ff4e306
Related: OS#6672
---
M proxy/src/clck_gen.c
1 file changed, 37 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/33/43633/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43633?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I256ff6a4f4a05ea5507b5d9395a946672ff4e306
Gerrit-Change-Number: 43633
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy(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>
Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-trx/+/43634?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: doc/examples: add a config file for osmo-trx-proxy
......................................................................
doc/examples: add a config file for osmo-trx-proxy
Change-Id: I401753bd1977057c3de481e7bf074d1f92f84499
Related: OS#6672
---
M doc/examples/Makefile.am
A doc/examples/osmo-trx-proxy/osmo-trx-proxy.cfg
2 files changed, 39 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/34/43634/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43634?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I401753bd1977057c3de481e7bf074d1f92f84499
Gerrit-Change-Number: 43634
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanitskiy(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>
Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-trx/+/43636?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: contrib/systemd: add a service file for osmo-trx-proxy
......................................................................
contrib/systemd: add a service file for osmo-trx-proxy
Add osmo-trx-proxy.service, and package it in osmo-trx-proxy along
with the postinst logic to create the 'osmocom' system user/group
it runs as, matching the other osmo-trx-* backends.
Change-Id: I8c5bc092dc5bf6bf07d6e80f64ac7fc06f0cfb96
Related: OS#6672
---
M contrib/systemd/Makefile.am
A contrib/systemd/osmo-trx-proxy.service
M debian/osmo-trx-proxy.install
A debian/osmo-trx-proxy.postinst
4 files changed, 52 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/36/43636/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43636?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I8c5bc092dc5bf6bf07d6e80f64ac7fc06f0cfb96
Gerrit-Change-Number: 43636
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanitskiy(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>
Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-trx/+/43635?usp=email
to look at the new patch set (#3).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: debian: add packaging for osmo-trx-proxy
......................................................................
debian: add packaging for osmo-trx-proxy
Ship the osmo-trx-proxy binary and its default/example VTY config
(doc/examples/osmo-trx-proxy/osmo-trx-proxy.cfg), following the same
layout as the osmo-trx-ipc package.
Change-Id: I995b44bf9a884cf5ba6db8333e6f08c9604a58a5
Related: OS#6672
---
M debian/control
A debian/osmo-trx-proxy.install
2 files changed, 12 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/35/43635/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43635?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I995b44bf9a884cf5ba6db8333e6f08c9604a58a5
Gerrit-Change-Number: 43635
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanitskiy(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>
Attention is currently required from: fixeria.
Hello Jenkins Builder, laforge, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-trx/+/43639?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
The change is no longer submittable: Verified is unsatisfied now.
Change subject: proxy/vty: add 'show ep [NAME]', rename 'show proxy' to 'show ep-list'
......................................................................
proxy/vty: add 'show ep [NAME]', rename 'show proxy' to 'show ep-list'
'show ep-list' keeps the brief one-line-per-endpoint summary (now
also showing the full address and power state), while the new
'show ep [NAME]' prints detailed per-channel information.
Change-Id: Ib0af5a47e7298bb0e961652c7a83bd0f6233efda
Related: OS#6672
---
M proxy/src/vty.c
1 file changed, 117 insertions(+), 9 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/39/43639/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43639?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Ib0af5a47e7298bb0e961652c7a83bd0f6233efda
Gerrit-Change-Number: 43639
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Attention is currently required from: pespin.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-trx/+/43612?usp=email )
Change subject: proxy: queue BURST.req and dispatch per TDMA frame tick
......................................................................
Patch Set 4:
(1 comment)
File proxy/src/burst_fwd.c:
https://gerrit.osmocom.org/c/osmo-trx/+/43612/comment/4a483cfe_5dba5890?usp… :
PS4, Line 156: llist_add(&msg->list, &cur->list);
> but shouldn't this be llist_add_tail now?
No. Despite we're iterating the list in reverse, we still want to add the new entry right after the `cur`, not before. Using `llist_add_tail` would break the sort (new entry would land ahead of an entry it must come after).
--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/43612?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Ib147983e97638d2a2e487ccf76184912caf0941d
Gerrit-Change-Number: 43612
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Sat, 19 Sep 2026 17:49:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>