openbsc.git branch neels/gtphub updated. 0.15.0-57-g3747d85

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)". The branch, neels/gtphub has been updated discards 93336f7b793b3bb90d1a9a942a03162eb4002f14 (commit) discards 9c4fb015669037c24f2181585210499a50003cf7 (commit) discards 9c434fd6b304c0aaaeb11f537f78a40b3efaa561 (commit) discards 34b4d6a0c746b3a0138dd1fe05e27738b23cd9dd (commit) discards 128f2961eae87ecc76fd3f83174685e74bb3cdc0 (commit) discards 220ff414089c7edfa5011c2d51c901cfbacc2c4d (commit) discards 8595132b70a0c5774824fe74a9d7c4df8d02f474 (commit) discards 9303897ba54260a10aae33344c283afc10eab4bf (commit) via 3747d8553f028b0857a0ca71c7170a8936ff1602 (commit) via 6bc37b2a774d62c0d7ce140344d9ce64326a83bd (commit) via c571a0550f912b7a2ebe85c45143471dd187a660 (commit) via d0073ad283c652a9ec1c05dd0bf0b623a6c9e97d (commit) via 131fc4edf6301f712d0afaf3e3072dc45de13cb5 (commit) via 0fd87fd88965c25bb9e0ec7c67012706d6ff8e83 (commit) via d4237299ec335497df47979faf4bcd498eeac252 (commit) via d7f934218dbe41f0ef8d2f217bfac26d5df466fe (commit) via fd79faead12ae06f1c187bcb962a801f8a250252 (commit) via 9eb20d33ba59fee35c0a91b0e2b24452d2be423c (commit) via c113189009c69fcefe42a245107503ec723502b6 (commit) via 493d49b98e52978917b4a01eaeb5ea7489c419f4 (commit) via 9f50c610b9b2851defb9323bff402edebe1420e4 (commit) via 5d60b06c58b1b3849c7b56991e722bcd11767783 (commit) via 84f8f38b5c1cd7ceeaafc221987421943e7a2040 (commit) via 6187e010a8fb6f503ce7bf0ee93530787acfc876 (commit) This update added new revisions after undoing existing revisions. That is to say, the old revision is not a strict subset of the new revision. This situation occurs when you --force push a change and generate a repository containing something like this: * -- * -- B -- O -- O -- O (93336f7b793b3bb90d1a9a942a03162eb4002f14) \ N -- N -- N (3747d8553f028b0857a0ca71c7170a8936ff1602) When this happens we assume that you've already had alert emails for all of the O revisions, and so we here report only the revisions in the N branch from the common base, B. Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://cgit.osmocom.org/openbsc/commit/?id=3747d8553f028b0857a0ca71c7170a89… commit 3747d8553f028b0857a0ca71c7170a8936ff1602 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 05:51:32 2015 +0100 tei map histogram http://cgit.osmocom.org/openbsc/commit/?id=6bc37b2a774d62c0d7ce140344d9ce64… commit 6bc37b2a774d62c0d7ce140344d9ce64326a83bd Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 04:20:47 2015 +0100 tei stats http://cgit.osmocom.org/openbsc/commit/?id=c571a0550f912b7a2ebe85c45143471d… commit c571a0550f912b7a2ebe85c45143471dd187a660 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 03:16:19 2015 +0100 vty_show http://cgit.osmocom.org/openbsc/commit/?id=d0073ad283c652a9ec1c05dd0bf0b623… commit d0073ad283c652a9ec1c05dd0bf0b623a6c9e97d Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 01:28:40 2015 +0100 gtphub: add first rate counters http://cgit.osmocom.org/openbsc/commit/?id=131fc4edf6301f712d0afaf3e3072dc4… commit 131fc4edf6301f712d0afaf3e3072dc45de13cb5 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 01:27:22 2015 +0100 gtphub: cosmetic: bind pointer clarification. Partly in preparation for upcoming rate counters. http://cgit.osmocom.org/openbsc/commit/?id=0fd87fd88965c25bb9e0ec7c67012706… commit 0fd87fd88965c25bb9e0ec7c67012706d6ff8e83 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 00:08:28 2015 +0100 gtphub: add explicit cleanup handles. Clean up functionality is added for the test suite only, to be able to clean out all allocations and test against memory leaks. So far, it was sufficient to expire everything to free a gtphub. In preparation for the upcoming rate counters, which will need to be freed explicitly, add gtphub functions to clean up everything. As added bonus, also close the sockets explicitly -- not really needed upon program exit, neither by the test suite, but *if* we have a cleanup function, it should clean up everything properly. Closing the sockets is however kept separate, for the test suite. gtphub_start() and gtphub_stop() are for normal use (published in gtphub.h), and gtphub_init() and gtphub_free() are for the test suite, without sockets. In gtphub_test.c, have a function to clean up the testing gtphub struct. First, expire everything by timeout, assert emptiness, then call the cleanup function. Call from each test in the end. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=d4237299ec335497df47979faf4bcd49… commit d4237299ec335497df47979faf4bcd498eeac252 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 00:04:41 2015 +0100 gtphub: ensure cleanup of peer addresses. Upon calling gtphub_peer_del(), all addresses and ports should already have expired (by force). Make sure the code heeds that with a so far missing assertion. http://cgit.osmocom.org/openbsc/commit/?id=d7f934218dbe41f0ef8d2f217bfac26d… commit d7f934218dbe41f0ef8d2f217bfac26d5df466fe Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 19 00:44:52 2015 +0100 gtphub: always send gtphub's own restart counter. http://cgit.osmocom.org/openbsc/commit/?id=fd79faead12ae06f1c187bcb962a801f… commit fd79faead12ae06f1c187bcb962a801f8a250252 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Wed Nov 18 18:11:32 2015 +0100 gtphub: ares vty and init From sgsn_vty.c, copy the cfg_grx_ggsn_cmd to add an ares server to the static sgsn_instance. This is sort of preliminary. As described in comments, the sgsn_ares functions should actually be separated from the static sgsn structure. gtphub keeps such an sgsn structure just for the sgsn_ares functions. http://cgit.osmocom.org/openbsc/commit/?id=9eb20d33ba59fee35c0a91b0e2b24452… commit 9eb20d33ba59fee35c0a91b0e2b24452d2be423c Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Wed Nov 18 18:11:09 2015 +0100 gtphub: review some logging. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=c113189009c69fcefe42a245107503ec… commit c113189009c69fcefe42a245107503ec723502b6 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Wed Nov 18 17:53:00 2015 +0100 gtphub: cosmetic: rename a file. gtphub_ext.c's initial purpose was to wrap a specific function. The file then turned into everything related to DNS, which fits pretty well. Rename to gtphub_ares.c. Tweak the header comment to reflect the new file name. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=493d49b98e52978917b4a01eaeb5ea74… commit 493d49b98e52978917b4a01eaeb5ea7489c419f4 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Wed Nov 18 17:09:22 2015 +0100 gtphub: properly add to debian build Copy all osmo-sgsn related things in debian and s/sgsn/gtphub/g (manually). Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=9f50c610b9b2851defb9323bff402ede… commit 9f50c610b9b2851defb9323bff402edebe1420e4 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 17 14:30:37 2015 +0100 gtphub: nr_map: add min,max and wrap. Implement min/max bounds for nr_pool, adjust nr_pool_init() and current tests, and create unit tests for nr_map wrapping. Sequence numbers range from 0 to 65535, while TEIs range from 1 to 0xffffffff. Both cause problems when the nr_pool surpasses the range: seq exit their valid range, causing unmappings to fail, and a TEI would be mapped as zero (invalid). Add a comment about TEI wrapping, and lose the comment about random TEIs (not really important). Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=5d60b06c58b1b3849c7b56991e722bcd… commit 5d60b06c58b1b3849c7b56991e722bcd11767783 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 17 14:24:46 2015 +0100 gtphub: fix number map range for TEIs. Use unsigned int for nr_map, just large enough to fit the TEI space. Adjust log output formats and casts accordingly. Fixes: TEIs are uint32_t, but the nr_map so far used int. This would cause TEIs from 0x80000000 on to be handled and printed as a negative value. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=84f8f38b5c1cd7ceeaafc22198742194… commit 84f8f38b5c1cd7ceeaafc221987421943e7a2040 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 17 14:21:05 2015 +0100 gtphub: lose obsolete comment. The del_cb is now also used for ares (GGSN resolution) timeouts, and expiry is anyway separated from nr_map, so this comment is void. Sponsored-by: On-Waves ehi ----------------------------------------------------------------------- Summary of changes: openbsc/include/openbsc/gtphub.h | 12 ++- openbsc/src/gprs/gtphub.c | 166 +++++++++++++++++++++++++++++++++---- openbsc/src/gprs/gtphub_main.c | 24 +++--- openbsc/src/gprs/gtphub_vty.c | 149 +++++++++++++++++++++++++++++++-- openbsc/tests/gtphub/gtphub_test.c | 103 ++++++++++++++++++----- 5 files changed, 398 insertions(+), 56 deletions(-) hooks/post-receive -- The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)