Attention is currently required from: jolly, laforge.

pespin has posted comments on this change by jolly. ( 
https://gerrit.osmocom.org/c/libosmocore/+/40725?usp=email )

Change subject: Automatically increase io_uring, if too small.
......................................................................


Patch Set 6:

(2 comments)

Patchset:

PS6:
> Due to the comments I wrote I still personally believe this is shooting us in 
> the foot, it's not the […]
More thoughts on this: With my proposal where we queue iofds waiting for an SQE 
we could end up in some sort of starvation or even deadlock, because we may end 
up with tons of fds/sockets with pending READs and nothing external (eg. a 
packet arriving) triggering a READ from it, leaving outside from the reading 
set some socket which may already contain some data to be read.


File src/core/osmo_io_uring.c:

https://gerrit.osmocom.org/c/libosmocore/+/40725/comment/bf6415a1_10d4d758?usp=email
 :
PS6, Line 172:  sqe = io_uring_get_sqe(&g_ring->ring);
> In the end, they are all empty msgb's waiting for some incoming data

Aren't we using it from read()? Then it's not simply empty msgbs waiting for 
packets, but buffers being filled from an in-order stream of bytes which needs 
to be kept in order when usbmitting the msgbs to upper layers (or when 
resegmenting internally).



--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/40725?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Id9230146acc8d54bfd44834e783c31b37bd64bca
Gerrit-Change-Number: 40725
Gerrit-PatchSet: 6
Gerrit-Owner: jolly <andr...@eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-CC: pespin <pes...@sysmocom.de>
Gerrit-Attention: jolly <andr...@eversberg.eu>
Gerrit-Attention: laforge <lafo...@osmocom.org>
Gerrit-Comment-Date: Thu, 31 Jul 2025 15:00:07 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <lafo...@osmocom.org>
Comment-In-Reply-To: pespin <pes...@sysmocom.de>

Reply via email to