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

Limit upload speed of firmware image #3

Merged
merged 1 commit into from
May 13, 2024

Conversation

dbahrdt
Copy link
Collaborator

@dbahrdt dbahrdt commented May 13, 2024

The BR62 has problems to process incoming packets fast enough. To mitigate this, it sends a lot of tcp window updates to indicate that the receive window is full.
However this somehow interacts with the upload of the firmware image which then fails on many occasions. As a temporary workaround we limit the upload speed to a safe value such that the BR62 is able to handle all incoming packets.

The BR62 has problems to process incoming packets fast enough.
To mitigate this, it sends a lot of tcp window updates to indicate that
the receive window is full.
However this somehow interacts with the upload of the firmware image
which then fails on many occasions. As a temporary workaround we limit
the upload speed to a safe value such that the BR62 is able to handle
all incoming packets.
@dbahrdt dbahrdt requested a review from ItRainsSmiles May 13, 2024 09:41
@ItRainsSmiles
Copy link

Nochmal mit allen HAP devices getestet - löst das Problem =)!

@dbahrdt dbahrdt merged commit 054cbd1 into develop May 13, 2024
@dbahrdt dbahrdt deleted the feature/rate-limit-image-upload branch May 13, 2024 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants