laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-pcap/+/27793 )
Change subject: client: Increase wqueue transmit length
......................................................................
client: Increase wqueue transmit length
Having a length of 10 packets is too low, it can be filled easily under
high load or really bursty traffic, where the input path could be polled
multiple times while the output path (write socket poll) is not called.
Related: SYS#5921
Change-Id: I72babfcc31e12624f30c16450dafd988192148be
---
M src/osmo_client_core.c
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Jenkins Builder: Verified
fixeria: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
diff --git a/src/osmo_client_core.c b/src/osmo_client_core.c
index 59a6777..8209afe 100644
--- a/src/osmo_client_core.c
+++ b/src/osmo_client_core.c
@@ -341,7 +341,7 @@
{
conn->client = client;
conn->tls_verify = true;
- osmo_wqueue_init(&conn->wqueue, 10);
+ osmo_wqueue_init(&conn->wqueue, 1000);
conn->wqueue.bfd.fd = -1;
}
--
To view, visit
https://gerrit.osmocom.org/c/osmo-pcap/+/27793
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcap
Gerrit-Branch: master
Gerrit-Change-Id: I72babfcc31e12624f30c16450dafd988192148be
Gerrit-Change-Number: 27793
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: merged