-
Notifications
You must be signed in to change notification settings - Fork 2k
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
gnrc_sixlowpan_frag_sfr_congure: add congure_abe support #16171
gnrc_sixlowpan_frag_sfr_congure: add congure_abe support #16171
Conversation
22fd6f4
to
75bc954
Compare
75bc954
to
cbb9711
Compare
Rebased and squashed to current master. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions. |
cbb9711
to
c48a547
Compare
Rebased and adopted for current master and dependencies, needs #18798 though, for the test application to link. |
Needs a rebase |
c48a547
to
26ab70a
Compare
Rebased to current master and #16170. |
26ab70a
to
690df90
Compare
Rebased again, to resolve merge conflicts caused by merging #16159 |
690df90
to
1cff487
Compare
The last rebase. |
bors merge |
Build succeeded: |
Finally the CC support for SFR is fully merged 🎉 Thanks @benpicco for reviewing! |
Contribution description
This provisions GNRC's 6LoWPAN SFR implementation with the CongURE implementation of TCP Reno. It also adds that implementation to the
tests/gnrc_sixlowpan_frag_sfr_congure_impl
test application.Testing procedure
The test
tests/gnrc_sixlowpan_frag_sfr_congure_impl
where extended to supportcongure_abe
:Ping some nodes with at least 1 hop separation with large payloads. Exchange between should work.
Running the tests with
CONGURE_IMPL=congure_reno
also should still work.Issues/PRs references
Depends on
#15968(merged) and #16170 and their respective dependencies.