Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/6871
to look at the new patch set (#2).
measurement.c: Fix sdcch4_meas_rep_fn102 / sdcch8_meas_rep_fn102
Since Change-Id: I23fba50f48415314da40cf5bf86fce2ed3e66af6 we were not
reporting measurements for SDCCH channel types due to the wrong
encoding of the sdcch{4,8}_meas_rep_fn102 table.
Let's fix the table by encoding the needed information:
"What is the modulo-102 remainder of the first burst of the last block
before fn%102 reaches 37?" (SDCCH/4)
"What is the modulo-102 remainder of the first burst of the last block
before fn%102 reaches 12?" (SDCCH/8)
The TS 45.002 Clause 7 tables have to be consulted carefully to
determine this information.
Change-Id: Icf02354872670126ab3297b787b216981ca6c309
Related: OS#2965
---
M src/common/measurement.c
1 file changed, 14 insertions(+), 15 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/71/6871/2
diff --git a/src/common/measurement.c b/src/common/measurement.c
index 69fbd94..6e9971b 100644
--- a/src/common/measurement.c
+++ b/src/common/measurement.c
@@ -64,25 +64,24 @@
* SDCCH/4 37 to 36
*/
-/* Added interleave offset to Meas period end Fn which
- * would reduce the Meas Res msg load at Abis */
-
+/* FN of the first burst whose block completes before reaching fn%102=11 */
static const uint8_t sdcch8_meas_rep_fn102[] = {
- [0] = 11 + 7,
- [1] = 11 + 11,
- [2] = 11 + 15,
- [3] = 11 + 19,
- [4] = 11 + 23,
- [5] = 11 + 27,
- [6] = 11 + 31,
- [7] = 11 + 35
+ [0] = 66, /* 15(SDCCH), 47(SACCH), 66(SDCCH) */
+ [1] = 70, /* 19(SDCCH), 51(SACCH), 70(SDCCH) */
+ [2] = 74, /* 23(SDCCH), 55(SACCH), 74(SDCCH) */
+ [3] = 78, /* 27(SDCCH), 59(SACCH), 78(SDCCH) */
+ [4] = 98, /* 31(SDCCH), 98(SACCH), 82(SDCCH) */
+ [5] = 0,/* 35(SDCCH), 0(SACCH), 86(SDCCH) */
+ [6] = 4,/* 39(SDCCH), 4(SACCH), 90(SDCCH) */
+ [7] = 8,/* 43(SDCCH), 8(SACCH), 94(SDCCH) */
};
+/* FN of the first burst whose block completes before reaching fn%102=37 */
static const uint8_t sdcch4_meas_rep_fn102[] = {
- [0] = 36 + 4,
- [1] = 36 + 8,
- [2] = 36 + 14,
- [3] = 36 + 18
+ [0] = 88, /* 37(SDCCH), 57(SACCH), 88(SDCCH) */
+ [1] = 92, /* 41(SDCCH), 61(SACCH), 92(SDCCH) */
+ [2] = 6,/* 6(SACCH), 47(SDCCH), 98(SDCCH) */
+ [3] = 10/* 10(SACCH), 0(SDCCH), 51(SDCCH) */
};
/* Note: The reporting of the measurement results is done via the SACCH
channel.
--
To view, visit https://gerrit.osmocom.org/6871
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Icf02354872670126ab3297b787b216981ca6c309
Gerrit-PatchSet: 2
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: Harald Welte
Gerrit-Reviewer: Jenkins Builder