Attention is currently required from: jolly, laforge.
pespin has posted comments on this change by jolly. (
https://gerrit.osmocom.org/c/libosmocore/+/40489?usp=email )
Change subject: Allow io_uring_submit batching just ahead of poll/select
......................................................................
Patch Set 3:
(2 comments)
Patchset:
PS3:
I'm still not convinced we want to add more envvars. This is something we can probably
configure through VTY.
Only reason I see for having a envvar is in the event we want to have this opt-in for a
while and later on enable it and remove the envvar.
File src/core/select.c:
https://gerrit.osmocom.org/c/libosmocore/+/40489/comment/325868c5_e2d4e50a?… :
PS3, Line 440: osmo_io_uring_submit();
I'd say it makes more sense to have the OSMO_UNLIKELY if here directly, to avoid an
extra call+ret on each loop iteration.
Maybe as a C macro if you want to keep it similar.
--
To view, visit
https://gerrit.osmocom.org/c/libosmocore/+/40489?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: Id34fe2ced32c63d15b14810e145744f7509064cc
Gerrit-Change-Number: 40489
Gerrit-PatchSet: 3
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Tue, 22 Jul 2025 14:35:34 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No