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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-sgsn/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  272s] [455/513] installing python3-3.9.0-3
[  272s] Processing triggers for man-db (2.9.3-2) ...
[  273s] [456/513] installing texlive-plain-generic-2020.20201129-1
[  277s] Processing triggers for tex-common (6.15) ...
[  277s] Running mktexlsr. This may take some time... done.
[  278s] [457/513] installing libosmo-netif-dev-1.0.0.8.bd77
[  278s] [458/513] installing texlive-latex-base-2020.20201129-2
[  278s] Processing triggers for man-db (2.9.3-2) ...
[  279s] Processing triggers for tex-common (6.15) ...
[  279s] Running mktexlsr. This may take some time... done.
[  280s] Running updmap-sys. This may take some time... done.
[  280s] Running mktexlsr /var/lib/texmf ... done.
[  280s] Building format(s) --all.
[  289s]This may take some time... 
[  289s] fmtutil failed. Output has been stored in
[  289s] /tmp/fmtutil.HJgsbIqc
[  289s] Please include this file if you report a bug.
[  289s] dpkg: error processing package tex-common (--install):
[  289s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  289s] Errors were encountered while processing:
[  289s]  tex-common
[  289s] exit ...
[  289s] ### VM INTERACTION START ###
[  292s] [  258.300858] sysrq: Power Off
[  292s] [  258.307440] reboot: Power down
[  292s] ### VM INTERACTION END ###
[  292s] 
[  292s] cloud129 failed "build osmo-sgsn_1.6.0.64.9e58.dsc" at Tue Dec  1 
05:52:44 UTC 2020.
[  292s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-msc/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  130s] [458/516] installing python3-3.9.0-3
[  131s] Processing triggers for man-db (2.9.3-2) ...
[  131s] [459/516] installing texlive-plain-generic-2020.20201129-1
[  134s] Processing triggers for tex-common (6.15) ...
[  134s] Running mktexlsr. This may take some time... done.
[  134s] [460/516] installing libosmo-netif-dev-1.0.0.8.bd77
[  134s] [461/516] installing texlive-latex-base-2020.20201129-2
[  134s] Processing triggers for man-db (2.9.3-2) ...
[  134s] Processing triggers for tex-common (6.15) ...
[  135s] Running mktexlsr. This may take some time... done.
[  135s] Running updmap-sys. This may take some time... done.
[  135s] Running mktexlsr /var/lib/texmf ... done.
[  135s] Building format(s) --all.
[  141s]This may take some time... 
[  141s] fmtutil failed. Output has been stored in
[  141s] /tmp/fmtutil.jWWK6yrI
[  141s] Please include this file if you report a bug.
[  141s] dpkg: error processing package tex-common (--install):
[  141s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  141s] Errors were encountered while processing:
[  141s]  tex-common
[  141s] exit ...
[  141s] ### VM INTERACTION START ###
[  144s] [  129.115815] sysrq: Power Off
[  144s] [  129.119877] reboot: Power down
[  144s] ### VM INTERACTION END ###
[  144s] 
[  144s] sheep85 failed "build osmo-msc_1.6.1.87.e917b.dsc" at Tue Dec  1 
05:40:06 UTC 2020.
[  144s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-ggsn/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  181s] Running mktexlsr /var/lib/texmf ... done.
[  181s] [436/492] installing python3-3.9.0-3
[  181s] Processing triggers for man-db (2.9.3-2) ...
[  181s] [437/492] installing texlive-plain-generic-2020.20201129-1
[  184s] Processing triggers for tex-common (6.15) ...
[  185s] Running mktexlsr. This may take some time... done.
[  185s] [438/492] installing texlive-latex-base-2020.20201129-2
[  185s] Processing triggers for man-db (2.9.3-2) ...
[  186s] Processing triggers for tex-common (6.15) ...
[  186s] Running mktexlsr. This may take some time... done.
[  186s] Running updmap-sys. This may take some time... done.
[  186s] Running mktexlsr /var/lib/texmf ... done.
[  187s] Building format(s) --all.
[  194s]This may take some time... 
[  194s] fmtutil failed. Output has been stored in
[  194s] /tmp/fmtutil.GVKgyIrr
[  194s] Please include this file if you report a bug.
[  194s] dpkg: error processing package tex-common (--install):
[  194s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  194s] Errors were encountered while processing:
[  194s]  tex-common
[  194s] exit ...
[  194s] ### VM INTERACTION START ###
[  197s] [  177.616826] sysrq: Power Off
[  197s] [  177.627636] reboot: Power down
[  197s] ### VM INTERACTION END ###
[  197s] 
[  197s] lamb12 failed "build osmo-ggsn_1.6.0.13.1230.dsc" at Tue Dec  1 
05:37:56 UTC 2020.
[  197s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-hlr/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  178s] [445/502] installing python3-3.9.0-3
[  178s] Processing triggers for man-db (2.9.3-2) ...
[  179s] [446/502] installing texlive-plain-generic-2020.20201129-1
[  182s] Processing triggers for tex-common (6.15) ...
[  182s] Running mktexlsr. This may take some time... done.
[  182s] [447/502] installing libosmo-netif-dev-1.0.0.8.bd77
[  182s] [448/502] installing texlive-latex-base-2020.20201129-2
[  183s] Processing triggers for man-db (2.9.3-2) ...
[  183s] Processing triggers for tex-common (6.15) ...
[  183s] Running mktexlsr. This may take some time... done.
[  184s] Running updmap-sys. This may take some time... done.
[  184s] Running mktexlsr /var/lib/texmf ... done.
[  184s] Building format(s) --all.
[  191s]This may take some time... 
[  191s] fmtutil failed. Output has been stored in
[  191s] /tmp/fmtutil.VHOYGTSY
[  191s] Please include this file if you report a bug.
[  191s] dpkg: error processing package tex-common (--install):
[  191s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  191s] Errors were encountered while processing:
[  191s]  tex-common
[  191s] exit ...
[  191s] ### VM INTERACTION START ###
[  194s] [  175.383169] sysrq: Power Off
[  194s] [  175.389959] reboot: Power down
[  194s] ### VM INTERACTION END ###
[  194s] 
[  194s] lamb23 failed "build osmo-hlr_1.2.0.67.dac8.dsc" at Tue Dec  1 
05:30:32 UTC 2020.
[  194s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-mgw/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  133s] [442/499] installing python3-3.9.0-3
[  133s] Processing triggers for man-db (2.9.3-2) ...
[  133s] [443/499] installing texlive-plain-generic-2020.20201129-1
[  136s] Processing triggers for tex-common (6.15) ...
[  136s] Running mktexlsr. This may take some time... done.
[  136s] [444/499] installing libosmo-netif-dev-1.0.0.8.bd77
[  136s] [445/499] installing texlive-latex-base-2020.20201129-2
[  137s] Processing triggers for man-db (2.9.3-2) ...
[  137s] Processing triggers for tex-common (6.15) ...
[  137s] Running mktexlsr. This may take some time... done.
[  138s] Running updmap-sys. This may take some time... done.
[  138s] Running mktexlsr /var/lib/texmf ... done.
[  138s] Building format(s) --all.
[  144s]This may take some time... 
[  144s] fmtutil failed. Output has been stored in
[  144s] /tmp/fmtutil.ckw8bbAn
[  144s] Please include this file if you report a bug.
[  144s] dpkg: error processing package tex-common (--install):
[  144s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  144s] Errors were encountered while processing:
[  144s]  tex-common
[  144s] exit ...
[  144s] ### VM INTERACTION START ###
[  147s] [  122.763822] sysrq: Power Off
[  147s] [  122.765654] reboot: Power down
[  147s] ### VM INTERACTION END ###
[  147s] 
[  147s] wildcard2 failed "build osmo-mgw_1.7.0.130.3c29.dsc" at Tue Dec  1 
05:28:20 UTC 2020.
[  147s] 

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


Build failure of network:osmocom:nightly/libosmo-sccp in Debian_Unstable/x86_64

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libosmo-sccp/Debian_Unstable/x86_64

Package network:osmocom:nightly/libosmo-sccp failed to build in 
Debian_Unstable/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly libosmo-sccp

Last lines of build log:
[  216s] [445/502] installing python3-3.9.0-3
[  216s] Processing triggers for man-db (2.9.3-2) ...
[  217s] [446/502] installing texlive-plain-generic-2020.20201129-1
[  220s] Processing triggers for tex-common (6.15) ...
[  220s] Running mktexlsr. This may take some time... done.
[  220s] [447/502] installing libosmo-netif-dev-1.0.0.8.bd77
[  220s] [448/502] installing texlive-latex-base-2020.20201129-2
[  221s] Processing triggers for man-db (2.9.3-2) ...
[  221s] Processing triggers for tex-common (6.15) ...
[  221s] Running mktexlsr. This may take some time... done.
[  222s] Running updmap-sys. This may take some time... done.
[  222s] Running mktexlsr /var/lib/texmf ... done.
[  222s] Building format(s) --all.
[  229s]This may take some time... 
[  229s] fmtutil failed. Output has been stored in
[  229s] /tmp/fmtutil.2Bd3v4Vg
[  229s] Please include this file if you report a bug.
[  229s] dpkg: error processing package tex-common (--install):
[  229s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  229s] Errors were encountered while processing:
[  229s]  tex-common
[  229s] exit ...
[  229s] ### VM INTERACTION START ###
[  232s] [  213.683223] sysrq: Power Off
[  232s] [  213.688206] reboot: Power down
[  232s] ### VM INTERACTION END ###
[  232s] 
[  232s] lamb15 failed "build libosmo-sccp_1.3.0.25.48e2.dsc" at Tue Dec  1 
05:27:54 UTC 2020.
[  232s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-gsm-manuals/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  171s] Running mktexlsr /var/lib/texmf ... done.
[  171s] [416/471] installing python3-3.9.0-3
[  171s] Processing triggers for man-db (2.9.3-2) ...
[  171s] [417/471] installing texlive-plain-generic-2020.20201129-1
[  174s] Processing triggers for tex-common (6.15) ...
[  175s] Running mktexlsr. This may take some time... done.
[  175s] [418/471] installing texlive-latex-base-2020.20201129-2
[  175s] Processing triggers for man-db (2.9.3-2) ...
[  176s] Processing triggers for tex-common (6.15) ...
[  176s] Running mktexlsr. This may take some time... done.
[  176s] Running updmap-sys. This may take some time... done.
[  176s] Running mktexlsr /var/lib/texmf ... done.
[  177s] Building format(s) --all.
[  184s]This may take some time... 
[  184s] fmtutil failed. Output has been stored in
[  184s] /tmp/fmtutil.rv6IIUoz
[  184s] Please include this file if you report a bug.
[  184s] dpkg: error processing package tex-common (--install):
[  184s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  184s] Errors were encountered while processing:
[  184s]  tex-common
[  184s] exit ...
[  184s] ### VM INTERACTION START ###
[  187s] [  170.185086] sysrq: Power Off
[  187s] [  170.192646] reboot: Power down
[  187s] ### VM INTERACTION END ###
[  187s] 
[  187s] lamb13 failed "build osmo-gsm-manuals-dev_0.3.0.37.c507.dsc" at Tue 
Dec  1 05:10:51 UTC 2020.
[  187s] 

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


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

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

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

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

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

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-smlc/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  138s] [438/496] installing python3-3.9.0-3
[  138s] Processing triggers for man-db (2.9.3-2) ...
[  139s] [439/496] installing texlive-plain-generic-2020.20201129-1
[  142s] Processing triggers for tex-common (6.15) ...
[  142s] Running mktexlsr. This may take some time... done.
[  142s] [440/496] installing libosmo-netif-dev-1.0.0.8.bd77
[  142s] [441/496] installing texlive-latex-base-2020.20201129-2
[  142s] Processing triggers for man-db (2.9.3-2) ...
[  142s] Processing triggers for tex-common (6.15) ...
[  143s] Running mktexlsr. This may take some time... done.
[  143s] Running updmap-sys. This may take some time... done.
[  143s] Running mktexlsr /var/lib/texmf ... done.
[  143s] Building format(s) --all.
[  149s]This may take some time... 
[  149s] fmtutil failed. Output has been stored in
[  149s] /tmp/fmtutil.UfCh5zwa
[  149s] Please include this file if you report a bug.
[  149s] dpkg: error processing package tex-common (--install):
[  149s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  149s] Errors were encountered while processing:
[  149s]  tex-common
[  149s] exit ...
[  149s] ### VM INTERACTION START ###
[  152s] [  136.949253] sysrq: Power Off
[  152s] [  136.957339] reboot: Power down
[  152s] ### VM INTERACTION END ###
[  152s] 
[  152s] sheep84 failed "build osmo-smlc_0.1.0.5.5dad.dsc" at Tue Dec  1 
04:25:24 UTC 2020.
[  152s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bsc/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  105s] [449/507] installing python3-3.9.0-3
[  106s] Processing triggers for man-db (2.9.3-2) ...
[  106s] [450/507] installing texlive-plain-generic-2020.20201129-1
[  108s] Processing triggers for tex-common (6.15) ...
[  108s] Running mktexlsr. This may take some time... done.
[  108s] [451/507] installing libosmo-netif-dev-1.0.0.8.bd77
[  108s] [452/507] installing texlive-latex-base-2020.20201129-2
[  109s] Processing triggers for man-db (2.9.3-2) ...
[  109s] Processing triggers for tex-common (6.15) ...
[  109s] Running mktexlsr. This may take some time... done.
[  109s] Running updmap-sys. This may take some time... done.
[  109s] Running mktexlsr /var/lib/texmf ... done.
[  110s] Building format(s) --all.
[  114s]This may take some time... 
[  114s] fmtutil failed. Output has been stored in
[  114s] /tmp/fmtutil.o3JVzCCt
[  114s] Please include this file if you report a bug.
[  114s] dpkg: error processing package tex-common (--install):
[  114s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  114s] Errors were encountered while processing:
[  114s]  tex-common
[  114s] exit ...
[  114s] ### VM INTERACTION START ###
[  117s] [  105.700740] sysrq: Power Off
[  117s] [  105.701974] reboot: Power down
[  117s] ### VM INTERACTION END ###
[  117s] 
[  117s] build70 failed "build osmo-bsc_1.6.0.413.c93b.dsc" at Tue Dec  1 
04:24:50 UTC 2020.
[  117s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-uecups/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  174s] [442/499] installing python3-3.9.0-3
[  174s] Processing triggers for man-db (2.9.3-2) ...
[  174s] [443/499] installing texlive-plain-generic-2020.20201129-1
[  178s] Processing triggers for tex-common (6.15) ...
[  178s] Running mktexlsr. This may take some time... done.
[  178s] [444/499] installing libosmo-netif-dev-1.0.0.8.bd77
[  178s] [445/499] installing texlive-latex-base-2020.20201129-2
[  179s] Processing triggers for man-db (2.9.3-2) ...
[  179s] Processing triggers for tex-common (6.15) ...
[  179s] Running mktexlsr. This may take some time... done.
[  180s] Running updmap-sys. This may take some time... done.
[  180s] Running mktexlsr /var/lib/texmf ... done.
[  180s] Building format(s) --all.
[  187s]This may take some time... 
[  187s] fmtutil failed. Output has been stored in
[  187s] /tmp/fmtutil.GOIDfsUh
[  187s] Please include this file if you report a bug.
[  187s] dpkg: error processing package tex-common (--install):
[  187s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  187s] Errors were encountered while processing:
[  187s]  tex-common
[  187s] exit ...
[  187s] ### VM INTERACTION START ###
[  190s] [  169.181599] sysrq: Power Off
[  190s] [  169.184615] reboot: Power down
[  190s] ### VM INTERACTION END ###
[  190s] 
[  190s] old-atreju2 failed "build osmo-uecups_0.1.3.1.8362.dsc" at Tue Dec  1 
04:19:51 UTC 2020.
[  190s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-trx/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  175s] Processing triggers for libc-bin (2.31-4) ...
[  175s] [575/656] installing python3-3.9.0-3
[  175s] Processing triggers for man-db (2.9.3-2) ...
[  175s] [576/656] installing texlive-plain-generic-2020.20201129-1
[  178s] Processing triggers for tex-common (6.15) ...
[  178s] Running mktexlsr. This may take some time... done.
[  178s] [577/656] installing texlive-latex-base-2020.20201129-2
[  178s] Processing triggers for man-db (2.9.3-2) ...
[  178s] Processing triggers for tex-common (6.15) ...
[  179s] Running mktexlsr. This may take some time... done.
[  179s] Running updmap-sys. This may take some time... done.
[  179s] Running mktexlsr /var/lib/texmf ... done.
[  179s] Building format(s) --all.
[  184s]This may take some time... 
[  184s] fmtutil failed. Output has been stored in
[  184s] /tmp/fmtutil.ZygzPj2a
[  184s] Please include this file if you report a bug.
[  184s] dpkg: error processing package tex-common (--install):
[  184s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  184s] Errors were encountered while processing:
[  184s]  tex-common
[  184s] exit ...
[  184s] ### VM INTERACTION START ###
[  187s] [  164.451151] sysrq: Power Off
[  187s] [  164.453522] reboot: Power down
[  187s] ### VM INTERACTION END ###
[  187s] 
[  187s] hci-cnode1-m0 failed "build osmo-trx_1.2.0.114.57db.dsc" at Tue Dec  1 
04:09:40 UTC 2020.
[  187s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-sip-connector/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  234s] Running mktexlsr /var/lib/texmf ... done.
[  234s] [454/515] installing python3-3.9.0-3
[  234s] Processing triggers for man-db (2.9.3-2) ...
[  235s] [455/515] installing texlive-plain-generic-2020.20201129-1
[  239s] Processing triggers for tex-common (6.15) ...
[  239s] Running mktexlsr. This may take some time... done.
[  239s] [456/515] installing texlive-latex-base-2020.20201129-2
[  240s] Processing triggers for man-db (2.9.3-2) ...
[  241s] Processing triggers for tex-common (6.15) ...
[  241s] Running mktexlsr. This may take some time... done.
[  242s] Running updmap-sys. This may take some time... done.
[  242s] Running mktexlsr /var/lib/texmf ... done.
[  242s] Building format(s) --all.
[  250s]This may take some time... 
[  250s] fmtutil failed. Output has been stored in
[  250s] /tmp/fmtutil.fg5n9hAY
[  250s] Please include this file if you report a bug.
[  251s] dpkg: error processing package tex-common (--install):
[  251s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  251s] Errors were encountered while processing:
[  251s]  tex-common
[  251s] exit ...
[  251s] ### VM INTERACTION START ###
[  254s] [  232.692789] sysrq: Power Off
[  254s] [  232.704390] reboot: Power down
[  254s] ### VM INTERACTION END ###
[  254s] 
[  254s] lamb28 failed "build osmo-sip-connector_1.4.1.6.05ee.dsc" at Tue Dec  
1 04:04:24 UTC 2020.
[  254s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-sgsn/Debian_8.0/x86_64

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

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

Last lines of build log:
[  232s] make[4]: Entering directory '/usr/src/packages/BUILD/src/gbproxy'
[  232s]   CC   gb_proxy.o
[  232s] gb_proxy.c: In function 'gbprox_relay2nse':
[  232s] gb_proxy.c:846:17: error: 'struct ' has no member named 
'link_selector'
[  232s]nsp.u.unitdata.link_selector = tlli;
[  232s]  ^
[  232s] Makefile:482: recipe for target 'gb_proxy.o' failed
[  232s] make[4]: *** [gb_proxy.o] Error 1
[  232s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/gbproxy'
[  232s] Makefile:383: recipe for target 'all-recursive' failed
[  232s] make[3]: *** [all-recursive] Error 1
[  232s] make[3]: Leaving directory '/usr/src/packages/BUILD/src'
[  232s] Makefile:449: recipe for target 'all-recursive' failed
[  232s] make[2]: *** [all-recursive] Error 1
[  232s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  232s] Makefile:380: recipe for target 'all' failed
[  232s] make[1]: *** [all] Error 2
[  232s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  232s] dh_auto_build: make -j1 returned exit code 2
[  232s] debian/rules:45: recipe for target 'build' failed
[  232s] make: *** [build] Error 2
[  232s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  232s] ### VM INTERACTION START ###
[  233s] Powering off.
[  233s] [  217.443598] reboot: Power down
[  233s] ### VM INTERACTION END ###
[  233s] 
[  233s] lamb03 failed "build osmo-sgsn_1.6.0.64.9e58.dsc" at Tue Dec  1 
04:03:21 UTC 2020.
[  233s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-pcu/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  131s] Running mktexlsr /var/lib/texmf ... done.
[  131s] [434/490] installing python3-3.9.0-3
[  131s] Processing triggers for man-db (2.9.3-2) ...
[  131s] [435/490] installing texlive-plain-generic-2020.20201129-1
[  134s] Processing triggers for tex-common (6.15) ...
[  134s] Running mktexlsr. This may take some time... done.
[  134s] [436/490] installing texlive-latex-base-2020.20201129-2
[  135s] Processing triggers for man-db (2.9.3-2) ...
[  135s] Processing triggers for tex-common (6.15) ...
[  135s] Running mktexlsr. This may take some time... done.
[  136s] Running updmap-sys. This may take some time... done.
[  136s] Running mktexlsr /var/lib/texmf ... done.
[  136s] Building format(s) --all.
[  141s]This may take some time... 
[  141s] fmtutil failed. Output has been stored in
[  141s] /tmp/fmtutil.lfzIxjec
[  141s] Please include this file if you report a bug.
[  141s] dpkg: error processing package tex-common (--install):
[  141s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  141s] Errors were encountered while processing:
[  141s]  tex-common
[  141s] exit ...
[  141s] ### VM INTERACTION START ###
[  144s] [  130.387967] sysrq: Power Off
[  144s] [  130.396194] reboot: Power down
[  144s] ### VM INTERACTION END ###
[  144s] 
[  144s] sheep87 failed "build osmo-pcu_0.8.0.280.58cd.dsc" at Tue Dec  1 
03:59:41 UTC 2020.
[  144s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-remsim/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  180s] Running mktexlsr /var/lib/texmf ... done.
[  180s] [470/528] installing python3-3.9.0-3
[  180s] Processing triggers for man-db (2.9.3-2) ...
[  180s] [471/528] installing texlive-plain-generic-2020.20201129-1
[  183s] Processing triggers for tex-common (6.15) ...
[  184s] Running mktexlsr. This may take some time... done.
[  184s] [472/528] installing texlive-latex-base-2020.20201129-2
[  184s] Processing triggers for man-db (2.9.3-2) ...
[  184s] Processing triggers for tex-common (6.15) ...
[  184s] Running mktexlsr. This may take some time... done.
[  185s] Running updmap-sys. This may take some time... done.
[  185s] Running mktexlsr /var/lib/texmf ... done.
[  185s] Building format(s) --all.
[  192s]This may take some time... 
[  192s] fmtutil failed. Output has been stored in
[  192s] /tmp/fmtutil.LQU9jcmJ
[  192s] Please include this file if you report a bug.
[  193s] dpkg: error processing package tex-common (--install):
[  193s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  193s] Errors were encountered while processing:
[  193s]  tex-common
[  193s] exit ...
[  193s] ### VM INTERACTION START ###
[  196s] [  174.682958] sysrq: Power Off
[  196s] [  174.686043] reboot: Power down
[  196s] ### VM INTERACTION END ###
[  196s] 
[  196s] old-atreju3 failed "build osmo-remsim_0.2.2.103.dc34.dsc" at Tue Dec  
1 03:56:51 UTC 2020.
[  196s] 

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

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

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

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

Last lines of build log:
[  113s] [COMPILING apps/dfu/main.c]
[  113s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu.c]
[  113s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_desc.c]
[  113s] [COMPILING ./atmel_softpack_libraries/usb/device/dfu/dfu_driver.c]
[  113s] Memory region Used Size  Region Size  %age Used
[  113s]  rom:   16580 B16 KB101.20%
[  113s]  ram:   11672 B48 KB 
23.75/usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: warning: 
changing start of section .stack by 4 bytes
[  113s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  113s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
bin/qmod-dfu-flash.elf section `.text' will not fit in region `rom'
[  113s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  113s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  113s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: 
warning: changing start of section .stack by 4 bytes
[  113s] /usr/lib/gcc/arm-none-eabi/9.2.1/../../../arm-none-eabi/bin/ld: region 
`rom' overflowed by 196 bytes
[  113s] collect2: error: ld returned 1 exit status
[  113s] %
[  113s] make[2]: *** [Makefile:234: flash] Error 1
[  113s] make[2]: Leaving directory '/usr/src/packages/BUILD/firmware'
[  113s] make[1]: *** [Makefile:13: fw-qmod-dfu] Error 2
[  113s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  113s] dh_auto_build: error: make -j1 returned exit code 2
[  113s] make: *** [debian/rules:16: build] Error 25
[  113s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  113s] ### VM INTERACTION START ###
[  116s] [  104.832597] sysrq: Power Off
[  116s] [  104.835770] reboot: Power down
[  116s] ### VM INTERACTION END ###
[  116s] 
[  116s] sheep85 failed "build simtrace2_0.7.0.69.aadd.dsc" at Tue Dec  1 
03:56:19 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/osmo-e1d in Debian_Unstable/x86_64

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-e1d/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  112s] Running mktexlsr /var/lib/texmf ... done.
[  112s] [434/490] installing python3-3.9.0-3
[  112s] Processing triggers for man-db (2.9.3-2) ...
[  112s] [435/490] installing texlive-plain-generic-2020.20201129-1
[  114s] Processing triggers for tex-common (6.15) ...
[  114s] Running mktexlsr. This may take some time... done.
[  114s] [436/490] installing texlive-latex-base-2020.20201129-2
[  115s] Processing triggers for man-db (2.9.3-2) ...
[  115s] Processing triggers for tex-common (6.15) ...
[  115s] Running mktexlsr. This may take some time... done.
[  116s] Running updmap-sys. This may take some time... done.
[  116s] Running mktexlsr /var/lib/texmf ... done.
[  116s] Building format(s) --all.
[  120s]This may take some time... 
[  120s] fmtutil failed. Output has been stored in
[  120s] /tmp/fmtutil.E0vr9b5H
[  120s] Please include this file if you report a bug.
[  120s] dpkg: error processing package tex-common (--install):
[  120s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  120s] Errors were encountered while processing:
[  120s]  tex-common
[  121s] exit ...
[  121s] ### VM INTERACTION START ###
[  124s] [  114.408950] sysrq: Power Off
[  124s] [  114.412724] reboot: Power down
[  124s] ### VM INTERACTION END ###
[  124s] 
[  124s] goat11 failed "build osmo-e1d_0.1.1.20.e542.dsc" at Tue Dec  1 
03:48:15 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/osmo-bts in Debian_Unstable/x86_64

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-bts/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  247s] Running mktexlsr /var/lib/texmf ... done.
[  247s] [445/501] installing python3-3.9.0-3
[  248s] Processing triggers for man-db (2.9.3-2) ...
[  248s] [446/501] installing texlive-plain-generic-2020.20201129-1
[  252s] Processing triggers for tex-common (6.15) ...
[  252s] Running mktexlsr. This may take some time... done.
[  252s] [447/501] installing texlive-latex-base-2020.20201129-2
[  252s] Processing triggers for man-db (2.9.3-2) ...
[  253s] Processing triggers for tex-common (6.15) ...
[  253s] Running mktexlsr. This may take some time... done.
[  254s] Running updmap-sys. This may take some time... done.
[  254s] Running mktexlsr /var/lib/texmf ... done.
[  254s] Building format(s) --all.
[  262s]This may take some time... 
[  262s] fmtutil failed. Output has been stored in
[  262s] /tmp/fmtutil.LWv3hnUf
[  262s] Please include this file if you report a bug.
[  262s] dpkg: error processing package tex-common (--install):
[  262s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  262s] Errors were encountered while processing:
[  262s]  tex-common
[  262s] exit ...
[  262s] ### VM INTERACTION START ###
[  265s] [  244.912299] sysrq: Power Off
[  265s] [  244.916848] reboot: Power down
[  265s] ### VM INTERACTION END ###
[  265s] 
[  265s] lamb09 failed "build osmo-bts_1.2.0.343.def2.dsc" at Tue Dec  1 
03:42:25 UTC 2020.
[  265s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-pcu/Debian_8.0/x86_64

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

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

Last lines of build log:
[  124s] make[2]: Nothing to be done for 'all'.
[  124s] make[2]: Leaving directory '/usr/src/packages/BUILD/include'
[  124s] Making all in src
[  124s] make[2]: Entering directory '/usr/src/packages/BUILD/src'
[  124s]   CXX  gprs_debug.lo
[  124s]   CC   csn1.lo
[  126s]   CC   gsm_rlcmac.lo
[  127s]   CXX  gprs_bssgp_pcu.lo
[  127s] gprs_bssgp_pcu.cpp: In function 'int ns_create_nsvc(gprs_rlcmac_bts*, 
uint16_t, const osmo_sockaddr*, const osmo_sockaddr*, const uint16_t*, 
uint16_t)':
[  127s] gprs_bssgp_pcu.cpp:983:13: error: 'GPRS_NS2_LL_UDP' was not declared 
in this scope
[  127s]  GPRS_NS2_LL_UDP);
[  127s]  ^
[  127s] Makefile:790: recipe for target 'gprs_bssgp_pcu.lo' failed
[  127s] make[2]: *** [gprs_bssgp_pcu.lo] Error 1
[  127s] make[2]: Leaving directory '/usr/src/packages/BUILD/src'
[  127s] Makefile:405: recipe for target 'all-recursive' failed
[  127s] make[1]: *** [all-recursive] Error 1
[  127s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  127s] dh_auto_build: make -j1 returned exit code 2
[  127s] debian/rules:12: recipe for target 'build' failed
[  127s] make: *** [build] Error 2
[  127s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  127s] ### VM INTERACTION START ###
[  128s] Powering off.
[  128s] [  119.775690] reboot: Power down
[  128s] ### VM INTERACTION END ###
[  128s] 
[  128s] sheep85 failed "build osmo-pcu_0.8.0.280.58cd.dsc" at Tue Dec  1 
03:18:05 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/osmo-ggsn in Debian_Unstable/x86_64

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-ggsn/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  143s] Running mktexlsr /var/lib/texmf ... done.
[  143s] [436/492] installing python3-3.9.0-3
[  144s] Processing triggers for man-db (2.9.3-2) ...
[  144s] [437/492] installing texlive-plain-generic-2020.20200925-1
[  147s] Processing triggers for tex-common (6.15) ...
[  147s] Running mktexlsr. This may take some time... done.
[  147s] [438/492] installing texlive-latex-base-2020.20201129-1
[  147s] Processing triggers for man-db (2.9.3-2) ...
[  148s] Processing triggers for tex-common (6.15) ...
[  148s] Running mktexlsr. This may take some time... done.
[  148s] Running updmap-sys. This may take some time... done.
[  148s] Running mktexlsr /var/lib/texmf ... done.
[  148s] Building format(s) --all.
[  154s]This may take some time... 
[  154s] fmtutil failed. Output has been stored in
[  154s] /tmp/fmtutil.5oxEeuzV
[  154s] Please include this file if you report a bug.
[  154s] dpkg: error processing package tex-common (--install):
[  154s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  154s] Errors were encountered while processing:
[  154s]  tex-common
[  154s] exit ...
[  154s] ### VM INTERACTION START ###
[  157s] [  140.829043] sysrq: Power Off
[  157s] [  140.832520] reboot: Power down
[  157s] ### VM INTERACTION END ###
[  157s] 
[  157s] hci-cnode1-m0 failed "build osmo-ggsn_1.6.0.13.1230.dsc" at Tue Dec  1 
03:03:37 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/osmo-hlr in Debian_Unstable/x86_64

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-hlr/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  167s] [445/502] installing python3-3.9.0-3
[  167s] Processing triggers for man-db (2.9.3-2) ...
[  168s] [446/502] installing texlive-plain-generic-2020.20200925-1
[  171s] Processing triggers for tex-common (6.15) ...
[  171s] Running mktexlsr. This may take some time... done.
[  171s] [447/502] installing libosmo-netif-dev-1.0.0.8.bd77
[  172s] [448/502] installing texlive-latex-base-2020.20201129-1
[  172s] Processing triggers for man-db (2.9.3-2) ...
[  172s] Processing triggers for tex-common (6.15) ...
[  173s] Running mktexlsr. This may take some time... done.
[  173s] Running updmap-sys. This may take some time... done.
[  173s] Running mktexlsr /var/lib/texmf ... done.
[  173s] Building format(s) --all.
[  181s]This may take some time... 
[  181s] fmtutil failed. Output has been stored in
[  181s] /tmp/fmtutil.dNwreZRN
[  181s] Please include this file if you report a bug.
[  182s] dpkg: error processing package tex-common (--install):
[  182s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  182s] Errors were encountered while processing:
[  182s]  tex-common
[  182s] exit ...
[  182s] ### VM INTERACTION START ###
[  185s] [  165.020548] sysrq: Power Off
[  185s] [  165.024950] reboot: Power down
[  185s] ### VM INTERACTION END ###
[  185s] 
[  185s] old-atreju4 failed "build osmo-hlr_1.2.0.67.dac8.dsc" at Tue Dec  1 
03:00:47 UTC 2020.
[  185s] 

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


Build failure of network:osmocom:nightly/libosmo-sccp in Debian_Unstable/x86_64

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libosmo-sccp/Debian_Unstable/x86_64

Package network:osmocom:nightly/libosmo-sccp failed to build in 
Debian_Unstable/x86_64

Check out the package for editing:
  osc checkout network:osmocom:nightly libosmo-sccp

Last lines of build log:
[  269s] [445/502] installing python3-3.9.0-3
[  269s] Processing triggers for man-db (2.9.3-2) ...
[  270s] [446/502] installing texlive-plain-generic-2020.20200925-1
[  274s] Processing triggers for tex-common (6.15) ...
[  274s] Running mktexlsr. This may take some time... done.
[  274s] [447/502] installing libosmo-netif-dev-1.0.0.8.bd77
[  274s] [448/502] installing texlive-latex-base-2020.20201129-1
[  275s] Processing triggers for man-db (2.9.3-2) ...
[  275s] Processing triggers for tex-common (6.15) ...
[  275s] Running mktexlsr. This may take some time... done.
[  276s] Running updmap-sys. This may take some time... done.
[  276s] Running mktexlsr /var/lib/texmf ... done.
[  276s] Building format(s) --all.
[  285s]This may take some time... 
[  285s] fmtutil failed. Output has been stored in
[  285s] /tmp/fmtutil.wRcC3IYH
[  285s] Please include this file if you report a bug.
[  285s] dpkg: error processing package tex-common (--install):
[  285s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  285s] Errors were encountered while processing:
[  285s]  tex-common
[  285s] exit ...
[  285s] ### VM INTERACTION START ###
[  288s] [  223.772454] sysrq: Power Off
[  288s] [  223.797700] reboot: Power down
[  288s] ### VM INTERACTION END ###
[  288s] 
[  288s] cloud107 failed "build libosmo-sccp_1.3.0.25.48e2.dsc" at Tue Dec  1 
02:51:17 UTC 2020.
[  288s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-mgw/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  228s] [442/499] installing python3-3.9.0-3
[  228s] Processing triggers for man-db (2.9.3-2) ...
[  229s] [443/499] installing texlive-plain-generic-2020.20200925-1
[  233s] Processing triggers for tex-common (6.15) ...
[  233s] Running mktexlsr. This may take some time... done.
[  233s] [444/499] installing libosmo-netif-dev-1.0.0.8.bd77
[  234s] [445/499] installing texlive-latex-base-2020.20201129-1
[  234s] Processing triggers for man-db (2.9.3-2) ...
[  235s] Processing triggers for tex-common (6.15) ...
[  235s] Running mktexlsr. This may take some time... done.
[  235s] Running updmap-sys. This may take some time... done.
[  235s] Running mktexlsr /var/lib/texmf ... done.
[  236s] Building format(s) --all.
[  243s]This may take some time... 
[  243s] fmtutil failed. Output has been stored in
[  243s] /tmp/fmtutil.6g3FJuAg
[  243s] Please include this file if you report a bug.
[  243s] dpkg: error processing package tex-common (--install):
[  243s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  243s] Errors were encountered while processing:
[  243s]  tex-common
[  243s] exit ...
[  243s] ### VM INTERACTION START ###
[  247s] [  227.305894] sysrq: Power Off
[  247s] [  227.310453] reboot: Power down
[  247s] ### VM INTERACTION END ###
[  247s] 
[  247s] lamb54 failed "build osmo-mgw_1.7.0.130.3c29.dsc" at Tue Dec  1 
02:50:27 UTC 2020.
[  247s] 

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


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

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-gsm-manuals/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  217s] Running mktexlsr /var/lib/texmf ... done.
[  217s] [416/471] installing python3-3.9.0-3
[  217s] Processing triggers for man-db (2.9.3-2) ...
[  217s] [417/471] installing texlive-plain-generic-2020.20200925-1
[  221s] Processing triggers for tex-common (6.15) ...
[  222s] Running mktexlsr. This may take some time... done.
[  222s] [418/471] installing texlive-latex-base-2020.20201129-1
[  222s] Processing triggers for man-db (2.9.3-2) ...
[  223s] Processing triggers for tex-common (6.15) ...
[  223s] Running mktexlsr. This may take some time... done.
[  223s] Running updmap-sys. This may take some time... done.
[  224s] Running mktexlsr /var/lib/texmf ... done.
[  224s] Building format(s) --all.
[  233s]This may take some time... 
[  233s] fmtutil failed. Output has been stored in
[  233s] /tmp/fmtutil.UpJM2cin
[  233s] Please include this file if you report a bug.
[  233s] dpkg: error processing package tex-common (--install):
[  233s]  installed tex-common package post-installation script subprocess 
returned error exit status 1
[  233s] Errors were encountered while processing:
[  233s]  tex-common
[  233s] exit ...
[  233s] ### VM INTERACTION START ###
[  236s] [  191.857361] sysrq: Power Off
[  236s] [  191.863392] reboot: Power down
[  236s] ### VM INTERACTION END ###
[  236s] 
[  236s] cloud117 failed "build osmo-gsm-manuals-dev_0.3.0.37.c507.dsc" at Tue 
Dec  1 02:32:00 UTC 2020.
[  236s] 

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


Build failure of network:osmocom:nightly/osmo-sgsn in Debian_8.0/i586

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-sgsn/Debian_8.0/i586

Package network:osmocom:nightly/osmo-sgsn failed to build in Debian_8.0/i586

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

Last lines of build log:
[  140s] make[4]: Entering directory '/usr/src/packages/BUILD/src/gbproxy'
[  140s]   CC   gb_proxy.o
[  140s] gb_proxy.c: In function 'gbprox_relay2nse':
[  140s] gb_proxy.c:846:17: error: 'struct ' has no member named 
'link_selector'
[  140s]nsp.u.unitdata.link_selector = tlli;
[  140s]  ^
[  140s] Makefile:482: recipe for target 'gb_proxy.o' failed
[  140s] make[4]: *** [gb_proxy.o] Error 1
[  140s] make[4]: Leaving directory '/usr/src/packages/BUILD/src/gbproxy'
[  140s] Makefile:383: recipe for target 'all-recursive' failed
[  140s] make[3]: *** [all-recursive] Error 1
[  140s] make[3]: Leaving directory '/usr/src/packages/BUILD/src'
[  140s] Makefile:449: recipe for target 'all-recursive' failed
[  140s] make[2]: *** [all-recursive] Error 1
[  140s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  140s] Makefile:380: recipe for target 'all' failed
[  140s] make[1]: *** [all] Error 2
[  140s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  140s] dh_auto_build: make -j1 returned exit code 2
[  140s] debian/rules:45: recipe for target 'build' failed
[  140s] make: *** [build] Error 2
[  140s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  140s] ### VM INTERACTION START ###
[  141s] Powering off.
[  141s] [  134.385400] reboot: Power down
[  141s] ### VM INTERACTION END ###
[  141s] 
[  141s] build83 failed "build osmo-sgsn_1.6.0.64.9e58.dsc" at Tue Dec  1 
02:22:08 UTC 2020.
[  141s] 

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


Build failure of network:osmocom:nightly/osmo-pcu in Debian_8.0/i586

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/osmo-pcu/Debian_8.0/i586

Package network:osmocom:nightly/osmo-pcu failed to build in Debian_8.0/i586

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

Last lines of build log:
[  154s] make[2]: Nothing to be done for 'all'.
[  154s] make[2]: Leaving directory '/usr/src/packages/BUILD/include'
[  154s] Making all in src
[  154s] make[2]: Entering directory '/usr/src/packages/BUILD/src'
[  154s]   CXX  gprs_debug.lo
[  155s]   CC   csn1.lo
[  158s]   CC   gsm_rlcmac.lo
[  159s]   CXX  gprs_bssgp_pcu.lo
[  159s] gprs_bssgp_pcu.cpp: In function 'int ns_create_nsvc(gprs_rlcmac_bts*, 
uint16_t, const osmo_sockaddr*, const osmo_sockaddr*, const uint16_t*, 
uint16_t)':
[  159s] gprs_bssgp_pcu.cpp:983:13: error: 'GPRS_NS2_LL_UDP' was not declared 
in this scope
[  159s]  GPRS_NS2_LL_UDP);
[  159s]  ^
[  159s] Makefile:790: recipe for target 'gprs_bssgp_pcu.lo' failed
[  159s] make[2]: *** [gprs_bssgp_pcu.lo] Error 1
[  159s] make[2]: Leaving directory '/usr/src/packages/BUILD/src'
[  159s] Makefile:405: recipe for target 'all-recursive' failed
[  159s] make[1]: *** [all-recursive] Error 1
[  159s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  159s] dh_auto_build: make -j1 returned exit code 2
[  159s] debian/rules:12: recipe for target 'build' failed
[  159s] make: *** [build] Error 2
[  159s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  159s] ### VM INTERACTION START ###
[  160s] Powering off.
[  160s] [  151.028803] reboot: Power down
[  160s] ### VM INTERACTION END ###
[  160s] 
[  160s] hci-cnode1-m0 failed "build osmo-pcu_0.8.0.280.58cd.dsc" at Tue Dec  1 
01:55:58 UTC 2020.
[  160s] 

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


Build failure of network:osmocom:nightly/libosmocore in Debian_8.0/x86_64

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libosmocore/Debian_8.0/x86_64

Package network:osmocom:nightly/libosmocore failed to build in Debian_8.0/x86_64

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

Last lines of build log:
[  218s] ^
[  218s] In file included from gprs_ns2_fr.c:45:0:
[  218s] /usr/include/linux/if.h:252:8: error: redefinition of 'struct ifconf'
[  218s]  struct ifconf  {
[  218s] ^
[  218s] In file included from gprs_ns2_fr.c:39:0:
[  218s] /usr/include/net/if.h:176:8: note: originally defined here
[  218s]  struct ifconf
[  218s] ^
[  218s] Makefile:542: recipe for target 'gprs_ns2_fr.lo' failed
[  218s] make[3]: *** [gprs_ns2_fr.lo] Error 1
[  218s] make[3]: Leaving directory '/usr/src/packages/BUILD/src/gb'
[  218s] Makefile:690: recipe for target 'all-recursive' failed
[  218s] make[2]: *** [all-recursive] Error 1
[  218s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  218s] Makefile:468: recipe for target 'all' failed
[  218s] make[1]: *** [all] Error 2
[  218s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  218s] dh_auto_build: make -j1 returned exit code 2
[  218s] debian/rules:15: recipe for target 'build' failed
[  218s] make: *** [build] Error 2
[  218s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  218s] ### VM INTERACTION START ###
[  219s] Powering off.
[  219s] [  200.234973] reboot: Power down
[  219s] ### VM INTERACTION END ###
[  219s] 
[  219s] old-atreju5 failed "build libosmocore_1.4.0.219.a24e7.dsc" at Tue Dec  
1 01:40:57 UTC 2020.
[  219s] 

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


Build failure of network:osmocom:nightly/libosmocore in Debian_8.0/i586

2020-11-30 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:nightly/libosmocore/Debian_8.0/i586

Package network:osmocom:nightly/libosmocore failed to build in Debian_8.0/i586

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

Last lines of build log:
[  176s] ^
[  176s] In file included from gprs_ns2_fr.c:45:0:
[  176s] /usr/include/linux/if.h:252:8: error: redefinition of 'struct ifconf'
[  176s]  struct ifconf  {
[  176s] ^
[  176s] In file included from gprs_ns2_fr.c:39:0:
[  176s] /usr/include/net/if.h:176:8: note: originally defined here
[  176s]  struct ifconf
[  176s] ^
[  176s] Makefile:542: recipe for target 'gprs_ns2_fr.lo' failed
[  176s] make[3]: *** [gprs_ns2_fr.lo] Error 1
[  176s] make[3]: Leaving directory '/usr/src/packages/BUILD/src/gb'
[  176s] Makefile:690: recipe for target 'all-recursive' failed
[  176s] make[2]: *** [all-recursive] Error 1
[  176s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  176s] Makefile:468: recipe for target 'all' failed
[  176s] make[1]: *** [all] Error 2
[  176s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  176s] dh_auto_build: make -j1 returned exit code 2
[  176s] debian/rules:15: recipe for target 'build' failed
[  176s] make: *** [build] Error 2
[  176s] dpkg-buildpackage: error: debian/rules build gave error exit status 2
[  176s] ### VM INTERACTION START ###
[  177s] Powering off.
[  177s] [  169.762850] reboot: Power down
[  177s] ### VM INTERACTION END ###
[  177s] 
[  177s] lamb05 failed "build libosmocore_1.4.0.219.a24e7.dsc" at Tue Dec  1 
01:33:04 UTC 2020.
[  177s] 

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


Change in docker-playground[master]: Add ttcn3-gbproxy-test-fr for FR related testing

2020-11-30 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/docker-playground/+/21412 )

Change subject: Add ttcn3-gbproxy-test-fr for FR related testing
..

Add ttcn3-gbproxy-test-fr for FR related testing

Testing FR support in osmo-gbproxy is a bit more complicated
as it involves the "hdlc" net-devices privded by the hdlc_fr.ko
kernel module.

So we need to
* run on a host with actual hdlc net-devices (e.g. dahdi_dyamic_loc)
* move those net-devices into the containers after starting them
* wait for the net-devices to appear in the containers before starting
  either gbproxy or the test suite

Change-Id: Id4b52877db53cb6e59f6d0d3f754aaae633949e8
---
A ttcn3-gbproxy-test-fr/GBProxy_Tests.cfg
A ttcn3-gbproxy-test-fr/jenkins.sh
A ttcn3-gbproxy-test-fr/netdev-to-docker.sh
A ttcn3-gbproxy-test-fr/osmo-gbproxy.cfg
M ttcn3-gbproxy-test/Dockerfile
A ttcn3-gbproxy-test/docker-entrypoint.sh
A ttcn3-gbproxy-test/pipework
7 files changed, 776 insertions(+), 5 deletions(-)

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



diff --git a/ttcn3-gbproxy-test-fr/GBProxy_Tests.cfg 
b/ttcn3-gbproxy-test-fr/GBProxy_Tests.cfg
new file mode 100644
index 000..adab61a
--- /dev/null
+++ b/ttcn3-gbproxy-test-fr/GBProxy_Tests.cfg
@@ -0,0 +1,100 @@
+[ORDERED_INCLUDE]
+# Common configuration, shared between test suites
+"/osmo-ttcn3-hacks/Common.cfg"
+# testsuite specific configuration, not expected to change
+"/osmo-ttcn3-hacks/gbproxy/GBProxy_Tests.default"
+
+# Local configuration below
+
+[LOGGING]
+
+[TESTPORT_PARAMETERS]
+*.GBPVTY.CTRL_HOSTNAME := "172.18.25.10"
+
+[MODULE_PARAMETERS]
+GBProxy_Tests.mp_nsconfig_sgsn := {
+   {
+   nsvc := {
+   {
+   provider := {
+   ip := {
+   address_family := AF_INET,
+   local_ip := "172.18.25.103",
+   local_udp_port := 23000,
+   remote_ip := "172.18.25.10"
+   }
+   }
+   }
+   }
+   }
+}
+GBProxy_Tests.mp_nsconfig_pcu := {
+   {
+   nsei := 1,
+   role_sgsn := false,
+   handle_sns := false,
+   nsvc := {
+   {
+   provider := {
+   fr := {
+   netdev := "hdlc1",
+   dlci := 16
+   }
+   },
+   nsvci := 1
+   }, {
+   provider := {
+   fr := {
+   netdev := "hdlc2",
+   dlci := 17
+   }
+   },
+   nsvci := 2
+   }, {
+   provider := {
+   fr := {
+   netdev := "hdlc3",
+   dlci := 18
+   }
+   },
+   nsvci := 3
+   }, {
+   provider := {
+   fr := {
+   netdev := "hdlc4",
+   dlci := 19
+   }
+   },
+   nsvci := 4
+   }
+   }
+   }
+}
+
+GBProxy_Tests.mp_gbconfigs := {
+   {
+   nsei := 1,
+   sgsn_role := false,
+   bvc := {
+   {
+   bvci := 196,
+   cell_id := {
+   ra_id := {
+   lai := {
+   mcc_mnc := '262F42'H,
+   lac := 13135
+   },
+   rac := 0
+   },
+   cell_id := 20960
+   },
+   depth := BSSGP_DECODE_DEPTH_BSSGP
+   }
+   }
+   }
+};
+
+[MAIN_CONTROLLER]
+
+[EXECUTE]
+GBProxy_Tests.control
diff --git a/ttcn3-gbproxy-test-fr/jenkins.sh 

Change in docker-playground[master]: Add ttcn3-gbproxy-test-fr for FR related testing

2020-11-30 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/docker-playground/+/21412 )

Change subject: Add ttcn3-gbproxy-test-fr for FR related testing
..


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


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

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: Id4b52877db53cb6e59f6d0d3f754aaae633949e8
Gerrit-Change-Number: 21412
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 30 Nov 2020 20:47:23 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in docker-playground[master]: osmo-gbproxy-master (derived from osmo-sgsn-master)

2020-11-30 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/docker-playground/+/21411 )

Change subject: osmo-gbproxy-master (derived from osmo-sgsn-master)
..

osmo-gbproxy-master (derived from osmo-sgsn-master)

Using a separate container will allow us to do gbproxy specific things
like waiting for physical net-devices to show up in the docker netns
before actually starting the osmo-gbproxy executable

Change-Id: I5e6351055ac9430f13e43291cf2a9d52683fe6bf
---
A osmo-gbproxy-master/Dockerfile
A osmo-gbproxy-master/Makefile
A osmo-gbproxy-master/docker-entrypoint.sh
A osmo-gbproxy-master/osmo-gbproxy.cfg
A osmo-gbproxy-master/pipework
5 files changed, 542 insertions(+), 0 deletions(-)

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



diff --git a/osmo-gbproxy-master/Dockerfile b/osmo-gbproxy-master/Dockerfile
new file mode 100644
index 000..8661ec9
--- /dev/null
+++ b/osmo-gbproxy-master/Dockerfile
@@ -0,0 +1,15 @@
+ARGUSER
+ARGDISTRO
+FROM   $USER/osmo-sgsn-master
+
+COPY   osmo-gbproxy.cfg/data/osmo-gbproxy.cfg
+
+# work-around for stupid docker not being able to properly deal with host 
netdevices or start
+# containers in pre-existing netns
+COPY   pipework/usr/bin/pipework
+COPY   docker-entrypoint.sh/docker-entrypoint.sh
+
+WORKDIR/data
+CMD["/docker-entrypoint.sh"]
+
+EXPOSE 23000/udp 4246/tcp 4263/tcp
diff --git a/osmo-gbproxy-master/Makefile b/osmo-gbproxy-master/Makefile
new file mode 100644
index 000..1614f9a
--- /dev/null
+++ b/osmo-gbproxy-master/Makefile
@@ -0,0 +1,3 @@
+RUN_ARGS?=--rm --network sigtran --ip 172.18.0.21 -v gbproxy-vol:/data
+
+include ../make/Makefile
diff --git a/osmo-gbproxy-master/docker-entrypoint.sh 
b/osmo-gbproxy-master/docker-entrypoint.sh
new file mode 100755
index 000..72c4e8c
--- /dev/null
+++ b/osmo-gbproxy-master/docker-entrypoint.sh
@@ -0,0 +1,9 @@
+#!/bin/bash
+set -e
+set -x
+
+if [[ -n ${WAIT_FOR_NETDEV:-} ]]; then
+   /usr/bin/pipework --wait -i ${WAIT_FOR_NETDEV}
+fi
+
+/usr/local/bin/osmo-gbproxy -c /data/osmo-gbproxy.cfg >/data/osmo-gbproxy.log 
2>&1
diff --git a/osmo-gbproxy-master/osmo-gbproxy.cfg 
b/osmo-gbproxy-master/osmo-gbproxy.cfg
new file mode 100644
index 000..29f698f
--- /dev/null
+++ b/osmo-gbproxy-master/osmo-gbproxy.cfg
@@ -0,0 +1,26 @@
+!
+! Osmocom Gb Proxy (0.9.0.404-6463) configuration saved from vty
+!!
+!
+line vty
+ no login
+!
+gbproxy
+ sgsn nsei 101
+ns
+ nse 101 nsvci 101
+ nse 101 remote-role sgsn
+ nse 101 encapsulation udp
+ nse 101 remote-ip 192.168.100.239
+ nse 101 remote-port 
+ timer tns-block 3
+ timer tns-block-retries 3
+ timer tns-reset 3
+ timer tns-reset-retries 3
+ timer tns-test 30
+ timer tns-alive 3
+ timer tns-alive-retries 10
+ encapsulation framerelay-gre enabled 0
+ encapsulation framerelay-gre local-ip 0.0.0.0
+ encapsulation udp local-ip 127.0.0.100
+ encapsulation udp local-port 23000
diff --git a/osmo-gbproxy-master/pipework b/osmo-gbproxy-master/pipework
new file mode 100755
index 000..97ce66b
--- /dev/null
+++ b/osmo-gbproxy-master/pipework
@@ -0,0 +1,489 @@
+#!/bin/sh
+# This code should (try to) follow Google's Shell Style Guide
+# (https://google.github.io/styleguide/shell.xml)
+set -e
+
+case "$1" in
+  --wait)
+WAIT=1
+;;
+  --direct-phys)
+DIRECT_PHYS=1
+shift
+;;
+esac
+
+IFNAME=$1
+
+# default value set further down if not set here
+CONTAINER_IFNAME=
+if [ "$2" = "-i" ]; then
+  CONTAINER_IFNAME=$3
+  shift 2
+fi
+
+if [ "$2" = "-l" ]; then
+  LOCAL_IFNAME=$3
+  shift 2
+fi
+
+#inet or inet6
+FAMILY_FLAG="-4"
+if [ "$2" = "-a" ]; then
+  FAMILY_FLAG="-$3"
+  shift 2
+fi
+
+GUESTNAME=$2
+IPADDR=$3
+MACADDR=$4
+
+case "$MACADDR" in
+  *@*)
+VLAN="${MACADDR#*@}"
+VLAN="${VLAN%%@*}"
+MACADDR="${MACADDR%%@*}"
+;;
+  *)
+VLAN=
+;;
+esac
+
+# did they ask to generate a custom MACADDR?
+# generate the unique string
+case "$MACADDR" in
+  U:*)
+macunique="${MACADDR#*:}"
+# now generate a 48-bit hash string from $macunique
+MACADDR=$(echo $macunique|md5sum|sed 
's/^\(..\)\(..\)\(..\)\(..\)\(..\).*$/02:\1:\2:\3:\4:\5/')
+   ;;
+esac
+
+
+[ "$IPADDR" ] || [ "$WAIT" ] || {
+  echo "Syntax:"
+  echo "pipework  [-i containerinterface] [-l 
localinterfacename] [-a addressfamily]  
/[@default_gateway] [macaddr][@vlan]"
+  echo "pipework  [-i containerinterface] [-l 
localinterfacename]  dhcp [macaddr][@vlan]"
+  echo "pipework mac: [-i containerinterface] [-l 
localinterfacename] [-a addressfamily]  
/[@default_gateway] [macaddr][@vlan]"
+  echo "pipework mac: [-i containerinterface] [-l 
localinterfacename]  dhcp [macaddr][@vlan]"
+  echo "pipework route  "
+  echo "pipework rule  "
+  echo "pipework tc  "
+  echo "pipework --wait [-i containerinterface]"
+  exit 1
+}
+
+# Succeed if the given utility is installed. Fail otherwise.
+# For explanations about `which` vs `type` vs `command`, see:
+# 

Change in docker-playground[master]: osmo-gbproxy-master (derived from osmo-sgsn-master)

2020-11-30 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/docker-playground/+/21411 )

Change subject: osmo-gbproxy-master (derived from osmo-sgsn-master)
..


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


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

Gerrit-Project: docker-playground
Gerrit-Branch: master
Gerrit-Change-Id: I5e6351055ac9430f13e43291cf2a9d52683fe6bf
Gerrit-Change-Number: 21411
Gerrit-PatchSet: 1
Gerrit-Owner: laforge 
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 30 Nov 2020 20:47:17 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: pcu: Verify LLC UI dummy frames are not appended at end of rlcmac DL ...

2020-11-30 Thread pespin
pespin has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21419 )


Change subject: pcu: Verify LLC UI dummy frames are not appended at end of 
rlcmac DL block
..

pcu: Verify LLC UI dummy frames are not appended at end of rlcmac DL block

LLC UI dummy frames are only to be sent alone in order to delay TBF
termination. Until recently (osmo-pcu.git
Ifae1a7b2b3dfad8df19585063088ba0df2749c8f), osmo-pcu was sending LLC UI
dummy frames instead of padding at RLCMAC layer, which made no sense.

Related: OS#4849
Change-Id: I7e0d9ed2475dbf989fbf932c8b83117ff5fb28fc
---
M pcu/GPRS_Components.ttcn
M pcu/PCU_Tests.ttcn
2 files changed, 81 insertions(+), 14 deletions(-)



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

diff --git a/pcu/GPRS_Components.ttcn b/pcu/GPRS_Components.ttcn
index 0850a3d..16036ad 100644
--- a/pcu/GPRS_Components.ttcn
+++ b/pcu/GPRS_Components.ttcn
@@ -1001,13 +1001,6 @@
f_shutdown(__BFILE__, __LINE__);
}

-   /* Check next data blocks contain dummy frames */
-   if (lengthof(data_block.blocks) > 1 and 
substr(data_block.blocks[1].payload, 0, 3) != '43C001'O) {
-   setverdict(fail, "Second data payload is not a dummy frame: ",
-  data_block.blocks[1].payload);
-   f_shutdown(__BFILE__, __LINE__);
-   }
-
if (not match(data_block.cs, exp_cs)) {
setverdict(fail, "Failed to match ", data_block.cs, " vs exp ", 
exp_cs);
f_shutdown(__BFILE__, __LINE__);
@@ -1036,13 +1029,6 @@
f_shutdown(__BFILE__, __LINE__);
}

-   /* Check next data blocks contain dummy frames */
-   if (lengthof(data_block.blocks) > 1 and 
substr(data_block.blocks[1].payload, 0, 3) != '43C001'O) {
-   setverdict(fail, "Second data payload is not a dummy frame: ",
-  data_block.blocks[1].payload);
-   f_shutdown(__BFILE__, __LINE__);
-   }
-
if (not match(data_block.mcs, exp_cs)) {
setverdict(fail, "Failed to match ", data_block.mcs, " vs exp 
", exp_cs);
f_shutdown(__BFILE__, __LINE__);
diff --git a/pcu/PCU_Tests.ttcn b/pcu/PCU_Tests.ttcn
index 67a811d..d187011 100644
--- a/pcu/PCU_Tests.ttcn
+++ b/pcu/PCU_Tests.ttcn
@@ -2546,6 +2546,85 @@
f_shutdown(__BFILE__, __LINE__, final := true);
 }

+private function f_tc_dl_data_no_llc_ui_dummy(template (omit) 
MSRadioAccessCapabilityV_BSSGP ms_racap := omit) runs on RAW_PCU_Test_CT {
+   var AckNackDescription ack_nack_desc := 
valueof(t_AckNackDescription_init);
+   var octetstring data := f_rnd_octstring(6);
+   var RlcmacDlBlock dl_block;
+   var GprsMS ms;
+   var uint32_t fn;
+
+   /* Initialize NS/BSSGP side */
+   f_init_bssgp();
+   /* Initialize GPRS MS side */
+   f_init_gprs_ms();
+   ms := g_ms[0]; /* We only use first MS in this test */
+
+   /* Initialize the PCU interface abstraction */
+   f_init_raw(testcasename());
+
+   /* Establish BSSGP connection to the PCU */
+   f_bssgp_establish();
+   f_bssgp_client_llgmm_assign(TLLI_UNUSED, ms.tlli);
+
+   /* SGSN sends some DL data, PCU will page on CCCH (PCH) */
+   BSSGP[0].send(ts_BSSGP_DL_UD(ms.tlli, data, ms_racap));
+   f_ms_exp_dl_tbf_ass_ccch(ms, PCU_IF_SAPI_PCH);
+
+   /* Wait timer X2002 and DL block is available after CCCH IMM ASS */
+   f_sleep(X2002);
+
+   /* Expect the first (GPRS DL) block with bsn=0 and rrbp_valid=1 */
+   f_rx_rlcmac_dl_block_exp_data(dl_block, fn, data, 0);
+
+   if (ischosen(dl_block.data_egprs)) {
+   if (lengthof(dl_block.data_egprs.blocks) != 2) {
+   setverdict(fail, "DL EGPRS block has unexpected number 
of LLC frames: ", dl_block.data_egprs);
+   f_shutdown(__BFILE__, __LINE__);
+   }
+   if (dl_block.data_egprs.blocks[1].hdr.length_ind != 127) {
+   setverdict(fail, "DL EGPRS block 2nd llc frame is not 
padding!: ", dl_block.data_egprs);
+   f_shutdown(__BFILE__, __LINE__);
+   }
+   if (dl_block.data_egprs.blocks[1].hdr.length_ind != 127) {
+   setverdict(fail, "DL EGPRS block 2nd llc frame is not 
padding!: ", dl_block.data_egprs);
+   f_shutdown(__BFILE__, __LINE__);
+   }
+   if (not match(dl_block.data_egprs.blocks[1].payload,
+ f_pad_oct(''O, 
lengthof(dl_block.data_egprs.blocks[1].payload), '2B'O))) {
+   setverdict(fail, "DL EGPRS block 2nd llc frame is not 
padding!: ", dl_block.data_egprs);
+   f_shutdown(__BFILE__, __LINE__);
+   }
+   } else if (lengthof(dl_block.data.blocks) > 1) {
+

Change in osmo-sgsn[master]: gbproxy: Separate function to move gbproxy_peer to different nse

2020-11-30 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21418 )

Change subject: gbproxy: Separate function to move gbproxy_peer to different nse
..

gbproxy: Separate function to move gbproxy_peer to different nse

Change-Id: I0a8b1e4b78384ea99d50109f050ca501b18ba5d9
Related: SYS#5226
---
M include/osmocom/sgsn/gb_proxy.h
M src/gbproxy/gb_proxy.c
M src/gbproxy/gb_proxy_peer.c
3 files changed, 9 insertions(+), 3 deletions(-)

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



diff --git a/include/osmocom/sgsn/gb_proxy.h b/include/osmocom/sgsn/gb_proxy.h
index 54d4548..9f91bd6 100644
--- a/include/osmocom/sgsn/gb_proxy.h
+++ b/include/osmocom/sgsn/gb_proxy.h
@@ -344,6 +344,7 @@
struct gbproxy_config *cfg, struct tlv_parsed *tp);
 struct gbproxy_peer *gbproxy_peer_alloc(struct gbproxy_nse *nse, uint16_t 
bvci);
 void gbproxy_peer_free(struct gbproxy_peer *peer);
+void gbproxy_peer_move(struct gbproxy_peer *peer, struct gbproxy_nse *nse);
 int gbproxy_cleanup_peers(struct gbproxy_config *cfg, uint16_t nsei, uint16_t 
bvci);

 /* NSE handling */
diff --git a/src/gbproxy/gb_proxy.c b/src/gbproxy/gb_proxy.c
index db7f55b..bd58617 100644
--- a/src/gbproxy/gb_proxy.c
+++ b/src/gbproxy/gb_proxy.c
@@ -1114,9 +1114,7 @@
LOGP(DGPRS, LOGL_NOTICE, "Peer for BVCI=%u 
moved from NSEI=%u to NSEI=%u\n", bvci, nse_old->nsei, nsei);

/* Move peer to different NSE */
-   llist_del(_peer->list);
-   llist_add(_peer->list, 
_new->bts_peers);
-   from_peer->nse = nse_new;
+   gbproxy_peer_move(from_peer, nse_new);
}

if (TLVP_PRESENT(, BSSGP_IE_CELL_ID)) {
diff --git a/src/gbproxy/gb_proxy_peer.c b/src/gbproxy/gb_proxy_peer.c
index 920547c..62c4d94 100644
--- a/src/gbproxy/gb_proxy_peer.c
+++ b/src/gbproxy/gb_proxy_peer.c
@@ -253,6 +253,13 @@
talloc_free(peer);
 }

+void gbproxy_peer_move(struct gbproxy_peer *peer, struct gbproxy_nse *nse)
+{
+   llist_del(>list);
+   llist_add(>list, >bts_peers);
+   peer->nse = nse;
+}
+
 int gbproxy_cleanup_peers(struct gbproxy_config *cfg, uint16_t nsei, uint16_t 
bvci)
 {
int counter = 0;

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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I0a8b1e4b78384ea99d50109f050ca501b18ba5d9
Gerrit-Change-Number: 21418
Gerrit-PatchSet: 2
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-sgsn[master]: gbproxy: Only send paging to each matching NSE once

2020-11-30 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21417 )

Change subject: gbproxy: Only send paging to each matching NSE once
..

gbproxy: Only send paging to each matching NSE once

Fixes: SYS#5226
Change-Id: Ia33e7816b354266b9d2659ef96b798db60c134f3
---
M src/gbproxy/gb_proxy.c
1 file changed, 34 insertions(+), 13 deletions(-)

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



diff --git a/src/gbproxy/gb_proxy.c b/src/gbproxy/gb_proxy.c
index 1ca4fc6..db7f55b 100644
--- a/src/gbproxy/gb_proxy.c
+++ b/src/gbproxy/gb_proxy.c
@@ -813,11 +813,10 @@
return rc;
 }

-/* feed a message down the NS-VC associated with the specified peer */
-static int gbprox_relay2peer(struct msgb *old_msg, struct gbproxy_peer *peer,
+/* feed a message down the NSE */
+static int gbprox_relay2nse(struct msgb *old_msg, struct gbproxy_nse *nse,
 uint16_t ns_bvci)
 {
-   struct gbproxy_nse *nse = peer->nse;
OSMO_ASSERT(nse);
OSMO_ASSERT(nse->cfg);

@@ -825,7 +824,7 @@
 * be free()d safely when we return from gbprox_rcvmsg() */
struct gprs_ns2_inst *nsi = nse->cfg->nsi;
struct osmo_gprs_ns2_prim nsp = {};
-   struct msgb *msg = bssgp_msgb_copy(old_msg, "msgb_relay2peer");
+   struct msgb *msg = bssgp_msgb_copy(old_msg, "msgb_relay2nse");
uint32_t tlli;
int rc;

@@ -849,6 +848,22 @@
osmo_prim_init(, SAP_NS, PRIM_NS_UNIT_DATA,
   PRIM_OP_REQUEST, msg);
rc = gprs_ns2_recv_prim(nsi, );
+   /* FIXME: We need a counter group for gbproxy_nse */
+   //if (rc < 0)
+   //  rate_ctr_inc(>ctrg->ctr[GBPROX_PEER_CTR_TX_ERR]);
+
+   return rc;
+}
+
+/* feed a message down the NS-VC associated with the specified peer */
+static int gbprox_relay2peer(struct msgb *old_msg, struct gbproxy_peer *peer,
+uint16_t ns_bvci)
+{
+   int rc;
+   struct gbproxy_nse *nse = peer->nse;
+   OSMO_ASSERT(nse);
+
+   rc = gbprox_relay2nse(old_msg, nse, ns_bvci);
if (rc < 0)
rate_ctr_inc(>ctrg->ctr[GBPROX_PEER_CTR_TX_ERR]);

@@ -1151,7 +1166,7 @@
 {
struct gbproxy_nse *nse;
struct gbproxy_peer *peer;
-   unsigned int n_peers = 0;
+   unsigned int n_nses = 0;
int errctr = GBPROX_GLOB_CTR_PROTO_ERR_SGSN;

/* FIXME: Handle paging logic to only page each matching NSE */
@@ -1176,9 +1191,11 @@
llist_for_each_entry(nse, >nse_peers, list) {
llist_for_each_entry(peer, >bts_peers, list) {
if (!memcmp(peer->ra, TLVP_VAL(tp, 
BSSGP_IE_ROUTEING_AREA), 6)) {
-   LOGPC(DGPRS, LOGL_INFO, "routing by RAI 
to peer BVCI=%u\n", peer->bvci);
-   gbprox_relay2peer(msg, peer, ns_bvci);
-   n_peers++;
+   LOGPC(DGPRS, LOGL_INFO, "routing by RAI 
to peer NSEI=%u\n", peer->bvci);
+   gbprox_relay2nse(msg, nse, ns_bvci);
+   n_nses++;
+   /* Only send it once to each NSE */
+   break;
}
}
}
@@ -1189,8 +1206,10 @@
llist_for_each_entry(peer, >bts_peers, list) {
if (!memcmp(peer->ra, TLVP_VAL(tp, 
BSSGP_IE_LOCATION_AREA), 5)) {
LOGPC(DGPRS, LOGL_INFO, "routing by LAI 
to peer BVCI=%u\n", peer->bvci);
-   gbprox_relay2peer(msg, peer, ns_bvci);
-   n_peers++;
+   gbprox_relay2nse(msg, nse, ns_bvci);
+   n_nses++;
+   /* Only send it once to each NSE */
+   break;
}
}
}
@@ -1199,8 +1218,10 @@
llist_for_each_entry(nse, >nse_peers, list) {
llist_for_each_entry(peer, >bts_peers, list) {
LOGPC(DGPRS, LOGL_INFO, "broadcasting to peer 
BVCI=%u\n", peer->bvci);
-   gbprox_relay2peer(msg, peer, ns_bvci);
-   n_peers++;
+   gbprox_relay2nse(msg, nse, ns_bvci);
+   n_nses++;
+   /* Only send it once to each NSE */
+   break;
}
}
} else {
@@ -1210,7 +1231,7 @@

Change in osmo-sgsn[master]: gbproxy: Ensure BVC0 is reset before handling PtP BVCs

2020-11-30 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21410 )

Change subject: gbproxy: Ensure BVC0 is reset before handling PtP BVCs
..

gbproxy: Ensure BVC0 is reset before handling PtP BVCs

Change-Id: I9c67f973f7bca00e8eb22a024ef2282c007dd84b
Related: SYS#5226
---
M src/gbproxy/gb_proxy.c
M tests/gbproxy/gbproxy_test.c
M tests/gbproxy/gbproxy_test.ok
3 files changed, 125 insertions(+), 6 deletions(-)

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



diff --git a/src/gbproxy/gb_proxy.c b/src/gbproxy/gb_proxy.c
index 736bd7a..1ca4fc6 100644
--- a/src/gbproxy/gb_proxy.c
+++ b/src/gbproxy/gb_proxy.c
@@ -1058,6 +1058,14 @@
LOGP(DGPRS, LOGL_INFO, "NSEI=%u Rx BVC RESET 
(BVCI=%u)\n",
nsei, bvci);
if (bvci == 0) {
+   struct gbproxy_nse *nse;
+   /* Ensure the NSE peer is there and clear all 
PtP BVCs */
+   nse = gbproxy_nse_by_nsei_or_new(cfg, nsei);
+   if (!nse)
+   LOGP(DGPRS, LOGL_ERROR, "Could not 
allocate NSE for NSEI=%u\n", nsei);
+
+   gbproxy_cleanup_peers(cfg, nsei, 0);
+
/* FIXME: only do this if SGSN is alive! */
LOGP(DGPRS, LOGL_INFO, "NSEI=%u Tx fake "
"BVC RESET ACK of BVCI=0\n", nsei);
@@ -1066,10 +1074,11 @@
}
from_peer = gbproxy_peer_by_bvci(cfg, bvci);
if (!from_peer) {
-   struct gbproxy_nse *nse = 
gbproxy_nse_by_nsei_or_new(cfg, nsei);
+   struct gbproxy_nse *nse = 
gbproxy_nse_by_nsei(cfg, nsei);
if (!nse) {
-   LOGP(DGPRS, LOGL_ERROR, "Could not 
allocate NSE for NSEI=%u\n", nsei);
-   return 
bssgp_tx_status(BSSGP_CAUSE_PROTO_ERR_UNSPEC, NULL, msg);
+   LOGP(DGPRS, LOGL_NOTICE, "Got PtP BVC 
reset before signalling reset for "
+   "BVCI=%u NSEI=%u\n", bvci, 
nsei);
+   return 
bssgp_tx_status(BSSGP_CAUSE_PDU_INCOMP_STATE, NULL, msg);
}
/* if a PTP-BVC is reset, and we don't know that
 * PTP-BVCI yet, we should allocate a new peer 
*/
@@ -1081,10 +1090,11 @@
/* Could have moved to a different NSE */
if (!check_peer_nsei(from_peer, nsei)) {
struct gbproxy_nse *nse_old = from_peer->nse;
-   struct gbproxy_nse *nse_new = 
gbproxy_nse_by_nsei_or_new(cfg, nsei);
+   struct gbproxy_nse *nse_new = 
gbproxy_nse_by_nsei(cfg, nsei);
if (!nse_new) {
-   LOGP(DGPRS, LOGL_ERROR, "Could not 
allocate NSE for NSEI=%u\n", nsei);
-   return 
bssgp_tx_status(BSSGP_CAUSE_PROTO_ERR_UNSPEC, NULL, msg);
+   LOGP(DGPRS, LOGL_NOTICE, "Got PtP BVC 
reset before signalling reset for "
+   "BVCI=%u NSEI=%u\n", bvci, 
nsei);
+   return 
bssgp_tx_status(BSSGP_CAUSE_PDU_INCOMP_STATE, NULL, msg);
}
LOGP(DGPRS, LOGL_NOTICE, "Peer for BVCI=%u 
moved from NSEI=%u to NSEI=%u\n", bvci, nse_old->nsei, nsei);

diff --git a/tests/gbproxy/gbproxy_test.c b/tests/gbproxy/gbproxy_test.c
index 06734b4..fec5914 100644
--- a/tests/gbproxy/gbproxy_test.c
+++ b/tests/gbproxy/gbproxy_test.c
@@ -930,6 +930,7 @@
   "NSEI 0x%04x(%d)\n\n",
   nsei, nsei);
send_ns_avail(nsi, nsei);
+   send_bssgp_reset(nsi, nsei, 0);
 }

 static void setup_bssgp(struct gprs_ns2_inst *nsi,
diff --git a/tests/gbproxy/gbproxy_test.ok b/tests/gbproxy/gbproxy_test.ok
index 14a2641..fbd5366 100644
--- a/tests/gbproxy/gbproxy_test.ok
+++ b/tests/gbproxy/gbproxy_test.ok
@@ -13,6 +13,15 @@

 NS2 CALLBACK, prim 2, bvci 0x

+PROCESSING BVC_RESET from NSEI 4096
+22 04 82 00 00 07 81 08 08 88 11 22 33 40 50 60 10 00
+
+NS2 CALLBACK, prim 0, msg length 18, bvci 0x
+22 04 82 00 00 07 81 08 08 88 11 22 33 40 50 60 10 00
+
+Message for BSS (NSEI=4096 BVCI=0):
+23 04 82 00 00
+
 Setup BSSGP: BVCI 0x1002(4098)

 PROCESSING BVC_RESET from NSEI 4096
@@ -48,6 +57,15 @@

 NS2 CALLBACK, prim 2, bvci 0x

+PROCESSING BVC_RESET from NSEI 8192
+22 04 82 

Change in osmo-sgsn[master]: gbproxy: Remove test testing NSVCI change

2020-11-30 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21413 )

Change subject: gbproxy: Remove test testing NSVCI change
..

gbproxy: Remove test testing NSVCI change

This was overlooked in commit 82182d which already removed lots of
NS-specific code in gbproxy_test.c

>From that commit message:
"""
Since NS2 has a different abstraction we mock up the prim send/recv
functions and don't test NS like the old tests did.
"""

Change-Id: Ic1d7e646e633c9fa62812f5005ed10c0108a06f2
Related: SYS#5226
---
M tests/gbproxy/gbproxy_test.c
M tests/gbproxy/gbproxy_test.ok
2 files changed, 2 insertions(+), 255 deletions(-)

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



diff --git a/tests/gbproxy/gbproxy_test.c b/tests/gbproxy/gbproxy_test.c
index 76fc000..78fa8cb 100644
--- a/tests/gbproxy/gbproxy_test.c
+++ b/tests/gbproxy/gbproxy_test.c
@@ -1309,7 +1309,7 @@
 {
struct gprs_ns2_inst *nsi = gprs_ns2_instantiate(tall_sgsn_ctx, 
gprs_ns2_callback, );
uint16_t bss_nsei[2] = {0x1000, 0x2000};
-   uint16_t bvci[4] = {0x1002, 0x2002, 0x3002, 0x4002};
+   uint16_t bvci[4] = {0x1002, 0x2002, 0x3002};

gbcfg.nsi = nsi;
gbcfg.nsip_sgsn_nsei = SGSN_NSEI;
@@ -1381,45 +1381,6 @@
send_ns_unitdata(nsi, NULL, bss_nsei[0], bvci[2], (uint8_t *)"", 0);
send_ns_unitdata(nsi, NULL, SGSN_NSEI, bvci[2], (uint8_t *)"", 0);

-   printf("--- Change NSVCI ---\n\n");
-
-   setup_ns(nsi, bss_nsei[1]);
-
-   printf("--- Setup BVCI 1 ---\n\n");
-
-   setup_bssgp(nsi, bss_nsei[1], bvci[0]);
-   send_bssgp_reset_ack(nsi, SGSN_NSEI, bvci[0]);
-   dump_peers(stdout, 0, 0, );
-
-   printf("--- Setup BVCI 4 ---\n\n");
-
-   setup_bssgp(nsi, bss_nsei[1], bvci[3]);
-   send_bssgp_reset_ack(nsi, SGSN_NSEI, bvci[3]);
-   dump_peers(stdout, 0, 0, );
-
-   printf("--- Send message from BSS 1 to SGSN and back, BVCI 1 ---\n\n");
-
-   send_ns_unitdata(nsi, NULL, bss_nsei[1], bvci[0], (uint8_t *)"", 0);
-   send_ns_unitdata(nsi, NULL, SGSN_NSEI, bvci[0], (uint8_t *)"", 0);
-
-   printf("--- Send message from BSS 1 to SGSN and back, BVCI 2 "
-  " (should fail) ---\n\n");
-
-   send_ns_unitdata(nsi, NULL, bss_nsei[1], bvci[1], (uint8_t *)"", 0);
-   dump_peers(stdout, 0, 0, );
-   send_ns_unitdata(nsi, NULL, SGSN_NSEI, bvci[1], (uint8_t *)"", 0);
-   dump_peers(stdout, 0, 0, );
-
-   printf("--- Send message from BSS 1 to SGSN and back, BVCI 3 ---\n\n");
-
-   send_ns_unitdata(nsi, NULL, bss_nsei[1], bvci[2], (uint8_t *)"", 0);
-   send_ns_unitdata(nsi, NULL, SGSN_NSEI, bvci[2], (uint8_t *)"", 0);
-
-   printf("--- Send message from BSS 1 to SGSN and back, BVCI 4 ---\n\n");
-
-   send_ns_unitdata(nsi, NULL, bss_nsei[1], bvci[3], (uint8_t *)"", 0);
-   send_ns_unitdata(nsi, NULL, SGSN_NSEI, bvci[3], (uint8_t *)"", 0);
-
dump_global(stdout, 0);
dump_peers(stdout, 0, 0, );

diff --git a/tests/gbproxy/gbproxy_test.ok b/tests/gbproxy/gbproxy_test.ok
index 6f20f12..74cedb3 100644
--- a/tests/gbproxy/gbproxy_test.ok
+++ b/tests/gbproxy/gbproxy_test.ok
@@ -654,227 +654,13 @@
 Message for BSS (NSEI=8192 BVCI=12290):
 [L2]> [L3]>

 Change NSVCI ---
-
-Setup NS-VC: NSEI 0x2000(8192)
-
-NS2 CALLBACK, prim 2, bvci 0x
-
 Setup BVCI 1 ---
-
-Setup BSSGP: BVCI 0x1002(4098)
-
-PROCESSING BVC_RESET from NSEI 8192
-22 04 82 10 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-NS2 CALLBACK, prim 0, msg length 18, bvci 0x
-22 04 82 10 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-NS2 UD REQUEST, prim 0, msg length 18, bvci 0x
-22 04 82 10 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-Message for SGSN (NSEI=256 BVCI=0):
-[L2]> [L3]> 22 04 82 10 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-PROCESSING BVC_RESET_ACK from NSEI 256
-23 04 82 10 02
-
-NS2 CALLBACK, prim 0, msg length 5, bvci 0x
-23 04 82 10 02
-
-NS2 UD REQUEST, prim 0, msg length 5, bvci 0x
-23 04 82 10 02
-
-Message for BSS (NSEI=8192 BVCI=0):
-[L2]> [L3]> 23 04 82 10 02
-
-Peers:
-  NSEI 8192, BVCI 12290, not blocked, RAI 112-332-16464-96
-NSEI mismatch   : 1
-TLLI-Cache: 0
-  NSEI 4096, BVCI 8194, not blocked, RAI 112-332-16464-96
-NSEI mismatch   : 1
-TLLI-Cache: 0
-  NSEI 8192, BVCI 4098, not blocked, RAI 112-332-16464-96
-NSEI mismatch   : 1
-TLLI-Cache: 0
 Setup BVCI 4 ---
-
-Setup BSSGP: BVCI 0x4002(16386)
-
-PROCESSING BVC_RESET from NSEI 8192
-22 04 82 40 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-NS2 CALLBACK, prim 0, msg length 18, bvci 0x
-22 04 82 40 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-NS2 UD REQUEST, prim 0, msg length 18, bvci 0x
-22 04 82 40 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-Message for SGSN (NSEI=256 BVCI=0):
-[L2]> [L3]> 22 

Change in osmo-sgsn[master]: gbproxy: Remove test testing NSVCI change

2020-11-30 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21413 )

Change subject: gbproxy: Remove test testing NSVCI change
..


Patch Set 2: Code-Review+2


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ic1d7e646e633c9fa62812f5005ed10c0108a06f2
Gerrit-Change-Number: 21413
Gerrit-PatchSet: 2
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 19:38:08 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Only send paging to each matching NSE once

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21417 )

Change subject: gbproxy: Only send paging to each matching NSE once
..


Patch Set 2: Code-Review+2


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ia33e7816b354266b9d2659ef96b798db60c134f3
Gerrit-Change-Number: 21417
Gerrit-PatchSet: 2
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 17:36:52 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Remove test testing NSVCI change

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21413 )

Change subject: gbproxy: Remove test testing NSVCI change
..


Patch Set 2: Code-Review+1


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ic1d7e646e633c9fa62812f5005ed10c0108a06f2
Gerrit-Change-Number: 21413
Gerrit-PatchSet: 2
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 17:32:42 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Remove test testing NSVCI change

2020-11-30 Thread daniel
daniel has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21413 )

Change subject: gbproxy: Remove test testing NSVCI change
..


Patch Set 2:

(1 comment)

https://gerrit.osmocom.org/c/osmo-sgsn/+/21413/1//COMMIT_MSG
Commit Message:

https://gerrit.osmocom.org/c/osmo-sgsn/+/21413/1//COMMIT_MSG@9
PS1, Line 9: Testing this here doesn't make much sense since ns2.
> Ack
Done (I hope)



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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ic1d7e646e633c9fa62812f5005ed10c0108a06f2
Gerrit-Change-Number: 21413
Gerrit-PatchSet: 2
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 17:31:46 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge 
Comment-In-Reply-To: pespin 
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Add NSE peer that can have multiple gbproxy_peers

2020-11-30 Thread daniel
Hello Jenkins Builder, laforge, pespin, lynxis lazus,

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

https://gerrit.osmocom.org/c/osmo-sgsn/+/21406

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

Change subject: gbproxy: Add NSE peer that can have multiple gbproxy_peers
..

gbproxy: Add NSE peer that can have multiple gbproxy_peers

We want this level of indirection to support multiple BVCs per NSE. The
current code assumes that an NSE only has one BVC which breaks messages
on the signalling BVC which should only be sent once to an NSE
regardless of the number of BVCs it contains.

Change-Id: I97cc6c8f8c0f1b91577ab8f679c4ae217cc88076
Related: SYS#5226
---
M include/osmocom/sgsn/gb_proxy.h
M src/gbproxy/gb_proxy.c
M src/gbproxy/gb_proxy_ctrl.c
M src/gbproxy/gb_proxy_patch.c
M src/gbproxy/gb_proxy_peer.c
M src/gbproxy/gb_proxy_tlli.c
M src/gbproxy/gb_proxy_vty.c
M tests/gbproxy/gbproxy_test.c
M tests/gbproxy/gbproxy_test.ok
9 files changed, 495 insertions(+), 285 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/06/21406/6
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/21406
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I97cc6c8f8c0f1b91577ab8f679c4ae217cc88076
Gerrit-Change-Number: 21406
Gerrit-PatchSet: 6
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: newpatchset


Change in osmo-sgsn[master]: gbproxy: Only send paging to each matching NSE once

2020-11-30 Thread daniel
Hello Jenkins Builder, laforge,

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

https://gerrit.osmocom.org/c/osmo-sgsn/+/21417

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

Change subject: gbproxy: Only send paging to each matching NSE once
..

gbproxy: Only send paging to each matching NSE once

Fixes: SYS#5226
Change-Id: Ia33e7816b354266b9d2659ef96b798db60c134f3
---
M src/gbproxy/gb_proxy.c
1 file changed, 34 insertions(+), 13 deletions(-)


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ia33e7816b354266b9d2659ef96b798db60c134f3
Gerrit-Change-Number: 21417
Gerrit-PatchSet: 2
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-MessageType: newpatchset


Change in osmo-sgsn[master]: gbproxy: Separate function to move gbproxy_peer to different nse

2020-11-30 Thread daniel
Hello Jenkins Builder, laforge, pespin,

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

https://gerrit.osmocom.org/c/osmo-sgsn/+/21418

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

Change subject: gbproxy: Separate function to move gbproxy_peer to different nse
..

gbproxy: Separate function to move gbproxy_peer to different nse

Change-Id: I0a8b1e4b78384ea99d50109f050ca501b18ba5d9
Related: SYS#5226
---
M include/osmocom/sgsn/gb_proxy.h
M src/gbproxy/gb_proxy.c
M src/gbproxy/gb_proxy_peer.c
3 files changed, 9 insertions(+), 3 deletions(-)


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I0a8b1e4b78384ea99d50109f050ca501b18ba5d9
Gerrit-Change-Number: 21418
Gerrit-PatchSet: 2
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: newpatchset


Change in osmo-sgsn[master]: gbproxy: Remove test testing NSVCI change

2020-11-30 Thread daniel
Hello Jenkins Builder, laforge, pespin,

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

https://gerrit.osmocom.org/c/osmo-sgsn/+/21413

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

Change subject: gbproxy: Remove test testing NSVCI change
..

gbproxy: Remove test testing NSVCI change

This was overlooked in commit 82182d which already removed lots of
NS-specific code in gbproxy_test.c

>From that commit message:
"""
Since NS2 has a different abstraction we mock up the prim send/recv
functions and don't test NS like the old tests did.
"""

Change-Id: Ic1d7e646e633c9fa62812f5005ed10c0108a06f2
Related: SYS#5226
---
M tests/gbproxy/gbproxy_test.c
M tests/gbproxy/gbproxy_test.ok
2 files changed, 2 insertions(+), 255 deletions(-)


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ic1d7e646e633c9fa62812f5005ed10c0108a06f2
Gerrit-Change-Number: 21413
Gerrit-PatchSet: 2
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: newpatchset


Change in osmo-sgsn[master]: gbproxy: Ensure BVC0 is reset before handling PtP BVCs

2020-11-30 Thread daniel
Hello Jenkins Builder, laforge, pespin,

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

https://gerrit.osmocom.org/c/osmo-sgsn/+/21410

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

Change subject: gbproxy: Ensure BVC0 is reset before handling PtP BVCs
..

gbproxy: Ensure BVC0 is reset before handling PtP BVCs

Change-Id: I9c67f973f7bca00e8eb22a024ef2282c007dd84b
Related: SYS#5226
---
M src/gbproxy/gb_proxy.c
M tests/gbproxy/gbproxy_test.c
M tests/gbproxy/gbproxy_test.ok
3 files changed, 125 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/10/21410/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/21410
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I9c67f973f7bca00e8eb22a024ef2282c007dd84b
Gerrit-Change-Number: 21410
Gerrit-PatchSet: 4
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: newpatchset


Change in osmo-gsm-tester[master]: Introduce Android UEs as new modems

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

Change subject: Introduce Android UEs as new modems
..


Patch Set 3: Code-Review-1

(13 comments)

This is starting to look good, there's only a few things here and there which 
needs polishing.

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_apn.py
File src/osmo_gsm_tester/obj/android_apn.py:

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_apn.py@30
PS3, Line 30: schema.register_resource_schema('apn', resource_schema)
This is probably not correct, as this should rpobably be under "modem" object 
or alike. So modem will probably end up adding this by using 
AndroidApn.schema() on its on_register_schemas()


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_apn.py@71
PS3, Line 71: def from_conf(cls, conf):
This one is PUBLIC INTERNAL API


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_apn.py@75
PS3, Line 75: @classmethod
This one is PUBLIC INTERNAL API


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_apn.py@87
PS3, Line 87: proc = self.run_androidue_cmd("set-apn", 
[qry_carrier_cmd])
"get-carrier-id" instead of "set-apn"? or is set-apn the binary? Not sure now 
if this is a description string or the binary name.


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_host.py
File src/osmo_gsm_tester/obj/android_host.py:

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_host.py@31
PS3, Line 31: def __init__(self, name, testenv, conf):
what's the point in passing these params to constructor if you don't use them? 
You can remove them


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_host.py@42
PS3, Line 42: log.dbg("Deleted the following files: %s" % 
clear_run_dir_proc.get_stdout())
You should better inherit from log.Origin in AndroidHost and use self.log. See 
how other classes do it.


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_host.py@50
PS3, Line 50: self.dbg("Deleted the following files: %s" % 
clear_diag_logs_proc.get_stdout())
see, you already expect to be inheriting from log.Origin, but you are not ;)
This probably works because subclasses are inheriting from log.Origin. You 
should be inheriting here instead, and call this constructor from child classes 
instead.


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/android_host.py@72
PS3, Line 72: if self._run_node.is_local():
I would welcome some description here too, specially what's the main difference 
with the method above and the rationale for having both.


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/iperf3.py
File src/osmo_gsm_tester/obj/iperf3.py:

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/iperf3.py@297
PS3, Line 297: elif self._run_node.ssh_port():
So the main difference between this  branch and the one below is that you need 
to force "root" user?
I'm not liking checking here based on whether ssh port is set or not. We should 
flag this somehow in the class implementing when passing its run_node by means 
of this class set_run_node().


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/iperf3.py@315
PS3, Line 315: self.process = process.Process(self.name(), 
self.run_dir, popen_args, env={})
Please add a new AdbProcess sublcass similar to NetNSProcess to hide this adb 
command stuff.


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/ms_android.py
File src/osmo_gsm_tester/obj/ms_android.py:

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/ms_android.py@43
PS3, Line 43: resource_schema['apn.%s' % key] = val
what I wrote in the first file of this patch: This is done here, so no need to 
have it in register_schemas() on the AndroidApn file.


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/ms_android.py@257
PS3, Line 257: ip_prefix = "172.16.0"
This API is probably PROTECTED?


https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/qc_diag.py
File src/osmo_gsm_tester/obj/qc_diag.py:

https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302/3/src/osmo_gsm_tester/obj/qc_diag.py@81
PS3, Line 81: while timer > 0:
I think in general you should be using event_loop.py's wait() method for this 
kind of sleeping loops.
See for instance:
"""
MainLoop.wait(self.dbus.has_interface, *req_ifaces, timeout=10)
"""



--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21302
To unsubscribe, or for 

Change in osmo-sgsn[master]: gbproxy: Remove test testing NSVCI change

2020-11-30 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21413 )

Change subject: gbproxy: Remove test testing NSVCI change
..


Patch Set 1: Code-Review+1

(1 comment)

would be good to get the commitlog amended so we can merge the other patches 
depending on this.

https://gerrit.osmocom.org/c/osmo-sgsn/+/21413/1//COMMIT_MSG
Commit Message:

https://gerrit.osmocom.org/c/osmo-sgsn/+/21413/1//COMMIT_MSG@9
PS1, Line 9: Testing this here doesn't make much sense since ns2.
> Why?
Ack



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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ic1d7e646e633c9fa62812f5005ed10c0108a06f2
Gerrit-Change-Number: 21413
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:34:48 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: pespin 
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Separate function to move gbproxy_peer to different nse

2020-11-30 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21418 )

Change subject: gbproxy: Separate function to move gbproxy_peer to different nse
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I0a8b1e4b78384ea99d50109f050ca501b18ba5d9
Gerrit-Change-Number: 21418
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:34:13 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Only send paging to each matching NSE once

2020-11-30 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21417 )

Change subject: gbproxy: Only send paging to each matching NSE once
..


Patch Set 1: Code-Review+1

one could probably introduce the per-NSE counters before this patch to avoid 
the FIXMe


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ia33e7816b354266b9d2659ef96b798db60c134f3
Gerrit-Change-Number: 21417
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:33:50 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Ensure BVC0 is reset before handling PtP BVCs

2020-11-30 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21410 )

Change subject: gbproxy: Ensure BVC0 is reset before handling PtP BVCs
..


Patch Set 3: Code-Review+2


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I9c67f973f7bca00e8eb22a024ef2282c007dd84b
Gerrit-Change-Number: 21410
Gerrit-PatchSet: 3
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:33:01 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: Process: Add remote_port as new parameter

2020-11-30 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21300 )

Change subject: Process: Add remote_port as new parameter
..

Process: Add remote_port as new parameter

remote_port defines a custom/additional port for
connections over ssh. It may be used in case several
ssh instances share one IP address.

Change-Id: I2c93fd2ea1c10c333d00eafd3c1066c35796e398
---
M src/osmo_gsm_tester/core/process.py
M src/osmo_gsm_tester/core/remote.py
M src/osmo_gsm_tester/obj/run_node.py
3 files changed, 21 insertions(+), 5 deletions(-)

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



diff --git a/src/osmo_gsm_tester/core/process.py 
b/src/osmo_gsm_tester/core/process.py
index eaf8de7..0b96f2e 100644
--- a/src/osmo_gsm_tester/core/process.py
+++ b/src/osmo_gsm_tester/core/process.py
@@ -381,12 +381,14 @@

 class RemoteProcess(Process):

-def __init__(self, name, run_dir, remote_user, remote_host, remote_cwd, 
popen_args, remote_env={}, **popen_kwargs):
+def __init__(self, name, run_dir, remote_user, remote_host, remote_cwd, 
popen_args,
+ remote_env={}, remote_port=None, **popen_kwargs):
 super().__init__(name, run_dir, popen_args, **popen_kwargs)
 self.remote_user = remote_user
 self.remote_host = remote_host
 self.remote_cwd = remote_cwd
 self.remote_env = remote_env
+self.remote_port = remote_port

 # hacky: instead of just prepending ssh, i.e. piping stdout and stderr
 # over the ssh link, we should probably run on the remote side,
@@ -402,6 +404,10 @@
'%s %s %s' % (cd,
  ' '.join(['%s=%r'%(k,v) for k,v in 
self.remote_env.items()]),
  ' '.join(self.popen_args))]
+if self.remote_port:
+self.popen_args.insert(1, '-p')
+self.popen_args.insert(2, self.remote_port)
+
 self.dbg(self.popen_args, dir=self.run_dir, conf=self.popen_kwargs, 
remote_env=self.remote_env)

 def RunError(self, msg_prefix):
diff --git a/src/osmo_gsm_tester/core/remote.py 
b/src/osmo_gsm_tester/core/remote.py
index 8deb25d..0ea873d 100644
--- a/src/osmo_gsm_tester/core/remote.py
+++ b/src/osmo_gsm_tester/core/remote.py
@@ -28,13 +28,14 @@

 WRAPPER_SCRIPT = 'ssh_sigkiller.sh'
 
-def __init__(self, run_dir, remote_user = 'root', remote_host = 
'localhost', remote_cwd=None):
+def __init__(self, run_dir, remote_user = 'root', remote_host = 
'localhost', remote_cwd=None, remote_port=None):
 super().__init__(log.C_RUN, 'host-' + remote_user + '@' + remote_host)
 self.run_dir = util.Dir(run_dir.new_dir(self.name()))
 self.remote_user = remote_user
 self.remote_host = remote_host
 self.remote_cwd = remote_cwd
 self.remote_env = {}
+self.remote_port = remote_port

 def user(self):
 return self.remote_user
@@ -51,9 +52,13 @@
 def get_remote_env(self):
 return self.remote_env

+def get_remote_port(self):
+return self.remote_port
+
 def RemoteProcess(self, name, popen_args, remote_env={}, **popen_kwargs):
 run_dir = self.run_dir.new_dir(name)
-return process.RemoteProcess(name, run_dir, self.user(), self.host(), 
self.cwd(), popen_args, remote_env=remote_env, **popen_kwargs)
+return process.RemoteProcess(name, run_dir, self.user(), self.host(), 
self.cwd(), popen_args,
+ remote_env=remote_env, 
remote_port=self.get_remote_port(), **popen_kwargs)

 def generate_wrapper_script(self, wait_time_sec):
 wrapper_script = self.run_dir.new_file(RemoteHost.WRAPPER_SCRIPT)
diff --git a/src/osmo_gsm_tester/obj/run_node.py 
b/src/osmo_gsm_tester/obj/run_node.py
index 6a030ac..7c41705 100644
--- a/src/osmo_gsm_tester/obj/run_node.py
+++ b/src/osmo_gsm_tester/obj/run_node.py
@@ -30,13 +30,14 @@
 T_LOCAL = 'local'
 T_REM_SSH = 'ssh'

-def __init__(self, type=None, run_addr=None, ssh_user=None, ssh_addr=None, 
run_label=None):
+def __init__(self, type=None, run_addr=None, ssh_user=None, ssh_addr=None, 
run_label=None, ssh_port=None):
 super().__init__(log.C_RUN, 'runnode')
 self._type = type
 self._run_addr = run_addr
 self._ssh_user = ssh_user
 self._ssh_addr = ssh_addr
 self._run_label = run_label
+self._ssh_port = ssh_port
 if not self._type:
 raise log.Error('run_type not set')
 if not self._run_addr:
@@ -55,7 +56,7 @@
 def from_conf(cls, conf):
 return cls(conf.get('run_type', None), conf.get('run_addr', None),
conf.get('ssh_user', None), conf.get('ssh_addr', None),
-   conf.get('run_label', None))
+   conf.get('run_label', None), conf.get('ssh_port', None))

 

Change in osmo-sgsn[master]: gbproxy: Add NSE peer that can have multiple gbproxy_peers

2020-11-30 Thread laforge
laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21406 )

Change subject: gbproxy: Add NSE peer that can have multiple gbproxy_peers
..


Patch Set 5: Code-Review+2


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I97cc6c8f8c0f1b91577ab8f679c4ae217cc88076
Gerrit-Change-Number: 21406
Gerrit-PatchSet: 5
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:32:42 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: Process: Add remote_port as new parameter

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

Change subject: Process: Add remote_port as new parameter
..


Patch Set 2: Code-Review+2


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

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I2c93fd2ea1c10c333d00eafd3c1066c35796e398
Gerrit-Change-Number: 21300
Gerrit-PatchSet: 2
Gerrit-Owner: ninjab3s 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:32:21 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Separate function to move gbproxy_peer to different nse

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21418 )

Change subject: gbproxy: Separate function to move gbproxy_peer to different nse
..


Patch Set 1: Code-Review+1


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I0a8b1e4b78384ea99d50109f050ca501b18ba5d9
Gerrit-Change-Number: 21418
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:30:17 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Add NSE peer that can have multiple gbproxy_peers

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21406 )

Change subject: gbproxy: Add NSE peer that can have multiple gbproxy_peers
..


Patch Set 5: Code-Review+1


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I97cc6c8f8c0f1b91577ab8f679c4ae217cc88076
Gerrit-Change-Number: 21406
Gerrit-PatchSet: 5
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-CC: laforge 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:28:29 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmocom-bb[master]: trxcon/scheduler: invalidate hard-coded Measurement results

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/21416 )

Change subject: trxcon/scheduler: invalidate hard-coded Measurement results
..


Patch Set 1: Code-Review+1


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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I7da767e146aec7cef1de71e4d735d6a02b6c5642
Gerrit-Change-Number: 21416
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:21:19 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmocom-bb[master]: trxcon/scheduler: clarify the content of Measurement results

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/21415 )

Change subject: trxcon/scheduler: clarify the content of Measurement results
..


Patch Set 1: Code-Review+1


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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I26546dcbc853166e351d00260936b1b9d584ae03
Gerrit-Change-Number: 21415
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:20:13 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmocom-bb[master]: trxcon/scheduler: clarify Measurement results padding

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/21414 )

Change subject: trxcon/scheduler: clarify Measurement results padding
..


Patch Set 1: Code-Review+1


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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I4db6845c98aded10291134f416da98fd0f4f58e3
Gerrit-Change-Number: 21414
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:18:42 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Remove test testing NSVCI change

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21413 )

Change subject: gbproxy: Remove test testing NSVCI change
..


Patch Set 1:

(1 comment)

https://gerrit.osmocom.org/c/osmo-sgsn/+/21413/1//COMMIT_MSG
Commit Message:

https://gerrit.osmocom.org/c/osmo-sgsn/+/21413/1//COMMIT_MSG@9
PS1, Line 9: Testing this here doesn't make much sense since ns2.
Why?



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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ic1d7e646e633c9fa62812f5005ed10c0108a06f2
Gerrit-Change-Number: 21413
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:18:16 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Ensure BVC0 is reset before handling PtP BVCs

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21410 )

Change subject: gbproxy: Ensure BVC0 is reset before handling PtP BVCs
..


Patch Set 3: Code-Review+1


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I9c67f973f7bca00e8eb22a024ef2282c007dd84b
Gerrit-Change-Number: 21410
Gerrit-PatchSet: 3
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 16:17:40 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Separate function to move gbproxy_peer to different nse

2020-11-30 Thread daniel
daniel has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21418 )


Change subject: gbproxy: Separate function to move gbproxy_peer to different nse
..

gbproxy: Separate function to move gbproxy_peer to different nse

Change-Id: I0a8b1e4b78384ea99d50109f050ca501b18ba5d9
Related: SYS#5226
---
M include/osmocom/sgsn/gb_proxy.h
M src/gbproxy/gb_proxy.c
M src/gbproxy/gb_proxy_peer.c
3 files changed, 9 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/18/21418/1

diff --git a/include/osmocom/sgsn/gb_proxy.h b/include/osmocom/sgsn/gb_proxy.h
index 54d4548..9f91bd6 100644
--- a/include/osmocom/sgsn/gb_proxy.h
+++ b/include/osmocom/sgsn/gb_proxy.h
@@ -344,6 +344,7 @@
struct gbproxy_config *cfg, struct tlv_parsed *tp);
 struct gbproxy_peer *gbproxy_peer_alloc(struct gbproxy_nse *nse, uint16_t 
bvci);
 void gbproxy_peer_free(struct gbproxy_peer *peer);
+void gbproxy_peer_move(struct gbproxy_peer *peer, struct gbproxy_nse *nse);
 int gbproxy_cleanup_peers(struct gbproxy_config *cfg, uint16_t nsei, uint16_t 
bvci);

 /* NSE handling */
diff --git a/src/gbproxy/gb_proxy.c b/src/gbproxy/gb_proxy.c
index db7f55b..bd58617 100644
--- a/src/gbproxy/gb_proxy.c
+++ b/src/gbproxy/gb_proxy.c
@@ -1114,9 +1114,7 @@
LOGP(DGPRS, LOGL_NOTICE, "Peer for BVCI=%u 
moved from NSEI=%u to NSEI=%u\n", bvci, nse_old->nsei, nsei);

/* Move peer to different NSE */
-   llist_del(_peer->list);
-   llist_add(_peer->list, 
_new->bts_peers);
-   from_peer->nse = nse_new;
+   gbproxy_peer_move(from_peer, nse_new);
}

if (TLVP_PRESENT(, BSSGP_IE_CELL_ID)) {
diff --git a/src/gbproxy/gb_proxy_peer.c b/src/gbproxy/gb_proxy_peer.c
index 920547c..62c4d94 100644
--- a/src/gbproxy/gb_proxy_peer.c
+++ b/src/gbproxy/gb_proxy_peer.c
@@ -253,6 +253,13 @@
talloc_free(peer);
 }

+void gbproxy_peer_move(struct gbproxy_peer *peer, struct gbproxy_nse *nse)
+{
+   llist_del(>list);
+   llist_add(>list, >bts_peers);
+   peer->nse = nse;
+}
+
 int gbproxy_cleanup_peers(struct gbproxy_config *cfg, uint16_t nsei, uint16_t 
bvci)
 {
int counter = 0;

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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I0a8b1e4b78384ea99d50109f050ca501b18ba5d9
Gerrit-Change-Number: 21418
Gerrit-PatchSet: 1
Gerrit-Owner: daniel 
Gerrit-MessageType: newchange


Change in osmo-pcu[master]: Dl TBF: Get rid of LLC UI dummy blocks following other data

2020-11-30 Thread pespin
Hello Jenkins Builder, fixeria,

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

https://gerrit.osmocom.org/c/osmo-pcu/+/21387

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

Change subject: Dl TBF: Get rid of LLC UI dummy blocks following other data
..

Dl TBF: Get rid of LLC UI dummy blocks following other data

According to:
* 3GPP TS 44.060 version 16.0.0 "9.3.1a Delayed release of downlink Temporary 
Block Flow"
* 3GPP TS 44.064 version 16.0.0 "6.4.2.2 Unconfirmed Information (UI) Dummy 
command"

LLC UI Dummy frames are to be used when there no more data to send, only
in order to delay the release of a TBF. Hence, while not incorrect per
se, makes no sense to send those LLC UI Dummy frames inserted into
rlcmac blocks which already contain other LLC frames, since the MS in
that case is already being kept active.
It only makes sense to send those LLC UI Dummy frames when we have
nothing else to send, that is, alone inside a RLCMAC block without other
LLC frames.

Related: OS#4849
Change-Id: Ifae1a7b2b3dfad8df19585063088ba0df2749c8f
---
M src/encoding.cpp
M src/encoding.h
M src/tbf_dl.cpp
M tests/tbf/TbfTest.err
4 files changed, 170 insertions(+), 198 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/87/21387/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/21387
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Ifae1a7b2b3dfad8df19585063088ba0df2749c8f
Gerrit-Change-Number: 21387
Gerrit-PatchSet: 4
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-MessageType: newpatchset


Change in osmo-sgsn[master]: gbproxy: Only send paging to each matching NSE once

2020-11-30 Thread daniel
daniel has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21417 )


Change subject: gbproxy: Only send paging to each matching NSE once
..

gbproxy: Only send paging to each matching NSE once

Fixes: SYS#5226
Change-Id: Ia33e7816b354266b9d2659ef96b798db60c134f3
---
M src/gbproxy/gb_proxy.c
1 file changed, 34 insertions(+), 13 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/17/21417/1

diff --git a/src/gbproxy/gb_proxy.c b/src/gbproxy/gb_proxy.c
index 1ca4fc6..db7f55b 100644
--- a/src/gbproxy/gb_proxy.c
+++ b/src/gbproxy/gb_proxy.c
@@ -813,11 +813,10 @@
return rc;
 }

-/* feed a message down the NS-VC associated with the specified peer */
-static int gbprox_relay2peer(struct msgb *old_msg, struct gbproxy_peer *peer,
+/* feed a message down the NSE */
+static int gbprox_relay2nse(struct msgb *old_msg, struct gbproxy_nse *nse,
 uint16_t ns_bvci)
 {
-   struct gbproxy_nse *nse = peer->nse;
OSMO_ASSERT(nse);
OSMO_ASSERT(nse->cfg);

@@ -825,7 +824,7 @@
 * be free()d safely when we return from gbprox_rcvmsg() */
struct gprs_ns2_inst *nsi = nse->cfg->nsi;
struct osmo_gprs_ns2_prim nsp = {};
-   struct msgb *msg = bssgp_msgb_copy(old_msg, "msgb_relay2peer");
+   struct msgb *msg = bssgp_msgb_copy(old_msg, "msgb_relay2nse");
uint32_t tlli;
int rc;

@@ -849,6 +848,22 @@
osmo_prim_init(, SAP_NS, PRIM_NS_UNIT_DATA,
   PRIM_OP_REQUEST, msg);
rc = gprs_ns2_recv_prim(nsi, );
+   /* FIXME: We need a counter group for gbproxy_nse */
+   //if (rc < 0)
+   //  rate_ctr_inc(>ctrg->ctr[GBPROX_PEER_CTR_TX_ERR]);
+
+   return rc;
+}
+
+/* feed a message down the NS-VC associated with the specified peer */
+static int gbprox_relay2peer(struct msgb *old_msg, struct gbproxy_peer *peer,
+uint16_t ns_bvci)
+{
+   int rc;
+   struct gbproxy_nse *nse = peer->nse;
+   OSMO_ASSERT(nse);
+
+   rc = gbprox_relay2nse(old_msg, nse, ns_bvci);
if (rc < 0)
rate_ctr_inc(>ctrg->ctr[GBPROX_PEER_CTR_TX_ERR]);

@@ -1151,7 +1166,7 @@
 {
struct gbproxy_nse *nse;
struct gbproxy_peer *peer;
-   unsigned int n_peers = 0;
+   unsigned int n_nses = 0;
int errctr = GBPROX_GLOB_CTR_PROTO_ERR_SGSN;

/* FIXME: Handle paging logic to only page each matching NSE */
@@ -1176,9 +1191,11 @@
llist_for_each_entry(nse, >nse_peers, list) {
llist_for_each_entry(peer, >bts_peers, list) {
if (!memcmp(peer->ra, TLVP_VAL(tp, 
BSSGP_IE_ROUTEING_AREA), 6)) {
-   LOGPC(DGPRS, LOGL_INFO, "routing by RAI 
to peer BVCI=%u\n", peer->bvci);
-   gbprox_relay2peer(msg, peer, ns_bvci);
-   n_peers++;
+   LOGPC(DGPRS, LOGL_INFO, "routing by RAI 
to peer NSEI=%u\n", peer->bvci);
+   gbprox_relay2nse(msg, nse, ns_bvci);
+   n_nses++;
+   /* Only send it once to each NSE */
+   break;
}
}
}
@@ -1189,8 +1206,10 @@
llist_for_each_entry(peer, >bts_peers, list) {
if (!memcmp(peer->ra, TLVP_VAL(tp, 
BSSGP_IE_LOCATION_AREA), 5)) {
LOGPC(DGPRS, LOGL_INFO, "routing by LAI 
to peer BVCI=%u\n", peer->bvci);
-   gbprox_relay2peer(msg, peer, ns_bvci);
-   n_peers++;
+   gbprox_relay2nse(msg, nse, ns_bvci);
+   n_nses++;
+   /* Only send it once to each NSE */
+   break;
}
}
}
@@ -1199,8 +1218,10 @@
llist_for_each_entry(nse, >nse_peers, list) {
llist_for_each_entry(peer, >bts_peers, list) {
LOGPC(DGPRS, LOGL_INFO, "broadcasting to peer 
BVCI=%u\n", peer->bvci);
-   gbprox_relay2peer(msg, peer, ns_bvci);
-   n_peers++;
+   gbprox_relay2nse(msg, nse, ns_bvci);
+   n_nses++;
+   /* Only send it once to each NSE */
+   break;
}
}
} else {
@@ -1210,7 +1231,7 @@
rate_ctr_inc(>ctrg->ctr[errctr]);
}

-   if 

Change in osmo-ttcn3-hacks[master]: bsc: add TC_assignment_codec_amr_startmode_cruft

2020-11-30 Thread neels
neels has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21373 )

Change subject: bsc: add TC_assignment_codec_amr_startmode_cruft
..

bsc: add TC_assignment_codec_amr_startmode_cruft

Make sure that when a 'start-mode auto' is set, that the previous start
mode setting does not linger in the unused bits.

- after I577ff590d7588fd7e3ee4846c7955ab8f84cf2b1, osmo-bsc sets its
  ICMI bit properly, but passes this test only because it *always* sends
  smod (start-mode) bits as zero.

- in I49691df01745a7c485bf165e897872c35fc4b147, the smod bits are
  properly sent on RSL, but this test shows that when ICMI becomes zero
  for 'start-mode auto', the smod bits will remain whatever start-mode
  was set in the previous osmo-bsc config. Instead, osmo-bsc should
  clear the smod bits for 'start-mode auto' so that its MultiRate Config
  does not vary depending on what was previously configured.

- in I1ec5bad0bce01cc425ee05ecf70c83ec662a226a, clearing smod is
  implemented and this test is expected to pass.

Change-Id: I151678f64e680f30f35b6bb2b0036d63efde9f2c
---
M bsc/BSC_Tests.ttcn
1 file changed, 37 insertions(+), 0 deletions(-)

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



diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn
index 251801b..7c12559 100644
--- a/bsc/BSC_Tests.ttcn
+++ b/bsc/BSC_Tests.ttcn
@@ -3711,6 +3711,42 @@
f_vty_amr_start_mode_restore(false);
 }

+/* 'amr start-mode auto' should not keep the (unused) 'smod' bits from 
previous configuration */
+testcase TC_assignment_codec_amr_startmode_cruft() runs on test_CT {
+   var TestHdlrParams pars := f_gen_test_hdlr_pars();
+   var MSC_ConnHdlr vc_conn;
+
+   var RSL_IE_Body mr_conf := {
+   other := {
+   len := 2,
+   payload := '2004'O /* <- expect ICMI=0, smod=00 */
+   }
+   };
+
+   pars.ass_codec_list := valueof(ts_BSSMAP_IE_CodecList({ts_CodecAMR_F}));
+   pars.ass_codec_list.codecElements[0].s0_7 := '0100'B; /* 5,90k */
+   pars.ass_codec_list.codecElements[0].s8_15 := '01010111'B;
+   pars.expect_mr_conf_ie := mr_conf;
+
+   f_init(1, true);
+   f_sleep(1.0);
+
+   /* First set nonzero start mode bits */
+   f_vty_amr_start_mode_set(true, "4");
+   /* Now set to auto, and expect the startmode bits to be zero in the 
message, i.e. ensure that osmo-bsc does not
+* let the startmode bits stick around and has deterministic MultiRate 
config for 'start-mode auto'; that is
+* ensured by above '2004'O, where 'x0xx'O indicates ICMI = 0, spare = 
0, smod = 00. */
+   f_vty_amr_start_mode_set(true, "auto");
+
+   vc_conn := f_start_handler(refers(f_TC_assignment_codec), pars);
+   vc_conn.done;
+   f_shutdown_helper();
+
+   /* Clear the startmode bits to not affect subsequent tests, in case the 
bits should indeed stick around. */
+   f_vty_amr_start_mode_set(true, "1");
+   f_vty_amr_start_mode_restore(true);
+}
+
 function f_TC_assignment_codec_amr(boolean fr, octetstring mrconf, bitstring 
s8_s0, bitstring exp_s8_s0,
charstring start_mode := "1")
 runs on test_CT {
@@ -8415,6 +8451,7 @@
execute( TC_assignment_codec_amr_h_S7() );
execute( TC_assignment_codec_amr_f_start_mode_auto() );
execute( TC_assignment_codec_amr_h_start_mode_auto() );
+   execute( TC_assignment_codec_amr_startmode_cruft() );
}

execute( TC_assignment_codec_fr_exhausted_req_hr() );

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


Change in osmo-ttcn3-hacks[master]: bsc: add tests for AMR 'start-mode auto'

2020-11-30 Thread neels
neels has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21371 )

Change subject: bsc: add tests for AMR 'start-mode auto'
..

bsc: add tests for AMR 'start-mode auto'

These tests will fail until osmo-bsc
I577ff590d7588fd7e3ee4846c7955ab8f84cf2b1

Related: OS#4868
Change-Id: Iec74b940009c8ec78a6da5c00615259e2edc1781
---
M bsc/BSC_Tests.ttcn
1 file changed, 15 insertions(+), 2 deletions(-)

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



diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn
index 4cbc256..251801b 100644
--- a/bsc/BSC_Tests.ttcn
+++ b/bsc/BSC_Tests.ttcn
@@ -3711,7 +3711,8 @@
f_vty_amr_start_mode_restore(false);
 }

-function f_TC_assignment_codec_amr(boolean fr, octetstring mrconf, bitstring 
s8_s0, bitstring exp_s8_s0)
+function f_TC_assignment_codec_amr(boolean fr, octetstring mrconf, bitstring 
s8_s0, bitstring exp_s8_s0,
+   charstring start_mode := "1")
 runs on test_CT {

var TestHdlrParams pars := f_gen_test_hdlr_pars();
@@ -3737,7 +3738,7 @@

f_init(1, true);
f_allow_amr_rate_4_75k_5_90k_7_40k_12_20k();
-   f_vty_amr_start_mode_set(fr, "1");
+   f_vty_amr_start_mode_set(fr, start_mode);
f_sleep(1.0);

vc_conn := f_start_handler(refers(f_TC_assignment_codec), pars);
@@ -3886,6 +3887,16 @@
f_shutdown_helper();
 }

+testcase TC_assignment_codec_amr_f_start_mode_auto() runs on test_CT {
+   f_TC_assignment_codec_amr(true, '209520882208'O, ''B, 
'0010'B,
+   start_mode := "auto");
+}
+
+testcase TC_assignment_codec_amr_h_start_mode_auto() runs on test_CT {
+   f_TC_assignment_codec_amr(false, '2015208820'O, '10010101'B, 
'00010101'B,
+   start_mode := "auto");
+}
+
 private function f_disable_all_tch_f() runs on test_CT {
f_vty_transceive(BSCVTY, "bts 0 trx 0 timeslot 1 sub-slot 0 borken");
f_vty_transceive(BSCVTY, "bts 0 trx 0 timeslot 2 sub-slot 0 borken");
@@ -8402,6 +8413,8 @@
execute( TC_assignment_codec_amr_f_zero() );
execute( TC_assignment_codec_amr_f_unsupp() );
execute( TC_assignment_codec_amr_h_S7() );
+   execute( TC_assignment_codec_amr_f_start_mode_auto() );
+   execute( TC_assignment_codec_amr_h_start_mode_auto() );
}

execute( TC_assignment_codec_fr_exhausted_req_hr() );

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


Change in osmo-ttcn3-hacks[master]: bsc: set the AMR start-mode that the tests expect

2020-11-30 Thread neels
neels has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21370 )

Change subject: bsc: set the AMR start-mode that the tests expect
..

bsc: set the AMR start-mode that the tests expect

osmo-bsc currently has a bug that fails to reflect the correct
start-mode in the AMR MultiRate config IE.

And it went unnoticed that the ttcn tests expect a MultiRate config of
ICMI = 1, even though the used configuration should yield ICMI = 0.
See mr_conf = '2804'O, where the '8' indicates ICMI = 1.

As a first fix of the ttcn3-bsc-tests, configure the BSC according to
the expected ICMI value and Start Mode, i.e. ICMI = 1 and StartMode = 0,
which is configured by 'amr tch-[fh] start-mode 1'. This should make
these tests pass as-is for both the current osmo-bsc as well as an
osmo-bsc where the bug is fixed, with minimal changes to the current
tests. See also OS#4868.

An upcoming patch will add tests for 'start-mode auto'.

Related: OS#4868
Change-Id: I4cff01c37d5c7e301e9a01f773b7e009a789519b
---
M bsc/BSC_Tests.ttcn
1 file changed, 28 insertions(+), 0 deletions(-)

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



diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn
index cda0687..4cbc256 100644
--- a/bsc/BSC_Tests.ttcn
+++ b/bsc/BSC_Tests.ttcn
@@ -3638,6 +3638,21 @@
});
 }

+private function f_vty_amr_start_mode_set(boolean fr, charstring startmode) 
runs on test_CT {
+   var charstring tch;
+   if (fr) {
+   tch := "tch-f";
+   } else {
+   tch := "tch-h";
+   }
+   f_vty_cfg_bts(BSCVTY, 0, { "amr " & tch & " start-mode " & startmode });
+}
+
+/* Set the AMR start-mode for this TCH back to the default configuration. */
+private function f_vty_amr_start_mode_restore(boolean fr) runs on test_CT {
+   f_vty_amr_start_mode_set(fr, "auto");
+}
+
 testcase TC_assignment_codec_amr_f() runs on test_CT {
var TestHdlrParams pars := f_gen_test_hdlr_pars();
var MSC_ConnHdlr vc_conn;
@@ -3659,10 +3674,13 @@

f_init(1, true);
f_sleep(1.0);
+   f_vty_amr_start_mode_set(true, "1");

vc_conn := f_start_handler(refers(f_TC_assignment_codec), pars);
vc_conn.done;
f_shutdown_helper();
+
+   f_vty_amr_start_mode_restore(true);
 }

 testcase TC_assignment_codec_amr_h() runs on test_CT {
@@ -3684,10 +3702,13 @@

f_init(1, true);
f_sleep(1.0);
+   f_vty_amr_start_mode_set(false, "1");

vc_conn := f_start_handler(refers(f_TC_assignment_codec), pars);
vc_conn.done;
f_shutdown_helper();
+
+   f_vty_amr_start_mode_restore(false);
 }

 function f_TC_assignment_codec_amr(boolean fr, octetstring mrconf, bitstring 
s8_s0, bitstring exp_s8_s0)
@@ -3716,11 +3737,13 @@

f_init(1, true);
f_allow_amr_rate_4_75k_5_90k_7_40k_12_20k();
+   f_vty_amr_start_mode_set(fr, "1");
f_sleep(1.0);

vc_conn := f_start_handler(refers(f_TC_assignment_codec), pars);
vc_conn.done;
f_allow_amr_rate_5_90k();
+   f_vty_amr_start_mode_restore(fr);
 }

 function f_TC_assignment_codec_amr_fail(boolean fr, bitstring s8_s0)
@@ -3739,11 +3762,13 @@

f_init(1, true);
f_allow_amr_rate_4_75k_5_90k_7_40k_12_20k();
+   f_vty_amr_start_mode_set(fr, "1");
f_sleep(1.0);

vc_conn := f_start_handler(refers(f_TC_assignment_codec_fail), pars);
vc_conn.done;
f_allow_amr_rate_5_90k();
+   f_vty_amr_start_mode_restore(fr);
 }


@@ -4164,10 +4189,13 @@

f_init(1, true, true);
f_sleep(1.0);
+   f_vty_amr_start_mode_set(false, "1");

vc_conn := f_start_handler(refers(f_TC_assignment_codec), pars);
vc_conn.done;
f_shutdown_helper();
+
+   f_vty_amr_start_mode_restore(false);
 }

 /* test the procedure of the MSC requesting a Classmark Update:

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


Change in osmo-ttcn3-hacks[master]: add f_vty_cfg_bts(), f_vty_cfg_msc()

2020-11-30 Thread neels
neels has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21369 )

Change subject: add f_vty_cfg_bts(), f_vty_cfg_msc()
..

add f_vty_cfg_bts(), f_vty_cfg_msc()

These allow passing N vty configurations on the bts / msc node without
requiring subsequent 'exit'.

As an example, use f_vty_cfg_msc() in BSC_Tests.ttcn AMR config.

Change-Id: I9f3e485f692acb3d2a7620e9b454b372651be78e
---
M bsc/BSC_Tests.ttcn
M library/Osmocom_VTY_Functions.ttcn
2 files changed, 28 insertions(+), 22 deletions(-)

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



diff --git a/bsc/BSC_Tests.ttcn b/bsc/BSC_Tests.ttcn
index a63d536..cda0687 100644
--- a/bsc/BSC_Tests.ttcn
+++ b/bsc/BSC_Tests.ttcn
@@ -3611,33 +3611,31 @@

 /* Allow 5,90k only (current default config) */
 private function f_allow_amr_rate_5_90k() runs on test_CT {
-   f_vty_enter_cfg_msc(BSCVTY, 0);
-   f_vty_transceive(BSCVTY, "amr-config 12_2k forbidden");
-   f_vty_transceive(BSCVTY, "amr-config 10_2k forbidden");
-   f_vty_transceive(BSCVTY, "amr-config 7_95k forbidden");
-   f_vty_transceive(BSCVTY, "amr-config 7_40k forbidden");
-   f_vty_transceive(BSCVTY, "amr-config 6_70k forbidden");
-   f_vty_transceive(BSCVTY, "amr-config 5_90k allowed");
-   f_vty_transceive(BSCVTY, "amr-config 5_15k forbidden");
-   f_vty_transceive(BSCVTY, "amr-config 4_75k forbidden");
-   f_vty_transceive(BSCVTY, "exit");
-   f_vty_transceive(BSCVTY, "exit");
+   f_vty_cfg_msc(BSCVTY, 0, {
+   "amr-config 12_2k forbidden",
+   "amr-config 10_2k forbidden",
+   "amr-config 7_95k forbidden",
+   "amr-config 7_40k forbidden",
+   "amr-config 6_70k forbidden",
+   "amr-config 5_90k allowed",
+   "amr-config 5_15k forbidden",
+   "amr-config 4_75k forbidden"
+   });
 }

 /* Allow 4,75k, 5,90k, 4,70k and 12,2k, which are the most common rates
  * ("Config-NB-Code = 1") */
 private function f_allow_amr_rate_4_75k_5_90k_7_40k_12_20k() runs on test_CT {
-   f_vty_enter_cfg_msc(BSCVTY, 0);
-   f_vty_transceive(BSCVTY, "amr-config 12_2k allowed");
-   f_vty_transceive(BSCVTY, "amr-config 10_2k forbidden");
-   f_vty_transceive(BSCVTY, "amr-config 7_95k forbidden");
-   f_vty_transceive(BSCVTY, "amr-config 7_40k allowed");
-   f_vty_transceive(BSCVTY, "amr-config 6_70k forbidden");
-   f_vty_transceive(BSCVTY, "amr-config 5_90k allowed");
-   f_vty_transceive(BSCVTY, "amr-config 5_15k forbidden");
-   f_vty_transceive(BSCVTY, "amr-config 4_75k allowed");
-   f_vty_transceive(BSCVTY, "exit");
-   f_vty_transceive(BSCVTY, "exit");
+   f_vty_cfg_msc(BSCVTY, 0, {
+   "amr-config 12_2k allowed",
+   "amr-config 10_2k forbidden",
+   "amr-config 7_95k forbidden",
+   "amr-config 7_40k allowed",
+   "amr-config 6_70k forbidden",
+   "amr-config 5_90k allowed",
+   "amr-config 5_15k forbidden",
+   "amr-config 4_75k allowed"
+   });
 }

 testcase TC_assignment_codec_amr_f() runs on test_CT {
diff --git a/library/Osmocom_VTY_Functions.ttcn 
b/library/Osmocom_VTY_Functions.ttcn
index a51794f..b291b03 100644
--- a/library/Osmocom_VTY_Functions.ttcn
+++ b/library/Osmocom_VTY_Functions.ttcn
@@ -174,6 +174,14 @@
f_vty_config2(pt, {config_node}, cmd);
 }

+function f_vty_cfg_bts(TELNETasp_PT pt, BtsNr bts := 0, rof_charstring cmds) {
+   f_vty_config3(pt, {"network", "bts " & int2str(bts)}, cmds);
+}
+
+function f_vty_cfg_msc(TELNETasp_PT pt, MscNr msc := 0, rof_charstring cmds) {
+   f_vty_config3(pt, {"msc " & int2str(msc)}, cmds);
+}
+
 function f_vty_transceive_match(TELNETasp_PT pt, charstring cmd, template 
charstring exp_ret) {
var charstring ret := f_vty_transceive_ret(pt, cmd);
if (not match(ret, exp_ret)) {

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


Change in osmo-ttcn3-hacks[master]: add f_vty_config3(), to run N commands on a sub-node

2020-11-30 Thread neels
neels has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21368 )

Change subject: add f_vty_config3(), to run N commands on a sub-node
..

add f_vty_config3(), to run N commands on a sub-node

f_vty_config2() makes it convenient to enter a specific vty node without
needing to send 'exit'/'end' explicitly. However, to pass multiple
commands on the same node, the VTY would enter and exit the node for
each call of f_vty_config2(). The new f_vty_config3() also allows
multiple commands to be run on that same node without intermediate
exiting.

Change-Id: If969ac645aa82e5a36245d974de2a251633de111
---
M library/Osmocom_VTY_Functions.ttcn
1 file changed, 9 insertions(+), 3 deletions(-)

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



diff --git a/library/Osmocom_VTY_Functions.ttcn 
b/library/Osmocom_VTY_Functions.ttcn
index 042f244..a51794f 100644
--- a/library/Osmocom_VTY_Functions.ttcn
+++ b/library/Osmocom_VTY_Functions.ttcn
@@ -149,19 +149,25 @@
}

 type record of charstring rof_charstring;
-function f_vty_config2(TELNETasp_PT pt, rof_charstring config_nodes, 
charstring cmd)
+function f_vty_config3(TELNETasp_PT pt, rof_charstring config_nodes, 
rof_charstring cmds)
 {
/* enter config mode; enter node */
f_vty_enter_config(pt);
for (var integer i := 0; i < sizeof(config_nodes); i := i+1) {
f_vty_transceive(pt, config_nodes[i]);
}
-   /* execute command */
-   f_vty_transceive(pt, cmd);
+   /* execute commands */
+   for (var integer i := 0; i < sizeof(cmds); i := i+1) {
+   f_vty_transceive(pt, cmds[i]);
+   }
/* leave config mode */
f_vty_transceive(pt, "end");
 }

+function f_vty_config2(TELNETasp_PT pt, rof_charstring config_nodes, 
charstring cmd)
+{
+   f_vty_config3(pt, config_nodes, { cmd });
+}

 function f_vty_config(TELNETasp_PT pt, charstring config_node, charstring cmd)
 {

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


Change in osmo-ttcn3-hacks[master]: bsc: add TC_assignment_codec_amr_startmode_cruft

2020-11-30 Thread neels
neels has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21373 )

Change subject: bsc: add TC_assignment_codec_amr_startmode_cruft
..


Patch Set 2: Code-Review+2

re-add +2 after adding comment


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21373
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: I151678f64e680f30f35b6bb2b0036d63efde9f2c
Gerrit-Change-Number: 21373
Gerrit-PatchSet: 2
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 15:58:49 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: bsc: add TC_assignment_codec_amr_startmode_cruft

2020-11-30 Thread neels
Hello Jenkins Builder, laforge, pespin,

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

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

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

Change subject: bsc: add TC_assignment_codec_amr_startmode_cruft
..

bsc: add TC_assignment_codec_amr_startmode_cruft

Make sure that when a 'start-mode auto' is set, that the previous start
mode setting does not linger in the unused bits.

- after I577ff590d7588fd7e3ee4846c7955ab8f84cf2b1, osmo-bsc sets its
  ICMI bit properly, but passes this test only because it *always* sends
  smod (start-mode) bits as zero.

- in I49691df01745a7c485bf165e897872c35fc4b147, the smod bits are
  properly sent on RSL, but this test shows that when ICMI becomes zero
  for 'start-mode auto', the smod bits will remain whatever start-mode
  was set in the previous osmo-bsc config. Instead, osmo-bsc should
  clear the smod bits for 'start-mode auto' so that its MultiRate Config
  does not vary depending on what was previously configured.

- in I1ec5bad0bce01cc425ee05ecf70c83ec662a226a, clearing smod is
  implemented and this test is expected to pass.

Change-Id: I151678f64e680f30f35b6bb2b0036d63efde9f2c
---
M bsc/BSC_Tests.ttcn
1 file changed, 37 insertions(+), 0 deletions(-)


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


Change in osmo-ttcn3-hacks[master]: bsc: add TC_assignment_codec_amr_startmode_cruft

2020-11-30 Thread neels
neels has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21373 )

Change subject: bsc: add TC_assignment_codec_amr_startmode_cruft
..


Patch Set 1:

(1 comment)

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

https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21373/1/bsc/BSC_Tests.ttcn@3714
PS1, Line 3714: testcase TC_assignment_codec_amr_startmode_cruft() runs on 
test_CT {
> in general a short description of the test in the line above is welcome.
forgot, thx



--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21373
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: I151678f64e680f30f35b6bb2b0036d63efde9f2c
Gerrit-Change-Number: 21373
Gerrit-PatchSet: 1
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 15:53:26 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin 
Gerrit-MessageType: comment


Change in osmo-pcu[master]: Dl TBF: Get rid of LLC UI dummy blocks following other data

2020-11-30 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/21387 )

Change subject: Dl TBF: Get rid of LLC UI dummy blocks following other data
..


Patch Set 3: Code-Review+1

(1 comment)

We need a TTCN-3 test case (of course)!

https://gerrit.osmocom.org/c/osmo-pcu/+/21387/3/src/encoding.cpp
File src/encoding.cpp:

https://gerrit.osmocom.org/c/osmo-pcu/+/21387/3/src/encoding.cpp@1688
PS3, Line 1688: prev_li = (struct rlc_li_field_egprs *)
no need for a line break here



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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Ifae1a7b2b3dfad8df19585063088ba0df2749c8f
Gerrit-Change-Number: 21387
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Comment-Date: Mon, 30 Nov 2020 15:53:03 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-bsc[master]: AMR start-mode auto: send smod bits as zero

2020-11-30 Thread neels
neels has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/21376 )

Change subject: AMR start-mode auto: send smod bits as zero
..

AMR start-mode auto: send smod bits as zero

When ICMI becomes zero for 'start-mode auto', the smod bits will remain
whatever start-mode was set in the previous osmo-bsc config. Instead, osmo-bsc
should clear the smod bits for 'start-mode auto' so that its MultiRate Config
does not vary depending on what was previously configured.

Change-Id: I1ec5bad0bce01cc425ee05ecf70c83ec662a226a
---
M src/osmo-bsc/bsc_vty.c
1 file changed, 3 insertions(+), 2 deletions(-)

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



diff --git a/src/osmo-bsc/bsc_vty.c b/src/osmo-bsc/bsc_vty.c
index 5022fe1..d8e9682 100644
--- a/src/osmo-bsc/bsc_vty.c
+++ b/src/osmo-bsc/bsc_vty.c
@@ -4199,9 +4199,10 @@
}
}

-   if (argv[0][0] == 'a' || num == 0)
+   if (argv[0][0] == 'a' || num == 0) {
mr_conf->icmi = 0;
-   else {
+   mr_conf->smod = 0;
+   } else {
mr_conf->icmi = 1;
if (num < atoi(argv[0]))
mr_conf->smod = num - 1;

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I1ec5bad0bce01cc425ee05ecf70c83ec662a226a
Gerrit-Change-Number: 21376
Gerrit-PatchSet: 3
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-bsc[master]: fix AMR start-mode auto: reflect proper ICMI in MultiRate Config IE

2020-11-30 Thread neels
neels has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/21372 )

Change subject: fix AMR start-mode auto: reflect proper ICMI in MultiRate 
Config IE
..

fix AMR start-mode auto: reflect proper ICMI in MultiRate Config IE

Send the proper ICMI value, instead of always sending ICMI=1 regardless
of the AMR start-mode setting.

To avoid test fallout from the fix, we should first merge
osmo-ttcn3-hacks I4cff01c37d5c7e301e9a01f773b7e009a789519b

Change-Id: I577ff590d7588fd7e3ee4846c7955ab8f84cf2b1
---
M src/osmo-bsc/lchan_fsm.c
1 file changed, 5 insertions(+), 1 deletion(-)

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



diff --git a/src/osmo-bsc/lchan_fsm.c b/src/osmo-bsc/lchan_fsm.c
index e97c1ba..0ca7f2c 100644
--- a/src/osmo-bsc/lchan_fsm.c
+++ b/src/osmo-bsc/lchan_fsm.c
@@ -467,6 +467,7 @@
mr = >mr_full;
else
mr = >mr_half;
+   mr_conf_bts = (struct gsm48_multi_rate_conf *)mr->gsm48_ie;

if (lchan->activate.info.activ_for == FOR_VTY)
/* If the channel is activated manually from VTY, then there is 
no
@@ -503,7 +504,6 @@
 * if the channel activation was triggerd by the VTY, do not
 * filter anything (see also comment above) */
if (lchan->activate.info.activ_for != FOR_VTY) {
-   mr_conf_bts = (struct gsm48_multi_rate_conf *)mr->gsm48_ie;
rc_rate = calc_amr_rate_intersection(_conf_filtered, 
mr_conf_bts, _conf_filtered);
if (rc_rate < 0) {
LOG_LCHAN(lchan, LOGL_ERROR,
@@ -512,6 +512,10 @@
}
}

+   /* Set the ICMI according to the BTS. Above 
gsm48_mr_cfg_from_gsm0808_sc_cfg() always sets ICMI = 1, which
+* carried through all of the above rate intersections. */
+   mr_conf_filtered.icmi = mr_conf_bts->icmi;
+
/* Proceed with the generation of the multirate configuration IE
 * (MS and BTS) */
rc = gsm48_multirate_config(lchan->mr_ms_lv, _conf_filtered, 
mr->ms_mode, mr->num_modes);

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I577ff590d7588fd7e3ee4846c7955ab8f84cf2b1
Gerrit-Change-Number: 21372
Gerrit-PatchSet: 3
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-bsc[master]: AMR start-mode: send proper smod bits

2020-11-30 Thread neels
neels has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/21375 )

Change subject: AMR start-mode: send proper smod bits
..

AMR start-mode: send proper smod bits

So far, the smod bits reflecting the start mode never made it to the
MultiRate Config IE: gsm48_mr_cfg_from_gsm0808_sc_cfg() always sets them
to zero, and the mode filtering fails to carry the BTS set bits.

Set smod bits according to BTS config after the filtering of available
modes.

Change-Id: I49691df01745a7c485bf165e897872c35fc4b147
---
M src/osmo-bsc/lchan_fsm.c
1 file changed, 1 insertion(+), 0 deletions(-)

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



diff --git a/src/osmo-bsc/lchan_fsm.c b/src/osmo-bsc/lchan_fsm.c
index 0ca7f2c..1903645 100644
--- a/src/osmo-bsc/lchan_fsm.c
+++ b/src/osmo-bsc/lchan_fsm.c
@@ -515,6 +515,7 @@
/* Set the ICMI according to the BTS. Above 
gsm48_mr_cfg_from_gsm0808_sc_cfg() always sets ICMI = 1, which
 * carried through all of the above rate intersections. */
mr_conf_filtered.icmi = mr_conf_bts->icmi;
+   mr_conf_filtered.smod = mr_conf_bts->smod;

/* Proceed with the generation of the multirate configuration IE
 * (MS and BTS) */

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I49691df01745a7c485bf165e897872c35fc4b147
Gerrit-Change-Number: 21375
Gerrit-PatchSet: 3
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-bsc[master]: AMR start-mode auto: send smod bits as zero

2020-11-30 Thread neels
neels has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/21376 )

Change subject: AMR start-mode auto: send smod bits as zero
..


Patch Set 2: Code-Review+2

combine


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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I1ec5bad0bce01cc425ee05ecf70c83ec662a226a
Gerrit-Change-Number: 21376
Gerrit-PatchSet: 2
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 15:49:41 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-bsc[master]: AMR start-mode: send proper smod bits

2020-11-30 Thread neels
neels has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/21375 )

Change subject: AMR start-mode: send proper smod bits
..


Patch Set 2: Code-Review+2

combine


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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I49691df01745a7c485bf165e897872c35fc4b147
Gerrit-Change-Number: 21375
Gerrit-PatchSet: 2
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 15:49:12 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-bsc[master]: fix AMR start-mode auto: reflect proper ICMI in MultiRate Config IE

2020-11-30 Thread neels
neels has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/21372 )

Change subject: fix AMR start-mode auto: reflect proper ICMI in MultiRate 
Config IE
..


Patch Set 2: Code-Review+2

combine


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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I577ff590d7588fd7e3ee4846c7955ab8f84cf2b1
Gerrit-Change-Number: 21372
Gerrit-PatchSet: 2
Gerrit-Owner: neels 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 15:48:39 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-sgsn[master]: gbproxy: Add NSE peer that can have multiple gbproxy_peers

2020-11-30 Thread daniel
Hello Jenkins Builder, pespin, lynxis lazus,

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

https://gerrit.osmocom.org/c/osmo-sgsn/+/21406

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

Change subject: gbproxy: Add NSE peer that can have multiple gbproxy_peers
..

gbproxy: Add NSE peer that can have multiple gbproxy_peers

We want this level of indirection to support multiple BVCs per NSE. The
current code assumes that an NSE only has one BVC which breaks messages
on the signalling BVC which should only be sent once to an NSE
regardless of the number of BVCs it contains.

Change-Id: I97cc6c8f8c0f1b91577ab8f679c4ae217cc88076
Related: SYS#5226
---
M include/osmocom/sgsn/gb_proxy.h
M src/gbproxy/gb_proxy.c
M src/gbproxy/gb_proxy_ctrl.c
M src/gbproxy/gb_proxy_patch.c
M src/gbproxy/gb_proxy_peer.c
M src/gbproxy/gb_proxy_tlli.c
M src/gbproxy/gb_proxy_vty.c
M tests/gbproxy/gbproxy_test.c
M tests/gbproxy/gbproxy_test.ok
9 files changed, 495 insertions(+), 285 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/06/21406/5
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/21406
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I97cc6c8f8c0f1b91577ab8f679c4ae217cc88076
Gerrit-Change-Number: 21406
Gerrit-PatchSet: 5
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-CC: laforge 
Gerrit-MessageType: newpatchset


Change in osmocom-bb[master]: trxcon/scheduler: clarify Measurement results padding

2020-11-30 Thread fixeria
fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/21414 )


Change subject: trxcon/scheduler: clarify Measurement results padding
..

trxcon/scheduler: clarify Measurement results padding

Table 10.5.2.20.0 "Measurement Results Contents" in 3GPP TS 44.018
is clear on what should be used as padding - '0**', i.e. zeroes.

Change-Id: I4db6845c98aded10291134f416da98fd0f4f58e3
---
M src/host/trxcon/sched_prim.c
1 file changed, 2 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/14/21414/1

diff --git a/src/host/trxcon/sched_prim.c b/src/host/trxcon/sched_prim.c
index fc0f05d..e5204ba 100644
--- a/src/host/trxcon/sched_prim.c
+++ b/src/host/trxcon/sched_prim.c
@@ -144,10 +144,10 @@
/* LAPDm header */
0x01, 0x03, 0x49,

-   /* Measurement report */
+   /* Measurement report (see 3GPP TS 44.018, section 10.5.2.20) */
0x06, 0x15, 0x36, 0x36, 0x01, 0xC0,

-   /* TODO: Padding? Randomize if so */
+   /* 0** -- Padding with zeroes */
0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
};

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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I4db6845c98aded10291134f416da98fd0f4f58e3
Gerrit-Change-Number: 21414
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-MessageType: newchange


Change in osmocom-bb[master]: trxcon/scheduler: clarify the content of Measurement results

2020-11-30 Thread fixeria
fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/21415 )


Change subject: trxcon/scheduler: clarify the content of Measurement results
..

trxcon/scheduler: clarify the content of Measurement results

Change-Id: I26546dcbc853166e351d00260936b1b9d584ae03
---
M src/host/trxcon/sched_prim.c
1 file changed, 15 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/15/21415/1

diff --git a/src/host/trxcon/sched_prim.c b/src/host/trxcon/sched_prim.c
index e5204ba..f4f5e40 100644
--- a/src/host/trxcon/sched_prim.c
+++ b/src/host/trxcon/sched_prim.c
@@ -144,8 +144,21 @@
/* LAPDm header */
0x01, 0x03, 0x49,

-   /* Measurement report (see 3GPP TS 44.018, section 10.5.2.20) */
-   0x06, 0x15, 0x36, 0x36, 0x01, 0xC0,
+   /* RR Management messages, Measurement Report */
+   0x06, 0x15,
+
+   /* Measurement results (see 3GPP TS 44.018, section 10.5.2.20):
+*   0...  = BA-USED: 0
+*   .0..  = DTX-USED: DTX was not used
+*   ..11 0110 = RXLEV-FULL-SERVING-CELL: -57 <= x < -56 dBm 
(54)
+*   0...  = 3G-BA-USED: 0
+*   .0..  = MEAS-VALID: The measurement results are valid
+*   ..11 0110 = RXLEV-SUB-SERVING-CELL: -57 <= x < -56 dBm (54)
+*   0...  = SI23_BA_USED: 0
+*   .000  = RXQUAL-FULL-SERVING-CELL: BER < 0.2%, Mean 
value 0.14% (0)
+*    000. = RXQUAL-SUB-SERVING-CELL: BER < 0.2%, Mean 
value 0.14% (0)
+*    ...1  11..  = NO-NCELL-M: Neighbour cell 
information not available */
+   0x36, 0x36, 0x01, 0xc0,

/* 0** -- Padding with zeroes */
0x00, 0x00, 0x00, 0x00, 0x00, 0x00,

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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I26546dcbc853166e351d00260936b1b9d584ae03
Gerrit-Change-Number: 21415
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-MessageType: newchange


Change in osmocom-bb[master]: trxcon/scheduler: invalidate hard-coded Measurement results

2020-11-30 Thread fixeria
fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmocom-bb/+/21416 )


Change subject: trxcon/scheduler: invalidate hard-coded Measurement results
..

trxcon/scheduler: invalidate hard-coded Measurement results

This is what trxcon sends to the network before the first SACCH
block is received from the higher layers.  The indicated values
are of course invalid because they're hard-coded.

According to 3GPP TS 44.018, table 10.5.2.20.1:

  0  The measurement results are valid
  1  The measurement results are not valid

Change-Id: I7da767e146aec7cef1de71e4d735d6a02b6c5642
Related: SYS#4918
---
M src/host/trxcon/sched_prim.c
1 file changed, 2 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/16/21416/1

diff --git a/src/host/trxcon/sched_prim.c b/src/host/trxcon/sched_prim.c
index f4f5e40..9473320 100644
--- a/src/host/trxcon/sched_prim.c
+++ b/src/host/trxcon/sched_prim.c
@@ -152,13 +152,13 @@
 *   .0..  = DTX-USED: DTX was not used
 *   ..11 0110 = RXLEV-FULL-SERVING-CELL: -57 <= x < -56 dBm 
(54)
 *   0...  = 3G-BA-USED: 0
-*   .0..  = MEAS-VALID: The measurement results are valid
+*   .1..  = MEAS-VALID: The measurement results are not 
valid
 *   ..11 0110 = RXLEV-SUB-SERVING-CELL: -57 <= x < -56 dBm (54)
 *   0...  = SI23_BA_USED: 0
 *   .000  = RXQUAL-FULL-SERVING-CELL: BER < 0.2%, Mean 
value 0.14% (0)
 *    000. = RXQUAL-SUB-SERVING-CELL: BER < 0.2%, Mean 
value 0.14% (0)
 *    ...1  11..  = NO-NCELL-M: Neighbour cell 
information not available */
-   0x36, 0x36, 0x01, 0xc0,
+   0x36, 0x76, 0x01, 0xc0,

/* 0** -- Padding with zeroes */
0x00, 0x00, 0x00, 0x00, 0x00, 0x00,

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

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I7da767e146aec7cef1de71e4d735d6a02b6c5642
Gerrit-Change-Number: 21416
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria 
Gerrit-MessageType: newchange


Change in osmo-sgsn[master]: gbproxy: Remove test testing NSVCI change

2020-11-30 Thread daniel
daniel has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/21413 )


Change subject: gbproxy: Remove test testing NSVCI change
..

gbproxy: Remove test testing NSVCI change

Testing this here doesn't make much sense since ns2.

Change-Id: Ic1d7e646e633c9fa62812f5005ed10c0108a06f2
Related: SYS#5226
---
M tests/gbproxy/gbproxy_test.c
M tests/gbproxy/gbproxy_test.ok
2 files changed, 2 insertions(+), 255 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/13/21413/1

diff --git a/tests/gbproxy/gbproxy_test.c b/tests/gbproxy/gbproxy_test.c
index 76fc000..78fa8cb 100644
--- a/tests/gbproxy/gbproxy_test.c
+++ b/tests/gbproxy/gbproxy_test.c
@@ -1309,7 +1309,7 @@
 {
struct gprs_ns2_inst *nsi = gprs_ns2_instantiate(tall_sgsn_ctx, 
gprs_ns2_callback, );
uint16_t bss_nsei[2] = {0x1000, 0x2000};
-   uint16_t bvci[4] = {0x1002, 0x2002, 0x3002, 0x4002};
+   uint16_t bvci[4] = {0x1002, 0x2002, 0x3002};

gbcfg.nsi = nsi;
gbcfg.nsip_sgsn_nsei = SGSN_NSEI;
@@ -1381,45 +1381,6 @@
send_ns_unitdata(nsi, NULL, bss_nsei[0], bvci[2], (uint8_t *)"", 0);
send_ns_unitdata(nsi, NULL, SGSN_NSEI, bvci[2], (uint8_t *)"", 0);

-   printf("--- Change NSVCI ---\n\n");
-
-   setup_ns(nsi, bss_nsei[1]);
-
-   printf("--- Setup BVCI 1 ---\n\n");
-
-   setup_bssgp(nsi, bss_nsei[1], bvci[0]);
-   send_bssgp_reset_ack(nsi, SGSN_NSEI, bvci[0]);
-   dump_peers(stdout, 0, 0, );
-
-   printf("--- Setup BVCI 4 ---\n\n");
-
-   setup_bssgp(nsi, bss_nsei[1], bvci[3]);
-   send_bssgp_reset_ack(nsi, SGSN_NSEI, bvci[3]);
-   dump_peers(stdout, 0, 0, );
-
-   printf("--- Send message from BSS 1 to SGSN and back, BVCI 1 ---\n\n");
-
-   send_ns_unitdata(nsi, NULL, bss_nsei[1], bvci[0], (uint8_t *)"", 0);
-   send_ns_unitdata(nsi, NULL, SGSN_NSEI, bvci[0], (uint8_t *)"", 0);
-
-   printf("--- Send message from BSS 1 to SGSN and back, BVCI 2 "
-  " (should fail) ---\n\n");
-
-   send_ns_unitdata(nsi, NULL, bss_nsei[1], bvci[1], (uint8_t *)"", 0);
-   dump_peers(stdout, 0, 0, );
-   send_ns_unitdata(nsi, NULL, SGSN_NSEI, bvci[1], (uint8_t *)"", 0);
-   dump_peers(stdout, 0, 0, );
-
-   printf("--- Send message from BSS 1 to SGSN and back, BVCI 3 ---\n\n");
-
-   send_ns_unitdata(nsi, NULL, bss_nsei[1], bvci[2], (uint8_t *)"", 0);
-   send_ns_unitdata(nsi, NULL, SGSN_NSEI, bvci[2], (uint8_t *)"", 0);
-
-   printf("--- Send message from BSS 1 to SGSN and back, BVCI 4 ---\n\n");
-
-   send_ns_unitdata(nsi, NULL, bss_nsei[1], bvci[3], (uint8_t *)"", 0);
-   send_ns_unitdata(nsi, NULL, SGSN_NSEI, bvci[3], (uint8_t *)"", 0);
-
dump_global(stdout, 0);
dump_peers(stdout, 0, 0, );

diff --git a/tests/gbproxy/gbproxy_test.ok b/tests/gbproxy/gbproxy_test.ok
index 6f20f12..74cedb3 100644
--- a/tests/gbproxy/gbproxy_test.ok
+++ b/tests/gbproxy/gbproxy_test.ok
@@ -654,227 +654,13 @@
 Message for BSS (NSEI=8192 BVCI=12290):
 [L2]> [L3]>

 Change NSVCI ---
-
-Setup NS-VC: NSEI 0x2000(8192)
-
-NS2 CALLBACK, prim 2, bvci 0x
-
 Setup BVCI 1 ---
-
-Setup BSSGP: BVCI 0x1002(4098)
-
-PROCESSING BVC_RESET from NSEI 8192
-22 04 82 10 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-NS2 CALLBACK, prim 0, msg length 18, bvci 0x
-22 04 82 10 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-NS2 UD REQUEST, prim 0, msg length 18, bvci 0x
-22 04 82 10 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-Message for SGSN (NSEI=256 BVCI=0):
-[L2]> [L3]> 22 04 82 10 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-PROCESSING BVC_RESET_ACK from NSEI 256
-23 04 82 10 02
-
-NS2 CALLBACK, prim 0, msg length 5, bvci 0x
-23 04 82 10 02
-
-NS2 UD REQUEST, prim 0, msg length 5, bvci 0x
-23 04 82 10 02
-
-Message for BSS (NSEI=8192 BVCI=0):
-[L2]> [L3]> 23 04 82 10 02
-
-Peers:
-  NSEI 8192, BVCI 12290, not blocked, RAI 112-332-16464-96
-NSEI mismatch   : 1
-TLLI-Cache: 0
-  NSEI 4096, BVCI 8194, not blocked, RAI 112-332-16464-96
-NSEI mismatch   : 1
-TLLI-Cache: 0
-  NSEI 8192, BVCI 4098, not blocked, RAI 112-332-16464-96
-NSEI mismatch   : 1
-TLLI-Cache: 0
 Setup BVCI 4 ---
-
-Setup BSSGP: BVCI 0x4002(16386)
-
-PROCESSING BVC_RESET from NSEI 8192
-22 04 82 40 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-NS2 CALLBACK, prim 0, msg length 18, bvci 0x
-22 04 82 40 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-NS2 UD REQUEST, prim 0, msg length 18, bvci 0x
-22 04 82 40 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-Message for SGSN (NSEI=256 BVCI=0):
-[L2]> [L3]> 22 04 82 40 02 07 81 08 08 88 11 22 33 40 50 60 10 00
-
-PROCESSING BVC_RESET_ACK from NSEI 256
-23 04 82 40 02
-
-NS2 CALLBACK, prim 0, msg length 5, bvci 0x
-23 04 82 40 02
-
-NS2 UD REQUEST, prim 0, msg length 5, bvci 0x
-23 04 82 40 02
-
-Message 

Change in osmo-sgsn[master]: gbproxy: Ensure BVC0 is reset before handling PtP BVCs

2020-11-30 Thread daniel
Hello Jenkins Builder,

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

https://gerrit.osmocom.org/c/osmo-sgsn/+/21410

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

Change subject: gbproxy: Ensure BVC0 is reset before handling PtP BVCs
..

gbproxy: Ensure BVC0 is reset before handling PtP BVCs

Change-Id: I9c67f973f7bca00e8eb22a024ef2282c007dd84b
Related: SYS#5226
---
M src/gbproxy/gb_proxy.c
M tests/gbproxy/gbproxy_test.c
M tests/gbproxy/gbproxy_test.ok
3 files changed, 125 insertions(+), 6 deletions(-)


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I9c67f973f7bca00e8eb22a024ef2282c007dd84b
Gerrit-Change-Number: 21410
Gerrit-PatchSet: 2
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset


Change in osmo-sgsn[master]: gbproxy: Add NSE peer that can have multiple gbproxy_peers

2020-11-30 Thread daniel
Hello Jenkins Builder, pespin, lynxis lazus,

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

https://gerrit.osmocom.org/c/osmo-sgsn/+/21406

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

Change subject: gbproxy: Add NSE peer that can have multiple gbproxy_peers
..

gbproxy: Add NSE peer that can have multiple gbproxy_peers

We want this level of indirection to support multiple BVCs per NSE. The
current code assumes that an NSE only has one BVC which breaks messages
on the signalling BVC which should only be sent once to an NSE
regardless of the number of BVCs it contains.

Change-Id: I97cc6c8f8c0f1b91577ab8f679c4ae217cc88076
Related: SYS#5226
---
M include/osmocom/sgsn/gb_proxy.h
M src/gbproxy/gb_proxy.c
M src/gbproxy/gb_proxy_ctrl.c
M src/gbproxy/gb_proxy_patch.c
M src/gbproxy/gb_proxy_peer.c
M src/gbproxy/gb_proxy_tlli.c
M src/gbproxy/gb_proxy_vty.c
M tests/gbproxy/gbproxy_test.c
M tests/gbproxy/gbproxy_test.ok
9 files changed, 499 insertions(+), 290 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/06/21406/4
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/21406
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I97cc6c8f8c0f1b91577ab8f679c4ae217cc88076
Gerrit-Change-Number: 21406
Gerrit-PatchSet: 4
Gerrit-Owner: daniel 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-CC: laforge 
Gerrit-MessageType: newpatchset


Change in osmo-pcu[master]: Dl TBF: Get rid of LLC UI dummy blocks following other data

2020-11-30 Thread pespin
Hello Jenkins Builder,

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

https://gerrit.osmocom.org/c/osmo-pcu/+/21387

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

Change subject: Dl TBF: Get rid of LLC UI dummy blocks following other data
..

Dl TBF: Get rid of LLC UI dummy blocks following other data

According to:
* 3GPP TS 44.060 version 16.0.0 "9.3.1a Delayed release of downlink Temporary 
Block Flow"
* 3GPP TS 44.064 version 16.0.0 "6.4.2.2 Unconfirmed Information (UI) Dummy 
command"

LLC UI Dummy frames are to be used when there no more data to send, only
in order to delay the release of a TBF. Hence, while not incorrect per
se, makes no sense to send those LLC UI Dummy frames inserted into
rlcmac blocks which already contain other LLC frames, since the MS in
that case is already being kept active.
It only makes sense to send those LLC UI Dummy frames when we have
nothing else to send, that is, alone inside a RLCMAC block without other
LLC frames.

Related: OS#4849
Change-Id: Ifae1a7b2b3dfad8df19585063088ba0df2749c8f
---
M src/encoding.cpp
M src/encoding.h
M src/tbf_dl.cpp
M tests/tbf/TbfTest.err
4 files changed, 171 insertions(+), 198 deletions(-)


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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Ifae1a7b2b3dfad8df19585063088ba0df2749c8f
Gerrit-Change-Number: 21387
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset


Change in docker-playground[master]: Add ttcn3-gbproxy-test-fr for FR related testing

2020-11-30 Thread laforge
laforge has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/docker-playground/+/21412 )


Change subject: Add ttcn3-gbproxy-test-fr for FR related testing
..

Add ttcn3-gbproxy-test-fr for FR related testing

Testing FR support in osmo-gbproxy is a bit more complicated
as it involves the "hdlc" net-devices privded by the hdlc_fr.ko
kernel module.

So we need to
* run on a host with actual hdlc net-devices (e.g. dahdi_dyamic_loc)
* move those net-devices into the containers after starting them
* wait for the net-devices to appear in the containers before starting
  either gbproxy or the test suite

Change-Id: Id4b52877db53cb6e59f6d0d3f754aaae633949e8
---
A ttcn3-gbproxy-test-fr/GBProxy_Tests.cfg
A ttcn3-gbproxy-test-fr/jenkins.sh
A ttcn3-gbproxy-test-fr/netdev-to-docker.sh
A ttcn3-gbproxy-test-fr/osmo-gbproxy.cfg
M ttcn3-gbproxy-test/Dockerfile
A ttcn3-gbproxy-test/docker-entrypoint.sh
A ttcn3-gbproxy-test/pipework
7 files changed, 776 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/docker-playground 
refs/changes/12/21412/1

diff --git a/ttcn3-gbproxy-test-fr/GBProxy_Tests.cfg 
b/ttcn3-gbproxy-test-fr/GBProxy_Tests.cfg
new file mode 100644
index 000..adab61a
--- /dev/null
+++ b/ttcn3-gbproxy-test-fr/GBProxy_Tests.cfg
@@ -0,0 +1,100 @@
+[ORDERED_INCLUDE]
+# Common configuration, shared between test suites
+"/osmo-ttcn3-hacks/Common.cfg"
+# testsuite specific configuration, not expected to change
+"/osmo-ttcn3-hacks/gbproxy/GBProxy_Tests.default"
+
+# Local configuration below
+
+[LOGGING]
+
+[TESTPORT_PARAMETERS]
+*.GBPVTY.CTRL_HOSTNAME := "172.18.25.10"
+
+[MODULE_PARAMETERS]
+GBProxy_Tests.mp_nsconfig_sgsn := {
+   {
+   nsvc := {
+   {
+   provider := {
+   ip := {
+   address_family := AF_INET,
+   local_ip := "172.18.25.103",
+   local_udp_port := 23000,
+   remote_ip := "172.18.25.10"
+   }
+   }
+   }
+   }
+   }
+}
+GBProxy_Tests.mp_nsconfig_pcu := {
+   {
+   nsei := 1,
+   role_sgsn := false,
+   handle_sns := false,
+   nsvc := {
+   {
+   provider := {
+   fr := {
+   netdev := "hdlc1",
+   dlci := 16
+   }
+   },
+   nsvci := 1
+   }, {
+   provider := {
+   fr := {
+   netdev := "hdlc2",
+   dlci := 17
+   }
+   },
+   nsvci := 2
+   }, {
+   provider := {
+   fr := {
+   netdev := "hdlc3",
+   dlci := 18
+   }
+   },
+   nsvci := 3
+   }, {
+   provider := {
+   fr := {
+   netdev := "hdlc4",
+   dlci := 19
+   }
+   },
+   nsvci := 4
+   }
+   }
+   }
+}
+
+GBProxy_Tests.mp_gbconfigs := {
+   {
+   nsei := 1,
+   sgsn_role := false,
+   bvc := {
+   {
+   bvci := 196,
+   cell_id := {
+   ra_id := {
+   lai := {
+   mcc_mnc := '262F42'H,
+   lac := 13135
+   },
+   rac := 0
+   },
+   cell_id := 20960
+   },
+   depth := BSSGP_DECODE_DEPTH_BSSGP
+   }
+   }
+   }
+};
+
+[MAIN_CONTROLLER]
+
+[EXECUTE]
+GBProxy_Tests.control
diff --git 

Change in docker-playground[master]: osmo-gbproxy-master (derived from osmo-sgsn-master)

2020-11-30 Thread laforge
laforge has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/docker-playground/+/21411 )


Change subject: osmo-gbproxy-master (derived from osmo-sgsn-master)
..

osmo-gbproxy-master (derived from osmo-sgsn-master)

Using a separate container will allow us to do gbproxy specific things
like waiting for physical net-devices to show up in the docker netns
before actually starting the osmo-gbproxy executable

Change-Id: I5e6351055ac9430f13e43291cf2a9d52683fe6bf
---
A osmo-gbproxy-master/Dockerfile
A osmo-gbproxy-master/Makefile
A osmo-gbproxy-master/docker-entrypoint.sh
A osmo-gbproxy-master/osmo-gbproxy.cfg
A osmo-gbproxy-master/pipework
5 files changed, 542 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/docker-playground 
refs/changes/11/21411/1

diff --git a/osmo-gbproxy-master/Dockerfile b/osmo-gbproxy-master/Dockerfile
new file mode 100644
index 000..8661ec9
--- /dev/null
+++ b/osmo-gbproxy-master/Dockerfile
@@ -0,0 +1,15 @@
+ARGUSER
+ARGDISTRO
+FROM   $USER/osmo-sgsn-master
+
+COPY   osmo-gbproxy.cfg/data/osmo-gbproxy.cfg
+
+# work-around for stupid docker not being able to properly deal with host 
netdevices or start
+# containers in pre-existing netns
+COPY   pipework/usr/bin/pipework
+COPY   docker-entrypoint.sh/docker-entrypoint.sh
+
+WORKDIR/data
+CMD["/docker-entrypoint.sh"]
+
+EXPOSE 23000/udp 4246/tcp 4263/tcp
diff --git a/osmo-gbproxy-master/Makefile b/osmo-gbproxy-master/Makefile
new file mode 100644
index 000..1614f9a
--- /dev/null
+++ b/osmo-gbproxy-master/Makefile
@@ -0,0 +1,3 @@
+RUN_ARGS?=--rm --network sigtran --ip 172.18.0.21 -v gbproxy-vol:/data
+
+include ../make/Makefile
diff --git a/osmo-gbproxy-master/docker-entrypoint.sh 
b/osmo-gbproxy-master/docker-entrypoint.sh
new file mode 100755
index 000..72c4e8c
--- /dev/null
+++ b/osmo-gbproxy-master/docker-entrypoint.sh
@@ -0,0 +1,9 @@
+#!/bin/bash
+set -e
+set -x
+
+if [[ -n ${WAIT_FOR_NETDEV:-} ]]; then
+   /usr/bin/pipework --wait -i ${WAIT_FOR_NETDEV}
+fi
+
+/usr/local/bin/osmo-gbproxy -c /data/osmo-gbproxy.cfg >/data/osmo-gbproxy.log 
2>&1
diff --git a/osmo-gbproxy-master/osmo-gbproxy.cfg 
b/osmo-gbproxy-master/osmo-gbproxy.cfg
new file mode 100644
index 000..29f698f
--- /dev/null
+++ b/osmo-gbproxy-master/osmo-gbproxy.cfg
@@ -0,0 +1,26 @@
+!
+! Osmocom Gb Proxy (0.9.0.404-6463) configuration saved from vty
+!!
+!
+line vty
+ no login
+!
+gbproxy
+ sgsn nsei 101
+ns
+ nse 101 nsvci 101
+ nse 101 remote-role sgsn
+ nse 101 encapsulation udp
+ nse 101 remote-ip 192.168.100.239
+ nse 101 remote-port 
+ timer tns-block 3
+ timer tns-block-retries 3
+ timer tns-reset 3
+ timer tns-reset-retries 3
+ timer tns-test 30
+ timer tns-alive 3
+ timer tns-alive-retries 10
+ encapsulation framerelay-gre enabled 0
+ encapsulation framerelay-gre local-ip 0.0.0.0
+ encapsulation udp local-ip 127.0.0.100
+ encapsulation udp local-port 23000
diff --git a/osmo-gbproxy-master/pipework b/osmo-gbproxy-master/pipework
new file mode 100755
index 000..97ce66b
--- /dev/null
+++ b/osmo-gbproxy-master/pipework
@@ -0,0 +1,489 @@
+#!/bin/sh
+# This code should (try to) follow Google's Shell Style Guide
+# (https://google.github.io/styleguide/shell.xml)
+set -e
+
+case "$1" in
+  --wait)
+WAIT=1
+;;
+  --direct-phys)
+DIRECT_PHYS=1
+shift
+;;
+esac
+
+IFNAME=$1
+
+# default value set further down if not set here
+CONTAINER_IFNAME=
+if [ "$2" = "-i" ]; then
+  CONTAINER_IFNAME=$3
+  shift 2
+fi
+
+if [ "$2" = "-l" ]; then
+  LOCAL_IFNAME=$3
+  shift 2
+fi
+
+#inet or inet6
+FAMILY_FLAG="-4"
+if [ "$2" = "-a" ]; then
+  FAMILY_FLAG="-$3"
+  shift 2
+fi
+
+GUESTNAME=$2
+IPADDR=$3
+MACADDR=$4
+
+case "$MACADDR" in
+  *@*)
+VLAN="${MACADDR#*@}"
+VLAN="${VLAN%%@*}"
+MACADDR="${MACADDR%%@*}"
+;;
+  *)
+VLAN=
+;;
+esac
+
+# did they ask to generate a custom MACADDR?
+# generate the unique string
+case "$MACADDR" in
+  U:*)
+macunique="${MACADDR#*:}"
+# now generate a 48-bit hash string from $macunique
+MACADDR=$(echo $macunique|md5sum|sed 
's/^\(..\)\(..\)\(..\)\(..\)\(..\).*$/02:\1:\2:\3:\4:\5/')
+   ;;
+esac
+
+
+[ "$IPADDR" ] || [ "$WAIT" ] || {
+  echo "Syntax:"
+  echo "pipework  [-i containerinterface] [-l 
localinterfacename] [-a addressfamily]  
/[@default_gateway] [macaddr][@vlan]"
+  echo "pipework  [-i containerinterface] [-l 
localinterfacename]  dhcp [macaddr][@vlan]"
+  echo "pipework mac: [-i containerinterface] [-l 
localinterfacename] [-a addressfamily]  
/[@default_gateway] [macaddr][@vlan]"
+  echo "pipework mac: [-i containerinterface] [-l 
localinterfacename]  dhcp [macaddr][@vlan]"
+  echo "pipework route  "
+  echo "pipework rule  "
+  echo "pipework tc  "
+  echo "pipework --wait [-i containerinterface]"
+  exit 1
+}
+
+# Succeed if the given utility is installed. Fail otherwise.
+# For explanations about 

Change in osmo-bts[master]: l1sap: add repeated uplink SACCH

2020-11-30 Thread dexter
dexter has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bts/+/21185 )

Change subject: l1sap: add repeated uplink SACCH
..


Patch Set 5:

(1 comment)

(ensure all comments are sent)

https://gerrit.osmocom.org/c/osmo-bts/+/21185/4/src/osmo-bts-trx/sched_lchan_xcch.c
File src/osmo-bts-trx/sched_lchan_xcch.c:

https://gerrit.osmocom.org/c/osmo-bts/+/21185/4/src/osmo-bts-trx/sched_lchan_xcch.c@37
PS4, Line 37: /* Add two arrays of sbits */
: static void add_sbits(sbit_t *current, const sbit_t *previous)
> I believe addition is fine, since it could be perceived as a diversity scheme 
> and the individual sof […]
See also Patchset 5 for the non saturated approach.



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

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I7e4cc33cc010866e41e3b594351a7f7bf93e08ac
Gerrit-Change-Number: 21185
Gerrit-PatchSet: 5
Gerrit-Owner: dexter 
Gerrit-Reviewer: Hoernchen 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: tnt 
Gerrit-CC: laforge 
Gerrit-Comment-Date: Mon, 30 Nov 2020 14:47:22 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Hoernchen 
Comment-In-Reply-To: laforge 
Comment-In-Reply-To: dexter 
Gerrit-MessageType: comment


Change in osmo-bsc[master]: ipaccess-proxy: generate coredump and exit upon SIGABRT received

2020-11-30 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/21346 )

Change subject: ipaccess-proxy: generate coredump and exit upon SIGABRT received
..

ipaccess-proxy: generate coredump and exit upon SIGABRT received

Previous code relied on abort() switching sigaction to SIG_FDL +
retriggering SIGABRT in case the signal handler returns, which would
then generate the coredump + terminate the process.
However, if a SIGABRT is received from somewhere else (kill -SIGABRT),
then the process would print the talloc report and continue running,
which is not desired.

Change-Id: Iff920ff3dbeb48bd871b7578470f27fe9d0f9516
Fixes: OS#4865
---
M src/ipaccess/ipaccess-proxy.c
1 file changed, 13 insertions(+), 5 deletions(-)

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



diff --git a/src/ipaccess/ipaccess-proxy.c b/src/ipaccess/ipaccess-proxy.c
index 47465b4..d676038 100644
--- a/src/ipaccess/ipaccess-proxy.c
+++ b/src/ipaccess/ipaccess-proxy.c
@@ -1086,14 +1086,22 @@
return ret;
 }

-static void signal_handler(int signal)
+static void signal_handler(int signum)
 {
-   fprintf(stdout, "signal %u received\n", signal);
+   fprintf(stdout, "signal %u received\n", signum);

-   switch (signal) {
+   switch (signum) {
case SIGABRT:
-   /* in case of abort, we want to obtain a talloc report
-* and then return to the caller, who will abort the process */
+   /* in case of abort, we want to obtain a talloc report and
+* then run default SIGABRT handler, who will generate coredump
+* and abort the process. abort() should do this for us after we
+* return, but program wouldn't exit if an external SIGABRT is
+* received.
+*/
+   talloc_report_full(tall_bsc_ctx, stderr);
+   signal(SIGABRT, SIG_DFL);
+   raise(SIGABRT);
+   break;
case SIGUSR1:
talloc_report_full(tall_bsc_ctx, stderr);
break;

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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Iff920ff3dbeb48bd871b7578470f27fe9d0f9516
Gerrit-Change-Number: 21346
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in osmo-bsc[master]: ipaccess-proxy: generate coredump and exit upon SIGABRT received

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/21346 )

Change subject: ipaccess-proxy: generate coredump and exit upon SIGABRT received
..


Patch Set 2: Code-Review+2

Merging myself, I already merged tons of similar patches in different projects.


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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Iff920ff3dbeb48bd871b7578470f27fe9d0f9516
Gerrit-Change-Number: 21346
Gerrit-PatchSet: 2
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 14:32:30 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: gbproxy: Fix log messages in timeout situations

2020-11-30 Thread laforge
laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21402 )

Change subject: gbproxy: Fix log messages in timeout situations
..

gbproxy: Fix log messages in timeout situations

Printing an unbound 'rx' variable when nothing was received due to
timeout is somehow not really useful.  Print what we expected to
receive.

Change-Id: Idab8bed06281468164006682aa6b4c2c3e236880
---
M gbproxy/GBProxy_Tests.ttcn
1 file changed, 2 insertions(+), 2 deletions(-)

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



diff --git a/gbproxy/GBProxy_Tests.ttcn b/gbproxy/GBProxy_Tests.ttcn
index b708fa0..2120fa7 100644
--- a/gbproxy/GBProxy_Tests.ttcn
+++ b/gbproxy/GBProxy_Tests.ttcn
@@ -571,7 +571,7 @@
mtc.stop;
}
[] T.timeout {
-   setverdict(fail, "Timeout waiting for BSSGP on SGSN side: ", 
rx);
+   setverdict(fail, "Timeout waiting for BSSGP on SGSN side: ", 
exp_rx);
mtc.stop;
}
}
@@ -606,7 +606,7 @@
mtc.stop;
}
[] T.timeout {
-   setverdict(fail, "Timeout waiting for BSSGP on PCU side: ", rx);
+   setverdict(fail, "Timeout waiting for BSSGP on PCU side: ", 
exp_rx);
mtc.stop;
}
}

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


Change in osmo-pcu[master]: Implement downgrade to DL MCS1-4 when USF for GPRS_only MS

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-pcu/+/21179 )

Change subject: Implement downgrade to DL MCS1-4 when USF for GPRS_only MS
..


Patch Set 5:

10 days passed since I submitted last version. Can we merge?


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

Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: I9af23e175435fe9ae7b0e4119ad52fcd4707b9ca
Gerrit-Change-Number: 21179
Gerrit-PatchSet: 5
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: neels 
Gerrit-Comment-Date: Mon, 30 Nov 2020 14:31:05 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in osmo-ttcn3-hacks[master]: pcu: Introduce test TC_mcs_max_dl

2020-11-30 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21128 )

Change subject: pcu: Introduce test TC_mcs_max_dl
..


Patch Set 5:

13 days passed since I pushed latest version. Can we merge?


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/21128
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: If6c58cc18b537c4a1354f1b8263db8321347fbd7
Gerrit-Change-Number: 21128
Gerrit-PatchSet: 5
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Comment-Date: Mon, 30 Nov 2020 14:30:26 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: gnuradio_zmq_broker_remote: fix eNB port mapping for carrier aggregation

2020-11-30 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21409 )

Change subject: gnuradio_zmq_broker_remote: fix eNB port mapping for carrier 
aggregation
..

gnuradio_zmq_broker_remote: fix eNB port mapping for carrier aggregation

the mapping of ports didn't consider when a eNB had two ports for
different EARFCNs

Change-Id: I5548f67a3b37b03482e4b6172828fcd891e704a5
---
M src/osmo_gsm_tester/external/gnuradio_zmq_broker_remote.py
1 file changed, 4 insertions(+), 4 deletions(-)

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



diff --git a/src/osmo_gsm_tester/external/gnuradio_zmq_broker_remote.py 
b/src/osmo_gsm_tester/external/gnuradio_zmq_broker_remote.py
index fa69efd..98942f3 100755
--- a/src/osmo_gsm_tester/external/gnuradio_zmq_broker_remote.py
+++ b/src/osmo_gsm_tester/external/gnuradio_zmq_broker_remote.py
@@ -22,7 +22,7 @@
 class GrBroker(gr.top_block):

 def __init__(self, args, cfg):
-gr.top_block.__init__(self, "Intra Handover Flowgraph")
+gr.top_block.__init__(self, "InterENB Handover Flowgraph")

 ##
 # Variables
@@ -83,10 +83,10 @@
 idx = 0
 for enb in self.cfg['enb']:
 for it in enb:
-print('Connecting ENB port %u to Adder[%u] for earfcn %u' % 
(it['bind_port'], blocks_add_next_avail_port[earfcn], it['earfcn']))
-self.connect((it['gr_block_multiply'], 0), 
(self.blocks_add[it['earfcn']], blocks_add_next_avail_port[earfcn]))
+print('Connecting ENB port %u to Adder[%u] for earfcn %u' % 
(it['bind_port'], blocks_add_next_avail_port[it['earfcn']], it['earfcn']))
+self.connect((it['gr_block_multiply'], 0), 
(self.blocks_add[it['earfcn']], blocks_add_next_avail_port[it['earfcn']]))
 # TODO: if it['use_mimo'], connect it['gr_block_multiply2'] to 
some adder...
-blocks_add_next_avail_port[earfcn] += 1
+blocks_add_next_avail_port[it['earfcn']] += 1

 # Connect the Adder to the UE-side (Dl):
 for earfcn, bl_add in self.blocks_add.items():

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

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


Change in osmo-gsm-tester[master]: jenkins-build-amarisoft.sh: set symlink to srslte_rf lib

2020-11-30 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21408 )

Change subject: jenkins-build-amarisoft.sh: set symlink to srslte_rf lib
..

jenkins-build-amarisoft.sh: set symlink to srslte_rf lib

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

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



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

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

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


Change in osmo-gsm-tester[master]: gnuradio_zmq_broker_remote: fix eNB port mapping for carrier aggregation

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

Change subject: gnuradio_zmq_broker_remote: fix eNB port mapping for carrier 
aggregation
..


Patch Set 1: Verified+1


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

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5548f67a3b37b03482e4b6172828fcd891e704a5
Gerrit-Change-Number: 21409
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Mon, 30 Nov 2020 14:12:25 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in osmo-gsm-tester[master]: ms_amarisoft: fix various regressions

2020-11-30 Thread pespin
pespin has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21258 )

Change subject: ms_amarisoft: fix various regressions
..

ms_amarisoft: fix various regressions

this patch fixes some regressions in the Amarisoft UE class,
the config template, etc. that have been undetected bc we never
executed tests with it.

Change-Id: I397e675a4018acf3372a3b7e29fd864703b2b919
---
M src/osmo_gsm_tester/obj/ms.py
M src/osmo_gsm_tester/obj/ms_amarisoft.py
M src/osmo_gsm_tester/templates/amarisoft_lteue.cfg.tmpl
M sysmocom/defaults.conf
A sysmocom/scenarios/cfg-ms-count@.conf
5 files changed, 28 insertions(+), 10 deletions(-)

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



diff --git a/src/osmo_gsm_tester/obj/ms.py b/src/osmo_gsm_tester/obj/ms.py
index 70ce558..7257769 100644
--- a/src/osmo_gsm_tester/obj/ms.py
+++ b/src/osmo_gsm_tester/obj/ms.py
@@ -36,6 +36,11 @@
 }
 schema.register_resource_schema('modem', resource_schema)

+config_schema = {
+'count': schema.UINT
+}
+schema.register_config_schema('modem', config_schema)
+
 class MS(log.Origin, metaclass=ABCMeta):
 """Base for everything about mobile/modem and SIMs."""

@@ -125,3 +130,6 @@

 def get_counter(self, counter_name):
 raise log.Error('get_counter() not implemented!')
+
+def features(self):
+return self._conf.get('features', [])
diff --git a/src/osmo_gsm_tester/obj/ms_amarisoft.py 
b/src/osmo_gsm_tester/obj/ms_amarisoft.py
index fa16d63..9b6d64a 100644
--- a/src/osmo_gsm_tester/obj/ms_amarisoft.py
+++ b/src/osmo_gsm_tester/obj/ms_amarisoft.py
@@ -98,6 +98,7 @@
 self.remote_config_rf_file =  None
 self.remote_log_file = None
 self.remote_ifup_file = None
+self.num_carriers = 1
 if not rf_type_valid(conf.get('rf_dev_type', None)):
 raise log.Error('Invalid rf_dev_type=%s' % conf.get('rf_dev_type', 
None))
 if conf.get('rf_dev_type') == 'zmq':
@@ -247,13 +248,17 @@
 # We need to set some specific variables programatically here to match 
IP addresses:
 if self._conf.get('rf_dev_type') == 'zmq':
 base_srate = num_prb2base_srate(self.enb.num_prb())
-enb_base_port = self.enb.zmq_base_bind_port()
-rf_dev_args = 'tx_port0=tcp://' + self.addr() + ':' + 
str(self._zmq_base_bind_port + 0) \
-+ ',tx_port1=tcp://' + self.addr() + ':' + 
str(self._zmq_base_bind_port + 1) \
-+ ',rx_port0=tcp://' + self.enb.addr() + ':' + 
str(enb_base_port + 0) \
-+ ',rx_port1=tcp://' + self.enb.addr() + ':' + 
str(enb_base_port + 1) \
-+ 
',tx_freq=2510e6,rx_freq=2630e6,tx_freq2=2530e6,rx_freq2=2650e6' \
-+ ',id=ue,base_srate='+ str(base_srate)
+rf_dev_args = self.enb.get_zmq_rf_dev_args_for_ue(self)
+
+# Single carrier
+if self.enb.num_ports() == 1:
+# SISO
+rf_dev_args += ',rx_freq0=2630e6,tx_freq0=2510e6'
+elif self.enb.num_ports() == 2:
+# MIMO
+rf_dev_args += 
',rx_freq0=2630e6,rx_freq1=2630e6,tx_freq0=2510e6,tx_freq1=2510e6'
+
+rf_dev_args += ',id=ue,base_srate='+ str(base_srate)
 config.overlay(values, dict(ue=dict(sample_rate = base_srate / 
(1000*1000),
 rf_dev_args = rf_dev_args)))

@@ -285,6 +290,7 @@
 # different cfg namespace 'trx'. Copy needed values over there:
 config.overlay(values, 
dict(trx=dict(rf_dev_type=values['ue'].get('rf_dev_type', None),
  
rf_dev_args=values['ue'].get('rf_dev_args', None),
+ 
rf_dev_sync=values['ue'].get('rf_dev_sync', None),
  
rx_gain=values['ue'].get('rx_gain', None),
  
tx_gain=values['ue'].get('tx_gain', None),
 )))
@@ -331,7 +337,7 @@
 proc.launch_sync()
 return proc

-def verify_metric(self, value, operation='avg', metric='dl_brate', 
criterion='gt'):
+def verify_metric(self, value, operation='avg', metric='dl_brate', 
criterion='gt', window=None):
 return 'metrics not yet implemented with Amarisoft UE'

 # vim: expandtab tabstop=4 shiftwidth=4
diff --git a/src/osmo_gsm_tester/templates/amarisoft_lteue.cfg.tmpl 
b/src/osmo_gsm_tester/templates/amarisoft_lteue.cfg.tmpl
index d6ba32d..6f38491 100644
--- a/src/osmo_gsm_tester/templates/amarisoft_lteue.cfg.tmpl
+++ b/src/osmo_gsm_tester/templates/amarisoft_lteue.cfg.tmpl
@@ -48,7 +48,7 @@
   ue_category: 4,

   /* USIM data */
-  sim_algo: "${sub.auth_algo}", /* USIM authentication algorithm: xor, 
milenage or tuak */
+ 

  1   2   >