manawyrm uploaded patch set #2 to this change.

View Change

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, 195 insertions(+), 39 deletions(-)

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

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: 2
Gerrit-Owner: manawyrm <osmocom.account@tbspace.de>
Gerrit-CC: Jenkins Builder