Attention is currently required from: fixeria, pespin. falconia has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/36706?usp=email )
Change subject: rsl.adoc: document RSL_IE_OSMO_RTP_EXTENSIONS ...................................................................... Patch Set 2: (1 comment) File doc/manuals/abis/rsl.adoc: https://gerrit.osmocom.org/c/osmo-bts/+/36706/comment/e1b69abb_41f5c0c7 PS1, Line 1110: This information element requests the use of non-standard enhanced RTP > @[email protected]: thank you for suggesting the idea of a Redmine ticket. > […] @[email protected]: I just studied Osmocom TLV parser in libosmocore/src/gsm/tlv_parser.c, and I don't see where it treats unknown IEIs as having TLV format. Instead my reading of the code is that it stops on the first unknown IEI and returns OSMO_TLVP_ERR_UNKNOWN_TLV_TYPE - see function tlv_parse_one(), line 321 in that source file. With this observation, I see no advantage with defining new IEs as TLV rather than TV, hence we can use either format for the new IE under consideration here. I'll add this question to OS#6448, so we can continue the discussion there. -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/36706?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: I6117049b17ced5fb6635ac70d9238169033af4de Gerrit-Change-Number: 36706 Gerrit-PatchSet: 2 Gerrit-Owner: falconia <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: fixeria <[email protected]> Gerrit-CC: pespin <[email protected]> Gerrit-Attention: fixeria <[email protected]> Gerrit-Attention: pespin <[email protected]> Gerrit-Comment-Date: Wed, 08 May 2024 18:58:37 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: falconia <[email protected]> Comment-In-Reply-To: fixeria <[email protected]> Comment-In-Reply-To: pespin <[email protected]> Gerrit-MessageType: comment
