tree 01e5693e21362087e9141cdae42e550bca8c020a
parent 8c090a80f3cd8928420f99bec145a2c523e33f53
author Alexei Frolov <frolv@pigweed.infra.roller.google.com> 1714155270 +0000
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> 1714155270 +0000

[roll third_party/pigweed] pw_transfer: Assume a minimum window size when reserving space

A pw_transfer receiver estimates how much data the sender can fit in a
chunk by trying to calculate the protobuf overhead of transfer metadata.
On the first chunk of a transfer, this calculation is performed before
a window size is set (as setting a window size requires a max chunk
size), potentially resulting in a larger initial chunk size.

This caused issues in the case of retries, as a retried chunk could
advertise a smaller (correct) window size than the initial chunk,
invalidating any in-flight data transmissions.

This updates the code to assume a minimum window size of one chunk when
making its initial overhead calculation.

Original-Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/206890

https://pigweed.googlesource.com/pigweed/pigweed
third_party/pigweed Rolled-Commits: 2593707a02b0c21..87126e11b80d206
Roller-URL: https://ci.chromium.org/b/8749572266749188753
GitWatcher: ignore
CQ-Do-Not-Cancel-Tryjobs: true
Change-Id: I8ddedeb1fb0c01013949c76e42413a58df16cb08
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/examples/+/206910
Bot-Commit: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
Lint: Lint 🤖 <android-build-ayeaye@system.gserviceaccount.com>
Commit-Queue: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>