Attention is currently required from: arehbein, pespin, daniel.
laforge 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 5:
(1 comment)
Patchset:
PS5:
implementation of osmo_stream should always use iofd, hence the old implementation is dropped (or #if 0 or ENV disabled for now).
I'd love to see that, given that osmo_io has the select/poll backend anyway for backwards compatibility.
The problem I see is: How would you handle compatibility with old code that registers a read_cb? That old code expects to do the actual read itself, from the file descriptor. But in osmo_io that already has happened by the infrastructure at the tiem you get to the read_cb...