Change in osmo-sgsn[master]: gbproxy: Pass TLLI as LSP towards NS to facilitate load sharing

[laforge]

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/21228 )

Change subject: gbproxy: Pass TLLI as LSP towards NS to facilitate load sharing
......................................................................

gbproxy: Pass TLLI as LSP towards NS to facilitate load sharing

Change-Id: I6aef26c126b330a393fc2be5f558b6d5f1d9f7f4
---
M src/gbproxy/gb_proxy.c
1 file changed, 9 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  pespin: Looks good to me, but someone else must approve
  lynxis lazus: Looks good to me, approved



diff --git a/src/gbproxy/gb_proxy.c b/src/gbproxy/gb_proxy.c
index 39b9ffd..9009651 100644
--- a/src/gbproxy/gb_proxy.c
+++ b/src/gbproxy/gb_proxy.c
@@ -811,6 +811,7 @@
 	struct gprs_ns2_inst *nsi = peer->cfg->nsi;
 	struct osmo_gprs_ns2_prim nsp = {};
 	struct msgb *msg = bssgp_msgb_copy(old_msg, "msgb_relay2peer");
+	uint32_t tlli;
 	int rc;
 
 	DEBUGP(DGPRS, "NSEI=%u proxying SGSN->BSS (NS_BVCI=%u, NSEI=%u)\n",
@@ -822,6 +823,14 @@
 	/* Strip the old NS header, it will be replaced with a new one */
 	strip_ns_hdr(msg);
 
+	/* TS 48.018 Section 5.4.2: The link selector parameter is
+	 * defined in 3GPP TS 48.016. At one side of the Gb interface,
+	 * all BSSGP UNITDATA PDUs related to an MS shall be passed with
+	 * the same LSP, e.g. the LSP contains the MS's TLLI, to the
+	 * underlying network service. */
+	if (gprs_gb_parse_tlli(msgb_data(msg), msgb_length(msg), &tlli) == 1)
+		nsp.u.unitdata.link_selector = tlli;
+
 	osmo_prim_init(&nsp.oph, SAP_NS, PRIM_NS_UNIT_DATA,
 		       PRIM_OP_REQUEST, msg);
 	rc = gprs_ns2_recv_prim(nsi, &nsp.oph);

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/21228
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I6aef26c126b330a393fc2be5f558b6d5f1d9f7f4
Gerrit-Change-Number: 21228
Gerrit-PatchSet: 6
Gerrit-Owner: laforge <laforge at osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis at fe80.eu>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-CC: daniel <dwillmann at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20201130/49366133/attachment.htm>