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

Address API review issues in MTRDeviceController. #22596

Merged

Conversation

bzbarsky-apple
Copy link
Contributor

  • Make MTRBaseDevice creation synchronous. This requires updates to MTRBaseDeviceOverXPC to do the possible async getting of the controller id it needs during its async operations, not when getting the device object.
  • Rename "pairDevice" to "setupCommissioningSessionWithPayload", fix its signature, improve documentation.
  • Rename "commissionDevice" to "commissionNodeWithID".
  • Rename "stopDevicePairing" to "cancelCommissioningForNodeID" and document.
  • Rename "getDeviceBeingCommissioned" to "getDeviceBeingCommissionedWithNodeID".
  • Various documentation improvements.
  • Add a way to generate a QR code from an MTRSetupPayload to allow correct recovery of long discriminators in setupCommissioningSessionWithPayload.

Addresses part of #22420

Issue Being Resolved

Change overview

See above.

@bzbarsky-apple bzbarsky-apple force-pushed the darwin-controller-api-fixes branch from 489bcf9 to f62606e Compare September 14, 2022 03:58
@github-actions
Copy link

github-actions bot commented Sep 14, 2022

PR #22596: Size comparison from 8626d39 to f62606e

Increases (5 builds for bl602, bl702, cc13x2_26x2, nrfconnect)
platform target config section 8626d39 f62606e change % change
bl602 lighting-app bl602+rpc (read/write) 1458330 1458338 8 0.0
.text 1099184 1099186 2 0.0
bl702 lighting-app bl702 .debug_info 37894018 37894019 1 0.0
.text 957170 957172 2 0.0
bl702+rpc .debug_info 41800588 41800589 1 0.0
.text 1030872 1030874 2 0.0
cc13x2_26x2 lock-ftd LP_CC2652R7 (read only) 678055 678063 8 0.0
.text 600280 600288 8 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 text 815932 815936 4 0.0
Decreases (4 builds for cc13x2_26x2, psoc6, qpg)
platform target config section 8626d39 f62606e change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read/write) 163680 163672 -8 -0.0
psoc6 all-clusters cy8ckit_062s2_43012 .debug_info 26813751 26813750 -1 -0.0
lock cy8ckit_062s2_43012 .debug_info 22396229 22396228 -1 -0.0
qpg lighting-app qpg6105+debug (read/write) 1130876 1130868 -8 -0.0
.text 577972 577964 -8 -0.0
Full report (35 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section 8626d39 f62606e change % change
bl602 lighting-app bl602 (read/write) 1412862 1412862 0 0.0
.bss 121585 121585 0 0.0
.data 9824 9824 0 0.0
.text 1067580 1067580 0 0.0
bl602+rpc (read/write) 1458330 1458338 8 0.0
.bss 129017 129017 0 0.0
.data 10208 10208 0 0.0
.text 1099184 1099186 2 0.0
bl702 lighting-app bl702 (read only) 3262 3262 0 0.0
(read/write) 1188427 1188427 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 66958 66958 0 0.0
.bss_psram 29696 29696 0 0.0
.comment 48 48 0 0.0
.data 4280 4280 0 0.0
.debug_abbrev 1506731 1506731 0 0.0
.debug_aranges 133072 133072 0 0.0
.debug_frame 486348 486348 0 0.0
.debug_info 37894018 37894019 1 0.0
.debug_line 5253486 5253486 0 0.0
.debug_loc 3363590 3363590 0 0.0
.debug_ranges 360256 360256 0 0.0
.debug_str 3455308 3455308 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 116504 116504 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 564895 564895 0 0.0
.symtab 171616 171616 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
957170 957172 2 0.0
bl702+rpc (read only) 3262 3262 0 0.0
(read/write) 1284339 1284339 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 75006 75006 0 0.0
.bss_psram 29936 29936 0 0.0
.comment 48 48 0 0.0
.data 4800 4800 0 0.0
.debug_abbrev 1644310 1644310 0 0.0
.debug_aranges 140576 140576 0 0.0
.debug_frame 511724 511724 0 0.0
.debug_info 41800588 41800589 1 0.0
.debug_line 5628025 5628025 0 0.0
.debug_loc 3556258 3556258 0 0.0
.debug_ranges 382712 382712 0 0.0
.debug_str 3851275 3851275 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 129896 129896 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 624068 624068 0 0.0
.symtab 189424 189424 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
1030872 1030874 2 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 676939 676939 0 0.0
(read/write) 174484 174484 0 0.0
.bss 74316 74316 0 0.0
.data 3380 3380 0 0.0
.rodata 89451 89451 0 0.0
.text 587176 587176 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 641651 641651 0 0.0
(read/write) 157884 157884 0 0.0
.bss 73588 73588 0 0.0
.data 3380 3380 0 0.0
.rodata 78595 78595 0 0.0
.text 562736 562736 0 0.0
lock-ftd LP_CC2652R7 (read only) 678055 678063 8 0.0
(read/write) 163680 163672 -8 -0.0
.bss 71516 71516 0 0.0
.data 3304 3304 0 0.0
.rodata 77295 77295 0 0.0
.text 600280 600288 8 0.0
lock-mtd LP_CC2652R7 (read only) 661883 661883 0 0.0
(read/write) 175540 175540 0 0.0
.bss 67204 67204 0 0.0
.data 3304 3304 0 0.0
.rodata 103131 103131 0 0.0
.text 558272 558272 0 0.0
pump-app LP_CC2652R7 (read only) 687291 687291 0 0.0
(read/write) 155148 155148 0 0.0
.bss 71452 71452 0 0.0
.data 3296 3296 0 0.0
.rodata 90563 90563 0 0.0
.text 596244 596244 0 0.0
pump-controller-app LP_CC2652R7 (read only) 671783 671783 0 0.0
(read/write) 170768 170768 0 0.0
.bss 71564 71564 0 0.0
.data 3292 3292 0 0.0
.rodata 86119 86119 0 0.0
.text 585184 585184 0 0.0
shell LP_CC2652R7 (read only) 667982 667982 0 0.0
(read/write) 178952 178952 0 0.0
.bss 76628 76628 0 0.0
.data 3376 3376 0 0.0
.rodata 86174 86174 0 0.0
.text 581492 581492 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 587410 587410 0 0.0
.app_xip_area 464068 464068 0 0.0
.bss 65776 65776 0 0.0
.data 744 744 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 594498 594498 0 0.0
.app_xip_area 465828 465828 0 0.0
.bss 71096 71096 0 0.0
.data 752 752 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 543482 543482 0 0.0
.app_xip_area 425164 425164 0 0.0
.bss 60784 60784 0 0.0
.data 716 716 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1110704 1110704 0 0.0
.bss 136332 136332 0 0.0
.data 2072 2072 0 0.0
.text 972280 972280 0 0.0
BRD4161A+rpc (read/write) 973588 973588 0 0.0
.bss 150844 150844 0 0.0
.data 2252 2252 0 0.0
.text 820472 820472 0 0.0
BRD4161A+rs911x (read/write) 1003968 1003968 0 0.0
.bss 169168 169168 0 0.0
.data 2064 2064 0 0.0
.text 832716 832716 0 0.0
lock-app BRD4161A+wf200 (read/write) 1151476 1151476 0 0.0
.bss 152248 152248 0 0.0
.data 2072 2072 0 0.0
.text 997136 997136 0 0.0
window-app BRD4161A (read/write) 1101928 1101928 0 0.0
.bss 137772 137772 0 0.0
.data 2096 2096 0 0.0
.text 962040 962040 0 0.0
k32w light k32w0+release (read/write) 649972 649972 0 0.0
.bss 70712 70712 0 0.0
.data 2068 2068 0 0.0
.text 574464 574464 0 0.0
lock k32w0+release (read/write) 706960 706960 0 0.0
.bss 71160 71160 0 0.0
.data 2076 2076 0 0.0
.text 630996 630996 0 0.0
linux chip-tool-ipv6only arm64 (read only) 10345860 10345860 0 0.0
(read/write) 706241 706241 0 0.0
.bss 33937 33937 0 0.0
.data 2864 2864 0 0.0
.data.rel.ro 650560 650560 0 0.0
.dynamic 560 560 0 0.0
.got 13904 13904 0 0.0
.init 24 24 0 0.0
.init_array 200 200 0 0.0
.rodata 503028 503028 0 0.0
.text 8188884 8188884 0 0.0
thermostat-no-ble arm64 (read only) 2368252 2368252 0 0.0
(read/write) 142497 142497 0 0.0
.bss 55233 55233 0 0.0
.data 1688 1688 0 0.0
.data.rel.ro 76704 76704 0 0.0
.dynamic 560 560 0 0.0
.got 5096 5096 0 0.0
.init 24 24 0 0.0
.init_array 416 416 0 0.0
.rodata 141588 141588 0 0.0
.text 1987664 1987664 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2455952 2455952 0 0.0
.bss 215044 215044 0 0.0
.data 5872 5872 0 0.0
.text 1418596 1418596 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1182723 1182723 0 0.0
bss 143641 143641 0 0.0
rodata 144212 144212 0 0.0
text 815932 815936 4 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1161923 1161923 0 0.0
bss 142868 142868 0 0.0
rodata 135804 135804 0 0.0
text 804328 804328 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 0 0 0 0.0
(read only) 841960 841960 0 0.0
(read/write) 1744780 1744780 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 188720 188720 0 0.0
.comment 204 204 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 1229158 1229158 0 0.0
.debug_aranges 111800 111800 0 0.0
.debug_frame 373224 373224 0 0.0
.debug_info 26813751 26813750 -1 -0.0
.debug_line 3668978 3668978 0 0.0
.debug_loc 3582717 3582717 0 0.0
.debug_ranges 340576 340576 0 0.0
.debug_str 3439101 3439101 0 0.0
.heap 841960 841960 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 571696 571696 0 0.0
.symtab 422080 422080 0 0.0
.text 1545008 1545008 0 0.0
.zero.table 8 8 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 0 0 0 0.0
(read only) 842696 842696 0 0.0
(read/write) 1687972 1687972 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 187984 187984 0 0.0
.comment 204 204 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 1220957 1220957 0 0.0
.debug_aranges 111272 111272 0 0.0
.debug_frame 376304 376304 0 0.0
.debug_info 26550599 26550599 0 0.0
.debug_line 3689694 3689694 0 0.0
.debug_loc 3570354 3570354 0 0.0
.debug_ranges 339192 339192 0 0.0
.debug_str 3428114 3428114 0 0.0
.heap 842696 842696 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 536170 536170 0 0.0
.symtab 408672 408672 0 0.0
.text 1488936 1488936 0 0.0
.zero.table 8 8 0 0.0
light cy8ckit_062s2_43012 0 0 0 0.0
(read only) 850928 850928 0 0.0
(read/write) 1605236 1605236 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 179960 179960 0 0.0
.comment 204 204 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 1055116 1055116 0 0.0
.debug_aranges 103448 103448 0 0.0
.debug_frame 346572 346572 0 0.0
.debug_info 22016853 22016853 0 0.0
.debug_line 3259861 3259861 0 0.0
.debug_loc 3268724 3268724 0 0.0
.debug_ranges 304664 304664 0 0.0
.debug_str 3233368 3233368 0 0.0
.heap 850928 850928 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 469469 469469 0 0.0
.symtab 375760 375760 0 0.0
.text 1414432 1414432 0 0.0
.zero.table 8 8 0 0.0
lock cy8ckit_062s2_43012 0 0 0 0.0
(read only) 845896 845896 0 0.0
(read/write) 1642924 1642924 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 184976 184976 0 0.0
.comment 204 204 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 1062571 1062571 0 0.0
.debug_aranges 104120 104120 0 0.0
.debug_frame 349400 349400 0 0.0
.debug_info 22396229 22396228 -1 -0.0
.debug_line 3268581 3268581 0 0.0
.debug_loc 3308561 3308561 0 0.0
.debug_ranges 308008 308008 0 0.0
.debug_str 3260785 3260785 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 475672 475672 0 0.0
.symtab 378944 378944 0 0.0
.text 1447088 1447088 0 0.0
.zero.table 8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1130876 1130868 -8 -0.0
.bss 106112 106112 0 0.0
.data 1028 1028 0 0.0
.text 577972 577964 -8 -0.0
lock-app qpg6105+debug (read/write) 1101888 1101888 0 0.0
.bss 102344 102344 0 0.0
.data 1032 1032 0 0.0
.text 548988 548988 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 813868 813868 0 0.0
bss 71372 71372 0 0.0
noinit 43488 43488 0 0.0
text 574710 574710 0 0.0
lighting-app tlsr9518adk80d (read/write) 835800 835800 0 0.0
bss 72228 72228 0 0.0
noinit 43488 43488 0 0.0
text 592850 592850 0 0.0
ota-requestor-app tlsr9518adk80d (read/write) 843916 843916 0 0.0
bss 73136 73136 0 0.0
noinit 43488 43488 0 0.0
text 599112 599112 0 0.0

@bzbarsky-apple bzbarsky-apple force-pushed the darwin-controller-api-fixes branch from f62606e to 49707b9 Compare September 14, 2022 05:12
@github-actions
Copy link

github-actions bot commented Sep 14, 2022

PR #22596: Size comparison from 8626d39 to 49707b9

Increases (9 builds for bl602, bl702, cc13x2_26x2, esp32, nrfconnect, psoc6, telink)
platform target config section 8626d39 49707b9 change % change
bl602 lighting-app bl602 .text 1067580 1067582 2 0.0
bl602+rpc (read/write) 1458330 1458338 8 0.0
.text 1099184 1099186 2 0.0
bl702 lighting-app bl702+rpc .debug_info 41800588 41800589 1 0.0
.text 1030872 1030874 2 0.0
cc13x2_26x2 lock-ftd LP_CC2652R7 (read only) 678055 678063 8 0.0
.text 600280 600288 8 0.0
esp32 all-clusters-app c3devkit (read/write) 1788102 1788110 8 0.0
.flash.rodata 257656 257664 8 0.0
m5stack (read/write) 563988 563996 8 0.0
.flash.rodata 314712 314720 8 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 text 815932 815936 4 0.0
psoc6 all-clusters cy8ckit_062s2_43012 .debug_info 26813751 26813752 1 0.0
telink ota-requestor-app tlsr9518adk80d (read/write) 843916 843924 8 0.0
text 599112 599114 2 0.0
Decreases (1 build for cc13x2_26x2)
platform target config section 8626d39 49707b9 change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read/write) 163680 163672 -8 -0.0
Full report (37 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section 8626d39 49707b9 change % change
bl602 lighting-app bl602 (read/write) 1412862 1412862 0 0.0
.bss 121585 121585 0 0.0
.data 9824 9824 0 0.0
.text 1067580 1067582 2 0.0
bl602+rpc (read/write) 1458330 1458338 8 0.0
.bss 129017 129017 0 0.0
.data 10208 10208 0 0.0
.text 1099184 1099186 2 0.0
bl702 lighting-app bl702 (read only) 3262 3262 0 0.0
(read/write) 1188427 1188427 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 66958 66958 0 0.0
.bss_psram 29696 29696 0 0.0
.comment 48 48 0 0.0
.data 4280 4280 0 0.0
.debug_abbrev 1506731 1506731 0 0.0
.debug_aranges 133072 133072 0 0.0
.debug_frame 486348 486348 0 0.0
.debug_info 37894018 37894018 0 0.0
.debug_line 5253486 5253486 0 0.0
.debug_loc 3363590 3363590 0 0.0
.debug_ranges 360256 360256 0 0.0
.debug_str 3455308 3455308 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 116504 116504 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 564895 564895 0 0.0
.symtab 171616 171616 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
957170 957170 0 0.0
bl702+rpc (read only) 3262 3262 0 0.0
(read/write) 1284339 1284339 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 75006 75006 0 0.0
.bss_psram 29936 29936 0 0.0
.comment 48 48 0 0.0
.data 4800 4800 0 0.0
.debug_abbrev 1644310 1644310 0 0.0
.debug_aranges 140576 140576 0 0.0
.debug_frame 511724 511724 0 0.0
.debug_info 41800588 41800589 1 0.0
.debug_line 5628025 5628025 0 0.0
.debug_loc 3556258 3556258 0 0.0
.debug_ranges 382712 382712 0 0.0
.debug_str 3851275 3851275 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 129896 129896 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 624068 624068 0 0.0
.symtab 189424 189424 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
1030872 1030874 2 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 676939 676939 0 0.0
(read/write) 174484 174484 0 0.0
.bss 74316 74316 0 0.0
.data 3380 3380 0 0.0
.rodata 89451 89451 0 0.0
.text 587176 587176 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 641651 641651 0 0.0
(read/write) 157884 157884 0 0.0
.bss 73588 73588 0 0.0
.data 3380 3380 0 0.0
.rodata 78595 78595 0 0.0
.text 562736 562736 0 0.0
lock-ftd LP_CC2652R7 (read only) 678055 678063 8 0.0
(read/write) 163680 163672 -8 -0.0
.bss 71516 71516 0 0.0
.data 3304 3304 0 0.0
.rodata 77295 77295 0 0.0
.text 600280 600288 8 0.0
lock-mtd LP_CC2652R7 (read only) 661883 661883 0 0.0
(read/write) 175540 175540 0 0.0
.bss 67204 67204 0 0.0
.data 3304 3304 0 0.0
.rodata 103131 103131 0 0.0
.text 558272 558272 0 0.0
pump-app LP_CC2652R7 (read only) 687291 687291 0 0.0
(read/write) 155148 155148 0 0.0
.bss 71452 71452 0 0.0
.data 3296 3296 0 0.0
.rodata 90563 90563 0 0.0
.text 596244 596244 0 0.0
pump-controller-app LP_CC2652R7 (read only) 671783 671783 0 0.0
(read/write) 170768 170768 0 0.0
.bss 71564 71564 0 0.0
.data 3292 3292 0 0.0
.rodata 86119 86119 0 0.0
.text 585184 585184 0 0.0
shell LP_CC2652R7 (read only) 667982 667982 0 0.0
(read/write) 178952 178952 0 0.0
.bss 76628 76628 0 0.0
.data 3376 3376 0 0.0
.rodata 86174 86174 0 0.0
.text 581492 581492 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 587410 587410 0 0.0
.app_xip_area 464068 464068 0 0.0
.bss 65776 65776 0 0.0
.data 744 744 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 594498 594498 0 0.0
.app_xip_area 465828 465828 0 0.0
.bss 71096 71096 0 0.0
.data 752 752 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 543482 543482 0 0.0
.app_xip_area 425164 425164 0 0.0
.bss 60784 60784 0 0.0
.data 716 716 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1110704 1110704 0 0.0
.bss 136332 136332 0 0.0
.data 2072 2072 0 0.0
.text 972280 972280 0 0.0
BRD4161A+rpc (read/write) 973588 973588 0 0.0
.bss 150844 150844 0 0.0
.data 2252 2252 0 0.0
.text 820472 820472 0 0.0
BRD4161A+rs911x (read/write) 1003968 1003968 0 0.0
.bss 169168 169168 0 0.0
.data 2064 2064 0 0.0
.text 832716 832716 0 0.0
lock-app BRD4161A+wf200 (read/write) 1151476 1151476 0 0.0
.bss 152248 152248 0 0.0
.data 2072 2072 0 0.0
.text 997136 997136 0 0.0
window-app BRD4161A (read/write) 1101928 1101928 0 0.0
.bss 137772 137772 0 0.0
.data 2096 2096 0 0.0
.text 962040 962040 0 0.0
esp32 all-clusters-app c3devkit (read only) 1223856 1223856 0 0.0
(read/write) 1788102 1788110 8 0.0
.dram0.bss 76952 76952 0 0.0
.dram0.data 13840 13840 0 0.0
.flash.rodata 257656 257664 8 0.0
.flash.text 1223856 1223856 0 0.0
.iram0.text 65204 65204 0 0.0
m5stack (read only) 1233723 1233723 0 0.0
(read/write) 563988 563996 8 0.0
.dram0.bss 82312 82312 0 0.0
.dram0.data 34296 34296 0 0.0
.flash.rodata 314712 314720 8 0.0
.flash.text 1228339 1228339 0 0.0
.iram0.text 123939 123939 0 0.0
k32w light k32w0+release (read/write) 649972 649972 0 0.0
.bss 70712 70712 0 0.0
.data 2068 2068 0 0.0
.text 574464 574464 0 0.0
lock k32w0+release (read/write) 706960 706960 0 0.0
.bss 71160 71160 0 0.0
.data 2076 2076 0 0.0
.text 630996 630996 0 0.0
linux chip-tool-ipv6only arm64 (read only) 10345860 10345860 0 0.0
(read/write) 706241 706241 0 0.0
.bss 33937 33937 0 0.0
.data 2864 2864 0 0.0
.data.rel.ro 650560 650560 0 0.0
.dynamic 560 560 0 0.0
.got 13904 13904 0 0.0
.init 24 24 0 0.0
.init_array 200 200 0 0.0
.rodata 503028 503028 0 0.0
.text 8188884 8188884 0 0.0
thermostat-no-ble arm64 (read only) 2368252 2368252 0 0.0
(read/write) 142497 142497 0 0.0
.bss 55233 55233 0 0.0
.data 1688 1688 0 0.0
.data.rel.ro 76704 76704 0 0.0
.dynamic 560 560 0 0.0
.got 5096 5096 0 0.0
.init 24 24 0 0.0
.init_array 416 416 0 0.0
.rodata 141588 141588 0 0.0
.text 1987664 1987664 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2455952 2455952 0 0.0
.bss 215044 215044 0 0.0
.data 5872 5872 0 0.0
.text 1418596 1418596 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1182723 1182723 0 0.0
bss 143641 143641 0 0.0
rodata 144212 144212 0 0.0
text 815932 815936 4 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1161923 1161923 0 0.0
bss 142868 142868 0 0.0
rodata 135804 135804 0 0.0
text 804328 804328 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read only) 841960 841960 0 0.0
(read/write) 1744780 1744780 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 188720 188720 0 0.0
.comment 204 204 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 1229158 1229158 0 0.0
.debug_aranges 111800 111800 0 0.0
.debug_frame 373224 373224 0 0.0
.debug_info 26813751 26813752 1 0.0
.debug_line 3668978 3668978 0 0.0
.debug_loc 3582717 3582717 0 0.0
.debug_ranges 340576 340576 0 0.0
.debug_str 3439101 3439101 0 0.0
.heap 841960 841960 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 571696 571696 0 0.0
.symtab 422080 422080 0 0.0
.text 1545008 1545008 0 0.0
.zero.table 8 8 0 0.0
text 0 0 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 (read only) 842696 842696 0 0.0
(read/write) 1687972 1687972 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 187984 187984 0 0.0
.comment 204 204 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 1220957 1220957 0 0.0
.debug_aranges 111272 111272 0 0.0
.debug_frame 376304 376304 0 0.0
.debug_info 26550599 26550599 0 0.0
.debug_line 3689694 3689694 0 0.0
.debug_loc 3570354 3570354 0 0.0
.debug_ranges 339192 339192 0 0.0
.debug_str 3428114 3428114 0 0.0
.heap 842696 842696 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 536170 536170 0 0.0
.symtab 408672 408672 0 0.0
.text 1488936 1488936 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
light cy8ckit_062s2_43012 (read only) 850928 850928 0 0.0
(read/write) 1605236 1605236 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 179960 179960 0 0.0
.comment 204 204 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 1055116 1055116 0 0.0
.debug_aranges 103448 103448 0 0.0
.debug_frame 346572 346572 0 0.0
.debug_info 22016853 22016853 0 0.0
.debug_line 3259861 3259861 0 0.0
.debug_loc 3268724 3268724 0 0.0
.debug_ranges 304664 304664 0 0.0
.debug_str 3233368 3233368 0 0.0
.heap 850928 850928 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 469469 469469 0 0.0
.symtab 375760 375760 0 0.0
.text 1414432 1414432 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) 1642924 1642924 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 184976 184976 0 0.0
.comment 204 204 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 1062571 1062571 0 0.0
.debug_aranges 104120 104120 0 0.0
.debug_frame 349400 349400 0 0.0
.debug_info 22396229 22396229 0 0.0
.debug_line 3268581 3268581 0 0.0
.debug_loc 3308561 3308561 0 0.0
.debug_ranges 308008 308008 0 0.0
.debug_str 3260785 3260785 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 475672 475672 0 0.0
.symtab 378944 378944 0 0.0
.text 1447088 1447088 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1130876 1130876 0 0.0
.bss 106112 106112 0 0.0
.data 1028 1028 0 0.0
.text 577972 577972 0 0.0
lock-app qpg6105+debug (read/write) 1101888 1101888 0 0.0
.bss 102344 102344 0 0.0
.data 1032 1032 0 0.0
.text 548988 548988 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 813868 813868 0 0.0
bss 71372 71372 0 0.0
noinit 43488 43488 0 0.0
text 574710 574710 0 0.0
lighting-app tlsr9518adk80d (read/write) 835800 835800 0 0.0
bss 72228 72228 0 0.0
noinit 43488 43488 0 0.0
text 592850 592850 0 0.0
ota-requestor-app tlsr9518adk80d (read/write) 843916 843924 8 0.0
bss 73136 73136 0 0.0
noinit 43488 43488 0 0.0
text 599112 599114 2 0.0

@bzbarsky-apple bzbarsky-apple force-pushed the darwin-controller-api-fixes branch from 49707b9 to 5b53dec Compare September 14, 2022 15:01
@bzbarsky-apple
Copy link
Contributor Author

Fast-tracking platform-specific fix reviewed by platform owner.

@github-actions
Copy link

github-actions bot commented Sep 14, 2022

PR #22596: Size comparison from f4f81f2 to 5b53dec

Increases (4 builds for cc13x2_26x2, esp32, psoc6, telink)
platform target config section f4f81f2 5b53dec change % change
cc13x2_26x2 shell LP_CC2652R7 (read only) 667974 667982 8 0.0
.text 581484 581492 8 0.0
esp32 all-clusters-app c3devkit (read/write) 1788102 1788110 8 0.0
.flash.rodata 257656 257664 8 0.0
psoc6 all-clusters cy8ckit_062s2_43012 .debug_info 26813750 26813751 1 0.0
telink ota-requestor-app tlsr9518adk80d (read/write) 843916 843924 8 0.0
text 599112 599114 2 0.0
Decreases (5 builds for cc13x2_26x2, esp32, psoc6, qpg, telink)
platform target config section f4f81f2 5b53dec change % change
cc13x2_26x2 shell LP_CC2652R7 (read/write) 178960 178952 -8 -0.0
esp32 all-clusters-app c3devkit (read only) 1223856 1223852 -4 -0.0
.flash.text 1223856 1223852 -4 -0.0
psoc6 light cy8ckit_062s2_43012 .debug_info 22016853 22016852 -1 -0.0
qpg lock-app qpg6105+debug (read/write) 1101888 1101880 -8 -0.0
.text 548988 548980 -8 -0.0
telink lighting-app tlsr9518adk80d text 592852 592850 -2 -0.0
Full report (35 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, psoc6, qpg, telink)
platform target config section f4f81f2 5b53dec change % change
bl602 lighting-app bl602 (read/write) 1412862 1412862 0 0.0
.bss 121585 121585 0 0.0
.data 9824 9824 0 0.0
.text 1067580 1067580 0 0.0
bl602+rpc (read/write) 1458330 1458330 0 0.0
.bss 129017 129017 0 0.0
.data 10208 10208 0 0.0
.text 1099184 1099184 0 0.0
bl702 lighting-app bl702 (read only) 3262 3262 0 0.0
(read/write) 1188427 1188427 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 66958 66958 0 0.0
.bss_psram 29696 29696 0 0.0
.comment 48 48 0 0.0
.data 4280 4280 0 0.0
.debug_abbrev 1506731 1506731 0 0.0
.debug_aranges 133072 133072 0 0.0
.debug_frame 486348 486348 0 0.0
.debug_info 37894018 37894018 0 0.0
.debug_line 5253486 5253486 0 0.0
.debug_loc 3363590 3363590 0 0.0
.debug_ranges 360256 360256 0 0.0
.debug_str 3455308 3455308 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 116504 116504 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 564895 564895 0 0.0
.symtab 171616 171616 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
957170 957170 0 0.0
bl702+rpc (read only) 3262 3262 0 0.0
(read/write) 1284339 1284339 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 75006 75006 0 0.0
.bss_psram 29936 29936 0 0.0
.comment 48 48 0 0.0
.data 4800 4800 0 0.0
.debug_abbrev 1644310 1644310 0 0.0
.debug_aranges 140576 140576 0 0.0
.debug_frame 511724 511724 0 0.0
.debug_info 41800589 41800589 0 0.0
.debug_line 5628025 5628025 0 0.0
.debug_loc 3556258 3556258 0 0.0
.debug_ranges 382712 382712 0 0.0
.debug_str 3851275 3851275 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 129896 129896 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 624068 624068 0 0.0
.symtab 189424 189424 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
1030874 1030874 0 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 676939 676939 0 0.0
(read/write) 174484 174484 0 0.0
.bss 74316 74316 0 0.0
.data 3380 3380 0 0.0
.rodata 89451 89451 0 0.0
.text 587176 587176 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 641651 641651 0 0.0
(read/write) 157884 157884 0 0.0
.bss 73588 73588 0 0.0
.data 3380 3380 0 0.0
.rodata 78595 78595 0 0.0
.text 562736 562736 0 0.0
lock-ftd LP_CC2652R7 (read only) 678063 678063 0 0.0
(read/write) 163672 163672 0 0.0
.bss 71516 71516 0 0.0
.data 3304 3304 0 0.0
.rodata 77295 77295 0 0.0
.text 600288 600288 0 0.0
lock-mtd LP_CC2652R7 (read only) 661883 661883 0 0.0
(read/write) 175540 175540 0 0.0
.bss 67204 67204 0 0.0
.data 3304 3304 0 0.0
.rodata 103131 103131 0 0.0
.text 558272 558272 0 0.0
pump-app LP_CC2652R7 (read only) 687291 687291 0 0.0
(read/write) 155148 155148 0 0.0
.bss 71452 71452 0 0.0
.data 3296 3296 0 0.0
.rodata 90563 90563 0 0.0
.text 596244 596244 0 0.0
pump-controller-app LP_CC2652R7 (read only) 671783 671783 0 0.0
(read/write) 170768 170768 0 0.0
.bss 71564 71564 0 0.0
.data 3292 3292 0 0.0
.rodata 86119 86119 0 0.0
.text 585184 585184 0 0.0
shell LP_CC2652R7 (read only) 667974 667982 8 0.0
(read/write) 178960 178952 -8 -0.0
.bss 76628 76628 0 0.0
.data 3376 3376 0 0.0
.rodata 86174 86174 0 0.0
.text 581484 581492 8 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 587410 587410 0 0.0
.app_xip_area 464068 464068 0 0.0
.bss 65776 65776 0 0.0
.data 744 744 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 594498 594498 0 0.0
.app_xip_area 465828 465828 0 0.0
.bss 71096 71096 0 0.0
.data 752 752 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 543482 543482 0 0.0
.app_xip_area 425164 425164 0 0.0
.bss 60784 60784 0 0.0
.data 716 716 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1110704 1110704 0 0.0
.bss 136332 136332 0 0.0
.data 2072 2072 0 0.0
.text 972280 972280 0 0.0
BRD4161A+rpc (read/write) 973588 973588 0 0.0
.bss 150844 150844 0 0.0
.data 2252 2252 0 0.0
.text 820472 820472 0 0.0
BRD4161A+rs911x (read/write) 1003968 1003968 0 0.0
.bss 169168 169168 0 0.0
.data 2064 2064 0 0.0
.text 832716 832716 0 0.0
lock-app BRD4161A+wf200 (read/write) 1151476 1151476 0 0.0
.bss 152248 152248 0 0.0
.data 2072 2072 0 0.0
.text 997136 997136 0 0.0
window-app BRD4161A (read/write) 1101928 1101928 0 0.0
.bss 137772 137772 0 0.0
.data 2096 2096 0 0.0
.text 962040 962040 0 0.0
esp32 all-clusters-app c3devkit (read only) 1223856 1223852 -4 -0.0
(read/write) 1788102 1788110 8 0.0
.dram0.bss 76952 76952 0 0.0
.dram0.data 13840 13840 0 0.0
.flash.rodata 257656 257664 8 0.0
.flash.text 1223856 1223852 -4 -0.0
.iram0.text 65204 65204 0 0.0
m5stack (read only) 1233723 1233723 0 0.0
(read/write) 563996 563996 0 0.0
.dram0.bss 82312 82312 0 0.0
.dram0.data 34296 34296 0 0.0
.flash.rodata 314720 314720 0 0.0
.flash.text 1228339 1228339 0 0.0
.iram0.text 123939 123939 0 0.0
k32w light k32w0+release (read/write) 649972 649972 0 0.0
.bss 70712 70712 0 0.0
.data 2068 2068 0 0.0
.text 574464 574464 0 0.0
lock k32w0+release (read/write) 706960 706960 0 0.0
.bss 71160 71160 0 0.0
.data 2076 2076 0 0.0
.text 630996 630996 0 0.0
linux chip-tool-ipv6only arm64 (read only) 10345860 10345860 0 0.0
(read/write) 706241 706241 0 0.0
.bss 33937 33937 0 0.0
.data 2864 2864 0 0.0
.data.rel.ro 650560 650560 0 0.0
.dynamic 560 560 0 0.0
.got 13904 13904 0 0.0
.init 24 24 0 0.0
.init_array 200 200 0 0.0
.rodata 503028 503028 0 0.0
.text 8188884 8188884 0 0.0
thermostat-no-ble arm64 (read only) 2368252 2368252 0 0.0
(read/write) 142497 142497 0 0.0
.bss 55233 55233 0 0.0
.data 1688 1688 0 0.0
.data.rel.ro 76704 76704 0 0.0
.dynamic 560 560 0 0.0
.got 5096 5096 0 0.0
.init 24 24 0 0.0
.init_array 416 416 0 0.0
.rodata 141588 141588 0 0.0
.text 1987664 1987664 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2455952 2455952 0 0.0
.bss 215044 215044 0 0.0
.data 5872 5872 0 0.0
.text 1418596 1418596 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read only) 841960 841960 0 0.0
(read/write) 1744780 1744780 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 188720 188720 0 0.0
.comment 204 204 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 1229158 1229158 0 0.0
.debug_aranges 111800 111800 0 0.0
.debug_frame 373224 373224 0 0.0
.debug_info 26813750 26813751 1 0.0
.debug_line 3668978 3668978 0 0.0
.debug_loc 3582717 3582717 0 0.0
.debug_ranges 340576 340576 0 0.0
.debug_str 3439101 3439101 0 0.0
.heap 841960 841960 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 571696 571696 0 0.0
.symtab 422080 422080 0 0.0
.text 0 0 0 0.0
1545008 1545008 0 0.0
.zero.table 8 8 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 (read only) 842696 842696 0 0.0
(read/write) 1687972 1687972 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 187984 187984 0 0.0
.comment 204 204 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 1220957 1220957 0 0.0
.debug_aranges 111272 111272 0 0.0
.debug_frame 376304 376304 0 0.0
.debug_info 26550600 26550600 0 0.0
.debug_line 3689694 3689694 0 0.0
.debug_loc 3570354 3570354 0 0.0
.debug_ranges 339192 339192 0 0.0
.debug_str 3428114 3428114 0 0.0
.heap 842696 842696 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 536170 536170 0 0.0
.symtab 408672 408672 0 0.0
.text 1488936 1488936 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
light cy8ckit_062s2_43012 (read only) 850928 850928 0 0.0
(read/write) 1605236 1605236 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 179960 179960 0 0.0
.comment 204 204 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 1055116 1055116 0 0.0
.debug_aranges 103448 103448 0 0.0
.debug_frame 346572 346572 0 0.0
.debug_info 22016853 22016852 -1 -0.0
.debug_line 3259861 3259861 0 0.0
.debug_loc 3268724 3268724 0 0.0
.debug_ranges 304664 304664 0 0.0
.debug_str 3233368 3233368 0 0.0
.heap 850928 850928 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 469469 469469 0 0.0
.symtab 375760 375760 0 0.0
.text 1414432 1414432 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) 1642924 1642924 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 184976 184976 0 0.0
.comment 204 204 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 1062571 1062571 0 0.0
.debug_aranges 104120 104120 0 0.0
.debug_frame 349400 349400 0 0.0
.debug_info 22396228 22396228 0 0.0
.debug_line 3268581 3268581 0 0.0
.debug_loc 3308561 3308561 0 0.0
.debug_ranges 308008 308008 0 0.0
.debug_str 3260785 3260785 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 475672 475672 0 0.0
.symtab 378944 378944 0 0.0
.text 1447088 1447088 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1130876 1130876 0 0.0
.bss 106112 106112 0 0.0
.data 1028 1028 0 0.0
.text 577972 577972 0 0.0
lock-app qpg6105+debug (read/write) 1101888 1101880 -8 -0.0
.bss 102344 102344 0 0.0
.data 1032 1032 0 0.0
.text 548988 548980 -8 -0.0
telink light-switch-app tlsr9518adk80d (read/write) 813868 813868 0 0.0
bss 71372 71372 0 0.0
noinit 43488 43488 0 0.0
text 574712 574712 0 0.0
lighting-app tlsr9518adk80d (read/write) 835800 835800 0 0.0
bss 72228 72228 0 0.0
noinit 43488 43488 0 0.0
text 592852 592850 -2 -0.0
ota-requestor-app tlsr9518adk80d (read/write) 843916 843924 8 0.0
bss 73136 73136 0 0.0
noinit 43488 43488 0 0.0
text 599112 599114 2 0.0

@bzbarsky-apple bzbarsky-apple force-pushed the darwin-controller-api-fixes branch from 5b53dec to 1307520 Compare September 14, 2022 16:40
* Make MTRBaseDevice creation synchronous.  This requires updates to
  MTRBaseDeviceOverXPC to do the possible async getting of the controller id it
  needs during its async operations, not when getting the device object.
* Rename "pairDevice" to "setupCommissioningSessionWithPayload", fix its
  signature, improve documentation.
* Rename "commissionDevice" to "commissionNodeWithID".
* Rename "stopDevicePairing" to "cancelCommissioningForNodeID" and document.
* Rename "getDeviceBeingCommissioned" to "getDeviceBeingCommissionedWithNodeID".
* Various documentation improvements.
* Add a way to generate a QR code from an MTRSetupPayload to allow correct
  recovery of long discriminators in setupCommissioningSessionWithPayload.
* Fix signature of computePaseVerifier.
* Fix a leak when we failed to start a controller because it wanted a fabric
  that does not exist, or wanted a new fabric and a matching one existed.  This
  used to not show up in LSan before, maybe because we did not have an
  autoreleasepool in place.

Fixes project-chip#22533

Addresses part of project-chip#22420
@bzbarsky-apple bzbarsky-apple force-pushed the darwin-controller-api-fixes branch from 1307520 to 20a83bb Compare September 14, 2022 16:50
@github-actions
Copy link

github-actions bot commented Sep 14, 2022

PR #22596: Size comparison from e0a9cca to 20a83bb

Increases (3 builds for cc13x2_26x2, nrfconnect, telink)
platform target config section e0a9cca 20a83bb change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read only) 678055 678063 8 0.0
.text 600280 600288 8 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 text 815932 815936 4 0.0
telink ota-requestor-app tlsr9518adk80d (read/write) 843916 843924 8 0.0
text 599112 599114 2 0.0
Decreases (9 builds for bl602, bl702, cc13x2_26x2, esp32, nrfconnect, qpg, telink)
platform target config section e0a9cca 20a83bb change % change
bl602 lighting-app bl602 .text 1067582 1067580 -2 -0.0
bl602+rpc (read/write) 1458338 1458330 -8 -0.0
.text 1099186 1099184 -2 -0.0
bl702 lighting-app bl702 .debug_info 37894019 37894018 -1 -0.0
.text 957172 957170 -2 -0.0
bl702+rpc .debug_info 41800589 41800588 -1 -0.0
.text 1030874 1030872 -2 -0.0
cc13x2_26x2 lock-ftd LP_CC2652R7 (read/write) 163680 163672 -8 -0.0
esp32 all-clusters-app c3devkit (read/write) 1788110 1788102 -8 -0.0
.flash.rodata 257664 257656 -8 -0.0
nrfconnect all-clusters-minimal-app nrf52840dk_nrf52840 text 804328 804324 -4 -0.0
qpg lighting-app qpg6105+debug (read/write) 1130876 1130868 -8 -0.0
.text 577972 577964 -8 -0.0
telink lighting-app tlsr9518adk80d text 592852 592850 -2 -0.0
Full report (37 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section e0a9cca 20a83bb change % change
bl602 lighting-app bl602 (read/write) 1412862 1412862 0 0.0
.bss 121585 121585 0 0.0
.data 9824 9824 0 0.0
.text 1067582 1067580 -2 -0.0
bl602+rpc (read/write) 1458338 1458330 -8 -0.0
.bss 129017 129017 0 0.0
.data 10208 10208 0 0.0
.text 1099186 1099184 -2 -0.0
bl702 lighting-app bl702 0 0 0 0.0
(read only) 3262 3262 0 0.0
(read/write) 1188427 1188427 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 66958 66958 0 0.0
.bss_psram 29696 29696 0 0.0
.comment 48 48 0 0.0
.data 4280 4280 0 0.0
.debug_abbrev 1506731 1506731 0 0.0
.debug_aranges 133072 133072 0 0.0
.debug_frame 486348 486348 0 0.0
.debug_info 37894019 37894018 -1 -0.0
.debug_line 5253486 5253486 0 0.0
.debug_loc 3363590 3363590 0 0.0
.debug_ranges 360256 360256 0 0.0
.debug_str 3455308 3455308 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 116504 116504 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 564895 564895 0 0.0
.symtab 171616 171616 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 957172 957170 -2 -0.0
bl702+rpc 0 0 0 0.0
(read only) 3262 3262 0 0.0
(read/write) 1284339 1284339 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 75006 75006 0 0.0
.bss_psram 29936 29936 0 0.0
.comment 48 48 0 0.0
.data 4800 4800 0 0.0
.debug_abbrev 1644310 1644310 0 0.0
.debug_aranges 140576 140576 0 0.0
.debug_frame 511724 511724 0 0.0
.debug_info 41800589 41800588 -1 -0.0
.debug_line 5628025 5628025 0 0.0
.debug_loc 3556258 3556258 0 0.0
.debug_ranges 382712 382712 0 0.0
.debug_str 3851275 3851275 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 129896 129896 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 624068 624068 0 0.0
.symtab 189424 189424 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 1030874 1030872 -2 -0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 676939 676939 0 0.0
(read/write) 174484 174484 0 0.0
.bss 74316 74316 0 0.0
.data 3380 3380 0 0.0
.rodata 89451 89451 0 0.0
.text 587176 587176 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 641651 641651 0 0.0
(read/write) 157884 157884 0 0.0
.bss 73588 73588 0 0.0
.data 3380 3380 0 0.0
.rodata 78595 78595 0 0.0
.text 562736 562736 0 0.0
lock-ftd LP_CC2652R7 (read only) 678055 678063 8 0.0
(read/write) 163680 163672 -8 -0.0
.bss 71516 71516 0 0.0
.data 3304 3304 0 0.0
.rodata 77295 77295 0 0.0
.text 600280 600288 8 0.0
lock-mtd LP_CC2652R7 (read only) 661883 661883 0 0.0
(read/write) 175540 175540 0 0.0
.bss 67204 67204 0 0.0
.data 3304 3304 0 0.0
.rodata 103131 103131 0 0.0
.text 558272 558272 0 0.0
pump-app LP_CC2652R7 (read only) 687291 687291 0 0.0
(read/write) 155148 155148 0 0.0
.bss 71452 71452 0 0.0
.data 3296 3296 0 0.0
.rodata 90563 90563 0 0.0
.text 596244 596244 0 0.0
pump-controller-app LP_CC2652R7 (read only) 671783 671783 0 0.0
(read/write) 170768 170768 0 0.0
.bss 71564 71564 0 0.0
.data 3292 3292 0 0.0
.rodata 86119 86119 0 0.0
.text 585184 585184 0 0.0
shell LP_CC2652R7 (read only) 667982 667982 0 0.0
(read/write) 178952 178952 0 0.0
.bss 76628 76628 0 0.0
.data 3376 3376 0 0.0
.rodata 86174 86174 0 0.0
.text 581492 581492 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 587410 587410 0 0.0
.app_xip_area 464068 464068 0 0.0
.bss 65776 65776 0 0.0
.data 744 744 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 594498 594498 0 0.0
.app_xip_area 465828 465828 0 0.0
.bss 71096 71096 0 0.0
.data 752 752 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 543482 543482 0 0.0
.app_xip_area 425164 425164 0 0.0
.bss 60784 60784 0 0.0
.data 716 716 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1110704 1110704 0 0.0
.bss 136332 136332 0 0.0
.data 2072 2072 0 0.0
.text 972280 972280 0 0.0
BRD4161A+rpc (read/write) 973588 973588 0 0.0
.bss 150844 150844 0 0.0
.data 2252 2252 0 0.0
.text 820472 820472 0 0.0
BRD4161A+rs911x (read/write) 1003968 1003968 0 0.0
.bss 169168 169168 0 0.0
.data 2064 2064 0 0.0
.text 832716 832716 0 0.0
lock-app BRD4161A+wf200 (read/write) 1151476 1151476 0 0.0
.bss 152248 152248 0 0.0
.data 2072 2072 0 0.0
.text 997136 997136 0 0.0
window-app BRD4161A (read/write) 1101928 1101928 0 0.0
.bss 137772 137772 0 0.0
.data 2096 2096 0 0.0
.text 962040 962040 0 0.0
esp32 all-clusters-app c3devkit (read only) 1223856 1223856 0 0.0
(read/write) 1788110 1788102 -8 -0.0
.dram0.bss 76952 76952 0 0.0
.dram0.data 13840 13840 0 0.0
.flash.rodata 257664 257656 -8 -0.0
.flash.text 1223856 1223856 0 0.0
.iram0.text 65204 65204 0 0.0
m5stack (read only) 1233723 1233723 0 0.0
(read/write) 563996 563996 0 0.0
.dram0.bss 82312 82312 0 0.0
.dram0.data 34296 34296 0 0.0
.flash.rodata 314720 314720 0 0.0
.flash.text 1228339 1228339 0 0.0
.iram0.text 123939 123939 0 0.0
k32w light k32w0+release (read/write) 649972 649972 0 0.0
.bss 70712 70712 0 0.0
.data 2068 2068 0 0.0
.text 574464 574464 0 0.0
lock k32w0+release (read/write) 706960 706960 0 0.0
.bss 71160 71160 0 0.0
.data 2076 2076 0 0.0
.text 630996 630996 0 0.0
linux chip-tool-ipv6only arm64 (read only) 10345860 10345860 0 0.0
(read/write) 706241 706241 0 0.0
.bss 33937 33937 0 0.0
.data 2864 2864 0 0.0
.data.rel.ro 650560 650560 0 0.0
.dynamic 560 560 0 0.0
.got 13904 13904 0 0.0
.init 24 24 0 0.0
.init_array 200 200 0 0.0
.rodata 503028 503028 0 0.0
.text 8188884 8188884 0 0.0
thermostat-no-ble arm64 (read only) 2368252 2368252 0 0.0
(read/write) 142497 142497 0 0.0
.bss 55233 55233 0 0.0
.data 1688 1688 0 0.0
.data.rel.ro 76704 76704 0 0.0
.dynamic 560 560 0 0.0
.got 5096 5096 0 0.0
.init 24 24 0 0.0
.init_array 416 416 0 0.0
.rodata 141588 141588 0 0.0
.text 1987664 1987664 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2455952 2455952 0 0.0
.bss 215044 215044 0 0.0
.data 5872 5872 0 0.0
.text 1418596 1418596 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1182723 1182723 0 0.0
bss 143641 143641 0 0.0
rodata 144212 144212 0 0.0
text 815932 815936 4 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1161923 1161923 0 0.0
bss 142868 142868 0 0.0
rodata 135804 135804 0 0.0
text 804328 804324 -4 -0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read only) 841960 841960 0 0.0
(read/write) 1744780 1744780 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 188720 188720 0 0.0
.comment 204 204 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 1229158 1229158 0 0.0
.debug_aranges 111800 111800 0 0.0
.debug_frame 373224 373224 0 0.0
.debug_info 26813750 26813750 0 0.0
.debug_line 3668978 3668978 0 0.0
.debug_loc 3582717 3582717 0 0.0
.debug_ranges 340576 340576 0 0.0
.debug_str 3439101 3439101 0 0.0
.heap 841960 841960 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 571696 571696 0 0.0
.symtab 422080 422080 0 0.0
.text 0 0 0 0.0
1545008 1545008 0 0.0
.zero.table 8 8 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 (read only) 842696 842696 0 0.0
(read/write) 1687972 1687972 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 187984 187984 0 0.0
.comment 204 204 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 1220957 1220957 0 0.0
.debug_aranges 111272 111272 0 0.0
.debug_frame 376304 376304 0 0.0
.debug_info 26550599 26550599 0 0.0
.debug_line 3689694 3689694 0 0.0
.debug_loc 3570354 3570354 0 0.0
.debug_ranges 339192 339192 0 0.0
.debug_str 3428114 3428114 0 0.0
.heap 842696 842696 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 536170 536170 0 0.0
.symtab 408672 408672 0 0.0
.text 1488936 1488936 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
light cy8ckit_062s2_43012 (read only) 850928 850928 0 0.0
(read/write) 1605236 1605236 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 179960 179960 0 0.0
.comment 204 204 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 1055116 1055116 0 0.0
.debug_aranges 103448 103448 0 0.0
.debug_frame 346572 346572 0 0.0
.debug_info 22016852 22016852 0 0.0
.debug_line 3259861 3259861 0 0.0
.debug_loc 3268724 3268724 0 0.0
.debug_ranges 304664 304664 0 0.0
.debug_str 3233368 3233368 0 0.0
.heap 850928 850928 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 469469 469469 0 0.0
.symtab 375760 375760 0 0.0
.text 1414432 1414432 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) 1642924 1642924 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 184976 184976 0 0.0
.comment 204 204 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 1062571 1062571 0 0.0
.debug_aranges 104120 104120 0 0.0
.debug_frame 349400 349400 0 0.0
.debug_info 22396228 22396228 0 0.0
.debug_line 3268581 3268581 0 0.0
.debug_loc 3308561 3308561 0 0.0
.debug_ranges 308008 308008 0 0.0
.debug_str 3260785 3260785 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 475672 475672 0 0.0
.symtab 378944 378944 0 0.0
.text 1447088 1447088 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1130876 1130868 -8 -0.0
.bss 106112 106112 0 0.0
.data 1028 1028 0 0.0
.text 577972 577964 -8 -0.0
lock-app qpg6105+debug (read/write) 1101888 1101888 0 0.0
.bss 102344 102344 0 0.0
.data 1032 1032 0 0.0
.text 548988 548988 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 813868 813868 0 0.0
bss 71372 71372 0 0.0
noinit 43488 43488 0 0.0
text 574710 574710 0 0.0
lighting-app tlsr9518adk80d (read/write) 835800 835800 0 0.0
bss 72228 72228 0 0.0
noinit 43488 43488 0 0.0
text 592852 592850 -2 -0.0
ota-requestor-app tlsr9518adk80d (read/write) 843916 843924 8 0.0
bss 73136 73136 0 0.0
noinit 43488 43488 0 0.0
text 599112 599114 2 0.0

@jtung-apple jtung-apple enabled auto-merge (squash) September 14, 2022 18:32
@jtung-apple jtung-apple merged commit 5757068 into project-chip:master Sep 14, 2022
@bzbarsky-apple bzbarsky-apple deleted the darwin-controller-api-fixes branch September 15, 2022 16:00
isiu-apple pushed a commit to isiu-apple/connectedhomeip that referenced this pull request Sep 16, 2022
* Address API review issues in MTRDeviceController.

* Make MTRBaseDevice creation synchronous.  This requires updates to
  MTRBaseDeviceOverXPC to do the possible async getting of the controller id it
  needs during its async operations, not when getting the device object.
* Rename "pairDevice" to "setupCommissioningSessionWithPayload", fix its
  signature, improve documentation.
* Rename "commissionDevice" to "commissionNodeWithID".
* Rename "stopDevicePairing" to "cancelCommissioningForNodeID" and document.
* Rename "getDeviceBeingCommissioned" to "getDeviceBeingCommissionedWithNodeID".
* Various documentation improvements.
* Add a way to generate a QR code from an MTRSetupPayload to allow correct
  recovery of long discriminators in setupCommissioningSessionWithPayload.
* Fix signature of computePaseVerifier.
* Fix a leak when we failed to start a controller because it wanted a fabric
  that does not exist, or wanted a new fabric and a matching one existed.  This
  used to not show up in LSan before, maybe because we did not have an
  autoreleasepool in place.

Fixes project-chip#22533

Addresses part of project-chip#22420

* Address review comments.
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this pull request Oct 13, 2022
…oller.

This is a re-landing of some of the API addition parts of
project-chip#22596 and
project-chip#22566, without including
any of the API changes/removals. The code is nearly identical to what was on
master before Darwin framework changes were reverted in
project-chip#23155; the differences have
to do with the different type and name of MTRSetupPayload's
rendezvousInformation.
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this pull request Oct 13, 2022
…oller.

This is a re-landing of some of the API addition parts of
project-chip#22596 and
project-chip#22566, without including
any of the API changes/removals. The code is nearly identical to what was on
master before Darwin framework changes were reverted in
project-chip#23155; the differences have
to do with the different type and name of MTRSetupPayload's
rendezvousInformation.
bzbarsky-apple added a commit that referenced this pull request Oct 13, 2022
…oller. (#23159)

This is a re-landing of some of the API addition parts of
#22596 and
#22566, without including
any of the API changes/removals. The code is nearly identical to what was on
master before Darwin framework changes were reverted in
#23155; the differences have
to do with the different type and name of MTRSetupPayload's
rendezvousInformation.
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this pull request Nov 4, 2022
This is a re-landing of PR project-chip#22596 but with changes made for backwards compat.

* Make MTRBaseDevice creation synchronous.  This requires updates to
  MTRBaseDeviceOverXPC to do the possible async getting of the controller id it
  needs during its async operations, not when getting the device object.
* Deprecate pairDevice methods.
* Rename "commissionDevice" to "commissionNodeWithID".
* Rename "stopDevicePairing" to "cancelCommissioningForNodeID" and document.
* Rename "getDeviceBeingCommissioned" to "getDeviceBeingCommissionedWithNodeID".
* Various documentation improvements.
* Fix signature of computePaseVerifier.

* Address review comments.
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this pull request Nov 4, 2022
This is a re-landing of PR project-chip#22596 but with changes made for backwards compat.

* Make MTRBaseDevice creation synchronous.  This requires updates to
  MTRBaseDeviceOverXPC to do the possible async getting of the controller id it
  needs during its async operations, not when getting the device object.
* Deprecate pairDevice methods.
* Rename "commissionDevice" to "commissionNodeWithID".
* Rename "stopDevicePairing" to "cancelCommissioningForNodeID" and document.
* Rename "getDeviceBeingCommissioned" to "getDeviceBeingCommissionedWithNodeID".
* Various documentation improvements.
* Fix signature of computePaseVerifier.

The header changes not accompanied by backwards-compat shims are OK for the
following reasons:

* In MTRDeviceController.h, the change to isRunning is binary and source
  compatible.
* MTRDeviceControllerOverXPC_Internal.h is not public API.
* MTRDeviceController_Internal.h is not public API.
* MTRDeviceOverXPC.h is not public API.
* The changes to MTRSetupPayload.h are backward-compatible.
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this pull request Nov 8, 2022
This is a re-landing of PR project-chip#22596 but with changes made for backwards compat.

* Make MTRBaseDevice creation synchronous.  This requires updates to
  MTRBaseDeviceOverXPC to do the possible async getting of the controller id it
  needs during its async operations, not when getting the device object.
* Deprecate pairDevice methods.
* Rename "commissionDevice" to "commissionNodeWithID".
* Rename "stopDevicePairing" to "cancelCommissioningForNodeID" and document.
* Rename "getDeviceBeingCommissioned" to "getDeviceBeingCommissionedWithNodeID".
* Various documentation improvements.
* Fix signature of computePaseVerifier.

The header changes not accompanied by backwards-compat shims are OK for the
following reasons:

* In MTRDeviceController.h, the change to isRunning is binary and source
  compatible.
* MTRDeviceControllerOverXPC_Internal.h is not public API.
* MTRDeviceController_Internal.h is not public API.
* MTRDeviceOverXPC.h is not public API.
* The changes to MTRSetupPayload.h are backward-compatible.
bzbarsky-apple added a commit that referenced this pull request Nov 9, 2022
This is a re-landing of PR #22596 but with changes made for backwards compat.

* Make MTRBaseDevice creation synchronous.  This requires updates to
  MTRBaseDeviceOverXPC to do the possible async getting of the controller id it
  needs during its async operations, not when getting the device object.
* Deprecate pairDevice methods.
* Rename "commissionDevice" to "commissionNodeWithID".
* Rename "stopDevicePairing" to "cancelCommissioningForNodeID" and document.
* Rename "getDeviceBeingCommissioned" to "getDeviceBeingCommissionedWithNodeID".
* Various documentation improvements.
* Fix signature of computePaseVerifier.

The header changes not accompanied by backwards-compat shims are OK for the
following reasons:

* In MTRDeviceController.h, the change to isRunning is binary and source
  compatible.
* MTRDeviceControllerOverXPC_Internal.h is not public API.
* MTRDeviceController_Internal.h is not public API.
* MTRDeviceOverXPC.h is not public API.
* The changes to MTRSetupPayload.h are backward-compatible.
adbridge pushed a commit to ARM-software/connectedhomeip that referenced this pull request Nov 18, 2022
This is a re-landing of PR project-chip#22596 but with changes made for backwards compat.

* Make MTRBaseDevice creation synchronous.  This requires updates to
  MTRBaseDeviceOverXPC to do the possible async getting of the controller id it
  needs during its async operations, not when getting the device object.
* Deprecate pairDevice methods.
* Rename "commissionDevice" to "commissionNodeWithID".
* Rename "stopDevicePairing" to "cancelCommissioningForNodeID" and document.
* Rename "getDeviceBeingCommissioned" to "getDeviceBeingCommissionedWithNodeID".
* Various documentation improvements.
* Fix signature of computePaseVerifier.

The header changes not accompanied by backwards-compat shims are OK for the
following reasons:

* In MTRDeviceController.h, the change to isRunning is binary and source
  compatible.
* MTRDeviceControllerOverXPC_Internal.h is not public API.
* MTRDeviceController_Internal.h is not public API.
* MTRDeviceOverXPC.h is not public API.
* The changes to MTRSetupPayload.h are backward-compatible.
adbridge pushed a commit to ARM-software/connectedhomeip that referenced this pull request Nov 18, 2022
This is a re-landing of PR project-chip#22596 but with changes made for backwards compat.

* Make MTRBaseDevice creation synchronous.  This requires updates to
  MTRBaseDeviceOverXPC to do the possible async getting of the controller id it
  needs during its async operations, not when getting the device object.
* Deprecate pairDevice methods.
* Rename "commissionDevice" to "commissionNodeWithID".
* Rename "stopDevicePairing" to "cancelCommissioningForNodeID" and document.
* Rename "getDeviceBeingCommissioned" to "getDeviceBeingCommissionedWithNodeID".
* Various documentation improvements.
* Fix signature of computePaseVerifier.

The header changes not accompanied by backwards-compat shims are OK for the
following reasons:

* In MTRDeviceController.h, the change to isRunning is binary and source
  compatible.
* MTRDeviceControllerOverXPC_Internal.h is not public API.
* MTRDeviceController_Internal.h is not public API.
* MTRDeviceOverXPC.h is not public API.
* The changes to MTRSetupPayload.h are backward-compatible.
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.

[Platform] Apply darwin API review changes to MTRDeviceController
4 participants