diff options
| author | Neels Hofmeyr <nhofmeyr@sysmocom.de> | 2015-12-06 19:11:45 +0100 |
|---|---|---|
| committer | Neels Hofmeyr <nhofmeyr@sysmocom.de> | 2015-12-07 13:37:11 +0100 |
| commit | ee07e4f75d1fc31dd6326e50535441a391c7de1e (patch) | |
| tree | 850f8eb1880d7e237bdd1411e7d48d099f23595f /openbsc/include/openbsc | |
| parent | f6e4d082194e41c8a5b51a46c578921006496309 (diff) | |
gtphub: simplify/fix: one TEI mapping per tunnel.
Because the sender is known, one unique TEI per tunnel suffices to map the TEIs
that the peers are sending to gtphub, instead of previously 4 (SGSN<->GGSN
interaction on User and Ctrl plane, where each had an own unique TEI).
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.
Thus simplify TEI reuse detection and improve VTY show readability and
debugging.
Adjust log and VTY output for tunnels.
Adjust tests accordingly.
Suggested-by: Holger Hans Peter Freyther <holger@moiji-mobile.com>
Sponsored-by: On-Waves ehi
Diffstat (limited to 'openbsc/include/openbsc')
| -rw-r--r-- | openbsc/include/openbsc/gtphub.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/openbsc/include/openbsc/gtphub.h b/openbsc/include/openbsc/gtphub.h index 97cfc60e9..9cb7605f8 100644 --- a/openbsc/include/openbsc/gtphub.h +++ b/openbsc/include/openbsc/gtphub.h @@ -398,7 +398,6 @@ struct gtphub_peer_port { struct gtphub_tunnel_endpoint { struct gtphub_peer_port *peer; uint32_t tei_orig; /* from/to peer */ - uint32_t tei_repl; /* from/to the other tunnel endpoint */ struct rate_ctr_group *counters_io; }; @@ -407,6 +406,7 @@ struct gtphub_tunnel { struct llist_head entry; struct expiring_item expiry_entry; + uint32_t tei_repl; /* unique TEI to replace peers' TEIs */ struct gtphub_tunnel_endpoint endpoint[GTPH_SIDE_N][GTPH_PLANE_N]; }; |