Visit https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/s…
Package network:osmocom:nightly/sofia-sip failed to build in openSUSE_Factory_ARM/armv7l
Check out the package for editing:
osc checkout network:osmocom:nightly sofia-sip
Last lines of build log:
[ 25s] checking target system type... armv7hl-suse-linux-gnueabi
[ 25s] checking cached information... ok
[ 25s] checking for a BSD-compatible install... /usr/bin/install -c
[ 25s] checking whether build environment is sane... yes
[ 25s] checking for a thread-safe mkdir -p... /usr/bin/mkdir -p
[ 25s] checking for gawk... gawk
[ 25s] checking whether make sets $(MAKE)... yes
[ 25s] checking whether make supports nested variables... yes
[ 25s] checking whether to enable maintainer-specific portions of Makefiles... no
[ 25s] checking for style of include used by make... GNU
[ 25s] checking for armv7hl-suse-linux-gnueabi-gcc... no
[ 25s] checking for gcc... no
[ 25s] checking for armv7hl-suse-linux-gnueabi-gcc... gcc
[ 25s] checking whether the C compiler works... no
[ 25s] configure: error: in `/home/abuild/rpmbuild/BUILD/sofia-sip-1.12.11+20110422.1':
[ 25s] configure: error: C compiler cannot create executables
[ 25s] See `config.log' for more details
[ 25s] error: Bad exit status from /var/tmp/rpm-tmp.DTbxDb (%build)
[ 25s]
[ 25s]
[ 25s] RPM build errors:
[ 25s] Bad exit status from /var/tmp/rpm-tmp.DTbxDb (%build)
[ 25s] ### VM INTERACTION START ###
[ 26s] [ 15.535111][ T1] sysrq: Power Off
[ 26s] [ 15.536883][ T8] reboot: Power down
[ 26s] ### VM INTERACTION END ###
[ 26s]
[ 26s] obs-arm-11 failed "build sofia-sip.spec" at Tue May 10 17:24:53 UTC 2022.
[ 26s]
--
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)
Visit https://build.opensuse.org/package/live_build_log/network:osmocom:latest/os…
Package network:osmocom:latest/osmo-mgw failed to build in openSUSE_Factory_ARM/armv7l
Check out the package for editing:
osc checkout network:osmocom:latest osmo-mgw
Last lines of build log:
[ 22s] checking for a race-free mkdir -p... /usr/bin/mkdir -p
[ 22s] checking for gawk... gawk
[ 22s] checking whether make sets $(MAKE)... yes
[ 22s] checking whether make supports nested variables... yes
[ 22s] checking whether make supports nested variables... (cached) yes
[ 22s] checking whether make sets $(MAKE)... (cached) yes
[ 22s] checking for armv7hl-suse-linux-gnueabi-gcc... no
[ 22s] checking for gcc... no
[ 22s] checking for armv7hl-suse-linux-gnueabi-cc... no
[ 22s] checking for cc... no
[ 22s] checking for armv7hl-suse-linux-gnueabi-cl.exe... no
[ 22s] checking for cl.exe... no
[ 22s] checking for armv7hl-suse-linux-gnueabi-clang... no
[ 22s] checking for clang... no
[ 22s] configure: error: in `/home/abuild/rpmbuild/BUILD/osmo-mgw-1.9.1':
[ 22s] configure: error: no acceptable C compiler found in $PATH
[ 22s] See `config.log' for more details
[ 22s] error: Bad exit status from /var/tmp/rpm-tmp.fj5q52 (%build)
[ 22s]
[ 22s]
[ 22s] RPM build errors:
[ 22s] Bad exit status from /var/tmp/rpm-tmp.fj5q52 (%build)
[ 22s] ### VM INTERACTION START ###
[ 23s] [ 17.387198][ T1] sysrq: Power Off
[ 23s] [ 17.387666][ T6] reboot: Power down
[ 23s] ### VM INTERACTION END ###
[ 23s]
[ 23s] obs-arm-11 failed "build osmo-mgw.spec" at Tue May 10 17:23:21 UTC 2022.
[ 23s]
--
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)
Attention is currently required from: laforge, fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/28086 )
Change subject: measurement: fix matching of SUB frames by TDMA FN
......................................................................
Patch Set 2: Code-Review+1
(1 comment)
File src/common/measurement.c:
https://gerrit.osmocom.org/c/osmo-bts/+/28086/comment/04ac7c59_8fc2a5b4
PS1, Line 65: * There is only one *complete* block in this subset starting at FN=52.
> Clarified in the new patchset.
Thanks, it's clearer now.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/28086
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I8cc3a755a8ad4dc564439aab2298c1e97ac0592d
Gerrit-Change-Number: 28086
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: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 10 May 2022 13:54:04 +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>
Gerrit-MessageType: comment
Attention is currently required from: laforge, pespin.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/28086 )
Change subject: measurement: fix matching of SUB frames by TDMA FN
......................................................................
Patch Set 2:
(1 comment)
File src/common/measurement.c:
https://gerrit.osmocom.org/c/osmo-bts/+/28086/comment/ffd57e6f_f42da8d8
PS1, Line 65: * There is only one *complete* block in this subset starting at FN=52.
> The first FN of incomplete block {52, 53, 54, 55} is actually several FNs earlier, because on TCH/F […]
Clarified in the new patchset.
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/28086
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I8cc3a755a8ad4dc564439aab2298c1e97ac0592d
Gerrit-Change-Number: 28086
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: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 10 May 2022 13:52:07 +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>
Gerrit-MessageType: comment
Attention is currently required from: pespin.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/28085 )
Change subject: measurement: move SACCH detection to process_l1sap_meas_data()
......................................................................
Patch Set 2:
(1 comment)
File src/common/l1sap.c:
https://gerrit.osmocom.org/c/osmo-bts/+/28085/comment/2e91b173_8e17a3f1
PS1, Line 725: .is_sub = info_meas_ind->is_sub,
> This is done on purpose because in AMR mode the caller of this function sets .is_sub itself. […]
Done
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/28085
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I507e96ee34ac0f8b7a2a6f16a8c7f92bc467df60
Gerrit-Change-Number: 28085
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 10 May 2022 13:50:37 +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>
Gerrit-MessageType: comment
Attention is currently required from: pespin.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-bts/+/28085
to look at the new patch set (#2).
Change subject: measurement: move SACCH detection to process_l1sap_meas_data()
......................................................................
measurement: move SACCH detection to process_l1sap_meas_data()
SACCH detection can be simplified by checking the RSL Link ID in
process_l1sap_meas_data(). This eliminates the need to lookup
the multiframe position by calling trx_sched_is_sacch_fn(), which
definitely takes more CPU time than just L1SAP_IS_LINK_SACCH().
Calling trx_sched_is_sacch_fn() is still required for BTS models
reporting the measurements via PRIM_MPH_INFO (legacy way),
separately from the related Uplink blocks.
This patch can be summarized as follows:
* detect SACCH and set .is_sub=1 in process_l1sap_meas_data();
** for PRIM_MPH_INFO use trx_sched_is_sacch_fn();
** for PRIM_PH_DATA use L1SAP_IS_LINK_SACCH();
* do not call trx_sched_is_sacch_fn() from ts45008_83_is_sub();
* modify the unit test - test_ts45008_83_is_sub_single();
** remove test_ts45008_83_is_sub_is_sacch().
Change-Id: I507e96ee34ac0f8b7a2a6f16a8c7f92bc467df60
Related: SYS#5853
---
M src/common/l1sap.c
M src/common/measurement.c
M tests/meas/meas_test.c
3 files changed, 16 insertions(+), 46 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/85/28085/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/28085
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I507e96ee34ac0f8b7a2a6f16a8c7f92bc467df60
Gerrit-Change-Number: 28085
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset