<p>laforge <strong>submitted</strong> this change.</p><p><a href="https://gerrit.osmocom.org/c/libosmo-netif/+/18265">View Change</a></p><div style="white-space:pre-wrap">Approvals:
laforge: Looks good to me, approved
pespin: Looks good to me, but someone else must approve
Jenkins Builder: Verified
</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">amr: Fix OA<->BWE conversion.<br><br>Size of a single AMR frame doesn't always shrink by a byte when<br>converted from octet-aligned to bandwidth-efficient mode. It does<br>shrink for AMR modes 2, 3, 4, 6, and 7 but doesn't shrink for<br>AMR modes 0, 1, 5, and SID frames because we only remove 6 bits.<br>So old code generated truncated AMR packets for those AMR modes.<br>This patch fixes the length calculation by properly counting bits.<br><br>Proper bit counting is also bringing us one small step closer to<br>properly handlig multi-frame AMR packets.<br><br>Change-Id: I0462e054a0adc9080456f3eeea9cab7c229cdb70<br>---<br>M include/osmocom/netif/amr.h<br>M src/amr.c<br>M tests/amr/amr_test.c<br>M tests/amr/amr_test.ok<br>4 files changed, 164 insertions(+), 83 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/include/osmocom/netif/amr.h b/include/osmocom/netif/amr.h</span><br><span>index 6e37c99..c5a8e28 100644</span><br><span>--- a/include/osmocom/netif/amr.h</span><br><span>+++ b/include/osmocom/netif/amr.h</span><br><span>@@ -78,20 +78,34 @@</span><br><span> #define AMR_FT_SID 8 /* SID */</span><br><span> #define AMR_FT_MAX 9</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-/* AMR voice frame length (in bytes, rounded),</span><br><span style="color: hsl(0, 100%, 40%);">- * See also RFC 3267, chapter 3.6 */</span><br><span style="color: hsl(0, 100%, 40%);">-#define AMR_FT_0_LEN 12 /* 4.75 */</span><br><span style="color: hsl(0, 100%, 40%);">-#define AMR_FT_1_LEN 13 /* 5.15 */</span><br><span style="color: hsl(0, 100%, 40%);">-#define AMR_FT_2_LEN 15 /* 5.90 */</span><br><span style="color: hsl(0, 100%, 40%);">-#define AMR_FT_3_LEN 17 /* 6.70 */</span><br><span style="color: hsl(0, 100%, 40%);">-#define AMR_FT_4_LEN 19 /* 7.40 */</span><br><span style="color: hsl(0, 100%, 40%);">-#define AMR_FT_5_LEN 20 /* 7.95 */</span><br><span style="color: hsl(0, 100%, 40%);">-#define AMR_FT_6_LEN 26 /* 10.2 */</span><br><span style="color: hsl(0, 100%, 40%);">-#define AMR_FT_7_LEN 31 /* 12.2 */</span><br><span style="color: hsl(0, 100%, 40%);">-#define AMR_FT_SID_LEN 5 /* SID */</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">-/* NOTE: the above constant refers to the length of one AMR speech frame-block,</span><br><span style="color: hsl(120, 100%, 40%);">+/* AMR voice frame length (in bits).</span><br><span style="color: hsl(120, 100%, 40%);">+ * See also RFC 3267, chapter 3.6.</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * NOTE: These constants refer to the length of one AMR speech frame-block,</span><br><span> * not counting CMR, TOC. */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_0_LEN_BITS 95 /* 4.75 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_1_LEN_BITS 103 /* 5.15 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_2_LEN_BITS 118 /* 5.90 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_3_LEN_BITS 134 /* 6.70 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_4_LEN_BITS 148 /* 7.40 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_5_LEN_BITS 159 /* 7.95 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_6_LEN_BITS 204 /* 10.2 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_7_LEN_BITS 244 /* 12.2 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_SID_LEN_BITS 39 /* SID */</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+/* AMR voice frame length (in bytes, rounded).</span><br><span style="color: hsl(120, 100%, 40%);">+ *</span><br><span style="color: hsl(120, 100%, 40%);">+ * NOTE: These constants refer to the length of one AMR speech frame-block,</span><br><span style="color: hsl(120, 100%, 40%);">+ * not counting CMR, TOC. */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_0_LEN ((AMR_FT_0_LEN_BITS+7)/8) /* 4.75 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_1_LEN ((AMR_FT_1_LEN_BITS+7)/8) /* 5.15 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_2_LEN ((AMR_FT_2_LEN_BITS+7)/8) /* 5.90 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_3_LEN ((AMR_FT_3_LEN_BITS+7)/8) /* 6.70 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_4_LEN ((AMR_FT_4_LEN_BITS+7)/8) /* 7.40 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_5_LEN ((AMR_FT_5_LEN_BITS+7)/8) /* 7.95 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_6_LEN ((AMR_FT_6_LEN_BITS+7)/8) /* 10.2 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_7_LEN ((AMR_FT_7_LEN_BITS+7)/8) /* 12.2 */</span><br><span style="color: hsl(120, 100%, 40%);">+#define AMR_FT_SID_LEN ((AMR_FT_SID_LEN_BITS+7)/8) /* SID */</span><br><span> </span><br><span> int osmo_amr_ft_valid(uint8_t amr_ft);</span><br><span> size_t osmo_amr_bytes(uint8_t amr_cmr);</span><br><span>diff --git a/src/amr.c b/src/amr.c</span><br><span>index 5609c46..e4c7bb5 100644</span><br><span>--- a/src/amr.c</span><br><span>+++ b/src/amr.c</span><br><span>@@ -29,6 +29,18 @@</span><br><span> * 7 12.20 244 31</span><br><span> */</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+static size_t amr_ft_to_bits[AMR_FT_MAX] = {</span><br><span style="color: hsl(120, 100%, 40%);">+ [AMR_FT_0] = AMR_FT_0_LEN_BITS,</span><br><span style="color: hsl(120, 100%, 40%);">+ [AMR_FT_1] = AMR_FT_1_LEN_BITS,</span><br><span style="color: hsl(120, 100%, 40%);">+ [AMR_FT_2] = AMR_FT_2_LEN_BITS,</span><br><span style="color: hsl(120, 100%, 40%);">+ [AMR_FT_3] = AMR_FT_3_LEN_BITS,</span><br><span style="color: hsl(120, 100%, 40%);">+ [AMR_FT_4] = AMR_FT_4_LEN_BITS,</span><br><span style="color: hsl(120, 100%, 40%);">+ [AMR_FT_5] = AMR_FT_5_LEN_BITS,</span><br><span style="color: hsl(120, 100%, 40%);">+ [AMR_FT_6] = AMR_FT_6_LEN_BITS,</span><br><span style="color: hsl(120, 100%, 40%);">+ [AMR_FT_7] = AMR_FT_7_LEN_BITS,</span><br><span style="color: hsl(120, 100%, 40%);">+ [AMR_FT_SID] = AMR_FT_SID_LEN_BITS,</span><br><span style="color: hsl(120, 100%, 40%);">+};</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> static size_t amr_ft_to_bytes[AMR_FT_MAX] = {</span><br><span> [AMR_FT_0] = AMR_FT_0_LEN,</span><br><span> [AMR_FT_1] = AMR_FT_1_LEN,</span><br><span>@@ -41,6 +53,11 @@</span><br><span> [AMR_FT_SID] = AMR_FT_SID_LEN,</span><br><span> };</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+size_t osmo_amr_bits(uint8_t amr_ft)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+ return amr_ft_to_bits[amr_ft];</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> size_t osmo_amr_bytes(uint8_t amr_ft)</span><br><span> {</span><br><span> return amr_ft_to_bytes[amr_ft];</span><br><span>@@ -119,8 +136,10 @@</span><br><span> int osmo_amr_oa_to_bwe(uint8_t *payload, unsigned int payload_len)</span><br><span> {</span><br><span> struct amr_hdr *oa_hdr = (struct amr_hdr *)payload;</span><br><span style="color: hsl(120, 100%, 40%);">+ unsigned int ft = oa_hdr->ft;</span><br><span> unsigned int frame_len = payload_len - sizeof(struct amr_hdr);</span><br><span> unsigned int i;</span><br><span style="color: hsl(120, 100%, 40%);">+ int bwe_payload_len;</span><br><span> </span><br><span> /* This implementation is not capable to handle multi-frame</span><br><span> * packets, so we need to make sure that the frame we operate on</span><br><span>@@ -128,6 +147,10 @@</span><br><span> if (oa_hdr->f != 0)</span><br><span> return -1;</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+ /* Check for valid FT (AMR mode) value */</span><br><span style="color: hsl(120, 100%, 40%);">+ if (!osmo_amr_ft_valid(oa_hdr->ft))</span><br><span style="color: hsl(120, 100%, 40%);">+ return -1;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> /* Move TOC close to CMR */</span><br><span> payload[0] |= (payload[1] >> 4) & 0x0f;</span><br><span> payload[1] = (payload[1] << 4) & 0xf0;</span><br><span>@@ -137,8 +160,10 @@</span><br><span> payload[i + 2] = payload[i + 2] << 6;</span><br><span> }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">- /* The overall saving is one byte! */</span><br><span style="color: hsl(0, 100%, 40%);">- return payload_len - 1;</span><br><span style="color: hsl(120, 100%, 40%);">+ /* Calculate new payload length */</span><br><span style="color: hsl(120, 100%, 40%);">+ bwe_payload_len = (10 + osmo_amr_bits(ft) + 7) / 8;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ return bwe_payload_len;</span><br><span> }</span><br><span> </span><br><span> /*! Convert an AMR frame from bandwith-efficient mode to octet-aligned mode.</span><br><span>@@ -150,8 +175,10 @@</span><br><span> unsigned int payload_maxlen)</span><br><span> {</span><br><span> uint8_t buf[256];</span><br><span style="color: hsl(0, 100%, 40%);">- unsigned int frame_len = payload_len - 1;</span><br><span style="color: hsl(120, 100%, 40%);">+ /* The header is only valid after shifting first two bytes to OA mode */</span><br><span style="color: hsl(120, 100%, 40%);">+ struct amr_hdr *oa_hdr;</span><br><span> unsigned int i;</span><br><span style="color: hsl(120, 100%, 40%);">+ int oa_payload_len;</span><br><span> </span><br><span> memset(buf, 0, sizeof(buf));</span><br><span> </span><br><span>@@ -165,12 +192,18 @@</span><br><span> buf[1] = payload[0] << 4;</span><br><span> buf[1] |= (payload[1] >> 4) & 0x0c;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">- for (i = 0; i < frame_len - 1; i++) {</span><br><span style="color: hsl(120, 100%, 40%);">+ /* Calculate new payload length */</span><br><span style="color: hsl(120, 100%, 40%);">+ oa_hdr = (struct amr_hdr *)buf;</span><br><span style="color: hsl(120, 100%, 40%);">+ if (!osmo_amr_ft_valid(oa_hdr->ft))</span><br><span style="color: hsl(120, 100%, 40%);">+ return -1;</span><br><span style="color: hsl(120, 100%, 40%);">+ oa_payload_len = 2 + osmo_amr_bytes(oa_hdr->ft);</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ for (i = 0; i < oa_payload_len - 2; i++) {</span><br><span> buf[i + 2] = payload[i + 1] << 2;</span><br><span> buf[i + 2] |= payload[i + 2] >> 6;</span><br><span> }</span><br><span> buf[i + 2] = payload[i + 1] << 2;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">- memcpy(payload, buf, payload_len + 1);</span><br><span style="color: hsl(0, 100%, 40%);">- return payload_len + 1;</span><br><span style="color: hsl(120, 100%, 40%);">+ memcpy(payload, buf, oa_payload_len);</span><br><span style="color: hsl(120, 100%, 40%);">+ return oa_payload_len;</span><br><span> }</span><br><span>diff --git a/tests/amr/amr_test.c b/tests/amr/amr_test.c</span><br><span>index fff686c..2f41fe9 100644</span><br><span>--- a/tests/amr/amr_test.c</span><br><span>+++ b/tests/amr/amr_test.c</span><br><span>@@ -44,7 +44,10 @@</span><br><span> "100c4e9ba850e30d5d53d04de41e7c",</span><br><span> "100c6c18e7b7fff53aeb055e7d1c54",</span><br><span> "100c1fb967f7f1fdf547bf2e61c060",</span><br><span style="color: hsl(0, 100%, 40%);">- "a038ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00fc", /* test pattern */</span><br><span style="color: hsl(120, 100%, 40%);">+ "0004f89d67f1160935bde1996840",</span><br><span style="color: hsl(120, 100%, 40%);">+ "0004633cc7f0630439ffe0000000",</span><br><span style="color: hsl(120, 100%, 40%);">+ "0004eb81fc0758973b9edc782552",</span><br><span style="color: hsl(120, 100%, 40%);">+ "a078ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00fc", /* sample with invalid FT */</span><br><span> "END",</span><br><span> };</span><br><span> </span><br><span>@@ -56,7 +59,7 @@</span><br><span> "f3d39a49a09e7a802852e297e8c9246aadf5a45928bfc27177fed8404d97d3b8",</span><br><span> "f3c2155b65131c68682079fab4810911200003b360ae0446000025f11e539dd0",</span><br><span> "f3c381bc7061c9f8507f6029de6115c16e5fa470c243b21b6e35dbb48bd84c00",</span><br><span style="color: hsl(0, 100%, 40%);">- "a7bfc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03f", /* test pattern */</span><br><span style="color: hsl(120, 100%, 40%);">+ "a7bfc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03f", /* sample with invalid FT */</span><br><span> "END",</span><br><span> };</span><br><span> </span><br><span>@@ -87,7 +90,6 @@</span><br><span> return;</span><br><span> printf("\n");</span><br><span> printf("Sample No.: %i\n", i);</span><br><span style="color: hsl(0, 100%, 40%);">- len = strlen(oa_amr_samples[i]);</span><br><span> </span><br><span> len = osmo_hexparse(oa_amr_samples[i], buf, sizeof(buf));</span><br><span> OSMO_ASSERT(len > 0);</span><br><span>@@ -102,11 +104,6 @@</span><br><span> dump_bits(buf, rc);</span><br><span> printf("\n");</span><br><span> printf(" rc: %i\n", rc);</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">- if (rc > 0) {</span><br><span style="color: hsl(0, 100%, 40%);">- OSMO_ASSERT(rc == len - 1);</span><br><span style="color: hsl(0, 100%, 40%);">- OSMO_ASSERT(buf[len - 1] == 0x00);</span><br><span style="color: hsl(0, 100%, 40%);">- }</span><br><span> i++;</span><br><span> }</span><br><span> }</span><br><span>@@ -126,7 +123,6 @@</span><br><span> return;</span><br><span> printf("\n");</span><br><span> printf("Sample No.: %i\n", i);</span><br><span style="color: hsl(0, 100%, 40%);">- len = strlen(bwe_amr_samples[i]);</span><br><span> </span><br><span> len = osmo_hexparse(bwe_amr_samples[i], buf, sizeof(buf));</span><br><span> OSMO_ASSERT(len > 0);</span><br><span>@@ -142,7 +138,6 @@</span><br><span> printf("\n");</span><br><span> printf(" rc: %i\n", rc);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">- OSMO_ASSERT(rc == len + 1);</span><br><span> i++;</span><br><span> }</span><br><span> }</span><br><span>@@ -151,6 +146,8 @@</span><br><span> {</span><br><span> uint8_t buf[256];</span><br><span> uint8_t buf_chk[256];</span><br><span style="color: hsl(120, 100%, 40%);">+ struct amr_hdr *oa_hd = (struct amr_hdr *)buf;</span><br><span style="color: hsl(120, 100%, 40%);">+ unsigned int ft;</span><br><span> </span><br><span> unsigned int i = 0;</span><br><span> int len;</span><br><span>@@ -163,18 +160,28 @@</span><br><span> while (1) {</span><br><span> if (strcmp(oa_amr_samples[i], "END") == 0)</span><br><span> return;</span><br><span style="color: hsl(0, 100%, 40%);">- printf("Sample No.: %i...\n", i);</span><br><span style="color: hsl(0, 100%, 40%);">- len = strlen(oa_amr_samples[i]);</span><br><span style="color: hsl(120, 100%, 40%);">+ printf("Sample No.: %i...", i);</span><br><span> </span><br><span> len = osmo_hexparse(oa_amr_samples[i], buf, sizeof(buf));</span><br><span> OSMO_ASSERT(len > 0);</span><br><span style="color: hsl(120, 100%, 40%);">+ i++;</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ ft = oa_hd->ft;</span><br><span style="color: hsl(120, 100%, 40%);">+ if (!osmo_amr_ft_valid(ft)) {</span><br><span style="color: hsl(120, 100%, 40%);">+ printf(" skipping a sample with a wrong FT\n");</span><br><span style="color: hsl(120, 100%, 40%);">+ continue;</span><br><span style="color: hsl(120, 100%, 40%);">+ }</span><br><span style="color: hsl(120, 100%, 40%);">+ OSMO_ASSERT(osmo_amr_bytes(ft) + 2 == len);</span><br><span style="color: hsl(120, 100%, 40%);">+ printf(" AMR mode: %d, OA: %d bytes,", ft, len);</span><br><span> memcpy(buf_chk, buf, sizeof(buf));</span><br><span> </span><br><span> rc = osmo_amr_oa_to_bwe(buf, len);</span><br><span> OSMO_ASSERT(rc > 0);</span><br><span style="color: hsl(120, 100%, 40%);">+ printf(" BE: %d bytes,", rc);</span><br><span> rc = osmo_amr_bwe_to_oa(buf, rc, sizeof(buf));</span><br><span style="color: hsl(120, 100%, 40%);">+ printf(" OA: %d bytes\n", rc);</span><br><span style="color: hsl(120, 100%, 40%);">+ OSMO_ASSERT(len == rc);</span><br><span> OSMO_ASSERT(memcmp(buf, buf_chk, len) == 0);</span><br><span style="color: hsl(0, 100%, 40%);">- i++;</span><br><span> }</span><br><span> }</span><br><span> </span><br><span>diff --git a/tests/amr/amr_test.ok b/tests/amr/amr_test.ok</span><br><span>index ea34fea..af1be82 100644</span><br><span>--- a/tests/amr/amr_test.ok</span><br><span>+++ b/tests/amr/amr_test.ok</span><br><span>@@ -47,23 +47,23 @@</span><br><span> Sample No.: 6</span><br><span> octet aligned: 502c98ab841e491ff7a1a555016a32a3c7f913210630</span><br><span> 01010000001011001001100010101011100001000001111001001001000111111111011110100001101001010101010100000001011010100011001010100011110001111111100100010011001000010000011000110000</span><br><span style="color: hsl(0, 100%, 40%);">- bw-efficient: 52e62ae1079247fde86955405a8ca8f1fe44c8418c</span><br><span style="color: hsl(0, 100%, 40%);">- 010100101110011000101010111000010000011110010010010001111111110111101000011010010101010101000000010110101000110010101000111100011111111001000100110010000100000110001100</span><br><span style="color: hsl(0, 100%, 40%);">- rc: 21</span><br><span style="color: hsl(120, 100%, 40%);">+ bw-efficient: 52e62ae1079247fde86955405a8ca8f1fe44c8418c00</span><br><span style="color: hsl(120, 100%, 40%);">+ 01010010111001100010101011100001000001111001001001000111111111011110100001101001010101010100000001011010100011001010100011110001111111100100010011001000010000011000110000000000</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: 22</span><br><span> </span><br><span> Sample No.: 7</span><br><span> octet aligned: 502cc5459a0d200e7097c4dfe86ec8d27f1756d776f0</span><br><span> 01010000001011001100010101000101100110100000110100100000000011100111000010010111110001001101111111101000011011101100100011010010011111110001011101010110110101110111011011110000</span><br><span style="color: hsl(0, 100%, 40%);">- bw-efficient: 52f151668348039c25f137fa1bb2349fc5d5b5ddbc</span><br><span style="color: hsl(0, 100%, 40%);">- 010100101111000101010001011001101000001101001000000000111001110000100101111100010011011111111010000110111011001000110100100111111100010111010101101101011101110110111100</span><br><span style="color: hsl(0, 100%, 40%);">- rc: 21</span><br><span style="color: hsl(120, 100%, 40%);">+ bw-efficient: 52f151668348039c25f137fa1bb2349fc5d5b5ddbc00</span><br><span style="color: hsl(120, 100%, 40%);">+ 01010010111100010101000101100110100000110100100000000011100111000010010111110001001101111111101000011011101100100011010010011111110001011101010110110101110111011011110000000000</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: 22</span><br><span> </span><br><span> Sample No.: 8</span><br><span> octet aligned: 502c42b332081813d7e916e7aa5e80d7fde812b8c080</span><br><span> 01010000001011000100001010110011001100100000100000011000000100111101011111101001000101101110011110101010010111101000000011010111111111011110100000010010101110001100000010000000</span><br><span style="color: hsl(0, 100%, 40%);">- bw-efficient: 52d0accc820604f5fa45b9ea97a035ff7a04ae3020</span><br><span style="color: hsl(0, 100%, 40%);">- 010100101101000010101100110011001000001000000110000001001111010111111010010001011011100111101010100101111010000000110101111111110111101000000100101011100011000000100000</span><br><span style="color: hsl(0, 100%, 40%);">- rc: 21</span><br><span style="color: hsl(120, 100%, 40%);">+ bw-efficient: 52d0accc820604f5fa45b9ea97a035ff7a04ae302000</span><br><span style="color: hsl(120, 100%, 40%);">+ 01010010110100001010110011001100100000100000011000000100111101011111101001000101101110011110101010010111101000000011010111111111011110100000010010101110001100000010000000000000</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: 22</span><br><span> </span><br><span> Sample No.: 9</span><br><span> octet aligned: 40240343e959c79bacd20c77501054880a718db200</span><br><span>@@ -110,30 +110,51 @@</span><br><span> Sample No.: 15</span><br><span> octet aligned: 100c4e9ba850e30d5d53d04de41e7c</span><br><span> 000100000000110001001110100110111010100001010000111000110000110101011101010100111101000001001101111001000001111001111100</span><br><span style="color: hsl(0, 100%, 40%);">- bw-efficient: 10d3a6ea1438c35754f41379079f</span><br><span style="color: hsl(0, 100%, 40%);">- 0001000011010011101001101110101000010100001110001100001101010111010101001111010000010011011110010000011110011111</span><br><span style="color: hsl(0, 100%, 40%);">- rc: 14</span><br><span style="color: hsl(120, 100%, 40%);">+ bw-efficient: 10d3a6ea1438c35754f41379079f00</span><br><span style="color: hsl(120, 100%, 40%);">+ 000100001101001110100110111010100001010000111000110000110101011101010100111101000001001101111001000001111001111100000000</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: 15</span><br><span> </span><br><span> Sample No.: 16</span><br><span> octet aligned: 100c6c18e7b7fff53aeb055e7d1c54</span><br><span> 000100000000110001101100000110001110011110110111111111111111010100111010111010110000010101011110011111010001110001010100</span><br><span style="color: hsl(0, 100%, 40%);">- bw-efficient: 10db0639edfffd4ebac1579f4715</span><br><span style="color: hsl(0, 100%, 40%);">- 0001000011011011000001100011100111101101111111111111110101001110101110101100000101010111100111110100011100010101</span><br><span style="color: hsl(0, 100%, 40%);">- rc: 14</span><br><span style="color: hsl(120, 100%, 40%);">+ bw-efficient: 10db0639edfffd4ebac1579f471500</span><br><span style="color: hsl(120, 100%, 40%);">+ 000100001101101100000110001110011110110111111111111111010100111010111010110000010101011110011111010001110001010100000000</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: 15</span><br><span> </span><br><span> Sample No.: 17</span><br><span> octet aligned: 100c1fb967f7f1fdf547bf2e61c060</span><br><span> 000100000000110000011111101110010110011111110111111100011111110111110101010001111011111100101110011000011100000001100000</span><br><span style="color: hsl(0, 100%, 40%);">- bw-efficient: 10c7ee59fdfc7f7d51efcb987018</span><br><span style="color: hsl(0, 100%, 40%);">- 0001000011000111111011100101100111111101111111000111111101111101010100011110111111001011100110000111000000011000</span><br><span style="color: hsl(0, 100%, 40%);">- rc: 14</span><br><span style="color: hsl(120, 100%, 40%);">+ bw-efficient: 10c7ee59fdfc7f7d51efcb98701800</span><br><span style="color: hsl(120, 100%, 40%);">+ 000100001100011111101110010110011111110111111100011111110111110101010001111011111100101110011000011100000001100000000000</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: 15</span><br><span> </span><br><span> Sample No.: 18</span><br><span style="color: hsl(0, 100%, 40%);">- octet aligned: a038ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00fc</span><br><span style="color: hsl(0, 100%, 40%);">- 101000000011100011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111100</span><br><span style="color: hsl(0, 100%, 40%);">- bw-efficient: a3bfc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03f</span><br><span style="color: hsl(0, 100%, 40%);">- 1010001110111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111</span><br><span style="color: hsl(0, 100%, 40%);">- rc: 32</span><br><span style="color: hsl(120, 100%, 40%);">+ octet aligned: 0004f89d67f1160935bde1996840</span><br><span style="color: hsl(120, 100%, 40%);">+ 0000000000000100111110001001110101100111111100010001011000001001001101011011110111100001100110010110100001000000</span><br><span style="color: hsl(120, 100%, 40%);">+ bw-efficient: 007e2759fc45824d6f78665a1000</span><br><span style="color: hsl(120, 100%, 40%);">+ 0000000001111110001001110101100111111100010001011000001001001101011011110111100001100110010110100001000000000000</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: 14</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 19</span><br><span style="color: hsl(120, 100%, 40%);">+ octet aligned: 0004633cc7f0630439ffe0000000</span><br><span style="color: hsl(120, 100%, 40%);">+ 0000000000000100011000110011110011000111111100000110001100000100001110011111111111100000000000000000000000000000</span><br><span style="color: hsl(120, 100%, 40%);">+ bw-efficient: 0058cf31fc18c10e7ff800000000</span><br><span style="color: hsl(120, 100%, 40%);">+ 0000000001011000110011110011000111111100000110001100000100001110011111111111100000000000000000000000000000000000</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: 14</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 20</span><br><span style="color: hsl(120, 100%, 40%);">+ octet aligned: 0004eb81fc0758973b9edc782552</span><br><span style="color: hsl(120, 100%, 40%);">+ 0000000000000100111010111000000111111100000001110101100010010111001110111001111011011100011110000010010101010010</span><br><span style="color: hsl(120, 100%, 40%);">+ bw-efficient: 007ae07f01d625cee7b71e095480</span><br><span style="color: hsl(120, 100%, 40%);">+ 0000000001111010111000000111111100000001110101100010010111001110111001111011011100011110000010010101010010000000</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: 14</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 21</span><br><span style="color: hsl(120, 100%, 40%);">+ octet aligned: a078ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00fc</span><br><span style="color: hsl(120, 100%, 40%);">+ 101000000111100011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111100</span><br><span style="color: hsl(120, 100%, 40%);">+ bw-efficient: </span><br><span style="color: hsl(120, 100%, 40%);">+ (no data)</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: -1</span><br><span> </span><br><span> </span><br><span> Testing conversion from bw-efficient to octet-aligned:</span><br><span>@@ -141,16 +162,16 @@</span><br><span> Sample No.: 0</span><br><span> bw-efficient: f4495c7cda8f80</span><br><span> 11110100010010010101110001111100110110101000111110000000</span><br><span style="color: hsl(0, 100%, 40%);">- octet aligned: f0442571f36a3e00</span><br><span style="color: hsl(0, 100%, 40%);">- 1111000001000100001001010111000111110011011010100011111000000000</span><br><span style="color: hsl(0, 100%, 40%);">- rc: 8</span><br><span style="color: hsl(120, 100%, 40%);">+ octet aligned: f0442571f36a3e</span><br><span style="color: hsl(120, 100%, 40%);">+ 11110000010001000010010101110001111100110110101000111110</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: 7</span><br><span> </span><br><span> Sample No.: 1</span><br><span> bw-efficient: f44aaa6c969780</span><br><span> 11110100010010101010101001101100100101101001011110000000</span><br><span style="color: hsl(0, 100%, 40%);">- octet aligned: f0442aa9b25a5e00</span><br><span style="color: hsl(0, 100%, 40%);">- 1111000001000100001010101010100110110010010110100101111000000000</span><br><span style="color: hsl(0, 100%, 40%);">- rc: 8</span><br><span style="color: hsl(120, 100%, 40%);">+ octet aligned: f0442aa9b25a5e</span><br><span style="color: hsl(120, 100%, 40%);">+ 11110000010001000010101010101001101100100101101001011110</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: 7</span><br><span> </span><br><span> Sample No.: 2</span><br><span> bw-efficient: f3d09c20e32da600c025a72e0a9b360386e40f87e19282094adc1a11e397d1d4</span><br><span>@@ -183,31 +204,34 @@</span><br><span> Sample No.: 6</span><br><span> bw-efficient: a7bfc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03fc03f</span><br><span> 1010011110111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111</span><br><span style="color: hsl(0, 100%, 40%);">- octet aligned: a078ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00ff00fc</span><br><span style="color: hsl(0, 100%, 40%);">- 101000000111100011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111111000000001111111100000000111111110000000011111100</span><br><span style="color: hsl(0, 100%, 40%);">- rc: 33</span><br><span style="color: hsl(120, 100%, 40%);">+ octet aligned: </span><br><span style="color: hsl(120, 100%, 40%);">+ (no data)</span><br><span style="color: hsl(120, 100%, 40%);">+ rc: -1</span><br><span> </span><br><span> </span><br><span> Testing conversion from octet-aligned to bw-efficient and inverse:</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 0...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 1...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 2...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 3...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 4...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 5...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 6...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 7...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 8...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 9...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 10...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 11...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 12...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 13...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 14...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 15...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 16...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 17...</span><br><span style="color: hsl(0, 100%, 40%);">-Sample No.: 18...</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 0... AMR mode: 7, OA: 33 bytes, BE: 32 bytes, OA: 33 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 1... AMR mode: 7, OA: 33 bytes, BE: 32 bytes, OA: 33 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 2... AMR mode: 7, OA: 33 bytes, BE: 32 bytes, OA: 33 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 3... AMR mode: 6, OA: 28 bytes, BE: 27 bytes, OA: 28 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 4... AMR mode: 6, OA: 28 bytes, BE: 27 bytes, OA: 28 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 5... AMR mode: 6, OA: 28 bytes, BE: 27 bytes, OA: 28 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 6... AMR mode: 5, OA: 22 bytes, BE: 22 bytes, OA: 22 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 7... AMR mode: 5, OA: 22 bytes, BE: 22 bytes, OA: 22 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 8... AMR mode: 5, OA: 22 bytes, BE: 22 bytes, OA: 22 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 9... AMR mode: 4, OA: 21 bytes, BE: 20 bytes, OA: 21 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 10... AMR mode: 4, OA: 21 bytes, BE: 20 bytes, OA: 21 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 11... AMR mode: 4, OA: 21 bytes, BE: 20 bytes, OA: 21 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 12... AMR mode: 2, OA: 17 bytes, BE: 16 bytes, OA: 17 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 13... AMR mode: 2, OA: 17 bytes, BE: 16 bytes, OA: 17 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 14... AMR mode: 2, OA: 17 bytes, BE: 16 bytes, OA: 17 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 15... AMR mode: 1, OA: 15 bytes, BE: 15 bytes, OA: 15 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 16... AMR mode: 1, OA: 15 bytes, BE: 15 bytes, OA: 15 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 17... AMR mode: 1, OA: 15 bytes, BE: 15 bytes, OA: 15 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 18... AMR mode: 0, OA: 14 bytes, BE: 14 bytes, OA: 14 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 19... AMR mode: 0, OA: 14 bytes, BE: 14 bytes, OA: 14 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 20... AMR mode: 0, OA: 14 bytes, BE: 14 bytes, OA: 14 bytes</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 21... skipping a sample with a wrong FT</span><br><span> </span><br><span> </span><br><span> Testing detection of octet-aligned mode payloads:</span><br><span>@@ -230,6 +254,9 @@</span><br><span> Sample No.: 16 ==>octet aligned</span><br><span> Sample No.: 17 ==>octet aligned</span><br><span> Sample No.: 18 ==>octet aligned</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 19 ==>octet aligned</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 20 ==>octet aligned</span><br><span style="color: hsl(120, 100%, 40%);">+Sample No.: 21 ==>bandwith efficient</span><br><span> Sample No.: 0 ==>bandwith efficient</span><br><span> Sample No.: 1 ==>bandwith efficient</span><br><span> Sample No.: 2 ==>bandwith efficient</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmo-netif/+/18265">change 18265</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/libosmo-netif/+/18265"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: libosmo-netif </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I0462e054a0adc9080456f3eeea9cab7c229cdb70 </div>
<div style="display:none"> Gerrit-Change-Number: 18265 </div>
<div style="display:none"> Gerrit-PatchSet: 8 </div>
<div style="display:none"> Gerrit-Owner: ipse <Alexander.Chemeris@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: ipse <Alexander.Chemeris@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: neels <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>