This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Harald Welte gerrit-no-reply at lists.osmocom.orgHarald Welte has submitted this change and it was merged. Change subject: dtx_check.gawk: Fix false-positives in DTX check ...................................................................... dtx_check.gawk: Fix false-positives in DTX check Previously, if ONSET happened exactly at the place where next SID FIRST or UPDATE should be it was incorrectly detected as error. Similarly, continuos FACCH was misinterpreted as error. Change-Id: I43fdbceea0dbdb0833c3b1cf0fc3b825803ed30e --- M contrib/dtx_check.gawk 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Harald Welte: Looks good to me, approved Jenkins Builder: Verified diff --git a/contrib/dtx_check.gawk b/contrib/dtx_check.gawk index 7ac8b6d..3a3fa26 100755 --- a/contrib/dtx_check.gawk +++ b/contrib/dtx_check.gawk @@ -40,7 +40,7 @@ CHK = "FAIL: " TYPE " followed by " $2 " instead of P2." ERR++ } - if ("OK" == CHK) { # check inter-SID distances: + if ("OK" == CHK && "ONSET" != $2) { # check inter-SID distances: if ("UPDATE" == TYPE) { if (DELTA > U_MAX) { CHK = "FAIL: delta (" $1 - FN "fn) from previous SID UPDATE (@" FN ") too big " DELTA "ms > " U_MAX "ms." @@ -62,8 +62,8 @@ } } } - if ("FACCH" == TYPE && "FIRST" != $2 && 1 == SILENCE) { # check FACCH handling - CHK = "FAIL: incorrect silence resume after FACCH." + if ("FACCH" == TYPE && "FIRST" != $2 && "FACCH" != $2 && 1 == SILENCE) { # check FACCH handling + CHK = "FAIL: incorrect silence resume with " $2 " after FACCH." ERR++ } } -- To view, visit https://gerrit.osmocom.org/1207 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I43fdbceea0dbdb0833c3b1cf0fc3b825803ed30e Gerrit-PatchSet: 4 Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Owner: Max <msuraev at sysmocom.de> Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org> Gerrit-Reviewer: Jenkins Builder