Open nspring opened 3 years ago
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it!
and we'll verify it.
ℹ️ Googlers: Go here for more info.
When I look at:
https://github.com/google/packetdrill/pull/56/commits
it says:
nspring wants to merge 3 commits into google:master from nspring:window_clamp
Can you please squash/fixup the 3 commits into a single commit? Thanks!
Gotcha - commits squashed, file/directory renamed, and a bit more test code added to exercise a scenario Eric Dumazet pointed out in the discussion on the kernel patch. I'm not quite sure the patch is perfect yet; maybe it's too conservative when raising the clamp.
I'm really happy you thought the test was good. I wonder if you can help me with two things though.
Still tuning receive windows after all these years!
Low hanging fruit! Nice to be able to work together (even if indirectly) again; this is a sweet project.
Adds a test of TCP_WINDOW_CLAMP, including lowering, raising, and lowering again the clamp while the connection is in progress.
Requires a patch to tcp_set_window_clamp, since prior linux ignores this field unless set before the connection exchanges data.
Signed-off-by: Neil Spring ntspring@fb.com