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

2019-08-17 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:
[  397s] ar: `u' modifier ignored since `D' is the default (see `U')
[  397s] libtool: link: ranlib .libs/libtransceiver_common.a
[  397s] libtool: link: ( cd ".libs" && rm -f "libtransceiver_common.la" && ln 
-s "../libtransceiver_common.la" "libtransceiver_common.la" )
[  397s] /bin/bash ../libtool  --tag=CXX   --mode=link g++ -lpthread 
-I/usr/include/ -I/usr/include/ -I/usr/include/ -g -O2 
-fdebug-prefix-map=/usr/src/packages/BUILD=. -fstack-protector-strong -Wformat 
-Werror=format-security  -Wl,-z,relro -Wl,-z,now -o osmo-trx-uhd 
osmo_trx_uhd-osmo-trx.o ./device/uhd/libdevice.la libtransceiver_common.la 
../Transceiver52M/arch/x86/libarch.la ../GSM/libGSM.la 
../CommonLibs/libcommon.la -lfftw3f -ltalloc -losmocore -ltalloc -losmoctrl 
-losmogsm -losmocore -ltalloc -losmovty -losmocore -luhd 
[  398s] libtool: link: g++ -I/usr/include/ -I/usr/include/ -I/usr/include/ -g 
-O2 -fdebug-prefix-map=/usr/src/packages/BUILD=. -fstack-protector-strong 
-Wformat -Werror=format-security -Wl,-z -Wl,relro -Wl,-z -Wl,now -o 
osmo-trx-uhd osmo_trx_uhd-osmo-trx.o  ./device/uhd/.libs/libdevice.a 
./.libs/libtransceiver_common.a ../Transceiver52M/arch/x86/.libs/libarch.a 
../GSM/.libs/libGSM.a ../CommonLibs/.libs/libcommon.a -lpthread -lfftw3f 
/usr/lib/x86_64-linux-gnu/libosmoctrl.so 
/usr/lib/x86_64-linux-gnu/libosmogsm.so -ltalloc 
/usr/lib/x86_64-linux-gnu/libosmovty.so 
/usr/lib/x86_64-linux-gnu/libosmocore.so -luhd
[  398s] /usr/bin/ld: ./device/uhd/.libs/libdevice.a(UHDDevice.o): undefined 
reference to symbol '_ZN5boost6system16generic_categoryEv'
[  398s] /usr/bin/ld: /usr/lib/x86_64-linux-gnu/libboost_system.so.1.67.0: 
error adding symbols: DSO missing from command line
[  398s] collect2: error: ld returned 1 exit status
[  398s] make[4]: *** [Makefile:681: osmo-trx-uhd] Error 1
[  398s] make[4]: Leaving directory '/usr/src/packages/BUILD/Transceiver52M'
[  398s] make[3]: *** [Makefile:820: all-recursive] Error 1
[  398s] make[3]: Leaving directory '/usr/src/packages/BUILD/Transceiver52M'
[  398s] make[2]: *** [Makefile:513: all-recursive] Error 1
[  398s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  398s] make[1]: *** [Makefile:444: all] Error 2
[  398s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  398s] dh_auto_build: make -j1 returned exit code 2
[  398s] make: *** [debian/rules:6: build] Error 255
[  398s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  398s] 
[  398s] lamb12 failed "build osmo-trx_1.1.1.dsc" at Sun Aug 18 01:43:55 UTC 
2019.
[  398s] 
[  398s] ### VM INTERACTION START ###
[  401s] [  382.620194] sysrq: SysRq : Power Off
[  401s] [  382.629657] reboot: Power down
[  401s] ### VM INTERACTION END ###
[  401s] 
[  401s] lamb12 failed "build osmo-trx_1.1.1.dsc" at Sun Aug 18 01:43:58 UTC 
2019.
[  401s] 

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


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

2019-08-17 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:latest/osmo-trx/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  426s] ar: `u' modifier ignored since `D' is the default (see `U')
[  426s] libtool: link: ranlib .libs/libtransceiver_common.a
[  426s] libtool: link: ( cd ".libs" && rm -f "libtransceiver_common.la" && ln 
-s "../libtransceiver_common.la" "libtransceiver_common.la" )
[  426s] /bin/bash ../libtool  --tag=CXX   --mode=link g++ -lpthread 
-I/usr/include/ -I/usr/include/ -I/usr/include/ -g -O2 
-fdebug-prefix-map=/usr/src/packages/BUILD=. -fstack-protector-strong -Wformat 
-Werror=format-security  -Wl,-z,relro -Wl,-z,now -o osmo-trx-uhd 
osmo_trx_uhd-osmo-trx.o ./device/uhd/libdevice.la libtransceiver_common.la 
../Transceiver52M/arch/x86/libarch.la ../GSM/libGSM.la 
../CommonLibs/libcommon.la -lfftw3f -ltalloc -losmocore -ltalloc -losmoctrl 
-losmogsm -losmocore -ltalloc -losmovty -losmocore -luhd 
[  426s] libtool: link: g++ -I/usr/include/ -I/usr/include/ -I/usr/include/ -g 
-O2 -fdebug-prefix-map=/usr/src/packages/BUILD=. -fstack-protector-strong 
-Wformat -Werror=format-security -Wl,-z -Wl,relro -Wl,-z -Wl,now -o 
osmo-trx-uhd osmo_trx_uhd-osmo-trx.o  ./device/uhd/.libs/libdevice.a 
./.libs/libtransceiver_common.a ../Transceiver52M/arch/x86/.libs/libarch.a 
../GSM/.libs/libGSM.a ../CommonLibs/.libs/libcommon.a -lpthread -lfftw3f 
/usr/lib/x86_64-linux-gnu/libosmoctrl.so 
/usr/lib/x86_64-linux-gnu/libosmogsm.so -ltalloc 
/usr/lib/x86_64-linux-gnu/libosmovty.so 
/usr/lib/x86_64-linux-gnu/libosmocore.so -luhd
[  426s] /usr/bin/ld: ./device/uhd/.libs/libdevice.a(UHDDevice.o): undefined 
reference to symbol '_ZN5boost6system16generic_categoryEv'
[  426s] /usr/bin/ld: /usr/lib/x86_64-linux-gnu/libboost_system.so.1.67.0: 
error adding symbols: DSO missing from command line
[  426s] collect2: error: ld returned 1 exit status
[  426s] make[4]: *** [Makefile:681: osmo-trx-uhd] Error 1
[  426s] make[4]: Leaving directory '/usr/src/packages/BUILD/Transceiver52M'
[  426s] make[3]: *** [Makefile:820: all-recursive] Error 1
[  426s] make[3]: Leaving directory '/usr/src/packages/BUILD/Transceiver52M'
[  426s] make[2]: *** [Makefile:513: all-recursive] Error 1
[  426s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  426s] make[1]: *** [Makefile:444: all] Error 2
[  426s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  426s] dh_auto_build: make -j1 returned exit code 2
[  426s] make: *** [debian/rules:6: build] Error 255
[  426s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  426s] 
[  426s] lamb57 failed "build osmo-trx_1.1.1.dsc" at Sat Aug 17 21:47:35 UTC 
2019.
[  426s] 
[  426s] ### VM INTERACTION START ###
[  429s] [  411.336469] sysrq: SysRq : Power Off
[  429s] [  411.343556] reboot: Power down
[  429s] ### VM INTERACTION END ###
[  429s] 
[  429s] lamb57 failed "build osmo-trx_1.1.1.dsc" at Sat Aug 17 21:47:38 UTC 
2019.
[  429s] 

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


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

2019-08-17 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:
[  407s] ar: `u' modifier ignored since `D' is the default (see `U')
[  407s] libtool: link: ranlib .libs/libtransceiver_common.a
[  407s] libtool: link: ( cd ".libs" && rm -f "libtransceiver_common.la" && ln 
-s "../libtransceiver_common.la" "libtransceiver_common.la" )
[  407s] /bin/bash ../libtool  --tag=CXX   --mode=link g++ -lpthread 
-I/usr/include/ -I/usr/include/ -I/usr/include/ -g -O2 
-fdebug-prefix-map=/usr/src/packages/BUILD=. -fstack-protector-strong -Wformat 
-Werror=format-security  -Wl,-z,relro -Wl,-z,now -o osmo-trx-uhd 
osmo_trx_uhd-osmo-trx.o ./device/uhd/libdevice.la libtransceiver_common.la 
../Transceiver52M/arch/x86/libarch.la ../GSM/libGSM.la 
../CommonLibs/libcommon.la -lfftw3f -ltalloc -losmocore -ltalloc -losmoctrl 
-losmogsm -losmocore -ltalloc -losmovty -losmocore -luhd 
[  408s] libtool: link: g++ -I/usr/include/ -I/usr/include/ -I/usr/include/ -g 
-O2 -fdebug-prefix-map=/usr/src/packages/BUILD=. -fstack-protector-strong 
-Wformat -Werror=format-security -Wl,-z -Wl,relro -Wl,-z -Wl,now -o 
osmo-trx-uhd osmo_trx_uhd-osmo-trx.o  ./device/uhd/.libs/libdevice.a 
./.libs/libtransceiver_common.a ../Transceiver52M/arch/x86/.libs/libarch.a 
../GSM/.libs/libGSM.a ../CommonLibs/.libs/libcommon.a -lpthread -lfftw3f 
/usr/lib/x86_64-linux-gnu/libosmoctrl.so 
/usr/lib/x86_64-linux-gnu/libosmogsm.so -ltalloc 
/usr/lib/x86_64-linux-gnu/libosmovty.so 
/usr/lib/x86_64-linux-gnu/libosmocore.so -luhd
[  408s] /usr/bin/ld: ./device/uhd/.libs/libdevice.a(UHDDevice.o): undefined 
reference to symbol '_ZN5boost6system16generic_categoryEv'
[  408s] /usr/bin/ld: /usr/lib/x86_64-linux-gnu/libboost_system.so.1.67.0: 
error adding symbols: DSO missing from command line
[  408s] collect2: error: ld returned 1 exit status
[  408s] make[4]: *** [Makefile:681: osmo-trx-uhd] Error 1
[  408s] make[4]: Leaving directory '/usr/src/packages/BUILD/Transceiver52M'
[  408s] make[3]: *** [Makefile:820: all-recursive] Error 1
[  408s] make[3]: Leaving directory '/usr/src/packages/BUILD/Transceiver52M'
[  408s] make[2]: *** [Makefile:513: all-recursive] Error 1
[  408s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  408s] make[1]: *** [Makefile:444: all] Error 2
[  408s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  408s] dh_auto_build: make -j1 returned exit code 2
[  408s] make: *** [debian/rules:6: build] Error 255
[  408s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  408s] 
[  408s] lamb27 failed "build osmo-trx_1.1.1.dsc" at Sat Aug 17 21:44:44 UTC 
2019.
[  408s] 
[  408s] ### VM INTERACTION START ###
[  411s] [  391.626558] sysrq: SysRq : Power Off
[  411s] [  391.634732] reboot: Power down
[  411s] ### VM INTERACTION END ###
[  411s] 
[  411s] lamb27 failed "build osmo-trx_1.1.1.dsc" at Sat Aug 17 21:44:47 UTC 
2019.
[  411s] 

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


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

2019-08-17 Thread OBS Notification
Visit 
https://build.opensuse.org/package/live_build_log/network:osmocom:latest/osmo-trx/Debian_Unstable/x86_64

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

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

Last lines of build log:
[  488s] ar: `u' modifier ignored since `D' is the default (see `U')
[  488s] libtool: link: ranlib .libs/libtransceiver_common.a
[  488s] libtool: link: ( cd ".libs" && rm -f "libtransceiver_common.la" && ln 
-s "../libtransceiver_common.la" "libtransceiver_common.la" )
[  488s] /bin/bash ../libtool  --tag=CXX   --mode=link g++ -lpthread 
-I/usr/include/ -I/usr/include/ -I/usr/include/ -g -O2 
-fdebug-prefix-map=/usr/src/packages/BUILD=. -fstack-protector-strong -Wformat 
-Werror=format-security  -Wl,-z,relro -Wl,-z,now -o osmo-trx-uhd 
osmo_trx_uhd-osmo-trx.o ./device/uhd/libdevice.la libtransceiver_common.la 
../Transceiver52M/arch/x86/libarch.la ../GSM/libGSM.la 
../CommonLibs/libcommon.la -lfftw3f -ltalloc -losmocore -ltalloc -losmoctrl 
-losmogsm -losmocore -ltalloc -losmovty -losmocore -luhd 
[  489s] libtool: link: g++ -I/usr/include/ -I/usr/include/ -I/usr/include/ -g 
-O2 -fdebug-prefix-map=/usr/src/packages/BUILD=. -fstack-protector-strong 
-Wformat -Werror=format-security -Wl,-z -Wl,relro -Wl,-z -Wl,now -o 
osmo-trx-uhd osmo_trx_uhd-osmo-trx.o  ./device/uhd/.libs/libdevice.a 
./.libs/libtransceiver_common.a ../Transceiver52M/arch/x86/.libs/libarch.a 
../GSM/.libs/libGSM.a ../CommonLibs/.libs/libcommon.a -lpthread -lfftw3f 
/usr/lib/x86_64-linux-gnu/libosmoctrl.so 
/usr/lib/x86_64-linux-gnu/libosmogsm.so -ltalloc 
/usr/lib/x86_64-linux-gnu/libosmovty.so 
/usr/lib/x86_64-linux-gnu/libosmocore.so -luhd
[  489s] /usr/bin/ld: ./device/uhd/.libs/libdevice.a(UHDDevice.o): undefined 
reference to symbol '_ZN5boost6system16generic_categoryEv'
[  489s] /usr/bin/ld: /usr/lib/x86_64-linux-gnu/libboost_system.so.1.67.0: 
error adding symbols: DSO missing from command line
[  489s] collect2: error: ld returned 1 exit status
[  489s] make[4]: *** [Makefile:681: osmo-trx-uhd] Error 1
[  489s] make[4]: Leaving directory '/usr/src/packages/BUILD/Transceiver52M'
[  489s] make[3]: *** [Makefile:820: all-recursive] Error 1
[  489s] make[3]: Leaving directory '/usr/src/packages/BUILD/Transceiver52M'
[  489s] make[2]: *** [Makefile:513: all-recursive] Error 1
[  489s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  489s] make[1]: *** [Makefile:444: all] Error 2
[  489s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  489s] dh_auto_build: make -j1 returned exit code 2
[  489s] make: *** [debian/rules:6: build] Error 255
[  489s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  489s] 
[  489s] cloud131 failed "build osmo-trx_1.1.1.dsc" at Sat Aug 17 16:47:54 UTC 
2019.
[  489s] 
[  489s] ### VM INTERACTION START ###
[  492s] [  449.334574] sysrq: SysRq : Power Off
[  492s] [  449.340962] reboot: Power down
[  494s] ### VM INTERACTION END ###
[  494s] 
[  494s] cloud131 failed "build osmo-trx_1.1.1.dsc" at Sat Aug 17 16:48:00 UTC 
2019.
[  494s] 

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


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

2019-08-17 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:
[  399s] ar: `u' modifier ignored since `D' is the default (see `U')
[  399s] libtool: link: ranlib .libs/libtransceiver_common.a
[  399s] libtool: link: ( cd ".libs" && rm -f "libtransceiver_common.la" && ln 
-s "../libtransceiver_common.la" "libtransceiver_common.la" )
[  399s] /bin/bash ../libtool  --tag=CXX   --mode=link g++ -lpthread 
-I/usr/include/ -I/usr/include/ -I/usr/include/ -g -O2 
-fdebug-prefix-map=/usr/src/packages/BUILD=. -fstack-protector-strong -Wformat 
-Werror=format-security  -Wl,-z,relro -Wl,-z,now -o osmo-trx-uhd 
osmo_trx_uhd-osmo-trx.o ./device/uhd/libdevice.la libtransceiver_common.la 
../Transceiver52M/arch/x86/libarch.la ../GSM/libGSM.la 
../CommonLibs/libcommon.la -lfftw3f -ltalloc -losmocore -ltalloc -losmoctrl 
-losmogsm -losmocore -ltalloc -losmovty -losmocore -luhd 
[  399s] libtool: link: g++ -I/usr/include/ -I/usr/include/ -I/usr/include/ -g 
-O2 -fdebug-prefix-map=/usr/src/packages/BUILD=. -fstack-protector-strong 
-Wformat -Werror=format-security -Wl,-z -Wl,relro -Wl,-z -Wl,now -o 
osmo-trx-uhd osmo_trx_uhd-osmo-trx.o  ./device/uhd/.libs/libdevice.a 
./.libs/libtransceiver_common.a ../Transceiver52M/arch/x86/.libs/libarch.a 
../GSM/.libs/libGSM.a ../CommonLibs/.libs/libcommon.a -lpthread -lfftw3f 
/usr/lib/x86_64-linux-gnu/libosmoctrl.so 
/usr/lib/x86_64-linux-gnu/libosmogsm.so -ltalloc 
/usr/lib/x86_64-linux-gnu/libosmovty.so 
/usr/lib/x86_64-linux-gnu/libosmocore.so -luhd
[  399s] /usr/bin/ld: ./device/uhd/.libs/libdevice.a(UHDDevice.o): undefined 
reference to symbol '_ZN5boost6system16generic_categoryEv'
[  399s] /usr/bin/ld: /usr/lib/x86_64-linux-gnu/libboost_system.so.1.67.0: 
error adding symbols: DSO missing from command line
[  399s] collect2: error: ld returned 1 exit status
[  399s] make[4]: *** [Makefile:681: osmo-trx-uhd] Error 1
[  399s] make[4]: Leaving directory '/usr/src/packages/BUILD/Transceiver52M'
[  399s] make[3]: *** [Makefile:820: all-recursive] Error 1
[  399s] make[3]: Leaving directory '/usr/src/packages/BUILD/Transceiver52M'
[  399s] make[2]: *** [Makefile:513: all-recursive] Error 1
[  399s] make[2]: Leaving directory '/usr/src/packages/BUILD'
[  399s] make[1]: *** [Makefile:444: all] Error 2
[  399s] make[1]: Leaving directory '/usr/src/packages/BUILD'
[  399s] dh_auto_build: make -j1 returned exit code 2
[  399s] make: *** [debian/rules:6: build] Error 255
[  399s] dpkg-buildpackage: error: debian/rules build subprocess returned exit 
status 2
[  399s] 
[  399s] lamb68 failed "build osmo-trx_1.1.1.dsc" at Sat Aug 17 16:44:05 UTC 
2019.
[  399s] 
[  399s] ### VM INTERACTION START ###
[  402s] [  384.271135] sysrq: SysRq : Power Off
[  402s] [  384.280272] reboot: Power down
[  402s] ### VM INTERACTION END ###
[  403s] 
[  403s] lamb68 failed "build osmo-trx_1.1.1.dsc" at Sat Aug 17 16:44:09 UTC 
2019.
[  403s] 

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


Change in ...osmo-ggsn[master]: libgtp: Remove packets in tx queue belonging pdp being freed

2019-08-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ggsn/+/15228 )

Change subject: libgtp: Remove packets in tx queue belonging pdp being freed
..


Patch Set 4:

Forgot an extra point:
* libgtp already uses LOGP()


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

Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: Id86d0b241454d3ad49c64c28087fd2710fa2d17a
Gerrit-Change-Number: 15228
Gerrit-PatchSet: 4
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: keith 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-CC: laforge 
Gerrit-Comment-Date: Sat, 17 Aug 2019 16:15:26 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in ...osmo-ggsn[master]: libgtp: Remove packets in tx queue belonging pdp being freed

2019-08-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ggsn/+/15228 )

Change subject: libgtp: Remove packets in tx queue belonging pdp being freed
..


Patch Set 4:

libgtp already depends on libosmocore
* It's already in osmo-ggsn.git's configure.ac as a PKG_CHECK_MODULES, so it 
depends on it at compile time.
* only known users to us (osmo-ggsn, osmo-sgsn, sgsnemu?) already depend on 
libosmocore both compile time and runtime.
* libgtp uses OSMO_DEPRECATED(), wich means again libosmocore is needed at 
compile time.
* libgtp uses OSMO_ASSERT(), so again needed at compile time.
* libgtp uses osmo_bcd2char(), so it requires libosmocore at runtime
* in Makefile.am, libgtp already links against libosmocore: "libgtp_la_LIBADD = 
$(LIBOSMOCORE_LIBS)"

So i'm in favour of actually moving libgtp to be more "osmocom" rather than 
trying to keep it away from it.


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

Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: Id86d0b241454d3ad49c64c28087fd2710fa2d17a
Gerrit-Change-Number: 15228
Gerrit-PatchSet: 4
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: keith 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-CC: laforge 
Gerrit-Comment-Date: Sat, 17 Aug 2019 14:20:15 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in ...osmo-sgsn[master]: gprs_sgsn.h: Flag MM_CTX_T_GERAN_Iu as not supported

2019-08-17 Thread pespin
pespin has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/15184 )

Change subject: gprs_sgsn.h: Flag MM_CTX_T_GERAN_Iu as not supported
..

gprs_sgsn.h: Flag MM_CTX_T_GERAN_Iu as not supported

Change-Id: I3b53a530ab25434e2b2f4d80ad70a8a5f22bfcac
---
M include/osmocom/sgsn/gprs_sgsn.h
M src/gprs/sgsn_libgtp.c
2 files changed, 5 insertions(+), 1 deletion(-)

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



diff --git a/include/osmocom/sgsn/gprs_sgsn.h b/include/osmocom/sgsn/gprs_sgsn.h
index a0a056f..10455d6 100644
--- a/include/osmocom/sgsn/gprs_sgsn.h
+++ b/include/osmocom/sgsn/gprs_sgsn.h
@@ -108,8 +108,10 @@
MM_CTX_T_GERAN_Gb,
/* UMTS via Iu */
MM_CTX_T_UTRAN_Iu,
-   /* GPRS/EDGE via Iu */
+#if 0
+   /* GPRS/EDGE via Iu, not supported */
MM_CTX_T_GERAN_Iu,
+#endif
 };

 struct service_info {
diff --git a/src/gprs/sgsn_libgtp.c b/src/gprs/sgsn_libgtp.c
index e9fffd8..88b8d04 100644
--- a/src/gprs/sgsn_libgtp.c
+++ b/src/gprs/sgsn_libgtp.c
@@ -279,7 +279,9 @@
pdp->userloc.l = 8;
switch (mmctx->ran_type) {
case MM_CTX_T_GERAN_Gb:
+#if 0
case MM_CTX_T_GERAN_Iu:
+#endif
pdp->rattype.v[0] = 2;
/* User Location Information */
pdp->userloc_given = 1;

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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I3b53a530ab25434e2b2f4d80ad70a8a5f22bfcac
Gerrit-Change-Number: 15184
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-CC: lynxis lazus 
Gerrit-MessageType: merged


Change in ...osmo-sgsn[master]: gprs_gmm: Introduce assert to guard against unexpected condition

2019-08-17 Thread pespin
pespin has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/15167 )

Change subject: gprs_gmm: Introduce assert to guard against unexpected condition
..

gprs_gmm: Introduce assert to guard against unexpected condition

This may well be the culprit of OS#3957, were already freed llme is accessed 
from
mmctx context later on, upon some timer is triggered in mmctx.

Related: OS#3957

Change-Id: I8e1eaeb9b3ebee8e45704b4fe007190c7db609e4
---
M src/gprs/gprs_gmm.c
1 file changed, 5 insertions(+), 0 deletions(-)

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



diff --git a/src/gprs/gprs_gmm.c b/src/gprs/gprs_gmm.c
index 0fcf1bb..718fc97 100644
--- a/src/gprs/gprs_gmm.c
+++ b/src/gprs/gprs_gmm.c
@@ -1741,6 +1741,11 @@
"The MM context cannot be used, RA: %03d-%0*d-%d-%d\n",
mmctx->ra.mcc, mmctx->ra.mnc_3_digits, mmctx->ra.mnc,
mmctx->ra.lac, mmctx->ra.rac);
+   /* mmctx is set to NULL and gprs_llgmm_unassign(llme) will be
+  called below, let's make sure we don't keep dangling llme
+  pointers in mmctx (OS#3957). */
+   if (mmctx->ran_type == MM_CTX_T_GERAN_Gb)
+   OSMO_ASSERT(mmctx->gb.llme == NULL);
mmctx = NULL;
}


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I8e1eaeb9b3ebee8e45704b4fe007190c7db609e4
Gerrit-Change-Number: 15167
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in ...osmo-sgsn[master]: gprs_gmm: Introduce assert to guard against unexpected condition

2019-08-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/15167 )

Change subject: gprs_gmm: Introduce assert to guard against unexpected condition
..


Patch Set 3: Code-Review+2


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I8e1eaeb9b3ebee8e45704b4fe007190c7db609e4
Gerrit-Change-Number: 15167
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Sat, 17 Aug 2019 14:10:23 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-sgsn[master]: gprs_gmm.c: Flag mmctx_set_(p)mm_state() functions static

2019-08-17 Thread pespin
pespin has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/15185 )

Change subject: gprs_gmm.c: Flag mmctx_set_(p)mm_state() functions static
..

gprs_gmm.c: Flag mmctx_set_(p)mm_state() functions static

Change-Id: I061144b6994ee40d5b32eb321dd4f3d3786d028d
---
M src/gprs/gprs_gmm.c
1 file changed, 2 insertions(+), 2 deletions(-)

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



diff --git a/src/gprs/gprs_gmm.c b/src/gprs/gprs_gmm.c
index bb0e002..e1bed0d 100644
--- a/src/gprs/gprs_gmm.c
+++ b/src/gprs/gprs_gmm.c
@@ -136,7 +136,7 @@
}
 }

-void mmctx_set_pmm_state(struct sgsn_mm_ctx *ctx, enum gprs_pmm_state state)
+static void mmctx_set_pmm_state(struct sgsn_mm_ctx *ctx, enum gprs_pmm_state 
state)
 {
if (ctx->ran_type != MM_CTX_T_UTRAN_Iu)
return;
@@ -162,7 +162,7 @@
ctx->pmm_state = state;
 }

-void mmctx_set_mm_state(struct sgsn_mm_ctx *ctx, enum gprs_pmm_state state)
+static void mmctx_set_mm_state(struct sgsn_mm_ctx *ctx, enum gprs_pmm_state 
state)
 {
if (ctx->ran_type != MM_CTX_T_GERAN_Gb)
return;

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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I061144b6994ee40d5b32eb321dd4f3d3786d028d
Gerrit-Change-Number: 15185
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in ...osmo-sgsn[master]: gprs_gmm.c: Use correct function to set MM_IDLE state during cleanup

2019-08-17 Thread pespin
pespin has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/15183 )

Change subject: gprs_gmm.c: Use correct function to set MM_IDLE state during 
cleanup
..

gprs_gmm.c: Use correct function to set MM_IDLE state during cleanup

Change-Id: Ic6912269d0d69c86f19e57f3271ebda1328e968f
---
M src/gprs/gprs_gmm.c
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/src/gprs/gprs_gmm.c b/src/gprs/gprs_gmm.c
index 718fc97..bb0e002 100644
--- a/src/gprs/gprs_gmm.c
+++ b/src/gprs/gprs_gmm.c
@@ -327,7 +327,7 @@
/* Mark MM state as deregistered */
ctx->gmm_state = GMM_DEREGISTERED;
mmctx_set_pmm_state(ctx, PMM_DETACHED);
-   mmctx_set_pmm_state(ctx, MM_IDLE);
+   mmctx_set_mm_state(ctx, MM_IDLE);

sgsn_mm_ctx_cleanup_free(ctx);
 }

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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: Ic6912269d0d69c86f19e57f3271ebda1328e968f
Gerrit-Change-Number: 15183
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: pespin 
Gerrit-MessageType: merged


Change in ...osmo-sgsn[master]: gprs_gmm: Introduce assert to guard against unexpected condition

2019-08-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/15167 )

Change subject: gprs_gmm: Introduce assert to guard against unexpected condition
..


Patch Set 3:

+1+1=+2


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I8e1eaeb9b3ebee8e45704b4fe007190c7db609e4
Gerrit-Change-Number: 15167
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Sat, 17 Aug 2019 14:10:17 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in ...osmo-iuh[master]: iu_client: introduce ranap_iu_free_ue() to free the UE connections

2019-08-17 Thread pespin
pespin has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-iuh/+/15161 )

Change subject: iu_client: introduce ranap_iu_free_ue() to free the UE 
connections
..


Patch Set 6: -Code-Review


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

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I6812edb4da487507ccf9f483de801c1c65c4b72b
Gerrit-Change-Number: 15161
Gerrit-PatchSet: 6
Gerrit-Owner: lynxis lazus 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Sat, 17 Aug 2019 14:06:38 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-sgsn[master]: gprs_sgsn.h: Flag MM_CTX_T_GERAN_Iu as not supported

2019-08-17 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/15184 )

Change subject: gprs_sgsn.h: Flag MM_CTX_T_GERAN_Iu as not supported
..


Patch Set 3: Code-Review+1


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I3b53a530ab25434e2b2f4d80ad70a8a5f22bfcac
Gerrit-Change-Number: 15184
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-CC: lynxis lazus 
Gerrit-Comment-Date: Sat, 17 Aug 2019 08:50:06 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in ...osmo-sgsn[master]: gprs_gmm: Introduce assert to guard against unexpected condition

2019-08-17 Thread fixeria
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-sgsn/+/15167 )

Change subject: gprs_gmm: Introduce assert to guard against unexpected condition
..


Patch Set 3: Code-Review+1

Much cleaner now.


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

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I8e1eaeb9b3ebee8e45704b4fe007190c7db609e4
Gerrit-Change-Number: 15167
Gerrit-PatchSet: 3
Gerrit-Owner: pespin 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria 
Gerrit-Reviewer: laforge 
Gerrit-Reviewer: lynxis lazus 
Gerrit-Reviewer: neels 
Gerrit-Reviewer: pespin 
Gerrit-Comment-Date: Sat, 17 Aug 2019 08:46:54 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment