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.

Sign up for GitHub

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

Jump to bottom

[OpenThread] Improve OpenThread SED polling logs #23780

Merged
merged 1 commit into from
Nov 29, 2022
Merged

[OpenThread] Improve OpenThread SED polling logs #23780

merged 1 commit into from
Nov 29, 2022

Conversation

jepenven-silabs
Copy link
Contributor

Problem :

Current Log assumes that the polling interval will be used if the code returns OT_NO_ERROR.

Actual behaviour is that the OpenThread stack will always use the smallest of the Polling Period and of the MLE_DEFAULT_TIMEOUT.

Update the Logs to show the actual value used by the OpenThread stack.

@github-actions
Copy link

github-actions bot commented Nov 28, 2022 •
edited
Loading

PR #23780: Size comparison from 5ec87ed to 2be5e0d

Increases (6 builds for bl702, esp32, k32w, psoc6, telink)
platform target config section 5ec87ed 2be5e0d change % change
bl702 lighting-app bl702 .debug_info 39168506 39168507 1 0.0
.text 957258 957260 2 0.0
bl702+rpc .debug_info 43466336 43466337 1 0.0
.text 1031098 1031100 2 0.0
esp32 all-clusters-app m5stack (read/write) 562692 562700 8 0.0
.flash.rodata 313664 313672 8 0.0
k32w lock k32w0+release (read/write) 634140 634156 16 0.0
.text 553660 553676 16 0.0
psoc6 all-clusters cy8ckit_062s2_43012 .debug_info 27036126 27036127 1 0.0
telink all-clusters-app tlsr9518adk80d (read/write) 939060 939068 8 0.0
text 650268 650270 2 0.0
Decreases (8 builds for bl602, esp32, nrfconnect, psoc6, telink)
platform target config section 5ec87ed 2be5e0d change % change
bl602 lighting-app bl602 .text 1061272 1061270 -2 -0.0
esp32 all-clusters-app c3devkit (read only) 1214742 1214740 -2 -0.0
.flash.text 1214742 1214740 -2 -0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 text 819872 819868 -4 -0.0
psoc6 all-clusters-minimal cy8ckit_062s2_43012 .debug_info 26773628 26773626 -2 -0.0
telink all-clusters-minimal-app tlsr9518adk80d text 614228 614226 -2 -0.0
light-switch-app tlsr9518adk80d text 555362 555358 -4 -0.0
ota-requestor-app tlsr9518adk80d (read/write) 806552 806544 -8 -0.0
text 566700 566696 -4 -0.0
thermostat tlsr9518adk80d (read/write) 793356 793348 -8 -0.0
text 554336 554332 -4 -0.0
Full report (42 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section 5ec87ed 2be5e0d change % change
bl602 lighting-app bl602 (read/write) 1377590 1377590 0 0.0
.bss 90041 90041 0 0.0
.data 9984 9984 0 0.0
.text 1061272 1061270 -2 -0.0
bl602+rpc (read/write) 1423270 1423270 0 0.0
.bss 98081 98081 0 0.0
.data 10376 10376 0 0.0
.text 1092604 1092604 0 0.0
bl702 lighting-app bl702 (read only) 3262 3262 0 0.0
(read/write) 1197127 1197127 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 67166 67166 0 0.0
.bss_psram 30048 30048 0 0.0
.comment 48 48 0 0.0
.data 4048 4048 0 0.0
.debug_abbrev 1524330 1524330 0 0.0
.debug_aranges 132440 132440 0 0.0
.debug_frame 485880 485880 0 0.0
.debug_info 39168506 39168507 1 0.0
.debug_line 5276299 5276299 0 0.0
.debug_loc 3373589 3373589 0 0.0
.debug_ranges 361264 361264 0 0.0
.debug_str 3475236 3475236 0 0.0
.hbn 509 509 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 144 144 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 116744 116744 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 568438 568438 0 0.0
.symtab 172080 172080 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
957258 957260 2 0.0
bl702+rpc (read only) 3262 3262 0 0.0
(read/write) 1285475 1285475 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 75214 75214 0 0.0
.bss_psram 30320 30320 0 0.0
.comment 48 48 0 0.0
.data 4576 4576 0 0.0
.debug_abbrev 1671433 1671433 0 0.0
.debug_aranges 140576 140576 0 0.0
.debug_frame 513304 513304 0 0.0
.debug_info 43466336 43466337 1 0.0
.debug_line 5695030 5695030 0 0.0
.debug_loc 3568815 3568815 0 0.0
.debug_ranges 384752 384752 0 0.0
.debug_str 3874745 3874745 0 0.0
.hbn 509 509 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 160 160 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 130440 130440 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 628592 628592 0 0.0
.symtab 190224 190224 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
1031098 1031100 2 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 679555 679555 0 0.0
(read/write) 171980 171980 0 0.0
.bss 81228 81228 0 0.0
.data 3380 3380 0 0.0
.rodata 89923 89923 0 0.0
.text 589320 589320 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 643811 643811 0 0.0
(read/write) 157996 157996 0 0.0
.bss 80500 80500 0 0.0
.data 3380 3380 0 0.0
.rodata 79059 79059 0 0.0
.text 564432 564432 0 0.0
lock-ftd LP_CC2652R7 (read only) 677087 677087 0 0.0
(read/write) 172016 172016 0 0.0
.bss 78884 78884 0 0.0
.data 3304 3304 0 0.0
.rodata 77351 77351 0 0.0
.text 599256 599256 0 0.0
lock-mtd LP_CC2652R7 (read only) 661875 661875 0 0.0
(read/write) 182484 182484 0 0.0
.bss 74140 74140 0 0.0
.data 3304 3304 0 0.0
.rodata 103435 103435 0 0.0
.text 557960 557960 0 0.0
pump-app LP_CC2652R7 (read only) 690095 690095 0 0.0
(read/write) 159744 159744 0 0.0
.bss 78852 78852 0 0.0
.data 3296 3296 0 0.0
.rodata 90927 90927 0 0.0
.text 598684 598684 0 0.0
pump-controller-app LP_CC2652R7 (read only) 673931 673931 0 0.0
(read/write) 176004 176004 0 0.0
.bss 78948 78948 0 0.0
.data 3292 3292 0 0.0
.rodata 86427 86427 0 0.0
.text 587024 587024 0 0.0
shell LP_CC2652R7 (read only) 670766 670766 0 0.0
(read/write) 183088 183088 0 0.0
.bss 83548 83548 0 0.0
.data 3376 3376 0 0.0
.rodata 86670 86670 0 0.0
.text 583780 583780 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 588446 588446 0 0.0
.app_xip_area 464696 464696 0 0.0
.bss 66208 66208 0 0.0
.data 728 728 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 592146 592146 0 0.0
.app_xip_area 463100 463100 0 0.0
.bss 71496 71496 0 0.0
.data 736 736 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 544826 544826 0 0.0
.app_xip_area 426588 426588 0 0.0
.bss 60736 60736 0 0.0
.data 684 684 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A+rpc (read/write) 976132 976132 0 0.0
.bss 152260 152260 0 0.0
.data 2168 2168 0 0.0
.text 821684 821684 0 0.0
BRD4161A+rs911x (read/write) 1037704 1037704 0 0.0
.bss 186736 186736 0 0.0
.data 2012 2012 0 0.0
.text 848936 848936 0 0.0
BRD4187C (read/write) 1149260 1149260 0 0.0
.bss 138656 138656 0 0.0
.data 2516 2516 0 0.0
.text 983492 983492 0 0.0
lock-app BRD4161A+wf200 (read/write) 1163256 1163256 0 0.0
.bss 158304 158304 0 0.0
.data 2020 2020 0 0.0
.text 1002912 1002912 0 0.0
window-app BRD4187C (read/write) 1141728 1141728 0 0.0
.bss 140072 140072 0 0.0
.data 2540 2540 0 0.0
.text 974520 974520 0 0.0
esp32 all-clusters-app c3devkit (read only) 1214742 1214740 -2 -0.0
(read/write) 1786718 1786718 0 0.0
.dram0.bss 76688 76688 0 0.0
.dram0.data 13840 13840 0 0.0
.flash.rodata 256544 256544 0 0.0
.flash.text 1214742 1214740 -2 -0.0
.iram0.text 65204 65204 0 0.0
m5stack (read only) 1225499 1225499 0 0.0
(read/write) 562692 562700 8 0.0
.dram0.bss 82064 82064 0 0.0
.dram0.data 34296 34296 0 0.0
.flash.rodata 313664 313672 8 0.0
.flash.text 1220115 1220115 0 0.0
.iram0.text 123939 123939 0 0.0
k32w contact k32w0+release (read/write) 662612 662612 0 0.0
.bss 77112 77112 0 0.0
.data 2104 2104 0 0.0
.text 564284 564284 0 0.0
light k32w0+release (read/write) 673264 673264 0 0.0
.bss 74912 74912 0 0.0
.data 2060 2060 0 0.0
.text 593564 593564 0 0.0
lock k32w0+release (read/write) 634140 634156 16 0.0
.bss 75672 75672 0 0.0
.data 2080 2080 0 0.0
.text 553660 553676 16 0.0
linux chip-tool-ipv6only arm64 (read only) 10450060 10450060 0 0.0
(read/write) 707713 707713 0 0.0
.bss 33905 33905 0 0.0
.data 2768 2768 0 0.0
.data.rel.ro 652008 652008 0 0.0
.dynamic 560 560 0 0.0
.got 13944 13944 0 0.0
.init 24 24 0 0.0
.init_array 208 208 0 0.0
.rodata 519468 519468 0 0.0
.text 8271988 8271988 0 0.0
thermostat-no-ble arm64 (read only) 2387012 2387012 0 0.0
(read/write) 143665 143665 0 0.0
.bss 55329 55329 0 0.0
.data 1816 1816 0 0.0
.data.rel.ro 77256 77256 0 0.0
.dynamic 560 560 0 0.0
.got 5192 5192 0 0.0
.init 24 24 0 0.0
.init_array 440 440 0 0.0
.rodata 145068 145068 0 0.0
.text 1998048 1998048 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2453760 2453760 0 0.0
.bss 215012 215012 0 0.0
.data 5872 5872 0 0.0
.text 1416404 1416404 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1190779 1190779 0 0.0
bss 145289 145289 0 0.0
rodata 144464 144464 0 0.0
text 819872 819868 -4 -0.0
nrf7002dk_nrf5340_cpuapp (read/write) 1360906 1360906 0 0.0
bss 105228 105228 0 0.0
rodata 211568 211568 0 0.0
text 759626 759626 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1136439 1136439 0 0.0
bss 144524 144524 0 0.0
rodata 120780 120780 0 0.0
text 790040 790040 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read only) 841968 841968 0 0.0
(read/write) 1748508 1748508 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 188712 188712 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2664 2664 0 0.0
.debug_abbrev 1235861 1235861 0 0.0
.debug_aranges 110752 110752 0 0.0
.debug_frame 371636 371636 0 0.0
.debug_info 27036126 27036127 1 0.0
.debug_line 3691230 3691230 0 0.0
.debug_loc 3616230 3616230 0 0.0
.debug_ranges 340824 340824 0 0.0
.debug_str 3444720 3444720 0 0.0
.heap 841968 841968 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 571529 571529 0 0.0
.symtab 422000 422000 0 0.0
.text 1548744 1548744 0 0.0
.zero.table 8 8 0 0.0
text 0 0 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 (read only) 842704 842704 0 0.0
(read/write) 1691028 1691028 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 187976 187976 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2664 2664 0 0.0
.debug_abbrev 1227752 1227752 0 0.0
.debug_aranges 110224 110224 0 0.0
.debug_frame 374692 374692 0 0.0
.debug_info 26773628 26773626 -2 -0.0
.debug_line 3712210 3712210 0 0.0
.debug_loc 3603836 3603836 0 0.0
.debug_ranges 339440 339440 0 0.0
.debug_str 3433732 3433732 0 0.0
.heap 842704 842704 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 535618 535618 0 0.0
.symtab 408432 408432 0 0.0
.text 1492000 1492000 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
light cy8ckit_062s2_43012 (read only) 850888 850888 0 0.0
(read/write) 1609228 1609228 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 180000 180000 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2456 2456 0 0.0
.debug_abbrev 1062275 1062275 0 0.0
.debug_aranges 102424 102424 0 0.0
.debug_frame 345028 345028 0 0.0
.debug_info 22239421 22239421 0 0.0
.debug_line 3281562 3281562 0 0.0
.debug_loc 3301679 3301679 0 0.0
.debug_ranges 304704 304704 0 0.0
.debug_str 3239318 3239318 0 0.0
.heap 850888 850888 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 471943 471943 0 0.0
.symtab 376800 376800 0 0.0
.text 1418384 1418384 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
lock cy8ckit_062s2_43012 (read only) 845896 845896 0 0.0
(read/write) 1642580 1642580 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 184976 184976 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2472 2472 0 0.0
.debug_abbrev 1064266 1064266 0 0.0
.debug_aranges 102816 102816 0 0.0
.debug_frame 346848 346848 0 0.0
.debug_info 22458796 22458796 0 0.0
.debug_line 3276288 3276288 0 0.0
.debug_loc 3323053 3323053 0 0.0
.debug_ranges 306080 306080 0 0.0
.debug_str 3255210 3255210 0 0.0
.heap 845896 845896 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 473659 473659 0 0.0
.symtab 378416 378416 0 0.0
.text 1446744 1446744 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1151028 1151028 0 0.0
.bss 97628 97628 0 0.0
.data 840 840 0 0.0
.text 598128 598128 0 0.0
lock-app qpg6105+debug (read/write) 1116828 1116828 0 0.0
.bss 93100 93100 0 0.0
.data 836 836 0 0.0
.text 563924 563924 0 0.0
telink all-clusters-app tlsr9518adk80d (read/write) 939060 939068 8 0.0
bss 81620 81620 0 0.0
noinit 43440 43440 0 0.0
text 650268 650270 2 0.0
all-clusters-minimal-app tlsr9518adk80d (read/write) 877028 877028 0 0.0
bss 80844 80844 0 0.0
noinit 43440 43440 0 0.0
text 614228 614226 -2 -0.0
light-switch-app tlsr9518adk80d (read/write) 792184 792184 0 0.0
bss 72468 72468 0 0.0
noinit 43520 43520 0 0.0
text 555362 555358 -4 -0.0
lighting-app tlsr9518adk80d (read/write) 876568 876568 0 0.0
bss 82232 82232 0 0.0
noinit 43520 43520 0 0.0
text 625612 625612 0 0.0
ota-requestor-app tlsr9518adk80d (read/write) 806552 806544 -8 -0.0
bss 74040 74040 0 0.0
noinit 43520 43520 0 0.0
text 566700 566696 -4 -0.0
thermostat tlsr9518adk80d (read/write) 793356 793348 -8 -0.0
bss 73852 73852 0 0.0
noinit 43480 43480 0 0.0
text 554336 554332 -4 -0.0

@andy31415 andy31415 merged commit 21b3c30 into project-chip:master Nov 29, 2022
yicheng810 pushed a commit to yicheng810/connectedhomeip that referenced this pull request Jun 27, 2023
@hsusid
Merge branch master into develop for project-chip#23780 and project-c…
e1f2199 
…hip#23861.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andy31415 andy31415 andy31415 approved these changes

@jmartinez-silabs jmartinez-silabs jmartinez-silabs approved these changes

@anush-apple anush-apple Awaiting requested review from anush-apple

@arkq arkq Awaiting requested review from arkq

@Byungjoo-Lee Byungjoo-Lee Awaiting requested review from Byungjoo-Lee

@bzbarsky-apple bzbarsky-apple Awaiting requested review from bzbarsky-apple

@carol-apple carol-apple Awaiting requested review from carol-apple

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@Damian-Nordic Damian-Nordic Awaiting requested review from Damian-Nordic

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@emargolis emargolis Awaiting requested review from emargolis

@franck-apple franck-apple Awaiting requested review from franck-apple

@gjc13 gjc13 Awaiting requested review from gjc13

@harimau-qirex harimau-qirex Awaiting requested review from harimau-qirex

@harsha-rajendran harsha-rajendran Awaiting requested review from harsha-rajendran

@hawk248 hawk248 Awaiting requested review from hawk248

@isiu-apple isiu-apple Awaiting requested review from isiu-apple

@jelderton jelderton Awaiting requested review from jelderton

@jtung-apple jtung-apple Awaiting requested review from jtung-apple

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

@mlepage-google mlepage-google Awaiting requested review from mlepage-google

@mrjerryjohns mrjerryjohns Awaiting requested review from mrjerryjohns

@msandstedt msandstedt Awaiting requested review from msandstedt

@mspang mspang Awaiting requested review from mspang

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

@tecimovic tecimovic Awaiting requested review from tecimovic

@turon turon Awaiting requested review from turon

@vijs vijs Awaiting requested review from vijs

@vivien-apple vivien-apple Awaiting requested review from vivien-apple

@woody-apple woody-apple Awaiting requested review from woody-apple

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

@yunhanw-google yunhanw-google Awaiting requested review from yunhanw-google

Assignees
No one assigned
Labels
platform review - approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jepenven-silabs @andy31415 @jmartinez-silabs