pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-pfcp/+/39580?usp=email )
Change subject: pfcp_endpoint: Increase sent_{requests,responses}_by_seq_nr hashtable size
......................................................................
pfcp_endpoint: Increase sent_{requests,responses}_by_seq_nr hashtable size
Change-Id: Ie0bb28240b3a83a1776702f26d79a09e8f0e30f6
---
M src/libosmo-pfcp/pfcp_endpoint.c
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
fixeria: Looks good to me, but someone else must approve
osmith: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/libosmo-pfcp/pfcp_endpoint.c b/src/libosmo-pfcp/pfcp_endpoint.c
index d6c109b..7c1058f 100644
--- a/src/libosmo-pfcp/pfcp_endpoint.c
+++ b/src/libosmo-pfcp/pfcp_endpoint.c
@@ -52,12 +52,12 @@
* For a transmitted Request message, wait for a matching Response from a remote peer; if none arrives,
* retransmit (see n1 and t1_ms). */
struct llist_head sent_requests;
- DECLARE_HASHTABLE(sent_requests_by_seq_nr, 10);
+ DECLARE_HASHTABLE(sent_requests_by_seq_nr, 12);
/* All transmitted PFCP Response messages, list of osmo_pfcp_queue_entry.
* For a transmitted Response message, keep it in the queue for a fixed amount of time. If the peer retransmits
* the original Request, do not dispatch the Request, but respond with the queued message directly. */
struct llist_head sent_responses;
- DECLARE_HASHTABLE(sent_responses_by_seq_nr, 10);
+ DECLARE_HASHTABLE(sent_responses_by_seq_nr, 12);
};
/*! Entry of pfcp_endpoint message queue of PFCP messages, for re-transsions. */
--
To view, visit https://gerrit.osmocom.org/c/libosmo-pfcp/+/39580?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: libosmo-pfcp
Gerrit-Branch: master
Gerrit-Change-Id: Ie0bb28240b3a83a1776702f26d79a09e8f0e30f6
Gerrit-Change-Number: 39580
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Attention is currently required from: lynxis lazus.
pespin has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/39560?usp=email )
Change subject: add libvlr based on MSC commit 25362cc295036a4671804db9003c3d22a88309d2
......................................................................
Patch Set 5:
(1 comment)
Patchset:
PS5:
> This is a synchronisation commit. It is about to sync two branches. […]
Then it sounds like this should really be fixed in osmo-msc and then copy-pasted here, or fix them in both sides at the same time. Otherwise code will end up differing in quite a lot of places for no good reason.
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/39560?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: I221114e00e6f338a75ae0dd7ac285e262b23635d
Gerrit-Change-Number: 39560
Gerrit-PatchSet: 5
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Tue, 25 Feb 2025 09:38:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>