Attention is currently required from: daniel.
pespin has posted comments on this change by daniel. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/42064?usp=email )
Change subject: tcap_as_loadshare: Fix asp selection if no tcap-range was found
......................................................................
Patch Set 1:
(1 comment)
File src/tcap_as_loadshare.c:
https://gerrit.osmocom.org/c/libosmo-sigtran/+/42064/comment/de9ec6d0_acd54… :
PS1, Line 247: return asp;
> How do you return back a NULL in case non of the given asp is valid?
ACK. It probably makes sense to assign last_asp_idx_sent and return asp inside the condition in the loop, and return NULL here.
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/42064?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I07e77548ecb75df9227487901b75018261bd1232
Gerrit-Change-Number: 42064
Gerrit-PatchSet: 1
Gerrit-Owner: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 05 Feb 2026 16:54:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>
Attention is currently required from: daniel, pespin.
lynxis lazus has posted comments on this change by daniel. ( https://gerrit.osmocom.org/c/libosmo-sigtran/+/42064?usp=email )
Change subject: tcap_as_loadshare: Fix asp selection if no tcap-range was found
......................................................................
Patch Set 1:
(2 comments)
File src/tcap_as_loadshare.c:
https://gerrit.osmocom.org/c/libosmo-sigtran/+/42064/comment/f7812d17_19c12… :
PS1, Line 233: struct osmo_ss7_asp *asp;
I would also set it to NULL
https://gerrit.osmocom.org/c/libosmo-sigtran/+/42064/comment/6ec7c0a0_35778… :
PS1, Line 247: return asp;
How do you return back a NULL in case non of the given asp is valid?
--
To view, visit https://gerrit.osmocom.org/c/libosmo-sigtran/+/42064?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I07e77548ecb75df9227487901b75018261bd1232
Gerrit-Change-Number: 42064
Gerrit-PatchSet: 1
Gerrit-Owner: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: daniel <dwillmann(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 05 Feb 2026 16:38:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: pespin.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-bts/+/42061?usp=email )
Change subject: power_control: lchan_ms_pwr_ctrl(): always log input values
......................................................................
Patch Set 2:
(1 comment)
File src/common/power_control.c:
https://gerrit.osmocom.org/c/osmo-bts/+/42061/comment/ce48ffd3_c87009de?usp… :
PS1, Line 233: LOGPLCHAN(lchan, DLOOP, LOGL_INFO, "Rx UL Measurement Report: "
> Good catch. `LOGL_DEBUG` was my original intention but I forgot to change it.
Acknowledged
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/42061?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I3b8466f062624fc8a2ea6f8220c057d3812f2c35
Gerrit-Change-Number: 42061
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 05 Feb 2026 14:53:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Attention is currently required from: pespin.
Hello Jenkins Builder, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-bts/+/42061?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review+1 by pespin, Verified+1 by Jenkins Builder
Change subject: power_control: lchan_ms_pwr_ctrl(): always log input values
......................................................................
power_control: lchan_ms_pwr_ctrl(): always log input values
Depending on the power control interval (P_Con_INTERVAL), the MS power
loop may delay the power control decision for a certain amount of SACCH
block periods. In this case lchan_ms_pwr_ctrl() returns early and no
logging is printed at all.
Let's always log the input values (reported level, UL measurements)
early in lchan_ms_pwr_ctrl(). This is useful for debugging, and
this is exactly what lchan_bs_pwr_ctrl() does.
Change-Id: I3b8466f062624fc8a2ea6f8220c057d3812f2c35
---
M src/common/power_control.c
M tests/power/ms_power_loop_test.err
2 files changed, 82 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/61/42061/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/42061?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I3b8466f062624fc8a2ea6f8220c057d3812f2c35
Gerrit-Change-Number: 42061
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Attention is currently required from: pespin.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmo-bts/+/42061?usp=email )
Change subject: power_control: lchan_ms_pwr_ctrl(): always log input values
......................................................................
Patch Set 1:
(1 comment)
File src/common/power_control.c:
https://gerrit.osmocom.org/c/osmo-bts/+/42061/comment/a5139c51_2079a79d?usp… :
PS1, Line 233: LOGPLCHAN(lchan, DLOOP, LOGL_INFO, "Rx UL Measurement Report: "
> shouldn't this be LOGL_DEBUG?
Good catch. `LOGL_DEBUG` was my original intention but I forgot to change it.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/42061?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I3b8466f062624fc8a2ea6f8220c057d3812f2c35
Gerrit-Change-Number: 42061
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 05 Feb 2026 14:51:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>