Attention is currently required from: laforge, manawyrm, pespin, tnt.

manawyrm uploaded patch set #7 to this change.

View Change

The following approvals got outdated and were removed: Verified-1 by Jenkins Builder

octoi: rework frame_rifo logic to decompress frames on the output side

The previous code tried to decompress frames against the last received
frame regardless of the ordering (even when the frames got re-ordered
by the network) and also when a frame didn't even arrive yet
(rendering the jitter buffer almost useless).

This commit stores the compressed frame and the timeslot mask in the
RIFO and only tries to decompress the frames in the RIFO pop logic
(when we had time to receive the other frames before it).

The rifo->last_in_fn logic would also reset the counter back to an
older frame even when a frame with a higher number was already
received (fixed now).

Change-Id: I66af4f01ebb38ca1bdd8ffbc8135776322941230
---
M src/octoi/e1oip.c
M src/octoi/e1oip.h
M src/octoi/frame_rifo.c
M src/octoi/frame_rifo.h
M src/octoi/octoi.c
M tests/rifo/rifo_test.c
M tests/rifo/rifo_test.ok
7 files changed, 191 insertions(+), 37 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-e1d refs/changes/43/43643/7

To view, visit change 43643. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-e1d
Gerrit-Branch: master
Gerrit-Change-Id: I66af4f01ebb38ca1bdd8ffbc8135776322941230
Gerrit-Change-Number: 43643
Gerrit-PatchSet: 7
Gerrit-Owner: manawyrm <osmocom.account@tbspace.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: tnt <tnt@246tNt.com>
Gerrit-Attention: manawyrm <osmocom.account@tbspace.de>
Gerrit-Attention: laforge <laforge@osmocom.org>
Gerrit-Attention: tnt <tnt@246tNt.com>
Gerrit-Attention: pespin <pespin@sysmocom.de>