Build failure of network:osmocom:nightly/simtrace2 in Debian_9.0/armv7l

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/Debian_9.0/armv7l

Package network:osmocom:nightly/simtrace2 failed to build in Debian_9.0/armv7l

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

Last lines of build log:
[  242s] [COMPILING libboard/qmod/source/i2c.c]
[  243s] [COMPILING libboard/qmod/source/board_qmod.c]
[  243s] [COMPILING apps/dfu/main.c]
[  243s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  244s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  244s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  244s] Memory region Used Size  Region Size  %age Used
[  244s]  rom:   16584 B16 KB101.22%
[  244s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/5.4.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  244s] /usr/lib/gcc/arm-none-eabi/5.4.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 200 bytes
[  244s] collect2: error: ld returned 1 exit status
[  244s] %
[  244s] Makefile:234: recipe for target 'flash' failed
[  244s] make[2]: *** [flash] Error 1
[  244s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  244s] Makefile:13: recipe for target 'fw-qmod-dfu' failed
[  244s] make[1]: *** [fw-qmod-dfu] Error 2
[  244s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  244s] dh_auto_build: make -j1 returned exit code 2
[  244s] debian/rules:16: recipe for target 'build' failed
[  244s] make: *** [build] Error 2
[  244s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  244s] ### VM INTERACTION START ###
[  247s] [  232.414922] sysrq: SysRq : Power Off
[  247s] [  232.434219] reboot: Power down
[  248s] ### VM INTERACTION END ###
[  248s] 
[  248s] armbuild24 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
02:36:14 UTC 2020.
[  248s] 

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

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

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/Debian_10/armv7l

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

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

Last lines of build log:
[  332s] [COMPILING libboard/qmod/source/wwan_perst.c]
[  333s] [COMPILING libboard/qmod/source/card_pres.c]
[  333s] [COMPILING libboard/qmod/source/wwan_led.c]
[  333s] [COMPILING libboard/qmod/source/i2c.c]
[  334s] [COMPILING libboard/qmod/source/board_qmod.c]
[  334s] [COMPILING apps/dfu/main.c]
[  335s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  335s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  336s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  336s] Memory region Used Size  Region Size  %age Used
[  336s]  rom:   16504 B16 KB100.73%
[  336s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  336s] /usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 120 bytes
[  336s] collect2: error: ld returned 1 exit status
[  336s] %
[  336s] make[2]: *** [Makefile:234: flash] Error 1
[  336s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  336s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  336s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  336s] dh_auto_build: make -j1 returned exit code 2
[  336s] make: *** [debian/rules:16: build] Error 2
[  336s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  336s] ### VM INTERACTION START ###
[  339s] [  309.178639] sysrq: SysRq : Power Off
[  339s] [  309.214252] reboot: Power down
[  340s] ### VM INTERACTION END ###
[  340s] 
[  340s] armbuild02 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
02:36:31 UTC 2020.
[  340s] 

-- 
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

[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:
[  372s] [COMPILING libboard/qmod/source/wwan_perst.c]
[  372s] [COMPILING libboard/qmod/source/card_pres.c]
[  372s] [COMPILING libboard/qmod/source/wwan_led.c]
[  373s] [COMPILING libboard/qmod/source/i2c.c]
[  373s] [COMPILING libboard/qmod/source/board_qmod.c]
[  374s] [COMPILING apps/dfu/main.c]
[  374s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  375s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  375s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  376s] Memory region Used Size  Region Size  %age Used
[  376s]  rom:   16504 B16 KB100.73%
[  376s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  376s] /usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 120 bytes
[  376s] collect2: error: ld returned 1 exit status
[  376s] %
[  376s] make[2]: *** [Makefile:234: flash] Error 1
[  376s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  376s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  376s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  376s] dh_auto_build: make -j1 returned exit code 2
[  376s] make: *** [debian/rules:16: build] Error 2
[  376s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  376s] ### VM INTERACTION START ###
[  379s] [  359.869115] sysrq: SysRq : Power Off
[  379s] [  359.879710] reboot: Power down
[  381s] ### VM INTERACTION END ###
[  381s] 
[  381s] obs-arm-5 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
02:15:09 UTC 2020.
[  381s] 

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

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

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/Debian_9.0/aarch64

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

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

Last lines of build log:
[  103s] [COMPILING libboard/qmod/source/i2c.c]
[  103s] [COMPILING libboard/qmod/source/board_qmod.c]
[  104s] [COMPILING apps/dfu/main.c]
[  104s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  104s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  104s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  104s] Memory region Used Size  Region Size  %age Used
[  104s]  rom:   16584 B16 KB101.22%
[  104s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/5.4.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  104s] /usr/lib/gcc/arm-none-eabi/5.4.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 200 bytes
[  104s] collect2: error: ld returned 1 exit status
[  104s] %
[  104s] Makefile:234: recipe for target 'flash' failed
[  104s] make[2]: *** [flash] Error 1
[  104s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  104s] Makefile:13: recipe for target 'fw-qmod-dfu' failed
[  104s] make[1]: *** [fw-qmod-dfu] Error 2
[  104s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  104s] dh_auto_build: make -j1 returned exit code 2
[  104s] debian/rules:16: recipe for target 'build' failed
[  104s] make: *** [build] Error 2
[  104s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  104s] ### VM INTERACTION START ###
[  107s] [   98.038047] sysrq: Power Off
[  107s] [   98.039281] reboot: Power down
[  108s] ### VM INTERACTION END ###
[  108s] 
[  108s] obs-arm-8 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:41:44 UTC 2020.
[  108s] 

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

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

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/Debian_10/aarch64

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

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

Last lines of build log:
[  111s] [COMPILING libboard/qmod/source/wwan_perst.c]
[  111s] [COMPILING libboard/qmod/source/card_pres.c]
[  111s] [COMPILING libboard/qmod/source/wwan_led.c]
[  111s] [COMPILING libboard/qmod/source/i2c.c]
[  111s] [COMPILING libboard/qmod/source/board_qmod.c]
[  112s] [COMPILING apps/dfu/main.c]
[  112s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  112s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  112s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  112s] Memory region Used Size  Region Size  %age Used
[  112s]  rom:   16504 B16 KB100.73%
[  112s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  112s] /usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 120 bytes
[  112s] collect2: error: ld returned 1 exit status
[  112s] %
[  112s] make[2]: *** [Makefile:234: flash] Error 1
[  112s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  112s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  112s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  112s] dh_auto_build: make -j1 returned exit code 2
[  112s] make: *** [debian/rules:16: build] Error 2
[  112s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  113s] ### VM INTERACTION START ###
[  116s] [  106.607101] sysrq: Power Off
[  116s] [  106.608065] reboot: Power down
[  116s] ### VM INTERACTION END ###
[  116s] 
[  116s] obs-arm-9 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:40:50 UTC 2020.
[  116s] 

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

Build failure of network:osmocom:nightly/simtrace2 in Debian_Unstable/x86_64

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/Debian_Unstable/x86_64

Package network:osmocom:nightly/simtrace2 failed to build in 
Debian_Unstable/x86_64

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

Last lines of build log:
[  153s] [COMPILING apps/dfu/main.c]
[  153s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  154s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  154s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  154s] Memory region Used Size  Region Size  %age Used
[  154s]  rom:   16464 B16 KB100.49%
[  154s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: warning: 
changing start of section .stack by 4 bytes
[  154s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  154s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  154s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  154s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  154s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  154s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 80 bytes
[  154s] collect2: error: ld returned 1 exit status
[  154s] %
[  154s] make[2]: *** [Makefile:234: flash] Error 1
[  154s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  154s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  154s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  154s] dh_auto_build: error: make -j1 returned exit code 2
[  154s] make: *** [debian/rules:16: build] Error 25
[  154s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  154s] ### VM INTERACTION START ###
[  157s] [  146.796647] sysrq: Power Off
[  157s] [  146.804046] reboot: Power down
[  157s] ### VM INTERACTION END ###
[  157s] 
[  157s] lamb62 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:40:24 UTC 2020.
[  157s] 

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

Build failure of network:osmocom:nightly/simtrace2 in Debian_Testing/x86_64

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/Debian_Testing/x86_64

Package network:osmocom:nightly/simtrace2 failed to build in 
Debian_Testing/x86_64

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

Last lines of build log:
[  155s] [COMPILING apps/dfu/main.c]
[  155s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  155s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  155s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  155s] Memory region Used Size  Region Size  %age Used
[  155s]  rom:   16464 B16 KB100.49%
[  155s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: warning: 
changing start of section .stack by 4 bytes
[  155s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  155s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  155s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  155s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  155s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  155s] /usr/lib/gcc/arm-none-eabi/8.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 80 bytes
[  155s] collect2: error: ld returned 1 exit status
[  155s] %
[  155s] make[2]: *** [Makefile:234: flash] Error 1
[  155s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  155s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  155s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  155s] dh_auto_build: error: make -j1 returned exit code 2
[  155s] make: *** [debian/rules:16: build] Error 25
[  155s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  155s] ### VM INTERACTION START ###
[  158s] [  139.870785] sysrq: Power Off
[  158s] [  139.874680] reboot: Power down
[  158s] ### VM INTERACTION END ###
[  158s] 
[  158s] cloud120 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:37:59 UTC 2020.
[  158s] 

-- 
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_19.04/x86_64

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/xUbuntu_19.04/x86_64

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

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

Last lines of build log:
[  201s] [COMPILING libboard/qmod/source/wwan_perst.c]
[  201s] [COMPILING libboard/qmod/source/card_pres.c]
[  201s] [COMPILING libboard/qmod/source/wwan_led.c]
[  201s] [COMPILING libboard/qmod/source/i2c.c]
[  202s] [COMPILING libboard/qmod/source/board_qmod.c]
[  202s] [COMPILING apps/dfu/main.c]
[  202s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  202s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  202s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  202s] Memory region Used Size  Region Size  %age Used
[  202s]  rom:   16504 B16 KB100.73%
[  202s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  202s] /usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 120 bytes
[  202s] collect2: error: ld returned 1 exit status
[  202s] %
[  202s] make[2]: *** [Makefile:234: flash] Error 1
[  202s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  202s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  202s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  202s] dh_auto_build: make -j1 returned exit code 2
[  202s] make: *** [debian/rules:16: build] Error 2
[  202s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  203s] ### VM INTERACTION START ###
[  206s] [  197.336157] sysrq: Power Off
[  206s] [  197.342008] reboot: Power down
[  206s] ### VM INTERACTION END ###
[  206s] 
[  206s] lamb02 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:37:46 UTC 2020.
[  206s] 

-- 
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_18.10/i586

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/xUbuntu_18.10/i586

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

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

Last lines of build log:
[  138s] [COMPILING libboard/qmod/source/wwan_perst.c]
[  138s] [COMPILING libboard/qmod/source/card_pres.c]
[  138s] [COMPILING libboard/qmod/source/wwan_led.c]
[  138s] [COMPILING libboard/qmod/source/i2c.c]
[  138s] [COMPILING libboard/qmod/source/board_qmod.c]
[  139s] [COMPILING apps/dfu/main.c]
[  139s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  139s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  139s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  139s] Memory region Used Size  Region Size  %age Used
[  139s]  rom:   16576 B16 KB101.17%
[  139s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/6.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  139s] /usr/lib/gcc/arm-none-eabi/6.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 192 bytes
[  139s] collect2: error: ld returned 1 exit status
[  139s] %
[  139s] make[2]: *** [Makefile:234: flash] Error 1
[  139s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  139s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  139s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  139s] dh_auto_build: make -j1 returned exit code 2
[  139s] make: *** [debian/rules:16: build] Error 2
[  139s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  139s] ### VM INTERACTION START ###
[  142s] [  130.861304] sysrq: Power Off
[  142s] [  130.866889] reboot: Power down
[  142s] ### VM INTERACTION END ###
[  142s] 
[  142s] cloud129 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:36:48 UTC 2020.
[  142s] 

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

Build failure of network:osmocom:nightly/simtrace2 in Debian_10/x86_64

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/Debian_10/x86_64

Package network:osmocom:nightly/simtrace2 failed to build in Debian_10/x86_64

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

Last lines of build log:
[   73s] [COMPILING libboard/qmod/source/wwan_perst.c]
[   73s] [COMPILING libboard/qmod/source/card_pres.c]
[   73s] [COMPILING libboard/qmod/source/wwan_led.c]
[   73s] [COMPILING libboard/qmod/source/i2c.c]
[   73s] [COMPILING libboard/qmod/source/board_qmod.c]
[   73s] [COMPILING apps/dfu/main.c]
[   73s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[   73s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[   73s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[   73s] Memory region Used Size  Region Size  %age Used
[   73s]  rom:   16504 B16 KB100.73%
[   73s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[   73s] /usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 120 bytes
[   73s] collect2: error: ld returned 1 exit status
[   73s] %
[   73s] make[2]: *** [Makefile:234: flash] Error 1
[   73s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[   73s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[   73s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[   73s] dh_auto_build: make -j1 returned exit code 2
[   73s] make: *** [debian/rules:16: build] Error 2
[   73s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[   73s] ### VM INTERACTION START ###
[   76s] [   71.612733] sysrq: Power Off
[   76s] [   71.618929] reboot: Power down
[   77s] ### VM INTERACTION END ###
[   77s] 
[   77s] goat19 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:36:03 UTC 2020.
[   77s] 

-- 
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

[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:
[  124s] [COMPILING libboard/common/source/uart_console.c]
[  124s] [COMPILING libboard/common/source/led.c]
[  124s] [COMPILING libboard/common/source/boardver_adc.c]
[  124s] [COMPILING libboard/common/source/manifest.c]
[  124s] [COMPILING libboard/simtrace/source/board_simtrace.c]
[  124s] [COMPILING apps/dfu/main.c]
[  124s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  124s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  124s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  125s] Memory region Used Size  Region Size  %age Used
[  125s]  rom:   16460 B16 KB100.46%
[  125s]  ram:   11664 B48 KB 
23.73/usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
bin/simtrace-dfu-flash.elf section `.text' will not fit in region `rom'
[  125s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 76 bytes
[  125s] collect2: error: ld returned 1 exit status
[  125s] %
[  125s] make[2]: *** [Makefile:234: flash] Error 1
[  125s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  125s] make[1]: *** [Makefile:10: fw-simtrace-dfu] Error 2
[  125s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  125s] dh_auto_build: error: make -j1 returned exit code 2
[  125s] make: *** [debian/rules:16: build] Error 25
[  125s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  125s] ### VM INTERACTION START ###
[  128s] [  119.157359] sysrq: Power Off
[  128s] [  119.163108] reboot: Power down
[  128s] ### VM INTERACTION END ###
[  128s] 
[  128s] lamb09 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:36:52 UTC 2020.
[  128s] 

-- 
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_19.10/x86_64

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/xUbuntu_19.10/x86_64

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

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

Last lines of build log:
[  120s] [COMPILING libboard/qmod/source/wwan_perst.c]
[  120s] [COMPILING libboard/qmod/source/card_pres.c]
[  120s] [COMPILING libboard/qmod/source/wwan_led.c]
[  120s] [COMPILING libboard/qmod/source/i2c.c]
[  120s] [COMPILING libboard/qmod/source/board_qmod.c]
[  120s] [COMPILING apps/dfu/main.c]
[  120s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  120s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  121s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  121s] Memory region Used Size  Region Size  %age Used
[  121s]  rom:   16504 B16 KB100.73%
[  121s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  121s] /usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 120 bytes
[  121s] collect2: error: ld returned 1 exit status
[  121s] %
[  121s] make[2]: *** [Makefile:234: flash] Error 1
[  121s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  121s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  121s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  121s] dh_auto_build: make -j1 returned exit code 2
[  121s] make: *** [debian/rules:16: build] Error 255
[  121s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  121s] ### VM INTERACTION START ###
[  124s] [  116.346399] sysrq: Power Off
[  124s] [  116.349047] reboot: Power down
[  124s] ### VM INTERACTION END ###
[  124s] 
[  124s] old-atreju3 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:36:13 UTC 2020.
[  124s] 

-- 
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_18.04/i586

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/xUbuntu_18.04/i586

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

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

Last lines of build log:
[  126s] [COMPILING libboard/qmod/source/i2c.c]
[  126s] [COMPILING libboard/qmod/source/board_qmod.c]
[  126s] [COMPILING apps/dfu/main.c]
[  126s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  127s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  127s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  127s] Memory region Used Size  Region Size  %age Used
[  127s]  rom:   16576 B16 KB101.17%
[  127s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/6.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  127s] /usr/lib/gcc/arm-none-eabi/6.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 192 bytes
[  127s] collect2: error: ld returned 1 exit status
[  127s] %
[  127s] Makefile:234: recipe for target 'flash' failed
[  127s] make[2]: *** [flash] Error 1
[  127s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  127s] Makefile:13: recipe for target 'fw-qmod-dfu' failed
[  127s] make[1]: *** [fw-qmod-dfu] Error 2
[  127s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  127s] dh_auto_build: make -j1 returned exit code 2
[  127s] debian/rules:16: recipe for target 'build' failed
[  127s] make: *** [build] Error 2
[  127s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  127s] ### VM INTERACTION START ###
[  130s] [  122.893656] sysrq: Power Off
[  130s] [  122.897713] reboot: Power down
[  130s] ### VM INTERACTION END ###
[  130s] 
[  130s] lamb05 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:36:31 UTC 2020.
[  130s] 

-- 
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_18.04/x86_64

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/xUbuntu_18.04/x86_64

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

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

Last lines of build log:
[  103s] [COMPILING libboard/qmod/source/i2c.c]
[  103s] [COMPILING libboard/qmod/source/board_qmod.c]
[  104s] [COMPILING apps/dfu/main.c]
[  104s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  104s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  104s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  104s] Memory region Used Size  Region Size  %age Used
[  104s]  rom:   16576 B16 KB101.17%
[  104s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/6.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  104s] /usr/lib/gcc/arm-none-eabi/6.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 192 bytes
[  104s] collect2: error: ld returned 1 exit status
[  104s] %
[  104s] Makefile:234: recipe for target 'flash' failed
[  104s] make[2]: *** [flash] Error 1
[  104s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  104s] Makefile:13: recipe for target 'fw-qmod-dfu' failed
[  104s] make[1]: *** [fw-qmod-dfu] Error 2
[  104s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  104s] dh_auto_build: make -j1 returned exit code 2
[  104s] debian/rules:16: recipe for target 'build' failed
[  104s] make: *** [build] Error 2
[  104s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  104s] ### VM INTERACTION START ###
[  107s] [  100.395015] sysrq: Power Off
[  107s] [  100.398664] reboot: Power down
[  107s] ### VM INTERACTION END ###
[  107s] 
[  107s] old-atreju5 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:35:47 UTC 2020.
[  107s] 

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

Build failure of network:osmocom:nightly/simtrace2 in Debian_9.0/i586

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/Debian_9.0/i586

Package network:osmocom:nightly/simtrace2 failed to build in Debian_9.0/i586

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

Last lines of build log:
[   98s] [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]
[   99s] Memory region Used Size  Region Size  %age Used
[   99s]  rom:   16584 B16 KB101.22%
[   99s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/5.4.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[   99s] /usr/lib/gcc/arm-none-eabi/5.4.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 200 bytes
[   99s] collect2: error: ld returned 1 exit status
[   99s] %
[   99s] Makefile:234: recipe for target 'flash' failed
[   99s] make[2]: *** [flash] Error 1
[   99s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[   99s] Makefile:13: recipe for target 'fw-qmod-dfu' failed
[   99s] make[1]: *** [fw-qmod-dfu] Error 2
[   99s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[   99s] dh_auto_build: make -j1 returned exit code 2
[   99s] debian/rules:16: recipe for target 'build' failed
[   99s] make: *** [build] Error 2
[   99s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[   99s] ### VM INTERACTION START ###
[  102s] [   95.639057] sysrq: Power Off
[  102s] [   95.645485] reboot: Power down
[  102s] ### VM INTERACTION END ###
[  102s] 
[  102s] lamb62 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:35:42 UTC 2020.
[  102s] 

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

Build failure of network:osmocom:nightly/simtrace2 in Debian_9.0/x86_64

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/Debian_9.0/x86_64

Package network:osmocom:nightly/simtrace2 failed to build in Debian_9.0/x86_64

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

Last lines of build log:
[   54s] [COMPILING libboard/qmod/source/i2c.c]
[   54s] [COMPILING libboard/qmod/source/board_qmod.c]
[   54s] [COMPILING apps/dfu/main.c]
[   54s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[   54s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[   54s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[   54s] Memory region Used Size  Region Size  %age Used
[   54s]  rom:   16584 B16 KB101.22%
[   54s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/5.4.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[   54s] /usr/lib/gcc/arm-none-eabi/5.4.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 200 bytes
[   54s] collect2: error: ld returned 1 exit status
[   54s] %
[   54s] Makefile:234: recipe for target 'flash' failed
[   54s] make[2]: *** [flash] Error 1
[   54s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[   54s] Makefile:13: recipe for target 'fw-qmod-dfu' failed
[   54s] make[1]: *** [fw-qmod-dfu] Error 2
[   54s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[   54s] dh_auto_build: make -j1 returned exit code 2
[   54s] debian/rules:16: recipe for target 'build' failed
[   54s] make: *** [build] Error 2
[   54s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[   54s] ### VM INTERACTION START ###
[   57s] [   53.508931] sysrq: Power Off
[   57s] [   53.516140] reboot: Power down
[   57s] ### VM INTERACTION END ###
[   57s] 
[   57s] goat11 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:36:02 UTC 2020.
[   57s] 

-- 
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_18.10/x86_64

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/xUbuntu_18.10/x86_64

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

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

Last lines of build log:
[   70s] [COMPILING libboard/qmod/source/wwan_perst.c]
[   70s] [COMPILING libboard/qmod/source/card_pres.c]
[   70s] [COMPILING libboard/qmod/source/wwan_led.c]
[   70s] [COMPILING libboard/qmod/source/i2c.c]
[   70s] [COMPILING libboard/qmod/source/board_qmod.c]
[   70s] [COMPILING apps/dfu/main.c]
[   71s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[   71s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[   71s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[   71s] Memory region Used Size  Region Size  %age Used
[   71s]  rom:   16576 B16 KB101.17%
[   71s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/6.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[   71s] /usr/lib/gcc/arm-none-eabi/6.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 192 bytes
[   71s] collect2: error: ld returned 1 exit status
[   71s] %
[   71s] make[2]: *** [Makefile:234: flash] Error 1
[   71s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[   71s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[   71s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[   71s] dh_auto_build: make -j1 returned exit code 2
[   71s] make: *** [debian/rules:16: build] Error 2
[   71s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[   71s] ### VM INTERACTION START ###
[   74s] [   69.301460] sysrq: Power Off
[   74s] [   69.307702] reboot: Power down
[   74s] ### VM INTERACTION END ###
[   74s] 
[   74s] goat09 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:34:53 UTC 2020.
[   74s] 

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

Build failure of network:osmocom:nightly/simtrace2 in Debian_10/i586

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/simtrace2/Debian_10/i586

Package network:osmocom:nightly/simtrace2 failed to build in Debian_10/i586

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

Last lines of build log:
[  104s] [COMPILING libboard/qmod/source/wwan_perst.c]
[  104s] [COMPILING libboard/qmod/source/card_pres.c]
[  104s] [COMPILING libboard/qmod/source/wwan_led.c]
[  104s] [COMPILING libboard/qmod/source/i2c.c]
[  104s] [COMPILING libboard/qmod/source/board_qmod.c]
[  104s] [COMPILING apps/dfu/main.c]
[  105s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  105s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  105s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  105s] Memory region Used Size  Region Size  %age Used
[  105s]  rom:   16504 B16 KB100.73%
[  105s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  105s] /usr/lib/gcc/arm-none-eabi/7.3.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 120 bytes
[  105s] collect2: error: ld returned 1 exit status
[  105s] %
[  105s] make[2]: *** [Makefile:234: flash] Error 1
[  105s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  105s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  105s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  105s] dh_auto_build: make -j1 returned exit code 2
[  105s] make: *** [debian/rules:16: build] Error 2
[  105s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  105s] ### VM INTERACTION START ###
[  108s] [  101.233992] sysrq: Power Off
[  108s] [  101.238367] reboot: Power down
[  108s] ### VM INTERACTION END ###
[  108s] 
[  108s] sheep88 failed "build simtrace2_0.7.0.61.1afb7.dsc" at Tue Aug 11 
01:33:34 UTC 2020.
[  108s] 

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

Change in osmo-bsc[master]: e1: encode line number as trunk number in MGCP endpoint

[dexter]

dexter has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19564 )


Change subject: e1: encode line number as trunk number in MGCP endpoint
..

e1: encode line number as trunk number in MGCP endpoint

The MGCP endpoint name, that is generated when an E1 endpoint is
selected does have a hardcoded trunk id number, which is permanantly set
to 1. Lets use the E1 line number instead.

Related: OS#2547
Change-Id: Ic5447bb4426e31d119667bdfddfd2c91fd591fc6
---
M src/osmo-bsc/bsc_subscr_conn_fsm.c
1 file changed, 5 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/64/19564/1

diff --git a/src/osmo-bsc/bsc_subscr_conn_fsm.c 
b/src/osmo-bsc/bsc_subscr_conn_fsm.c
index 0faf3b2..ec8a4df 100644
--- a/src/osmo-bsc/bsc_subscr_conn_fsm.c
+++ b/src/osmo-bsc/bsc_subscr_conn_fsm.c
@@ -495,8 +495,11 @@
if (is_ipaccess_bts(for_lchan->ts->trx->bts))
/* use dynamic RTPBRIDGE endpoint allocation in MGW */
epname = 
mgcp_client_rtpbridge_wildcard(conn->network->mgw.client);
-   else
-   epname = mgcp_client_e1_epname(conn, 
conn->network->mgw.client, 1, for_lchan->ts->e1_link.e1_ts, 16, 
for_lchan->ts->e1_link.e1_ts_ss*2);
+   else {
+   epname = mgcp_client_e1_epname(conn, 
conn->network->mgw.client, for_lchan->ts->e1_link.e1_nr,
+  
for_lchan->ts->e1_link.e1_ts, 16,
+  
for_lchan->ts->e1_link.e1_ts_ss*2);
+   }

conn->user_plane.mgw_endpoint =
osmo_mgcpc_ep_alloc(conn->fi, 
GSCON_EV_FORGET_MGW_ENDPOINT,

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ic5447bb4426e31d119667bdfddfd2c91fd591fc6
Gerrit-Change-Number: 19564
Gerrit-PatchSet: 1
Gerrit-Owner: dexter 
Gerrit-MessageType: newchange

Change in osmo-mgw[master]: mgcp_trunk: drop "trunk 0" limitation

[dexter]

dexter has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-mgw/+/19563 )


Change subject: mgcp_trunk: drop "trunk 0" limitation
..

mgcp_trunk: drop "trunk 0" limitation

Due to the internal handling of the trunks it was not possible to allow
an E1 trunk that has the ID 0. However this limitation is no longer
present, so we now can allow an E1 trunk with ID 0.

Change-Id: I302c2007628f607033686e277c407232351e66ad
Related: OS#2659
---
M src/libosmo-mgcp/mgcp_trunk.c
M src/libosmo-mgcp/mgcp_vty.c
2 files changed, 3 insertions(+), 8 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/63/19563/1

diff --git a/src/libosmo-mgcp/mgcp_trunk.c b/src/libosmo-mgcp/mgcp_trunk.c
index 617993f..84ad220 100644
--- a/src/libosmo-mgcp/mgcp_trunk.c
+++ b/src/libosmo-mgcp/mgcp_trunk.c
@@ -190,7 +190,7 @@

errno = 0;
trunk_nr = strtoul(epname + prefix_len, _trunk_nr_end, 10);
-   if (errno == ERANGE || trunk_nr > 64 || trunk_nr == 0
+   if (errno == ERANGE || trunk_nr > 64
|| epname + prefix_len == str_trunk_nr_end
|| str_trunk_nr_end[0] != '/')
return -EINVAL;
@@ -217,7 +217,7 @@
}

trunk_nr = e1_trunk_nr_from_epname(epname);
-   if (trunk_nr > 0)
+   if (trunk_nr >= 0)
return mgcp_trunk_by_num(cfg, MGCP_TRUNK_E1, trunk_nr);

/* Earlier versions of osmo-mgw were accepting endpoint names
diff --git a/src/libosmo-mgcp/mgcp_vty.c b/src/libosmo-mgcp/mgcp_vty.c
index d2d4b23..22863c7 100644
--- a/src/libosmo-mgcp/mgcp_vty.c
+++ b/src/libosmo-mgcp/mgcp_vty.c
@@ -866,16 +866,11 @@
 "IPv4 Address of the callagent\n")

 DEFUN(cfg_mgcp_trunk, cfg_mgcp_trunk_cmd,
-  "trunk <1-64>", "Configure a SS7 trunk\n" "Trunk Nr\n")
+  "trunk <0-64>", "Configure a SS7 trunk\n" "Trunk Nr\n")
 {
struct mgcp_trunk *trunk;
int index = atoi(argv[0]);

-   /* Due to historical reasons, the trunk id number 0 is reserved for the
-* virtual trunk. This trunk is configured with separate VTY
-* parameters, so we restrict the access to trunks with id numbers
-* greater than 0. */
-
trunk = mgcp_trunk_by_num(g_cfg, MGCP_TRUNK_E1, index);
if (!trunk) {
trunk = mgcp_trunk_alloc(g_cfg, MGCP_TRUNK_E1, index);

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

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I302c2007628f607033686e277c407232351e66ad
Gerrit-Change-Number: 19563
Gerrit-PatchSet: 1
Gerrit-Owner: dexter 
Gerrit-MessageType: newchange

Change in osmo-mgw[master]: mgcp_e1: finish E1 support, add E1 support from libosmoabis

[dexter]

Hello Jenkins Builder,

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

https://gerrit.osmocom.org/c/osmo-mgw/+/19524

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

Change subject: mgcp_e1: finish E1 support, add E1 support from libosmoabis
..

mgcp_e1: finish E1 support, add E1 support from libosmoabis

Currently only the endpoint handling for E1 exists, but there is no
actual code behind it that handles the E1 traffic.

Change-Id: I6b93809b5ac7d01af55888347dd787b0bc997ae1
Related: OS#2659
---
M configure.ac
M include/osmocom/mgcp/Makefile.am
M include/osmocom/mgcp/debug.h
M include/osmocom/mgcp/mgcp_conn.h
A include/osmocom/mgcp/mgcp_e1.h
M include/osmocom/mgcp/mgcp_endp.h
M include/osmocom/mgcp/mgcp_internal.h
M include/osmocom/mgcp/mgcp_ratectr.h
M include/osmocom/mgcp/mgcp_trunk.h
M src/libosmo-mgcp/Makefile.am
M src/libosmo-mgcp/mgcp_conn.c
A src/libosmo-mgcp/mgcp_e1.c
M src/libosmo-mgcp/mgcp_endp.c
M src/libosmo-mgcp/mgcp_network.c
M src/libosmo-mgcp/mgcp_protocol.c
M src/libosmo-mgcp/mgcp_ratectr.c
M src/libosmo-mgcp/mgcp_trunk.c
M src/libosmo-mgcp/mgcp_vty.c
M src/osmo-mgw/Makefile.am
M src/osmo-mgw/mgw_main.c
M tests/mgcp/Makefile.am
M tests/mgcp/mgcp_test.c
22 files changed, 1,076 insertions(+), 62 deletions(-)


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

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I6b93809b5ac7d01af55888347dd787b0bc997ae1
Gerrit-Change-Number: 19524
Gerrit-PatchSet: 3
Gerrit-Owner: dexter 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge 
Gerrit-MessageType: newpatchset

Change in libosmocore[master]: add osmo_sockaddr_local_ip() to determine the local address for a rem...

[lynxis lazus]

Hello Jenkins Builder,

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

https://gerrit.osmocom.org/c/libosmocore/+/19416

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

Change subject: add osmo_sockaddr_local_ip() to determine the local address for 
a remote.
..

add osmo_sockaddr_local_ip() to determine the local address for a remote.

Similiar to osmo_sock_local_ip but for osmo_sockaddr.

Change-Id: I9cd2c5ceb28183e2fd2d28f9c9088c3fcac643d2
---
M include/osmocom/core/socket.h
M src/socket.c
2 files changed, 27 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/16/19416/11
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/19416
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I9cd2c5ceb28183e2fd2d28f9c9088c3fcac643d2
Gerrit-Change-Number: 19416
Gerrit-PatchSet: 11
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus 
Gerrit-CC: laforge 
Gerrit-MessageType: newpatchset

Change in libosmocore[master]: add osmo_sockaddr_local_ip() to determine the local address for a rem...

[lynxis lazus]

lynxis lazus has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/19416 )

Change subject: add osmo_sockaddr_local_ip() to determine the local address for 
a remote.
..


Patch Set 10:

(1 comment)

https://gerrit.osmocom.org/c/libosmocore/+/19416/9/src/socket.c
File src/socket.c:

https://gerrit.osmocom.org/c/libosmocore/+/19416/9/src/socket.c@1567
PS9, Line 1567: remote_ip
> ping?
Done



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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I9cd2c5ceb28183e2fd2d28f9c9088c3fcac643d2
Gerrit-Change-Number: 19416
Gerrit-PatchSet: 10
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus 
Gerrit-CC: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 21:32:05 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge 
Gerrit-MessageType: comment

Change in osmo-ttcn3-hacks[master]: bsc: test RR Channel Release cause codes from Clear Command cause codes

[neels]

Hello fixeria, pespin, laforge, Jenkins Builder,

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

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

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

Change subject: bsc: test RR Channel Release cause codes from Clear Command 
cause codes
..

bsc: test RR Channel Release cause codes from Clear Command cause codes

Invoke Clear Command with various Cause codes and verify that the RR Channel
Release reflects them.

Depends: I734cc55c501d61bbdadee81a223b26f9df57f959 (osmo-bsc)
Change-Id: Ie6c99f28b610a67f2d59ec00b3541940e882251b
---
M bsc/BSC_Tests.ttcn
M library/GSM_RR_Types.ttcn
2 files changed, 89 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks 
refs/changes/17/19217/5
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19217
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: Ie6c99f28b610a67f2d59ec00b3541940e882251b
Gerrit-Change-Number: 19217
Gerrit-PatchSet: 5
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: newpatchset

Change in osmo-ttcn3-hacks[master]: bsc: test RR Channel Release EARFCNs

[neels]

Hello laforge, Jenkins Builder,

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

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

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

Change subject: bsc: test RR Channel Release EARFCNs
..

bsc: test RR Channel Release EARFCNs

Add a DCHAN and release to recently added SI2quater tests (because these tests
already configure various amounts of EARFCNs in osmo-bsc).

Verify that the RR Channel Release for CSFB contains all configured EARFCNs.

In GSM_RR_Types.ttcn, add coding for "Cell selection indicator after release of
all TCH and SDCCH IE".

In f_expect_chan_rel(), add optional arg csfb_expect_cells, and, if present,
decode the RR Channel Release message's L3 part, and in turn the Cell Selection
Indicator Value contained. Match against csfb_expect_cells.

In f_tc_si2quater_n_earfcns(), also compose a list of EARFCNs as found in the
RR Channel Release, and pass to f_expect_chan_rel().

Depends: I59e427e4ebb1c6af99b27a15c40fed82457ac8ab (osmo-bsc)
Change-Id: I882c5e1f70bcc4833fc837a95c900ce291919cc5
---
M bsc/BSC_Tests.ttcn
M library/GSM_RR_Types.ttcn
2 files changed, 170 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-ttcn3-hacks 
refs/changes/16/19216/5
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19216
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: I882c5e1f70bcc4833fc837a95c900ce291919cc5
Gerrit-Change-Number: 19216
Gerrit-PatchSet: 5
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-CC: fixeria 
Gerrit-MessageType: newpatchset

Change in osmo-ttcn3-hacks[master]: bsc: test RR Channel Release cause codes from Clear Command cause codes

[neels]

neels has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19217 )

Change subject: bsc: test RR Channel Release cause codes from Clear Command 
cause codes
..


Patch Set 4:

(1 comment)

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19217/3/bsc/BSC_Tests.ttcn
File bsc/BSC_Tests.ttcn:

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19217/3/bsc/BSC_Tests.ttcn@1446
PS3, Line 1446: template RR_Cause expect_rr_cause := ?
> I don't share this concern. […]
(I think adding expected things is the best choice, explained in comment on 
previous patch)



--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19217
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: Ie6c99f28b610a67f2d59ec00b3541940e882251b
Gerrit-Change-Number: 19217
Gerrit-PatchSet: 4
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:34:22 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria 
Comment-In-Reply-To: laforge 
Gerrit-MessageType: comment

Change in osmo-ttcn3-hacks[master]: fix mscpool cleanup (fixes LCLS tests)

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19554 )

Change subject: fix mscpool cleanup (fixes LCLS tests)
..

fix mscpool cleanup (fixes LCLS tests)

osmo-bsc takes a while to notice that a connected MSC is no longer connected.
Once the mscpool tests have run, the additional msc 1 and msc 2 still linger
around even though the BSSMAP link is no longer served by the bsc-tester.

The easiest way to ensure that only expected MSCs are contacted is to set
'no allow-attach' for each MSC that should not be in use.

So, the default setup is 'allow-attach' on msc 0, and 'no allow-attach' on mscs
1 and 2. In f_init(), allow attach on those MSCs indicated by the nr_msc
amount. The entire vty transaction to configure attach/no attach for all three
MSCs takes about 4 micro seconds in my test setup, so it is fine to do this
during f_init() for each BSC test.

After this, tests running after the MSC pooling tests (the LCLS tests) no
longer round-robin their subscribers across disconnected MSCs.

NOTE: it would be good to somehow detect more reliably in osmo-bsc that an MSC
is gone and not use it anymore. That is however not so trivial. To get the LCLS
tests back online, this is a workaround to avoid that complexity for now.

Change-Id: I02ad58ed7d0d0aac61e393b415e09c6c5c8a70ca
---
M bsc/BSC_Tests.ttcn
1 file changed, 28 insertions(+), 22 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  fixeria: Looks good to me, but someone else must approve
  Jenkins Builder: Verified



diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn
index 5bd0e4f..1040a31 100644
--- a/bsc/BSC_Tests.ttcn
+++ b/bsc/BSC_Tests.ttcn
@@ -832,6 +832,24 @@
}
 }

+/* TODO: use BooleanList from COMMON/src/General_Types.ttcn */
+private type record of boolean my_BooleanList;
+
+private function f_vty_msc_allow_attach(TELNETasp_PT pt, my_BooleanList 
allow_attach_list)
+{
+   for (var integer msc_nr := 0; msc_nr < sizeof(allow_attach_list); 
msc_nr := msc_nr+1) {
+   f_vty_enter_cfg_msc(pt, msc_nr);
+   if (allow_attach_list[msc_nr]) {
+   /* strict := false: ignore if osmo-bsc does not support 
this config option (latest build) */
+   f_vty_transceive(pt, "allow-attach", strict := false);
+   } else {
+   f_vty_transceive(pt, "no allow-attach", strict := 
false);
+   }
+   f_vty_transceive(pt, "exit");
+   f_vty_transceive(pt, "exit");
+   }
+}
+
 /* global initialization function
  * \param nr_bts Number of BTSs we should start/bring up
  * \param handler_mode Start an RSL_Emulation_CT component (true) or not 
(false).
@@ -854,7 +872,9 @@
f_vty_allow_osmux(allow_osmux);
}

+   var my_BooleanList allow_attach := { false, false, false };
for (bssap_idx := 0; bssap_idx < nr_msc; bssap_idx := bssap_idx+1) {
+   allow_attach[bssap_idx] := true;
/* Call a function of our 'parent component' RAN_Adapter_CT to 
start the
 * MSC-side BSSAP emulation */
if (handler_mode) {
@@ -871,6 +891,9 @@
}
}
 
+   /* start the test with exactly all enabled MSCs allowed to attach */
+   f_vty_msc_allow_attach(BSCVTY, allow_attach);
+
f_ipa_ctrl_start(mp_bsc_ip, mp_bsc_ctrl_port);

f_init_mgcp("VirtMSC");
@@ -5850,17 +5873,13 @@

f_init(nr_bts := 3, handler_mode := true, nr_msc := 3);
f_sleep(1.0);
+   /* Mark the second MSC as offloading, round-robin should skip this MSC 
now. */
+   f_vty_msc_allow_attach(BSCVTY, {true, false, true});

/* Control which MSC gets chosen next by the round-robin, otherwise
 * would be randomly affected by which other tests ran before this. */
f_vty_transceive(BSCVTY, "mscpool roundrobin next 0");

-   /* Mark the second MSC as offloading, round-robin should skip this MSC 
now. */
-   f_vty_enter_cfg_msc(BSCVTY, 1);
-   f_vty_transceive(BSCVTY, "no allow-attach");
-   f_vty_transceive(BSCVTY, "exit");
-   f_vty_transceive(BSCVTY, "exit");
-
f_ctrs_msc_init();

var MSC_ConnHdlr vc_conn1;
@@ -5886,11 +5905,6 @@
vc_conn3 := f_start_handler(refers(f_tc_mscpool_compl_l3), pars3);
vc_conn3.done;
f_ctrs_msc_expect(0, "mscpool:subscr:new");
-
-   f_vty_enter_cfg_msc(BSCVTY, 1);
-   f_vty_transceive(BSCVTY, "allow-attach");
-   f_vty_transceive(BSCVTY, "exit");
-   f_vty_transceive(BSCVTY, "exit");
 }

 /* An MSC that has 'no allow-attach' set should still serve subscribers that 
are already attached according to their
@@ -5900,16 +5914,13 @@
f_init(nr_bts := 3, handler_mode := true, nr_msc := 3);
f_sleep(1.0);

+   /* Mark the second MSC as offloading, round-robin should skip this MSC 
now. */
+   

Change in osmo-ttcn3-hacks[master]: fix mscpool cleanup (fixes LCLS tests)

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19554 )

Change subject: fix mscpool cleanup (fixes LCLS tests)
..


Patch Set 2: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19554
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: I02ad58ed7d0d0aac61e393b415e09c6c5c8a70ca
Gerrit-Change-Number: 19554
Gerrit-PatchSet: 2
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:15:38 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-ttcn3-hacks[master]: f_vty_transceive: allow to ignore unknown vty commands

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19555 )

Change subject: f_vty_transceive: allow to ignore unknown vty commands
..

f_vty_transceive: allow to ignore unknown vty commands

When a new test uses a VTY configuration that may not yet be available in the
'latest' build, it can be useful to ignore the "Unknown VTY Command" error.

To be used by f_init() for multiple MSCs, setting a default 'allow-attach' flag
per MSC implicitly -- such vty config is not yet supported in the latest build.

Change-Id: I284c42e10c0cb282c8410db87959b471867edef6
---
M library/Osmocom_VTY_Functions.ttcn
1 file changed, 10 insertions(+), 6 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  fixeria: Looks good to me, but someone else must approve
  Jenkins Builder: Verified



diff --git a/library/Osmocom_VTY_Functions.ttcn 
b/library/Osmocom_VTY_Functions.ttcn
index 823d79f..4cf37c4 100644
--- a/library/Osmocom_VTY_Functions.ttcn
+++ b/library/Osmocom_VTY_Functions.ttcn
@@ -56,7 +56,7 @@
}

/* wait for any of the permitted prompts; buffer + return all 
intermediate output */
-   function f_vty_wait_for_prompt(TELNETasp_PT pt) return charstring {
+   function f_vty_wait_for_prompt(TELNETasp_PT pt, boolean strict := true) 
return charstring {
var charstring rx, buf := "";
var integer fd;
timer T := 2.0;
@@ -67,7 +67,11 @@
[] pt.receive(pattern "[\w-]+\# ") { };
[] pt.receive(pattern "[\w-]+\(*\)\# ") { };
[] pt.receive(t_vty_unknown) {
-   testcase.stop(fail, "VTY: Unknown Command");
+   if (strict) {
+   testcase.stop(fail, "VTY: Unknown 
Command");
+   } else {
+   log("VTY: Unknown Command (ignored)");
+   }
};
[] pt.receive(charstring:?) -> value rx { buf := buf & 
rx; repeat };
[] pt.receive(integer:?) -> value fd {
@@ -88,14 +92,14 @@
}

/* send a VTY command and obtain response until prompt is received */
-   function f_vty_transceive_ret(TELNETasp_PT pt, charstring tx) return 
charstring {
+   function f_vty_transceive_ret(TELNETasp_PT pt, charstring tx, boolean 
strict := true) return charstring {
pt.send(tx);
-   return f_vty_wait_for_prompt(pt);
+   return f_vty_wait_for_prompt(pt, strict);
}

/* send a VTY command and obtain response until prompt is received */
-   function f_vty_transceive(TELNETasp_PT pt, charstring tx) {
-   var charstring unused := f_vty_transceive_ret(pt, tx);
+   function f_vty_transceive(TELNETasp_PT pt, charstring tx, boolean 
strict := true) {
+   var charstring unused := f_vty_transceive_ret(pt, tx, strict);
}

type integer BtsNr (0..255);

--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19555
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: I284c42e10c0cb282c8410db87959b471867edef6
Gerrit-Change-Number: 19555
Gerrit-PatchSet: 2
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged

Change in osmo-ttcn3-hacks[master]: f_vty_transceive: allow to ignore unknown vty commands

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19555 )

Change subject: f_vty_transceive: allow to ignore unknown vty commands
..


Patch Set 1: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19555
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: I284c42e10c0cb282c8410db87959b471867edef6
Gerrit-Change-Number: 19555
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:16:03 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in libosmocore[master]: add osmo_sockaddr_local_ip() to determine the local address for a rem...

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/19416 )

Change subject: add osmo_sockaddr_local_ip() to determine the local address for 
a remote.
..


Patch Set 10:

(1 comment)

https://gerrit.osmocom.org/c/libosmocore/+/19416/9/src/socket.c
File src/socket.c:

https://gerrit.osmocom.org/c/libosmocore/+/19416/9/src/socket.c@1567
PS9, Line 1567: remote_ip
> remote should be 'const' if it's an input argument.
ping?



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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I9cd2c5ceb28183e2fd2d28f9c9088c3fcac643d2
Gerrit-Change-Number: 19416
Gerrit-PatchSet: 10
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:14:37 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge 
Gerrit-MessageType: comment

Change in libosmocore[master]: socket: add osmo_sockaddr_cmp()

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/19415 )

Change subject: socket: add osmo_sockaddr_cmp()
..


Patch Set 10: Code-Review+1


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I2d12ebae2710ffd17cf071e6ada0804e73f87dd6
Gerrit-Change-Number: 19415
Gerrit-PatchSet: 10
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:13:51 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in libosmocore[master]: socket: introduce osmo_sock_init3 & osmo_sock_init3_ofd

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/19143 )

Change subject: socket: introduce osmo_sock_init3 & osmo_sock_init3_ofd
..


Patch Set 12: Code-Review+1


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I1eece543e3241ef0e095eb63bb831f7c15a16794
Gerrit-Change-Number: 19143
Gerrit-PatchSet: 12
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:13:25 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in libosmocore[master]: socket.h: introduce osmo_sockaddr to hold v4 and v6 endpoints

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/19141 )

Change subject: socket.h: introduce osmo_sockaddr to hold v4 and v6 endpoints
..


Patch Set 9: Code-Review+2


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: If80172373735193401af872b18e1ff00c93880e7
Gerrit-Change-Number: 19141
Gerrit-PatchSet: 9
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:12:36 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-ccid-firmware[master]: proper card powerup handling

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/19473 )

Change subject: proper card powerup handling
..


Patch Set 5: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/19473
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: I6105ce7f827aaec8ac36b940766fe05d47145edb
Gerrit-Change-Number: 19473
Gerrit-PatchSet: 5
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:11:56 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-ttcn3-hacks[master]: PCU_Tests: verify handling of frequency hopping parameters

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19324 )

Change subject: PCU_Tests: verify handling of frequency hopping parameters
..

PCU_Tests: verify handling of frequency hopping parameters

Change-Id: I1beda9224998848f9243237413e5ae3e0ddab8a5
Related: SYS#4868, OS#4547
---
M pcu/PCU_Tests.ttcn
1 file changed, 271 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved



diff --git a/pcu/PCU_Tests.ttcn b/pcu/PCU_Tests.ttcn
index dab87e0..1c77256 100644
--- a/pcu/PCU_Tests.ttcn
+++ b/pcu/PCU_Tests.ttcn
@@ -2069,6 +2069,266 @@
f_shutdown(__BFILE__, __LINE__, final := true);
 }

+private function f_TC_pcuif_fh_check_imm_ass(in PCUIF_info_ind info_ind,
+in GsmRrMessage rr_msg)
+{
+   var ImmediateAssignment ia := rr_msg.payload.imm_ass;
+   var PCUIF_InfoTrxTs ts := info_ind.trx.v10[0].ts[ia.pkt_chan_desc.tn];
+
+   var template PacketChannelDescription tr_pkt_chan_desc := {
+   channel_Type_spare := ?,
+   tn := ?,
+   tsc := ts.tsc,
+   presence := '1'B,
+   zero := omit,
+   one := {
+   maio := ts.maio,
+   hsn := ts.hsn
+   }
+   };
+
+   if (not match(ia.pkt_chan_desc, tr_pkt_chan_desc)) {
+   setverdict(fail, "Packet Channel Description does not match: ",
+  ia.pkt_chan_desc, " vs ", tr_pkt_chan_desc);
+   }
+
+   var MobileAllocation tr_ma := { len := 0, ma := ''B };
+   var octetstring ma_oct := bit2oct(ts.ma);
+
+   /* Calculate length of the useful part of Mobile Allocation */
+   for (var integer i := 0; i < lengthof(ma_oct); i := i + 1) {
+   if (ma_oct[i] != '00'O) {
+   tr_ma.len := i + 1;
+   }
+   }
+
+   /* Strip zero bytes from the right side of MA */
+   tr_ma.ma := substr(ts.ma, 0, tr_ma.len * 8);
+
+   if (not match(ia.mobile_allocation, tr_ma)) {
+   setverdict(fail, "Mobile Allocation does not match: ",
+  ia.mobile_allocation, " vs ", tr_ma);
+   }
+
+   setverdict(pass);
+}
+
+/* Make sure that Immediate (UL EGPRS TBF) Assignment contains hopping 
parameters */
+testcase TC_pcuif_fh_imm_ass_ul_egprs() runs on RAW_PCU_Test_CT {
+   var template PCUIF_info_ind info_ind := ts_PCUIF_INFO_default;
+   var GprsMS ms := valueof(t_GprsMS_def);
+
+   /* Enable frequency hopping of TRX0/TS7 */
+   info_ind.trx.v10[0].ts[7] := ts_PCUIF_InfoTrxTsH1(
+   tsc := f_rnd_int(7),
+   hsn := f_rnd_int(63),
+   maio := f_rnd_int(63),
+   ma := f_rnd_bitstring(32));
+
+   /* Initialize the PCU interface abstraction */
+   f_init_raw(testcasename(), info_ind);
+
+   /* EGPRS Packet Channel Request (cause=Signalling) */
+   f_ms_use_ra(ms, bit2int('11001101010'B), ra_is_11bit := 1);
+
+   /* Establish an Uplink EGPRS TBF */
+   f_ms_establish_ul_tbf(ms);
+
+   f_TC_pcuif_fh_check_imm_ass(valueof(info_ind), ms.ul_tbf.rr_imm_ass);
+   f_shutdown(__BFILE__, __LINE__, final := true);
+}
+
+/* Make sure that Immediate (UL TBF) Assignment contains hopping parameters */
+testcase TC_pcuif_fh_imm_ass_ul() runs on RAW_PCU_Test_CT {
+   var template PCUIF_info_ind info_ind := ts_PCUIF_INFO_default;
+   var GprsMS ms := valueof(t_GprsMS_def);
+
+   /* Enable frequency hopping of TRX0/TS7 */
+   info_ind.trx.v10[0].ts[7] := ts_PCUIF_InfoTrxTsH1(
+   tsc := f_rnd_int(7),
+   hsn := f_rnd_int(63),
+   maio := f_rnd_int(63),
+   ma := f_rnd_bitstring(32));
+
+   /* Initialize the PCU interface abstraction */
+   f_init_raw(testcasename(), info_ind);
+
+   /* Establish an Uplink TBF */
+   f_ms_establish_ul_tbf(ms);
+
+   f_TC_pcuif_fh_check_imm_ass(valueof(info_ind), ms.ul_tbf.rr_imm_ass);
+   f_shutdown(__BFILE__, __LINE__, final := true);
+}
+
+/* Make sure that Immediate (DL TBF) Assignment contains hopping parameters */
+testcase TC_pcuif_fh_imm_ass_dl() runs on RAW_PCU_Test_CT {
+   var template PCUIF_info_ind info_ind := ts_PCUIF_INFO_default;
+   var GprsMS ms := valueof(t_GprsMS_def);
+
+   /* Enable frequency hopping of TRX0/TS7 */
+   info_ind.trx.v10[0].ts[7] := ts_PCUIF_InfoTrxTsH1(
+   tsc := f_rnd_int(7),
+   hsn := f_rnd_int(63),
+   maio := f_rnd_int(63),
+   ma := f_rnd_bitstring(16));
+
+   /* Initialize NS/BSSGP side */
+   f_init_bssgp();
+
+   /* Initialize the PCU interface abstraction */
+   f_init_raw(testcasename(), info_ind);
+
+   /* Establish BSSGP connection to 

Change in osmo-ttcn3-hacks[master]: BTS_Tests: compose the MA bitmask in f_resolve_fh_params()

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19517 )

Change subject: BTS_Tests: compose the MA bitmask in f_resolve_fh_params()
..

BTS_Tests: compose the MA bitmask in f_resolve_fh_params()

Change-Id: Ic0a07d9deb60712347a42d9aa7813c684eaccd01
Related: SYS#4868, OS#4546
---
M bts/BTS_Tests.ttcn
1 file changed, 10 insertions(+), 2 deletions(-)

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



diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn
index dfda043..8a9e24a 100644
--- a/bts/BTS_Tests.ttcn
+++ b/bts/BTS_Tests.ttcn
@@ -545,13 +545,21 @@
}

if (ispresent(pars.maio_hsn)) {
-   /* Compose the Mobile Allocation */
+   /* Prepare the Mobile Allocation bitmask (length & 
padding) */
+   pars.ma_map.len := mp_transceiver_num / 8;
+   if (mp_transceiver_num mod 8 > 0)
+   { pars.ma_map.len := pars.ma_map.len + 1; }
+   pars.ma_map.ma := f_pad_bit('0'B, pars.ma_map.len * 8, 
'0'B);
+
+   /* Compose the actual Mobile Allocation and the bitmask 
*/
for (j := 0; j < lengthof(g.trx_maio); j := j + 1) {
var FreqHopGroupItem gi := g.trx_maio[j];
pars.ma := pars.ma & { l1ctl_ma_def[gi.trx_nr] 
};
+   pars.ma_map.ma[gi.trx_nr] := '1'B;
}

-   log("Freq. hopping parameters: ", pars.maio_hsn, 
pars.ma);
+   log("Freq. hopping parameters: maio_hsn := ", 
pars.maio_hsn,
+   ", ma := ", pars.ma, ", ma_map := ", pars.ma_map);
break; /* We're done */
}
}

--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19517
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: Ic0a07d9deb60712347a42d9aa7813c684eaccd01
Gerrit-Change-Number: 19517
Gerrit-PatchSet: 5
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged

Change in osmo-ttcn3-hacks[master]: BTS_Tests: verify hopping parameters in the INFO.ind message

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19518 )

Change subject: BTS_Tests: verify hopping parameters in the INFO.ind message
..

BTS_Tests: verify hopping parameters in the INFO.ind message

Change-Id: I9bb164fd2c7c48b91e0d7bd1abaf3cfec155342c
Related: I0478b499d0488269544910fbd4ed9929c05d
Related: SYS#4868, OS#4546, OS#4547
---
M bts/BTS_Tests.ttcn
1 file changed, 45 insertions(+), 0 deletions(-)

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



diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn
index 8a9e24a..8a478ba 100644
--- a/bts/BTS_Tests.ttcn
+++ b/bts/BTS_Tests.ttcn
@@ -5442,6 +5442,48 @@
setverdict(pass);
 }

+/* Verify hopping parameters in the INFO.ind message (version >= 10) */
+testcase TC_pcu_info_ind_fh_params() runs on test_CT {
+   var PCUIF_info_ind info_ind;
+   var ConnHdlrPars pars;
+
+   f_init();
+
+   info_ind := g_pcu_last_info.u.info_ind;
+
+   for (var integer i := 0; i < mp_transceiver_num; i := i + 1) {
+   for (var integer tn := 0; tn < 8; tn := tn + 1) {
+   if (info_ind.trx.v10[i].pdch_mask[tn] != '1'B) {
+   /* Skip inactive timeslots */
+   continue;
+   }
+
+   pars := valueof(t_Pars(t_RslChanNr_PDCH(tn), 
ts_RSL_ChanMode_SIGN));
+   if (mp_freq_hop_enabled and mp_transceiver_num > 1)
+   { f_resolve_fh_params(pars); }
+
+   var template PCUIF_InfoTrxTs tr_ts;
+   if (ispresent(pars.maio_hsn)) {
+   tr_ts := tr_PCUIF_InfoTrxTsH1(
+   hsn := pars.maio_hsn.hsn,
+   maio := pars.maio_hsn.maio,
+   ma := f_pad_bit(pars.ma_map.ma, 
64, '0'B));
+   } else {
+   tr_ts := tr_PCUIF_InfoTrxTsH0;
+   }
+
+   var PCUIF_InfoTrxTs ts := info_ind.trx.v10[i].ts[tn];
+   log("Checking timeslot #", tn, " of trx#", i, ": ", ts);
+   if (not match(ts, tr_ts)) {
+   setverdict(fail, "Hopping parameters do not 
match: ",
+  "received ", ts, " vs expected ", 
tr_ts);
+   } else {
+   setverdict(pass);
+   }
+   }
+   }
+}
+
 /***
  * Osmocom Style Dynamic Timeslot Support
  ***/
@@ -6844,6 +6886,9 @@
execute( TC_dyn_ipa_pdch_act_deact() );
execute( TC_dyn_ipa_pdch_act_tchf_act_nack() );

+   if (PCUIF_Types.mp_pcuif_version >= 10) {
+   execute( TC_pcu_info_ind_fh_params() );
+   }
} else {
log("PCU socket path not available, skipping PCU tests");
}

--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19518
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: I9bb164fd2c7c48b91e0d7bd1abaf3cfec155342c
Gerrit-Change-Number: 19518
Gerrit-PatchSet: 5
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged

Change in osmo-bts[master]: common: Support setting rt prio through new libosmovty sched VTY cmds

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/19444 )

Change subject: common: Support setting rt prio through new libosmovty sched 
VTY cmds
..


Patch Set 2: Code-Review+2


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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ice46e406b84fa11afcc7ba31e521e7677df73cf3
Gerrit-Change-Number: 19444
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:10:47 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-ttcn3-hacks[master]: library/PCUIF_Types: version 10: add frequency hopping parameters

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19323 )

Change subject: library/PCUIF_Types: version 10: add frequency hopping 
parameters
..

library/PCUIF_Types: version 10: add frequency hopping parameters

This change introduces new version 10 specific extensions, in
particular: the frequency hopping parameters of each timeslot.
These parameters are used to compose Channel Description IE
in the packet resource assignment messages.

In order to maintain backwards compatibility with version 9 of
the PCUIF, and thus to still be able to run test cases against
the latest release of osmo-pcu, I kept the old parts of the
INFO.ind and gruoped them together with the new records
into union 'PCUIF_InfoTrxs'.

During decoding, the content of this union is resolved by the
TITAN's RAW codec itself, depending on value of the 'version'
field.  During the encoding, it's the responsibility of the
API user to set a proper field of the union.  I implemented
both f_PCUIF_ver_INFO_{Trxs,PDCHMask} helpers for that.

Version 9 is kept as default, so this change can be merged
independently of the actual implementation.  We can bump
it and remove the compatibility glue once the new versions
of both osmo-bts and osmo-pcu are released.

Change-Id: Idf11bc4ba3ff0b00b32f2beab8fd020c67119d05
Related: SYS#4868, OS#4547
---
M bts/BTS_Tests.ttcn
M library/PCUIF_Types.ttcn
M pcu/PCU_Tests.ttcn
3 files changed, 107 insertions(+), 13 deletions(-)

Approvals:
  neels: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/bts/BTS_Tests.ttcn b/bts/BTS_Tests.ttcn
index 1c8127b..dfda043 100644
--- a/bts/BTS_Tests.ttcn
+++ b/bts/BTS_Tests.ttcn
@@ -5444,7 +5444,7 @@
var PCUIF_send_data sd;

[] PCU.receive(t_SD_PCUIF(pcu_conn_id, tr_PCUIF_INFO_IND(bts_nr, ?))) 
-> value sd {
-   var bitstring pdch_mask := 
sd.data.u.info_ind.trx[trx_nr].pdch_mask;
+   var bitstring pdch_mask := 
f_PCUIF_ver_INFO_PDCHMask(sd.data.u.info_ind, trx_nr);
if (substr(pdch_mask, g_chan_nr.tn, 1) != exp) {
repeat;
}
diff --git a/library/PCUIF_Types.ttcn b/library/PCUIF_Types.ttcn
index 089403c..1c5b0af 100644
--- a/library/PCUIF_Types.ttcn
+++ b/library/PCUIF_Types.ttcn
@@ -20,7 +20,7 @@
 };

 const charstring PCU_SOCK_DEFAULT := "/tmp/pcu_bts";
-type integer PCUIF_Version (9); /* supported versions */
+type integer PCUIF_Version (9..10); /* supported versions */

 type enumerated PCUIF_MsgType {
PCU_IF_MSG_DATA_REQ ('00'O),
@@ -140,14 +140,38 @@
uint8_t ts_nr
 } with { variant "" };

-type record PCUIF_InfoTrx {
+type record PCUIF_InfoTrxTs {
+   uint8_t tsc,
+   uint8_t hopping,
+   uint8_t hsn,
+   uint8_t maio,
+   bitstring   ma length(64)
+} with { variant (ma) "BYTEORDER(first), BITORDER(msb)" };
+private type record length(8) of PCUIF_InfoTrxTs PCUIF_InfoTrxTsList;
+
+/* Version >= 10 specific coding */
+private type record PCUIF_InfoV10Trx {
+   uint16_tarfcn,
+   BIT8pdch_mask,
+   uint32_thLayer1,
+   PCUIF_InfoTrxTsList ts
+} with { variant (pdch_mask) "BITORDER(msb)" };
+private type record length(8) of PCUIF_InfoV10Trx PCUIF_InfoV10TrxList;
+
+/* Version <= 9 specific coding */
+private type record PCUIF_InfoV09Trx {
uint16_tarfcn,
BIT8pdch_mask,
OCT1spare,
OCT8tsc,
uint32_thLayer1
 } with { variant (pdch_mask) "BITORDER(msb)" };
-type record length(8) of PCUIF_InfoTrx PCUIF_InfoTrxs;
+private type record length(8) of PCUIF_InfoV09Trx PCUIF_InfoV09TrxList;
+
+type union PCUIF_InfoTrxs {
+   PCUIF_InfoV09TrxListv09,
+   PCUIF_InfoV10TrxListv10
+} with { variant "" };

 type record PCUIF_info_ind {
uint32_tversion,
@@ -187,7 +211,11 @@
record length(2) of uint16_tlocal_pprt,
record length(2) of uint16_tremote_port,
record length(2) of OCT4remote_ip
-} with { variant "" }
+} with {
+   /* NOTE: TITAN is not smart enough to handle 'version < 10' and 
'version > 9',
+* so we cannot support more than two versions at the same time here. 
Sigh. */
+   variant (trx) "CROSSTAG(v09, version = 9; v10, version = 10)"
+};

 type record PCUIF_act_req {
uint8_t is_activate,
@@ -676,9 +704,41 @@
spare2 := '000'B
 };

-template (value) PCUIF_InfoTrx ts_PCUIF_InfoTrx(template (value) uint16_t 
arfcn := 871,
-   template (value) BIT8 pdch_mask 
:= '0001'B,
-   OCT1 tsc := '07'O) := {
+template (value) 

Change in osmo-bts[master]: common: Support setting rt prio through new libosmovty sched VTY cmds

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/19444 )

Change subject: common: Support setting rt prio through new libosmovty sched 
VTY cmds
..

common: Support setting rt prio through new libosmovty sched VTY cmds

We gain other features from libosmovty for free, like configuring
cpu-affinity of the only thread in the process.

Depends: libosmocore.git Change-Id If76a4bd2cc7b3c7adf5d84790a944d78be70e10a
Depends: osmo-gsm-masnuals.git Change-Id 
Icd75769ef630c3fa985fc5e2154d5521689cdd3c

Related: SYS#4986
Change-Id: Ice46e406b84fa11afcc7ba31e521e7677df73cf3
---
M TODO-RELEASE
M doc/manuals/osmobts-usermanual.adoc
M src/common/main.c
3 files changed, 9 insertions(+), 2 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  fixeria: Looks good to me, but someone else must approve
  Jenkins Builder: Verified



diff --git a/TODO-RELEASE b/TODO-RELEASE
index 1dd5e79..72e4757 100644
--- a/TODO-RELEASE
+++ b/TODO-RELEASE
@@ -1 +1,2 @@
-* update libosmocore dependency to > 1.3.x for lapdm_channel_init3()
+* update libosmocore dependency to > 1.3.x for lapdm_channel_init3(), 
osmo_cpu_sched_vty_init()
+* update osmo-gsm-manuals dependency to > 0.3.0 for vty_cpu_sched.adoc include.
diff --git a/doc/manuals/osmobts-usermanual.adoc 
b/doc/manuals/osmobts-usermanual.adoc
index 55087b2..b1b0747 100644
--- a/doc/manuals/osmobts-usermanual.adoc
+++ b/doc/manuals/osmobts-usermanual.adoc
@@ -30,6 +30,8 @@

 include::{srcdir}/chapters/architecture.adoc[]

+include::./common/chapters/vty_cpu_sched.adoc[]
+
 include::./common/chapters/trx_if.adoc[]

 include::./common/chapters/control_if.adoc[]
diff --git a/src/common/main.c b/src/common/main.c
index f4dc505..dfdc2a7 100644
--- a/src/common/main.c
+++ b/src/common/main.c
@@ -38,6 +38,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 

@@ -74,7 +75,7 @@
"  -T   --timestamp Prefix every log line with a 
timestamp\n"
"  -V   --version   Print version information and exit\n"
"  -e   --log-level Set a global log-level\n"
-   "  -r   --realtime PRIO Use SCHED_RR with the specified 
priority\n"
+   "  -r   --realtime PRIO Use SCHED_RR with the specified 
priority (deprecated, use VTY instead)\n"
"  -i   --gsmtap-ip The destination IP used for GSMTAP.\n"
);
bts_model_print_help();
@@ -145,6 +146,8 @@
break;
case 'r':
rt_prio = atoi(optarg);
+   fprintf(stderr, "Parameter -r is deprecated, use VTY 
cpu-sched "
+   "node setting 'policy rr %d' instead\n", 
rt_prio);
break;
case 'i':
gsmtap_ip = optarg;
@@ -238,6 +241,7 @@
osmo_stats_init(tall_bts_ctx);
vty_init(_vty_info);
ctrl_vty_init(tall_bts_ctx);
+   osmo_cpu_sched_vty_init(tall_bts_ctx);
rate_ctr_init(tall_bts_ctx);

handle_options(argc, argv);

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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ice46e406b84fa11afcc7ba31e521e7677df73cf3
Gerrit-Change-Number: 19444
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged

Change in osmo-trx[master]: Use new libosmovty cpu sched config features

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/19445 )

Change subject: Use new libosmovty cpu sched config features
..


Patch Set 2: Code-Review+2


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

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I3798603779b88ea37da03033cf7737a6e4751d6e
Gerrit-Change-Number: 19445
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:10:24 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-trx[master]: Use new libosmovty cpu sched config features

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/19445 )

Change subject: Use new libosmovty cpu sched config features
..

Use new libosmovty cpu sched config features

Using the new libosmovty features allow for:
* Setting different cpu-affinity masks for each thread in the process,
  both at startup through .cfg file as well as changing it at runtime.
* Unified VTY interface to change the scheduling policy of the process
  inherited by all osmocom processes enabling the feature.

Depends: libosmocore.git Change-Id If76a4bd2cc7b3c7adf5d84790a944d78be70e10a
Depends: osmo-gsm-masnuals.git Change-Id 
Icd75769ef630c3fa985fc5e2154d5521689cdd3c

Related: SYS#4986
Change-Id: I3798603779b88ea37da03033cf7737a6e4751d6e
---
M CommonLibs/trx_vty.c
A TODO-RELEASE
M Transceiver52M/Transceiver.cpp
M Transceiver52M/device/uhd/UHDDevice.cpp
M Transceiver52M/osmo-trx.cpp
M Transceiver52M/radioInterface.cpp
M doc/examples/osmo-trx-lms/osmo-trx-limesdr.cfg
M doc/examples/osmo-trx-uhd/osmo-trx-limesdr.cfg
M doc/examples/osmo-trx-uhd/osmo-trx-umtrx.cfg
M doc/examples/osmo-trx-uhd/osmo-trx-usrp_b200.cfg
M doc/manuals/osmotrx-usermanual.adoc
11 files changed, 32 insertions(+), 7 deletions(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved



diff --git a/CommonLibs/trx_vty.c b/CommonLibs/trx_vty.c
index e37ecaf..1cf4f5a 100644
--- a/CommonLibs/trx_vty.c
+++ b/CommonLibs/trx_vty.c
@@ -291,7 +291,7 @@
return CMD_SUCCESS;
 }
 
-DEFUN(cfg_rt_prio, cfg_rt_prio_cmd,
+DEFUN_DEPRECATED(cfg_rt_prio, cfg_rt_prio_cmd,
"rt-prio <1-32>",
"Set the SCHED_RR real-time priority\n"
"Real time priority\n")
@@ -299,6 +299,8 @@
struct trx_ctx *trx = trx_from_vty(vty);

trx->cfg.sched_rr = atoi(argv[0]);
+   vty_out (vty, "%% 'rt-prio %u' is deprecated, use 'policy rr %u' under 
'sched' node instead%s",
+trx->cfg.sched_rr, trx->cfg.sched_rr, VTY_NEWLINE);

return CMD_SUCCESS;
 }
diff --git a/TODO-RELEASE b/TODO-RELEASE
new file mode 100644
index 000..75fe7b8
--- /dev/null
+++ b/TODO-RELEASE
@@ -0,0 +1,2 @@
+* update libosmocore dependency to > 1.3.x for osmo_sched_vty_init(), 
osmo_sched_vty_apply_localthread()
+* update osmo-gsm-manuals dependency to > 0.3.0 for vty_cpu_sched.adoc include.
diff --git a/Transceiver52M/Transceiver.cpp b/Transceiver52M/Transceiver.cpp
index c3ef377..32e0226 100644
--- a/Transceiver52M/Transceiver.cpp
+++ b/Transceiver52M/Transceiver.cpp
@@ -37,6 +37,7 @@
 #include 
 #include 
 #include 
+#include 
 }

 #ifdef HAVE_CONFIG_H
@@ -1273,6 +1274,7 @@

   snprintf(thread_name, 16, "RxUpper%zu", num);
   set_selfthread_name(thread_name);
+  OSMO_ASSERT(osmo_cpu_sched_vty_apply_localthread() == 0);

   while (1) {
 if (!trx->driveReceiveFIFO(num)) {
@@ -1288,6 +1290,7 @@
 void *RxLowerLoopAdapter(Transceiver *transceiver)
 {
   set_selfthread_name("RxLower");
+  OSMO_ASSERT(osmo_cpu_sched_vty_apply_localthread() == 0);

   while (1) {
 if (!transceiver->driveReceiveRadio()) {
@@ -1303,6 +1306,7 @@
 void *TxLowerLoopAdapter(Transceiver *transceiver)
 {
   set_selfthread_name("TxLower");
+  OSMO_ASSERT(osmo_cpu_sched_vty_apply_localthread() == 0);

   while (1) {
 transceiver->driveTxFIFO();
@@ -1321,6 +1325,7 @@

   snprintf(thread_name, 16, "TxUpper%zu", num);
   set_selfthread_name(thread_name);
+  OSMO_ASSERT(osmo_cpu_sched_vty_apply_localthread() == 0);

   while (1) {
 if (!trx->driveTxPriorityQueue(num)) {
diff --git a/Transceiver52M/device/uhd/UHDDevice.cpp 
b/Transceiver52M/device/uhd/UHDDevice.cpp
index 854ed2e..b5dda28 100644
--- a/Transceiver52M/device/uhd/UHDDevice.cpp
+++ b/Transceiver52M/device/uhd/UHDDevice.cpp
@@ -34,7 +34,9 @@
 #endif

 extern "C" {
+#include 
 #include 
+#include 
 }

 #ifdef USE_UHD_3_11
@@ -150,6 +152,7 @@
 void *async_event_loop(uhd_device *dev)
 {
set_selfthread_name("UHDAsyncEvent");
+   OSMO_ASSERT(osmo_cpu_sched_vty_apply_localthread() == 0);

while (1) {
dev->recv_async_msg();
diff --git a/Transceiver52M/osmo-trx.cpp b/Transceiver52M/osmo-trx.cpp
index 9fcbda5..4a92447 100644
--- a/Transceiver52M/osmo-trx.cpp
+++ b/Transceiver52M/osmo-trx.cpp
@@ -55,6 +55,7 @@
 #include 
 #include 
 #include 
+#include 

 #include "convolve.h"
 #include "convert.h"
@@ -435,7 +436,9 @@
int rc;
memset(, 0, sizeof(param));
param.sched_priority = prio;
-   LOG(INFO) << "Setting SCHED_RR priority " << param.sched_priority;
+   LOG(INFO) << "Setting SCHED_RR priority " << param.sched_priority
+ << ". This setting is DEPRECATED, please use 'policy rr " << 
param.sched_priority
+ << "' under the 'sched' VTY node instead.";
rc = sched_setscheduler(getpid(), SCHED_RR, );
if (rc != 0) {
LOG(ERROR) << 

Change in osmo-ttcn3-hacks[master]: library/PCUIF_Types: version 10: add frequency hopping parameters

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19323 )

Change subject: library/PCUIF_Types: version 10: add frequency hopping 
parameters
..


Patch Set 11: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19323
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: Idf11bc4ba3ff0b00b32f2beab8fd020c67119d05
Gerrit-Change-Number: 19323
Gerrit-PatchSet: 11
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: neels 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:09:54 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-ttcn3-hacks[master]: library/PCUIF_Types: add receive templates for PCUIF_InfoTrxTs

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19516 )

Change subject: library/PCUIF_Types: add receive templates for PCUIF_InfoTrxTs
..

library/PCUIF_Types: add receive templates for PCUIF_InfoTrxTs

Change-Id: I062079de4440d2c9c0a079cc4cc4328ea0ea37dd
---
M library/PCUIF_Types.ttcn
1 file changed, 16 insertions(+), 0 deletions(-)

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



diff --git a/library/PCUIF_Types.ttcn b/library/PCUIF_Types.ttcn
index 1c5b0af..dbb1436 100644
--- a/library/PCUIF_Types.ttcn
+++ b/library/PCUIF_Types.ttcn
@@ -710,6 +710,12 @@
hsn := 0, maio := 0,
ma := f_pad_bit(''B, 64, '0'B)
 };
+template PCUIF_InfoTrxTs tr_PCUIF_InfoTrxTsH0(template uint3_t tsc := ?) := {
+   tsc := tsc,
+   hopping := 0,
+   hsn := ?, maio := ?,
+   ma := ?
+};

 template (value) PCUIF_InfoTrxTs ts_PCUIF_InfoTrxTsH1(template (value) uint3_t 
tsc := 7,
  template (value) uint6_t 
hsn := 0,
@@ -721,6 +727,16 @@
maio := maio,
ma := f_pad_bit(valueof(ma), 64, '0'B)
 };
+template PCUIF_InfoTrxTs tr_PCUIF_InfoTrxTsH1(template uint3_t tsc := ?,
+ template uint6_t hsn := ?,
+ template uint6_t maio := ?,
+ template bitstring ma := ?) := {
+   tsc := tsc,
+   hopping := 1,
+   hsn := hsn,
+   maio := maio,
+   ma := ma
+};

 template (value) PCUIF_InfoV10Trx ts_PCUIF_InfoV10Trx(template (value) 
uint16_t arfcn := 871,
  template (value) BIT8 
pdch_mask := '0001'B,

--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19516
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: I062079de4440d2c9c0a079cc4cc4328ea0ea37dd
Gerrit-Change-Number: 19516
Gerrit-PatchSet: 4
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged

Change in osmo-ttcn3-hacks[master]: MGCP_Test: add E1 related testscases

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19561 )

Change subject: MGCP_Test: add E1 related testscases
..


Patch Set 1: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19561
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: I673eeffcb3012b42f039789960c54d99282e1aad
Gerrit-Change-Number: 19561
Gerrit-PatchSet: 1
Gerrit-Owner: dexter 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:09:32 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-bsc[master]: hodec2: log requirements of best candidate, log rxlev in dBm

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19560 )

Change subject: hodec2: log requirements of best candidate, log rxlev in dBm
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I1b30a6e98bdb4bd92e72864fafdd2f4f3ae3134c
Gerrit-Change-Number: 19560
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:08:57 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-bsc[master]: hodec2: do not keep candidates with zero requirements met

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19559 )

Change subject: hodec2: do not keep candidates with zero requirements met
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I12e48292d5731cb601165c870b9570003bc488ec
Gerrit-Change-Number: 19559
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:08:21 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-bsc[master]: handover_test.c: typo s/more/less in test 19 description

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19556 )

Change subject: handover_test.c: typo s/more/less in test 19 description
..

handover_test.c: typo s/more/less in test 19 description

Change-Id: Ia46fa9bb4f340d35515168f82dd23a1270f6fcd5
---
M tests/handover/handover_test.c
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/tests/handover/handover_test.c b/tests/handover/handover_test.c
index f9bafe5..6fb145e 100644
--- a/tests/handover/handover_test.c
+++ b/tests/handover/handover_test.c
@@ -956,7 +956,7 @@
"2",

"Congestion check: Balancing over congested cells\n\n"
-   "Two cells are congested, but the second cell is more congested.\n"
+   "Two cells are congested, but the second cell is less congested.\n"
"Handover is performed to solve the congestion.\n",

"create-bts", "2",

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ia46fa9bb4f340d35515168f82dd23a1270f6fcd5
Gerrit-Change-Number: 19556
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged

Change in osmo-bsc[master]: hodec2 congestion: consider only congested pchan types

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19558 )

Change subject: hodec2 congestion: consider only congested pchan types
..

hodec2 congestion: consider only congested pchan types

If no TCH/H need to be moved, then don't collect TCH/H candidates.
Same for TCH/F.

Examining a specific congestion scenario, I realized that ho decision 2
collects candidates of pchan types that are not actually considered congested:

- handover decision 2 wants to move 2 TCH/F only; TCH/H is fine.
- still it collects active TCH/H lchans as candidates.

  (BTS 0) Congestion check: (free/want-free) TCH/F=0/2 TCH/H=12/0
  (BTS 0) Attempting to resolve congestion...
  (BTS 0) congested: 2 TCH/F and 0 TCH/H should be moved
  [...]
  (BTS 0) Considering 8 candidates to solve congestion:
  (lchan 0.020 TCH_F SPEECH_V1)->(BTS 0) (subscr unknown) #0: req=0x0 
avg-rxlev=42
  (lchan 0.030 TCH_F SPEECH_V1)->(BTS 0) (subscr unknown) #1: req=0x0 
avg-rxlev=42
  (lchan 0.040 TCH_F SPEECH_V1)->(BTS 0) (subscr unknown) #2: req=0x0 
avg-rxlev=42
  (lchan 0.050 TCH_F SPEECH_V1)->(BTS 0) (subscr unknown) #3: req=0x0 
avg-rxlev=42
  (lchan 0.160 TCH_H SPEECH_V1)->(BTS 0) (subscr IMSI:901700x) #4: 
req=0x0 avg-rxlev=42
  (lchan 0.161 TCH_H SPEECH_V1)->(BTS 0) (subscr IMSI:901700x) #5: 
req=0x0 avg-rxlev=42
  (lchan 0.170 TCH_H SPEECH_V1)->(BTS 0) (subscr IMSI:901700x) #6: 
req=0x0 avg-rxlev=42
  (lchan 0.171 TCH_H SPEECH_V1)->(BTS 0) (subscr IMSI:901700x) #7: 
req=0x0 avg-rxlev=42

Change-Id: I5077a62c6d77838513eecf3f0e4810c3d3d3a247
---
M src/osmo-bsc/handover_decision_2.c
1 file changed, 8 insertions(+), 0 deletions(-)

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



diff --git a/src/osmo-bsc/handover_decision_2.c 
b/src/osmo-bsc/handover_decision_2.c
index 4c32c38..26a8840 100644
--- a/src/osmo-bsc/handover_decision_2.c
+++ b/src/osmo-bsc/handover_decision_2.c
@@ -1445,6 +1445,10 @@
/* (Do not consider dynamic TS that are in PDCH mode) */
switch (ts->pchan_is) {
case GSM_PCHAN_TCH_F:
+   /* No need to collect TCH/F candidates if no 
TCH/F needs to be moved. */
+   if (tchf_congestion == 0)
+   continue;
+
lc = >lchan[0];
/* omit if channel not active */
if (lc->type != GSM_LCHAN_TCH_F
@@ -1459,6 +1463,10 @@
collect_candidates_for_lchan(lc, clist, 
, NULL, true);
break;
case GSM_PCHAN_TCH_H:
+   /* No need to collect TCH/H candidates if no 
TCH/H needs to be moved. */
+   if (tchh_congestion == 0)
+   continue;
+
for (j = 0; j < 2; j++) {
lc = >lchan[j];
/* omit if channel not active */

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I5077a62c6d77838513eecf3f0e4810c3d3d3a247
Gerrit-Change-Number: 19558
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged

Change in osmo-bsc[master]: hodec2: log requirements of best candidate, log rxlev in dBm

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19560 )

Change subject: hodec2: log requirements of best candidate, log rxlev in dBm
..

hodec2: log requirements of best candidate, log rxlev in dBm

Instead of logging a hex value for the met requirements, fully expand the "ABC"
flags for both TCH/F and TCH/H.

>From HO_CANDIDATE_FMT/_ARGS, split off into REQUIREMENTS_FMT/_ARGS and use that
when logging the chosen HO candidates.

Also change the RX level to dBm, to match general logging and reduce confusion
between rxlev number variants in the log.

Change-Id: I1b30a6e98bdb4bd92e72864fafdd2f4f3ae3134c
---
M src/osmo-bsc/handover_decision_2.c
1 file changed, 21 insertions(+), 14 deletions(-)

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



diff --git a/src/osmo-bsc/handover_decision_2.c 
b/src/osmo-bsc/handover_decision_2.c
index 3191957..c818dbb 100644
--- a/src/osmo-bsc/handover_decision_2.c
+++ b/src/osmo-bsc/handover_decision_2.c
@@ -797,26 +797,30 @@
return trigger_remote_bss_ho(c, requirements);
 }

+#define REQUIREMENTS_FMT "[%s%s%s]%s"
+#define REQUIREMENTS_ARGS(REQUIREMENTS, TCHX) \
+(REQUIREMENTS) & REQUIREMENT_A_TCH##TCHX ? "A" : \
+   ((REQUIREMENTS) & REQUIREMENT_TCH##TCHX##_MASK) == 0? "-" : "", 
\
+(REQUIREMENTS) & REQUIREMENT_B_TCH##TCHX ? "B" : "", \
+(REQUIREMENTS) & REQUIREMENT_C_TCH##TCHX ? "C" : "", \
+((REQUIREMENTS) & REQUIREMENT_TCH##TCHX##_MASK) == 0 ? " not a 
candidate" : \
+  (((REQUIREMENTS) & REQUIREMENT_TCH##TCHX##_MASK) == 
REQUIREMENT_A_TCH##TCHX ? \
+   " more congestion" : \
+   ((REQUIREMENTS) & REQUIREMENT_B_TCH##TCHX ? \
+" good" : \
+ /* now has to be (REQUIREMENTS) & REQUIREMENT_C_TCHX != 0: */ 
\
+ " less-or-equal congestion"))
+
 /* verbosely log about a handover candidate */
 static inline void debug_candidate(struct ho_candidate *candidate,
   int8_t rxlev, int tchf_count, int tchh_count)
 {
struct gsm_lchan *lchan = candidate->lchan;

-#define HO_CANDIDATE_FMT(tchx, TCHX) "TCH/" #TCHX "={free %d (want %d), 
[%s%s%s]%s}"
+#define HO_CANDIDATE_FMT(tchx, TCHX) "TCH/" #TCHX "={free %d (want %d), " 
REQUIREMENTS_FMT "}"
 #define HO_CANDIDATE_ARGS(tchx, TCHX) \
 tch##tchx##_count, 
ho_get_hodec2_tch##tchx##_min_slots(candidate->bts->ho), \
-candidate->requirements & REQUIREMENT_A_TCH##TCHX ? "A" : \
-   (candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 0? 
"-" : "", \
-candidate->requirements & REQUIREMENT_B_TCH##TCHX ? "B" : "", \
-candidate->requirements & REQUIREMENT_C_TCH##TCHX ? "C" : "", \
-(candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 0 ? " 
not a candidate" : \
-  ((candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 
REQUIREMENT_A_TCH##TCHX ? \
-   " more congestion" : \
-   (candidate->requirements & REQUIREMENT_B_TCH##TCHX ? \
-" good" : \
- /* now has to be candidate->requirements & REQUIREMENT_C_TCHX 
!= 0: */ \
- " less-or-equal congestion"))
+REQUIREMENTS_ARGS(candidate->requirements, TCHX)

if (!candidate->bts && !candidate->cil)
LOGPHOLCHAN(lchan, LOGL_DEBUG, "Empty candidate\n");
@@ -1503,8 +1507,11 @@
if (log_check_level(DHODEC, LOGL_DEBUG)) {
LOGPHOBTS(bts, LOGL_DEBUG, "Considering %u candidates to solve 
congestion:\n", candidates);
for (i = 0; i < candidates; i++) {
-   LOGPHOCAND([i], LOGL_DEBUG, "#%d: req=0x%x 
avg-rxlev=%d\n",
-  i, clist[i].requirements, clist[i].avg);
+
+   LOGPHOCAND([i], LOGL_DEBUG, "#%d: req={TCH/F:" 
REQUIREMENTS_FMT ", TCH/H:" REQUIREMENTS_FMT "} avg-rxlev=%d dBm\n",
+  i, REQUIREMENTS_ARGS(clist[i].requirements, 
F),
+  REQUIREMENTS_ARGS(clist[i].requirements, H),
+  rxlev2dbm(clist[i].avg));
}
}


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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I1b30a6e98bdb4bd92e72864fafdd2f4f3ae3134c
Gerrit-Change-Number: 19560
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged

Change in osmo-bsc[master]: hodec2: fix logging of requirements flags

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19557 )

Change subject: hodec2: fix logging of requirements flags
..

hodec2: fix logging of requirements flags

Fix typo to log the "C" requirements accurately.

Change-Id: I15ce607f354918167ba81192230db63a5948c742
---
M src/osmo-bsc/handover_decision_2.c
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/src/osmo-bsc/handover_decision_2.c 
b/src/osmo-bsc/handover_decision_2.c
index 8caa2e3..4c32c38 100644
--- a/src/osmo-bsc/handover_decision_2.c
+++ b/src/osmo-bsc/handover_decision_2.c
@@ -809,7 +809,7 @@
 candidate->requirements & REQUIREMENT_A_TCH##TCHX ? "A" : \
(candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 0? 
"-" : "", \
 candidate->requirements & REQUIREMENT_B_TCH##TCHX ? "B" : "", \
-candidate->requirements & REQUIREMENT_B_TCH##TCHX ? "C" : "", \
+candidate->requirements & REQUIREMENT_C_TCH##TCHX ? "C" : "", \
 (candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 0 ? " 
not a candidate" : \
   ((candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 
REQUIREMENT_A_TCH##TCHX ? \
" more congestion" : \

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I15ce607f354918167ba81192230db63a5948c742
Gerrit-Change-Number: 19557
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged

Change in osmo-bsc[master]: hodec2: do not keep candidates with zero requirements met

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19559 )

Change subject: hodec2: do not keep candidates with zero requirements met
..

hodec2: do not keep candidates with zero requirements met

When check_requirements() returns zero, do not keep such an entry in the
candidates list at all. This removes logging confusion, where some "candidates"
are still listed even though not meeting any handover requirements.

Change-Id: I12e48292d5731cb601165c870b9570003bc488ec
---
M src/osmo-bsc/handover_decision_2.c
1 file changed, 7 insertions(+), 0 deletions(-)

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



diff --git a/src/osmo-bsc/handover_decision_2.c 
b/src/osmo-bsc/handover_decision_2.c
index 26a8840..3191957 100644
--- a/src/osmo-bsc/handover_decision_2.c
+++ b/src/osmo-bsc/handover_decision_2.c
@@ -861,6 +861,10 @@
};

debug_candidate(, 0, tchf_count, tchh_count);
+
+   if (!c.requirements)
+   return;
+
clist[*candidates] = c;
(*candidates)++;
 }
@@ -966,6 +970,9 @@

debug_candidate(, av_rxlev, tchf_count, tchh_count);

+   if (!c.requirements)
+   return;
+
clist[*candidates] = c;
(*candidates)++;
 }

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I12e48292d5731cb601165c870b9570003bc488ec
Gerrit-Change-Number: 19559
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-MessageType: merged

Change in osmo-bsc[master]: hodec2 congestion: consider only congested pchan types

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19558 )

Change subject: hodec2 congestion: consider only congested pchan types
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I5077a62c6d77838513eecf3f0e4810c3d3d3a247
Gerrit-Change-Number: 19558
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:08:09 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-bsc[master]: handover_test.c: typo s/more/less in test 19 description

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19556 )

Change subject: handover_test.c: typo s/more/less in test 19 description
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ia46fa9bb4f340d35515168f82dd23a1270f6fcd5
Gerrit-Change-Number: 19556
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:07:26 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-bsc[master]: hodec2: fix logging of requirements flags

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19557 )

Change subject: hodec2: fix logging of requirements flags
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I15ce607f354918167ba81192230db63a5948c742
Gerrit-Change-Number: 19557
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:07:37 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-mgw[master]: mgcp_ratectr: fix comments in header file

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-mgw/+/19562 )

Change subject: mgcp_ratectr: fix comments in header file
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: Idd9d7b108e81b44501b78264284dfa46e679d994
Gerrit-Change-Number: 19562
Gerrit-PatchSet: 1
Gerrit-Owner: dexter 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:07:11 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-mgw[master]: mgcp_e1: finish E1 support, add E1 support from libosmoabis

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-mgw/+/19524 )

Change subject: mgcp_e1: finish E1 support, add E1 support from libosmoabis
..


Patch Set 2:

(2 comments)

https://gerrit.osmocom.org/c/osmo-mgw/+/19524/2/src/libosmo-mgcp/mgcp_e1.c
File src/libosmo-mgcp/mgcp_e1.c:

https://gerrit.osmocom.org/c/osmo-mgw/+/19524/2/src/libosmo-mgcp/mgcp_e1.c@57
PS2, Line 57: const ubit_t idle_tf_efr[] = { 0, 0, 0, 0, 0, 0, 0, 0,
well, now it's const.  But it shoud still be static, if it's only used within 
this file. static +const are not a contadiction...


https://gerrit.osmocom.org/c/osmo-mgw/+/19524/2/src/libosmo-mgcp/mgcp_e1.c@142
PS2, Line 142: /* Idle speech frame, see also GSM 08.60, chapter 3.4 */
same as above.



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

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I6b93809b5ac7d01af55888347dd787b0bc997ae1
Gerrit-Change-Number: 19524
Gerrit-PatchSet: 2
Gerrit-Owner: dexter 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:06:58 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Change in osmo-mgw[master]: get rid of mgcp_internal.h

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-mgw/+/19525 )

Change subject: get rid of mgcp_internal.h
..


Patch Set 2: Code-Review+2


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

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I837eaad771ed7252304db4a81c37953b70766fff
Gerrit-Change-Number: 19525
Gerrit-PatchSet: 2
Gerrit-Owner: dexter 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 20:00:40 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-mgw[master]: mgcp_ratectr: fix comments in header file

[dexter]

dexter has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-mgw/+/19562 )


Change subject: mgcp_ratectr: fix comments in header file
..

mgcp_ratectr: fix comments in header file

Change-Id: Idd9d7b108e81b44501b78264284dfa46e679d994
---
M include/osmocom/mgcp/mgcp_ratectr.h
1 file changed, 3 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/62/19562/1

diff --git a/include/osmocom/mgcp/mgcp_ratectr.h 
b/include/osmocom/mgcp/mgcp_ratectr.h
index ff59ea4..657e897 100644
--- a/include/osmocom/mgcp/mgcp_ratectr.h
+++ b/include/osmocom/mgcp/mgcp_ratectr.h
@@ -11,7 +11,7 @@
MGCP_GENERAL_RX_FAIL_NO_ENDPOINT,
 };

-/* Global MCGP CRCX related rate counters */
+/* Trunk-global MCGP CRCX related rate counters */
 enum {
MGCP_CRCX_SUCCESS,
MGCP_CRCX_FAIL_BAD_ACTION,
@@ -32,7 +32,7 @@
MGCP_CRCX_FAIL_CLAIM,
 };

-/* Global MCGP MDCX related rate counters */
+/* Trunk-global MCGP MDCX related rate counters */
 enum {
MGCP_MDCX_SUCCESS,
MGCP_MDCX_FAIL_WILDCARD,
@@ -51,7 +51,7 @@
MGCP_MDCX_FAIL_AVAIL,
 };

-/* Global MCGP DLCX related rate counters */
+/* Trunk-global MCGP DLCX related rate counters */
 enum {
MGCP_DLCX_SUCCESS,
MGCP_DLCX_FAIL_WILDCARD,

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

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: Idd9d7b108e81b44501b78264284dfa46e679d994
Gerrit-Change-Number: 19562
Gerrit-PatchSet: 1
Gerrit-Owner: dexter 
Gerrit-MessageType: newchange

Change in osmo-mgw[master]: get rid of mgcp_internal.h

[dexter]

Hello laforge, Jenkins Builder,

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

https://gerrit.osmocom.org/c/osmo-mgw/+/19525

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

Change subject: get rid of mgcp_internal.h
..

get rid of mgcp_internal.h

The file mgcp_internal.h still contains mostly definitions and types
that are relevant for mgcp_network.c and mgcp_protocol.c. Lets give
the network and protocol module its own header files, also move stuff
that does not relate to protocol and network to the appropiate places.

Change-Id: I837eaad771ed7252304db4a81c37953b70766fff
---
M include/Makefile.am
M include/osmocom/mgcp/Makefile.am
M include/osmocom/mgcp/mgcp_codec.h
M include/osmocom/mgcp/mgcp_conn.h
D include/osmocom/mgcp/mgcp_internal.h
A include/osmocom/mgcp/mgcp_network.h
A include/osmocom/mgcp/mgcp_protocol.h
M include/osmocom/mgcp/mgcp_stat.h
M src/libosmo-mgcp/mgcp_codec.c
M src/libosmo-mgcp/mgcp_conn.c
M src/libosmo-mgcp/mgcp_e1.c
M src/libosmo-mgcp/mgcp_endp.c
M src/libosmo-mgcp/mgcp_msg.c
M src/libosmo-mgcp/mgcp_network.c
M src/libosmo-mgcp/mgcp_osmux.c
M src/libosmo-mgcp/mgcp_protocol.c
M src/libosmo-mgcp/mgcp_ratectr.c
M src/libosmo-mgcp/mgcp_sdp.c
M src/libosmo-mgcp/mgcp_stat.c
M src/libosmo-mgcp/mgcp_trunk.c
M src/libosmo-mgcp/mgcp_vty.c
M src/osmo-mgw/mgw_main.c
M tests/mgcp/mgcp_test.c
23 files changed, 306 insertions(+), 331 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/25/19525/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/19525
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I837eaad771ed7252304db4a81c37953b70766fff
Gerrit-Change-Number: 19525
Gerrit-PatchSet: 2
Gerrit-Owner: dexter 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-MessageType: newpatchset

Change in osmo-mgw[master]: mgcp_e1: finish E1 support, add E1 support from libosmoabis

[dexter]

Hello Jenkins Builder,

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

https://gerrit.osmocom.org/c/osmo-mgw/+/19524

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

Change subject: mgcp_e1: finish E1 support, add E1 support from libosmoabis
..

mgcp_e1: finish E1 support, add E1 support from libosmoabis

Currently only the endpoint handling for E1 exists, but there is no
actual code behind it that handles the E1 traffic.

Change-Id: I6b93809b5ac7d01af55888347dd787b0bc997ae1
Related: OS#2659
---
M configure.ac
M include/osmocom/mgcp/Makefile.am
M include/osmocom/mgcp/debug.h
M include/osmocom/mgcp/mgcp_conn.h
A include/osmocom/mgcp/mgcp_e1.h
M include/osmocom/mgcp/mgcp_endp.h
M include/osmocom/mgcp/mgcp_internal.h
M include/osmocom/mgcp/mgcp_ratectr.h
M include/osmocom/mgcp/mgcp_trunk.h
M src/libosmo-mgcp/Makefile.am
M src/libosmo-mgcp/mgcp_conn.c
A src/libosmo-mgcp/mgcp_e1.c
M src/libosmo-mgcp/mgcp_endp.c
M src/libosmo-mgcp/mgcp_network.c
M src/libosmo-mgcp/mgcp_protocol.c
M src/libosmo-mgcp/mgcp_ratectr.c
M src/libosmo-mgcp/mgcp_trunk.c
M src/libosmo-mgcp/mgcp_vty.c
M src/osmo-mgw/Makefile.am
M src/osmo-mgw/mgw_main.c
M tests/mgcp/Makefile.am
M tests/mgcp/mgcp_test.c
22 files changed, 1,076 insertions(+), 62 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/24/19524/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/19524
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I6b93809b5ac7d01af55888347dd787b0bc997ae1
Gerrit-Change-Number: 19524
Gerrit-PatchSet: 2
Gerrit-Owner: dexter 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge 
Gerrit-MessageType: newpatchset

Change in osmo-ttcn3-hacks[master]: MGCP_Test: add E1 related testscases

[dexter]

dexter has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19561 )


Change subject: MGCP_Test: add E1 related testscases
..

MGCP_Test: add E1 related testscases

osmo-mgw recently added support for E1 trunks via libosmoabis. While the
actual E1 plane may be difficult to test, there is some functionality
that can already be tested without having E1 support in TTCN3.

Lets add three testcases:
- TC_e1_crcx_and_dlcx_ep:
  Does a CRCX to an E1 endpoint followed by a DLCX
- TC_e1_crcx_with_overlap:
  Not all E1 endpoint combinations are possible at the same time. This
  test verifies that invalid endpoint combinations are prevented.
- TC_e1_crcx_loopback:
  Opes an E1 endpoint and tests if RTP loopback works (NO E1 traffic
  involved)

Change-Id: I673eeffcb3012b42f039789960c54d99282e1aad
Related: OS#2659
---
M mgw/MGCP_Test.ttcn
1 file changed, 87 insertions(+), 0 deletions(-)



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

diff --git a/mgw/MGCP_Test.ttcn b/mgw/MGCP_Test.ttcn
index 6caef79..24bfdaf 100644
--- a/mgw/MGCP_Test.ttcn
+++ b/mgw/MGCP_Test.ttcn
@@ -2146,6 +2146,88 @@
setverdict(pass);
}

+   /* Test (valid) CRCX followed by (valid) DLCX containing EP (E1) */
+   testcase TC_e1_crcx_and_dlcx_ep() runs on dummy_CT {
+   var template MgcpCommand cmd;
+   var MgcpResponse resp;
+   var MgcpEndpoint ep := "ds/e1-1/s-1/su16-0@" & c_mgw_domain;
+   var MgcpCallId call_id := '8376F297'H;
+
+   f_init(ep);
+
+   cmd := ts_CRCX(get_next_trans_id(), ep, "recvonly", call_id);
+   resp := mgcp_transceive_mgw(cmd, tr_CRCX_ACK);
+
+   f_dlcx_ok(ep);
+
+   setverdict(pass);
+   }
+
+   /* Test what happens when overlapping endpoints are selected (E1) */
+   testcase TC_e1_crcx_with_overlap() runs on dummy_CT {
+   var template MgcpCommand cmd;
+   var MgcpResponse resp;
+   var MgcpEndpoint ep_1 := "ds/e1-1/s-1/su8-0@" & c_mgw_domain;
+   var MgcpEndpoint ep_2 := "ds/e1-1/s-1/su16-0@" & c_mgw_domain;
+   var MgcpCallId call_id_1 := '8376F297'H;
+   var MgcpCallId call_id_2 := '837AF2A7'H;
+
+   f_init();
+
+   /* ep_1 and ep_2 are overlapping, selecting both one after
+* another should work fine: */
+   cmd := ts_CRCX(get_next_trans_id(), ep_1, "recvonly", 
call_id_1);
+   resp := mgcp_transceive_mgw(cmd, tr_CRCX_ACK);
+   f_dlcx_ok(ep_1);
+   cmd := ts_CRCX(get_next_trans_id(), ep_2, "recvonly", 
call_id_2);
+   resp := mgcp_transceive_mgw(cmd, tr_CRCX_ACK);
+   f_dlcx_ok(ep_2);
+
+   /* When ep_1 is serving a call we can not select ep_2 becaus
+* it is overlapping with ep_1 */
+   cmd := ts_CRCX(get_next_trans_id(), ep_1, "recvonly", 
call_id_1);
+   resp := mgcp_transceive_mgw(cmd, tr_CRCX_ACK);
+   cmd := ts_CRCX(get_next_trans_id(), ep_2, "recvonly", 
call_id_2);
+   resp := mgcp_transceive_mgw(cmd, ?);
+   if (resp.line.code != "501") {
+   setverdict(fail, "unexpected CRCX returncode, CRCX 
should fail!");
+   }
+   f_dlcx_ok(ep_1);
+
+   setverdict(pass);
+   }
+
+   /* Create one connection in loopback mode, test if the RTP packets are
+* actually reflected */
+   testcase TC_e1_crcx_loopback() runs on dummy_CT {
+   var RtpFlowData flow;
+   var MgcpEndpoint ep := "ds/e1-1/s-1/su16-0@" & c_mgw_domain;
+   var MgcpCallId call_id := '12250989'H;
+   var RtpemStats stats;
+
+   f_init(ep);
+   flow := valueof(t_RtpFlow(mp_local_ip, mp_remote_ip, 111, 
"GSM-HR-08/8000/1"));
+   flow.em.portnr := 1;
+   f_flow_create(RTPEM[0], ep, call_id, "loopback", flow);
+
+   f_rtpem_mode(RTPEM[0], RTPEM_MODE_BIDIR);
+   f_sleep(1.0);
+   f_flow_delete(RTPEM[0], ep, call_id);
+
+   stats := f_rtpem_stats_get(RTPEM[0]);
+
+   if (stats.num_pkts_tx != stats.num_pkts_rx) {
+   setverdict(fail);
+   }
+   if (stats.bytes_payload_tx != stats.bytes_payload_rx) {
+   setverdict(fail);
+   }
+
+   f_rtpem_stats_err_check(stats);
+
+   setverdict(pass);
+   }
+
control {
execute(TC_selftest());
execute(TC_crcx());
@@ -2201,5 +2283,10 @@
execute(TC_amr_bwe_bwe_rtp_conversion());

execute(TC_conn_timeout());
+
+   execute(TC_e1_crcx_and_dlcx_ep());
+  

Change in osmo-bsc[master]: hodec2: do not keep candidates with zero requirements met

[neels]

neels has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19559 )


Change subject: hodec2: do not keep candidates with zero requirements met
..

hodec2: do not keep candidates with zero requirements met

When check_requirements() returns zero, do not keep such an entry in the
candidates list at all. This removes logging confusion, where some "candidates"
are still listed even though not meeting any handover requirements.

Change-Id: I12e48292d5731cb601165c870b9570003bc488ec
---
M src/osmo-bsc/handover_decision_2.c
1 file changed, 7 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/59/19559/1

diff --git a/src/osmo-bsc/handover_decision_2.c 
b/src/osmo-bsc/handover_decision_2.c
index 26a8840..3191957 100644
--- a/src/osmo-bsc/handover_decision_2.c
+++ b/src/osmo-bsc/handover_decision_2.c
@@ -861,6 +861,10 @@
};

debug_candidate(, 0, tchf_count, tchh_count);
+
+   if (!c.requirements)
+   return;
+
clist[*candidates] = c;
(*candidates)++;
 }
@@ -966,6 +970,9 @@

debug_candidate(, av_rxlev, tchf_count, tchh_count);

+   if (!c.requirements)
+   return;
+
clist[*candidates] = c;
(*candidates)++;
 }

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I12e48292d5731cb601165c870b9570003bc488ec
Gerrit-Change-Number: 19559
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-MessageType: newchange

Change in osmo-bsc[master]: handover_test.c: typo s/more/less in test 19 description

[neels]

neels has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19556 )


Change subject: handover_test.c: typo s/more/less in test 19 description
..

handover_test.c: typo s/more/less in test 19 description

Change-Id: Ia46fa9bb4f340d35515168f82dd23a1270f6fcd5
---
M tests/handover/handover_test.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/56/19556/1

diff --git a/tests/handover/handover_test.c b/tests/handover/handover_test.c
index f9bafe5..6fb145e 100644
--- a/tests/handover/handover_test.c
+++ b/tests/handover/handover_test.c
@@ -956,7 +956,7 @@
"2",

"Congestion check: Balancing over congested cells\n\n"
-   "Two cells are congested, but the second cell is more congested.\n"
+   "Two cells are congested, but the second cell is less congested.\n"
"Handover is performed to solve the congestion.\n",

"create-bts", "2",

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ia46fa9bb4f340d35515168f82dd23a1270f6fcd5
Gerrit-Change-Number: 19556
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-MessageType: newchange

Change in osmo-bsc[master]: hodec2 congestion: consider only congested pchan types

[neels]

neels has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19558 )


Change subject: hodec2 congestion: consider only congested pchan types
..

hodec2 congestion: consider only congested pchan types

If no TCH/H need to be moved, then don't collect TCH/H candidates.
Same for TCH/F.

Examining a specific congestion scenario, I realized that ho decision 2
collects candidates of pchan types that are not actually considered congested:

- handover decision 2 wants to move 2 TCH/F only; TCH/H is fine.
- still it collects active TCH/H lchans as candidates.

  (BTS 0) Congestion check: (free/want-free) TCH/F=0/2 TCH/H=12/0
  (BTS 0) Attempting to resolve congestion...
  (BTS 0) congested: 2 TCH/F and 0 TCH/H should be moved
  [...]
  (BTS 0) Considering 8 candidates to solve congestion:
  (lchan 0.020 TCH_F SPEECH_V1)->(BTS 0) (subscr unknown) #0: req=0x0 
avg-rxlev=42
  (lchan 0.030 TCH_F SPEECH_V1)->(BTS 0) (subscr unknown) #1: req=0x0 
avg-rxlev=42
  (lchan 0.040 TCH_F SPEECH_V1)->(BTS 0) (subscr unknown) #2: req=0x0 
avg-rxlev=42
  (lchan 0.050 TCH_F SPEECH_V1)->(BTS 0) (subscr unknown) #3: req=0x0 
avg-rxlev=42
  (lchan 0.160 TCH_H SPEECH_V1)->(BTS 0) (subscr IMSI:901700x) #4: 
req=0x0 avg-rxlev=42
  (lchan 0.161 TCH_H SPEECH_V1)->(BTS 0) (subscr IMSI:901700x) #5: 
req=0x0 avg-rxlev=42
  (lchan 0.170 TCH_H SPEECH_V1)->(BTS 0) (subscr IMSI:901700x) #6: 
req=0x0 avg-rxlev=42
  (lchan 0.171 TCH_H SPEECH_V1)->(BTS 0) (subscr IMSI:901700x) #7: 
req=0x0 avg-rxlev=42

Change-Id: I5077a62c6d77838513eecf3f0e4810c3d3d3a247
---
M src/osmo-bsc/handover_decision_2.c
1 file changed, 8 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/58/19558/1

diff --git a/src/osmo-bsc/handover_decision_2.c 
b/src/osmo-bsc/handover_decision_2.c
index 4c32c38..26a8840 100644
--- a/src/osmo-bsc/handover_decision_2.c
+++ b/src/osmo-bsc/handover_decision_2.c
@@ -1445,6 +1445,10 @@
/* (Do not consider dynamic TS that are in PDCH mode) */
switch (ts->pchan_is) {
case GSM_PCHAN_TCH_F:
+   /* No need to collect TCH/F candidates if no 
TCH/F needs to be moved. */
+   if (tchf_congestion == 0)
+   continue;
+
lc = >lchan[0];
/* omit if channel not active */
if (lc->type != GSM_LCHAN_TCH_F
@@ -1459,6 +1463,10 @@
collect_candidates_for_lchan(lc, clist, 
, NULL, true);
break;
case GSM_PCHAN_TCH_H:
+   /* No need to collect TCH/H candidates if no 
TCH/H needs to be moved. */
+   if (tchh_congestion == 0)
+   continue;
+
for (j = 0; j < 2; j++) {
lc = >lchan[j];
/* omit if channel not active */

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I5077a62c6d77838513eecf3f0e4810c3d3d3a247
Gerrit-Change-Number: 19558
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-MessageType: newchange

Change in osmo-bsc[master]: hodec2: fix logging of requirements flags

[neels]

neels has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19557 )


Change subject: hodec2: fix logging of requirements flags
..

hodec2: fix logging of requirements flags

Fix typo to log the "C" requirements accurately.

Change-Id: I15ce607f354918167ba81192230db63a5948c742
---
M src/osmo-bsc/handover_decision_2.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/57/19557/1

diff --git a/src/osmo-bsc/handover_decision_2.c 
b/src/osmo-bsc/handover_decision_2.c
index 8caa2e3..4c32c38 100644
--- a/src/osmo-bsc/handover_decision_2.c
+++ b/src/osmo-bsc/handover_decision_2.c
@@ -809,7 +809,7 @@
 candidate->requirements & REQUIREMENT_A_TCH##TCHX ? "A" : \
(candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 0? 
"-" : "", \
 candidate->requirements & REQUIREMENT_B_TCH##TCHX ? "B" : "", \
-candidate->requirements & REQUIREMENT_B_TCH##TCHX ? "C" : "", \
+candidate->requirements & REQUIREMENT_C_TCH##TCHX ? "C" : "", \
 (candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 0 ? " 
not a candidate" : \
   ((candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 
REQUIREMENT_A_TCH##TCHX ? \
" more congestion" : \

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I15ce607f354918167ba81192230db63a5948c742
Gerrit-Change-Number: 19557
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-MessageType: newchange

Change in osmo-bsc[master]: hodec2: log requirements of best candidate, log rxlev in dBm

[neels]

neels has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/19560 )


Change subject: hodec2: log requirements of best candidate, log rxlev in dBm
..

hodec2: log requirements of best candidate, log rxlev in dBm

Instead of logging a hex value for the met requirements, fully expand the "ABC"
flags for both TCH/F and TCH/H.

>From HO_CANDIDATE_FMT/_ARGS, split off into REQUIREMENTS_FMT/_ARGS and use that
when logging the chosen HO candidates.

Also change the RX level to dBm, to match general logging and reduce confusion
between rxlev number variants in the log.

Change-Id: I1b30a6e98bdb4bd92e72864fafdd2f4f3ae3134c
---
M src/osmo-bsc/handover_decision_2.c
1 file changed, 21 insertions(+), 14 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/60/19560/1

diff --git a/src/osmo-bsc/handover_decision_2.c 
b/src/osmo-bsc/handover_decision_2.c
index 3191957..c818dbb 100644
--- a/src/osmo-bsc/handover_decision_2.c
+++ b/src/osmo-bsc/handover_decision_2.c
@@ -797,26 +797,30 @@
return trigger_remote_bss_ho(c, requirements);
 }

+#define REQUIREMENTS_FMT "[%s%s%s]%s"
+#define REQUIREMENTS_ARGS(REQUIREMENTS, TCHX) \
+(REQUIREMENTS) & REQUIREMENT_A_TCH##TCHX ? "A" : \
+   ((REQUIREMENTS) & REQUIREMENT_TCH##TCHX##_MASK) == 0? "-" : "", 
\
+(REQUIREMENTS) & REQUIREMENT_B_TCH##TCHX ? "B" : "", \
+(REQUIREMENTS) & REQUIREMENT_C_TCH##TCHX ? "C" : "", \
+((REQUIREMENTS) & REQUIREMENT_TCH##TCHX##_MASK) == 0 ? " not a 
candidate" : \
+  (((REQUIREMENTS) & REQUIREMENT_TCH##TCHX##_MASK) == 
REQUIREMENT_A_TCH##TCHX ? \
+   " more congestion" : \
+   ((REQUIREMENTS) & REQUIREMENT_B_TCH##TCHX ? \
+" good" : \
+ /* now has to be (REQUIREMENTS) & REQUIREMENT_C_TCHX != 0: */ 
\
+ " less-or-equal congestion"))
+
 /* verbosely log about a handover candidate */
 static inline void debug_candidate(struct ho_candidate *candidate,
   int8_t rxlev, int tchf_count, int tchh_count)
 {
struct gsm_lchan *lchan = candidate->lchan;

-#define HO_CANDIDATE_FMT(tchx, TCHX) "TCH/" #TCHX "={free %d (want %d), 
[%s%s%s]%s}"
+#define HO_CANDIDATE_FMT(tchx, TCHX) "TCH/" #TCHX "={free %d (want %d), " 
REQUIREMENTS_FMT "}"
 #define HO_CANDIDATE_ARGS(tchx, TCHX) \
 tch##tchx##_count, 
ho_get_hodec2_tch##tchx##_min_slots(candidate->bts->ho), \
-candidate->requirements & REQUIREMENT_A_TCH##TCHX ? "A" : \
-   (candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 0? 
"-" : "", \
-candidate->requirements & REQUIREMENT_B_TCH##TCHX ? "B" : "", \
-candidate->requirements & REQUIREMENT_C_TCH##TCHX ? "C" : "", \
-(candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 0 ? " 
not a candidate" : \
-  ((candidate->requirements & REQUIREMENT_TCH##TCHX##_MASK) == 
REQUIREMENT_A_TCH##TCHX ? \
-   " more congestion" : \
-   (candidate->requirements & REQUIREMENT_B_TCH##TCHX ? \
-" good" : \
- /* now has to be candidate->requirements & REQUIREMENT_C_TCHX 
!= 0: */ \
- " less-or-equal congestion"))
+REQUIREMENTS_ARGS(candidate->requirements, TCHX)

if (!candidate->bts && !candidate->cil)
LOGPHOLCHAN(lchan, LOGL_DEBUG, "Empty candidate\n");
@@ -1503,8 +1507,11 @@
if (log_check_level(DHODEC, LOGL_DEBUG)) {
LOGPHOBTS(bts, LOGL_DEBUG, "Considering %u candidates to solve 
congestion:\n", candidates);
for (i = 0; i < candidates; i++) {
-   LOGPHOCAND([i], LOGL_DEBUG, "#%d: req=0x%x 
avg-rxlev=%d\n",
-  i, clist[i].requirements, clist[i].avg);
+
+   LOGPHOCAND([i], LOGL_DEBUG, "#%d: req={TCH/F:" 
REQUIREMENTS_FMT ", TCH/H:" REQUIREMENTS_FMT "} avg-rxlev=%d dBm\n",
+  i, REQUIREMENTS_ARGS(clist[i].requirements, 
F),
+  REQUIREMENTS_ARGS(clist[i].requirements, H),
+  rxlev2dbm(clist[i].avg));
}
}


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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I1b30a6e98bdb4bd92e72864fafdd2f4f3ae3134c
Gerrit-Change-Number: 19560
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-MessageType: newchange

Change in osmo-ttcn3-hacks[master]: library/PCUIF_Types: version 10: add frequency hopping parameters

[neels]

neels has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19323 )

Change subject: library/PCUIF_Types: version 10: add frequency hopping 
parameters
..


Patch Set 11: Code-Review+1

i don't really understand deeply, but looks sane to me


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/19323
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: Idf11bc4ba3ff0b00b32f2beab8fd020c67119d05
Gerrit-Change-Number: 19323
Gerrit-PatchSet: 11
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: neels 
Gerrit-CC: pespin 
Gerrit-Comment-Date: Mon, 10 Aug 2020 16:31:14 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-bts[master]: common: Support setting rt prio through new libosmovty sched VTY cmds

[fixeria]

fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/19444 )

Change subject: common: Support setting rt prio through new libosmovty sched 
VTY cmds
..


Patch Set 2: Code-Review+1


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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ice46e406b84fa11afcc7ba31e521e7677df73cf3
Gerrit-Change-Number: 19444
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Comment-Date: Mon, 10 Aug 2020 16:01:53 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-trx[master]: Use new libosmovty cpu sched config features

[fixeria]

fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/19445 )

Change subject: Use new libosmovty cpu sched config features
..


Patch Set 2: Code-Review+1


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

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I3798603779b88ea37da03033cf7737a6e4751d6e
Gerrit-Change-Number: 19445
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Comment-Date: Mon, 10 Aug 2020 16:00:53 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in osmo-ccid-firmware[master]: proper card powerup handling

[Hoernchen]

Hello Jenkins Builder,

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

https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/19473

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

Change subject: proper card powerup handling
..

proper card powerup handling

Now that the timers work as intended other voltages (that lead to
timeouts) may be used.

Change-Id: I6105ce7f827aaec8ac36b940766fe05d47145edb
---
M ccid_common/ccid_slot_fsm.c
M ccid_common/cuart.h
M ccid_host/cuart_driver_tty.c
M ccid_host/cuart_fsm_test.c
M sysmoOCTSIM/cuart_driver_asf4_usart_async.c
5 files changed, 40 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-ccid-firmware 
refs/changes/73/19473/5
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/19473
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: I6105ce7f827aaec8ac36b940766fe05d47145edb
Gerrit-Change-Number: 19473
Gerrit-PatchSet: 5
Gerrit-Owner: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset

Change in osmo-gsm-manuals[master]: chapters: Introduce vty_cpu_sched.adoc chapter

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/19443 )

Change subject: chapters: Introduce vty_cpu_sched.adoc chapter
..

chapters: Introduce vty_cpu_sched.adoc chapter

Documents new features added in libosmocore.git Change-Id
If76a4bd2cc7b3c7adf5d84790a944d78be70e10a.

This file will be imported in user manuals of programs using the
mentioned libosmovty features.

Related: SYS#4986
Change-Id: Icd75769ef630c3fa985fc5e2154d5521689cdd3c
---
A common/chapters/vty_cpu_sched.adoc
1 file changed, 117 insertions(+), 0 deletions(-)

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



diff --git a/common/chapters/vty_cpu_sched.adoc 
b/common/chapters/vty_cpu_sched.adoc
new file mode 100644
index 000..f3a7c09
--- /dev/null
+++ b/common/chapters/vty_cpu_sched.adoc
@@ -0,0 +1,117 @@
+[[vty_cpu_sched]]
+== VTY Process and Thread management
+
+Most Osmocom programs provide, some support to tune some system
+settings related to the running process, its threads, its scheduling policies,
+etc.
+
+All of these settings can be configured through the VTY, either during startup
+by means of usual config files or through direct human interaction at the 
telnet
+VTY interface while the process is running.
+
+[[vty_cpu_sched_policy]]
+=== Scheduling Policy
+
+The scheduler to use as well as some of its properties (such as realtime
+priority) can be configured at any time for the entire process. This sort of
+functionality is useful in order to increase priority for processes running
+time-constrained procedures, such as those acting on the Um interface, like
+_osmo-trx_ or _osmo-bts_, where use of this feature is highly recommended.
+
+.Example: Set process to use RR scheduler
+
+cpu-sched
+ policy rr 1 <1>
+
+<1> Configure process to use _SCHED_RR_ policy with real time priority 1
+
+[[vty_cpu_sched_cpu_affinity_mask]]
+=== CPU-Affinity Mask
+
+Most operating systems allow for some sort of configuration on restricting the
+amount of CPUs a given process or thread can run on. The procedure is sometimes
+called as _cpu-pinning_ since it allows to keep different processes pinned on a
+subset of CPUs to make sure the scheduler won't run two CPU-hungry processes on
+the same CPU.
+
+The set of CPUs where each thread is allowed to run on is expressed by means of
+a bitmask in hexadecimal representation, where the right most bit relates to
+CPU 0, and the Nth most significant bit relates to CPU _N-1_. Setting the bit
+means the process is allowed to run on that CPU, while clearing it means the
+process is forbidden to run on that CPU.
+
+Hence, for instance a cpu-affinity mask of _0x00_ means the thread is not
+allowed on any CPU, which will cause the thread to stall until a new value is
+applied. A mask of _0x01_ means the thread is only allowed to run on the 1st 
CPU
+(CPU 0). A mask of _0xff00_ means CPUs 8-15 are allowed, while 0-7 are not.
+
+For single-threaded processes (most of Osmocom are), it is usually enough to 
set
+this line in VTY config file as follows:
+
+cpu-sched
+ cpu-affinity self 0x01 <1>
+
+<1> Allow main thread (the one managing the VTY) only on CPU 0
+
+Or otherwise:
+
+cpu-sched
+ cpu-affinity all 0x01 <1>
+
+<1> Allow all threads only on CPU 0
+
+
+For multi-threaded processes, it may be desired to run some threads on a subset
+of CPUs while another subset may run on another one. In order to identify
+threads, one can either use the TID of the thread (each thread has its own PID
+in Linux), or its specific Thread Name in case it has been set by the
+application.
+
+The related information on all threads available in the process can be listed
+through VTY. This allows identifying quickly the different threads, its current
+cpu-affinity mask, etc.
+
+.Example: Get osmo-trx Thread list information from VTY
+
+OsmoTRX> show cpu-sched threads
+Thread list for PID 338609:
+ TID: 338609, NAME: 'osmo-trx-uhd', cpu-affinity: 0x3
+ TID: 338610, NAME: 'osmo-trx-uhd', cpu-affinity: 0x3
+ TID: 338611, NAME: 'osmo-trx-uhd', cpu-affinity: 0x3
+ TID: 338629, NAME: 'osmo-trx-uhd', cpu-affinity: 0x3
+ TID: 338630, NAME: 'osmo-trx-uhd', cpu-affinity: 0x3
+ TID: 338631, NAME: 'osmo-trx-uhd', cpu-affinity: 0x3
+ TID: 338634, NAME: 'UHDAsyncEvent', cpu-affinity: 0x3
+ TID: 338635, NAME: 'TxLower', cpu-affinity: 0x3
+ TID: 338636, NAME: 'RxLower', cpu-affinity: 0x3
+ TID: 338637, NAME: 'RxUpper0', cpu-affinity: 0x3
+ TID: 338638, NAME: 'TxUpper0', cpu-affinity: 0x3
+ TID: 338639, NAME: 'RxUpper1', cpu-affinity: 0x3
+ TID: 338640, NAME: 'TxUpper1', cpu-affinity: 0x3
+
+
+At runtime, one can change the cpu-affinity mask for a given thread identifying
+it by either TID or name:
+
+.Example: Set CPU-affinity from VTY telnet interface
+
+OsmoTRX> cpu-affinity TxLower 0x02 <1>
+OsmoTRX> cpu-affinity TxLower 0x03 <2>
+
+<1> Allow thread named _TxLower_ 

Change in osmo-gsm-manuals[master]: chapters: Introduce vty_cpu_sched.adoc chapter

[laforge]

laforge has uploaded a new patch set (#3) to the change originally created by 
pespin. ( https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/19443 )

Change subject: chapters: Introduce vty_cpu_sched.adoc chapter
..

chapters: Introduce vty_cpu_sched.adoc chapter

Documents new features added in libosmocore.git Change-Id
If76a4bd2cc7b3c7adf5d84790a944d78be70e10a.

This file will be imported in user manuals of programs using the
mentioned libosmovty features.

Related: SYS#4986
Change-Id: Icd75769ef630c3fa985fc5e2154d5521689cdd3c
---
A common/chapters/vty_cpu_sched.adoc
1 file changed, 117 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-manuals 
refs/changes/43/19443/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/19443
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-manuals
Gerrit-Branch: master
Gerrit-Change-Id: Icd75769ef630c3fa985fc5e2154d5521689cdd3c
Gerrit-Change-Number: 19443
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge 
Gerrit-MessageType: newpatchset

Change in osmo-gsm-manuals[master]: chapters: Introduce vty_cpu_sched.adoc chapter

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/19443 )

Change subject: chapters: Introduce vty_cpu_sched.adoc chapter
..


Patch Set 3: Code-Review+2


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

Gerrit-Project: osmo-gsm-manuals
Gerrit-Branch: master
Gerrit-Change-Id: Icd75769ef630c3fa985fc5e2154d5521689cdd3c
Gerrit-Change-Number: 19443
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 10 Aug 2020 14:35:14 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Change in libosmocore[master]: vty: Introduce support to set cpu-affinity and scheduler policy

[laforge]

laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/19442 )

Change subject: vty: Introduce support to set cpu-affinity and scheduler policy
..

vty: Introduce support to set cpu-affinity and scheduler policy

Process willing to support this kind of configuration through VTY simply
need to call "osmo_sched_vty_init(tall_ctx);" during startup to register
the commands.
For multithreaded processes, new threads willing to get their
cpu-affinity mask according to VTY config should call
osmo_sched_vty_apply_localthread() (potentially after
setting the thread name through pthread_setname_np()).

Related: SYS#4986
Change-Id: If76a4bd2cc7b3c7adf5d84790a944d78be70e10a
---
M include/Makefile.am
M include/osmocom/vty/command.h
A include/osmocom/vty/cpu_sched_vty.h
M src/vty/Makefile.am
A src/vty/cpu_sched_vty.c
5 files changed, 714 insertions(+), 1 deletion(-)

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



diff --git a/include/Makefile.am b/include/Makefile.am
index 7af7e01..b0a72bc 100644
--- a/include/Makefile.am
+++ b/include/Makefile.am
@@ -164,6 +164,7 @@
   osmocom/vty/vector.h \
   osmocom/vty/vty.h \
   osmocom/vty/ports.h \
+  osmocom/vty/cpu_sched_vty.h \
   osmocom/vty/tdef_vty.h \
   osmocom/ctrl/control_vty.h
 endif
diff --git a/include/osmocom/vty/command.h b/include/osmocom/vty/command.h
index 29206ce..e3692dc 100644
--- a/include/osmocom/vty/command.h
+++ b/include/osmocom/vty/command.h
@@ -97,6 +97,7 @@
L_CS7_SCCPADDR_NODE,/*!< SS7 SCCP Address */
L_CS7_SCCPADDR_GT_NODE, /*!< SS7 SCCP Global Title */

+   L_CPU_SCHED_NODE,   /*!< CPU Sched related options node */
/*
 * When adding new nodes to the libosmocore project, these nodes can be
 * used to avoid ABI changes for unrelated projects.
diff --git a/include/osmocom/vty/cpu_sched_vty.h 
b/include/osmocom/vty/cpu_sched_vty.h
new file mode 100644
index 000..171f168
--- /dev/null
+++ b/include/osmocom/vty/cpu_sched_vty.h
@@ -0,0 +1,37 @@
+/*! \file cpu_sched_vty.h
+ * API to CPU / Threading / Scheduler properties from VTY configuration.
+ */
+/* (C) 2020 by sysmocom - s.f.m.c. GmbH 
+ *
+ * Author: Pau Espin Pedrol 
+ *
+ * All Rights Reserved
+ *
+ * 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 2 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 .
+ *
+ * SPDX-License-Identifier: GPL-2.0+
+ */
+#pragma once
+
+#include 
+
+/*! \defgroup cpu_sched_VTY Configuration
+ * @{
+ * \file cpu_sched_vty.h
+ */
+
+void osmo_cpu_sched_vty_init(void *tall_ctx);
+int osmo_cpu_sched_vty_apply_localthread(void);
+
+/*! @} */
diff --git a/src/vty/Makefile.am b/src/vty/Makefile.am
index 35350cc..81ff104 100644
--- a/src/vty/Makefile.am
+++ b/src/vty/Makefile.am
@@ -12,7 +12,7 @@
 libosmovty_la_SOURCES = buffer.c command.c vty.c vector.c utils.c \
telnet_interface.c logging_vty.c stats_vty.c \
fsm_vty.c talloc_ctx_vty.c \
-   tdef_vty.c
+   cpu_sched_vty.c tdef_vty.c
 libosmovty_la_LDFLAGS = -version-info $(LIBVERSION) -no-undefined
 libosmovty_la_LIBADD = $(top_builddir)/src/libosmocore.la $(TALLOC_LIBS)
 endif
diff --git a/src/vty/cpu_sched_vty.c b/src/vty/cpu_sched_vty.c
new file mode 100644
index 000..3b6067a
--- /dev/null
+++ b/src/vty/cpu_sched_vty.c
@@ -0,0 +1,674 @@
+/*! \file cpu_sched_vty.c
+ * Implementation to CPU / Threading / Scheduler properties from VTY 
configuration.
+ */
+/* (C) 2020 by sysmocom - s.f.m.c. GmbH 
+ *
+ * Author: Pau Espin Pedrol 
+ *
+ * All Rights Reserved
+ *
+ * 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 2 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 

Change in libosmocore[master]: vty: Introduce support to set cpu-affinity and scheduler policy

[laforge]

laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/19442 )

Change subject: vty: Introduce support to set cpu-affinity and scheduler policy
..


Patch Set 5: Code-Review+2


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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: If76a4bd2cc7b3c7adf5d84790a944d78be70e10a
Gerrit-Change-Number: 19442
Gerrit-PatchSet: 5
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 10 Aug 2020 14:11:16 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Build failure of network:osmocom:nightly/ulfius in Debian_Unstable/x86_64

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/ulfius/Debian_Unstable/x86_64

Package network:osmocom:nightly/ulfius failed to build in Debian_Unstable/x86_64

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

Last lines of build log:
[   83s] /usr/src/packages/BUILD/src/ulfius.c: In function 
'ulfius_run_mhd_daemon':
[   83s] /usr/src/packages/BUILD/src/ulfius.c:839:48: error: passing argument 5 
of 'MHD_start_daemon' from incompatible pointer type 
[-Werror=incompatible-pointer-types]
[   83s]   839 |   mhd_flags, u_instance->port, NULL, NULL, 
_webservice_dispatcher, (void *)u_instance,
[   83s]   |
^
[   83s]   ||
[   83s]   |int (*)(void *, 
struct MHD_Connection *, const char *, const char *, const char *, const char 
*, size_t *, void **) {aka int (*)(void *, struct MHD_Connection *, const char 
*, const char *, const char *, const char *, long unsigned int *, void **)}
[   83s] In file included from /usr/src/packages/BUILD/include/ulfius.h:46,
[   83s]  from /usr/src/packages/BUILD/include/u_private.h:29,
[   83s]  from /usr/src/packages/BUILD/src/ulfius.c:29:
[   83s] /usr/include/microhttpd.h:2428:45: note: expected 
'MHD_AccessHandlerCallback' {aka 'enum MHD_Result (*)(void *, struct 
MHD_Connection *, const char *, const char *, const char *, const char *, long 
unsigned int *, void **)'} but argument is of type 'int (*)(void *, struct 
MHD_Connection *, const char *, const char *, const char *, const char *, 
size_t *, void **)' {aka 'int (*)(void *, struct MHD_Connection *, const char 
*, const char *, const char *, const char *, long unsigned int *, void **)'}
[   83s]  2428 |   MHD_AccessHandlerCallback dh, void *dh_cls,
[   83s]   |   ~~^~
[   84s] cc1: all warnings being treated as errors
[   84s] make[3]: *** [CMakeFiles/ulfius.dir/build.make:144: 
CMakeFiles/ulfius.dir/src/ulfius.c.o] Error 1
[   84s] make[3]: Leaving directory '/usr/src/packages/BUILD/build'
[   84s] make[2]: *** [CMakeFiles/Makefile2:87: CMakeFiles/ulfius.dir/all] 
Error 2
[   84s] make[2]: Leaving directory '/usr/src/packages/BUILD/build'
[   84s] make[1]: *** [Makefile:166: all] Error 2
[   84s] make[1]: Leaving directory '/usr/src/packages/BUILD/build'
[   84s] dh_auto_build: error: cd build && make -j6 "INSTALL=install 
--strip-program=true" returned exit code 2
[   84s] make: *** [debian/rules:9: build] Error 25
[   84s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[   84s] ### VM INTERACTION START ###
[   87s] [   77.903937] sysrq: Power Off
[   87s] [   77.908639] reboot: Power down
[   87s] ### VM INTERACTION END ###
[   87s] 
[   87s] sheep86 failed "build ulfius_2.5.2-4.1.dsc" at Mon Aug 10 11:54:27 UTC 
2020.
[   87s] 

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

Build failure of network:osmocom:latest/ulfius in Debian_Unstable/x86_64

[OBS Notification]

Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:latest/ulfius/Debian_Unstable/x86_64

Package network:osmocom:latest/ulfius failed to build in Debian_Unstable/x86_64

Check out the package for editing:
  osc checkout network:osmocom:latest ulfius

Last lines of build log:
[  121s] /usr/src/packages/BUILD/src/ulfius.c:839:48: error: passing argument 5 
of 'MHD_start_daemon' from incompatible pointer type 
[-Werror=incompatible-pointer-types]
[  121s]   839 |   mhd_flags, u_instance->port, NULL, NULL, 
_webservice_dispatcher, (void *)u_instance,
[  121s]   |
^
[  121s]   ||
[  121s]   |int (*)(void *, 
struct MHD_Connection *, const char *, const char *, const char *, const char 
*, size_t *, void **) {aka int (*)(void *, struct MHD_Connection *, const char 
*, const char *, const char *, const char *, long unsigned int *, void **)}
[  121s] In file included from /usr/src/packages/BUILD/include/ulfius.h:46,
[  121s]  from /usr/src/packages/BUILD/include/u_private.h:29,
[  121s]  from /usr/src/packages/BUILD/src/ulfius.c:29:
[  121s] /usr/include/microhttpd.h:2428:45: note: expected 
'MHD_AccessHandlerCallback' {aka 'enum MHD_Result (*)(void *, struct 
MHD_Connection *, const char *, const char *, const char *, const char *, long 
unsigned int *, void **)'} but argument is of type 'int (*)(void *, struct 
MHD_Connection *, const char *, const char *, const char *, const char *, 
size_t *, void **)' {aka 'int (*)(void *, struct MHD_Connection *, const char 
*, const char *, const char *, const char *, long unsigned int *, void **)'}
[  121s]  2428 |   MHD_AccessHandlerCallback dh, void *dh_cls,
[  121s]   |   ~~^~
[  122s] cc1: all warnings being treated as errors
[  122s] make[3]: *** [CMakeFiles/ulfius.dir/build.make:144: 
CMakeFiles/ulfius.dir/src/ulfius.c.o] Error 1
[  122s] make[3]: *** Waiting for unfinished jobs
[  122s] make[3]: Leaving directory '/usr/src/packages/BUILD/build'
[  122s] make[2]: *** [CMakeFiles/Makefile2:87: CMakeFiles/ulfius.dir/all] 
Error 2
[  122s] make[2]: Leaving directory '/usr/src/packages/BUILD/build'
[  122s] make[1]: *** [Makefile:166: all] Error 2
[  122s] make[1]: Leaving directory '/usr/src/packages/BUILD/build'
[  122s] dh_auto_build: error: cd build && make -j8 "INSTALL=install 
--strip-program=true" returned exit code 2
[  122s] make: *** [debian/rules:9: build] Error 25
[  122s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  122s] ### VM INTERACTION START ###
[  125s] [  114.713127] sysrq: Power Off
[  125s] [  114.723216] reboot: Power down
[  125s] ### VM INTERACTION END ###
[  125s] 
[  125s] lamb21 failed "build ulfius_2.5.2-4.1.dsc" at Mon Aug 10 11:54:51 UTC 
2020.
[  125s] 

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