Attention is currently required from: laforge. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-abis/+/28063 )

Change subject: input/ipaccess: Don't start zero-ms timer on every write ......................................................................

Patch Set 1: Code-Review+1

(1 comment)

File src/input/ipaccess.c:

https://gerrit.osmocom.org/c/libosmo-abis/+/28063/comment/91b3ebb5_763f36bc PS1, Line 535: } I think with this change you are leaving write enabled even if there's no more stuff in the queue to send for this TS. So that means poll will trigger again and end up in the path of line 496 (e1inp_tx_ts() returning NULL).

So ideally, in this line there would be an: else { if (list_for_this_ts_is_empty()) osmo_fd_write_disable(bfd); }

This can of course be seen as an extra optimization, done in a different patch.