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

Fix fabric removal logic in group counter #15977

Conversation

jepenven-silabs
Copy link
Contributor

Problem

PR #15874 highlighted an issue with fabric removal in GroupPeerMessageCounter

Change overview

Clear the table entry before compacting the list.

Testing

Unit Test

@github-actions
Copy link

github-actions bot commented Mar 8, 2022

PR #15977: Size comparison from a69fde3 to 1fb2ba5

Increases (23 builds for cyw30739, efr32, k32w, linux, p6, telink)
platform target config section a69fde3 1fb2ba5 change % change
cyw30739 light cyw930739m2evb_01 (read/write) 602090 602114 24 0.0
.app_xip_area 506608 506632 24 0.0
lock cyw930739m2evb_01 (read/write) 559998 560022 24 0.0
.app_xip_area 466052 466076 24 0.0
ota-requestor cyw930739m2evb_01 (read/write) 582190 582214 24 0.0
.app_xip_area 478564 478588 24 0.0
efr32 lighting-app BRD4161A (read only) 921344 921440 96 0.0
.text 921336 921432 96 0.0
BRD4161A+rpc (read only) 950180 950276 96 0.0
.text 950172 950268 96 0.0
window-app BRD4161A (read only) 854380 854476 96 0.0
.text 854372 854468 96 0.0
k32w light k32w061+release (read/write) 695256 695272 16 0.0
.text 609764 609780 16 0.0
lock k32w061+release (read/write) 696860 696876 16 0.0
.text 611392 611408 16 0.0
linux all-clusters-app debug (read only) 2428721 2428785 64 0.0
.text 2060194 2060258 64 0.0
bridge-app debug+rpc (read only) 1733421 1733485 64 0.0
.text 1475701 1475765 64 0.0
chip-tool debug (read only) 9762741 9762805 64 0.0
.text 8512389 8512453 64 0.0
chip-tool-ipv6only arm64 (read only) 9434788 9434836 48 0.0
.text 7944148 7944196 48 0.0
door-lock-app debug (read only) 1947745 1947809 64 0.0
.text 1625442 1625506 64 0.0
lighting-app debug+rpc (read only) 2080025 2080089 64 0.0
.text 1758546 1758610 64 0.0
ota-provider-app debug (read only) 1880601 1880665 64 0.0
.text 1570978 1571042 64 0.0
ota-requestor-app debug (read only) 1902617 1902681 64 0.0
.text 1598338 1598402 64 0.0
shell debug (read only) 2394833 2394897 64 0.0
.text 2032802 2032866 64 0.0
thermostat-no-ble arm64 (read only) 2176500 2176548 48 0.0
.text 1824912 1824960 48 0.0
tv-app debug (read only) 2620537 2620601 64 0.0
.text 2246994 2247058 64 0.0
p6 all-clusters-app default (read/write) 2494568 2494664 96 0.0
.text 1452832 1452928 96 0.0
light-app default (read/write) 2398000 2398096 96 0.0
.text 1356264 1356360 96 0.0
lock-app default (read/write) 2361432 2361528 96 0.0
.text 1319696 1319792 96 0.0
telink lighting-app tlsr9518adk80d (read/write) 889534 889614 80 0.0
text 628840 628914 74 0.0
Full report (23 builds for cyw30739, efr32, k32w, linux, p6, telink)
platform target config section a69fde3 1fb2ba5 change % change
cyw30739 light cyw930739m2evb_01 (read/write) 602090 602114 24 0.0
.app_xip_area 506608 506632 24 0.0
.bss 78172 78172 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) 559998 560022 24 0.0
.app_xip_area 466052 466076 24 0.0
.bss 76676 76676 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) 582190 582214 24 0.0
.app_xip_area 478564 478588 24 0.0
.bss 86004 86004 0 0.0
.data 584 584 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 921344 921440 96 0.0
(read/write) 128872 128872 0 0.0
.bss 126800 126800 0 0.0
.data 2072 2072 0 0.0
.text 921336 921432 96 0.0
BRD4161A+rpc (read only) 950180 950276 96 0.0
(read/write) 144828 144828 0 0.0
.bss 142576 142576 0 0.0
.data 2252 2252 0 0.0
.text 950172 950268 96 0.0
window-app BRD4161A (read only) 854380 854476 96 0.0
(read/write) 126852 126852 0 0.0
.bss 124936 124936 0 0.0
.data 1916 1916 0 0.0
.text 854372 854468 96 0.0
k32w light k32w061+release (read/write) 695256 695272 16 0.0
.bss 77736 77736 0 0.0
.data 1956 1956 0 0.0
.text 609764 609780 16 0.0
lock k32w061+release (read/write) 696860 696876 16 0.0
.bss 77704 77704 0 0.0
.data 1964 1964 0 0.0
.text 611392 611408 16 0.0
linux all-clusters-app debug (read only) 2428721 2428785 64 0.0
(read/write) 145712 145712 0 0.0
.bss 60192 60192 0 0.0
.data 1360 1360 0 0.0
.data.rel.ro 78424 78424 0 0.0
.dynamic 592 592 0 0.0
.got 4176 4176 0 0.0
.init 27 27 0 0.0
.init_array 936 936 0 0.0
.rodata 205541 205541 0 0.0
.text 2060194 2060258 64 0.0
bridge-app debug+rpc (read only) 1733421 1733485 64 0.0
(read/write) 92656 92656 0 0.0
.bss 47496 47496 0 0.0
.data 2016 2016 0 0.0
.data.rel.ro 38064 38064 0 0.0
.dynamic 592 592 0 0.0
.got 3928 3928 0 0.0
.init 27 27 0 0.0
.init_array 544 544 0 0.0
.rodata 141748 141748 0 0.0
.text 1475701 1475765 64 0.0
chip-tool debug (read only) 9762741 9762805 64 0.0
(read/write) 338616 338616 0 0.0
.bss 25512 25512 0 0.0
.data 1152 1152 0 0.0
.data.rel.ro 305928 305928 0 0.0
.dynamic 608 608 0 0.0
.got 4776 4776 0 0.0
.init 27 27 0 0.0
.init_array 616 616 0 0.0
.rodata 510868 510868 0 0.0
.text 8512389 8512453 64 0.0
chip-tool-ipv6only arm64 (read only) 9434788 9434836 48 0.0
(read/write) 454337 454337 0 0.0
.bss 43825 43825 0 0.0
.data 1200 1200 0 0.0
.data.rel.ro 351136 351136 0 0.0
.dynamic 560 560 0 0.0
.got 54376 54376 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 486588 486588 0 0.0
.text 7944148 7944196 48 0.0
door-lock-app debug (read only) 1947745 1947809 64 0.0
(read/write) 119136 119136 0 0.0
.bss 50496 50496 0 0.0
.data 1056 1056 0 0.0
.data.rel.ro 62184 62184 0 0.0
.dynamic 592 592 0 0.0
.got 4112 4112 0 0.0
.init 27 27 0 0.0
.init_array 664 664 0 0.0
.rodata 173660 173660 0 0.0
.text 1625442 1625506 64 0.0
lighting-app debug+rpc (read only) 2080025 2080089 64 0.0
(read/write) 124456 124456 0 0.0
.bss 51296 51296 0 0.0
.data 1536 1536 0 0.0
.data.rel.ro 66104 66104 0 0.0
.dynamic 608 608 0 0.0
.got 4144 4144 0 0.0
.init 27 27 0 0.0
.init_array 736 736 0 0.0
.rodata 165820 165820 0 0.0
.text 1758546 1758610 64 0.0
ota-provider-app debug (read only) 1880601 1880665 64 0.0
(read/write) 114736 114736 0 0.0
.bss 50304 50304 0 0.0
.data 1320 1320 0 0.0
.data.rel.ro 57464 57464 0 0.0
.dynamic 608 608 0 0.0
.got 4368 4368 0 0.0
.init 27 27 0 0.0
.init_array 616 616 0 0.0
.rodata 158899 158899 0 0.0
.text 1570978 1571042 64 0.0
ota-requestor-app debug (read only) 1902617 1902681 64 0.0
(read/write) 117448 117448 0 0.0
.bss 51360 51360 0 0.0
.data 1480 1480 0 0.0
.data.rel.ro 59176 59176 0 0.0
.dynamic 592 592 0 0.0
.got 4160 4160 0 0.0
.init 27 27 0 0.0
.init_array 640 640 0 0.0
.rodata 154220 154220 0 0.0
.text 1598338 1598402 64 0.0
shell debug (read only) 2394833 2394897 64 0.0
(read/write) 149928 149928 0 0.0
.bss 70312 70312 0 0.0
.data 864 864 0 0.0
.data.rel.ro 73064 73064 0 0.0
.dynamic 592 592 0 0.0
.got 4160 4160 0 0.0
.init 27 27 0 0.0
.init_array 920 920 0 0.0
.rodata 205458 205458 0 0.0
.text 2032802 2032866 64 0.0
thermostat-no-ble arm64 (read only) 2176500 2176548 48 0.0
(read/write) 148673 148673 0 0.0
.bss 65281 65281 0 0.0
.data 1048 1048 0 0.0
.data.rel.ro 75136 75136 0 0.0
.dynamic 560 560 0 0.0
.got 4240 4240 0 0.0
.init 24 24 0 0.0
.init_array 352 352 0 0.0
.rodata 133012 133012 0 0.0
.text 1824912 1824960 48 0.0
tv-app debug (read only) 2620537 2620601 64 0.0
(read/write) 250840 250840 0 0.0
.bss 168000 168000 0 0.0
.data 3168 3168 0 0.0
.data.rel.ro 73624 73624 0 0.0
.dynamic 592 592 0 0.0
.got 4544 4544 0 0.0
.init 27 27 0 0.0
.init_array 880 880 0 0.0
.rodata 199453 199453 0 0.0
.text 2246994 2247058 64 0.0
p6 all-clusters-app default (read/write) 2494568 2494664 96 0.0
.bss 119544 119544 0 0.0
.data 2680 2680 0 0.0
.text 1452832 1452928 96 0.0
light-app default (read/write) 2398000 2398096 96 0.0
.bss 113008 113008 0 0.0
.data 2536 2536 0 0.0
.text 1356264 1356360 96 0.0
lock-app default (read/write) 2361432 2361528 96 0.0
.bss 112752 112752 0 0.0
.data 2496 2496 0 0.0
.text 1319696 1319792 96 0.0
telink lighting-app tlsr9518adk80d (read/write) 889534 889614 80 0.0
bss 86968 86968 0 0.0
noinit 37160 37160 0 0.0
text 628840 628914 74 0.0

@github-actions
Copy link

github-actions bot commented Mar 8, 2022

PR #15977: Size comparison from a69fde3 to 6a0864b

Increases (26 builds for cyw30739, efr32, esp32, k32w, linux, nrfconnect, p6, telink)
platform target config section a69fde3 6a0864b change % change
cyw30739 light cyw930739m2evb_01 (read/write) 602090 602114 24 0.0
.app_xip_area 506608 506632 24 0.0
lock cyw930739m2evb_01 (read/write) 559998 560022 24 0.0
.app_xip_area 466052 466076 24 0.0
ota-requestor cyw930739m2evb_01 (read/write) 582190 582214 24 0.0
.app_xip_area 478564 478588 24 0.0
efr32 lighting-app BRD4161A (read only) 921344 921440 96 0.0
.text 921336 921432 96 0.0
BRD4161A+rpc (read only) 950180 950276 96 0.0
.text 950172 950268 96 0.0
window-app BRD4161A (read only) 854380 854476 96 0.0
.text 854372 854468 96 0.0
esp32 all-clusters-app c3devkit (read only) 957942 957988 46 0.0
.flash.text 957942 957988 46 0.0
m5stack (read only) 1012743 1012787 44 0.0
.flash.text 1007359 1007403 44 0.0
k32w light k32w061+release (read/write) 695256 695272 16 0.0
.text 609764 609780 16 0.0
lock k32w061+release (read/write) 696860 696876 16 0.0
.text 611392 611408 16 0.0
linux all-clusters-app debug (read only) 2428721 2428785 64 0.0
.text 2060194 2060258 64 0.0
bridge-app debug+rpc (read only) 1733421 1733485 64 0.0
.text 1475701 1475765 64 0.0
chip-tool debug (read only) 9762741 9762805 64 0.0
.text 8512389 8512453 64 0.0
chip-tool-ipv6only arm64 (read only) 9434788 9434836 48 0.0
.text 7944148 7944196 48 0.0
door-lock-app debug (read only) 1947745 1947809 64 0.0
.text 1625442 1625506 64 0.0
lighting-app debug+rpc (read only) 2080025 2080089 64 0.0
.text 1758546 1758610 64 0.0
ota-provider-app debug (read only) 1880601 1880665 64 0.0
.text 1570978 1571042 64 0.0
ota-requestor-app debug (read only) 1902617 1902681 64 0.0
.text 1598338 1598402 64 0.0
shell debug (read only) 2394833 2394897 64 0.0
.text 2032802 2032866 64 0.0
thermostat-no-ble arm64 (read only) 2176500 2176548 48 0.0
.text 1824912 1824960 48 0.0
tv-app debug (read only) 2620537 2620601 64 0.0
.text 2246994 2247058 64 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1152335 1152383 48 0.0
text 785912 785964 52 0.0
p6 all-clusters-app default (read/write) 2494568 2494664 96 0.0
.text 1452832 1452928 96 0.0
light-app default (read/write) 2398000 2398096 96 0.0
.text 1356264 1356360 96 0.0
lock-app default (read/write) 2361432 2361528 96 0.0
.text 1319696 1319792 96 0.0
telink lighting-app tlsr9518adk80d (read/write) 889534 889614 80 0.0
text 628840 628914 74 0.0
Full report (26 builds for cyw30739, efr32, esp32, k32w, linux, nrfconnect, p6, telink)
platform target config section a69fde3 6a0864b change % change
cyw30739 light cyw930739m2evb_01 (read/write) 602090 602114 24 0.0
.app_xip_area 506608 506632 24 0.0
.bss 78172 78172 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) 559998 560022 24 0.0
.app_xip_area 466052 466076 24 0.0
.bss 76676 76676 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) 582190 582214 24 0.0
.app_xip_area 478564 478588 24 0.0
.bss 86004 86004 0 0.0
.data 584 584 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 921344 921440 96 0.0
(read/write) 128872 128872 0 0.0
.bss 126800 126800 0 0.0
.data 2072 2072 0 0.0
.text 921336 921432 96 0.0
BRD4161A+rpc (read only) 950180 950276 96 0.0
(read/write) 144828 144828 0 0.0
.bss 142576 142576 0 0.0
.data 2252 2252 0 0.0
.text 950172 950268 96 0.0
window-app BRD4161A (read only) 854380 854476 96 0.0
(read/write) 126852 126852 0 0.0
.bss 124936 124936 0 0.0
.data 1916 1916 0 0.0
.text 854372 854468 96 0.0
esp32 all-clusters-app c3devkit (read only) 957942 957988 46 0.0
(read/write) 1393674 1393674 0 0.0
.dram0.bss 63480 63480 0 0.0
.dram0.data 14220 14220 0 0.0
.flash.rodata 197056 197056 0 0.0
.flash.text 957942 957988 46 0.0
.iram0.text 62016 62016 0 0.0
m5stack (read only) 1012743 1012787 44 0.0
(read/write) 460772 460772 0 0.0
.dram0.bss 69000 69000 0 0.0
.dram0.data 34064 34064 0 0.0
.flash.rodata 225872 225872 0 0.0
.flash.text 1007359 1007403 44 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 695256 695272 16 0.0
.bss 77736 77736 0 0.0
.data 1956 1956 0 0.0
.text 609764 609780 16 0.0
lock k32w061+release (read/write) 696860 696876 16 0.0
.bss 77704 77704 0 0.0
.data 1964 1964 0 0.0
.text 611392 611408 16 0.0
linux all-clusters-app debug (read only) 2428721 2428785 64 0.0
(read/write) 145712 145712 0 0.0
.bss 60192 60192 0 0.0
.data 1360 1360 0 0.0
.data.rel.ro 78424 78424 0 0.0
.dynamic 592 592 0 0.0
.got 4176 4176 0 0.0
.init 27 27 0 0.0
.init_array 936 936 0 0.0
.rodata 205541 205541 0 0.0
.text 2060194 2060258 64 0.0
bridge-app debug+rpc (read only) 1733421 1733485 64 0.0
(read/write) 92656 92656 0 0.0
.bss 47496 47496 0 0.0
.data 2016 2016 0 0.0
.data.rel.ro 38064 38064 0 0.0
.dynamic 592 592 0 0.0
.got 3928 3928 0 0.0
.init 27 27 0 0.0
.init_array 544 544 0 0.0
.rodata 141748 141748 0 0.0
.text 1475701 1475765 64 0.0
chip-tool debug (read only) 9762741 9762805 64 0.0
(read/write) 338616 338616 0 0.0
.bss 25512 25512 0 0.0
.data 1152 1152 0 0.0
.data.rel.ro 305928 305928 0 0.0
.dynamic 608 608 0 0.0
.got 4776 4776 0 0.0
.init 27 27 0 0.0
.init_array 616 616 0 0.0
.rodata 510868 510868 0 0.0
.text 8512389 8512453 64 0.0
chip-tool-ipv6only arm64 (read only) 9434788 9434836 48 0.0
(read/write) 454337 454337 0 0.0
.bss 43825 43825 0 0.0
.data 1200 1200 0 0.0
.data.rel.ro 351136 351136 0 0.0
.dynamic 560 560 0 0.0
.got 54376 54376 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 486588 486588 0 0.0
.text 7944148 7944196 48 0.0
door-lock-app debug (read only) 1947745 1947809 64 0.0
(read/write) 119136 119136 0 0.0
.bss 50496 50496 0 0.0
.data 1056 1056 0 0.0
.data.rel.ro 62184 62184 0 0.0
.dynamic 592 592 0 0.0
.got 4112 4112 0 0.0
.init 27 27 0 0.0
.init_array 664 664 0 0.0
.rodata 173660 173660 0 0.0
.text 1625442 1625506 64 0.0
lighting-app debug+rpc (read only) 2080025 2080089 64 0.0
(read/write) 124456 124456 0 0.0
.bss 51296 51296 0 0.0
.data 1536 1536 0 0.0
.data.rel.ro 66104 66104 0 0.0
.dynamic 608 608 0 0.0
.got 4144 4144 0 0.0
.init 27 27 0 0.0
.init_array 736 736 0 0.0
.rodata 165820 165820 0 0.0
.text 1758546 1758610 64 0.0
ota-provider-app debug (read only) 1880601 1880665 64 0.0
(read/write) 114736 114736 0 0.0
.bss 50304 50304 0 0.0
.data 1320 1320 0 0.0
.data.rel.ro 57464 57464 0 0.0
.dynamic 608 608 0 0.0
.got 4368 4368 0 0.0
.init 27 27 0 0.0
.init_array 616 616 0 0.0
.rodata 158899 158899 0 0.0
.text 1570978 1571042 64 0.0
ota-requestor-app debug (read only) 1902617 1902681 64 0.0
(read/write) 117448 117448 0 0.0
.bss 51360 51360 0 0.0
.data 1480 1480 0 0.0
.data.rel.ro 59176 59176 0 0.0
.dynamic 592 592 0 0.0
.got 4160 4160 0 0.0
.init 27 27 0 0.0
.init_array 640 640 0 0.0
.rodata 154220 154220 0 0.0
.text 1598338 1598402 64 0.0
shell debug (read only) 2394833 2394897 64 0.0
(read/write) 149928 149928 0 0.0
.bss 70312 70312 0 0.0
.data 864 864 0 0.0
.data.rel.ro 73064 73064 0 0.0
.dynamic 592 592 0 0.0
.got 4160 4160 0 0.0
.init 27 27 0 0.0
.init_array 920 920 0 0.0
.rodata 205458 205458 0 0.0
.text 2032802 2032866 64 0.0
thermostat-no-ble arm64 (read only) 2176500 2176548 48 0.0
(read/write) 148673 148673 0 0.0
.bss 65281 65281 0 0.0
.data 1048 1048 0 0.0
.data.rel.ro 75136 75136 0 0.0
.dynamic 560 560 0 0.0
.got 4240 4240 0 0.0
.init 24 24 0 0.0
.init_array 352 352 0 0.0
.rodata 133012 133012 0 0.0
.text 1824912 1824960 48 0.0
tv-app debug (read only) 2620537 2620601 64 0.0
(read/write) 250840 250840 0 0.0
.bss 168000 168000 0 0.0
.data 3168 3168 0 0.0
.data.rel.ro 73624 73624 0 0.0
.dynamic 592 592 0 0.0
.got 4544 4544 0 0.0
.init 27 27 0 0.0
.init_array 880 880 0 0.0
.rodata 199453 199453 0 0.0
.text 2246994 2247058 64 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1152335 1152383 48 0.0
bss 146832 146832 0 0.0
rodata 144328 144328 0 0.0
text 785912 785964 52 0.0
p6 all-clusters-app default (read/write) 2494568 2494664 96 0.0
.bss 119544 119544 0 0.0
.data 2680 2680 0 0.0
.text 1452832 1452928 96 0.0
light-app default (read/write) 2398000 2398096 96 0.0
.bss 113008 113008 0 0.0
.data 2536 2536 0 0.0
.text 1356264 1356360 96 0.0
lock-app default (read/write) 2361432 2361528 96 0.0
.bss 112752 112752 0 0.0
.data 2496 2496 0 0.0
.text 1319696 1319792 96 0.0
telink lighting-app tlsr9518adk80d (read/write) 889534 889614 80 0.0
bss 86968 86968 0 0.0
noinit 37160 37160 0 0.0
text 628840 628914 74 0.0

@jepenven-silabs jepenven-silabs merged commit b6e4b58 into project-chip:master Mar 9, 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