Attention is currently required from: falconia, fixeria, jolly, laforge.
pespin has posted comments on this change by falconia. (
https://gerrit.osmocom.org/c/libosmo-netif/+/39280?usp=email )
Change subject: bring twjit into libosmo-netif
......................................................................
Patch Set 6:
(2 comments)
Patchset:
PS6:
Other than the public structs discussion, other stuff looks good to me already.
File include/osmocom/netif/twjit.h:
https://gerrit.osmocom.org/c/libosmo-netif/+/39280/comment/bae21d5e_5c88941… :
PS6, Line 50: struct osmo_twjit_config {
The only thing left which I'm now concerned is regarding exposing all these structs
here, which means extending them later on would break the ABI.
Hence, my proposal (I'm happy to discuss or get back other proposals):
* Add setter APIs for fields in struct osmo_twjit_config, eg:
int osmo_twjit_set_bd_start(struct osmo_twjit *twjit, uint16_t bf_start);
* osmo_twjit_stats: Have an API to obtain stats based on a enum,similar to getsocktopt.
eg:
uint32_t rx_packets;
int osmo_twjit_get_stat_(twjit, OSMO_TWJIT_STAT_RX_PACKETS, &rx_packets)
* osmo_twjit_rr_info: Probably the same as osmo_twjit_stats?
--
To view, visit
https://gerrit.osmocom.org/c/libosmo-netif/+/39280?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-Change-Id: Ia3be5834571ca18b68939abbcf1ce3a879156658
Gerrit-Change-Number: 39280
Gerrit-PatchSet: 6
Gerrit-Owner: falconia <falcon(a)freecalypso.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: jolly <andreas(a)eversberg.eu>
Gerrit-Attention: falconia <falcon(a)freecalypso.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 25 Aug 2025 09:45:17 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No