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

Disallow opening commissioning windows when fail-safe is not fully disarmed. #21111

Merged
merged 1 commit into from
Jul 22, 2022
Merged

Disallow opening commissioning windows when fail-safe is not fully disarmed. #21111

merged 1 commit into from
Jul 22, 2022

Conversation

bzbarsky-apple
Copy link
Contributor

We were allowing opening commissioning windows in the "fail-safe is
busy" state. We should probably not allow that.

Problem

Possible to open a commissioning window while an async fail-safe disarm is in progress.

Change overview

Disallow that, just list we disallow doing it while a fail-safe is fully armed.

@bzbarsky-apple
Disallow opening commissioning windows when fail-safe is not fully di…
26685dd 
…sarmed.

We were allowing opening commissioning windows in the "fail-safe is
busy" state.  We should probably not allow that.
@github-actions github-actions bot added the app label Jul 22, 2022
@github-actions
Copy link

github-actions bot commented Jul 22, 2022 •
edited
Loading

PR #21111: Size comparison from a671405 to 26685dd

Increases (40 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, nrfconnect, p6, telink)
platform target config section a671405 26685dd change % change
bl602 lighting-app bl602 (read/write) 1380802 1380866 64 0.0
.text 1050932 1050996 64 0.0
bl602+rpc (read/write) 1426210 1426282 72 0.0
.text 1082584 1082656 72 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 668431 668455 24 0.0
.text 579756 579780 24 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 634031 634055 24 0.0
.text 556124 556148 24 0.0
lock-ftd LP_CC2652R7 (read only) 671627 671659 32 0.0
.text 594672 594704 32 0.0
lock-mtd LP_CC2652R7 (read only) 653895 653903 8 0.0
.text 552224 552232 8 0.0
pump-app LP_CC2652R7 (read only) 681319 681327 8 0.0
.text 591628 591636 8 0.0
pump-controller-app LP_CC2652R7 (read only) 667095 667103 8 0.0
.text 581544 581552 8 0.0
shell LP_CC2652R7 (read only) 660898 660922 24 0.0
.text 575452 575476 24 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 582082 582106 24 0.0
.app_xip_area 460472 460496 24 0.0
lock cyw930739m2evb_01 (read/write) 587998 588014 16 0.0
.app_xip_area 461660 461676 16 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 586534 586558 24 0.0
.app_xip_area 465772 465796 24 0.0
efr32 lighting-app BRD4161A (read/write) 1087720 1087752 32 0.0
.text 952432 952464 32 0.0
BRD4161A+rpc (read/write) 1141996 1142044 48 0.0
.text 989820 989868 48 0.0
BRD4161A+rs911x (read/write) 972940 972988 48 0.0
.text 809208 809256 48 0.0
lock-app BRD4161A+wf200 (read/write) 1128464 1128480 16 0.0
.text 982084 982100 16 0.0
window-app BRD4161A (read/write) 1081196 1081228 32 0.0
.text 944408 944440 32 0.0
esp32 all-clusters-app c3devkit (read only) 1021974 1022000 26 0.0
.flash.text 1021974 1022000 26 0.0
m5stack (read only) 1075751 1075771 20 0.0
.flash.text 1070367 1070387 20 0.0
k32w light k32w061+release (read/write) 660816 660832 16 0.0
.text 583356 583372 16 0.0
lock k32w061+release (read/write) 687868 687884 16 0.0
.text 609924 609940 16 0.0
linux all-clusters-app debug (read only) 2981817 2981945 128 0.0
.text 2537778 2537906 128 0.0
all-clusters-minimal-app debug (read only) 2827713 2827841 128 0.0
.text 2385490 2385618 128 0.0
bridge-app debug+rpc (read only) 2342417 2342545 128 0.0
.text 1980706 1980834 128 0.0
lighting-app debug+rpc (read only) 2565569 2565697 128 0.0
.text 2180082 2180210 128 0.0
lock-app debug (read only) 2530561 2530689 128 0.0
.text 2134866 2134994 128 0.0
ota-provider-app debug (read only) 2334177 2334305 128 0.0
.text 1966050 1966178 128 0.0
ota-requestor-app debug (read only) 2452633 2452761 128 0.0
.text 2072386 2072514 128 0.0
shell debug (read only) 2568425 2568553 128 0.0
.text 2181666 2181794 128 0.0
thermostat-no-ble arm64 (read only) 2342012 2342204 192 0.0
.text 1965568 1965760 192 0.0
tv-app debug (read only) 3116193 3116321 128 0.0
.text 2677170 2677298 128 0.0
tv-casting-app debug (read only) 5369905 5370033 128 0.0
.text 4768594 4768722 128 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1177131 1177163 32 0.0
text 812608 812628 20 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1157203 1157219 16 0.0
text 801924 801940 16 0.0
p6 all-clusters-app default (read/write) 1686660 1686708 48 0.0
.text 1526560 1526608 48 0.0
all-clusters-minimal-app default (read/write) 1630772 1630804 32 0.0
.text 1471392 1471424 32 0.0
light-app default (read/write) 1551020 1551052 32 0.0
.text 1399944 1399976 32 0.0
lock-app default (read/write) 1588612 1588644 32 0.0
.text 1433064 1433096 32 0.0
telink light-switch-app tlsr9518adk80d (read/write) 799664 799696 32 0.0
text 567228 567262 34 0.0
lighting-app tlsr9518adk80d (read/write) 819712 819744 32 0.0
text 583772 583802 30 0.0
Decreases (6 builds for cc13x2_26x2)
platform target config section a671405 26685dd change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read/write) 182912 182888 -24 -0.0
lock-ftd LP_CC2652R7 (read/write) 169892 169860 -32 -0.0
lock-mtd LP_CC2652R7 (read/write) 183312 183304 -8 -0.0
pump-app LP_CC2652R7 (read/write) 161056 161048 -8 -0.0
pump-controller-app LP_CC2652R7 (read/write) 175400 175392 -8 -0.0
shell LP_CC2652R7 (read/write) 185948 185924 -24 -0.0
Full report (43 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section a671405 26685dd change % change
bl602 lighting-app bl602 (read/write) 1380802 1380866 64 0.0
.bss 117474 117474 0 0.0
.data 4480 4480 0 0.0
.text 1050932 1050996 64 0.0
bl602+rpc (read/write) 1426210 1426282 72 0.0
.bss 124922 124922 0 0.0
.data 4600 4600 0 0.0
.text 1082584 1082656 72 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 668431 668455 24 0.0
(read/write) 182912 182888 -24 -0.0
.bss 74236 74236 0 0.0
.data 3356 3356 0 0.0
.rodata 88359 88359 0 0.0
.text 579756 579780 24 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 634031 634055 24 0.0
(read/write) 157804 157804 0 0.0
.bss 73532 73532 0 0.0
.data 3356 3356 0 0.0
.rodata 77583 77583 0 0.0
.text 556124 556148 24 0.0
lock-ftd LP_CC2652R7 (read only) 671627 671659 32 0.0
(read/write) 169892 169860 -32 -0.0
.bss 71300 71300 0 0.0
.data 3280 3280 0 0.0
.rodata 76475 76475 0 0.0
.text 594672 594704 32 0.0
lock-mtd LP_CC2652R7 (read only) 653895 653903 8 0.0
(read/write) 183312 183304 -8 -0.0
.bss 66988 66988 0 0.0
.data 3280 3280 0 0.0
.rodata 101191 101191 0 0.0
.text 552224 552232 8 0.0
pump-app LP_CC2652R7 (read only) 681319 681327 8 0.0
(read/write) 161056 161048 -8 -0.0
.bss 71388 71388 0 0.0
.data 3280 3280 0 0.0
.rodata 89207 89207 0 0.0
.text 591628 591636 8 0.0
pump-controller-app LP_CC2652R7 (read only) 667095 667103 8 0.0
(read/write) 175400 175392 -8 -0.0
.bss 71508 71508 0 0.0
.data 3276 3276 0 0.0
.rodata 85071 85071 0 0.0
.text 581544 581552 8 0.0
shell LP_CC2652R7 (read only) 660898 660922 24 0.0
(read/write) 185948 185924 -24 -0.0
.bss 76540 76540 0 0.0
.data 3360 3360 0 0.0
.rodata 85130 85130 0 0.0
.text 575452 575476 24 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 582082 582106 24 0.0
.app_xip_area 460472 460496 24 0.0
.bss 64404 64404 0 0.0
.data 716 716 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 587998 588014 16 0.0
.app_xip_area 461660 461676 16 0.0
.bss 69132 69132 0 0.0
.data 720 720 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 586534 586558 24 0.0
.app_xip_area 465772 465796 24 0.0
.bss 63612 63612 0 0.0
.data 660 660 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1087720 1087752 32 0.0
.bss 133220 133220 0 0.0
.data 2048 2048 0 0.0
.text 952432 952464 32 0.0
BRD4161A+rpc (read/write) 1141996 1142044 48 0.0
.bss 149892 149892 0 0.0
.data 2260 2260 0 0.0
.text 989820 989868 48 0.0
BRD4161A+rs911x (read/write) 972940 972988 48 0.0
.bss 161664 161664 0 0.0
.data 2048 2048 0 0.0
.text 809208 809256 48 0.0
lock-app BRD4161A+wf200 (read/write) 1128464 1128480 16 0.0
.bss 144304 144304 0 0.0
.data 2056 2056 0 0.0
.text 982084 982100 16 0.0
window-app BRD4161A (read/write) 1081196 1081228 32 0.0
.bss 134692 134692 0 0.0
.data 2076 2076 0 0.0
.text 944408 944440 32 0.0
esp32 all-clusters-app c3devkit (read only) 1021974 1022000 26 0.0
(read/write) 1486466 1486466 0 0.0
.dram0.bss 70232 70232 0 0.0
.dram0.data 14600 14600 0 0.0
.flash.rodata 216192 216192 0 0.0
.flash.text 1021974 1022000 26 0.0
.iram0.text 62902 62902 0 0.0
m5stack (read only) 1075751 1075771 20 0.0
(read/write) 488472 488472 0 0.0
.dram0.bss 75752 75752 0 0.0
.dram0.data 34144 34144 0 0.0
.flash.rodata 246580 246580 0 0.0
.flash.text 1070367 1070387 20 0.0
.iram0.text 123267 123267 0 0.0
k32w light k32w061+release (read/write) 660816 660832 16 0.0
.bss 69668 69668 0 0.0
.data 1992 1992 0 0.0
.text 583356 583372 16 0.0
lock k32w061+release (read/write) 687868 687884 16 0.0
.bss 70140 70140 0 0.0
.data 2004 2004 0 0.0
.text 609924 609940 16 0.0
linux all-clusters-app debug (read only) 2981817 2981945 128 0.0
(read/write) 155344 155344 0 0.0
.bss 61792 61792 0 0.0
.data 2064 2064 0 0.0
.data.rel.ro 85224 85224 0 0.0
.dynamic 608 608 0 0.0
.got 4568 4568 0 0.0
.init 27 27 0 0.0
.init_array 1064 1064 0 0.0
.rodata 265899 265899 0 0.0
.text 2537778 2537906 128 0.0
all-clusters-minimal-app debug (read only) 2827713 2827841 128 0.0
(read/write) 147016 147016 0 0.0
.bss 60992 60992 0 0.0
.data 2064 2064 0 0.0
.data.rel.ro 77784 77784 0 0.0
.dynamic 608 608 0 0.0
.got 4488 4488 0 0.0
.init 27 27 0 0.0
.init_array 1056 1056 0 0.0
.rodata 266763 266763 0 0.0
.text 2385490 2385618 128 0.0
bridge-app debug+rpc (read only) 2342417 2342545 128 0.0
(read/write) 126920 126920 0 0.0
.bss 50080 50080 0 0.0
.data 3824 3824 0 0.0
.data.rel.ro 67240 67240 0 0.0
.dynamic 608 608 0 0.0
.got 4392 4392 0 0.0
.init 27 27 0 0.0
.init_array 736 736 0 0.0
.rodata 199400 199400 0 0.0
.text 1980706 1980834 128 0.0
chip-tool debug (read only) 10349921 10349921 0 0.0
(read/write) 631912 631912 0 0.0
.bss 24760 24760 0 0.0
.data 3266 3266 0 0.0
.data.rel.ro 597480 597480 0 0.0
.dynamic 608 608 0 0.0
.got 5088 5088 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 522005 522005 0 0.0
.text 8374324 8374324 0 0.0
chip-tool-ipv6only arm64 (read only) 9779356 9779356 0 0.0
(read/write) 679441 679441 0 0.0
.bss 32833 32833 0 0.0
.data 3272 3272 0 0.0
.data.rel.ro 624824 624824 0 0.0
.dynamic 560 560 0 0.0
.got 13552 13552 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 457724 457724 0 0.0
.text 7737236 7737236 0 0.0
lighting-app debug+rpc (read only) 2565569 2565697 128 0.0
(read/write) 129888 129888 0 0.0
.bss 49632 49632 0 0.0
.data 2096 2096 0 0.0
.data.rel.ro 72296 72296 0 0.0
.dynamic 608 608 0 0.0
.got 4392 4392 0 0.0
.init 27 27 0 0.0
.init_array 824 824 0 0.0
.rodata 215088 215088 0 0.0
.text 2180082 2180210 128 0.0
lock-app debug (read only) 2530561 2530689 128 0.0
(read/write) 124872 124872 0 0.0
.bss 48032 48032 0 0.0
.data 1712 1712 0 0.0
.data.rel.ro 69272 69272 0 0.0
.dynamic 608 608 0 0.0
.got 4424 4424 0 0.0
.init 27 27 0 0.0
.init_array 800 800 0 0.0
.rodata 230096 230096 0 0.0
.text 2134866 2134994 128 0.0
ota-provider-app debug (read only) 2334177 2334305 128 0.0
(read/write) 118672 118672 0 0.0
.bss 47680 47680 0 0.0
.data 1936 1936 0 0.0
.data.rel.ro 63256 63256 0 0.0
.dynamic 608 608 0 0.0
.got 4488 4488 0 0.0
.init 27 27 0 0.0
.init_array 680 680 0 0.0
.rodata 204920 204920 0 0.0
.text 1966050 1966178 128 0.0
ota-requestor-app debug (read only) 2452633 2452761 128 0.0
(read/write) 125576 125576 0 0.0
.bss 50016 50016 0 0.0
.data 2240 2240 0 0.0
.data.rel.ro 67480 67480 0 0.0
.dynamic 608 608 0 0.0
.got 4480 4480 0 0.0
.init 27 27 0 0.0
.init_array 736 736 0 0.0
.rodata 208640 208640 0 0.0
.text 2072386 2072514 128 0.0
shell debug (read only) 2568425 2568553 128 0.0
(read/write) 141400 141400 0 0.0
.bss 57608 57608 0 0.0
.data 1248 1248 0 0.0
.data.rel.ro 76856 76856 0 0.0
.dynamic 608 608 0 0.0
.got 4136 4136 0 0.0
.init 27 27 0 0.0
.init_array 936 936 0 0.0
.rodata 229202 229202 0 0.0
.text 2181666 2181794 128 0.0
thermostat-no-ble arm64 (read only) 2342012 2342204 192 0.0
(read/write) 141265 141265 0 0.0
.bss 55233 55233 0 0.0
.data 1672 1672 0 0.0
.data.rel.ro 75616 75616 0 0.0
.dynamic 560 560 0 0.0
.got 4984 4984 0 0.0
.init 24 24 0 0.0
.init_array 400 400 0 0.0
.rodata 139604 139604 0 0.0
.text 1965568 1965760 192 0.0
tv-app debug (read only) 3116193 3116321 128 0.0
(read/write) 257024 257024 0 0.0
.bss 167064 167064 0 0.0
.data 4736 4736 0 0.0
.data.rel.ro 78792 78792 0 0.0
.dynamic 608 608 0 0.0
.got 4848 4848 0 0.0
.init 27 27 0 0.0
.init_array 960 960 0 0.0
.rodata 250600 250600 0 0.0
.text 2677170 2677298 128 0.0
tv-casting-app debug (read only) 5369905 5370033 128 0.0
(read/write) 158328 158328 0 0.0
.bss 51256 51256 0 0.0
.data 2432 2432 0 0.0
.data.rel.ro 98352 98352 0 0.0
.dynamic 608 608 0 0.0
.got 4736 4736 0 0.0
.init 27 27 0 0.0
.init_array 920 920 0 0.0
.rodata 334785 334785 0 0.0
.text 4768594 4768722 128 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2449264 2449264 0 0.0
.bss 214444 214444 0 0.0
.data 5872 5872 0 0.0
.text 1411908 1411908 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1177131 1177163 32 0.0
bss 143068 143068 0 0.0
rodata 142596 142596 0 0.0
text 812608 812628 20 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1157203 1157219 16 0.0
bss 142304 142304 0 0.0
rodata 134132 134132 0 0.0
text 801924 801940 16 0.0
p6 all-clusters-app default (read only) 881632 881632 0 0.0
(read/write) 1686660 1686708 48 0.0
.bss 149064 149064 0 0.0
.data 2648 2648 0 0.0
.text 1526560 1526608 48 0.0
all-clusters-minimal-app default (read only) 882352 882352 0 0.0
(read/write) 1630772 1630804 32 0.0
.bss 148344 148344 0 0.0
.data 2648 2648 0 0.0
.text 1471392 1471424 32 0.0
light-app default (read only) 890656 890656 0 0.0
(read/write) 1551020 1551052 32 0.0
.bss 140248 140248 0 0.0
.data 2440 2440 0 0.0
.text 1399944 1399976 32 0.0
lock-app default (read only) 886184 886184 0 0.0
(read/write) 1588612 1588644 32 0.0
.bss 144704 144704 0 0.0
.data 2456 2456 0 0.0
.text 1433064 1433096 32 0.0
telink light-switch-app tlsr9518adk80d (read/write) 799664 799696 32 0.0
bss 70744 70744 0 0.0
noinit 40416 40416 0 0.0
text 567228 567262 34 0.0
lighting-app tlsr9518adk80d (read/write) 819712 819744 32 0.0
bss 71588 71588 0 0.0
noinit 40416 40416 0 0.0
text 583772 583802 30 0.0

@andy31415 andy31415 enabled auto-merge (squash) July 22, 2022 18:34
@andy31415 andy31415 merged commit 8274265 into project-chip:master Jul 22, 2022
@bzbarsky-apple bzbarsky-apple deleted the failsafe-busy-handling branch July 23, 2022 00:57
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this pull request Jul 31, 2022
@bzbarsky-apple
Disallow opening commissioning windows when fail-safe is not fully di…
ac0c573 
…sarmed. (project-chip#21111)

We were allowing opening commissioning windows in the "fail-safe is
busy" state.  We should probably not allow that.
woody-apple pushed a commit that referenced this pull request Aug 1, 2022
@bzbarsky-apple
Disallow opening commissioning windows when fail-safe is not fully di…
a14c225 
…sarmed. (#21111) (#21474)

We were allowing opening commissioning windows in the "fail-safe is
busy" state.  We should probably not allow that.
isiu-apple pushed a commit to isiu-apple/connectedhomeip that referenced this pull request Sep 16, 2022
@bzbarsky-apple @isiu-apple
Disallow opening commissioning windows when fail-safe is not fully di…
00117ea 
…sarmed. (project-chip#21111)

We were allowing opening commissioning windows in the "fail-safe is
busy" state.  We should probably not allow that.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@woody-apple woody-apple woody-apple approved these changes

@andy31415 andy31415 andy31415 approved these changes

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

@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

@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

@erjiaqing erjiaqing Awaiting requested review from erjiaqing

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

@gjc13 gjc13 Awaiting requested review from gjc13

@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

@jepenven-silabs jepenven-silabs Awaiting requested review from jepenven-silabs

@jmartinez-silabs jmartinez-silabs Awaiting requested review from jmartinez-silabs

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

@kpschoedel kpschoedel Awaiting requested review from kpschoedel

@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

@rgoliver rgoliver Awaiting requested review from rgoliver

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tecimovic tecimovic Awaiting requested review from tecimovic

@vijs vijs Awaiting requested review from vijs

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

@wbschiller wbschiller Awaiting requested review from wbschiller

@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
app review - approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bzbarsky-apple @andy31415 @woody-apple