Attention is currently required from: laforge.
jolly has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmocore/+/34987?usp=email )
Change subject: LAPDm: Add an extra queue for UI frames
......................................................................
Patch Set 9:
(2 comments)
File include/osmocom/gsm/lapdm.h:
https://gerrit.osmocom.org/c/libosmocore/+/34987/comment/026bd399_0400e9ad
PS8, Line 37: struct llist_head tx_ui_queue; /*!< UI frames to L1 */
TODO-RELEASE due to ABI breakage
Done
File src/gsm/lapdm.c:
https://gerrit.osmocom.org/c/libosmocore/+/34987/comment/d6af0bf0_1a13513a
PS8, Line 295: while ((msg = msgb_dequeue(&dl->tx_ui_queue)))
this function reimplements the msgb_queue_free()
function which we already have in msgb. […]
removed.
--
To view, visit
https://gerrit.osmocom.org/c/libosmocore/+/34987?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I00c8ee73be8b7c564a4dee3fca3e893484f567da
Gerrit-Change-Number: 34987
Gerrit-PatchSet: 9
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Wed, 15 Nov 2023 13:58:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment