Attention is currently required from: arehbein, laforge, pespin.
daniel has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmo-netif/+/33193 )
Change subject: Add osmo_io support to osmo_stream_cli and osmo_stream_srv
......................................................................
Patch Set 6:
(1 comment)
Patchset:
PS5:
Internally, the osmo_stream code calls the read_cb()
if non NULL, or/and the iofd_read_cb() if not NULL (the only difference afaiu is the extra
msg ptr being passed).
I really don't like the and/or here since the difference is read_cb() will need to
read from the fd itself and iofd does it for you. So there can only ever be an XOR
relationship between these. I'd go as far as (if we went for this API) to assert that
you never call the new api on a conn created with the old one.
Since we need to stay backwards compatible with the old API we need to keep the old
functions:
osmo_stream_cli_create(), osmo_stream_cli_set_read_cb(), osmo_stream_srv_create()
New functions using osmo_io would then be:
* osmo_stream_cli_create2() with name param
* osmo_stream_cli_set_iofd_read_cb() (?)
* osmo_stream_srv_create2() with name param and no function pointers
* osmo_stream_srv_set_iofd_read_cb() (could be just read_cb, but would be confusing as the
client already has that)
* osmo_stream_srv_set_closed_cb()
--
To view, visit
https://gerrit.osmocom.org/c/libosmo-netif/+/33193
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: I2f52c7107c392b6f4b0bf2a84f8c873c084a200c
Gerrit-Change-Number: 33193
Gerrit-PatchSet: 6
Gerrit-Owner: arehbein <arehbein(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: daniel <dwillmann(a)sysmocom.de>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: arehbein <arehbein(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 08 Jun 2023 15:25:05 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment