laforge submitted this change.

View Change

Approvals: Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve laforge: Looks good to me, approved
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(-)

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 change 27793. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-pcap
Gerrit-Branch: master
Gerrit-Change-Id: I72babfcc31e12624f30c16450dafd988192148be
Gerrit-Change-Number: 27793
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged