Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FR: Suppress log @158858584:ACKACK out of order, skipping RTT calculation (ACK number: 2960, last ACK sent: 2972, RTT (EWMA): 244797 #2739

Closed
bander-saeed94 opened this issue May 16, 2023 · 4 comments · Fixed by #3007
Labels
[core] Area: Changes in SRT library core Type: Enhancement Indicates new feature requests
Milestone

Comments

@bander-saeed94
Copy link

hi,

I was running the srt application locally and it is working fine.
I have deploy it to cloud and start streaming and this is what I got:

22:35:07.548905/SRT:RcvQ:w1!W:SRT.in: @158858584:ACKACK out of order, skipping RTT calculation (ACK number: 2960, last ACK sent: 2972, RTT (EWMA): 244797

I am streaming with OBS

srt://IP:PORT?streamid=#!::r=livestream3,m=publish,t=stream&transtype=live&mode=caller&latency=1000000
@bander-saeed94 bander-saeed94 added the Type: Question Questions or things that require clarification label May 16, 2023
@ethouris
Copy link
Collaborator

If this appears occasionally, that's not a big deal. The problem would be if you get flooded with this kind of warning.

This is in the mechanism used to calculate RTT, where the receiver party sends ACK and saves a record with the journal number for it. When the sender party sends ACKACK with the same journal number (it's this "ACK number" you can see in the log), this record is found and times compared for RTT calculation. If this record cannot be found, this warning is reported.

@maxsharabayko maxsharabayko closed this as not planned Won't fix, can't repro, duplicate, stale Jun 5, 2023
@gregfr
Copy link

gregfr commented Dec 18, 2023

Hello
What if the warning appears a lot? is there anything we can do?
Thanks in advance
Regards

@maxsharabayko
Copy link
Collaborator

Add the log message to the suppression mechanism.
See #2733

@maxsharabayko maxsharabayko added this to the v1.6.0 milestone Dec 20, 2023
@maxsharabayko maxsharabayko added Type: Enhancement Indicates new feature requests [core] Area: Changes in SRT library core and removed Type: Question Questions or things that require clarification labels Dec 20, 2023
@maxsharabayko maxsharabayko changed the title @158858584:ACKACK out of order, skipping RTT calculation (ACK number: 2960, last ACK sent: 2972, RTT (EWMA): 244797 FR: Suppress log @158858584:ACKACK out of order, skipping RTT calculation (ACK number: 2960, last ACK sent: 2972, RTT (EWMA): 244797 Dec 20, 2023
@maxsharabayko maxsharabayko reopened this Dec 20, 2023
@RadegasTCZ
Copy link

I happen to run into this while bonding with the help of BELABOX's STRLA relay, it gets spammed real fast while second connection is added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[core] Area: Changes in SRT library core Type: Enhancement Indicates new feature requests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants