Attention is currently required from: fixeria.
pespin has posted comments on this change by pespin. ( https://gerrit.osmocom.org/c/libosmo-netif/+/38894?usp=email )
Change subject: stream_srv: Add osmo_stream_srv_link_set_{ip_dscp,priority}() APIs ......................................................................
Patch Set 4:
(1 comment)
File src/stream_srv.c:
https://gerrit.osmocom.org/c/libosmo-netif/+/38894/comment/484302d3_62a3cc88... : PS4, Line 258: return 0;
Maybe just change to `void`?
No, I prefer keeping with a return in case we want to add some sort of param check or set it through setsockopt at the future, etc. We are already hit by some APIs which were not returning an error code initially and now are left with no information of error despite being able to fail, like osmo_stream_cli_send() (eg. iofd queue going full).