osmocom-commitlog

osmocom-commitlog@lists.osmocom.org

7948 discussions

openbsc.git branch master updated. 0.15.0-55-gff252bb
by gitosis＠osmocom.org 30 Nov '15

30 Nov '15

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, master has been updated via ff252bb80172826391b6286fd240266fe842cc83 (commit) from 8fbfaa51bb3f583eeebc1239388832b2e966d424 (commit) 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=ff252bb80172826391b6286fd240266f… commit ff252bb80172826391b6286fd240266fe842cc83 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Sun Nov 29 18:44:55 2015 +0100 gtphub_test: build only when c-ares and gtp are present. ----------------------------------------------------------------------- Summary of changes: openbsc/tests/gtphub/Makefile.am | 4 ++++ 1 file changed, 4 insertions(+) hooks/post-receive -- The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)

1 0

osmo-pcu.git branch jerlbeck/wip/edge updated. 0.2-612-gebe7628
by gitosis＠osmocom.org 28 Nov '15

28 Nov '15

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 "UNNAMED PROJECT". The branch, jerlbeck/wip/edge has been updated discards fcd7558506adb0d02d8cb18dbb7da23643dcd195 (commit) discards e4f82efe8202c26c690430e92359df87193dd96b (commit) discards 0ff208a5e14289f293bab8ce113d3d72b460a702 (commit) discards 212171cb67d9bc3add09fc1ba566d0f2db9d63ae (commit) discards 6ea0240e894376d85e83fdad8d433da95412735e (commit) via ebe7628ee120988381a35c1b2b5d13694757b56b (commit) via 483de5ea056b0d5259a81735a7faec2c2d177301 (commit) via e44582acdccda82484e68691393507e6e064b901 (commit) via a19e5a536b9f8d0b38abc86ddda1cdda5e2e74f2 (commit) via 193f5a45f44d23c28613dfc636d997fe74af7262 (commit) via f3a2df2c1e07ac06f002d2196a315037ad97af5c (commit) via f0cf3d6325de6e695434954cd8514a9b72be8488 (commit) via edfd7e3d94b7868c0a5eb18cde68bb03805fbbb4 (commit) via acfb883011552a54c1d5b1b727466a3f29768e61 (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 (fcd7558506adb0d02d8cb18dbb7da23643dcd195) \ N -- N -- N (ebe7628ee120988381a35c1b2b5d13694757b56b) 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/osmo-pcu/commit/?id=ebe7628ee120988381a35c1b2b5d136… commit ebe7628ee120988381a35c1b2b5d13694757b56b Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Fri Nov 27 18:10:39 2015 +0100 edge: Extend gprs_rlcmac_dl_tbf::handle by egprs_ms_class (TODO) TODO - commit message - get egprs_ms_class from BSSGP RA cap - check usage ms_class in append http://cgit.osmocom.org/osmo-pcu/commit/?id=483de5ea056b0d5259a81735a7faec2… commit 483de5ea056b0d5259a81735a7faec2c2d177301 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Mon Sep 28 18:54:32 2015 +0200 edge: Enable EGPRS if configured and egprs_ms_class present Enable the TBF to use EGPRS if the bts->egprs_enabled config variable has been set via the VTY "egprs" command and if the MS has signaled a EGPRS multislot class. Tell the MS to use EGPRS if the condition above holds. Note that this will cause the MS to use EGPRS RLC block formats for further messages which are not yet understood by the PCU. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=e44582acdccda82484e68691393507e… commit e44582acdccda82484e68691393507e6e064b901 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Fri Nov 27 15:17:34 2015 +0100 edge: Support EGPRS multislot class handling in tbf_alloc Add an egprs_ms_class argument to the allocation functions and set/pass it where necessary. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=a19e5a536b9f8d0b38abc86ddda1cdd… commit a19e5a536b9f8d0b38abc86ddda1cdda5e2e74f2 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Fri Nov 27 16:17:40 2015 +0100 edge: Add m_egprs_enabled and related methods to TBF Add the following methods to gprs_rlcmac_tbf: - is_egprs_enabled - enable_egprs - disable_egprs Also show the value of the flag in name() by displaying "EGPRS" if it is set. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=193f5a45f44d23c28613dfc636d997f… commit 193f5a45f44d23c28613dfc636d997fe74af7262 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Mon Sep 28 18:54:32 2015 +0200 edge: Support EGPRS in packet uplink assignment message Currently the Encoding::write_packet_uplink_assignment method only supports the GPRS variant of the message. This commit adds the missing EGPRS variant to the encoder. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=f3a2df2c1e07ac06f002d2196a31503… commit f3a2df2c1e07ac06f002d2196a315037ad97af5c Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Mon Sep 28 18:12:57 2015 +0200 edge: Add egprs config command Add a global config flag to enable the use EDGE/EGPRS. The following VTY commands are added to node config-pcu: - egprs Enables EGPRS - no egprs Disable EGPRS Note that enabling EGPRS is experimental and will most likely break packet transmission until a minimal required set of EGPRS functionality is implemented. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=f0cf3d6325de6e695434954cd8514a9… commit f0cf3d6325de6e695434954cd8514a9b72be8488 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Mon Sep 28 17:55:32 2015 +0200 edge: Get EGPRS multislot class The EGPRS MS class ist contained in the MS_RA_capability information. Its presence indicates, that the MS is able (and willing) to use EGPRS. This commit implements basic support for retrieving, storing, and showing it in the VTY. The information is stored in the MS object. Sponsored-by: On-Waves ehf ----------------------------------------------------------------------- Summary of changes: examples/osmo-pcu.cfg | 4 ++++ src/gprs_bssgp_pcu.cpp | 4 +++- src/pcu_vty.c | 5 ++++- src/tbf.cpp | 7 ++++--- src/tbf.h | 6 +++--- src/tbf_dl.cpp | 10 ++++++---- tests/alloc/AllocTest.cpp | 15 ++++++++------- tests/tbf/TbfTest.cpp | 14 +++++++------- tests/tbf/TbfTest.err | 20 ++++++++++---------- 9 files changed, 49 insertions(+), 36 deletions(-) hooks/post-receive -- UNNAMED PROJECT

1 0

osmo-pcu.git branch master updated. 0.2-605-gedfd7e3
by gitosis＠osmocom.org 27 Nov '15

27 Nov '15

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 "UNNAMED PROJECT". The branch, master has been updated via edfd7e3d94b7868c0a5eb18cde68bb03805fbbb4 (commit) via acfb883011552a54c1d5b1b727466a3f29768e61 (commit) from 42aba81c2f8c8313fcbf85ee422d6b747107d98a (commit) 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/osmo-pcu/commit/?id=edfd7e3d94b7868c0a5eb18cde68bb0… commit edfd7e3d94b7868c0a5eb18cde68bb03805fbbb4 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Tue Nov 17 16:30:58 2015 +0100 encoder: Whitespace fixes Sponsored-by: On-Waves ehf http://cgit.osmocom.org/osmo-pcu/commit/?id=acfb883011552a54c1d5b1b727466a3… commit acfb883011552a54c1d5b1b727466a3f29768e61 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Thu Nov 26 11:55:56 2015 +0100 pcu: Improve default config Currently the optional features dl-tbf-idle-time and idle-ack-delay are not enabled when using the default config. Without the former, the packet loss is significantly increased since CoDel is enabled by default, eventually throwing away packets from ongoing paging and TBF establishment procedures. This commit changes the default config for satisfactory results even with a single PDCH. Sponsored-by: On-Waves ehf ----------------------------------------------------------------------- Summary of changes: examples/osmo-pcu.cfg | 4 ++++ src/encoding.cpp | 8 ++++---- 2 files changed, 8 insertions(+), 4 deletions(-) hooks/post-receive -- UNNAMED PROJECT

1 0

openbsc.git branch neels/gtphub updated. 0.15.0-76-g0321c1a
by gitosis＠osmocom.org 27 Nov '15

27 Nov '15

1 0

openbsc.git branch neels/gtphub updated. 0.15.0-73-g346d506
by gitosis＠osmocom.org 27 Nov '15

27 Nov '15

1 0

openbsc.git branch neels/gtphub updated. 0.15.0-73-gb9d5c9b
by gitosis＠osmocom.org 27 Nov '15

27 Nov '15

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 041995dacf26082cb4883a99f8d5a30c109efff7 (commit) discards ae1e7a00efcdea016008c54cd442e38c141e4200 (commit) discards cf6ba14087de5ae464528280be3a784f8c6c6d6d (commit) discards 22174abe556b8419a42b3b67b191ac94e30d81c5 (commit) discards 79dce146fbb4096f737776d5c217b71a31b16eb9 (commit) discards c33ac58604665455699eddb2bad5cfd8242f193c (commit) discards 310b8bf647c8b2478dec3a27c11942cf4b6f1e6a (commit) discards 97c5e61b0a58a507a5ad5ee0559da9182596d287 (commit) discards 9586e51efd243cce41a887448bf442f3f49d4c5f (commit) discards 58e14c99f4c6b9be3a7b15c72d0f669b05d713cf (commit) discards 4395a3d823a2243531ef15d05ef498427d9c409b (commit) discards d4c79db2905f6380773adc8d12ddcddc5e21077e (commit) discards a5f12d907cc6554b7c1a0e7c63b6a7287796f149 (commit) discards 27883dd1891686ed50d9fff017b07416ef785cce (commit) discards f9f6739cece7073aca938b5258ab7d590efd9f0f (commit) discards d9e3579689f5678d761ac331c152d1286e8f94ef (commit) discards c80f953d117a8d2406e37f200bf4aa62ef7d3cf2 (commit) discards 70cb65941457b29bf811ee570b3cf21ae48e99bc (commit) discards 3d53bfd7668f491f2904580c33f1a461fa631132 (commit) via b9d5c9b7922965469a6304fa65bd96c9ce8a50eb (commit) via dabb2430cfe8e176bcf83a1cef0a7607c2f72577 (commit) via 8c02ef0a2dd678dd8bf63c15c77c350ad6104e56 (commit) via 3aa01614d1332b9a10920062b04127e5db7d7740 (commit) via a0cb29c321573d84b4e93cfc658cd4ceadbf5041 (commit) via 532e9d7ceb76d1409ede36c9b304cf32a7648d40 (commit) via f321b223d4dc9ad0fc300a781945bf43cca23da4 (commit) via 47eb5e2f2bcf8add821b076ed35b45e4af61672b (commit) via 80524ff23caa213ce14ca7fa0df9da91ac2ffaf8 (commit) via c27f03c0d5208de86c698cf532b56c4f1692c17a (commit) via 62580504d1756f506711a6e96bf063d3ce1dea31 (commit) via 1fcdcc21eecf0cd56b21ac5e4f76a7f9a2ce559e (commit) via 38a6177f76a1425feaec1045d9e28535abc0db21 (commit) via 4c92268cbe855a9451b7acf0bff5e7c53bad2809 (commit) via 4231731d4125bdcf68fc5cd99b6aee0eaa9c9b37 (commit) via 3f40114e0285108d71fba3780566bd2c436ccc92 (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 (041995dacf26082cb4883a99f8d5a30c109efff7) \ N -- N -- N (b9d5c9b7922965469a6304fa65bd96c9ce8a50eb) 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=b9d5c9b7922965469a6304fa65bd96c9… commit b9d5c9b7922965469a6304fa65bd96c9ce8a50eb Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 26 22:19:22 2015 +0100 gtphub: implement restart counter properly. Force passing a restart counter, by adding such arg to gtphub_start() (test suite is not affected by this). In gtphub_main.c, add -r,--restart-file <path> and next_restart_count() to maintain the counter file. While at it, tweak the cmdline help to unify the formatting (mostly commas and a missing line break). Send gtphub's own restart counter. So far, the sender's restart counter was copied through, which would break as soon as more than one GSN would talk to the same peer with differing restart counters. Also fix the in-mem restart counter data type (one octet, not two). Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=dabb2430cfe8e176bcf83a1cef0a7607… commit dabb2430cfe8e176bcf83a1cef0a7607c2f72577 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 26 05:20:18 2015 +0100 gtphub: complain about excess cmdline args. http://cgit.osmocom.org/openbsc/commit/?id=8c02ef0a2dd678dd8bf63c15c77c350a… commit 8c02ef0a2dd678dd8bf63c15c77c350ad6104e56 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:31:06 2015 +0100 gtphub: track tunnels explicitly. So far, gtphub worked perfectly by only tracking single TEIs ... for probably most uses. But a Ctrl plane tunnel may have expired despite a still active corresponding User plane tunnel. The User plane would continue to work indefinitely, but if any Ctrl messages followed after more than six hours of Ctrl silence, they would have been dropped due to an expired TEI mapping. We want to - combine expiry of a user TEI with its ctrl TEI. (done in this patch) - upon delete PDP context, remove both user and ctrl TEI mappings. (future) - when a peer indicates a restart counter bump, invalidate its tunnels. (future) To facilitate these, track tunnels, complete with both SGSN's and GGSN's address, original and replaced TEIs, all for both user and ctrl plane, in a single struct. A single expiry entry handles the entire tunnel, instead of previously four separate expiries for each endpoint identifier. Add the concept of a "side", being either GGSN or SGSN, to index tunnel endpoint structs, and so on. Track the originating side in the gtp_packet_desc. Add header_tei_rx: set_tei() overwrites header_tei, but the originally received header TEI is still needed to match a Create PDP Context Response up with its Request (and for logging). Adjust the test suite to expect tunnel listing strings instead of TEI mappings, with a bonus of making it a lot easier to grok, and including the IP addresses. Add regression test for refreshing tunnel expiry upon use. Note: the current implementation is as slow as can possibly be, iterating all the tunnels all the time. Optimizations are kept for a future commit, on purpose. BTW, the sequence number mapping/unmapping structures remain unchanged. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=3aa01614d1332b9a10920062b04127e5… commit 3aa01614d1332b9a10920062b04127e5db7d7740 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Wed Nov 25 16:45:59 2015 +0100 gtphub: cosmetic/prepare: rename expiry queues. The expiry queues are already used for resolved GGSN addresses, and will soon enlist tunnel structs. Hence the naming should be more general. http://cgit.osmocom.org/openbsc/commit/?id=a0cb29c321573d84b4e93cfc658cd4ce… commit a0cb29c321573d84b4e93cfc658cd4ceadbf5041 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:32:23 2015 +0100 gtphub: add assertion to ensure expiry ordering. Make 100% sure the user adds expiring_items in chronological order by asserting that a newly added expiry is >= the last expiry in the queue. Add llist_last() to facilitate. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=532e9d7ceb76d1409ede36c9b304cf32… commit 532e9d7ceb76d1409ede36c9b304cf32a7648d40 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:30:38 2015 +0100 gtphub: cosmetic/prepare: add nr_map_refresh(). http://cgit.osmocom.org/openbsc/commit/?id=f321b223d4dc9ad0fc300a781945bf43… commit f321b223d4dc9ad0fc300a781945bf43cca23da4 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:27:13 2015 +0100 gtphub: move timestamp into packet struct. Instead of passing the current time around in function arguments ('now'), rather store the current time once upon decoding a GTP packet in the gtp_packet_desc passed around anyway ('p->timestamp'). Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=47eb5e2f2bcf8add821b076ed35b45e4… commit 47eb5e2f2bcf8add821b076ed35b45e4af61672b Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:23:44 2015 +0100 gtphub: cosmetic: fix an argument name. http://cgit.osmocom.org/openbsc/commit/?id=80524ff23caa213ce14ca7fa0df9da91… commit 80524ff23caa213ce14ca7fa0df9da91ac2ffaf8 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 03:16:19 2015 +0100 gtphub: first vty show commands. Start adding VTY commands to show rate counters / statistics / cache dumps. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=c27f03c0d5208de86c698cf532b56c4f… commit c27f03c0d5208de86c698cf532b56c4f1692c17a Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 01:28:40 2015 +0100 gtphub: add first rate counters Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=62580504d1756f506711a6e96bf063d3… commit 62580504d1756f506711a6e96bf063d3ce1dea31 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 01:27:22 2015 +0100 gtphub: cosmetic: clarify bind pointer naming. Some gtphub_bind pointers point to an array of binds, some point directly at instances. Make the distinction between the two more obvious by adding an '_arr' suffix to the array ones. Partly in preparation for upcoming rate counters. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=1fcdcc21eecf0cd56b21ac5e4f76a7f9… commit 1fcdcc21eecf0cd56b21ac5e4f76a7f9a2ce559e 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. (gtphub_stop() will probably never be called by anyone, but its existence completes the picture.) 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=38a6177f76a1425feaec1045d9e28535… commit 38a6177f76a1425feaec1045d9e28535abc0db21 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. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=4c92268cbe855a9451b7acf0bff5e7c5… commit 4c92268cbe855a9451b7acf0bff5e7c53bad2809 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. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=4231731d4125bdcf68fc5cd99b6aee0e… commit 4231731d4125bdcf68fc5cd99b6aee0eaa9c9b37 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=3f40114e0285108d71fba3780566bd2c… commit 3f40114e0285108d71fba3780566bd2c436ccc92 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 ----------------------------------------------------------------------- Summary of changes: hooks/post-receive -- The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)

1 0

openbsc.git branch master updated. 0.15.0-54-g8fbfaa5
by gitosis＠osmocom.org 27 Nov '15

27 Nov '15

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, master has been updated via 8fbfaa51bb3f583eeebc1239388832b2e966d424 (commit) from 4ae165da61243c8ea63f1736d9b1366e6a864d94 (commit) 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 ----------------------------------------------------------------- ----------------------------------------------------------------------- Summary of changes: openbsc/src/gprs/gtphub.c | 69 ----------------------------------------------- 1 file changed, 69 deletions(-) hooks/post-receive -- The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)

1 0

openbsc.git branch neels/gtphub updated. 0.15.0-76-g041995d
by gitosis＠osmocom.org 27 Nov '15

27 Nov '15

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 50a88f8eb4ad183bfc1f870a99c7f0f1f2022acb (commit) discards e201bf5778b073e7101a707435de0777a38b832f (commit) discards 192cf8b56ace2ea168153571f9c5eae53fd47365 (commit) discards 7f2d40ceb755943d52fcae8ddb12c09e2d0c623f (commit) discards c1e297a9f030429a075392b4888d26539a333dbf (commit) discards 444fe1be787fd291b5bc9c347303047e78089c35 (commit) discards 3bf4d4ddd97383a98d3da56f3b57e7aac2adcf54 (commit) discards da53b650561f5c2e81bd499657497713a7276725 (commit) discards 036874d27b458d4359a9132e3361e47b42935117 (commit) discards 3c4c17178f6d41ee83b441b9ce7d55d4e404b73d (commit) discards 89a83524dc4dcf73bedd8c98659bfd006a422a3b (commit) discards 5b3ed5a6cd4bc03b225e59910d647a5d4d21243a (commit) discards 5b231c38318754de6b9b16e91d0f1dc29290872d (commit) discards a0e2c562de54d87a48bdd059e18ef782f18c3cc8 (commit) discards d369720ec8bda4f2fb3ee0b1a719211d66df247a (commit) discards f397ca83bea6f15f194b522d9278695c7a27765f (commit) discards 402b117fe1ec63fe9c30eaa38850a477601799e9 (commit) discards fce0d1ee3b0717d18d081db9ceadbab2525441b8 (commit) discards 786d555de00d65e6c0b07008fe1cc8f593118c2e (commit) discards 42d114b846a4a1852f22ecfd56d8872cae83018a (commit) discards 03203415845cd646c748983f8ab49ecb8ca0581a (commit) discards fa74d6dd746e65166d4f7d62d7c6ff698bec2900 (commit) discards 6de6ea937eae341577dedb93d95bd9ded7fd440f (commit) via 041995dacf26082cb4883a99f8d5a30c109efff7 (commit) via ae1e7a00efcdea016008c54cd442e38c141e4200 (commit) via cf6ba14087de5ae464528280be3a784f8c6c6d6d (commit) via 22174abe556b8419a42b3b67b191ac94e30d81c5 (commit) via 79dce146fbb4096f737776d5c217b71a31b16eb9 (commit) via c33ac58604665455699eddb2bad5cfd8242f193c (commit) via 310b8bf647c8b2478dec3a27c11942cf4b6f1e6a (commit) via 97c5e61b0a58a507a5ad5ee0559da9182596d287 (commit) via 9586e51efd243cce41a887448bf442f3f49d4c5f (commit) via 58e14c99f4c6b9be3a7b15c72d0f669b05d713cf (commit) via 4395a3d823a2243531ef15d05ef498427d9c409b (commit) via d4c79db2905f6380773adc8d12ddcddc5e21077e (commit) via a5f12d907cc6554b7c1a0e7c63b6a7287796f149 (commit) via 27883dd1891686ed50d9fff017b07416ef785cce (commit) via f9f6739cece7073aca938b5258ab7d590efd9f0f (commit) via d9e3579689f5678d761ac331c152d1286e8f94ef (commit) via c80f953d117a8d2406e37f200bf4aa62ef7d3cf2 (commit) via 70cb65941457b29bf811ee570b3cf21ae48e99bc (commit) via 3d53bfd7668f491f2904580c33f1a461fa631132 (commit) via e243b5c6b6183d911dc6a9a245e1c95a798ec280 (commit) via a63c177aa3ab50a7a00c04ac10ea282376ea4bc7 (commit) via 42acf2cd2cc4ebbd2611c67aeb0744778e0ba72a (commit) via 8fbfaa51bb3f583eeebc1239388832b2e966d424 (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 (50a88f8eb4ad183bfc1f870a99c7f0f1f2022acb) \ N -- N -- N (041995dacf26082cb4883a99f8d5a30c109efff7) 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=041995dacf26082cb4883a99f8d5a30c… commit 041995dacf26082cb4883a99f8d5a30c109efff7 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 26 22:19:22 2015 +0100 gtphub: use file for incremental restart counter. Force passing a restart counter, by adding such arg to gtphub_start(). In gtphub_main.c, add -r,--restart-file <path> and next_restart_count() to maintain the counter file. While at it, tweak the cmdline help to unify the formatting. http://cgit.osmocom.org/openbsc/commit/?id=ae1e7a00efcdea016008c54cd442e38c… commit ae1e7a00efcdea016008c54cd442e38c141e4200 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 26 05:20:18 2015 +0100 gtphub: complain about excess cmdline args. http://cgit.osmocom.org/openbsc/commit/?id=cf6ba14087de5ae464528280be3a784f… commit cf6ba14087de5ae464528280be3a784f8c6c6d6d Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 26 03:33:29 2015 +0100 gtphub: track tunnels explicitly. So far, gtphub worked perfectly by only tracking single TEIs ... for probably most uses. But a Ctrl plane tunnel may have expired despite a still active corresponding User plane tunnel. The User plane would continue to work indefinitely, but if any Ctrl messages followed after more than six hours of Ctrl silence, they would have been dropped due to an expired TEI mapping. We want to - combine expiry of a user TEI with its ctrl TEI. (done in this patch) - upon delete PDP context, remove both user and ctrl TEI mappings. (future) - when a peer indicates a restart counter bump, invalidate its tunnels. (future) To facilitate these, track tunnels, complete with both SGSN's and GGSN's address, original and replaced TEIs, all for both user and ctrl plane, in a single struct. A single expiry entry handles the entire tunnel, instead of previously four separate expiries for each endpoint identifier. Add the concept of a "side", being either GGSN or SGSN. Track the originating side in the gtp_packet_desc. For clarity, explicitly track the received header TEI in header_tei_rx, since set_tei() overwrites header_tei to reflect the packet's replaced header TEI, and the originally received header TEI still needs to be used for matching a Create PDP Context Response up with its Request (and for logging). Adjust the test suite to expect tunnel listing strings instead of TEI mappings, with a bonus of making it a lot easier to grok and including the IP addresses. Note: the current implementation is as slow as can possibly be, iterating all the tunnels all the time. Optimizations are kept for a future commit, on purpose. BTW, the sequence number mapping/unmapping structures remain unchanged. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=22174abe556b8419a42b3b67b191ac94… commit 22174abe556b8419a42b3b67b191ac94e30d81c5 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Wed Nov 25 16:50:19 2015 +0100 gtphub: fix filename mentioned in comment http://cgit.osmocom.org/openbsc/commit/?id=79dce146fbb4096f737776d5c217b71a… commit 79dce146fbb4096f737776d5c217b71a31b16eb9 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Wed Nov 25 16:45:59 2015 +0100 gtphub: cosmetic/prepare: rename expiry queues. The expiry queues are already used for resolved GGSN addresses, and will soon enlist tunnel structs. Hence the naming should be more general. http://cgit.osmocom.org/openbsc/commit/?id=c33ac58604665455699eddb2bad5cfd8… commit c33ac58604665455699eddb2bad5cfd8242f193c Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:32:23 2015 +0100 gtphub: add assertion to ensure expiry ordering. Make 100% sure the user adds expiring_items in chronological order by asserting that a newly added expiry is >= the last expiry in the queue. Add llist_last() to facilitate. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=310b8bf647c8b2478dec3a27c11942cf… commit 310b8bf647c8b2478dec3a27c11942cf4b6f1e6a Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:31:06 2015 +0100 gtphub: refresh TEI timeout when TEI is used. Make sure a TEI mapping remains valid when it is in use (so far all mappings would expire after 6 hours, in use or not). Add regression test for this. Note: this does not yet refresh a Ctrl TEI mapping when a corresponding User TEI is used (todo?). Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=97c5e61b0a58a507a5ad5ee0559da918… commit 97c5e61b0a58a507a5ad5ee0559da9182596d287 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:30:38 2015 +0100 gtphub: cosmetic/prepare: add nr_map_refresh(). http://cgit.osmocom.org/openbsc/commit/?id=9586e51efd243cce41a887448bf442f3… commit 9586e51efd243cce41a887448bf442f3f49d4c5f Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:27:13 2015 +0100 gtphub: move timestamp into packet struct. Instead of passing the current time around in function arguments ('now'), rather store the current time once upon decoding a GTP packet in the gtp_packet_desc passed around anyway ('p->timestamp'). Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=58e14c99f4c6b9be3a7b15c72d0f669b… commit 58e14c99f4c6b9be3a7b15c72d0f669b05d713cf Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:23:44 2015 +0100 gtphub: cosmetic: fix an argument name. http://cgit.osmocom.org/openbsc/commit/?id=4395a3d823a2243531ef15d05ef49842… commit 4395a3d823a2243531ef15d05ef498427d9c409b Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 03:16:19 2015 +0100 gtphub: first vty show commands. Start adding VTY commands to show rate counters / statistics / cache dumps. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=d4c79db2905f6380773adc8d12ddcddc… commit d4c79db2905f6380773adc8d12ddcddc5e21077e Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 01:28:40 2015 +0100 gtphub: add first rate counters Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=a5f12d907cc6554b7c1a0e7c63b6a728… commit a5f12d907cc6554b7c1a0e7c63b6a7287796f149 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 01:27:22 2015 +0100 gtphub: cosmetic: clarify bind pointer naming. Some gtphub_bind pointers point to an array of binds, some point directly at instances. Make the distinction between the two more obvious by adding an '_arr' suffix to the array ones. Partly in preparation for upcoming rate counters. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=27883dd1891686ed50d9fff017b07416… commit 27883dd1891686ed50d9fff017b07416ef785cce 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. (gtphub_stop() will probably never be called by anyone, but its existence completes the picture.) 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=f9f6739cece7073aca938b5258ab7d59… commit f9f6739cece7073aca938b5258ab7d590efd9f0f 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. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=d9e3579689f5678d761ac331c152d128… commit d9e3579689f5678d761ac331c152d1286e8f94ef Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 19 00:44:52 2015 +0100 gtphub: send gtphub's own restart counter, fix type. So far, the sender's restart counter was copied through, which would break as soon as more than one SGSN or more than one GGSN would talk to the same peer. Also fix the restart counter data type (just one octet after all). gtphub should also persist its restart counter and handle peers that have rebooted, both of which is still not implemented. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=c80f953d117a8d2406e37f200bf4aa62… commit c80f953d117a8d2406e37f200bf4aa62ef7d3cf2 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. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=70cb65941457b29bf811ee570b3cf21a… commit 70cb65941457b29bf811ee570b3cf21ae48e99bc 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=3d53bfd7668f491f2904580c33f1a461… commit 3d53bfd7668f491f2904580c33f1a461fa631132 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=e243b5c6b6183d911dc6a9a245e1c95a… commit e243b5c6b6183d911dc6a9a245e1c95a798ec280 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=a63c177aa3ab50a7a00c04ac10ea2823… commit a63c177aa3ab50a7a00c04ac10ea282376ea4bc7 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=42acf2cd2cc4ebbd2611c67aeb074477… commit 42acf2cd2cc4ebbd2611c67aeb0744778e0ba72a Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Wed Nov 18 17:09:22 2015 +0100 gtphub: add to debian build By the example of osmo-sgsn, package osmo-gtphub for debian. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=8fbfaa51bb3f583eeebc1239388832b2… commit 8fbfaa51bb3f583eeebc1239388832b2e966d424 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 26 22:20:50 2015 +0100 gtphub: remove another obsolete comment. ----------------------------------------------------------------------- Summary of changes: hooks/post-receive -- The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)

1 0

openbsc.git branch neels/gtphub updated. 0.15.0-76-g50a88f8
by gitosis＠osmocom.org 27 Nov '15

27 Nov '15

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 cb591d0e9175dad898de019612a68f9adfd25f4b (commit) discards 88d06caa712a3fd40960c0d11d45373dc9341659 (commit) discards 2d228573f2df28be5d185d8d93ec3b887f526eb7 (commit) discards f0dc0f3bd9d702c59d46cd358ba9915377136cfc (commit) discards ad51b759181064c87f26ed702c16d3edc3bc12ec (commit) discards e6b56c49fd36757aa507bbbef868cdc98d2cd3ca (commit) discards 5d2e92036d0bc6fbcab522e3eb3cb71cf62d80cc (commit) discards 628c0c8f82673390e92cc5f532f7f1e0d368c26c (commit) discards eb7b0a83b4555222215a0ec6155e0eecaadf3f24 (commit) discards 1c31e9a0793c90c856f6537d31b8f20e5d93ff4b (commit) discards e68fee7637ac953837adddb8c3428909a07ecfcb (commit) discards b30bfedb11e10ea7f99a5e98d5b8274cb866baa8 (commit) discards 5171ef9d2059625cf4d4c5775fe8e0c18ee5cd8b (commit) discards 25a88f932df6ea89d33da5ffafc883f735462636 (commit) discards 105b984f6af0907838529755efc6cf3ba3f5bad8 (commit) discards bbef1e556ad3d2968ad2baa9f41b4e932b0ec556 (commit) discards 8a9ebc14bfc2937d0402aa9f8cba82a323f4889a (commit) discards bf6ac16ae3f906accb7b68ed9fffd343bcc1f148 (commit) discards 5bdd48daaf45e824f3a683933617f41f65543d71 (commit) discards 47493b5d258e66dcc8760f477aa6ecfb9dcbd442 (commit) discards c0e13b2dbd36015f25bdeb8c07abb13ac4ef65ec (commit) via 50a88f8eb4ad183bfc1f870a99c7f0f1f2022acb (commit) via e201bf5778b073e7101a707435de0777a38b832f (commit) via 192cf8b56ace2ea168153571f9c5eae53fd47365 (commit) via 7f2d40ceb755943d52fcae8ddb12c09e2d0c623f (commit) via c1e297a9f030429a075392b4888d26539a333dbf (commit) via 444fe1be787fd291b5bc9c347303047e78089c35 (commit) via 3bf4d4ddd97383a98d3da56f3b57e7aac2adcf54 (commit) via da53b650561f5c2e81bd499657497713a7276725 (commit) via 036874d27b458d4359a9132e3361e47b42935117 (commit) via 3c4c17178f6d41ee83b441b9ce7d55d4e404b73d (commit) via 89a83524dc4dcf73bedd8c98659bfd006a422a3b (commit) via 5b3ed5a6cd4bc03b225e59910d647a5d4d21243a (commit) via 5b231c38318754de6b9b16e91d0f1dc29290872d (commit) via a0e2c562de54d87a48bdd059e18ef782f18c3cc8 (commit) via d369720ec8bda4f2fb3ee0b1a719211d66df247a (commit) via f397ca83bea6f15f194b522d9278695c7a27765f (commit) via 402b117fe1ec63fe9c30eaa38850a477601799e9 (commit) via fce0d1ee3b0717d18d081db9ceadbab2525441b8 (commit) via 786d555de00d65e6c0b07008fe1cc8f593118c2e (commit) via 42d114b846a4a1852f22ecfd56d8872cae83018a (commit) via 03203415845cd646c748983f8ab49ecb8ca0581a (commit) via fa74d6dd746e65166d4f7d62d7c6ff698bec2900 (commit) via 6de6ea937eae341577dedb93d95bd9ded7fd440f (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 (cb591d0e9175dad898de019612a68f9adfd25f4b) \ N -- N -- N (50a88f8eb4ad183bfc1f870a99c7f0f1f2022acb) 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=50a88f8eb4ad183bfc1f870a99c7f0f1… commit 50a88f8eb4ad183bfc1f870a99c7f0f1f2022acb Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 26 22:19:22 2015 +0100 gtphub: use file for incremental restart counter. Force passing a restart counter, by adding such arg to gtphub_start(). In gtphub_main.c, add -r,--restart-file <path> and next_restart_count() to maintain the counter file. While at it, tweak the cmdline help to unify the formatting. http://cgit.osmocom.org/openbsc/commit/?id=e201bf5778b073e7101a707435de0777… commit e201bf5778b073e7101a707435de0777a38b832f Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 26 05:20:18 2015 +0100 gtphub: complain about excess cmdline args. http://cgit.osmocom.org/openbsc/commit/?id=192cf8b56ace2ea168153571f9c5eae5… commit 192cf8b56ace2ea168153571f9c5eae53fd47365 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 26 03:33:29 2015 +0100 gtphub: track tunnels explicitly. So far, gtphub worked perfectly by only tracking single TEIs ... for probably most uses. But a Ctrl plane tunnel may have expired despite a still active corresponding User plane tunnel. The User plane would continue to work indefinitely, but if any Ctrl messages followed after more than six hours of Ctrl silence, they would have been dropped due to an expired TEI mapping. We want to - combine expiry of a user TEI with its ctrl TEI. (done in this patch) - upon delete PDP context, remove both user and ctrl TEI mappings. (future) - when a peer indicates a restart counter bump, invalidate its tunnels. (future) To facilitate these, track tunnels, complete with both SGSN's and GGSN's address, original and replaced TEIs, all for both user and ctrl plane, in a single struct. A single expiry entry handles the entire tunnel, instead of previously four separate expiries for each endpoint identifier. Add the concept of a "side", being either GGSN or SGSN. Track the originating side in the gtp_packet_desc. For clarity, explicitly track the received header TEI in header_tei_rx, since set_tei() overwrites header_tei to reflect the packet's replaced header TEI, and the originally received header TEI still needs to be used for matching a Create PDP Context Response up with its Request (and for logging). Adjust the test suite to expect tunnel listing strings instead of TEI mappings, with a bonus of making it a lot easier to grok and including the IP addresses. Note: the current implementation is as slow as can possibly be, iterating all the tunnels all the time. Optimizations are kept for a future commit, on purpose. BTW, the sequence number mapping/unmapping structures remain unchanged. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=7f2d40ceb755943d52fcae8ddb12c09e… commit 7f2d40ceb755943d52fcae8ddb12c09e2d0c623f Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Wed Nov 25 16:50:19 2015 +0100 gtphub: fix filename mentioned in comment http://cgit.osmocom.org/openbsc/commit/?id=c1e297a9f030429a075392b4888d2653… commit c1e297a9f030429a075392b4888d26539a333dbf Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Wed Nov 25 16:45:59 2015 +0100 gtphub: cosmetic/prepare: rename expiry queues. The expiry queues are already used for resolved GGSN addresses, and will soon enlist tunnel structs. Hence the naming should be more general. http://cgit.osmocom.org/openbsc/commit/?id=444fe1be787fd291b5bc9c347303047e… commit 444fe1be787fd291b5bc9c347303047e78089c35 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:32:23 2015 +0100 gtphub: add assertion to ensure expiry ordering. Make 100% sure the user adds expiring_items in chronological order by asserting that a newly added expiry is >= the last expiry in the queue. Add llist_last() to facilitate. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=3bf4d4ddd97383a98d3da56f3b57e7aa… commit 3bf4d4ddd97383a98d3da56f3b57e7aac2adcf54 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:31:06 2015 +0100 gtphub: refresh TEI timeout when TEI is used. Make sure a TEI mapping remains valid when it is in use (so far all mappings would expire after 6 hours, in use or not). Add regression test for this. Note: this does not yet refresh a Ctrl TEI mapping when a corresponding User TEI is used (todo?). Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=da53b650561f5c2e81bd499657497713… commit da53b650561f5c2e81bd499657497713a7276725 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:30:38 2015 +0100 gtphub: cosmetic/prepare: add nr_map_refresh(). http://cgit.osmocom.org/openbsc/commit/?id=036874d27b458d4359a9132e3361e47b… commit 036874d27b458d4359a9132e3361e47b42935117 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:27:13 2015 +0100 gtphub: move timestamp into packet struct. Instead of passing the current time around in function arguments ('now'), rather store the current time once upon decoding a GTP packet in the gtp_packet_desc passed around anyway ('p->timestamp'). Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=3c4c17178f6d41ee83b441b9ce7d55d4… commit 3c4c17178f6d41ee83b441b9ce7d55d4e404b73d Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Tue Nov 24 13:23:44 2015 +0100 gtphub: cosmetic: fix an argument name. http://cgit.osmocom.org/openbsc/commit/?id=89a83524dc4dcf73bedd8c98659bfd00… commit 89a83524dc4dcf73bedd8c98659bfd006a422a3b Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 03:16:19 2015 +0100 gtphub: first vty show commands. Start adding VTY commands to show rate counters / statistics / cache dumps. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=5b3ed5a6cd4bc03b225e59910d647a5d… commit 5b3ed5a6cd4bc03b225e59910d647a5d4d21243a Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 01:28:40 2015 +0100 gtphub: add first rate counters Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=5b231c38318754de6b9b16e91d0f1dc2… commit 5b231c38318754de6b9b16e91d0f1dc29290872d Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Fri Nov 20 01:27:22 2015 +0100 gtphub: cosmetic: clarify bind pointer naming. Some gtphub_bind pointers point to an array of binds, some point directly at instances. Make the distinction between the two more obvious by adding an '_arr' suffix to the array ones. Partly in preparation for upcoming rate counters. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=a0e2c562de54d87a48bdd059e18ef782… commit a0e2c562de54d87a48bdd059e18ef782f18c3cc8 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. (gtphub_stop() will probably never be called by anyone, but its existence completes the picture.) 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=d369720ec8bda4f2fb3ee0b1a719211d… commit d369720ec8bda4f2fb3ee0b1a719211d66df247a 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. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=f397ca83bea6f15f194b522d9278695c… commit f397ca83bea6f15f194b522d9278695c7a27765f Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 19 00:44:52 2015 +0100 gtphub: send gtphub's own restart counter, fix type. So far, the sender's restart counter was copied through, which would break as soon as more than one SGSN or more than one GGSN would talk to the same peer. Also fix the restart counter data type (just one octet after all). gtphub should also persist its restart counter and handle peers that have rebooted, both of which is still not implemented. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=402b117fe1ec63fe9c30eaa38850a477… commit 402b117fe1ec63fe9c30eaa38850a477601799e9 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. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=fce0d1ee3b0717d18d081db9ceadbab2… commit fce0d1ee3b0717d18d081db9ceadbab2525441b8 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=786d555de00d65e6c0b07008fe1cc8f5… commit 786d555de00d65e6c0b07008fe1cc8f593118c2e 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=42d114b846a4a1852f22ecfd56d8872c… commit 42d114b846a4a1852f22ecfd56d8872cae83018a 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=03203415845cd646c748983f8ab49ecb… commit 03203415845cd646c748983f8ab49ecb8ca0581a 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=fa74d6dd746e65166d4f7d62d7c6ff69… commit fa74d6dd746e65166d4f7d62d7c6ff698bec2900 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Wed Nov 18 17:09:22 2015 +0100 gtphub: add to debian build By the example of osmo-sgsn, package osmo-gtphub for debian. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openbsc/commit/?id=6de6ea937eae341577dedb93d95bd9de… commit 6de6ea937eae341577dedb93d95bd9ded7fd440f Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Thu Nov 26 22:20:50 2015 +0100 gtphub: remove obsolete comment. ----------------------------------------------------------------------- Summary of changes: openbsc/include/openbsc/gtphub.h | 3 +- openbsc/src/gprs/gtphub.c | 131 ++++++++++++++------------------------- openbsc/src/gprs/gtphub_main.c | 88 +++++++++++++++++++++++--- 3 files changed, 128 insertions(+), 94 deletions(-) hooks/post-receive -- The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)

1 0

libosmocore.git branch master updated. 0.9.0-27-gee702cd
by gitosis＠osmocom.org 26 Nov '15

26 Nov '15

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 "An utility library for Open Source Mobile Communications". The branch, master has been updated via ee702cd590688c1e65296319c5bc4c9cf160ba65 (commit) via f13de868be052f96325c337f25453f1d97777005 (commit) via 46b703d083a57d7718340f8f71da29d96ce15170 (commit) via 2e8f9ed5560ac6286b1fe70ade868569d30cb392 (commit) via b6e6bea4ecea4b0769fc56136321f01f553459ba (commit) from 5fa0dcc2750a79920d4a6404e960c50667840058 (commit) 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/libosmocore/commit/?id=ee702cd590688c1e65296319c5bc… commit ee702cd590688c1e65296319c5bc4c9cf160ba65 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Tue Nov 10 11:38:25 2015 +0100 stats: Add TODO comment to rate_ctr Currently the counters are scanned twice, once for interval computation and once for reporting. This adds a reminder to move the interval computation code to a special stats reporter which just shall update the fields. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/libosmocore/commit/?id=f13de868be052f96325c337f2545… commit f13de868be052f96325c337f25453f1d97777005 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Tue Nov 10 11:36:58 2015 +0100 stats/test: Add memory leak check Adds a rudimentary leak check for the counters and stat items. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/libosmocore/commit/?id=46b703d083a57d7718340f8f71da… commit 46b703d083a57d7718340f8f71da29d96ce15170 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Mon Nov 9 17:25:27 2015 +0100 stats/test: Add test for reporting This tests uses a dedicated test reported to check several aspects of the value reporting. - addition/removal of stats reporter - addition/removal of counters/items - setting of max_class - initial value flush - updating single counters/items - reporter retrieval - enable/disable Sponsored-by: On-Waves ehf http://cgit.osmocom.org/libosmocore/commit/?id=2e8f9ed5560ac6286b1fe70ade86… commit 2e8f9ed5560ac6286b1fe70ade868569d30cb392 Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Mon Nov 9 15:48:25 2015 +0100 stats: Reorder functions in stats.c Due to prior refactoring, the functions do not have an sensible order in the file. This commit tries to improve that a little bit. Sponsored-by: On-Waves ehf http://cgit.osmocom.org/libosmocore/commit/?id=b6e6bea4ecea4b0769fc56136321… commit b6e6bea4ecea4b0769fc56136321f01f553459ba Author: Jacob Erlbeck <jerlbeck(a)sysmocom.de> Date: Mon Nov 9 15:33:44 2015 +0100 stats: Move statsd related code into a separate file This commit moves the stats specific code parts into stats_statsd.c while keeping the generic parts in stats.c. The code in stats.c no longer contains references to statsd symbols. Note that the VTY code still needs to know about every stats reporter backend. Sponsored-by: On-Waves ehf ----------------------------------------------------------------------- Summary of changes: include/osmocom/core/stats.h | 16 ++- src/Makefile.am | 2 +- src/rate_ctr.c | 3 + src/stats.c | 256 ++++++++++---------------------------- src/stats_statsd.c | 158 ++++++++++++++++++++++++ tests/stats/stats_test.c | 283 ++++++++++++++++++++++++++++++++++++++++--- tests/stats/stats_test.ok | 108 +++++++++++++++++ 7 files changed, 612 insertions(+), 214 deletions(-) create mode 100644 src/stats_statsd.c hooks/post-receive -- An utility library for Open Source Mobile Communications

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

osmocom-commitlog