laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-sgsn/+/21613 )
Change subject: gbproxy: Use "(nsei << 16) | bvci" as rate_ctr_group index
......................................................................
gbproxy: Use "(nsei << 16) | bvci" as rate_ctr_group index
As we now have gbproxy_bvc on both the SGSN and the BSS side
with the same BVCI, using the BVCI alone will no longer render
unique indexes.
Related: OS#4472
Change-Id: I13f3c9e69562a56ad7d3742fdeb2ba48f134fdaa
---
M src/gbproxy/gb_proxy_peer.c
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
daniel: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/gbproxy/gb_proxy_peer.c b/src/gbproxy/gb_proxy_peer.c
index a0586fe..ed7df32 100644
--- a/src/gbproxy/gb_proxy_peer.c
+++ b/src/gbproxy/gb_proxy_peer.c
@@ -77,7 +77,7 @@
return NULL;
bvc->bvci = bvci;
- bvc->ctrg = rate_ctr_group_alloc(bvc, &bvc_ctrg_desc, bvci);
+ bvc->ctrg = rate_ctr_group_alloc(bvc, &bvc_ctrg_desc, (nse->nsei << 16)
| bvci);
if (!bvc->ctrg) {
talloc_free(bvc);
return NULL;
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/21613
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I13f3c9e69562a56ad7d3742fdeb2ba48f134fdaa
Gerrit-Change-Number: 21613
Gerrit-PatchSet: 5
Gerrit-Owner: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillm...@sysmocom.de>
Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: lynxis lazus <lyn...@fe80.eu>
Gerrit-MessageType: merged
