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 dac1a00b4cf02f2627662f9ad0e61b7dc1892e17 (commit)
discards 29c4c4999932990dfa61fe3e79802a52aef4264f (commit)
discards 02d6365ef16bfcd838d792a4797409d84f104f59 (commit)
discards e43c8c949bee886947e6759eaac5aa7c5bd453c9 (commit)
discards 5b319dd18936e75a5448a02833cc75d2bbf3a098 (commit)
discards 00362b567db211024f793d0fa850ee5736055a2b (commit)
discards b6cb4e695ecb303ce446769bdea1712a7d2ae329 (commit)
discards 33217200aa2536ecdb7e986a071134232ab9f0f6 (commit)
discards aee727f5e5b6e9877621407b93f81bab0526a614 (commit)
discards 887ed65c1828279c359cf261644d438d32c09b25 (commit)
discards 942e425d8264df165154905e6e6c7ceb04b9d667 (commit)
discards a1747a446e44ea30f12f0a4bdd1d03ee89ee106c (commit)
discards 5cfa3fcdfcb789ab9216445e2d649400b1f5d635 (commit)
discards 1676898db0638f896f963b43c7b6ce884a787706 (commit)
discards 38d337d9408623d93af46b4ec2dc2411ec7d1ca4 (commit)
discards 54520814ff53b3eadc09a484fa3e61d4a445455d (commit)
discards fc97e4549562e283d005bcb145265d585e667e3a (commit)
discards 7bc3c88667c1e96ad46f2d12d02d5d7883e281ff (commit)
discards d156ccf9ff5f29c8dc131dccd6beb0db3e207497 (commit)
discards ce4735c7fb8c8a1733028c1fd0f429c0f0af365c (commit)
discards 211b3362635d1a085c3eff05f4836f06f252b606 (commit)
discards 107cfad51f1740669c694629c0d1981538d5cf94 (commit)
discards d5662d2decd46fbc2a38130df9bdd8a485543d1a (commit)
discards 519009ecaacffbcc5d8a56690fcc7d1d810e8488 (commit)
discards ef97389fefe87d8107b4fc0cf7b34fb83cdbb3ea (commit)
discards b079b9923a812476af6ec6185c9932ea92363151 (commit)
discards a6a173ff84cf6112fc10bbbb41d13df504793942 (commit)
discards 5bab5c395c48b4dee5374d42ea6c7b895360b296 (commit)
discards cd711490c2df8f87e49adc6b82003cd915ed8d28 (commit)
discards da979ef8bbacaf1641cbf40e841a05f1e5421cba (commit)
discards 173c36b20b3163817d519aa21428448f5b16618d (commit)
discards 4defa42cf51330c087f506587f4171c3c75dc53f (commit)
discards e36599fecd68ffabd3b5d7e68aa34d48d829f93b (commit)
discards dff8c375f07b49e6cdf36352fa0d2c0a69c3faaf (commit)
discards fc78950790586b66c6f87b92603039b8703ab6a7 (commit)
discards 54bb59814699042da2c294da8eddd012c52598f8 (commit)
discards 6357d8e218408bdd2b4df861fae88e8bdeea244e (commit)
discards 9e16284e3307f0689ff1b41554da1820f1584355 (commit)
discards e82037a0873be819a65550c7948182dd3685f375 (commit)
discards 4d7dd42d9714e9c4a6aaf7ac010453f93f80cd71 (commit)
discards 91d1bb23fa35d175969382a573c7470e38c86041 (commit)
discards fa0fd33199eeba9e69c165aa5ff6f718fefab5a0 (commit)
discards 2586ba4045fa84fc0daf6ac444a757adeb033d5c (commit)
discards 3643ee36f32ca326881d78fc55a30cd6bfabb261 (commit)
discards 1d19d62d810d284a883f42814a5c389652bb2950 (commit)
via 8defadbdd2953482ad53fe9ef37bb81ff76a2097 (commit)
via 59c1b645e2d86457ee1c9df56860aec8ee5a606f (commit)
via 87c83d0b895624ac3968a85edc9de18e36d5d1ea (commit)
via d8660ef090b66e7fbbb960ce8a8c8c26f6981c6f (commit)
via c6d51f5fb477c9ebf0345edb088b86d70eb119a1 (commit)
via 996ec1d73150366fa105b7d42ba8003fb018fcf8 (commit)
via 18d304961249c7815cf3a78ebd1697a4efd9ab42 (commit)
via 237fee649e691fc3a7ef851731305b8a41921436 (commit)
via 936b890f71f6603bdd95fc6283030395743dfae5 (commit)
via be4bebaad05dd360200c38b7da02aa8ca66850f2 (commit)
via 4573ca88345c553653eeea627cf4b51fa26ab9df (commit)
via 52c0bd3025f319597ce40621ef44e39e01594605 (commit)
via bc44330671c019be92b335149a2aefa3cc08b8f7 (commit)
via 3fdba2ed5ab7ad6e0814aac37f440ea1e72c906b (commit)
via 7559910e8ced14f718820d8bfeb419878185d153 (commit)
via 99a50b3529a53429343f767023fe9ca42b61a90f (commit)
via 10fc02447279a322e1d362a31dec5c8f8bafb6f8 (commit)
via 2f67125c3237f63800c5601f1aaddda62ce192a2 (commit)
via 800126b1f33986ca9ffc6f2a65c91264a969d078 (commit)
via cd865d62f0446135258189184664a146b28db975 (commit)
via 085500807c3a66e4d6e0a6a2d6516b2a4a13e69c (commit)
via ff4b630de9b8cb760e3d30861fda807f708351d9 (commit)
via 817bc32696422ac4662b4b706c6e03961e45257b (commit)
via a9905a51f96acdddf761d0b3e4c21ac0711b57ae (commit)
via 7174b169e0346d2a44ec50b309bd1e332503c825 (commit)
via d121ea612460b8a6f78a3bc91ab164651240f48a (commit)
via f977320736fe15ffc3a0e6110bec7823a57c65ee (commit)
via ba9e9f63bce9a868b8b3e15adca616ac4dec9414 (commit)
via 8d1ffbd3ba44939612d453b161dd86ba8302d0c1 (commit)
via 005f175c3b9a3182d429de9177438564b8a7e4f6 (commit)
via fc1be3a0c1cb7a220c174c7d7d3d4bd1cd4dedf4 (commit)
via e54cd1555a874b132116b1269264de7d6cc3d24d (commit)
via 2c8b58139f3e3c29309dd6518f77a62414de560c (commit)
via 1aa0e47104d9529ecf448b81adb9ef6fb0a50c5f (commit)
via 508514c7c5a4cc2d4eae27a7e6df80b10ec4e863 (commit)
via 29d926be12fccfb2a0240535d252b7f7d8452df8 (commit)
via 231653a8d076c7c793a809de7b953c19a5e91284 (commit)
via 4b2cbdab3e29dfbd8c4b5f9d44a9a63e7553253d (commit)
via 1ba50c659801fd8ba811dd898236134c0f0a7011 (commit)
via dba6d1af697a027e9529b61925a6c78603b552a7 (commit)
via 20bd6bfef5417f540e7fdd624832d1f31e38fe42 (commit)
via 1ed9a8673d67f4224a0bdbebae012c855c5d7bdc (commit)
via b6c2db569f03bf5eb0f9a12a9bca5c0d7637beca (commit)
via d9b1d49485195a79095bae162f8bd8d0abc04781 (commit)
via 4960fab767084f9d5937960f68b84c801350f74c (commit)
via e2ed8e6cc3eef9f93d8603634072720d20821ebb (commit)
via 334af5dd9dcee962a25ca9cca9425da5d7f57a2c (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 (dac1a00b4cf02f2627662f9ad0e61b7dc1892e17)
\
N -- N -- N (8defadbdd2953482ad53fe9ef37bb81ff76a2097)
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=8defadbdd2953482ad53fe9ef37bb81f…
commit 8defadbdd2953482ad53fe9ef37bb81ff76a2097
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Nov 29 19:14:58 2015 +0100
tests: fix condition to run sgsn, oap, gtphub tests
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=59c1b645e2d86457ee1c9df56860aec8…
commit 59c1b645e2d86457ee1c9df56860aec8ee5a606f
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Thu Dec 3 11:30:43 2015 +0100
gtphub: check TEI presence in Create PDP Ctx Response
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=87c83d0b895624ac3968a85edc9de18e…
commit 87c83d0b895624ac3968a85edc9de18e36d5d1ea
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Thu Dec 3 11:25:27 2015 +0100
gtphub: improve logging for invalid packet
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=d8660ef090b66e7fbbb960ce8a8c8c26…
commit d8660ef090b66e7fbbb960ce8a8c8c26f6981c6f
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Thu Dec 3 11:24:39 2015 +0100
gtphub: cosmetic
During the peer review session with Holger, these things were deemed fixable.
No need to have a static gtp_packet_desc in gtphub_handle_buf.
No need to memcpy, direct assignment does the job.
Remove obsolete comments.
Fix a stray space.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=c6d51f5fb477c9ebf0345edb088b86d7…
commit c6d51f5fb477c9ebf0345edb088b86d70eb119a1
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Wed Dec 2 15:44:34 2015 +0100
gtphub: fix restart cleanup peer matching.
Adjust test expectations accordingly.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=996ec1d73150366fa105b7d42ba8003f…
commit 996ec1d73150366fa105b7d42ba8003fb018fcf8
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Wed Dec 2 15:43:10 2015 +0100
gtphub: wrap gtphub_write() for test suite.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=18d304961249c7815cf3a78ebd1697a4…
commit 18d304961249c7815cf3a78ebd1697a4efd9ab42
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Wed Dec 2 15:00:50 2015 +0100
gtphub: fix use after free.
A tunnel pointer was still being checked after deleting by a call to
expiring_item_del(). 'continue' to the next tun.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=237fee649e691fc3a7ef851731305b8a…
commit 237fee649e691fc3a7ef851731305b8a41921436
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Wed Dec 2 14:31:45 2015 +0100
gtphub: add restart counter test
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=936b890f71f6603bdd95fc6283030395…
commit 936b890f71f6603bdd95fc6283030395743dfae5
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Wed Dec 2 14:31:08 2015 +0100
gtphub: Add a debug log
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=be4bebaad05dd360200c38b7da02aa8c…
commit be4bebaad05dd360200c38b7da02aa8ca66850f2
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Wed Dec 2 14:18:26 2015 +0100
gtphub: add test for reused TEI.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=4573ca88345c553653eeea627cf4b51f…
commit 4573ca88345c553653eeea627cf4b51fa26ab9df
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Wed Dec 2 14:15:36 2015 +0100
gtphub: fix: add a missing NULL check
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=52c0bd3025f319597ce40621ef44e39e…
commit 52c0bd3025f319597ce40621ef44e39e01594605
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Wed Dec 2 14:14:32 2015 +0100
gtphub: cosmetic: early continue for less indent
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=bc44330671c019be92b335149a2aefa3…
commit bc44330671c019be92b335149a2aefa3cc08b8f7
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Tue Dec 1 15:20:18 2015 +0100
gtphub: monitor GSNs' restart counters.
If a GSN indicates that it has reset, tear down each known tunnel for that GSN
individually (don't send the GSNs on the other side a different restart
counter, because they represent more than just this GSN).
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=3fdba2ed5ab7ad6e0814aac37f440ea1…
commit 3fdba2ed5ab7ad6e0814aac37f440ea1e72c906b
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Mon Nov 30 14:17:21 2015 +0100
gtphub: cosmetic
Move the SGSN read callback next to the GGSN read callback.
Comments.
Tweak an initialization to NULL.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=7559910e8ced14f718820d8bfeb41987…
commit 7559910e8ced14f718820d8bfeb419878185d153
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Tue Dec 1 01:01:16 2015 +0100
gtphub_test: add test for Del PDP from GGSN side.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=99a50b3529a53429343f767023fe9ca4…
commit 99a50b3529a53429343f767023fe9ca42b61a90f
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Wed Dec 2 01:15:30 2015 +0100
gtphub_test: add some cleanup asserts.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=10fc02447279a322e1d362a31dec5c8f…
commit 10fc02447279a322e1d362a31dec5c8f8bafb6f8
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Tue Dec 1 00:23:45 2015 +0100
gtphub: handle Delete PDP Context.
During resolution of the header TEI, also return the tunnel struct that
resolved the TEI, so the Delete PDP Ctx code does not need to look it up
again.
Upon Delete PDP Ctx Request, remember the IEs and that a request was made.
Upon Delete PDP Ctx Response, find the pending delete and remove the
corresponding tunnel, iff the response indicates success.
Add a context deletion to regression tests, rename the test appropriately.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=2f67125c3237f63800c5601f1aaddda6…
commit 2f67125c3237f63800c5601f1aaddda62ce192a2
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Mon Nov 30 14:13:33 2015 +0100
gtphub: remove obsolete todo comment
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=800126b1f33986ca9ffc6f2a65c91264…
commit 800126b1f33986ca9ffc6f2a65c91264a969d078
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Mon Nov 30 14:13:19 2015 +0100
gtphub: fix segfault when empty config.
gsn_addr_from_str(): return error upon NULL string.
Add some debug logging.
With an empty config, no bind addresses were set, and the address parser
did not check for a NULL pointer, resulting in a segfault.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=cd865d62f0446135258189184664a146…
commit cd865d62f0446135258189184664a146b28db975
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Mon Nov 30 12:58:48 2015 +0100
gtphub: be more fatal about not finding an unused TEI.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=085500807c3a66e4d6e0a6a2d6516b2a…
commit 085500807c3a66e4d6e0a6a2d6516b2a4a13e69c
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Mon Nov 30 12:19:50 2015 +0100
gtphub: fix some style complaints from cppcheck
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=ff4b630de9b8cb760e3d30861fda807f…
commit ff4b630de9b8cb760e3d30861fda807f708351d9
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Mon Nov 30 00:07:02 2015 +0100
gtphub: add two stubs for Delete PDP msgs
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=817bc32696422ac4662b4b706c6e0396…
commit 817bc32696422ac4662b4b706c6e03961e45257b
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Nov 29 23:50:45 2015 +0100
gtphub: vty: add missing SGSN-proxy output.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=a9905a51f96acdddf761d0b3e4c21ac0…
commit a9905a51f96acdddf761d0b3e4c21ac0711b57ae
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Nov 29 23:49:48 2015 +0100
gtphub: refactor: use side_idx everywhere.
This is a mostly cosmetic change. Instead of separate buffer handling
functions, reduce some code duplication by using a side_idx just like the
plane_idx, with arrays.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=7174b169e0346d2a44ec50b309bd1e33…
commit 7174b169e0346d2a44ec50b309bd1e332503c825
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Fri Nov 27 01:22:13 2015 +0100
gtphub: make sure mapped TEIs aren't occupied (incomplete).
This could be done way better, discussion is pending/ongoing. It is indeed
quite unlikely that any user will ever hit this situation, so there is no
strong drive to invest effort in a more comprehensive implementation.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=d121ea612460b8a6f78a3bc91ab16465…
commit d121ea612460b8a6f78a3bc91ab164651240f48a
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Fri Nov 27 01:20:53 2015 +0100
gtphub: use a single TEI pool across planes.
There's no need to keep two separate number pools when both can be fed
from the same pool. User and Ctrl plane TEIs can technically overlap without
colliding, but it doesn't hurt if they don't overlap, either.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=f977320736fe15ffc3a0e6110bec7823…
commit f977320736fe15ffc3a0e6110bec7823a57c65ee
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Fri Nov 27 00:05:56 2015 +0100
gtphub: cosmetic: for_each_side,_plane macros.
Simplify looping over sides and planes. I'm tired of typing the same for
loops all the time.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=ba9e9f63bce9a868b8b3e15adca616ac…
commit ba9e9f63bce9a868b8b3e15adca616ac4dec9414
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=8d1ffbd3ba44939612d453b161dd86ba…
commit 8d1ffbd3ba44939612d453b161dd86ba8302d0c1
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Thu Nov 26 05:20:18 2015 +0100
gtphub: complain about excess cmdline args.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=005f175c3b9a3182d429de9177438564…
commit 005f175c3b9a3182d429de9177438564b8a7e4f6
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Mon Nov 30 12:19:11 2015 +0100
gtphub: fix a conditional for log output
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=fc1be3a0c1cb7a220c174c7d7d3d4bd1…
commit fc1be3a0c1cb7a220c174c7d7d3d4bd1cd4dedf4
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Wed Dec 2 00:31:31 2015 +0100
gtphub: avoid segfault for incomplete tunnels.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=e54cd1555a874b132116b1269264de7d…
commit e54cd1555a874b132116b1269264de7d6cc3d24d
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=2c8b58139f3e3c29309dd6518f77a624…
commit 2c8b58139f3e3c29309dd6518f77a62414de560c
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.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=1aa0e47104d9529ecf448b81adb9ef6f…
commit 1aa0e47104d9529ecf448b81adb9ef6fb0a50c5f
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=508514c7c5a4cc2d4eae27a7e6df80b1…
commit 508514c7c5a4cc2d4eae27a7e6df80b10ec4e863
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Tue Nov 24 13:30:38 2015 +0100
gtphub: cosmetic/prepare: add nr_map_refresh().
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=29d926be12fccfb2a0240535d252b7f7…
commit 29d926be12fccfb2a0240535d252b7f7d8452df8
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=231653a8d076c7c793a809de7b953c19…
commit 231653a8d076c7c793a809de7b953c19a5e91284
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Tue Nov 24 13:23:44 2015 +0100
gtphub: cosmetic: fix an argument name.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=4b2cbdab3e29dfbd8c4b5f9d44a9a63e…
commit 4b2cbdab3e29dfbd8c4b5f9d44a9a63e7553253d
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=1ba50c659801fd8ba811dd898236134c…
commit 1ba50c659801fd8ba811dd898236134c0f0a7011
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=dba6d1af697a027e9529b61925a6c786…
commit dba6d1af697a027e9529b61925a6c78603b552a7
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=20bd6bfef5417f540e7fdd624832d1f3…
commit 20bd6bfef5417f540e7fdd624832d1f31e38fe42
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=1ed9a8673d67f4224a0bdbebae012c85…
commit 1ed9a8673d67f4224a0bdbebae012c855c5d7bdc
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=b6c2db569f03bf5eb0f9a12a9bca5c0d…
commit b6c2db569f03bf5eb0f9a12a9bca5c0d7637beca
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=d9b1d49485195a79095bae162f8bd8d0…
commit d9b1d49485195a79095bae162f8bd8d0abc04781
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=4960fab767084f9d5937960f68b84c80…
commit 4960fab767084f9d5937960f68b84c801350f74c
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=e2ed8e6cc3eef9f93d8603634072720d…
commit e2ed8e6cc3eef9f93d8603634072720d20821ebb
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=334af5dd9dcee962a25ca9cca9425da5…
commit 334af5dd9dcee962a25ca9cca9425da5d7f57a2c
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
-----------------------------------------------------------------------
Summary of changes:
openbsc/src/gprs/gtphub.c | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
hooks/post-receive
--
The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)