6 Dec
2015
6 Dec
'15
11:27 p.m.
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 e54f092e2356de924b5711e8f6396f0b031816e2 (commit)
discards 78c88ff85f038dc27ba56823b27b7b1a130cfb24 (commit)
discards b9790de60290cd63aca7b4d31cdf54eabf602219 (commit)
discards 520e79c9254643bf04e50cf2426a44300201473e (commit)
discards 483c1e846a6781dfa16fd4e4d6c9f48ee4ed7ffe (commit)
discards 4301e3f2886b26dc42aa41bbc3aa8a5932d0dd38 (commit)
discards bd6351d16b8e08cd35a21d9ac6ac3b33fac1e0bd (commit)
discards 4613c6901300f316e28ebbba432f3a598967a306 (commit)
discards 807b757720958886dbc07448e1b6bf3016f364b1 (commit)
discards 6927cbcf7af1fc777f79de2d30ec8fda6745cfc6 (commit)
discards ff03117fcfbe6d187dc61e326553f1c886aaae8d (commit)
discards 9cc8eaed10e50db92de858c82bf400637e468fe8 (commit)
discards f03f63a185b6a3e941f63f4c0acc644ec6c9c24c (commit)
via 15ef85da763e71a0d3d206e88618bffbcc732527 (commit)
via 66e7b5bff16e65cac8efd6ee3a5375e3d5033e09 (commit)
via 70062d5353ed729876545dd6dcd51a15dbba816e (commit)
via 7fa6c3e4225134302b1e4c31077d1d15bd48ea65 (commit)
via 186ec1bac98d555b1c845a5ab76aa1a969c39005 (commit)
via b63e8997929b3357fcf414806e2f9c0778004aff (commit)
via 47e6e204c2c185d5b66996f74922707f61710093 (commit)
via 7c23a8d0f7bfaea211f4302a2f9dec50d51a278b (commit)
via 5d09e1e9562bd0815d1e94345eb4215c0877033b (commit)
via a7f19ce6780a506b2b53814d0aa1edecb01dd197 (commit)
via 668cffedb6700c898f761b5ec3415d95758ac58a (commit)
via 3830117081054865c9bcf976c5de1a99f1981663 (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 (e54f092e2356de924b5711e8f6396f0b031816e2)
\
N -- N -- N (15ef85da763e71a0d3d206e88618bffbcc732527)
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=15ef85da763e71a0d3d206e88618bffb…
commit 15ef85da763e71a0d3d206e88618bffbcc732527
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Dec 6 23:12:02 2015 +0100
gtphub: improve handling of restarted peer.
Handle peer restart earlier, so that the tunnels are deleted due to peer
restart instead of the first one being deleted due to reused TEI. That caused
confusing logging messages.
Also, when receiving Delete confirmations from the peer that didn't restart,
don't complain about unknown peer, but acknowledge and remove the half
invalidated tunnel.
The test test_peer_restarted_reusing_tei() hits the situation where a tunnel is
removed because of a reused TEI rather than the restart counter. Adjust the
test to expect the "out-of-band" delete request earlier on, and to still
see
the half invalidated tunnel around. Enhance the test by adding the delete
response from the peer that didn't restart, and add a final tunnels_are()
verification.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=66e7b5bff16e65cac8efd6ee3a5375e3…
commit 66e7b5bff16e65cac8efd6ee3a5375e3d5033e09
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Dec 6 23:07:02 2015 +0100
gtphub: log most common message type names.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=70062d5353ed729876545dd6dcd51a15…
commit 70062d5353ed729876545dd6dcd51a15dbba816e
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Dec 6 19:11:45 2015 +0100
gtphub: simplify/fix: one TEI mapping per tunnel.
One unique TEI per tunnel (SGSN<->GGSN interaction on User and Ctrl plane)
suffices to map the TEIs that the peers are sending to gtphub.
This simplifies TEI reuse detection and improves VTY show readability and
debugging.
Also, previously, a tunnel's endpoints should also have been checked against
each other for TEI reuse, not only against the endpoints of other tunnels. This
simplification fixes that problem for free.
Adjust log and VTY output for tunnels.
Adjust tests accordingly.
Suggested-by: Holger Hans Peter Freyther <holger(a)moiji-mobile.com>
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=7fa6c3e4225134302b1e4c31077d1d15…
commit 7fa6c3e4225134302b1e4c31077d1d15bd48ea65
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Dec 6 19:03:35 2015 +0100
gtphub: fix: when checking TEIs, skip 0.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=186ec1bac98d555b1c845a5ab76aa1a9…
commit 186ec1bac98d555b1c845a5ab76aa1a969c39005
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Dec 6 19:13:21 2015 +0100
gtphub: log: add TEI reuse msg, fix another TEI msg.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=b63e8997929b3357fcf414806e2f9c07…
commit b63e8997929b3357fcf414806e2f9c0778004aff
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Dec 6 19:02:43 2015 +0100
gtphub_test: add test_parallel_context_creation()
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=47e6e204c2c185d5b66996f74922707f…
commit 47e6e204c2c185d5b66996f74922707f61710093
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Dec 6 17:18:49 2015 +0100
gtphub: log: add side str to msg for reused TEI
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=7c23a8d0f7bfaea211f4302a2f9dec50…
commit 7c23a8d0f7bfaea211f4302a2f9dec50d51a278b
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Dec 6 16:44:14 2015 +0100
gtphub: add more detailed I/O rate counters.
Count bytes and packets per peer port, as well es per tunnel enpoint, which
adds two more levels of detail.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=5d09e1e9562bd0815d1e94345eb4215c…
commit 5d09e1e9562bd0815d1e94345eb4215c0877033b
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Dec 6 16:40:24 2015 +0100
gtphub: be strict about unknown cmdline args
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=a7f19ce6780a506b2b53814d0aa1edec…
commit a7f19ce6780a506b2b53814d0aa1edecb01dd197
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Dec 6 16:39:23 2015 +0100
gtphub: tweak logging.
Less spaces in tunnel strings, adjust tests accordingly.
Use side_idx to remove code dup in rate counter output.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=668cffedb6700c898f761b5ec3415d95…
commit 668cffedb6700c898f761b5ec3415d95758ac58a
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Sun Dec 6 15:22:54 2015 +0100
gtphub: logging: have one newline per log.
Some logging was multiline to ease human reading of debug output. However,
in the VTY output, these newlines lack a CR motion. Split multiline logs into
separate lines.
Also add one missing space.
Sponsored-by: On-Waves ehi
http://cgit.osmocom.org/openbsc/commit/?id=3830117081054865c9bcf976c5de1a99…
commit 3830117081054865c9bcf976c5de1a99f1981663
Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de>
Date: Thu Dec 3 14:52:33 2015 +0100
gtphub: fix missing newline in log output
Sponsored-by: On-Waves ehi
-----------------------------------------------------------------------
Summary of changes:
openbsc/src/gprs/gtphub.c | 92 ++++++++++++++++++++++++++++++-------
openbsc/src/gprs/gtphub_vty.c | 8 ++--
openbsc/tests/gtphub/gtphub_test.c | 50 +++++++++++++++++---
openbsc/tests/gtphub/gtphub_test.ok | 7 ++-
4 files changed, 127 insertions(+), 30 deletions(-)
hooks/post-receive
--
The OpenBSC GSM Base Station Controller (+MSC/HLR/SGSN)