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

[QPG6100] Patch linkerscript for size issues #12047 #12480

Merged

Conversation

tima-q
Copy link
Contributor

@tima-q tima-q commented Dec 2, 2021

Problem

#12047 introduced additional flash and RAM use, reaching QPG6100 limits.
This PR was however merged, with failing CI for QPG6100 (and K32W)

Failing CI:
https://github.com/project-chip/connectedhomeip/runs/4397616649?check_suite_focus=true

Change overview

Temporary patch of QPG6100 linkerscript to allow CHIP dev to continue.
Investigation to mitigate size limits and removal of patch scheduled as internal follow-up

Testing

  • Manual build QPG light example (failing build)
  • CI

@jepenven-silabs
Copy link
Contributor

fast tracked since platform specific submodule update

@tima-q tima-q added the qpg QPG6xxx platform label Dec 2, 2021
@github-actions
Copy link

github-actions bot commented Dec 2, 2021

PR #12480: Size comparison from a5513c2 to f3d191e

Full report (33 builds for efr32, esp32, linux, mbed, nrfconnect, p6, telink)
platform target config section a5513c2 f3d191e change % change
efr32 lighting-app BRD4161A (read only) 750760 750760 0 0.0
(read/write) 119976 119976 0 0.0
.bss 118160 118160 0 0.0
.data 1812 1812 0 0.0
.text 750752 750752 0 0.0
BRD4161A+rpc (read only) 779416 779416 0 0.0
(read/write) 138280 138280 0 0.0
.bss 136360 136360 0 0.0
.data 1920 1920 0 0.0
.text 779408 779408 0 0.0
lock-app BRD4161A (read only) 724616 724616 0 0.0
(read/write) 117680 117680 0 0.0
.bss 115912 115912 0 0.0
.data 1768 1768 0 0.0
.text 724608 724608 0 0.0
window-app BRD4161A (read only) 727928 727928 0 0.0
(read/write) 118104 118104 0 0.0
.bss 116328 116328 0 0.0
.data 1776 1776 0 0.0
.text 727920 727920 0 0.0
esp32 all-clusters-app c3devkit (read only) 846930 846930 0 0.0
(read/write) 1223314 1223314 0 0.0
.dram0.bss 56456 56456 0 0.0
.dram0.data 14036 14036 0 0.0
.flash.rodata 167360 167360 0 0.0
.flash.text 846930 846930 0 0.0
.iram0.text 61394 61394 0 0.0
m5stack (read only) 917483 917483 0 0.0
(read/write) 422308 422308 0 0.0
.dram0.bss 61848 61848 0 0.0
.dram0.data 34000 34000 0 0.0
.flash.rodata 195180 195180 0 0.0
.flash.text 912099 912099 0 0.0
.iram0.text 122943 122943 0 0.0
linux all-clusters-app debug (read only) 1832833 1832833 0 0.0
(read/write) 123648 123648 0 0.0
.bss 50640 50640 0 0.0
.data 1104 1104 0 0.0
.data.rel.ro 66624 66624 0 0.0
.dynamic 592 592 0 0.0
.got 4112 4112 0 0.0
.init 27 27 0 0.0
.init_array 552 552 0 0.0
.rodata 146261 146261 0 0.0
.text 1545202 1545202 0 0.0
bridge-app debug+rpc (read only) 1416061 1416061 0 0.0
(read/write) 73808 73808 0 0.0
.bss 36048 36048 0 0.0
.data 1680 1680 0 0.0
.data.rel.ro 31056 31056 0 0.0
.dynamic 592 592 0 0.0
.got 3984 3984 0 0.0
.init 27 27 0 0.0
.init_array 408 408 0 0.0
.rodata 119300 119300 0 0.0
.text 1193189 1193189 0 0.0
chip-tool debug (read only) 6589629 6589629 0 0.0
(read/write) 201616 201616 0 0.0
.bss 34344 34344 0 0.0
.data 1008 1008 0 0.0
.data.rel.ro 160696 160696 0 0.0
.dynamic 592 592 0 0.0
.got 4464 4464 0 0.0
.init 27 27 0 0.0
.init_array 480 480 0 0.0
.rodata 306104 306104 0 0.0
.text 5874181 5874181 0 0.0
lighting-app debug+rpc (read only) 1699865 1699865 0 0.0
(read/write) 106896 106896 0 0.0
.bss 41744 41744 0 0.0
.data 1264 1264 0 0.0
.data.rel.ro 58560 58560 0 0.0
.dynamic 608 608 0 0.0
.got 4136 4136 0 0.0
.init 27 27 0 0.0
.init_array 536 536 0 0.0
.rodata 139089 139089 0 0.0
.text 1419458 1419458 0 0.0
ota-provider-app debug (read only) 1377977 1377977 0 0.0
(read/write) 72264 72264 0 0.0
.bss 38624 38624 0 0.0
.data 912 912 0 0.0
.data.rel.ro 27592 27592 0 0.0
.dynamic 592 592 0 0.0
.got 4048 4048 0 0.0
.init 27 27 0 0.0
.init_array 448 448 0 0.0
.rodata 120776 120776 0 0.0
.text 1153394 1153394 0 0.0
ota-requestor-app debug (read only) 1478033 1478033 0 0.0
(read/write) 76160 76160 0 0.0
.bss 40736 40736 0 0.0
.data 976 976 0 0.0
.data.rel.ro 29304 29304 0 0.0
.dynamic 592 592 0 0.0
.got 4032 4032 0 0.0
.init 27 27 0 0.0
.init_array 472 472 0 0.0
.rodata 132720 132720 0 0.0
.text 1238066 1238066 0 0.0
shell debug (read only) 812841 812841 0 0.0
(read/write) 60264 60264 0 0.0
.bss 16904 16904 0 0.0
.data 240 240 0 0.0
.data.rel.ro 38656 38656 0 0.0
.dynamic 592 592 0 0.0
.got 3504 3504 0 0.0
.init 27 27 0 0.0
.init_array 344 344 0 0.0
.rodata 83506 83506 0 0.0
.text 623250 623250 0 0.0
tv-app debug (read only) 1993585 1993585 0 0.0
(read/write) 316536 316536 0 0.0
.bss 246112 246112 0 0.0
.data 1504 1504 0 0.0
.data.rel.ro 63248 63248 0 0.0
.dynamic 592 592 0 0.0
.got 4424 4424 0 0.0
.init 27 27 0 0.0
.init_array 624 624 0 0.0
.rodata 169128 169128 0 0.0
.text 1672562 1672562 0 0.0
mbed all-clusters-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2308000 2308000 0 0.0
.bss 179516 179516 0 0.0
.data 5168 5168 0 0.0
.heap 851760 851760 0 0.0
.text 1270576 1270576 0 0.0
lighting-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2294648 2294648 0 0.0
.bss 173208 173208 0 0.0
.data 5480 5480 0 0.0
.heap 857760 857760 0 0.0
.text 1257248 1257248 0 0.0
lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2267744 2267744 0 0.0
.bss 172024 172024 0 0.0
.data 5480 5480 0 0.0
.heap 858944 858944 0 0.0
.text 1230344 1230344 0 0.0
pigweed-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 1140008 1140008 0 0.0
.bss 11756 11756 0 0.0
.data 4376 4376 0 0.0
.heap 1020312 1020312 0 0.0
.text 103392 103392 0 0.0
shell CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2046576 2046576 0 0.0
.bss 156704 156704 0 0.0
.data 4864 4864 0 0.0
.heap 874880 874880 0 0.0
.text 1009176 1009176 0 0.0
nrfconnect lighting-app nrf52840dk_nrf52840 (read/write) 890887 890887 0 0.0
bss 113712 113712 0 0.0
rodata 99112 99112 0 0.0
text 602512 602512 0 0.0
nrf52840dk_nrf52840+rpc (read/write) 854247 854247 0 0.0
bss 110060 110060 0 0.0
rodata 90472 90472 0 0.0
text 577508 577508 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 816786 816786 0 0.0
bss 115084 115084 0 0.0
rodata 94372 94372 0 0.0
text 532852 532852 0 0.0
lock-app nrf52840dk_nrf52840 (read/write) 861715 861715 0 0.0
bss 110748 110748 0 0.0
rodata 94852 94852 0 0.0
text 580764 580764 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 787866 787866 0 0.0
bss 112160 112160 0 0.0
rodata 90140 90140 0 0.0
text 511196 511196 0 0.0
pigweed-app nrf52840dk_nrf52840 (read/write) 497463 497463 0 0.0
bss 51820 51820 0 0.0
rodata 45852 45852 0 0.0
text 339492 339492 0 0.0
pump-app nrf52840dk_nrf52840 (read/write) 867759 867759 0 0.0
bss 110884 110884 0 0.0
rodata 96584 96584 0 0.0
text 584832 584832 0 0.0
pump-controller-app nrf52840dk_nrf52840 (read/write) 860823 860823 0 0.0
bss 110764 110764 0 0.0
rodata 94720 94720 0 0.0
text 579892 579892 0 0.0
shell nrf52840dk_nrf52840 (read/write) 779903 779903 0 0.0
bss 109696 109696 0 0.0
rodata 73796 73796 0 0.0
text 521924 521924 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 694946 694946 0 0.0
bss 110680 110680 0 0.0
rodata 68436 68436 0 0.0
text 442528 442528 0 0.0
p6 all-clusters-app default (read/write) 2337560 2337560 0 0.0
.bss 107696 107696 0 0.0
.data 2456 2456 0 0.0
.heap 923192 923192 0 0.0
.text 1295824 1295824 0 0.0
light-app default (read/write) 2277344 2277344 0 0.0
.bss 98428 98428 0 0.0
.data 2328 2328 0 0.0
.heap 932584 932584 0 0.0
.text 1235608 1235608 0 0.0
lock-app default (read/write) 2252680 2252680 0 0.0
.bss 97084 97084 0 0.0
.data 2288 2288 0 0.0
.heap 933968 933968 0 0.0
.text 1210944 1210944 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 793890 793890 0 0.0
bss 80288 80288 0 0.0
noinit 37160 37160 0 0.0
text 554246 554246 0 0.0

@jepenven-silabs jepenven-silabs merged commit 541ebfc into project-chip:master Dec 2, 2021
@tima-q tima-q deleted the qpg6100/size_patch_for_12047 branch December 2, 2021 18:32
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.

3 participants