Build failure of network:osmocom:nightly/osmo-msc in openSUSE_Factory_ARM/armv7l

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-msc/openSUSE_Factory_ARM/armv7l

Package network:osmocom:nightly/osmo-msc failed to build in 
openSUSE_Factory_ARM/armv7l

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-msc

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was armbuild13:4)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-bts in openSUSE_Leap_15.1_ARM/armv7l

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bts/openSUSE_Leap_15.1_ARM/armv7l

Package network:osmocom:nightly/osmo-bts failed to build in 
openSUSE_Leap_15.1_ARM/armv7l

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-bts

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was obs-arm-5:6)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-gsm-manuals in Debian_10/armv7l

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-gsm-manuals/Debian_10/armv7l

Package network:osmocom:nightly/osmo-gsm-manuals failed to build in 
Debian_10/armv7l

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-gsm-manuals

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was obs-arm-6:12)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/simtrace2 in Raspbian_10/armv7l

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/Raspbian_10/armv7l

Package network:osmocom:nightly/simtrace2 failed to build in Raspbian_10/armv7l

Check out the package for editing:
  osc checkout network:osmocom:nightly simtrace2

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was armbuild13:4)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-iuh in Debian_Unstable/x86_64

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-iuh/Debian_Unstable/x86_64

Package network:osmocom:nightly/osmo-iuh failed to build in 
Debian_Unstable/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-iuh

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was lamb61:10)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-iuh in openSUSE_Tumbleweed/x86_64

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-iuh/openSUSE_Tumbleweed/x86_64

Package network:osmocom:nightly/osmo-iuh failed to build in 
openSUSE_Tumbleweed/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-iuh

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was goat17:2)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-sysmon in Debian_8.0/x86_64

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-sysmon/Debian_8.0/x86_64

Package network:osmocom:nightly/osmo-sysmon failed to build in Debian_8.0/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-sysmon

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was old-atreju5:4)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-bts in Debian_9.0/x86_64

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bts/Debian_9.0/x86_64

Package network:osmocom:nightly/osmo-bts failed to build in Debian_9.0/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-bts

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was lamb01:13)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-pcu in Debian_Testing/x86_64

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-pcu/Debian_Testing/x86_64

Package network:osmocom:nightly/osmo-pcu failed to build in 
Debian_Testing/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-pcu

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was lamb58:4)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-uecups in xUbuntu_16.04/i586

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-uecups/xUbuntu_16.04/i586

Package network:osmocom:nightly/osmo-uecups failed to build in 
xUbuntu_16.04/i586

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-uecups

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was hci-cnode2-m1:11)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/osmo-sip-connector in xUbuntu_18.10/i586

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-sip-connector/xUbuntu_18.10/i586

Package network:osmocom:nightly/osmo-sip-connector failed to build in 
xUbuntu_18.10/i586

Check out the package for editing:
  osc checkout network:osmocom:nightly osmo-sip-connector

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was lamb12:7)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/simtrace2 in xUbuntu_20.04/x86_64

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/xUbuntu_20.04/x86_64

Package network:osmocom:nightly/simtrace2 failed to build in 
xUbuntu_20.04/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly simtrace2

Last lines of build log:
[   99s] [COMPILING libboard/qmod/source/wwan_perst.c]
[   99s] [COMPILING libboard/qmod/source/card_pres.c]
[   99s] [COMPILING libboard/qmod/source/wwan_led.c]
[   99s] [COMPILING libboard/qmod/source/i2c.c]
[   99s] [COMPILING libboard/qmod/source/board_qmod.c]
[   99s] [COMPILING apps/dfu/main.c]
[   99s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[   99s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[   99s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  100s] Memory region Used Size  Region Size  %age Used
[  100s]  rom:   16588 B16 KB101.25%
[  100s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  100s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 204 bytes
[  100s] collect2: error: ld returned 1 exit status
[  100s] %
[  100s] make[2]: *** [Makefile:234: flash] Error 1
[  100s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  100s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  100s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  100s] dh_auto_build: error: make -j1 returned exit code 2
[  100s] make: *** [debian/rules:16: build] Error 25
[  100s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  100s] ### VM INTERACTION START ###
[  103s] [   96.581759] sysrq: Power Off
[  103s] [   96.586180] reboot: Power down
[  103s] ### VM INTERACTION END ###
[  103s] 
[  103s] goat05 failed "build simtrace2_0.7.0.63.39070.dsc" at Wed Oct 14 
01:46:02 UTC 2020.
[  103s] 

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/libusrp in xUbuntu_18.10/i586

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libusrp/xUbuntu_18.10/i586

Package network:osmocom:nightly/libusrp failed to build in xUbuntu_18.10/i586

Check out the package for editing:
  osc checkout network:osmocom:nightly libusrp

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was lamb15:5)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/libasn1c in openSUSE_Leap_15.1_ARM/aarch64

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libasn1c/openSUSE_Leap_15.1_ARM/aarch64

Package network:osmocom:nightly/libasn1c failed to build in 
openSUSE_Leap_15.1_ARM/aarch64

Check out the package for editing:
  osc checkout network:osmocom:nightly libasn1c

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was obs-arm-9:30)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/libusrp in Debian_10/aarch64

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libusrp/Debian_10/aarch64

Package network:osmocom:nightly/libusrp failed to build in Debian_10/aarch64

Check out the package for editing:
  osc checkout network:osmocom:nightly libusrp

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was obs-arm-9:36)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/libgtpnl in xUbuntu_18.04/i586

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libgtpnl/xUbuntu_18.04/i586

Package network:osmocom:nightly/libgtpnl failed to build in xUbuntu_18.04/i586

Check out the package for editing:
  osc checkout network:osmocom:nightly libgtpnl

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was old-atreju5:9)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/libusrp in xUbuntu_18.10/x86_64

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libusrp/xUbuntu_18.10/x86_64

Package network:osmocom:nightly/libusrp failed to build in xUbuntu_18.10/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly libusrp

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was cloud113:3)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/libosmocore in xUbuntu_19.10/x86_64

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libosmocore/xUbuntu_19.10/x86_64

Package network:osmocom:nightly/libosmocore failed to build in 
xUbuntu_19.10/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly libosmocore

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was cloud113:4)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Build failure of network:osmocom:nightly/libasn1c in Debian_9.0/aarch64

2020-10-13 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libasn1c/Debian_9.0/aarch64

Package network:osmocom:nightly/libasn1c failed to build in Debian_9.0/aarch64

Check out the package for editing:
  osc checkout network:osmocom:nightly libasn1c

Last lines of build log:
getoldpackages: connect to back-other-opensuse:5253: Connection timed out
(worker was obs-arm-2:5)

-- 
Configure notifications at https://build.opensuse.org/my/subscriptions
openSUSE Build Service (https://build.opensuse.org/)


Change in osmo-ccid-firmware[master]: HACK: cuart: Artificially extend WTIME by 1s

2020-10-13 Thread laforge
laforge has abandoned this change. ( 
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/19952 )

Change subject: HACK: cuart: Artificially extend WTIME by 1s
..


Abandoned

we now have fixed the acual timeouts, no need for this hack anymore
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/19952
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: I85a7a0ae6195856fad17f249f0903f144a9aec85
Gerrit-Change-Number: 19952
Gerrit-PatchSet: 3
Gerrit-Owner: laforge 
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: abandon


Change in osmo-trx[master]: Transceiver: Pass config struct instead of large list of params

2020-10-13 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/20638 )

Change subject: Transceiver: Pass config struct instead of large list of params
..


Patch Set 1:

(1 comment)

https://gerrit.osmocom.org/c/osmo-trx/+/20638/1/CommonLibs/trx_vty.h
File CommonLibs/trx_vty.h:

https://gerrit.osmocom.org/c/osmo-trx/+/20638/1/CommonLibs/trx_vty.h@47
PS1, Line 47: struct trx_cfg
In general, this stricture (together with the other definitions in this file) 
is not something VTY specific, so I believe 'CommonLibs/config_defs.h' is a 
better place. If we move them there, there will be no need to #include 
 in Transceiver.{c,h}, which has nothing to do with the VTY.



--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/20638
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Ifb43cb11f3e7a69b0a88f632f0a0c90ada7f939e
Gerrit-Change-Number: 20638
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-CC: fixeria 
Gerrit-Comment-Date: Tue, 13 Oct 2020 19:50:14 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: pcu/PCU_Tests_SNS.cfg: fix configuration to match new NS provider

2020-10-13 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20635 )

Change subject: pcu/PCU_Tests_SNS.cfg: fix configuration to match new NS 
provider
..

pcu/PCU_Tests_SNS.cfg: fix configuration to match new NS provider

The NS uses now a provider to support FR and IP at the same time.

Change-Id: I6b0fa97a1098070a4f63228568092084503d6164
---
M pcu/PCU_Tests_SNS.cfg
1 file changed, 9 insertions(+), 4 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/pcu/PCU_Tests_SNS.cfg b/pcu/PCU_Tests_SNS.cfg
index 022f91c..50f6129 100644
--- a/pcu/PCU_Tests_SNS.cfg
+++ b/pcu/PCU_Tests_SNS.cfg
@@ -8,10 +8,15 @@

 [MODULE_PARAMETERS]
 SGSN_Components.mp_nsconfig := {
-   local_ip := "127.0.0.1",
-   local_udp_port := 23000,
-   remote_ip := "127.0.0.1",
-   remote_udp_port := 22000,
+   provider := {
+   ip := {
+   address_family := AF_INET,
+   local_ip := "127.0.0.1",
+   local_udp_port := 23000,
+   remote_ip := "127.0.0.1",
+   remote_udp_port := 22000
+   }
+   },
nsvci := 1234,
nsei := 1234
 }

--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20635
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I6b0fa97a1098070a4f63228568092084503d6164
Gerrit-Change-Number: 20635
Gerrit-PatchSet: 2
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged


Change in osmo-ttcn3-hacks[master]: pcu/PCU_Tests_SNS.cfg: fix configuration to match new NS provider

2020-10-13 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20635 )

Change subject: pcu/PCU_Tests_SNS.cfg: fix configuration to match new NS 
provider
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20635
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I6b0fa97a1098070a4f63228568092084503d6164
Gerrit-Change-Number: 20635
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Tue, 13 Oct 2020 19:49:09 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: PCU: add IPv6 support for SNS

2020-10-13 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20003 )

Change subject: PCU: add IPv6 support for SNS
..


Patch Set 3: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20003
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I43d64caca600fff78f3fbbb3e8179f447f235d46
Gerrit-Change-Number: 20003
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Comment-Date: Tue, 13 Oct 2020 19:48:56 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: Add initial TTCN3 test for GbProxy

2020-10-13 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20637 )

Change subject: Add initial TTCN3 test for GbProxy
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20637
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ib3dc05fe5598b53e963ca863968e387cc36b9de3
Gerrit-Change-Number: 20637
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Tue, 13 Oct 2020 19:47:37 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: Send NS_ALIVE only after NS-RESET

2020-10-13 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20636 )

Change subject: Send NS_ALIVE only after NS-RESET
..


Patch Set 1:

that is true only for FR and non-spec NS-over-IP without SNS and UNBLOCK/RESET 
procedure.

I think this change would break the behavior for proper 3GPP NSIP with IP-SNS, 
right?


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20636
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ie01fee70297255b3d9c091bc2cec75b0f915c588
Gerrit-Change-Number: 20636
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Comment-Date: Tue, 13 Oct 2020 19:47:13 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in osmo-trx[master]: Transceiver: Pass config struct instead of large list of params

2020-10-13 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/20638 )

Change subject: Transceiver: Pass config struct instead of large list of params
..


Patch Set 1:

(3 comments)

https://gerrit.osmocom.org/c/osmo-trx/+/20638/1/Transceiver52M/Transceiver.h
File Transceiver52M/Transceiver.h:

https://gerrit.osmocom.org/c/osmo-trx/+/20638/1/Transceiver52M/Transceiver.h@110
PS1, Line 110: struct
Can we use 'const' here? If yes, let's use it.


https://gerrit.osmocom.org/c/osmo-trx/+/20638/1/Transceiver52M/Transceiver.h@111
PS1, Line 111:   //const char *TRXAddress,
Any reason why you're keeping the old arguments commented out?


https://gerrit.osmocom.org/c/osmo-trx/+/20638/1/Transceiver52M/Transceiver.h@158
PS1, Line 158: struct
Same here, 'const' if possible.



--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/20638
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Ifb43cb11f3e7a69b0a88f632f0a0c90ada7f939e
Gerrit-Change-Number: 20638
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-CC: fixeria 
Gerrit-Comment-Date: Tue, 13 Oct 2020 19:45:44 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in osmo-trx[master]: Calculate RSSI offset based on RxGain configuration

2020-10-13 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/20639 )

Change subject: Calculate RSSI offset based on RxGain configuration
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/20639
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I8ef78fd20c22c60d61bfb18d80a4a36df4fd6c20
Gerrit-Change-Number: 20639
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Tue, 13 Oct 2020 19:45:35 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-trx[master]: Transceiver: Pass config struct instead of large list of params

2020-10-13 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/20638 )

Change subject: Transceiver: Pass config struct instead of large list of params
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/20638
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Ifb43cb11f3e7a69b0a88f632f0a0c90ada7f939e
Gerrit-Change-Number: 20638
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Tue, 13 Oct 2020 19:44:24 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: {enb, epc, ms}_srs: remove uneeded class variable

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20640 )

Change subject: {enb,epc,ms}_srs: remove uneeded class variable
..

{enb,epc,ms}_srs: remove uneeded class variable

self.kpis is define in base class

Change-Id: I49c9c9cdf7bc31270d3cfe182a79b197a742dd2c
---
M src/osmo_gsm_tester/obj/enb_srs.py
M src/osmo_gsm_tester/obj/epc_srs.py
M src/osmo_gsm_tester/obj/ms_srs.py
3 files changed, 0 insertions(+), 3 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/enb_srs.py 
b/src/osmo_gsm_tester/obj/enb_srs.py
index 9d7eca3..77f196f 100644
--- a/src/osmo_gsm_tester/obj/enb_srs.py
+++ b/src/osmo_gsm_tester/obj/enb_srs.py
@@ -74,7 +74,6 @@
 self.metrics_file = None
 self.stop_sleep_time = 6 # We require at most 5s to stop
 self.testenv = testenv
-self.kpis = None
 self._additional_args = []
 if not rf_type_valid(conf.get('rf_dev_type', None)):
 raise log.Error('Invalid rf_dev_type=%s' % conf.get('rf_dev_type', 
None))
diff --git a/src/osmo_gsm_tester/obj/epc_srs.py 
b/src/osmo_gsm_tester/obj/epc_srs.py
index dae8a56..80f66e8 100644
--- a/src/osmo_gsm_tester/obj/epc_srs.py
+++ b/src/osmo_gsm_tester/obj/epc_srs.py
@@ -57,7 +57,6 @@
 self.remote_log_file = None
 self.remote_pcap_file = None
 self.enable_pcap = False
-self.kpis = None
 self.subscriber_list = []

 def cleanup(self):
diff --git a/src/osmo_gsm_tester/obj/ms_srs.py 
b/src/osmo_gsm_tester/obj/ms_srs.py
index be3e799..084edcb 100644
--- a/src/osmo_gsm_tester/obj/ms_srs.py
+++ b/src/osmo_gsm_tester/obj/ms_srs.py
@@ -96,7 +96,6 @@
 self.remote_metrics_file = None
 self.enable_pcap = False
 self.num_carriers = 1
-self.kpis = None
 self.testenv = testenv
 self._additional_args = []
 if not rf_type_valid(conf.get('rf_dev_type', None)):

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20640
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I49c9c9cdf7bc31270d3cfe182a79b197a742dd2c
Gerrit-Change-Number: 20640
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: rfemu_srsenb_stdin: use negative dB value to set attenuation

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20641 )

Change subject: rfemu_srsenb_stdin: use negative dB value to set attenuation
..

rfemu_srsenb_stdin: use negative dB value to set attenuation

the logic for the RF emulator on the command line is, like the
Amarisoft equivalent, to set a negative gain value to express
attenuation. Therefore, the sign needs to be inverted.

Change-Id: I7b6ac611a9f711bbdc1570602bfd797a5346ab86
---
M src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py 
b/src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py
index ca23020..e4aec19 100644
--- a/src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py
+++ b/src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py
@@ -42,7 +42,7 @@
 # PUBLIC (test API included)
 #
 def set_attenuation(self, db):
-msg_str = 'cell_gain %d %f' % (self.cell_id, db)
+msg_str = 'cell_gain %d %f' % (self.cell_id, -db)
 self.dbg('sending stdin msg: "%s"' % msg_str)
 self.enb.process.stdin_write(msg_str + '\n')


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20641
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I7b6ac611a9f711bbdc1570602bfd797a5346ab86
Gerrit-Change-Number: 20641
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: rfemu_srsenb_stdin: use negative dB value to set attenuation

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20641 )

Change subject: rfemu_srsenb_stdin: use negative dB value to set attenuation
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20641
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I7b6ac611a9f711bbdc1570602bfd797a5346ab86
Gerrit-Change-Number: 20641
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 18:54:56 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: {enb, epc, ms}_srs: remove uneeded class variable

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20640 )

Change subject: {enb,epc,ms}_srs: remove uneeded class variable
..


Patch Set 1: Verified+1 Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20640
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I49c9c9cdf7bc31270d3cfe182a79b197a742dd2c
Gerrit-Change-Number: 20640
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 18:54:33 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: rfemu_srsenb_stdin: use negative dB value to set attenuation

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20641 )

Change subject: rfemu_srsenb_stdin: use negative dB value to set attenuation
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20641
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I7b6ac611a9f711bbdc1570602bfd797a5346ab86
Gerrit-Change-Number: 20641
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 18:54:59 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: srsenb: Fix ECI in handover cell list

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20518 )

Change subject: srsenb: Fix ECI in handover cell list
..


Patch Set 2: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20518
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I1780bda130cde0faab15d4032117e0aa2b9c5dbe
Gerrit-Change-Number: 20518
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Reviewer: srs_andre 
Gerrit-Comment-Date: Tue, 13 Oct 2020 18:54:15 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: srsenb: Fix ECI in handover cell list

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20518 )

Change subject: srsenb: Fix ECI in handover cell list
..

srsenb: Fix ECI in handover cell list

So far we only support setting cell IDs from same ENB. Support for
handover to other ENB will come later, probably by adding some API to be
called during the test to add own's CellID's to other ENB's config.t

Change-Id: I1780bda130cde0faab15d4032117e0aa2b9c5dbe
---
M src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  pespin: Verified
  srs_andre: Looks good to me, approved



diff --git a/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
index b1e9cb2..8c06c0d 100644
--- a/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
@@ -88,7 +88,7 @@
 ${',' if loop.my_num_items != 0 else ''}
 <% loop.my_num_items += 1 %>
 {
-  eci = ${ncell.cell_id};
+  eci = ${hex((int(enb.id)<<8) + int(ncell.cell_id))};
   dl_earfcn = ${ncell.dl_earfcn};
   pci = ${ncell.pci};
 }

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20518
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I1780bda130cde0faab15d4032117e0aa2b9c5dbe
Gerrit-Change-Number: 20518
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Reviewer: srs_andre 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: srsenb: Activate handover in config file

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20519 )

Change subject: srsenb: Activate handover in config file
..


Patch Set 2: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20519
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ifa8462f4d1e183e73b059a7af37bcdf7b3e247f6
Gerrit-Change-Number: 20519
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Reviewer: srs_andre 
Gerrit-Comment-Date: Tue, 13 Oct 2020 18:54:25 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: Introduce RFemu stdin implementation for srsENB

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20456 )

Change subject: Introduce RFemu stdin implementation for srsENB
..

Introduce RFemu stdin implementation for srsENB

Change-Id: I5487dec3691cbf424306369f94f337ed34a10c70
---
M src/osmo_gsm_tester/obj/enb_srs.py
M src/osmo_gsm_tester/obj/rfemu.py
A src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py
M sysmocom/defaults.conf
4 files changed, 64 insertions(+), 0 deletions(-)

Approvals:
  pespin: Verified
  srs_andre: Looks good to me, approved



diff --git a/src/osmo_gsm_tester/obj/enb_srs.py 
b/src/osmo_gsm_tester/obj/enb_srs.py
index 63e5d7c..9d7eca3 100644
--- a/src/osmo_gsm_tester/obj/enb_srs.py
+++ b/src/osmo_gsm_tester/obj/enb_srs.py
@@ -259,6 +259,10 @@
 rfemu_cfg = cell_list[cell].get('dl_rfemu', None)
 if rfemu_cfg is None:
 raise log.Error('rfemu attribute not found in cell_list item!')
+if rfemu_cfg['type'] == 'srsenb_stdin':
+# These fields are required so the rfemu class can interact with 
us:
+ config.overlay(rfemu_cfg, dict(enb=self,
+
cell_id=cell_list[cell]['cell_id']))

 rfemu_obj = rfemu.get_instance_by_type(rfemu_cfg['type'], rfemu_cfg)
 return rfemu_obj
diff --git a/src/osmo_gsm_tester/obj/rfemu.py b/src/osmo_gsm_tester/obj/rfemu.py
index ac34ee1..ec8ed68 100644
--- a/src/osmo_gsm_tester/obj/rfemu.py
+++ b/src/osmo_gsm_tester/obj/rfemu.py
@@ -52,6 +52,9 @@
 elif rfemu_type == 'minicircuits':
 from .rfemu_minicircuits import RFemulationMinicircuitsHTTP
 obj = RFemulationMinicircuitsHTTP
+elif rfemu_type == 'srsenb_stdin':
+from .rfemu_srsenb_stdin import RFemulationSrsStdin
+obj = RFemulationSrsStdin
 else:
 raise log.Error('RFemulation type not supported:', rfemu_type)

diff --git a/src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py 
b/src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py
new file mode 100644
index 000..ca23020
--- /dev/null
+++ b/src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py
@@ -0,0 +1,52 @@
+# osmo_gsm_tester: class defining a RF emulation object implemented using SRS 
ENB stdin interface
+#
+# Copyright (C) 2020 by sysmocom - s.f.m.c. GmbH
+#
+# Author: Pau Espin Pedrol 
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as
+# published by the Free Software Foundation, either version 3 of the
+# License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program.  If not, see .
+
+import json
+
+from ..core import log
+from .rfemu import RFemulation
+
+class RFemulationSrsStdin(RFemulation):
+##
+# PROTECTED
+##
+def __init__(self, conf):
+super().__init__(conf, 'srsenb_stdin')
+self.cell_id = int(conf.get('cell_id'))
+if self.cell_id is None:
+raise log.Error('No "cell_id" attribute provided in rfemu conf!')
+self.enb = conf.get('enb')
+if self.enb is None:
+raise log.Error('No "srsenb" attribute provided in rfemu conf!')
+
+def __del__(self):
+self.enb = None
+
+#
+# PUBLIC (test API included)
+#
+def set_attenuation(self, db):
+msg_str = 'cell_gain %d %f' % (self.cell_id, db)
+self.dbg('sending stdin msg: "%s"' % msg_str)
+self.enb.process.stdin_write(msg_str + '\n')
+
+def get_max_attenuation(self):
+return 200 # maximum cell_gain value in srs. Is this correct value?
+
+# vim: expandtab tabstop=4 shiftwidth=4
diff --git a/sysmocom/defaults.conf b/sysmocom/defaults.conf
index ed0b575..419c3a4 100644
--- a/sysmocom/defaults.conf
+++ b/sysmocom/defaults.conf
@@ -148,6 +148,11 @@
   tx_gain: 80
   rx_gain: 40
   log_all_level: warning
+  cell_list:
+   - dl_rfemu:
+   type: srsenb_stdin
+   - dl_rfemu:
+   type: srsenb_stdin

 amarisoftenb:
   num_prb: 100

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20456
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5487dec3691cbf424306369f94f337ed34a10c70
Gerrit-Change-Number: 20456
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Reviewer: srs_andre 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: srsenb: Activate handover in config file

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20519 )

Change subject: srsenb: Activate handover in config file
..

srsenb: Activate handover in config file

This is needed to test handover features.

Change-Id: Ifa8462f4d1e183e73b059a7af37bcdf7b3e247f6
---
M src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  pespin: Verified
  srs_andre: Looks good to me, approved



diff --git a/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
index 8c06c0d..3bb2c8c 100644
--- a/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
@@ -64,7 +64,7 @@
 root_seq_idx = ${loop.index + 204};
 dl_earfcn = ${cell.dl_earfcn};
 //ul_earfcn = 20850;
-ho_active = false;
+ho_active = true;

 // CA cells
 scell_list = (

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20519
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ifa8462f4d1e183e73b059a7af37bcdf7b3e247f6
Gerrit-Change-Number: 20519
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Reviewer: srs_andre 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: Introduce RFemu stdin implementation for srsENB

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20456 )

Change subject: Introduce RFemu stdin implementation for srsENB
..


Patch Set 3: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20456
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5487dec3691cbf424306369f94f337ed34a10c70
Gerrit-Change-Number: 20456
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Reviewer: srs_andre 
Gerrit-Comment-Date: Tue, 13 Oct 2020 18:54:04 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: {enb, epc, ms}_srs: remove uneeded class variable

2020-10-13 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20640 )


Change subject: {enb,epc,ms}_srs: remove uneeded class variable
..

{enb,epc,ms}_srs: remove uneeded class variable

self.kpis is define in base class

Change-Id: I49c9c9cdf7bc31270d3cfe182a79b197a742dd2c
---
M src/osmo_gsm_tester/obj/enb_srs.py
M src/osmo_gsm_tester/obj/epc_srs.py
M src/osmo_gsm_tester/obj/ms_srs.py
3 files changed, 0 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/40/20640/1

diff --git a/src/osmo_gsm_tester/obj/enb_srs.py 
b/src/osmo_gsm_tester/obj/enb_srs.py
index 9d7eca3..77f196f 100644
--- a/src/osmo_gsm_tester/obj/enb_srs.py
+++ b/src/osmo_gsm_tester/obj/enb_srs.py
@@ -74,7 +74,6 @@
 self.metrics_file = None
 self.stop_sleep_time = 6 # We require at most 5s to stop
 self.testenv = testenv
-self.kpis = None
 self._additional_args = []
 if not rf_type_valid(conf.get('rf_dev_type', None)):
 raise log.Error('Invalid rf_dev_type=%s' % conf.get('rf_dev_type', 
None))
diff --git a/src/osmo_gsm_tester/obj/epc_srs.py 
b/src/osmo_gsm_tester/obj/epc_srs.py
index dae8a56..80f66e8 100644
--- a/src/osmo_gsm_tester/obj/epc_srs.py
+++ b/src/osmo_gsm_tester/obj/epc_srs.py
@@ -57,7 +57,6 @@
 self.remote_log_file = None
 self.remote_pcap_file = None
 self.enable_pcap = False
-self.kpis = None
 self.subscriber_list = []

 def cleanup(self):
diff --git a/src/osmo_gsm_tester/obj/ms_srs.py 
b/src/osmo_gsm_tester/obj/ms_srs.py
index be3e799..084edcb 100644
--- a/src/osmo_gsm_tester/obj/ms_srs.py
+++ b/src/osmo_gsm_tester/obj/ms_srs.py
@@ -96,7 +96,6 @@
 self.remote_metrics_file = None
 self.enable_pcap = False
 self.num_carriers = 1
-self.kpis = None
 self.testenv = testenv
 self._additional_args = []
 if not rf_type_valid(conf.get('rf_dev_type', None)):

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20640
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I49c9c9cdf7bc31270d3cfe182a79b197a742dd2c
Gerrit-Change-Number: 20640
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: rfemu_srsenb_stdin: use negative dB value to set attenuation

2020-10-13 Thread srs_andre
srs_andre has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20641 )


Change subject: rfemu_srsenb_stdin: use negative dB value to set attenuation
..

rfemu_srsenb_stdin: use negative dB value to set attenuation

the logic for the RF emulator on the command line is, like the
Amarisoft equivalent, to set a negative gain value to express
attenuation. Therefore, the sign needs to be inverted.

Change-Id: I7b6ac611a9f711bbdc1570602bfd797a5346ab86
---
M src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/41/20641/1

diff --git a/src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py 
b/src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py
index ca23020..e4aec19 100644
--- a/src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py
+++ b/src/osmo_gsm_tester/obj/rfemu_srsenb_stdin.py
@@ -42,7 +42,7 @@
 # PUBLIC (test API included)
 #
 def set_attenuation(self, db):
-msg_str = 'cell_gain %d %f' % (self.cell_id, db)
+msg_str = 'cell_gain %d %f' % (self.cell_id, -db)
 self.dbg('sending stdin msg: "%s"' % msg_str)
 self.enb.process.stdin_write(msg_str + '\n')


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20641
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I7b6ac611a9f711bbdc1570602bfd797a5346ab86
Gerrit-Change-Number: 20641
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-MessageType: newchange


Change in osmo-trx[master]: Calculate RSSI offset based on RxGain configuration

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/20639 )

Change subject: Calculate RSSI offset based on RxGain configuration
..


Patch Set 1:

@Hoernchen, what to do with osmo-trx-ipc? Retrieve this offset from the IPC 
Driver? Expect the IPC Driver to always provide bursts with rssi_offset=0?


--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/20639
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I8ef78fd20c22c60d61bfb18d80a4a36df4fd6c20
Gerrit-Change-Number: 20639
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-Reviewer: Hoernchen 
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Tue, 13 Oct 2020 18:30:01 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in osmo-trx[master]: Transceiver: Pass config struct instead of large list of params

2020-10-13 Thread pespin
pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/20638 )


Change subject: Transceiver: Pass config struct instead of large list of params
..

Transceiver: Pass config struct instead of large list of params

Change-Id: Ifb43cb11f3e7a69b0a88f632f0a0c90ada7f939e
---
M CommonLibs/trx_vty.h
M Transceiver52M/Transceiver.cpp
M Transceiver52M/Transceiver.h
M Transceiver52M/osmo-trx.cpp
4 files changed, 78 insertions(+), 102 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/38/20638/1

diff --git a/CommonLibs/trx_vty.h b/CommonLibs/trx_vty.h
index c0d54cf..fcbc8eb 100644
--- a/CommonLibs/trx_vty.h
+++ b/CommonLibs/trx_vty.h
@@ -44,29 +44,31 @@
char *tx_path;
 };

+struct trx_cfg {
+   char *bind_addr;
+   char *remote_addr;
+   char *dev_args;
+   unsigned int base_port;
+   unsigned int tx_sps;
+   unsigned int rx_sps;
+   unsigned int rtsc;
+   unsigned int rach_delay;
+   enum ReferenceType clock_ref;
+   enum FillerType filler;
+   bool multi_arfcn;
+   double offset;
+   double rssi_offset;
+   bool swap_channels;
+   bool ext_rach;
+   bool egprs;
+   unsigned int sched_rr;
+   unsigned int stack_size;
+   unsigned int num_chans;
+   struct trx_chan chans[TRX_CHAN_MAX];
+};
+
 struct trx_ctx {
-   struct {
-   char *bind_addr;
-   char *remote_addr;
-   char *dev_args;
-   unsigned int base_port;
-   unsigned int tx_sps;
-   unsigned int rx_sps;
-   unsigned int rtsc;
-   unsigned int rach_delay;
-   enum ReferenceType clock_ref;
-   enum FillerType filler;
-   bool multi_arfcn;
-   double offset;
-   double rssi_offset;
-   bool swap_channels;
-   bool ext_rach;
-   bool egprs;
-   unsigned int sched_rr;
-   unsigned int stack_size;
-   unsigned int num_chans;
-   struct trx_chan chans[TRX_CHAN_MAX];
-   } cfg;
+   struct trx_cfg cfg;
 };

 int trx_vty_init(struct trx_ctx* trx);
diff --git a/Transceiver52M/Transceiver.cpp b/Transceiver52M/Transceiver.cpp
index 676ffde..ff0b95b 100644
--- a/Transceiver52M/Transceiver.cpp
+++ b/Transceiver52M/Transceiver.cpp
@@ -132,20 +132,14 @@
   return false;
 }

-Transceiver::Transceiver(int wBasePort,
- const char *TRXAddress,
- const char *GSMcoreAddress,
- size_t tx_sps, size_t rx_sps, size_t chans,
+Transceiver::Transceiver(struct trx_cfg *cfg,
  GSM::Time wTransmitLatency,
- RadioInterface *wRadioInterface,
- double wRssiOffset, int wStackSize)
-  : mBasePort(wBasePort), mLocalAddr(TRXAddress), mRemoteAddr(GSMcoreAddress),
-mClockSocket(-1), mTransmitLatency(wTransmitLatency), 
mRadioInterface(wRadioInterface),
-rssiOffset(wRssiOffset), stackSize(wStackSize),
-mSPSTx(tx_sps), mSPSRx(rx_sps), mChans(chans), mExtRACH(false), 
mEdge(false),
-mOn(false), mForceClockInterface(false),
-mTxFreq(0.0), mRxFreq(0.0), mTSC(0), mMaxExpectedDelayAB(0), 
mMaxExpectedDelayNB(0),
-mWriteBurstToDiskMask(0)
+ RadioInterface *wRadioInterface)
+  : cfg(cfg), mClockSocket(-1),
+mTransmitLatency(wTransmitLatency), mRadioInterface(wRadioInterface),
+mChans(cfg->num_chans), mOn(false), mForceClockInterface(false),
+mTxFreq(0.0), mRxFreq(0.0), mTSC(0), mMaxExpectedDelayAB(0),
+mMaxExpectedDelayNB(0), mWriteBurstToDiskMask(0)
 {
   txFullScale = mRadioInterface->fullScaleInputValue();
   rxFullScale = mRadioInterface->fullScaleOutputValue();
@@ -199,11 +193,9 @@
  * are still expected to report clock indications through control channel
  * activity.
  */
-bool Transceiver::init(FillerType filler, size_t rtsc, unsigned rach_delay,
-   bool edge, bool ext_rach)
+bool Transceiver::init()
 {
   int d_srcport, d_dstport, c_srcport, c_dstport;
-
   if (!mChans) {
 LOG(FATAL) << "No channels assigned";
 return false;
@@ -214,9 +206,6 @@
 return false;
   }

-  mExtRACH = ext_rach;
-  mEdge = edge;
-
   mDataSockets.resize(mChans, -1);
   mCtrlSockets.resize(mChans);
   mTxPriorityQueueServiceLoopThreads.resize(mChans);
@@ -228,27 +217,28 @@
   mVersionTRXD.resize(mChans);

   /* Filler table retransmissions - support only on channel 0 */
-  if (filler == FILLER_DUMMY)
+  if (cfg->filler == FILLER_DUMMY)
 mStates[0].mRetrans = true;

   /* Setup sockets */
   mClockSocket = osmo_sock_init2(AF_UNSPEC, SOCK_DGRAM, IPPROTO_UDP,
-   mLocalAddr.c_str(), mBasePort,
-   mRemoteAddr.c_str(), mBasePort + 100,
+ 

Change in osmo-trx[master]: Calculate RSSI offset based on RxGain configuration

2020-10-13 Thread pespin
pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/20639 )


Change subject: Calculate RSSI offset based on RxGain configuration
..

Calculate RSSI offset based on RxGain configuration

Prior to this patch, osmo-trx relied totally on proper VTY configuration
being set in "rssi-offset" together with the RxGain set through TRXC in
order to provide correct Uplink RSSI measurements to bts-trx.

With this patch, RSSI is now by default calculated (in LMS and UHD
backends) based on the currently set RxGain, by providing empirically
discovered values. Still, for backward compatibility, the old
"rssi-offset" command will overwrite completely the per-default
calculated rssi offset.
A new optional parameter "relative" is added at the end of the
"rssi-offset" VTY command to flag the value as relative to the newly
per-default calculated value. This way specific setups (like adding a
LNA / RF fronted) can still be expressed while still keeping the
automatic per-default offset.

Related: OS#4468
Change-Id: I8ef78fd20c22c60d61bfb18d80a4a36df4fd6c20
---
M CommonLibs/trx_vty.c
M CommonLibs/trx_vty.h
M Transceiver52M/Transceiver.cpp
M Transceiver52M/Transceiver.h
M Transceiver52M/device/common/radioDevice.h
M Transceiver52M/device/ipc/IPCDevice.h
M Transceiver52M/device/lms/LMSDevice.cpp
M Transceiver52M/device/lms/LMSDevice.h
M Transceiver52M/device/uhd/UHDDevice.cpp
M Transceiver52M/device/uhd/UHDDevice.h
M Transceiver52M/device/usrp1/USRPDevice.h
M Transceiver52M/osmo-trx.cpp
M Transceiver52M/radioInterface.cpp
M Transceiver52M/radioInterface.h
M Transceiver52M/radioInterfaceMulti.cpp
M doc/examples/osmo-trx-uhd/osmo-trx-usrp_b200.cfg
16 files changed, 187 insertions(+), 70 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-trx refs/changes/39/20639/1

diff --git a/CommonLibs/trx_vty.c b/CommonLibs/trx_vty.c
index cf36547..fd6faed 100644
--- a/CommonLibs/trx_vty.c
+++ b/CommonLibs/trx_vty.c
@@ -231,13 +231,15 @@
 }

 DEFUN(cfg_rssi_offset, cfg_rssi_offset_cmd,
-   "rssi-offset FLOAT",
+   "rssi-offset FLOAT [relative]",
"Set the RSSI to dBm offset in dB (default=0)\n"
-   "RSSI to dBm offset in dB\n")
+   "RSSI to dBm offset in dB\n"
+   "Add to the default rssi-offset value instead of completely replacing 
it\n")
 {
struct trx_ctx *trx = trx_from_vty(vty);

trx->cfg.rssi_offset = atof(argv[0]);
+   trx->cfg.force_rssi_offset = (argc == 1);

return CMD_SUCCESS;
 }
@@ -580,8 +582,9 @@
vty_out(vty, " multi-arfcn %s%s", trx->cfg.multi_arfcn ? "enable" : 
"disable", VTY_NEWLINE);
if (trx->cfg.offset != 0)
vty_out(vty, " offset %f%s", trx->cfg.offset, VTY_NEWLINE);
-   if (trx->cfg.rssi_offset != 0)
-   vty_out(vty, " rssi-offset %f%s", trx->cfg.rssi_offset, 
VTY_NEWLINE);
+   if (!(trx->cfg.rssi_offset == 0 && !trx->cfg.force_rssi_offset))
+   vty_out(vty, " rssi-offset %f%s%s", trx->cfg.rssi_offset,
+   trx->cfg.force_rssi_offset ? " relative": "", 
VTY_NEWLINE);
vty_out(vty, " swap-channels %s%s", trx->cfg.swap_channels ? "enable" : 
"disable", VTY_NEWLINE);
vty_out(vty, " egprs %s%s", trx->cfg.egprs ? "enable" : "disable", 
VTY_NEWLINE);
vty_out(vty, " ext-rach %s%s", trx->cfg.ext_rach ? "enable" : 
"disable", VTY_NEWLINE);
@@ -716,6 +719,7 @@
trx->cfg.tx_sps = DEFAULT_TX_SPS;
trx->cfg.rx_sps = DEFAULT_RX_SPS;
trx->cfg.filler = FILLER_ZERO;
+   trx->cfg.rssi_offset = 0.0f;

return trx;
 }
diff --git a/CommonLibs/trx_vty.h b/CommonLibs/trx_vty.h
index fcbc8eb..b7c753e 100644
--- a/CommonLibs/trx_vty.h
+++ b/CommonLibs/trx_vty.h
@@ -58,6 +58,7 @@
bool multi_arfcn;
double offset;
double rssi_offset;
+   bool force_rssi_offset; /* Force value set in VTY? */
bool swap_channels;
bool ext_rach;
bool egprs;
diff --git a/Transceiver52M/Transceiver.cpp b/Transceiver52M/Transceiver.cpp
index ff0b95b..c5fe8f6 100644
--- a/Transceiver52M/Transceiver.cpp
+++ b/Transceiver52M/Transceiver.cpp
@@ -612,6 +612,13 @@
   outfile.close();
 }

+double Transceiver::rssiOffset(size_t chan)
+{
+  if (cfg->force_rssi_offset)
+return cfg->rssi_offset;
+  return mRadioInterface->rssiOffset(chan) + cfg->rssi_offset;
+}
+
 /*
  * Pull bursts from the FIFO and handle according to the slot
  * and burst correlation type. Equalzation is currently disabled.
@@ -631,6 +638,7 @@
   SoftVector *rxBurst;
   TransceiverState *state = [chan];
   bool ctr_changed = false;
+  double rssi_offset;

   /* Blocking FIFO read */
   radioVector *radio_burst = mReceiveFIFO[chan]->read();
@@ -700,8 +708,9 @@
 state->mNoiseLev = state->mNoises.avg();
   }

-  bi->rssi = 20.0 * log10(rxFullScale / avg) + cfg->rssi_offset;
-  bi->noise = 20.0 * log10(rxFullScale / state->mNoiseLev) + cfg->rssi_offset;

Change in osmo-ttcn3-hacks[master]: Add initial TTCN3 test for GbProxy

2020-10-13 Thread daniel
daniel has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20637 )


Change subject: Add initial TTCN3 test for GbProxy
..

Add initial TTCN3 test for GbProxy

The first testcase, TC_BVC_bringup just waits for NS/BSSGP Emulation to
do their thing. If nothing fails there then we pass.

Related: SYS#5002
Change-Id: Ib3dc05fe5598b53e963ca863968e387cc36b9de3
---
M Makefile
A gbproxy/GBProxy_Tests.cfg
A gbproxy/GBProxy_Tests.default
A gbproxy/GBProxy_Tests.ttcn
A gbproxy/README.md
A gbproxy/gen_links.sh
A gbproxy/osmo-gbproxy.cfg
A gbproxy/regen_makefile.sh
8 files changed, 688 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks 
refs/changes/37/20637/1

diff --git a/Makefile b/Makefile
index 03e7a70..c4e8aff 100644
--- a/Makefile
+++ b/Makefile
@@ -13,8 +13,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.

-SUBDIRS=bsc bsc-nat bts ccid ggsn_tests hlr mgw mme msc pcu pgw remsim sccp 
selftest sgsn \
-   simtrace sip stp sysinfo
+SUBDIRS=bsc bsc-nat bts ccid gbproxy ggsn_tests hlr mgw mme msc pcu pgw remsim 
\
+   sccp selftest sgsn simtrace sip stp sysinfo

 NPROC=$(shell nproc 2>/dev/null)
 ifeq ($(NPROC),)
diff --git a/gbproxy/GBProxy_Tests.cfg b/gbproxy/GBProxy_Tests.cfg
new file mode 100644
index 000..3209be6
--- /dev/null
+++ b/gbproxy/GBProxy_Tests.cfg
@@ -0,0 +1,33 @@
+[ORDERED_INCLUDE]
+# Common configuration, shared between test suites
+"../Common.cfg"
+# testsuite specific configuration, not expected to change
+"./GBProxy_Tests.default"
+
+# Local configuration below
+
+[LOGGING]
+
+[TESTPORT_PARAMETERS]
+*.GBPVTY.CTRL_HOSTNAME := "127.0.0.1"
+
+[MODULE_PARAMETERS]
+GBProxy_Tests.mp_nsconfig_pcu := {
+   {provider := { ip := {
+   local_ip := "127.0.0.1",
+   remote_ip := "127.0.0.1"
+   }}},
+   {provider := { ip := {
+   local_ip := "127.0.0.1",
+   remote_ip := "127.0.0.1"
+   }}},
+   {provider := { ip := {
+   local_ip := "127.0.0.1",
+   remote_ip := "127.0.0.1"
+   }}}
+}
+
+[MAIN_CONTROLLER]
+
+[EXECUTE]
+GBProxy_Tests.control
diff --git a/gbproxy/GBProxy_Tests.default b/gbproxy/GBProxy_Tests.default
new file mode 100644
index 000..799ac32
--- /dev/null
+++ b/gbproxy/GBProxy_Tests.default
@@ -0,0 +1,27 @@
+[LOGGING]
+FileMask := LOG_ALL | TTCN_MATCHING;
+
+"IPA-CTRL-IPA".FileMask := ERROR | WARNING | PARALLEL | VERDICTOP;
+"GBProxy-GSUP-IPA".FileMask := ERROR | WARNING | PARALLEL | VERDICTOP;
+"GBProxy-Gb0-NS".FileMask := ERROR | WARNING | PARALLEL | VERDICTOP;
+"GBProxy-Gb0-BSSGP".FileMask := ERROR | WARNING | PARALLEL | VERDICTOP;
+"GBProxy-GSUP".FileMask := ERROR | WARNING | PARALLEL | VERDICTOP;
+
+[TESTPORT_PARAMETERS]
+*.GBPVTY.CTRL_MODE := "client"
+*.GBPVTY.CTRL_HOSTNAME := "127.0.0.1"
+*.GBPVTY.CTRL_PORTNUM := "4246"
+*.GBPVTY.CTRL_LOGIN_SKIPPED := "yes"
+*.GBPVTY.CTRL_DETECT_SERVER_DISCONNECTED := "yes"
+*.GBPVTY.CTRL_READMODE := "buffered"
+*.GBPVTY.CTRL_CLIENT_CLEANUP_LINEFEED := "yes"
+*.GBPVTY.CTRL_DETECT_CONNECTION_ESTABLISHMENT_RESULT := "yes"
+*.GBPVTY.PROMPT1 := "OsmoGbProxy> "
+
+
+[MODULE_PARAMETERS]
+Osmocom_VTY_Functions.mp_prompt_prefix := "OsmoGbProxy";
+
+[MAIN_CONTROLLER]
+
+[EXECUTE]
diff --git a/gbproxy/GBProxy_Tests.ttcn b/gbproxy/GBProxy_Tests.ttcn
new file mode 100644
index 000..b4e1739
--- /dev/null
+++ b/gbproxy/GBProxy_Tests.ttcn
@@ -0,0 +1,480 @@
+module GBProxy_Tests {
+
+/* Osmocom GBProxy test suite in TTCN-3
+ * (C) 2020 sysmocom - s.f.m.c. GmbH
+ * All rights reserved.
+ *
+ * Author: Daniel Willmann 
+
+ * Released under the terms of GNU General Public License, Version 2 or
+ * (at your option) any later version.
+ *
+ * SPDX-License-Identifier: GPL-2.0-or-later
+ */
+
+import from General_Types all;
+import from Osmocom_Types all;
+import from GSM_Types all;
+import from Native_Functions all;
+import from NS_Types all;
+import from NS_Emulation all;
+import from BSSGP_Types all;
+import from BSSGP_Emulation all;
+import from SCCPasp_Types all;
+import from Osmocom_Gb_Types all;
+
+import from MobileL3_CommonIE_Types all;
+import from MobileL3_GMM_SM_Types all;
+import from MobileL3_Types all;
+import from L3_Templates all;
+import from L3_Common all;
+
+import from TELNETasp_PortType all;
+import from Osmocom_VTY_Functions all;
+
+import from LLC_Types all;
+import from LLC_Templates all;
+
+import from GSM_RR_Types all;
+
+modulepar {
+   /* IP/port on which we run our internal GSUP/HLR emulation */
+   NSConfigurations_SGSN mp_nsconfig_sgsn := {
+   {
+   provider := {
+   ip := {
+   address_family := AF_INET,
+   local_udp_port := ,
+   local_ip := 

Change in osmo-ttcn3-hacks[master]: Send NS_ALIVE only after NS-RESET

2020-10-13 Thread daniel
daniel has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20636 )


Change subject: Send NS_ALIVE only after NS-RESET
..

Send NS_ALIVE only after NS-RESET

We should wait for an NS-RESET before we start sending NS-ALIVE. Also
unified the log messages a bit (received -> Rx).

Related: SYS#5002
Change-Id: Ie01fee70297255b3d9c091bc2cec75b0f915c588
---
M library/NS_Emulation.ttcnpp
1 file changed, 5 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks 
refs/changes/36/20636/1

diff --git a/library/NS_Emulation.ttcnpp b/library/NS_Emulation.ttcnpp
index 5fb2aa6..f984aa7 100644
--- a/library/NS_Emulation.ttcnpp
+++ b/library/NS_Emulation.ttcnpp
@@ -218,9 +218,7 @@
}

[config.role_sgsn] 
NSCP.receive(NS_Provider_Evt:{link_status:=NS_PROV_LINK_STATUS_UP}) {
-   log("Provider Link came up: sending NS-ALIVE");
-   f_sendAlive();
-   Tns_test.start;
+   log("Provider Link came up: waiting for NS-RESET");
}

[not config.role_sgsn] 
NSCP.receive(NS_Provider_Evt:{link_status:=NS_PROV_LINK_STATUS_UP}) {
@@ -230,7 +228,7 @@

/* Stop t_alive when receiving ALIVE-ACK */
[Tns_alive.running] NSCP.receive(t_NS_ALIVE_ACK) {
-   log("NS-ALIVE-ACK received: stopping Tns-alive; 
starting Tns-test");
+   log("Rx NS-ALIVE-ACK: stopping Tns-alive; starting 
Tns-test");
Tns_alive.stop;
Tns_test.start;
}
@@ -250,6 +248,9 @@
[] NSCP.receive(tr_NS_RESET(?, config.nsvci, config.nsei)) -> 
value rf {
f_change_state(NSE_S_ALIVE_BLOCKED);
NSCP.send(ts_NS_RESET_ACK(config.nsvci, config.nsei));
+   log("Rx NS-RESET: Sending NS-ALIVE");
+   f_sendAlive();
+   Tns_test.start;
}

/* Respond to RESET with wrong NSEI/NSVCI */

--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20636
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ie01fee70297255b3d9c091bc2cec75b0f915c588
Gerrit-Change-Number: 20636
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-MessageType: newchange


Change in osmo-ttcn3-hacks[master]: PCU: add IPv6 support for SNS

2020-10-13 Thread lynxis lazus
lynxis lazus has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20003 )


Change subject: PCU: add IPv6 support for SNS
..


Removed Code-Review-2 by lynxis lazus 
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20003
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I43d64caca600fff78f3fbbb3e8179f447f235d46
Gerrit-Change-Number: 20003
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus 
Gerrit-CC: laforge 
Gerrit-MessageType: deleteVote


Change in osmo-ttcn3-hacks[master]: PCU: add IPv6 support for SNS

2020-10-13 Thread lynxis lazus
Hello Jenkins Builder,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20003

to look at the new patch set (#3).

Change subject: PCU: add IPv6 support for SNS
..

PCU: add IPv6 support for SNS

With PCUIF 10 the remote can be IPv4 or IPv6.
Add all missing parts including SNS IPv6 elements,
the support to omit IPv4 elements and a PCU_Tests_SNSv6.cfg
configuration to run all tests with IPv6

Change-Id: I43d64caca600fff78f3fbbb3e8179f447f235d46
---
M library/Osmocom_Gb_Types.ttcn
M pcu/PCU_Tests_SNS.cfg
M pcu/PCU_Tests_SNS.ttcn
A pcu/PCU_Tests_SNSv6.cfg
4 files changed, 171 insertions(+), 39 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks 
refs/changes/03/20003/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20003
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I43d64caca600fff78f3fbbb3e8179f447f235d46
Gerrit-Change-Number: 20003
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus 
Gerrit-CC: laforge 
Gerrit-MessageType: newpatchset


Change in osmo-ttcn3-hacks[master]: pcu/PCU_Tests_SNS.cfg: fix configuration to match new NS provider

2020-10-13 Thread lynxis lazus
lynxis lazus has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20635 )


Change subject: pcu/PCU_Tests_SNS.cfg: fix configuration to match new NS 
provider
..

pcu/PCU_Tests_SNS.cfg: fix configuration to match new NS provider

The NS uses now a provider to support FR and IP at the same time.

Change-Id: I6b0fa97a1098070a4f63228568092084503d6164
---
M pcu/PCU_Tests_SNS.cfg
1 file changed, 9 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks 
refs/changes/35/20635/1

diff --git a/pcu/PCU_Tests_SNS.cfg b/pcu/PCU_Tests_SNS.cfg
index 022f91c..50f6129 100644
--- a/pcu/PCU_Tests_SNS.cfg
+++ b/pcu/PCU_Tests_SNS.cfg
@@ -8,10 +8,15 @@

 [MODULE_PARAMETERS]
 SGSN_Components.mp_nsconfig := {
-   local_ip := "127.0.0.1",
-   local_udp_port := 23000,
-   remote_ip := "127.0.0.1",
-   remote_udp_port := 22000,
+   provider := {
+   ip := {
+   address_family := AF_INET,
+   local_ip := "127.0.0.1",
+   local_udp_port := 23000,
+   remote_ip := "127.0.0.1",
+   remote_udp_port := 22000
+   }
+   },
nsvci := 1234,
nsei := 1234
 }

--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/20635
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I6b0fa97a1098070a4f63228568092084503d6164
Gerrit-Change-Number: 20635
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: templates/osmo-bts-trx: Increase osmo-trx rx-gain

2020-10-13 Thread pespin
pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20634 )


Change subject: templates/osmo-bts-trx: Increase osmo-trx rx-gain
..

templates/osmo-bts-trx: Increase osmo-trx rx-gain

Otherwise osmo-trx with a B200 seems to be having issues receiving some
data from the MS otherwise, making "sms" test fail. In general LU works
fine, and when MS sends the SMS, CM Service Request+Accept also works
fine, but then the BTS never sees the CP-DATA afterwards probably due to
MS power loop.

Change-Id: I5f7e3d22a142521ab83db81635ded252dabcf111
---
M src/osmo_gsm_tester/templates/osmo-bts-trx.cfg.tmpl
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester 
refs/changes/34/20634/1

diff --git a/src/osmo_gsm_tester/templates/osmo-bts-trx.cfg.tmpl 
b/src/osmo_gsm_tester/templates/osmo-bts-trx.cfg.tmpl
index 0459dd0..b202880 100644
--- a/src/osmo_gsm_tester/templates/osmo-bts-trx.cfg.tmpl
+++ b/src/osmo_gsm_tester/templates/osmo-bts-trx.cfg.tmpl
@@ -28,7 +28,7 @@
 %endif
 %for chan in osmo_bts_trx.osmo_trx.channels:
  instance ${loop.index}
-  osmotrx rx-gain 25
+  osmotrx rx-gain 50
   osmotrx tx-attenuation oml
 %endfor
 bts 0

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20634
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5f7e3d22a142521ab83db81635ded252dabcf111
Gerrit-Change-Number: 20634
Gerrit-PatchSet: 1
Gerrit-Owner: pespin 
Gerrit-MessageType: newchange


Change in osmo-gsm-tester[master]: srs{enb, ue}: add rf_dev_sync option to config templates

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615 )

Change subject: srs{enb,ue}: add rf_dev_sync option to config templates
..

srs{enb,ue}: add rf_dev_sync option to config templates

pass-through the option so they can be used in templates

just concatenate with rf_dev_args for srsLTE eNB/UE, arguments
parsing will handle them

Change-Id: I3818026c159780f2996f547163cdf730afad
---
M src/osmo_gsm_tester/obj/enb_srs.py
M src/osmo_gsm_tester/obj/ms_srs.py
M src/osmo_gsm_tester/templates/srsenb.conf.tmpl
M src/osmo_gsm_tester/templates/srsue.conf.tmpl
4 files changed, 7 insertions(+), 2 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/enb_srs.py 
b/src/osmo_gsm_tester/obj/enb_srs.py
index ad92af5..63e5d7c 100644
--- a/src/osmo_gsm_tester/obj/enb_srs.py
+++ b/src/osmo_gsm_tester/obj/enb_srs.py
@@ -191,6 +191,7 @@
 config.overlay(values, dict(enb={'enable_pcap': self.enable_pcap}))

 config.overlay(values, dict(enb={'enable_dl_awgn': 
util.str2bool(values['enb'].get('enable_dl_awgn', 'false'))}))
+config.overlay(values, dict(enb={'rf_dev_sync': 
values['enb'].get('rf_dev_sync', None)}))

 self._additional_args = []
 for add_args in values['enb'].get('additional_args', []):
diff --git a/src/osmo_gsm_tester/obj/ms_srs.py 
b/src/osmo_gsm_tester/obj/ms_srs.py
index 2546624..be3e799 100644
--- a/src/osmo_gsm_tester/obj/ms_srs.py
+++ b/src/osmo_gsm_tester/obj/ms_srs.py
@@ -34,6 +34,7 @@
 resource_schema = {
 'rf_dev_type': schema.STR,
 'rf_dev_args': schema.STR,
+'rf_dev_sync': schema.STR,
 'num_carriers': schema.UINT,
 'additional_args[]': schema.STR,
 'airplane_t_on_ms': schema.INT,
@@ -257,6 +258,9 @@

 self.num_carriers = int(values['ue'].get('num_carriers', 1))

+# Simply pass-through the sync options
+config.overlay(values, dict(ue={'rf_dev_sync': 
values['ue'].get('rf_dev_sync', None)}))
+
 # We need to set some specific variables programatically here to match 
IP addresses:
 if self._conf.get('rf_dev_type') == 'zmq':
 base_srate = num_prb2base_srate(self.enb.num_prb())
diff --git a/src/osmo_gsm_tester/templates/srsenb.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsenb.conf.tmpl
index 7409b3a..5df432b 100644
--- a/src/osmo_gsm_tester/templates/srsenb.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsenb.conf.tmpl
@@ -75,7 +75,7 @@
 # For best performance when BW<5 MHz (25 PRB), use the following device_args 
settings:
 # USRP B210: send_frame_size=512,recv_frame_size=512

-device_args = ${enb.rf_dev_args}
+device_args = ${enb.rf_dev_args},${enb.rf_dev_sync}
 #time_adv_nsamples = auto
 #burst_preamble_us = auto

diff --git a/src/osmo_gsm_tester/templates/srsue.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsue.conf.tmpl
index be75347..284922b 100644
--- a/src/osmo_gsm_tester/templates/srsue.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsue.conf.tmpl
@@ -54,7 +54,7 @@
 # For best performance when BW<5 MHz (25 PRB), use the following device_args 
settings:
 # USRP B210: send_frame_size=512,recv_frame_size=512

-device_args = ${ue.rf_dev_args}
+device_args = ${ue.rf_dev_args},${ue.rf_dev_sync}
 #time_adv_nsamples = auto
 #burst_preamble_us = auto
 #continuous_tx = auto

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I3818026c159780f2996f547163cdf730afad
Gerrit-Change-Number: 20615
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: srs{enb, ue}: add rf_dev_sync option to config templates

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615 )

Change subject: srs{enb,ue}: add rf_dev_sync option to config templates
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I3818026c159780f2996f547163cdf730afad
Gerrit-Change-Number: 20615
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 11:23:55 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: srs{enb, ue}: add rf_dev_sync option to config templates

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615 )

Change subject: srs{enb,ue}: add rf_dev_sync option to config templates
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I3818026c159780f2996f547163cdf730afad
Gerrit-Change-Number: 20615
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 11:23:51 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: srs{enb, ue}: add rf_dev_sync option to config templates

2020-10-13 Thread srs_andre
srs_andre has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615 )

Change subject: srs{enb,ue}: add rf_dev_sync option to config templates
..


Patch Set 1:

(1 comment)

see below

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615/1/src/osmo_gsm_tester/templates/srsenb.conf.tmpl
File src/osmo_gsm_tester/templates/srsenb.conf.tmpl:

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615/1/src/osmo_gsm_tester/templates/srsenb.conf.tmpl@78
PS1, Line 78: device_args = ${enb.rf_dev_args},${enb.rf_dev_sync}
> So does it work if you get Python's "None" here in rf_dev_args? what do you 
> get? String "None" is pr […]
correct. It will add "None", but the RF driver will simply ignore it. Amarisoft 
needs "none" though. So the default is needed and it should be lower-case.



--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I3818026c159780f2996f547163cdf730afad
Gerrit-Change-Number: 20615
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 11:01:50 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin 
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: {enb, epc, ms}_srs: refactor start/stop and KPI helpers

2020-10-13 Thread srs_andre
srs_andre has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20601 )

Change subject: {enb,epc,ms}_srs: refactor start/stop and KPI helpers
..


Patch Set 1:

(1 comment)

will do, thanks

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20601/1/src/osmo_gsm_tester/obj/srslte_common.py
File src/osmo_gsm_tester/obj/srslte_common.py:

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20601/1/src/osmo_gsm_tester/obj/srslte_common.py@30
PS1, Line 30: self.kpis = None
> Yes it's in the base class, please drop the others.
Ack



--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20601
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I2ff36df6f1a07ed8ddac39d296d62fe44a215283
Gerrit-Change-Number: 20601
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:59:18 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin 
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: {enb, epc, ms}_srs: refactor start/stop and KPI helpers

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20601 )

Change subject: {enb,epc,ms}_srs: refactor start/stop and KPI helpers
..

{enb,epc,ms}_srs: refactor start/stop and KPI helpers

refactor some of the functionality to srslte_common

Change-Id: I2ff36df6f1a07ed8ddac39d296d62fe44a215283
---
M src/osmo_gsm_tester/obj/enb_srs.py
M src/osmo_gsm_tester/obj/epc_srs.py
M src/osmo_gsm_tester/obj/ms_srs.py
M src/osmo_gsm_tester/obj/srslte_common.py
4 files changed, 57 insertions(+), 24 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/enb_srs.py 
b/src/osmo_gsm_tester/obj/enb_srs.py
index 13d18f3..ec2d1fc 100644
--- a/src/osmo_gsm_tester/obj/enb_srs.py
+++ b/src/osmo_gsm_tester/obj/enb_srs.py
@@ -51,6 +51,7 @@

 def __init__(self, testenv, conf):
 super().__init__(testenv, conf, srsENB.BINFILE)
+srslte_common.__init__(self)
 self.ue = None
 self.run_dir = None
 self.gen_conf = None
@@ -73,6 +74,7 @@
 self.metrics_file = None
 self.stop_sleep_time = 6 # We require at most 5s to stop
 self.testenv = testenv
+self.kpis = None
 self._additional_args = []
 if not rf_type_valid(conf.get('rf_dev_type', None)):
 raise log.Error('Invalid rf_dev_type=%s' % conf.get('rf_dev_type', 
None))
@@ -100,12 +102,6 @@
 # Collect KPIs for each TC
 self.testenv.test().set_kpis(self.get_kpis())

-def sleep_after_stop(self):
-# Only sleep once
-if self.stop_sleep_time > 0:
-MainLoop.sleep(self.stop_sleep_time)
-self.stop_sleep_time = 0
-
 def start(self, epc):
 self.log('Starting srsENB')
 self._epc = epc
diff --git a/src/osmo_gsm_tester/obj/epc_srs.py 
b/src/osmo_gsm_tester/obj/epc_srs.py
index 44e06a5..dae8a56 100644
--- a/src/osmo_gsm_tester/obj/epc_srs.py
+++ b/src/osmo_gsm_tester/obj/epc_srs.py
@@ -23,6 +23,7 @@
 from ..core import log, util, config, template, process, remote
 from ..core import schema
 from . import epc
+from .srslte_common import srslte_common

 def on_register_schemas():
 config_schema = {
@@ -31,7 +32,7 @@
 }
 schema.register_config_schema('epc', config_schema)

-class srsEPC(epc.EPC):
+class srsEPC(epc.EPC, srslte_common):

 REMOTE_DIR = '/osmo-gsm-tester-srsepc'
 BINFILE = 'srsepc'
@@ -42,6 +43,7 @@

 def __init__(self, testenv, run_node):
 super().__init__(testenv, run_node, 'srsepc')
+srslte_common.__init__(self)
 self.run_dir = None
 self.config_file = None
 self.db_file = None
@@ -55,6 +57,7 @@
 self.remote_log_file = None
 self.remote_pcap_file = None
 self.enable_pcap = False
+self.kpis = None
 self.subscriber_list = []

 def cleanup(self):
diff --git a/src/osmo_gsm_tester/obj/ms_srs.py 
b/src/osmo_gsm_tester/obj/ms_srs.py
index 1c0f728..b5f5d70 100644
--- a/src/osmo_gsm_tester/obj/ms_srs.py
+++ b/src/osmo_gsm_tester/obj/ms_srs.py
@@ -24,7 +24,6 @@
 from ..core import log, util, config, template, process, remote
 from ..core import schema
 from .run_node import RunNode
-from ..core.event_loop import MainLoop
 from .ms import MS
 from .srslte_common import srslte_common

@@ -77,6 +76,7 @@
 def __init__(self, testenv, conf):
 self._run_node = RunNode.from_conf(conf.get('run_node', {}))
 super().__init__('srsue_%s' % self.addr(), conf)
+srslte_common.__init__(self)
 self.enb = None
 self.run_dir = None
 self.config_file = None
@@ -92,9 +92,9 @@
 self.remote_log_file = None
 self.remote_pcap_file = None
 self.remote_metrics_file = None
-self.stop_sleep_time = 6 # We require at most 5s to stop
 self.enable_pcap = False
 self.num_carriers = 1
+self.kpis = None
 self.testenv = testenv
 self._additional_args = []
 if not rf_type_valid(conf.get('rf_dev_type', None)):
@@ -150,16 +150,6 @@
 def netns(self):
 return "srsue1"

-def sleep_after_stop(self):
-# Only sleep once
-if self.stop_sleep_time > 0:
-MainLoop.sleep(self.stop_sleep_time)
-self.stop_sleep_time = 0
-
-def stop(self):
-self.testenv.stop_process(self.process)
-self.sleep_after_stop()
-
 def connect(self, enb):
 self.log('Starting srsue')
 self.enb = enb
diff --git a/src/osmo_gsm_tester/obj/srslte_common.py 
b/src/osmo_gsm_tester/obj/srslte_common.py
index 33f12d4..9619fb6 100644
--- a/src/osmo_gsm_tester/obj/srslte_common.py
+++ b/src/osmo_gsm_tester/obj/srslte_common.py
@@ -18,6 +18,7 @@
 # along with this program.  If not, see .

 from ..core import log
+from ..core.event_loop import MainLoop

 class srslte_common(): # don't inherit from 

Change in osmo-gsm-tester[master]: jenkins-build-amarisoft: fix build script for ZMQ module

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20613 )

Change subject: jenkins-build-amarisoft: fix build script for ZMQ module
..

jenkins-build-amarisoft: fix build script for ZMQ module

newer versions of srsLTE create symlinks with the rf library
including the version, see below. This caused the trx_zmq.so
lib to load .so.0 instead of .so only. this patch adds
another symlink so loading works fine again

libsrslte_rf.so -> libsrslte_rf.so.0
libsrslte_rf.so.0 -> libsrslte_rf.so.20.10.0
libsrslte_rf.so.20.10.0

Change-Id: Ib5302c9872f4c82a1ee5074f59de86709aa6c5a9
---
M contrib/jenkins-build-amarisoft.sh
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/contrib/jenkins-build-amarisoft.sh 
b/contrib/jenkins-build-amarisoft.sh
index fe106f1..f880876 100755
--- a/contrib/jenkins-build-amarisoft.sh
+++ b/contrib/jenkins-build-amarisoft.sh
@@ -50,6 +50,7 @@
 cp ${base}/${project_name_srslte}/build/lib/src/phy/rf/libsrslte_rf.so 
inst-amarisoftenb/
 cp ${base}/${project_name_zmq}/build/libtrx_zmq-linux-2018-10-18.so 
inst-amarisoftenb/trx_zmq.so
 patchelf --set-rpath '$ORIGIN/' inst-amarisoftenb/trx_zmq.so
+cd inst-amarisoftenb && ln -s libsrslte_rf.so libsrslte_rf.so.0 && cd 
..
 fi
 cp ${base}/inst-tmp/trx_uhd-linux*/trx_uhd.so inst-amarisoftenb/
 this="amarisoftenb.build-${BUILD_NUMBER-$(date +%Y-%m-%d_%H_%M_%S)}"

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20613
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ib5302c9872f4c82a1ee5074f59de86709aa6c5a9
Gerrit-Change-Number: 20613
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: amarisoft_enb.cfg.tmpl: seperate PRACH root sequences for cells

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20609 )

Change subject: amarisoft_enb.cfg.tmpl: seperate PRACH root sequences for cells
..

amarisoft_enb.cfg.tmpl: seperate PRACH root sequences for cells

seperate the root sequence index for each cell a bit more to
avoid detection of a single PRACH in both cells

Change-Id: I5aea1a27939c880944779c282e902465ab513715
---
M src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl 
b/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
index 18a37c4..c5b8a1e 100644
--- a/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
+++ b/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
@@ -74,7 +74,7 @@
 cell_id: ${cell.cell_id},
 n_id_cell: ${cell.pci},
 tac: 0x0007,
-root_sequence_index: ${loop.index + 204}, /* PRACH root sequence index */
+root_sequence_index: ${loop.index * 10 + 204}, /* PRACH root sequence 
index */

 ncell_list: [
 %for ncell in enb.cell_list:

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20609
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5aea1a27939c880944779c282e902465ab513715
Gerrit-Change-Number: 20609
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: enb_amarisoft: reduce max achievable UL rate for 100 PRB

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20608 )

Change subject: enb_amarisoft: reduce max achievable UL rate for 100 PRB
..

enb_amarisoft: reduce max achievable UL rate for 100 PRB

Change-Id: Ib7a0b8e2948cdf6c0a15d4ad2bf7d218a6b28ea2
---
M src/osmo_gsm_tester/obj/enb_amarisoft.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/enb_amarisoft.py 
b/src/osmo_gsm_tester/obj/enb_amarisoft.py
index 4495f47..0f8944f 100644
--- a/src/osmo_gsm_tester/obj/enb_amarisoft.py
+++ b/src/osmo_gsm_tester/obj/enb_amarisoft.py
@@ -272,7 +272,7 @@
25 : 10e6,
50 : 21e6,
75 : 32e6,
-   100 : 51e6 }
+   100 : 47e6 }
 if downlink:
 max_rate = max_phy_rate_tm1_dl[self.num_prb()]
 else:

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20608
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ib7a0b8e2948cdf6c0a15d4ad2bf7d218a6b28ea2
Gerrit-Change-Number: 20608
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: srslte_common: fix indent in whole file

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20611 )

Change subject: srslte_common: fix indent in whole file
..

srslte_common: fix indent in whole file

Change-Id: I8aee79e9d451761dde8f35f84d75483dc4e23040
---
M src/osmo_gsm_tester/obj/srslte_common.py
1 file changed, 65 insertions(+), 65 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/srslte_common.py 
b/src/osmo_gsm_tester/obj/srslte_common.py
index c4d8e62..e221940 100644
--- a/src/osmo_gsm_tester/obj/srslte_common.py
+++ b/src/osmo_gsm_tester/obj/srslte_common.py
@@ -22,79 +22,79 @@

 class srslte_common(): # don't inherit from log.Origin here but instead use 
.name() from whoever inherits from us

-  def __init__(self):
-self.log_file = None
-self.process = None
-self.metrics_file = None
-self.stop_sleep_time = 6 # We require at most 5s to stop
-self.kpis = None
+def __init__(self):
+self.log_file = None
+self.process = None
+self.metrics_file = None
+self.stop_sleep_time = 6 # We require at most 5s to stop
+self.kpis = None

-  def sleep_after_stop(self):
-# Only sleep once
-if self.stop_sleep_time > 0:
-MainLoop.sleep(self.stop_sleep_time)
-self.stop_sleep_time = 0
+def sleep_after_stop(self):
+# Only sleep once
+if self.stop_sleep_time > 0:
+MainLoop.sleep(self.stop_sleep_time)
+self.stop_sleep_time = 0

-  def stop(self):
-  self.testenv.stop_process(self.process)
-  self.sleep_after_stop()
+def stop(self):
+self.testenv.stop_process(self.process)
+self.sleep_after_stop()

-  def get_kpis(self):
-  ''' Return all KPI '''
-  if self.kpis is None:
-  self.extract_kpis()
-  return self.kpis
+def get_kpis(self):
+''' Return all KPI '''
+if self.kpis is None:
+self.extract_kpis()
+return self.kpis

-  def get_log_kpis(self):
-  ''' Return KPIs extracted from log '''
-  if self.kpis is None:
-  self.extract_kpis()
+def get_log_kpis(self):
+''' Return KPIs extracted from log '''
+if self.kpis is None:
+self.extract_kpis()

-  # Use log KPIs if they exist for this node
-  if "log_" + self.name() in self.kpis:
-  log_kpi = self.kpis["log_" + self.name()]
-  else:
-  log_kpi = {}
+# Use log KPIs if they exist for this node
+if "log_" + self.name() in self.kpis:
+log_kpi = self.kpis["log_" + self.name()]
+else:
+log_kpi = {}

-  # Make sure we have the errors and warnings counter in the dict
-  if 'total_errors' not in log_kpi:
-  log_kpi['total_errors'] = 0
-  if 'total_warnings' not in log_kpi:
-  log_kpi['total_warnings'] = 0
-  return log_kpi
+# Make sure we have the errors and warnings counter in the dict
+if 'total_errors' not in log_kpi:
+log_kpi['total_errors'] = 0
+if 'total_warnings' not in log_kpi:
+log_kpi['total_warnings'] = 0
+return log_kpi

-  def extract_kpis(self):
-  ''' Use the srsLTE KPI analyzer module (part of srsLTE.git) if available 
to collect KPIs '''
+def extract_kpis(self):
+''' Use the srsLTE KPI analyzer module (part of srsLTE.git) if 
available to collect KPIs '''

-  # Stop application, copy back logs and process them
-  if self.running():
-  self.stop()
-  self.cleanup()
+# Stop application, copy back logs and process them
+if self.running():
+self.stop()
+self.cleanup()

-  self.kpis = {}
-  try:
-  # Please make sure the srsLTE scripts folder is included in your 
PYTHONPATH env variable
-  from kpi_analyzer import kpi_analyzer
-  analyzer = kpi_analyzer(self.name())
-  if self.log_file is not None:
-  self.kpis["log_" + self.name()] = 
analyzer.get_kpi_from_logfile(self.log_file)
-  if self.process.get_output_file('stdout') is not None:
-  self.kpis["stdout_" + self.name()] = 
analyzer.get_kpi_from_stdout(self.process.get_output_file('stdout'))
-  if self.metrics_file is not None:
-  self.kpis["csv_" + self.name()] = 
analyzer.get_kpi_from_csv(self.metrics_file)
-  except ImportError:
-  self.log("Can't load KPI analyzer module.")
-  self.kpis = {}
+self.kpis = {}
+try:
+# Please make sure the srsLTE scripts folder is included in your 
PYTHONPATH env variable
+from kpi_analyzer import kpi_analyzer
+analyzer = kpi_analyzer(self.name())
+if self.log_file is not None:
+self.kpis["log_" + self.name()] = 
analyzer.get_kpi_from_logfile(self.log_file)
+if 

Change in osmo-gsm-tester[master]: enb_amarisoft: add rf_dev_sync option to specify the sync type

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20614 )

Change subject: enb_amarisoft: add rf_dev_sync option to specify the sync type
..

enb_amarisoft: add rf_dev_sync option to specify the sync type

this allows to specify "external_clock" for example to use
the external REF input of an USRP

Change-Id: Ie066993440840cbd2c18e8f0c0c577b92ac7092d
---
M src/osmo_gsm_tester/obj/enb.py
M src/osmo_gsm_tester/obj/enb_amarisoft.py
M src/osmo_gsm_tester/templates/amarisoft_rf_driver.cfg.tmpl
M sysmocom/defaults.conf
4 files changed, 4 insertions(+), 2 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/enb.py b/src/osmo_gsm_tester/obj/enb.py
index 6384f15..fea78de 100644
--- a/src/osmo_gsm_tester/obj/enb.py
+++ b/src/osmo_gsm_tester/obj/enb.py
@@ -34,6 +34,7 @@
 'rx_gain': schema.UINT,
 'rf_dev_type': schema.STR,
 'rf_dev_args': schema.STR,
+'rf_dev_sync': schema.STR,
 'additional_args[]': schema.STR,
 'inactivity_timer': schema.INT,
 'enable_measurements': schema.BOOL_STR,
diff --git a/src/osmo_gsm_tester/obj/enb_amarisoft.py 
b/src/osmo_gsm_tester/obj/enb_amarisoft.py
index 0f8944f..1fc4485 100644
--- a/src/osmo_gsm_tester/obj/enb_amarisoft.py
+++ b/src/osmo_gsm_tester/obj/enb_amarisoft.py
@@ -214,6 +214,7 @@
 # different cfg namespace 'trx'. Copy needed values over there:
 config.overlay(values, 
dict(trx=dict(rf_dev_type=values['enb'].get('rf_dev_type', None),
  
rf_dev_args=values['enb'].get('rf_dev_args', None),
+ 
rf_dev_sync=values['enb'].get('rf_dev_sync', None),
  
rx_gain=values['enb'].get('rx_gain', None),
  
tx_gain=values['enb'].get('tx_gain', None),
 )))
diff --git a/src/osmo_gsm_tester/templates/amarisoft_rf_driver.cfg.tmpl 
b/src/osmo_gsm_tester/templates/amarisoft_rf_driver.cfg.tmpl
index cd08835..05689c7 100644
--- a/src/osmo_gsm_tester/templates/amarisoft_rf_driver.cfg.tmpl
+++ b/src/osmo_gsm_tester/templates/amarisoft_rf_driver.cfg.tmpl
@@ -1,8 +1,7 @@
 rf_driver: {
 name: "${trx.rf_dev_type}",
-sync: "${'1' if trx.rf_dev_type == 'zmq' else 'none'}",
-
 args: "${trx.rf_dev_args}",
+sync: "${trx.rf_dev_sync}",

 % if trx.rf_dev_type == 'zmq':
 dl_sample_bits: 16,
diff --git a/sysmocom/defaults.conf b/sysmocom/defaults.conf
index b175cbb..ed0b575 100644
--- a/sysmocom/defaults.conf
+++ b/sysmocom/defaults.conf
@@ -154,6 +154,7 @@
   enable_pcap: false
   tx_gain: 89
   rx_gain: 60
+  rf_dev_sync: none
   log_options: 
all.level=error,all.max_size=0,nas.level=debug,nas.max_size=1,s1ap.level=debug,s1ap.max_size=1,x2ap.level=debug,x2ap.max_size=1,rrc.level=debug,rrc.max_size=1

 srsue:

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20614
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ie066993440840cbd2c18e8f0c0c577b92ac7092d
Gerrit-Change-Number: 20614
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: amarisoft_enb.cfg.tmpl: use default tac 0007

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20605 )

Change subject: amarisoft_enb.cfg.tmpl: use default tac 0007
..

amarisoft_enb.cfg.tmpl: use default tac 0007

this is the tac used in srsepc.

Change-Id: I1de432a52281ea210abae5648239fd89ba5eb4ce
---
M src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl 
b/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
index 2893b9e..f46b77e 100644
--- a/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
+++ b/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
@@ -73,7 +73,7 @@
 rf_port: ${cell.rf_port},
 cell_id: ${cell.cell_id},
 n_id_cell: ${cell.pci},
-tac: 0x0001,
+tac: 0x0007,
 root_sequence_index: ${loop.index + 204}, /* PRACH root sequence index */

 ncell_list: [

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20605
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I1de432a52281ea210abae5648239fd89ba5eb4ce
Gerrit-Change-Number: 20605
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: enb_amarisoft: add rf_dev_sync option to specify the sync type

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20614 )

Change subject: enb_amarisoft: add rf_dev_sync option to specify the sync type
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20614
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ie066993440840cbd2c18e8f0c0c577b92ac7092d
Gerrit-Change-Number: 20614
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:25:20 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: amarisoft_enb.cfg.tmpl: increas threshold for failed UL retransmissions

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20607 )

Change subject: amarisoft_enb.cfg.tmpl: increas threshold for failed UL 
retransmissions
..

amarisoft_enb.cfg.tmpl: increas threshold for failed UL retransmissions

this avoids the eNB from releasing the UE too early if the UL channel
stays very bad for a longer time but many PUSCH are scheduled

Change-Id: Ib82616e920eecc70fbded12433e54bd90ae1bcd4
---
M src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl 
b/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
index f46b77e..18a37c4 100644
--- a/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
+++ b/src/osmo_gsm_tester/templates/amarisoft_enb.cfg.tmpl
@@ -206,6 +206,7 @@
 mac_config: {
ul_max_harq_tx: 5, /* max number of HARQ transmissions for uplink */
dl_max_harq_tx: 5, /* max number of HARQ transmissions for downlink */
+   ul_max_consecutive_retx: 1000,
 },

 /* CPU load limitation */

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20607
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ib82616e920eecc70fbded12433e54bd90ae1bcd4
Gerrit-Change-Number: 20607
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: lte: update UHD device args for MIMO usage

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20606 )

Change subject: lte: update UHD device args for MIMO usage
..

lte: update UHD device args for MIMO usage

the USRP B210 device args were not ideal for MIMO configs for
both Amarisoft and srsLTE.

Change-Id: I3785bb96eb7c10a6b10b3d1a86fa71d1f787a7a3
---
M src/osmo_gsm_tester/obj/enb_amarisoft.py
M src/osmo_gsm_tester/obj/enb_srs.py
M src/osmo_gsm_tester/obj/ms_srs.py
3 files changed, 47 insertions(+), 19 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/enb_amarisoft.py 
b/src/osmo_gsm_tester/obj/enb_amarisoft.py
index 5b8e05e..4495f47 100644
--- a/src/osmo_gsm_tester/obj/enb_amarisoft.py
+++ b/src/osmo_gsm_tester/obj/enb_amarisoft.py
@@ -183,16 +183,24 @@
 rf_dev_args = values['enb'].get('rf_dev_args', '')
 rf_dev_args += ',' if rf_dev_args != '' and not 
rf_dev_args.endswith(',') else ''

-if self._num_prb < 25:
-rf_dev_args += 'send_frame_size=512,recv_frame_size=512'
-elif self._num_prb == 25:
-rf_dev_args += 'send_frame_size=1024,recv_frame_size=1024'
-elif self._num_prb > 25:
-rf_dev_args += 'num_recv_frames=64,num_send_frames=64'
+if self._txmode == 1:
+# SISO config
+if self._num_prb < 25:
+rf_dev_args += 
'send_frame_size=512,recv_frame_size=512'
+elif self._num_prb == 25:
+rf_dev_args += 
'send_frame_size=1024,recv_frame_size=1024'
+else:
+rf_dev_args += ''
+else:
+# MIMO config
+if self._num_prb == 6:
+rf_dev_args += 
'send_frame_size=512,recv_frame_size=512'
+else:
+rf_dev_args += 'num_recv_frames=64,num_send_frames=64'

-if self._num_prb > 50:
-# Reduce over the wire format to sc12
-rf_dev_args += ',otw_format=sc12'
+if self._num_prb > 50:
+# Reduce over the wire format to sc12
+rf_dev_args += ',otw_format=sc12'

 config.overlay(values, dict(enb=dict(rf_dev_args=rf_dev_args)))

diff --git a/src/osmo_gsm_tester/obj/enb_srs.py 
b/src/osmo_gsm_tester/obj/enb_srs.py
index ec2d1fc..ad92af5 100644
--- a/src/osmo_gsm_tester/obj/enb_srs.py
+++ b/src/osmo_gsm_tester/obj/enb_srs.py
@@ -207,12 +207,23 @@
 rf_dev_args = values['enb'].get('rf_dev_args', '')
 rf_dev_args += ',' if rf_dev_args != '' and not 
rf_dev_args.endswith(',') else ''

-if self._num_prb < 25:
-rf_dev_args += 'send_frame_size=512,recv_frame_size=512'
-elif self._num_prb == 25:
-rf_dev_args += 'send_frame_size=1024,recv_frame_size=1024'
-elif self._num_prb > 25:
+if self._num_prb == 75:
+rf_dev_args += 'master_clock_rate=15.36e6,'
+
+if self._txmode <= 2:
+# SISO config
+if self._num_prb < 25:
+rf_dev_args += 
'send_frame_size=512,recv_frame_size=512'
+elif self._num_prb == 25:
+rf_dev_args += 
'send_frame_size=1024,recv_frame_size=1024'
+else:
+rf_dev_args += ''
+else:
+# MIMO config
 rf_dev_args += 'num_recv_frames=64,num_send_frames=64'
+if self._num_prb > 50:
+# Reduce over the wire format to sc12
+rf_dev_args += ',otw_format=sc12'

 config.overlay(values, dict(enb=dict(rf_dev_args=rf_dev_args)))

diff --git a/src/osmo_gsm_tester/obj/ms_srs.py 
b/src/osmo_gsm_tester/obj/ms_srs.py
index 701794f..22e110d 100644
--- a/src/osmo_gsm_tester/obj/ms_srs.py
+++ b/src/osmo_gsm_tester/obj/ms_srs.py
@@ -306,12 +306,21 @@
 rf_dev_args = values['ue'].get('rf_dev_args', '')
 rf_dev_args += ',' if rf_dev_args != '' and not 
rf_dev_args.endswith(',') else ''

-if self.enb.num_prb() < 25:
-rf_dev_args += 'send_frame_size=512,recv_frame_size=512'
-elif self.enb.num_prb() == 25:
-rf_dev_args += 'send_frame_size=1024,recv_frame_size=1024'
-elif self.enb.num_prb() > 50:
+if self.enb.num_prb() == 75:
+rf_dev_args += 'master_clock_rate=15.36e6,'
+
+if self.enb.num_ports() == 1:
+# SISO config
+if 

Change in osmo-gsm-tester[master]: amarisoft_sib23.asn.tmpl: set p-b value in SIB23 depending on TM

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20604 )

Change subject: amarisoft_sib23.asn.tmpl: set p-b value in SIB23 depending on TM
..

amarisoft_sib23.asn.tmpl: set p-b value in SIB23 depending on TM

this fixes a wrong p-b value used for TM2 and above

Change-Id: I89c557c456f3330f491cc66bc4931a0234bf6e02
---
M src/osmo_gsm_tester/templates/amarisoft_sib23.asn.tmpl
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/templates/amarisoft_sib23.asn.tmpl 
b/src/osmo_gsm_tester/templates/amarisoft_sib23.asn.tmpl
index c08b645..f26cebd 100644
--- a/src/osmo_gsm_tester/templates/amarisoft_sib23.asn.tmpl
+++ b/src/osmo_gsm_tester/templates/amarisoft_sib23.asn.tmpl
@@ -37,7 +37,11 @@
 },
 pdsch-ConfigCommon {
   referenceSignalPower -5,
+  % if int(enb.get('transmission_mode')) == 1:
+  p-b 0
+  % else:
   p-b 1
+  % endif
 },
 pusch-ConfigCommon {
   pusch-ConfigBasic {

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20604
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I89c557c456f3330f491cc66bc4931a0234bf6e02
Gerrit-Change-Number: 20604
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: srslte_common: add get_num_phy_errors() method

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20610 )

Change subject: srslte_common: add get_num_phy_errors() method
..

srslte_common: add get_num_phy_errors() method

the get_num_phy_errors() uses the external KPI analyzer module
to calculate the number of PHY layers errors in a vector of log metrics

Change-Id: I0c8de87e7f83f986e288585640ce56847959c497
---
M src/osmo_gsm_tester/obj/srslte_common.py
1 file changed, 11 insertions(+), 0 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/srslte_common.py 
b/src/osmo_gsm_tester/obj/srslte_common.py
index 9619fb6..c4d8e62 100644
--- a/src/osmo_gsm_tester/obj/srslte_common.py
+++ b/src/osmo_gsm_tester/obj/srslte_common.py
@@ -87,3 +87,14 @@
   self.kpis = {}

   return self.kpis
+
+   def get_num_phy_errors(self, kpi):
+   """ Use KPI analyzer to calculate the number PHY errors for either UE 
or eNB components from parsed KPI vector """
+   try:
+   # Same as above, make sure the srsLTE scripts folder is included in 
your PYTHONPATH env variable
+   from kpi_analyzer import kpi_analyzer
+   analyzer = kpi_analyzer(self.name())
+   return analyzer.get_num_phy_errors(kpi)
+   except ImportError:
+   self.log("Can't load KPI analyzer module.")
+   return 0
\ No newline at end of file

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20610
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I0c8de87e7f83f986e288585640ce56847959c497
Gerrit-Change-Number: 20610
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: srsenb_rr.conf.tmpl: set TAC for cells to 0x0007

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20602 )

Change subject: srsenb_rr.conf.tmpl: set TAC for cells to 0x0007
..

srsenb_rr.conf.tmpl: set TAC for cells to 0x0007

this is the default TAC used in the EPC. A recent EPC patch
will reject the S1 setup request if the TACs don't match

Change-Id: I94fdb96631ce3ae35a1b2a9f90d1101df9161233
---
M src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
index 6097a66..b1e9cb2 100644
--- a/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
@@ -59,7 +59,7 @@
   {
 rf_port = ${cell.rf_port};
 cell_id = ${cell.cell_id};
-tac = 0x0001;
+tac = 0x0007;
 pci = ${loop.index + 1};
 root_seq_idx = ${loop.index + 204};
 dl_earfcn = ${cell.dl_earfcn};

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20602
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I94fdb96631ce3ae35a1b2a9f90d1101df9161233
Gerrit-Change-Number: 20602
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: srs{enb, ue}: add rf_dev_sync option to config templates

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615 )


Change subject: srs{enb,ue}: add rf_dev_sync option to config templates
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20615
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I3818026c159780f2996f547163cdf730afad
Gerrit-Change-Number: 20615
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: ms_srs: add freq_offset as resource param

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20612 )

Change subject: ms_srs: add freq_offset as resource param
..

ms_srs: add freq_offset as resource param

this allows to specify a freq tuning offset in Hz in the MS resource config

Change-Id: Iab726edd27e890d2c2dddbea91afe80d8a5068aa
---
M src/osmo_gsm_tester/obj/ms_srs.py
M src/osmo_gsm_tester/templates/srsue.conf.tmpl
M sysmocom/defaults.conf
3 files changed, 3 insertions(+), 1 deletion(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/ms_srs.py 
b/src/osmo_gsm_tester/obj/ms_srs.py
index 22e110d..2546624 100644
--- a/src/osmo_gsm_tester/obj/ms_srs.py
+++ b/src/osmo_gsm_tester/obj/ms_srs.py
@@ -40,6 +40,7 @@
 'airplane_t_off_ms': schema.INT,
 'tx_gain': schema.UINT,
 'rx_gain': schema.UINT,
+'freq_offset': schema.INT,
 }
 for key, val in RunNode.schema().items():
 resource_schema['run_node.%s' % key] = val
diff --git a/src/osmo_gsm_tester/templates/srsue.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsue.conf.tmpl
index 05c6c55..be75347 100644
--- a/src/osmo_gsm_tester/templates/srsue.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsue.conf.tmpl
@@ -38,7 +38,7 @@
 % else:
 dl_earfcn = 2850
 % endif
-freq_offset = 0
+freq_offset = ${ue.freq_offset}
 tx_gain = ${ue.tx_gain}
 rx_gain = ${ue.rx_gain}

diff --git a/sysmocom/defaults.conf b/sysmocom/defaults.conf
index 2f2e317..b175cbb 100644
--- a/sysmocom/defaults.conf
+++ b/sysmocom/defaults.conf
@@ -163,6 +163,7 @@
   num_carriers: 1
   tx_gain: 80
   rx_gain: 40
+  freq_offset: 0
   log_all_level: warning

 amarisoftue:

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20612
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Iab726edd27e890d2c2dddbea91afe80d8a5068aa
Gerrit-Change-Number: 20612
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: ms_srs: fix ZMQ config for 4xCA

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20603 )

Change subject: ms_srs: fix ZMQ config for 4xCA
..

ms_srs: fix ZMQ config for 4xCA

add missing freq config for 4xCA

Change-Id: I88a389f5b03d6b55ff5ef403d5107c648a11686f
---
M src/osmo_gsm_tester/obj/ms_srs.py
1 file changed, 11 insertions(+), 0 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/ms_srs.py 
b/src/osmo_gsm_tester/obj/ms_srs.py
index b5f5d70..701794f 100644
--- a/src/osmo_gsm_tester/obj/ms_srs.py
+++ b/src/osmo_gsm_tester/obj/ms_srs.py
@@ -285,6 +285,17 @@
 elif self.enb.num_ports() == 2:
 # MIMO
 rf_dev_args += 
',rx_freq0=2630e6,rx_freq1=2630e6,rx_freq2=2650e6,rx_freq3=2650e6,tx_freq0=2510e6,tx_freq1=2510e6,tx_freq2=2530e6,tx_freq3=2530e6'
+elif self.num_carriers == 4:
+# 4x CA
+if self.enb.num_ports() == 1:
+# SISO
+rf_dev_args += 
',rx_freq0=2630e6,rx_freq1=2650e6,rx_freq2=2670e6,rx_freq3=2680e6,tx_freq0=2510e6,tx_freq1=2530e6,tx_freq2=2550e6,tx_freq3=2560e6'
+elif self.enb.num_ports() == 2:
+# MIMO
+raise log.Error("4 carriers with MIMO isn't supported")
+else:
+# flag
+raise log.Error('No rx/tx frequencies given for {} carriers' % 
self.num_carriers)

 rf_dev_args += ',id=ue,base_srate='+ str(base_srate)
 config.overlay(values, dict(ue=dict(rf_dev_args=rf_dev_args)))

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20603
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I88a389f5b03d6b55ff5ef403d5107c648a11686f
Gerrit-Change-Number: 20603
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: srsenb_rr.conf.example: comment CQI and SR specific subframes

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20599 )

Change subject: srsenb_rr.conf.example: comment CQI and SR specific subframes
..

srsenb_rr.conf.example: comment CQI and SR specific subframes

this enables auto-selection by eNB which has been implemented
recently

Change-Id: I17e16c5c47a2fba5c2955a1c3e6e0e16f83bab72
---
M src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl 
b/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
index cc1b894..6097a66 100644
--- a/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
+++ b/src/osmo_gsm_tester/templates/srsenb_rr.conf.tmpl
@@ -36,7 +36,7 @@
   {
 dsr_trans_max = 64;
 period = 20;   // in ms
-subframe = [1, 11, 21, 31]; // vector of subframe indices allowed for SR 
transmissions
+//subframe = [1, 11]; // vector of subframe indices allowed for SR 
transmissions

 // number of PRBs on each extreme used for SR (total prb is twice this 
number)
 nof_prb = 1;
@@ -46,7 +46,7 @@
 mode = "periodic";
 simultaneousAckCQI = true;
 period = 40; // in ms
-subframe = [0, 10, 20, 30];
+//subframe = [0, 10, 20, 30];
 nof_prb = 1;
 m_ri = 8; // RI period in CQI period
   };

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20599
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I17e16c5c47a2fba5c2955a1c3e6e0e16f83bab72
Gerrit-Change-Number: 20599
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: enb_amarisoft: temporary reduce max UL rate on 2nd CC

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20598 )

Change subject: enb_amarisoft: temporary reduce max UL rate on 2nd CC
..

enb_amarisoft: temporary reduce max UL rate on 2nd CC

this is due to a confirmed bug in Amarisoft eNB. will
be reverted as soon as the fix is released.

Change-Id: I182b46cf9e6c2f428c9535edf3c7df3daa912f18
---
M src/osmo_gsm_tester/obj/enb_amarisoft.py
1 file changed, 5 insertions(+), 0 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/enb_amarisoft.py 
b/src/osmo_gsm_tester/obj/enb_amarisoft.py
index f2d328d..5b8e05e 100644
--- a/src/osmo_gsm_tester/obj/enb_amarisoft.py
+++ b/src/osmo_gsm_tester/obj/enb_amarisoft.py
@@ -281,6 +281,11 @@
 # Assume we schedule all carriers
 max_rate *= num_carriers

+# Reduce expected UL rate due to bug in UCI scheduling in Amarisoft eNB
+if downlink == False and num_carriers == 2:
+# 2nd carrier @ 25%
+max_rate = max_rate / 2 + (.25 * max_rate / 2)
+
 return max_rate

 # vim: expandtab tabstop=4 shiftwidth=4

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20598
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I182b46cf9e6c2f428c9535edf3c7df3daa912f18
Gerrit-Change-Number: 20598
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: srsenb_rr.conf.tmpl: set TAC for cells to 0x0007

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20602 )

Change subject: srsenb_rr.conf.tmpl: set TAC for cells to 0x0007
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20602
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I94fdb96631ce3ae35a1b2a9f90d1101df9161233
Gerrit-Change-Number: 20602
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:24:21 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: reduce max expected UL rate for 4xCA 100 PRB

2020-10-13 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20600 )

Change subject: reduce max expected UL rate for 4xCA 100 PRB
..

reduce max expected UL rate for 4xCA 100 PRB

this needs to be reverte once extendedBSRs are supported

Change-Id: I8432c744868afe43e2b150f6fcecd8cb3a0458a3
---
M src/osmo_gsm_tester/obj/enb_srs.py
1 file changed, 5 insertions(+), 0 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/obj/enb_srs.py 
b/src/osmo_gsm_tester/obj/enb_srs.py
index 68a3a19..13d18f3 100644
--- a/src/osmo_gsm_tester/obj/enb_srs.py
+++ b/src/osmo_gsm_tester/obj/enb_srs.py
@@ -286,6 +286,11 @@
 # Assume we schedule all carriers
 max_rate *= num_carriers

+# Reduce expected UL rate due to missing extendedBSR support (see 
issue #1708)
+if downlink == False and num_carriers == 4 and self.num_prb() == 100:
+# all carriers run at 70% approx.
+max_rate *= 0.7
+
 return max_rate

 # vim: expandtab tabstop=4 shiftwidth=4

--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20600
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I8432c744868afe43e2b150f6fcecd8cb3a0458a3
Gerrit-Change-Number: 20600
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-gsm-tester[master]: srslte_common: fix indent in whole file

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20611 )


Change subject: srslte_common: fix indent in whole file
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20611
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I8aee79e9d451761dde8f35f84d75483dc4e23040
Gerrit-Change-Number: 20611
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: ms_srs: add freq_offset as resource param

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20612 )

Change subject: ms_srs: add freq_offset as resource param
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20612
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Iab726edd27e890d2c2dddbea91afe80d8a5068aa
Gerrit-Change-Number: 20612
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:25:11 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: enb_amarisoft: add rf_dev_sync option to specify the sync type

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20614 )


Change subject: enb_amarisoft: add rf_dev_sync option to specify the sync type
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20614
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ie066993440840cbd2c18e8f0c0c577b92ac7092d
Gerrit-Change-Number: 20614
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: amarisoft_enb.cfg.tmpl: seperate PRACH root sequences for cells

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20609 )

Change subject: amarisoft_enb.cfg.tmpl: seperate PRACH root sequences for cells
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20609
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5aea1a27939c880944779c282e902465ab513715
Gerrit-Change-Number: 20609
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:24:58 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: amarisoft_enb.cfg.tmpl: use default tac 0007

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20605 )


Change subject: amarisoft_enb.cfg.tmpl: use default tac 0007
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20605
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I1de432a52281ea210abae5648239fd89ba5eb4ce
Gerrit-Change-Number: 20605
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: enb_amarisoft: reduce max achievable UL rate for 100 PRB

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20608 )

Change subject: enb_amarisoft: reduce max achievable UL rate for 100 PRB
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20608
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ib7a0b8e2948cdf6c0a15d4ad2bf7d218a6b28ea2
Gerrit-Change-Number: 20608
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:24:52 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: amarisoft_enb.cfg.tmpl: seperate PRACH root sequences for cells

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20609 )


Change subject: amarisoft_enb.cfg.tmpl: seperate PRACH root sequences for cells
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20609
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5aea1a27939c880944779c282e902465ab513715
Gerrit-Change-Number: 20609
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: jenkins-build-amarisoft: fix build script for ZMQ module

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20613 )


Change subject: jenkins-build-amarisoft: fix build script for ZMQ module
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20613
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ib5302c9872f4c82a1ee5074f59de86709aa6c5a9
Gerrit-Change-Number: 20613
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: enb_amarisoft: reduce max achievable UL rate for 100 PRB

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20608 )


Change subject: enb_amarisoft: reduce max achievable UL rate for 100 PRB
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20608
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ib7a0b8e2948cdf6c0a15d4ad2bf7d218a6b28ea2
Gerrit-Change-Number: 20608
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: srslte_common: fix indent in whole file

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20611 )

Change subject: srslte_common: fix indent in whole file
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20611
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I8aee79e9d451761dde8f35f84d75483dc4e23040
Gerrit-Change-Number: 20611
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:25:06 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: srslte_common: add get_num_phy_errors() method

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20610 )


Change subject: srslte_common: add get_num_phy_errors() method
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20610
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I0c8de87e7f83f986e288585640ce56847959c497
Gerrit-Change-Number: 20610
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: srslte_common: add get_num_phy_errors() method

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20610 )

Change subject: srslte_common: add get_num_phy_errors() method
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20610
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I0c8de87e7f83f986e288585640ce56847959c497
Gerrit-Change-Number: 20610
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:25:02 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: jenkins-build-amarisoft: fix build script for ZMQ module

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20613 )

Change subject: jenkins-build-amarisoft: fix build script for ZMQ module
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20613
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ib5302c9872f4c82a1ee5074f59de86709aa6c5a9
Gerrit-Change-Number: 20613
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:25:15 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: ms_srs: fix ZMQ config for 4xCA

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20603 )

Change subject: ms_srs: fix ZMQ config for 4xCA
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20603
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I88a389f5b03d6b55ff5ef403d5107c648a11686f
Gerrit-Change-Number: 20603
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:24:26 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: lte: update UHD device args for MIMO usage

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20606 )


Change subject: lte: update UHD device args for MIMO usage
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20606
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I3785bb96eb7c10a6b10b3d1a86fa71d1f787a7a3
Gerrit-Change-Number: 20606
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: amarisoft_sib23.asn.tmpl: set p-b value in SIB23 depending on TM

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20604 )

Change subject: amarisoft_sib23.asn.tmpl: set p-b value in SIB23 depending on TM
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20604
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I89c557c456f3330f491cc66bc4931a0234bf6e02
Gerrit-Change-Number: 20604
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:24:33 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: ms_srs: fix ZMQ config for 4xCA

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20603 )


Change subject: ms_srs: fix ZMQ config for 4xCA
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20603
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I88a389f5b03d6b55ff5ef403d5107c648a11686f
Gerrit-Change-Number: 20603
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: amarisoft_enb.cfg.tmpl: increas threshold for failed UL retransmissions

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20607 )


Change subject: amarisoft_enb.cfg.tmpl: increas threshold for failed UL 
retransmissions
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20607
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ib82616e920eecc70fbded12433e54bd90ae1bcd4
Gerrit-Change-Number: 20607
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: ms_srs: add freq_offset as resource param

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20612 )


Change subject: ms_srs: add freq_offset as resource param
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20612
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Iab726edd27e890d2c2dddbea91afe80d8a5068aa
Gerrit-Change-Number: 20612
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: amarisoft_enb.cfg.tmpl: use default tac 0007

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20605 )

Change subject: amarisoft_enb.cfg.tmpl: use default tac 0007
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20605
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I1de432a52281ea210abae5648239fd89ba5eb4ce
Gerrit-Change-Number: 20605
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:24:38 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: srsenb_rr.conf.tmpl: set TAC for cells to 0x0007

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20602 )


Change subject: srsenb_rr.conf.tmpl: set TAC for cells to 0x0007
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20602
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I94fdb96631ce3ae35a1b2a9f90d1101df9161233
Gerrit-Change-Number: 20602
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: amarisoft_sib23.asn.tmpl: set p-b value in SIB23 depending on TM

2020-10-13 Thread pespin
pespin has removed a vote from this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20604 )


Change subject: amarisoft_sib23.asn.tmpl: set p-b value in SIB23 depending on TM
..


Removed Verified-1 by Jenkins Builder (102)
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20604
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I89c557c456f3330f491cc66bc4931a0234bf6e02
Gerrit-Change-Number: 20604
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: deleteVote


Change in osmo-gsm-tester[master]: amarisoft_enb.cfg.tmpl: increas threshold for failed UL retransmissions

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20607 )

Change subject: amarisoft_enb.cfg.tmpl: increas threshold for failed UL 
retransmissions
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20607
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: Ib82616e920eecc70fbded12433e54bd90ae1bcd4
Gerrit-Change-Number: 20607
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:24:48 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: lte: update UHD device args for MIMO usage

2020-10-13 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20606 )

Change subject: lte: update UHD device args for MIMO usage
..


Patch Set 1: Verified+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/20606
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I3785bb96eb7c10a6b10b3d1a86fa71d1f787a7a3
Gerrit-Change-Number: 20606
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Tue, 13 Oct 2020 10:24:43 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


  1   2   3   >