[M] Change in osmo-msc[master]: libmsc: add X5 timer for delaying LU connections

Attention is currently required from: laforge, neels, pespin. fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-msc/+/36760?usp=email ) Change subject: libmsc: add X5 timer for delaying LU connections ...................................................................... Patch Set 4: (4 comments) Patchset: PS4: I suggested this improvement to the customer (see note-16 in SYS#6913), but they refused. I would still like to implement this, but I ran out of the estimated time, so I'll leave this as a potential improvement for a separate patch. PS4: This sounds logical and would be nice, but there exists no such thing in MAP protocol and thus should not be in GSUP, unless we really want to deviate from the former. See 3GPP TS 29.002, Table 12.4/1. File src/libmsc/msc_a.c: https://gerrit.osmocom.org/c/osmo-msc/+/36760/comment/db8c35e7_5129ac6b PS4, Line 164: static void msc_a_put_use_lu(struct msc_a *msc_a) Done File src/libmsc/ran_infra.c: https://gerrit.osmocom.org/c/osmo-msc/+/36760/comment/cd9271a5_0f3703ef PS4, Line 47: { .T = -5, .default_val = 0, .unit = OSMO_TDEF_MS, .desc = "Additional delay for LU connections" }, \ (two separate things in one thread making it a bit hard to respond) doc string change: ack, updated in the new patchset. Done. I remember this was discussed somewhere, but I am not sure if we ever reached an agreement on the numbering scope of those Osmocom specific X-timers. According to that table, timer numbers like X1 and X2 are re-used in several projects, but AFAIU this is because they were added before you suggested to make X-timer numbers globally unique? Also, there is nothing on that wiki page saying "please avoid re-using the same timer numbers"... I don't have a strong opinion here, so I'll just follow your suggestion. -- To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/36760?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-msc Gerrit-Branch: master Gerrit-Change-Id: Ic519cab55d65e47b2636124427dab1a1d80fab78 Gerrit-Change-Number: 36760 Gerrit-PatchSet: 4 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: laforge <laforge(a)osmocom.org> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-Comment-Date: Tue, 28 May 2024 07:55:23 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: neels <nhofmeyr(a)sysmocom.de> Gerrit-MessageType: comment