Attention is currently required from: arehbein.
4 comments:
Patchset:
I think the problem is that you are storing the config value directly in the wqueue, and then when initializing the wqueue you may erase it? Then add an extra field which is updated by VTY and use that when initializing the wqueue.
File src/osmo-bsc/bsc_vty.c:
Patch Set #1, Line 2487: if (fd >= 0 && ((ofd = osmo_fd_get_by_fd(fd)) != NULL)) {
This is to check whether or not the socket and hence has been (properly) opened via `pcu_sock_init`, […]
why do you care about the socket being opened here?
if you want to check for pcu-state presence, then simply: if(net->pcu_state) { ...}
That's what this does; osmo write queue code (the code in `osmo_wqueue_bfd_cb()`) will use the addre […]
ah I thought there may be an API so I didn't see it sorry.
Actually, there should be an API, since it should internally free the extra messages already enequeued if the new max_len is smaller.
File src/osmo-bsc/pcu_sock.c:
Patch Set #1, Line 956: net->pcu_sock_fd = rc;
it enables the check commented on [above](https://gerrit.osmocom. […]
That check is not needed imho.
To view, visit change 33892. To unsubscribe, or for help writing mail filters, visit settings.