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

[linux] Use generic PersistentStorageDelegate instead of ChipLinuxSto… #17371

Merged
merged 2 commits into from
Apr 14, 2022
Merged

[linux] Use generic PersistentStorageDelegate instead of ChipLinuxSto… #17371

merged 2 commits into from
Apr 14, 2022

Conversation

yufengwangca
Copy link
Contributor

…rage for userlabel

Problem

What is being fixed? Examples:

  • We already have the PersistentStorageDelegate setup in the parent class of DeviceInfoProviderImpl, we should use this one instead of Linux specific ChipLinuxStorage class to read and write user label to persistent memory on Linux platform.

Change overview

Use generic PersistentStorageDelegate instead of ChipLinuxStorage to read and write user label to persistent memory on Linux platform.

Testing

How was this tested? (at least one bullet point required)

  • Covered by the existing TestUserLabelList yaml test

@github-actions
Copy link

github-actions bot commented Apr 14, 2022

PR #17371: Size comparison from 376ce66 to 00463bb

Full report (21 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, mbed, nrfconnect, p6, telink)
platform target config section 376ce66 00463bb change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 683351 683351 0 0.0
(read/write) 169840 169840 0 0.0
.bss 76176 76176 0 0.0
.data 3380 3380 0 0.0
.rodata 102975 102975 0 0.0
.text 579896 579896 0 0.0
lock-ftd LP_CC2652R7 (read only) 639823 639823 0 0.0
(read/write) 151220 151220 0 0.0
.bss 74152 74152 0 0.0
.data 3212 3212 0 0.0
.rodata 79951 79951 0 0.0
.text 559380 559380 0 0.0
lock-mtd LP_CC2652R7 (read only) 588567 588567 0 0.0
(read/write) 146940 146940 0 0.0
.bss 69872 69872 0 0.0
.data 3212 3212 0 0.0
.rodata 79831 79831 0 0.0
.text 508244 508244 0 0.0
pump-app LP_CC2652R7 (read only) 648571 648571 0 0.0
(read/write) 152516 152516 0 0.0
.bss 74648 74648 0 0.0
.data 3244 3244 0 0.0
.rodata 75435 75435 0 0.0
.text 572648 572648 0 0.0
pump-controller-app LP_CC2652R7 (read only) 641871 641871 0 0.0
(read/write) 152184 152184 0 0.0
.bss 74352 74352 0 0.0
.data 3208 3208 0 0.0
.rodata 78767 78767 0 0.0
.text 562616 562616 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 618458 618458 0 0.0
.app_xip_area 525160 525160 0 0.0
.bss 75964 75964 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) 576058 576058 0 0.0
.app_xip_area 484296 484296 0 0.0
.bss 74460 74460 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) 565234 565234 0 0.0
.app_xip_area 463844 463844 0 0.0
.bss 83792 83792 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) 907396 907396 0 0.0
(read/write) 133152 133152 0 0.0
.bss 131112 131112 0 0.0
.data 2040 2040 0 0.0
.text 907388 907388 0 0.0
BRD4161A+rpc (read only) 941772 941772 0 0.0
(read/write) 149836 149836 0 0.0
.bss 147592 147592 0 0.0
.data 2244 2244 0 0.0
.text 941764 941764 0 0.0
window-app BRD4161A (read only) 844052 844052 0 0.0
(read/write) 131156 131156 0 0.0
.bss 129208 129208 0 0.0
.data 1948 1948 0 0.0
.text 844044 844044 0 0.0
esp32 all-clusters-app c3devkit (read only) 979666 979666 0 0.0
(read/write) 1397410 1397410 0 0.0
.dram0.bss 62632 62632 0 0.0
.dram0.data 14420 14420 0 0.0
.flash.rodata 201432 201432 0 0.0
.flash.text 979666 979666 0 0.0
.iram0.text 62016 62016 0 0.0
m5stack (read only) 1035075 1035075 0 0.0
(read/write) 465164 465164 0 0.0
.dram0.bss 68152 68152 0 0.0
.dram0.data 34152 34152 0 0.0
.flash.rodata 231024 231024 0 0.0
.flash.text 1029691 1029691 0 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 686876 686876 0 0.0
.bss 78136 78136 0 0.0
.data 2036 2036 0 0.0
.text 600904 600904 0 0.0
lock k32w061+release (read/write) 691320 691320 0 0.0
.bss 78712 78712 0 0.0
.data 1996 1996 0 0.0
.text 604812 604812 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2368820 2368820 0 0.0
.bss 185244 185244 0 0.0
.data 5840 5840 0 0.0
.text 1331420 1331420 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1162867 1162867 0 0.0
bss 136536 136536 0 0.0
rodata 147092 147092 0 0.0
text 800624 800624 0 0.0
p6 all-clusters-app default (read/write) 2514624 2514624 0 0.0
.bss 118648 118648 0 0.0
.data 2768 2768 0 0.0
.text 1472888 1472888 0 0.0
light-app default (read/write) 2415080 2415080 0 0.0
.bss 112144 112144 0 0.0
.data 2576 2576 0 0.0
.text 1373344 1373344 0 0.0
lock-app default (read/write) 2378648 2378648 0 0.0
.bss 111888 111888 0 0.0
.data 2536 2536 0 0.0
.text 1336912 1336912 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 800852 800852 0 0.0
bss 69996 69996 0 0.0
noinit 40416 40416 0 0.0
text 569742 569742 0 0.0

@github-actions
Copy link

github-actions bot commented Apr 14, 2022

PR #17371: Size comparison from 91b08e0 to cf88cea

Increases above 0.2%:

platform target config section 91b08e0 cf88cea change % change
linux all-clusters-app debug .data 1888 1920 32 1.7
chip-tool debug .data 1104 1136 32 2.9
chip-tool-no-interactive-ipv6only arm64 .data 1168 1184 16 1.4
ota-provider-app debug .data 1608 1640 32 2.0
ota-requestor-app debug .data 1864 1896 32 1.7
shell debug .data 1264 1296 32 2.5
thermostat-no-ble arm64 .data 1424 1440 16 1.1
tv-app debug .data 4448 4480 32 0.7
Increases (8 builds for linux)
platform target config section 91b08e0 cf88cea change % change
linux all-clusters-app debug .data 1888 1920 32 1.7
chip-tool debug .data 1104 1136 32 2.9
chip-tool-no-interactive-ipv6only arm64 .data 1168 1184 16 1.4
ota-provider-app debug .data 1608 1640 32 2.0
ota-requestor-app debug .data 1864 1896 32 1.7
shell debug .data 1264 1296 32 2.5
thermostat-no-ble arm64 .data 1424 1440 16 1.1
tv-app debug .data 4448 4480 32 0.7
Decreases (11 builds for linux)
platform target config section 91b08e0 cf88cea change % change
linux all-clusters-app debug (read only) 2697785 2697425 -360 -0.0
(read/write) 149248 149088 -160 -0.1
.bss 60224 60032 -192 -0.3
.text 2291490 2291106 -384 -0.0
bridge-app debug+rpc (read only) 1837453 1837045 -408 -0.0
(read/write) 91856 91672 -184 -0.2
.bss 44480 44296 -184 -0.4
.rodata 148473 148441 -32 -0.0
.text 1570725 1570325 -400 -0.0
chip-tool debug (read only) 10719701 1071929 -408 -0.0
(read/write) 371832 371672 -160 -0.0
.bss 22752 22560 -192 -0.8
.rodata 539221 539189 -32 -0.0
.text 9351173 9350773 -400 -0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 10343572 10342980 -592 -0.0
(read/write) 492273 492049 -224 -0.0
.bss 41025 40881 -144 -0.4
.data.rel.ro 388840 388752 -88 -0.0
.got 57440 57424 -16 -0.0
.rodata 513204 513172 -32 -0.0
.text 8726948 8726628 -320 -0.0
door-lock-app debug (read only) 2108721 2108329 -392 -0.0
(read/write) 119568 119408 -160 -0.1
.bss 48064 47904 -160 -0.3
.rodata 186697 186665 -32 -0.0
.text 1766498 1766114 -384 -0.0
lighting-app debug+rpc (read only) 2313393 2312985 -408 -0.0
(read/write) 127952 127760 -192 -0.2
.bss 50304 50112 -192 -0.4
.rodata 182633 182601 -32 -0.0
.text 1965010 1964610 -400 -0.0
ota-provider-app debug (read only) 2045881 2045481 -400 -0.0
(read/write) 115296 115136 -160 -0.1
.bss 48224 48032 -192 -0.4
.rodata 172547 172523 -24 -0.0
.text 1717298 1716898 -400 -0.0
ota-requestor-app debug (read only) 2076345 2075953 -392 -0.0
(read/write) 118360 118168 -192 -0.2
.bss 48960 48736 -224 -0.5
.rodata 169420 169388 -32 -0.0
.text 1749042 1748658 -384 -0.0
shell debug (read only) 2526577 2526185 -392 -0.0
(read/write) 150064 149904 -160 -0.1
.bss 67624 67432 -192 -0.3
.rodata 214834 214802 -32 -0.0
.text 2150578 2150194 -384 -0.0
thermostat-no-ble arm64 (read only) 2353332 2352756 -576 -0.0
(read/write) 151121 150881 -240 -0.2
.bss 63169 63009 -160 -0.3
.data.rel.ro 78752 78664 -88 -0.1
.got 4768 4752 -16 -0.3
.rodata 144548 144532 -16 -0.0
.text 1980272 1979952 -320 -0.0
tv-app debug (read only) 2798185 2797825 -360 -0.0
(read/write) 250784 250632 -152 -0.1
.bss 164144 163960 -184 -0.1
.text 2403586 2403202 -384 -0.0
Full report (32 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 91b08e0 cf88cea change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 683415 683415 0 0.0
(read/write) 169776 169776 0 0.0
.bss 76176 76176 0 0.0
.data 3380 3380 0 0.0
.rodata 103023 103023 0 0.0
.text 579912 579912 0 0.0
lock-ftd LP_CC2652R7 (read only) 676447 676447 0 0.0
(read/write) 167704 167704 0 0.0
.bss 74192 74192 0 0.0
.data 3212 3212 0 0.0
.rodata 98639 98639 0 0.0
.text 577324 577324 0 0.0
lock-mtd LP_CC2652R7 (read only) 625199 625199 0 0.0
(read/write) 146980 146980 0 0.0
.bss 69912 69912 0 0.0
.data 3212 3212 0 0.0
.rodata 98519 98519 0 0.0
.text 526188 526188 0 0.0
pump-app LP_CC2652R7 (read only) 648643 648643 0 0.0
(read/write) 152516 152516 0 0.0
.bss 74648 74648 0 0.0
.data 3244 3244 0 0.0
.rodata 75475 75475 0 0.0
.text 572680 572680 0 0.0
pump-controller-app LP_CC2652R7 (read only) 641951 641951 0 0.0
(read/write) 152184 152184 0 0.0
.bss 74352 74352 0 0.0
.data 3208 3208 0 0.0
.rodata 78815 78815 0 0.0
.text 562648 562648 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 618538 618538 0 0.0
.app_xip_area 525240 525240 0 0.0
.bss 75964 75964 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) 612786 612786 0 0.0
.app_xip_area 520992 520992 0 0.0
.bss 74492 74492 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) 565298 565298 0 0.0
.app_xip_area 463908 463908 0 0.0
.bss 83792 83792 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) 907492 907492 0 0.0
(read/write) 133152 133152 0 0.0
.bss 131112 131112 0 0.0
.data 2040 2040 0 0.0
.text 907484 907484 0 0.0
BRD4161A+rpc (read only) 941852 941852 0 0.0
(read/write) 149836 149836 0 0.0
.bss 147592 147592 0 0.0
.data 2244 2244 0 0.0
.text 941844 941844 0 0.0
window-app BRD4161A (read only) 844132 844132 0 0.0
(read/write) 131156 131156 0 0.0
.bss 129208 129208 0 0.0
.data 1948 1948 0 0.0
.text 844124 844124 0 0.0
esp32 all-clusters-app c3devkit (read only) 979708 979708 0 0.0
(read/write) 1397466 1397466 0 0.0
.dram0.bss 62632 62632 0 0.0
.dram0.data 14420 14420 0 0.0
.flash.rodata 201488 201488 0 0.0
.flash.text 979708 979708 0 0.0
.iram0.text 62016 62016 0 0.0
m5stack (read only) 1035111 1035111 0 0.0
(read/write) 465220 465220 0 0.0
.dram0.bss 68152 68152 0 0.0
.dram0.data 34152 34152 0 0.0
.flash.rodata 231080 231080 0 0.0
.flash.text 1029727 1029727 0 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 684008 684008 0 0.0
.bss 77952 77952 0 0.0
.data 1992 1992 0 0.0
.text 598264 598264 0 0.0
lock k32w061+release (read/write) 724868 724868 0 0.0
.bss 78560 78560 0 0.0
.data 1952 1952 0 0.0
.text 638556 638556 0 0.0
linux all-clusters-app debug (read only) 2697785 2697425 -360 -0.0
(read/write) 149248 149088 -160 -0.1
.bss 60224 60032 -192 -0.3
.data 1888 1920 32 1.7
.data.rel.ro 81080 81080 0 0.0
.dynamic 608 608 0 0.0
.got 4448 4448 0 0.0
.init 27 27 0 0.0
.init_array 984 984 0 0.0
.rodata 232389 232389 0 0.0
.text 2291490 2291106 -384 -0.0
bridge-app debug+rpc (read only) 1837453 1837045 -408 -0.0
(read/write) 91856 91672 -184 -0.2
.bss 44480 44296 -184 -0.4
.data 2912 2912 0 0.0
.data.rel.ro 39376 39376 0 0.0
.dynamic 592 592 0 0.0
.got 3920 3920 0 0.0
.init 27 27 0 0.0
.init_array 552 552 0 0.0
.rodata 148473 148441 -32 -0.0
.text 1570725 1570325 -400 -0.0
chip-tool debug (read only) 10719701 1071929 -408 -0.0
(read/write) 371832 371672 -160 -0.0
.bss 22752 22560 -192 -0.8
.data 1104 1136 32 2.9
.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 539221 539189 -32 -0.0
.text 9351173 9350773 -400 -0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 10343572 10342980 -592 -0.0
(read/write) 492273 492049 -224 -0.0
.bss 41025 40881 -144 -0.4
.data 1168 1184 16 1.4
.data.rel.ro 388840 388752 -88 -0.0
.dynamic 560 560 0 0.0
.got 57440 57424 -16 -0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 513204 513172 -32 -0.0
.text 8726948 8726628 -320 -0.0
door-lock-app debug (read only) 2108721 2108329 -392 -0.0
(read/write) 119568 119408 -160 -0.1
.bss 48064 47904 -160 -0.3
.data 1472 1472 0 0.0
.data.rel.ro 64504 64504 0 0.0
.dynamic 592 592 0 0.0
.got 4248 4248 0 0.0
.init 27 27 0 0.0
.init_array 680 680 0 0.0
.rodata 186697 186665 -32 -0.0
.text 1766498 1766114 -384 -0.0
lighting-app debug+rpc (read only) 2313393 2312985 -408 -0.0
(read/write) 127952 127760 -192 -0.2
.bss 50304 50112 -192 -0.4
.data 1952 1952 0 0.0
.data.rel.ro 69992 69992 0 0.0
.dynamic 608 608 0 0.0
.got 4304 4304 0 0.0
.init 27 27 0 0.0
.init_array 776 776 0 0.0
.rodata 182633 182601 -32 -0.0
.text 1965010 1964610 -400 -0.0
ota-provider-app debug (read only) 2045881 2045481 -400 -0.0
(read/write) 115296 115136 -160 -0.1
.bss 48224 48032 -192 -0.4
.data 1608 1640 32 2.0
.data.rel.ro 59720 59720 0 0.0
.dynamic 608 608 0 0.0
.got 4448 4448 0 0.0
.init 27 27 0 0.0
.init_array 632 632 0 0.0
.rodata 172547 172523 -24 -0.0
.text 1717298 1716898 -400 -0.0
ota-requestor-app debug (read only) 2076345 2075953 -392 -0.0
(read/write) 118360 118168 -192 -0.2
.bss 48960 48736 -224 -0.5
.data 1864 1896 32 1.7
.data.rel.ro 61960 61960 0 0.0
.dynamic 592 592 0 0.0
.got 4288 4288 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 169420 169388 -32 -0.0
.text 1749042 1748658 -384 -0.0
shell debug (read only) 2526577 2526185 -392 -0.0
(read/write) 150064 149904 -160 -0.1
.bss 67624 67432 -192 -0.3
.data 1264 1296 32 2.5
.data.rel.ro 75464 75464 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 214834 214802 -32 -0.0
.text 2150578 2150194 -384 -0.0
thermostat-no-ble arm64 (read only) 2353332 2352756 -576 -0.0
(read/write) 151121 150881 -240 -0.2
.bss 63169 63009 -160 -0.3
.data 1424 1440 16 1.1
.data.rel.ro 78752 78664 -88 -0.1
.dynamic 560 560 0 0.0
.got 4768 4752 -16 -0.3
.init 24 24 0 0.0
.init_array 368 368 0 0.0
.rodata 144548 144532 -16 -0.0
.text 1980272 1979952 -320 -0.0
tv-app debug (read only) 2798185 2797825 -360 -0.0
(read/write) 250784 250632 -152 -0.1
.bss 164144 163960 -184 -0.1
.data 4448 4480 32 0.7
.data.rel.ro 76008 76008 0 0.0
.dynamic 592 592 0 0.0
.got 4680 4680 0 0.0
.init 27 27 0 0.0
.init_array 904 904 0 0.0
.rodata 213387 213387 0 0.0
.text 2403586 2403202 -384 -0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2409404 2409404 0 0.0
.bss 185268 185268 0 0.0
.data 5840 5840 0 0.0
.text 1372004 1372004 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1162943 1162943 0 0.0
bss 136536 136536 0 0.0
rodata 147136 147136 0 0.0
text 800656 800656 0 0.0
p6 all-clusters-app default (read/write) 2514704 2514704 0 0.0
.bss 118648 118648 0 0.0
.data 2768 2768 0 0.0
.text 1472968 1472968 0 0.0
light-app default (read/write) 2415160 2415160 0 0.0
.bss 112144 112144 0 0.0
.data 2576 2576 0 0.0
.text 1373424 1373424 0 0.0
lock-app default (read/write) 2418640 2418640 0 0.0
.bss 111920 111920 0 0.0
.data 2536 2536 0 0.0
.text 1376904 1376904 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 800936 800936 0 0.0
bss 69996 69996 0 0.0
noinit 40416 40416 0 0.0
text 569780 569780 0 0.0

@yufengwangca yufengwangca merged commit 30edacd into project-chip:master Apr 14, 2022
@yufengwangca yufengwangca deleted the pr/linux/devinfo branch April 14, 2022 21:38
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.

4 participants