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

[EFR32] OTA: Use word-aligned buffer in bootloader storage APIs #17281

Merged
merged 31 commits into from
Apr 13, 2022

Conversation

selissia
Copy link
Contributor

@selissia selissia commented Apr 12, 2022

Problem

EFR32 bootloader storage API must use word-aligned write calls and be wrapped into a critical section.

Change overview

  • Use word-aligned buffer and critical section in calls to the EFR32 bootloader storage APIs
  • Update documentation

Testing

Tested OTA scenario on EFR32 MG12 and MG24 boards

@boring-cyborg boring-cyborg bot added the documentation Improvements or additions to documentation label Apr 12, 2022
@selissia selissia changed the title Work In Progress, please ignore [EFR32] OTA: Use word-aligned buffer in bootloader storage APIs Apr 12, 2022
@selissia selissia marked this pull request as ready for review April 12, 2022 15:59
@pullapprove pullapprove bot requested a review from andy31415 April 12, 2022 15:59
@github-actions
Copy link

github-actions bot commented Apr 12, 2022

PR #17281: Size comparison from 2c15978 to 2714cbe

Increases (2 builds for efr32)
platform target config section 2c15978 2714cbe change % change
efr32 lighting-app BRD4161A (read only) 907004 907340 336 0.0
(read/write) 133080 133144 64 0.0
.bss 131040 131104 64 0.0
.text 906996 907332 336 0.0
BRD4161A+rpc (read only) 941372 941708 336 0.0
(read/write) 149764 149828 64 0.0
.bss 147520 147584 64 0.0
.text 941364 941700 336 0.0
Full report (31 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 2c15978 2714cbe change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read only) 639911 639911 0 0.0
(read/write) 151212 151212 0 0.0
.bss 74144 74144 0 0.0
.data 3212 3212 0 0.0
.rodata 80143 80143 0 0.0
.text 559276 559276 0 0.0
lock-mtd LP_CC2652R7 (read only) 588655 588655 0 0.0
(read/write) 146932 146932 0 0.0
.bss 69864 69864 0 0.0
.data 3212 3212 0 0.0
.rodata 80023 80023 0 0.0
.text 508140 508140 0 0.0
pump-app LP_CC2652R7 (read only) 647995 647995 0 0.0
(read/write) 152508 152508 0 0.0
.bss 74640 74640 0 0.0
.data 3244 3244 0 0.0
.rodata 75331 75331 0 0.0
.text 572176 572176 0 0.0
pump-controller-app LP_CC2652R7 (read only) 641967 641967 0 0.0
(read/write) 152176 152176 0 0.0
.bss 74344 74344 0 0.0
.data 3208 3208 0 0.0
.rodata 78967 78967 0 0.0
.text 562512 562512 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 618514 618514 0 0.0
.app_xip_area 525224 525224 0 0.0
.bss 75956 75956 0 0.0
.data 684 684 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 576126 576126 0 0.0
.app_xip_area 484372 484372 0 0.0
.bss 74452 74452 0 0.0
.data 648 648 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 564642 564642 0 0.0
.app_xip_area 463260 463260 0 0.0
.bss 83784 83784 0 0.0
.data 564 564 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 907004 907340 336 0.0
(read/write) 133080 133144 64 0.0
.bss 131040 131104 64 0.0
.data 2040 2040 0 0.0
.text 906996 907332 336 0.0
BRD4161A+rpc (read only) 941372 941708 336 0.0
(read/write) 149764 149828 64 0.0
.bss 147520 147584 64 0.0
.data 2244 2244 0 0.0
.text 941364 941700 336 0.0
window-app BRD4161A (read only) 843996 843996 0 0.0
(read/write) 131148 131148 0 0.0
.bss 129200 129200 0 0.0
.data 1948 1948 0 0.0
.text 843988 843988 0 0.0
esp32 all-clusters-app c3devkit (read only) 979666 979666 0 0.0
(read/write) 1397514 1397514 0 0.0
.dram0.bss 62624 62624 0 0.0
.dram0.data 14420 14420 0 0.0
.flash.rodata 201552 201552 0 0.0
.flash.text 979666 979666 0 0.0
.iram0.text 62016 62016 0 0.0
m5stack (read only) 1035255 1035255 0 0.0
(read/write) 465252 465252 0 0.0
.dram0.bss 68144 68144 0 0.0
.dram0.data 34152 34152 0 0.0
.flash.rodata 231120 231120 0 0.0
.flash.text 1029871 1029871 0 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 686728 686728 0 0.0
.bss 78128 78128 0 0.0
.data 2036 2036 0 0.0
.text 600764 600764 0 0.0
lock k32w061+release (read/write) 691360 691360 0 0.0
.bss 78704 78704 0 0.0
.data 1996 1996 0 0.0
.text 604860 604860 0 0.0
linux all-clusters-app debug (read only) 2696353 2696353 0 0.0
(read/write) 149120 149120 0 0.0
.bss 60192 60192 0 0.0
.data 1888 1888 0 0.0
.data.rel.ro 80984 80984 0 0.0
.dynamic 608 608 0 0.0
.got 4456 4456 0 0.0
.init 27 27 0 0.0
.init_array 984 984 0 0.0
.rodata 232421 232421 0 0.0
.text 2289938 2289938 0 0.0
bridge-app debug+rpc (read only) 1835933 1835933 0 0.0
(read/write) 91752 91752 0 0.0
.bss 44472 44472 0 0.0
.data 2912 2912 0 0.0
.data.rel.ro 39280 39280 0 0.0
.dynamic 592 592 0 0.0
.got 3936 3936 0 0.0
.init 27 27 0 0.0
.init_array 552 552 0 0.0
.rodata 148473 148473 0 0.0
.text 1569029 1569029 0 0.0
chip-tool debug (read only) 10676501 10676501 0 0.0
(read/write) 371832 371832 0 0.0
.bss 22752 22752 0 0.0
.data 1104 1104 0 0.0
.data.rel.ro 341728 341728 0 0.0
.dynamic 624 624 0 0.0
.got 4936 4936 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 537973 537973 0 0.0
.text 9309221 9309221 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 1027722 1027722 0 0.0
(read/write) 492193 492193 0 0.0
.bss 41009 41009 0 0.0
.data 1168 1168 0 0.0
.data.rel.ro 388816 388816 0 0.0
.dynamic 560 560 0 0.0
.got 57392 57392 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 512228 512228 0 0.0
.text 8661732 8661732 0 0.0
door-lock-app debug (read only) 2107313 2107313 0 0.0
(read/write) 119504 119504 0 0.0
.bss 48064 48064 0 0.0
.data 1472 1472 0 0.0
.data.rel.ro 64408 64408 0 0.0
.dynamic 592 592 0 0.0
.got 4264 4264 0 0.0
.init 27 27 0 0.0
.init_array 680 680 0 0.0
.rodata 186697 186697 0 0.0
.text 1764914 1764914 0 0.0
lighting-app debug+rpc (read only) 2312137 2312137 0 0.0
(read/write) 127824 127824 0 0.0
.bss 50272 50272 0 0.0
.data 1952 1952 0 0.0
.data.rel.ro 69896 69896 0 0.0
.dynamic 608 608 0 0.0
.got 4312 4312 0 0.0
.init 27 27 0 0.0
.init_array 776 776 0 0.0
.rodata 182633 182633 0 0.0
.text 1963666 1963666 0 0.0
ota-provider-app debug (read only) 2044465 2044465 0 0.0
(read/write) 114944 114944 0 0.0
.bss 47968 47968 0 0.0
.data 1608 1608 0 0.0
.data.rel.ro 59624 59624 0 0.0
.dynamic 608 608 0 0.0
.got 4464 4464 0 0.0
.init 27 27 0 0.0
.init_array 632 632 0 0.0
.rodata 172451 172451 0 0.0
.text 1715826 1715826 0 0.0
ota-requestor-app debug (read only) 2074873 2074873 0 0.0
(read/write) 118264 118264 0 0.0
.bss 48928 48928 0 0.0
.data 1864 1864 0 0.0
.data.rel.ro 61864 61864 0 0.0
.dynamic 592 592 0 0.0
.got 4304 4304 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 169356 169356 0 0.0
.text 1747458 1747458 0 0.0
shell debug (read only) 2524145 2524145 0 0.0
(read/write) 149968 149968 0 0.0
.bss 67624 67624 0 0.0
.data 1264 1264 0 0.0
.data.rel.ro 75368 75368 0 0.0
.dynamic 592 592 0 0.0
.got 4168 4168 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 214674 214674 0 0.0
.text 2148306 2148306 0 0.0
thermostat-no-ble arm64 (read only) 2352700 2352700 0 0.0
(read/write) 150993 150993 0 0.0
.bss 63153 63153 0 0.0
.data 1424 1424 0 0.0
.data.rel.ro 78632 78632 0 0.0
.dynamic 560 560 0 0.0
.got 4768 4768 0 0.0
.init 24 24 0 0.0
.init_array 368 368 0 0.0
.rodata 144668 144668 0 0.0
.text 1979456 1979456 0 0.0
tv-app debug (read only) 2795649 2795649 0 0.0
(read/write) 250680 250680 0 0.0
.bss 164104 164104 0 0.0
.data 4448 4448 0 0.0
.data.rel.ro 75912 75912 0 0.0
.dynamic 592 592 0 0.0
.got 4688 4688 0 0.0
.init 27 27 0 0.0
.init_array 904 904 0 0.0
.rodata 213259 213259 0 0.0
.text 2401090 2401090 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2368260 2368260 0 0.0
.bss 185236 185236 0 0.0
.data 5840 5840 0 0.0
.text 1330860 1330860 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1155895 1155895 0 0.0
bss 135464 135464 0 0.0
rodata 146768 146768 0 0.0
text 794976 794976 0 0.0
p6 all-clusters-app default (read/write) 2514688 2514688 0 0.0
.bss 118640 118640 0 0.0
.data 2768 2768 0 0.0
.text 1472952 1472952 0 0.0
light-app default (read/write) 2415160 2415160 0 0.0
.bss 112136 112136 0 0.0
.data 2576 2576 0 0.0
.text 1373424 1373424 0 0.0
lock-app default (read/write) 2378712 2378712 0 0.0
.bss 111880 111880 0 0.0
.data 2536 2536 0 0.0
.text 1336976 1336976 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 800992 800992 0 0.0
bss 69988 69988 0 0.0
noinit 40416 40416 0 0.0
text 569690 569690 0 0.0

@selissia selissia requested a review from andy31415 April 12, 2022 19:48
@github-actions
Copy link

github-actions bot commented Apr 12, 2022

PR #17281: Size comparison from 2e33dec to f8f961f

Increases (2 builds for efr32)
platform target config section 2e33dec f8f961f change % change
efr32 lighting-app BRD4161A (read only) 907004 907340 336 0.0
(read/write) 133080 133144 64 0.0
.bss 131040 131104 64 0.0
.text 906996 907332 336 0.0
BRD4161A+rpc (read only) 941372 941708 336 0.0
(read/write) 149764 149828 64 0.0
.bss 147520 147584 64 0.0
.text 941364 941700 336 0.0
Full report (31 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 2e33dec f8f961f change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read only) 639911 639911 0 0.0
(read/write) 151212 151212 0 0.0
.bss 74144 74144 0 0.0
.data 3212 3212 0 0.0
.rodata 80143 80143 0 0.0
.text 559276 559276 0 0.0
lock-mtd LP_CC2652R7 (read only) 588655 588655 0 0.0
(read/write) 146932 146932 0 0.0
.bss 69864 69864 0 0.0
.data 3212 3212 0 0.0
.rodata 80023 80023 0 0.0
.text 508140 508140 0 0.0
pump-app LP_CC2652R7 (read only) 647995 647995 0 0.0
(read/write) 152508 152508 0 0.0
.bss 74640 74640 0 0.0
.data 3244 3244 0 0.0
.rodata 75331 75331 0 0.0
.text 572176 572176 0 0.0
pump-controller-app LP_CC2652R7 (read only) 641967 641967 0 0.0
(read/write) 152176 152176 0 0.0
.bss 74344 74344 0 0.0
.data 3208 3208 0 0.0
.rodata 78967 78967 0 0.0
.text 562512 562512 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 618514 618514 0 0.0
.app_xip_area 525224 525224 0 0.0
.bss 75956 75956 0 0.0
.data 684 684 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 576126 576126 0 0.0
.app_xip_area 484372 484372 0 0.0
.bss 74452 74452 0 0.0
.data 648 648 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 564642 564642 0 0.0
.app_xip_area 463260 463260 0 0.0
.bss 83784 83784 0 0.0
.data 564 564 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 907004 907340 336 0.0
(read/write) 133080 133144 64 0.0
.bss 131040 131104 64 0.0
.data 2040 2040 0 0.0
.text 906996 907332 336 0.0
BRD4161A+rpc (read only) 941372 941708 336 0.0
(read/write) 149764 149828 64 0.0
.bss 147520 147584 64 0.0
.data 2244 2244 0 0.0
.text 941364 941700 336 0.0
window-app BRD4161A (read only) 843996 843996 0 0.0
(read/write) 131148 131148 0 0.0
.bss 129200 129200 0 0.0
.data 1948 1948 0 0.0
.text 843988 843988 0 0.0
esp32 all-clusters-app c3devkit (read only) 979666 979666 0 0.0
(read/write) 1397514 1397514 0 0.0
.dram0.bss 62624 62624 0 0.0
.dram0.data 14420 14420 0 0.0
.flash.rodata 201552 201552 0 0.0
.flash.text 979666 979666 0 0.0
.iram0.text 62016 62016 0 0.0
m5stack (read only) 1035255 1035255 0 0.0
(read/write) 465252 465252 0 0.0
.dram0.bss 68144 68144 0 0.0
.dram0.data 34152 34152 0 0.0
.flash.rodata 231120 231120 0 0.0
.flash.text 1029871 1029871 0 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 686728 686728 0 0.0
.bss 78128 78128 0 0.0
.data 2036 2036 0 0.0
.text 600764 600764 0 0.0
lock k32w061+release (read/write) 691360 691360 0 0.0
.bss 78704 78704 0 0.0
.data 1996 1996 0 0.0
.text 604860 604860 0 0.0
linux all-clusters-app debug (read only) 2696353 2696353 0 0.0
(read/write) 149120 149120 0 0.0
.bss 60192 60192 0 0.0
.data 1888 1888 0 0.0
.data.rel.ro 80984 80984 0 0.0
.dynamic 608 608 0 0.0
.got 4456 4456 0 0.0
.init 27 27 0 0.0
.init_array 984 984 0 0.0
.rodata 232421 232421 0 0.0
.text 2289938 2289938 0 0.0
bridge-app debug+rpc (read only) 1835933 1835933 0 0.0
(read/write) 91752 91752 0 0.0
.bss 44472 44472 0 0.0
.data 2912 2912 0 0.0
.data.rel.ro 39280 39280 0 0.0
.dynamic 592 592 0 0.0
.got 3936 3936 0 0.0
.init 27 27 0 0.0
.init_array 552 552 0 0.0
.rodata 148473 148473 0 0.0
.text 1569029 1569029 0 0.0
chip-tool debug (read only) 10676501 10676501 0 0.0
(read/write) 371832 371832 0 0.0
.bss 22752 22752 0 0.0
.data 1104 1104 0 0.0
.data.rel.ro 341728 341728 0 0.0
.dynamic 624 624 0 0.0
.got 4936 4936 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 537973 537973 0 0.0
.text 9309221 9309221 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 1027722 1027722 0 0.0
(read/write) 492193 492193 0 0.0
.bss 41009 41009 0 0.0
.data 1168 1168 0 0.0
.data.rel.ro 388816 388816 0 0.0
.dynamic 560 560 0 0.0
.got 57392 57392 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 512228 512228 0 0.0
.text 8661732 8661732 0 0.0
door-lock-app debug (read only) 2107313 2107313 0 0.0
(read/write) 119504 119504 0 0.0
.bss 48064 48064 0 0.0
.data 1472 1472 0 0.0
.data.rel.ro 64408 64408 0 0.0
.dynamic 592 592 0 0.0
.got 4264 4264 0 0.0
.init 27 27 0 0.0
.init_array 680 680 0 0.0
.rodata 186697 186697 0 0.0
.text 1764914 1764914 0 0.0
lighting-app debug+rpc (read only) 2312137 2312137 0 0.0
(read/write) 127824 127824 0 0.0
.bss 50272 50272 0 0.0
.data 1952 1952 0 0.0
.data.rel.ro 69896 69896 0 0.0
.dynamic 608 608 0 0.0
.got 4312 4312 0 0.0
.init 27 27 0 0.0
.init_array 776 776 0 0.0
.rodata 182633 182633 0 0.0
.text 1963666 1963666 0 0.0
ota-provider-app debug (read only) 2044465 2044465 0 0.0
(read/write) 114944 114944 0 0.0
.bss 47968 47968 0 0.0
.data 1608 1608 0 0.0
.data.rel.ro 59624 59624 0 0.0
.dynamic 608 608 0 0.0
.got 4464 4464 0 0.0
.init 27 27 0 0.0
.init_array 632 632 0 0.0
.rodata 172451 172451 0 0.0
.text 1715826 1715826 0 0.0
ota-requestor-app debug (read only) 2074873 2074873 0 0.0
(read/write) 118264 118264 0 0.0
.bss 48928 48928 0 0.0
.data 1864 1864 0 0.0
.data.rel.ro 61864 61864 0 0.0
.dynamic 592 592 0 0.0
.got 4304 4304 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 169356 169356 0 0.0
.text 1747458 1747458 0 0.0
shell debug (read only) 2524145 2524145 0 0.0
(read/write) 149968 149968 0 0.0
.bss 67624 67624 0 0.0
.data 1264 1264 0 0.0
.data.rel.ro 75368 75368 0 0.0
.dynamic 592 592 0 0.0
.got 4168 4168 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 214674 214674 0 0.0
.text 2148306 2148306 0 0.0
thermostat-no-ble arm64 (read only) 2352700 2352700 0 0.0
(read/write) 150993 150993 0 0.0
.bss 63153 63153 0 0.0
.data 1424 1424 0 0.0
.data.rel.ro 78632 78632 0 0.0
.dynamic 560 560 0 0.0
.got 4768 4768 0 0.0
.init 24 24 0 0.0
.init_array 368 368 0 0.0
.rodata 144668 144668 0 0.0
.text 1979456 1979456 0 0.0
tv-app debug (read only) 2795649 2795649 0 0.0
(read/write) 250680 250680 0 0.0
.bss 164104 164104 0 0.0
.data 4448 4448 0 0.0
.data.rel.ro 75912 75912 0 0.0
.dynamic 592 592 0 0.0
.got 4688 4688 0 0.0
.init 27 27 0 0.0
.init_array 904 904 0 0.0
.rodata 213259 213259 0 0.0
.text 2401090 2401090 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2368260 2368260 0 0.0
.bss 185236 185236 0 0.0
.data 5840 5840 0 0.0
.text 1330860 1330860 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1155895 1155895 0 0.0
bss 135464 135464 0 0.0
rodata 146768 146768 0 0.0
text 794976 794976 0 0.0
p6 all-clusters-app default (read/write) 2514688 2514688 0 0.0
.bss 118640 118640 0 0.0
.data 2768 2768 0 0.0
.text 1472952 1472952 0 0.0
light-app default (read/write) 2415160 2415160 0 0.0
.bss 112136 112136 0 0.0
.data 2576 2576 0 0.0
.text 1373424 1373424 0 0.0
lock-app default (read/write) 2378712 2378712 0 0.0
.bss 111880 111880 0 0.0
.data 2536 2536 0 0.0
.text 1336976 1336976 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 800992 800992 0 0.0
bss 69988 69988 0 0.0
noinit 40416 40416 0 0.0
text 569690 569690 0 0.0

@github-actions
Copy link

github-actions bot commented Apr 13, 2022

PR #17281: Size comparison from faabdf9 to 86310c3

Increases (2 builds for efr32)
platform target config section faabdf9 86310c3 change % change
efr32 lighting-app BRD4161A (read only) 907300 907636 336 0.0
(read/write) 133080 133144 64 0.0
.bss 131040 131104 64 0.0
.text 907292 907628 336 0.0
BRD4161A+rpc (read only) 941668 942004 336 0.0
(read/write) 149764 149828 64 0.0
.bss 147520 147584 64 0.0
.text 941660 941996 336 0.0
Full report (31 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section faabdf9 86310c3 change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read only) 640211 640211 0 0.0
(read/write) 151212 151212 0 0.0
.bss 74144 74144 0 0.0
.data 3212 3212 0 0.0
.rodata 80163 80163 0 0.0
.text 559556 559556 0 0.0
lock-mtd LP_CC2652R7 (read only) 588947 588947 0 0.0
(read/write) 146932 146932 0 0.0
.bss 69864 69864 0 0.0
.data 3212 3212 0 0.0
.rodata 80043 80043 0 0.0
.text 508412 508412 0 0.0
pump-app LP_CC2652R7 (read only) 648287 648287 0 0.0
(read/write) 152508 152508 0 0.0
.bss 74640 74640 0 0.0
.data 3244 3244 0 0.0
.rodata 75351 75351 0 0.0
.text 572448 572448 0 0.0
pump-controller-app LP_CC2652R7 (read only) 642267 642267 0 0.0
(read/write) 152176 152176 0 0.0
.bss 74344 74344 0 0.0
.data 3208 3208 0 0.0
.rodata 78987 78987 0 0.0
.text 562792 562792 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 618818 618818 0 0.0
.app_xip_area 525528 525528 0 0.0
.bss 75956 75956 0 0.0
.data 684 684 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 576414 576414 0 0.0
.app_xip_area 484660 484660 0 0.0
.bss 74452 74452 0 0.0
.data 648 648 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 564938 564938 0 0.0
.app_xip_area 463556 463556 0 0.0
.bss 83784 83784 0 0.0
.data 564 564 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 907300 907636 336 0.0
(read/write) 133080 133144 64 0.0
.bss 131040 131104 64 0.0
.data 2040 2040 0 0.0
.text 907292 907628 336 0.0
BRD4161A+rpc (read only) 941668 942004 336 0.0
(read/write) 149764 149828 64 0.0
.bss 147520 147584 64 0.0
.data 2244 2244 0 0.0
.text 941660 941996 336 0.0
window-app BRD4161A (read only) 844292 844292 0 0.0
(read/write) 131148 131148 0 0.0
.bss 129200 129200 0 0.0
.data 1948 1948 0 0.0
.text 844284 844284 0 0.0
esp32 all-clusters-app c3devkit (read only) 979702 979702 0 0.0
(read/write) 1397514 1397514 0 0.0
.dram0.bss 62624 62624 0 0.0
.dram0.data 14420 14420 0 0.0
.flash.rodata 201552 201552 0 0.0
.flash.text 979702 979702 0 0.0
.iram0.text 62016 62016 0 0.0
m5stack (read only) 1035283 1035283 0 0.0
(read/write) 465252 465252 0 0.0
.dram0.bss 68144 68144 0 0.0
.dram0.data 34152 34152 0 0.0
.flash.rodata 231120 231120 0 0.0
.flash.text 1029899 1029899 0 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 687036 687036 0 0.0
.bss 78128 78128 0 0.0
.data 2036 2036 0 0.0
.text 601072 601072 0 0.0
lock k32w061+release (read/write) 691652 691652 0 0.0
.bss 78704 78704 0 0.0
.data 1996 1996 0 0.0
.text 605152 605152 0 0.0
linux all-clusters-app debug (read only) 2696513 2696513 0 0.0
(read/write) 149120 149120 0 0.0
.bss 60192 60192 0 0.0
.data 1888 1888 0 0.0
.data.rel.ro 80984 80984 0 0.0
.dynamic 608 608 0 0.0
.got 4456 4456 0 0.0
.init 27 27 0 0.0
.init_array 984 984 0 0.0
.rodata 232421 232421 0 0.0
.text 2290098 2290098 0 0.0
bridge-app debug+rpc (read only) 1836109 1836109 0 0.0
(read/write) 91760 91760 0 0.0
.bss 44480 44480 0 0.0
.data 2912 2912 0 0.0
.data.rel.ro 39280 39280 0 0.0
.dynamic 592 592 0 0.0
.got 3936 3936 0 0.0
.init 27 27 0 0.0
.init_array 552 552 0 0.0
.rodata 148473 148473 0 0.0
.text 1569205 1569205 0 0.0
chip-tool debug (read only) 10676677 10676677 0 0.0
(read/write) 371832 371832 0 0.0
.bss 22752 22752 0 0.0
.data 1104 1104 0 0.0
.data.rel.ro 341728 341728 0 0.0
.dynamic 624 624 0 0.0
.got 4936 4936 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 538005 538005 0 0.0
.text 9309365 9309365 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 10277348 10277348 0 0.0
(read/write) 492209 492209 0 0.0
.bss 41025 41025 0 0.0
.data 1168 1168 0 0.0
.data.rel.ro 388816 388816 0 0.0
.dynamic 560 560 0 0.0
.got 57392 57392 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 512244 512244 0 0.0
.text 8661844 8661844 0 0.0
door-lock-app debug (read only) 2107489 2107489 0 0.0
(read/write) 119504 119504 0 0.0
.bss 48064 48064 0 0.0
.data 1472 1472 0 0.0
.data.rel.ro 64408 64408 0 0.0
.dynamic 592 592 0 0.0
.got 4264 4264 0 0.0
.init 27 27 0 0.0
.init_array 680 680 0 0.0
.rodata 186697 186697 0 0.0
.text 1765090 1765090 0 0.0
lighting-app debug+rpc (read only) 2312329 2312329 0 0.0
(read/write) 127824 127824 0 0.0
.bss 50272 50272 0 0.0
.data 1952 1952 0 0.0
.data.rel.ro 69896 69896 0 0.0
.dynamic 608 608 0 0.0
.got 4312 4312 0 0.0
.init 27 27 0 0.0
.init_array 776 776 0 0.0
.rodata 182665 182665 0 0.0
.text 1963826 1963826 0 0.0
ota-provider-app debug (read only) 2044673 2044673 0 0.0
(read/write) 114944 114944 0 0.0
.bss 47968 47968 0 0.0
.data 1608 1608 0 0.0
.data.rel.ro 59624 59624 0 0.0
.dynamic 608 608 0 0.0
.got 4464 4464 0 0.0
.init 27 27 0 0.0
.init_array 632 632 0 0.0
.rodata 172483 172483 0 0.0
.text 1716002 1716002 0 0.0
ota-requestor-app debug (read only) 2075049 2075049 0 0.0
(read/write) 118296 118296 0 0.0
.bss 48960 48960 0 0.0
.data 1864 1864 0 0.0
.data.rel.ro 61864 61864 0 0.0
.dynamic 592 592 0 0.0
.got 4304 4304 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 169356 169356 0 0.0
.text 1747634 1747634 0 0.0
shell debug (read only) 2524353 2524353 0 0.0
(read/write) 149968 149968 0 0.0
.bss 67624 67624 0 0.0
.data 1264 1264 0 0.0
.data.rel.ro 75368 75368 0 0.0
.dynamic 592 592 0 0.0
.got 4168 4168 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 214706 214706 0 0.0
.text 2148482 2148482 0 0.0
thermostat-no-ble arm64 (read only) 2352444 2352444 0 0.0
(read/write) 150993 150993 0 0.0
.bss 63153 63153 0 0.0
.data 1424 1424 0 0.0
.data.rel.ro 78632 78632 0 0.0
.dynamic 560 560 0 0.0
.got 4768 4768 0 0.0
.init 24 24 0 0.0
.init_array 368 368 0 0.0
.rodata 144540 144540 0 0.0
.text 1979328 1979328 0 0.0
tv-app debug (read only) 2795825 2795825 0 0.0
(read/write) 250688 250688 0 0.0
.bss 164112 164112 0 0.0
.data 4448 4448 0 0.0
.data.rel.ro 75912 75912 0 0.0
.dynamic 592 592 0 0.0
.got 4688 4688 0 0.0
.init 27 27 0 0.0
.init_array 904 904 0 0.0
.rodata 213259 213259 0 0.0
.text 2401266 2401266 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2368260 2368260 0 0.0
.bss 185236 185236 0 0.0
.data 5840 5840 0 0.0
.text 1330860 1330860 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1156151 1156151 0 0.0
bss 135464 135464 0 0.0
rodata 146768 146768 0 0.0
text 795236 795236 0 0.0
p6 all-clusters-app default (read/write) 2514704 2514704 0 0.0
.bss 118640 118640 0 0.0
.data 2768 2768 0 0.0
.text 1472968 1472968 0 0.0
light-app default (read/write) 2415176 2415176 0 0.0
.bss 112136 112136 0 0.0
.data 2576 2576 0 0.0
.text 1373440 1373440 0 0.0
lock-app default (read/write) 2378744 2378744 0 0.0
.bss 111880 111880 0 0.0
.data 2536 2536 0 0.0
.text 1337008 1337008 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 801264 801264 0 0.0
bss 69988 69988 0 0.0
noinit 40416 40416 0 0.0
text 569958 569958 0 0.0

@github-actions
Copy link

github-actions bot commented Apr 13, 2022

PR #17281: Size comparison from faabdf9 to bbf022d

Increases (2 builds for efr32)
platform target config section faabdf9 bbf022d change % change
efr32 lighting-app BRD4161A (read only) 907300 907636 336 0.0
(read/write) 133080 133144 64 0.0
.bss 131040 131104 64 0.0
.text 907292 907628 336 0.0
BRD4161A+rpc (read only) 941668 942004 336 0.0
(read/write) 149764 149828 64 0.0
.bss 147520 147584 64 0.0
.text 941660 941996 336 0.0
Full report (31 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section faabdf9 bbf022d change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read only) 640211 640211 0 0.0
(read/write) 151212 151212 0 0.0
.bss 74144 74144 0 0.0
.data 3212 3212 0 0.0
.rodata 80163 80163 0 0.0
.text 559556 559556 0 0.0
lock-mtd LP_CC2652R7 (read only) 588947 588947 0 0.0
(read/write) 146932 146932 0 0.0
.bss 69864 69864 0 0.0
.data 3212 3212 0 0.0
.rodata 80043 80043 0 0.0
.text 508412 508412 0 0.0
pump-app LP_CC2652R7 (read only) 648287 648287 0 0.0
(read/write) 152508 152508 0 0.0
.bss 74640 74640 0 0.0
.data 3244 3244 0 0.0
.rodata 75351 75351 0 0.0
.text 572448 572448 0 0.0
pump-controller-app LP_CC2652R7 (read only) 642267 642267 0 0.0
(read/write) 152176 152176 0 0.0
.bss 74344 74344 0 0.0
.data 3208 3208 0 0.0
.rodata 78987 78987 0 0.0
.text 562792 562792 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 618818 618818 0 0.0
.app_xip_area 525528 525528 0 0.0
.bss 75956 75956 0 0.0
.data 684 684 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 576414 576414 0 0.0
.app_xip_area 484660 484660 0 0.0
.bss 74452 74452 0 0.0
.data 648 648 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 564938 564938 0 0.0
.app_xip_area 463556 463556 0 0.0
.bss 83784 83784 0 0.0
.data 564 564 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 907300 907636 336 0.0
(read/write) 133080 133144 64 0.0
.bss 131040 131104 64 0.0
.data 2040 2040 0 0.0
.text 907292 907628 336 0.0
BRD4161A+rpc (read only) 941668 942004 336 0.0
(read/write) 149764 149828 64 0.0
.bss 147520 147584 64 0.0
.data 2244 2244 0 0.0
.text 941660 941996 336 0.0
window-app BRD4161A (read only) 844292 844292 0 0.0
(read/write) 131148 131148 0 0.0
.bss 129200 129200 0 0.0
.data 1948 1948 0 0.0
.text 844284 844284 0 0.0
esp32 all-clusters-app c3devkit (read only) 979702 979702 0 0.0
(read/write) 1397514 1397514 0 0.0
.dram0.bss 62624 62624 0 0.0
.dram0.data 14420 14420 0 0.0
.flash.rodata 201552 201552 0 0.0
.flash.text 979702 979702 0 0.0
.iram0.text 62016 62016 0 0.0
m5stack (read only) 1035283 1035283 0 0.0
(read/write) 465252 465252 0 0.0
.dram0.bss 68144 68144 0 0.0
.dram0.data 34152 34152 0 0.0
.flash.rodata 231120 231120 0 0.0
.flash.text 1029899 1029899 0 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 687036 687036 0 0.0
.bss 78128 78128 0 0.0
.data 2036 2036 0 0.0
.text 601072 601072 0 0.0
lock k32w061+release (read/write) 691652 691652 0 0.0
.bss 78704 78704 0 0.0
.data 1996 1996 0 0.0
.text 605152 605152 0 0.0
linux all-clusters-app debug (read only) 2696513 2696513 0 0.0
(read/write) 149120 149120 0 0.0
.bss 60192 60192 0 0.0
.data 1888 1888 0 0.0
.data.rel.ro 80984 80984 0 0.0
.dynamic 608 608 0 0.0
.got 4456 4456 0 0.0
.init 27 27 0 0.0
.init_array 984 984 0 0.0
.rodata 232421 232421 0 0.0
.text 2290098 2290098 0 0.0
bridge-app debug+rpc (read only) 1836109 1836109 0 0.0
(read/write) 91760 91760 0 0.0
.bss 44480 44480 0 0.0
.data 2912 2912 0 0.0
.data.rel.ro 39280 39280 0 0.0
.dynamic 592 592 0 0.0
.got 3936 3936 0 0.0
.init 27 27 0 0.0
.init_array 552 552 0 0.0
.rodata 148473 148473 0 0.0
.text 1569205 1569205 0 0.0
chip-tool debug (read only) 10676677 10676677 0 0.0
(read/write) 371832 371832 0 0.0
.bss 22752 22752 0 0.0
.data 1104 1104 0 0.0
.data.rel.ro 341728 341728 0 0.0
.dynamic 624 624 0 0.0
.got 4936 4936 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 538005 538005 0 0.0
.text 9309365 9309365 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 10277348 10277348 0 0.0
(read/write) 492209 492209 0 0.0
.bss 41025 41025 0 0.0
.data 1168 1168 0 0.0
.data.rel.ro 388816 388816 0 0.0
.dynamic 560 560 0 0.0
.got 57392 57392 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 512244 512244 0 0.0
.text 8661844 8661844 0 0.0
door-lock-app debug (read only) 2107489 2107489 0 0.0
(read/write) 119504 119504 0 0.0
.bss 48064 48064 0 0.0
.data 1472 1472 0 0.0
.data.rel.ro 64408 64408 0 0.0
.dynamic 592 592 0 0.0
.got 4264 4264 0 0.0
.init 27 27 0 0.0
.init_array 680 680 0 0.0
.rodata 186697 186697 0 0.0
.text 1765090 1765090 0 0.0
lighting-app debug+rpc (read only) 2312329 2312329 0 0.0
(read/write) 127824 127824 0 0.0
.bss 50272 50272 0 0.0
.data 1952 1952 0 0.0
.data.rel.ro 69896 69896 0 0.0
.dynamic 608 608 0 0.0
.got 4312 4312 0 0.0
.init 27 27 0 0.0
.init_array 776 776 0 0.0
.rodata 182665 182665 0 0.0
.text 1963826 1963826 0 0.0
ota-provider-app debug (read only) 2044673 2044673 0 0.0
(read/write) 114944 114944 0 0.0
.bss 47968 47968 0 0.0
.data 1608 1608 0 0.0
.data.rel.ro 59624 59624 0 0.0
.dynamic 608 608 0 0.0
.got 4464 4464 0 0.0
.init 27 27 0 0.0
.init_array 632 632 0 0.0
.rodata 172483 172483 0 0.0
.text 1716002 1716002 0 0.0
ota-requestor-app debug (read only) 2075049 2075049 0 0.0
(read/write) 118296 118296 0 0.0
.bss 48960 48960 0 0.0
.data 1864 1864 0 0.0
.data.rel.ro 61864 61864 0 0.0
.dynamic 592 592 0 0.0
.got 4304 4304 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 169356 169356 0 0.0
.text 1747634 1747634 0 0.0
shell debug (read only) 2524353 2524353 0 0.0
(read/write) 149968 149968 0 0.0
.bss 67624 67624 0 0.0
.data 1264 1264 0 0.0
.data.rel.ro 75368 75368 0 0.0
.dynamic 592 592 0 0.0
.got 4168 4168 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 214706 214706 0 0.0
.text 2148482 2148482 0 0.0
thermostat-no-ble arm64 (read only) 2352444 2352444 0 0.0
(read/write) 150993 150993 0 0.0
.bss 63153 63153 0 0.0
.data 1424 1424 0 0.0
.data.rel.ro 78632 78632 0 0.0
.dynamic 560 560 0 0.0
.got 4768 4768 0 0.0
.init 24 24 0 0.0
.init_array 368 368 0 0.0
.rodata 144540 144540 0 0.0
.text 1979328 1979328 0 0.0
tv-app debug (read only) 2795825 2795825 0 0.0
(read/write) 250688 250688 0 0.0
.bss 164112 164112 0 0.0
.data 4448 4448 0 0.0
.data.rel.ro 75912 75912 0 0.0
.dynamic 592 592 0 0.0
.got 4688 4688 0 0.0
.init 27 27 0 0.0
.init_array 904 904 0 0.0
.rodata 213259 213259 0 0.0
.text 2401266 2401266 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2368260 2368260 0 0.0
.bss 185236 185236 0 0.0
.data 5840 5840 0 0.0
.text 1330860 1330860 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1156151 1156151 0 0.0
bss 135464 135464 0 0.0
rodata 146768 146768 0 0.0
text 795236 795236 0 0.0
p6 all-clusters-app default (read/write) 2514704 2514704 0 0.0
.bss 118640 118640 0 0.0
.data 2768 2768 0 0.0
.text 1472968 1472968 0 0.0
light-app default (read/write) 2415176 2415176 0 0.0
.bss 112136 112136 0 0.0
.data 2576 2576 0 0.0
.text 1373440 1373440 0 0.0
lock-app default (read/write) 2378744 2378744 0 0.0
.bss 111880 111880 0 0.0
.data 2536 2536 0 0.0
.text 1337008 1337008 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 801264 801264 0 0.0
bss 69988 69988 0 0.0
noinit 40416 40416 0 0.0
text 569958 569958 0 0.0

@selissia selissia merged commit 94e8197 into project-chip:master Apr 13, 2022
andrei-menzopol pushed a commit to andrei-menzopol/connectedhomeip that referenced this pull request Apr 14, 2022
…ect-chip#17281)

* Test added march 8 (project-chip#15957)

* Added new manual scripts

* Added Auto generated File

* [OTA] Fix OTARequestorDriverImpl inclusion (project-chip#15981)

* Regen to fix CI failures (project-chip#15990)

* [ota] Store Default OTA Providers in flash (project-chip#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>

* Use an intermediate buffer for writing the image to the booloader storage

* Use critical section in the bootloader API call

* Cleanup log messages, move variables into a class

* Remove merge artifacts

* Update EFR32 documentation

* Fix typo

* Restyled by whitespace

* Restyled by clang-format

* Restyled by prettier-markdown

* Rename array size parameter, add aligned attribute

* Trivial change to restart the CI (restyle job need to be kicked)

* Restyled by clang-format

* Update comments

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>
@selissia selissia deleted the internal-flash-ota-1 branch May 12, 2022 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation efr32 platform review - approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants