Hi Mychaela,
sorry for the late response...
On Thu, Mar 09, 2023 at 06:30:43PM -0800, Mychaela Falconia wrote:
Adding the just-described SID logic to osmo_rtp2trau() would be quite easy:
- Add a function to libosmocore that would be just like
osmo_fr_check_sid(), but for EFR;
- Add the SID logic to osmo_rtp2trau() in libosmo-abis, using
osmo_fr_check_sid() for FR1 and the new function for EFR.
I would be happy to submit Gerrit patches doing 1 and 2 above, *iff* there is at least one person on Earth (OP or anyone else) who would find them useful.
Thanks for your offer.
I hereby declare that I am a person on Earth and would find it useful.
- In the DL direction, I do see a problem in osmo_rtp2trau() that can
be fixed with a patch. GSM 08.60 sections 4.5.2 and 4.5.3 say: "If no speech is received from the MSC side of the interface (downlink direction), idle speech frames shall be transferred instead of speech frames." - however, the code in osmo_rtp2trau() fails to obey this requirement, instead it sends TRAU speech frames with garbage payload when the rtp_len argument to the function is 0, indicating missing speech data. Once again, I would be happy to produce a patch fixing this bug if someone (OP or anyone else) would like to see one.
Likewise, I would like to see a patch for that.
Thanks, Harald