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

[chip-tool] Rely on the readclient/writeclient/commandsender onDone c… #15571

Merged

Conversation

vivien-apple
Copy link
Contributor

…allback to be called for returning any errors

Problem

chip-tool exit on the first error when using a wildcard.
See #14896

Change overview

  • Wait for onDone delegates method before exiting

Testing

It was manually tested doing: ./out/debug/standalone/chip-tool any read-by-id 0xFFFFFFFF 0xFFFFFFFF 0x12344321 3

…allback to be called for returning any errors
@vivien-apple vivien-apple force-pushed the ChipTool_WaitOnDoneForExiting branch from 1161a71 to 44eab55 Compare February 25, 2022 13:14
@github-actions
Copy link

github-actions bot commented Feb 25, 2022

PR #15571: Size comparison from 980e177 to 44eab55

Decreases (2 builds for linux)
platform target config section 980e177 44eab55 change % change
linux chip-tool debug (read only) 8994413 8994365 -48 -0.0
.text 7905429 7905381 -48 -0.0
chip-tool-ipv6only arm64 (read only) 8748748 8748716 -32 -0.0
.text 7427620 7427588 -32 -0.0
Full report (45 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
platform target config section 980e177 44eab55 change % change
cyw30739 light cyw930739m2evb_01 (read/write) 599010 599010 0 0.0
.app_xip_area 503856 503856 0 0.0
.bss 77844 77844 0 0.0
.data 652 652 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 557094 557094 0 0.0
.app_xip_area 463476 463476 0 0.0
.bss 76348 76348 0 0.0
.data 616 616 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor cyw930739m2evb_01 (read/write) 577598 577598 0 0.0
.app_xip_area 474556 474556 0 0.0
.bss 85444 85444 0 0.0
.data 560 560 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 915892 915892 0 0.0
(read/write) 128608 128608 0 0.0
.bss 126560 126560 0 0.0
.data 2048 2048 0 0.0
.text 915884 915884 0 0.0
BRD4161A+rpc (read only) 945988 945988 0 0.0
(read/write) 145548 145548 0 0.0
.bss 143336 143336 0 0.0
.data 2212 2212 0 0.0
.text 945980 945980 0 0.0
lock-app BRD4161A+wf200 (read only) 985812 985812 0 0.0
(read/write) 120360 120360 0 0.0
.bss 118436 118436 0 0.0
.data 1924 1924 0 0.0
.text 985804 985804 0 0.0
window-app BRD4161A (read only) 849984 849984 0 0.0
(read/write) 126840 126840 0 0.0
.bss 124920 124920 0 0.0
.data 1916 1916 0 0.0
.text 849976 849976 0 0.0
esp32 all-clusters-app c3devkit (read only) 952386 952386 0 0.0
(read/write) 1392858 1392858 0 0.0
.dram0.bss 62936 62936 0 0.0
.dram0.data 14172 14172 0 0.0
.flash.rodata 196848 196848 0 0.0
.flash.text 952386 952386 0 0.0
.iram0.text 62056 62056 0 0.0
m5stack (read only) 1002411 1002411 0 0.0
(read/write) 458336 458336 0 0.0
.dram0.bss 68080 68080 0 0.0
.dram0.data 34072 34072 0 0.0
.flash.rodata 224056 224056 0 0.0
.flash.text 997027 997027 0 0.0
.iram0.text 123399 123399 0 0.0
k32w light k32w061+release (read/write) 691360 691360 0 0.0
.bss 77472 77472 0 0.0
.data 1924 1924 0 0.0
.text 606164 606164 0 0.0
lock k32w061+release (read/write) 694648 694648 0 0.0
.bss 77688 77688 0 0.0
.data 1964 1964 0 0.0
.text 609196 609196 0 0.0
linux all-clusters-app debug (read only) 2392145 2392145 0 0.0
(read/write) 144688 144688 0 0.0
.bss 59232 59232 0 0.0
.data 1360 1360 0 0.0
.data.rel.ro 78392 78392 0 0.0
.dynamic 592 592 0 0.0
.got 4160 4160 0 0.0
.init 27 27 0 0.0
.init_array 904 904 0 0.0
.rodata 204197 204197 0 0.0
.text 2025202 2025202 0 0.0
bridge-app debug+rpc (read only) 1718253 1718253 0 0.0
(read/write) 92736 92736 0 0.0
.bss 47552 47552 0 0.0
.data 2042 2042 0 0.0
.data.rel.ro 38088 38088 0 0.0
.dynamic 592 592 0 0.0
.got 3912 3912 0 0.0
.init 27 27 0 0.0
.init_array 536 536 0 0.0
.rodata 141012 141012 0 0.0
.text 1461461 1461461 0 0.0
chip-tool debug (read only) 8994413 8994365 -48 -0.0
(read/write) 298488 298488 0 0.0
.bss 40384 40384 0 0.0
.data 1164 1164 0 0.0
.data.rel.ro 250872 250872 0 0.0
.dynamic 608 608 0 0.0
.got 4784 4784 0 0.0
.init 27 27 0 0.0
.init_array 624 624 0 0.0
.rodata 480533 480533 0 0.0
.text 7905429 7905381 -48 -0.0
chip-tool-ipv6only arm64 (read only) 8748748 8748716 -32 -0.0
(read/write) 411601 411601 0 0.0
.bss 58609 58609 0 0.0
.data 1216 1216 0 0.0
.data.rel.ro 296568 296568 0 0.0
.dynamic 560 560 0 0.0
.got 51408 51408 0 0.0
.init 24 24 0 0.0
.init_array 208 208 0 0.0
.rodata 456676 456676 0 0.0
.text 7427620 7427588 -32 -0.0
door-lock-app debug (read only) 1933881 1933881 0 0.0
(read/write) 119128 119128 0 0.0
.bss 50496 50496 0 0.0
.data 1050 1050 0 0.0
.data.rel.ro 62200 62200 0 0.0
.dynamic 592 592 0 0.0
.got 4096 4096 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 173052 173052 0 0.0
.text 1612370 1612370 0 0.0
lighting-app debug+rpc (read only) 2068713 2068713 0 0.0
(read/write) 124488 124488 0 0.0
.bss 51424 51424 0 0.0
.data 1472 1472 0 0.0
.data.rel.ro 66144 66144 0 0.0
.dynamic 608 608 0 0.0
.got 4128 4128 0 0.0
.init 27 27 0 0.0
.init_array 704 704 0 0.0
.rodata 165617 165617 0 0.0
.text 1747666 1747666 0 0.0
ota-provider-app debug (read only) 1866449 1866449 0 0.0
(read/write) 114792 114792 0 0.0
.bss 50368 50368 0 0.0
.data 1320 1320 0 0.0
.data.rel.ro 57480 57480 0 0.0
.dynamic 608 608 0 0.0
.got 4352 4352 0 0.0
.init 27 27 0 0.0
.init_array 608 608 0 0.0
.rodata 158195 158195 0 0.0
.text 1557714 1557714 0 0.0
ota-requestor-app debug (read only) 1876945 1876945 0 0.0
(read/write) 116560 116560 0 0.0
.bss 51232 51232 0 0.0
.data 1192 1192 0 0.0
.data.rel.ro 58744 58744 0 0.0
.dynamic 592 592 0 0.0
.got 4152 4152 0 0.0
.init 27 27 0 0.0
.init_array 616 616 0 0.0
.rodata 151940 151940 0 0.0
.text 1576194 1576194 0 0.0
shell debug (read only) 2360553 2360553 0 0.0
(read/write) 147168 147168 0 0.0
.bss 67552 67552 0 0.0
.data 864 864 0 0.0
.data.rel.ro 73096 73096 0 0.0
.dynamic 592 592 0 0.0
.got 4144 4144 0 0.0
.init 27 27 0 0.0
.init_array 888 888 0 0.0
.rodata 204178 204178 0 0.0
.text 1999842 1999842 0 0.0
thermostat-no-ble arm64 (read only) 2161012 2161012 0 0.0
(read/write) 148577 148577 0 0.0
.bss 65265 65265 0 0.0
.data 1056 1056 0 0.0
.data.rel.ro 75080 75080 0 0.0
.dynamic 560 560 0 0.0
.got 4216 4216 0 0.0
.init 24 24 0 0.0
.init_array 352 352 0 0.0
.rodata 132756 132756 0 0.0
.text 1809952 1809952 0 0.0
tv-app debug (read only) 2586937 2586937 0 0.0
(read/write) 148624 148624 0 0.0
.bss 65824 65824 0 0.0
.data 3168 3168 0 0.0
.data.rel.ro 73624 73624 0 0.0
.dynamic 592 592 0 0.0
.got 4528 4528 0 0.0
.init 27 27 0 0.0
.init_array 872 872 0 0.0
.rodata 198173 198173 0 0.0
.text 2215026 2215026 0 0.0
mbed all-clusters-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2449520 2449520 0 0.0
.bss 191016 191016 0 0.0
.data 5512 5512 0 0.0
.text 1412092 1412092 0 0.0
lighting-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2414280 2414280 0 0.0
.bss 188188 188188 0 0.0
.data 5800 5800 0 0.0
.text 1376852 1376852 0 0.0
lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2350036 2350036 0 0.0
.bss 187092 187092 0 0.0
.data 5776 5776 0 0.0
.text 1312636 1312636 0 0.0
pigweed-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 1155428 1155428 0 0.0
.bss 11952 11952 0 0.0
.data 4512 4512 0 0.0
.text 118812 118812 0 0.0
shell CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2340032 2340032 0 0.0
.bss 185728 185728 0 0.0
.data 5608 5608 0 0.0
.text 1302604 1302604 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1038379 1038379 0 0.0
bss 125752 125752 0 0.0
rodata 135772 135772 0 0.0
text 699640 699640 0 0.0
lighting-app nrf52840dk_nrf52840 (read/write) 1001607 1001607 0 0.0
bss 122060 122060 0 0.0
rodata 116016 116016 0 0.0
text 686020 686020 0 0.0
nrf52840dk_nrf52840+rpc (read/write) 967919 967919 0 0.0
bss 119144 119144 0 0.0
rodata 107392 107392 0 0.0
text 663244 663244 0 0.0
nrf52840dongle_nrf52840 (read/write) 1017679 1017679 0 0.0
bss 123108 123108 0 0.0
rodata 115700 115700 0 0.0
text 690724 690724 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 904418 904418 0 0.0
bss 119348 119348 0 0.0
rodata 108128 108128 0 0.0
text 595732 595732 0 0.0
lock-app nrf52840dk_nrf52840 (read/write) 930695 930695 0 0.0
bss 120276 120276 0 0.0
rodata 104848 104848 0 0.0
text 628504 628504 0 0.0
nrf5340dk_nrf5340_cpuapp (read/write) 834318 834318 0 0.0
bss 117596 117596 0 0.0
rodata 96876 96876 0 0.0
text 538980 538980 0 0.0
pigweed-app nrf52840dk_nrf52840 (read/write) 501603 501603 0 0.0
bss 52584 52584 0 0.0
rodata 44500 44500 0 0.0
text 342712 342712 0 0.0
pump-app nrf52840dk_nrf52840 (read/write) 928939 928939 0 0.0
bss 120008 120008 0 0.0
rodata 103828 103828 0 0.0
text 627932 627932 0 0.0
pump-controller-app nrf52840dk_nrf52840 (read/write) 924915 924915 0 0.0
bss 120012 120012 0 0.0
rodata 103524 103524 0 0.0
text 624188 624188 0 0.0
shell nrf52840dk_nrf52840 (read/write) 788795 788795 0 0.0
bss 112360 112360 0 0.0
rodata 74836 74836 0 0.0
text 525496 525496 0 0.0
p6 all-clusters-app default (read/write) 2487472 2487472 0 0.0
.bss 118656 118656 0 0.0
.data 2688 2688 0 0.0
.text 1445736 1445736 0 0.0
light-app default (read/write) 2394696 2394696 0 0.0
.bss 112976 112976 0 0.0
.data 2536 2536 0 0.0
.text 1352960 1352960 0 0.0
lock-app default (read/write) 2358256 2358256 0 0.0
.bss 112720 112720 0 0.0
.data 2496 2496 0 0.0
.text 1316520 1316520 0 0.0
qpg lighting-app qpg6105+debug (read only) 600648 600648 0 0.0
(read/write) 146936 146936 0 0.0
.bss 90904 90904 0 0.0
.data 1124 1124 0 0.0
.text 595328 595328 0 0.0
lock-app qpg6105+debug (read only) 566452 566452 0 0.0
(read/write) 146936 146936 0 0.0
.bss 90912 90912 0 0.0
.data 1076 1076 0 0.0
.text 561132 561132 0 0.0
persistent-storage-app qpg6105+debug (read only) 99536 99536 0 0.0
(read/write) 146941 146941 0 0.0
.bss 24001 24001 0 0.0
.data 180 180 0 0.0
.text 94216 94216 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 882546 882546 0 0.0
bss 86336 86336 0 0.0
noinit 37160 37160 0 0.0
text 623544 623544 0 0.0

@andy31415 andy31415 merged commit 86c036c into project-chip:master Feb 25, 2022
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