RTT is miscalculated if ACKs are disordered
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 48/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Stale
- Tech stack
- cpp
- Domain
- networking
Research direction
Start in ScreamV2Tx.cpp and inspect where currRtt is updated when ACKs arrive. Exercise the RTT calculation with disordered ACK sequence numbers; the issue is done when currRtt updates only for ACKed sequence numbers that increase and detailed logs no longer show confusing RTT variation.
Written by the indexing model from the issue text.
Description
Disordered ACKs cause RTT values to vary unexpectedly. While this is not exactly wrong, it can give confusing readouts in especially the detailed logs. The suggested fix is to only update currRtt in ScreamV2Tx.cpp when the ACKed sequence number increases.
- Dominant language
- Shell
- Stars
- 219
- Forks
- 69
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from EricssonResearch/scream
-
Difficulty 3/5 1-2 days Newbie friendliness 35/100
EricssonResearch/scream#69 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
EricssonResearch/scream#55 · 3 comments ·
-
build on WindowsOpen
Difficulty 4/5 3-5 days Newbie friendliness 25/100
EricssonResearch/scream#50 · 13 comments ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
EricssonResearch/scream#48 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
EricssonResearch/scream#45 · 4 comments ·
All issues in EricssonResearch/scream
Similar issues
-
afk-ok area:agents size:S
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
enorm-labs/event-junkie#3523 ·
Maintainers usually reply within 1 day
-
Container fails to start when a mounted neo4j.conf uses spaces around `=`Possibly taken @RaphaelFakhri claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
neo4j/docker-neo4j#596 ·
-
agent/quality help wanted origin/quality quality stage/010-actionable testing
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
hivecommons/hotshot#227 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
pterodactyl/game-eggs#657 ·
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
sleeyax/promptfiles#20 ·