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

[OTA] Set MRP Active Retry interval for ota-provider-app on Linux #20613

Merged
merged 1 commit into from
Jul 12, 2022

Conversation

woody-apple
Copy link
Contributor

Problem

Currently the ota-provider-app uses the default value of 300ms for MRP Active Retry interval. This value is too low for Thread networks where a packet/ACK round-trip time routinely exceeds 300ms. This results in hundreds of re-transmissions during OTA downloads.

Change overview

Set MRP Active Retry interval for ota-provider-app on Linux to 2000ms

Testing

Tested OTA Software update with an EFR32 BRD4161 in the following network configuration:
ota-provider-app(Linux)---ethernet---OTBR---thread---EndDevice
The number of re-transmissions went from an average of 1050 to 8.

…0570)

* Test added march 8 (#15957)

* Added new manual scripts

* Added Auto generated File

* [OTA] Fix OTARequestorDriverImpl inclusion (#15981)

* Regen to fix CI failures (#15990)

* [ota] Store Default OTA Providers in flash (#15970)

* [ota] Store Default OTA Providers in flash

Store Default OTA Providers in flash each time the attribute
is modified and load it back on the application startup.

* Restyled by clang-format

* Fix build and reduce flash usage

Co-authored-by: Restyled.io <commits@restyled.io>

* Set MRP Active Retry interval in the Linux ota-provider-app config

* Remove merge artifacts

* Restyled by whitespace

Co-authored-by: kowsisoundhar12 <57476670+kowsisoundhar12@users.noreply.github.com>
Co-authored-by: Carol Yang <clyang@apple.com>
Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>
Co-authored-by: Damian Królik <66667989+Damian-Nordic@users.noreply.github.com>
Co-authored-by: Restyled.io <commits@restyled.io>
@github-actions
Copy link

PR #20613: Size comparison from 7037068 to 2d8a006

Full report (2 builds for nrfconnect)
platform target config section 7037068 2d8a006 change % change
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1174903 1174903 0 0.0
bss 142900 142900 0 0.0
rodata 141808 141808 0 0.0
text 811300 811300 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1155099 1155099 0 0.0
bss 142136 142136 0 0.0
rodata 133340 133340 0 0.0
text 800744 800744 0 0.0

@andy31415 andy31415 merged commit 39acdbf into sve Jul 12, 2022
@andy31415 andy31415 deleted the cherry-pick-f68f7e5a1e296bd4288d4ce85c0563df1c797759 branch July 12, 2022 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants