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

[Fabric-Sync] Port commissioner control protocol implementation #36427

Merged
merged 9 commits into from
Nov 14, 2024

Conversation

yufengwangca
Copy link
Contributor

@yufengwangca yufengwangca commented Nov 7, 2024

Port commissioner control protocol implementation from fabric-admin/fabric-bridge-app to fabric-sync.

Most of the code are ported from fabric-admin/fabric-bridge-app as is, the key changes are replacing all RPC call with direct API call.

  1. Build Fabric Sync (FS) on E1 and E2
    ./scripts/examples/gn_build_example.sh examples/fabric-sync out/debug/standalone

  2. Run the Fabric Sync example app on E1 and E2

rm -rf /tmp/fabric_sync.log 
sudo rm -rf /tmp/chip_*
./fabric-sync 
  1. Initiate the FS Setup Process from E1 to E2 (192.168.86.246)
> app add-bridge 1 20202021 192.168.86.246 5540
Done
> New device with Node ID: 0000000000000001 has been successfully added.
A new device has been added on Endpoint: 2.
  1. Verify Reverse Commissioning of the Fabric-Bridge from E1 on E2
    > New device with Node ID: 0000000000000002 has been successfully added.

Copy link

Review changes with  SemanticDiff

Copy link

github-actions bot commented Nov 8, 2024

PR #36427: Size comparison from e338404 to 1b16bb1

Increases above 0.2%:

platform target config section e338404 1b16bb1 change % change
linux fabric-sync debug FLASH 5309445 5450405 140960 2.7
RAM 466960 470824 3864 0.8
Full report (66 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
platform target config section e338404 1b16bb1 change % change
bl602 lighting-app bl602+mfd+littlefs+rpc FLASH 1349358 1349358 0 0.0
RAM 104080 104080 0 0.0
bl702 lighting-app bl702+eth FLASH 649376 649376 0 0.0
RAM 25321 25321 0 0.0
bl702+wifi FLASH 826956 826956 0 0.0
RAM 14053 14053 0 0.0
bl706+mfd+rpc+littlefs FLASH 1055154 1055154 0 0.0
RAM 23893 23893 0 0.0
bl702l lighting-app bl702l+mfd+littlefs FLASH 976520 976520 0 0.0
RAM 16556 16556 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 837860 837860 0 0.0
RAM 123628 123628 0 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 823776 823776 0 0.0
RAM 125516 125516 0 0.0
pump-app LP_EM_CC1354P10_6 FLASH 770516 770516 0 0.0
RAM 113984 113984 0 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 754720 754720 0 0.0
RAM 114192 114192 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 625910 625910 0 0.0
RAM 205780 205780 0 0.0
lock CC3235SF_LAUNCHXL FLASH 667222 667222 0 0.0
RAM 205932 205932 0 0.0
cyw30739 light CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 679497 679497 0 0.0
RAM 78684 78684 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 699341 699341 0 0.0
RAM 81324 81324 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 699341 699341 0 0.0
RAM 81324 81324 0 0.0
CYW930739M2EVB-02 unknown 2040 2040 0 0.0
FLASH 656285 656285 0 0.0
RAM 73752 73752 0 0.0
light-switch CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 616345 616345 0 0.0
RAM 71644 71644 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 635973 635973 0 0.0
RAM 74188 74188 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 635973 635973 0 0.0
RAM 74188 74188 0 0.0
lock CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 635801 635801 0 0.0
RAM 74692 74692 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 655509 655509 0 0.0
RAM 77236 77236 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 655509 655509 0 0.0
RAM 77236 77236 0 0.0
thermostat CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 611197 611197 0 0.0
RAM 68772 68772 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 631049 631049 0 0.0
RAM 71412 71412 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 631049 631049 0 0.0
RAM 71412 71412 0 0.0
efr32 lock-app BRD4187C FLASH 927628 927628 0 0.0
RAM 160160 160160 0 0.0
BRD4338a FLASH 743680 743680 0 0.0
RAM 233288 233288 0 0.0
window-app BRD4187C FLASH 1018784 1018776 -8 -0.0
RAM 128260 128260 0 0.0
esp32 all-clusters-app c3devkit DRAM 95328 95328 0 0.0
FLASH 1540452 1540452 0 0.0
IRAM 82542 82542 0 0.0
m5stack DRAM 116256 116256 0 0.0
FLASH 1549250 1549250 0 0.0
IRAM 117039 117039 0 0.0
linux air-purifier-app debug unknown 4712 4712 0 0.0
FLASH 2698811 2698811 0 0.0
RAM 129648 129648 0 0.0
all-clusters-app debug unknown 5552 5552 0 0.0
FLASH 5992540 5992540 0 0.0
RAM 523968 523968 0 0.0
all-clusters-minimal-app debug unknown 5448 5448 0 0.0
FLASH 5328148 5328148 0 0.0
RAM 242480 242480 0 0.0
bridge-app debug unknown 5432 5432 0 0.0
FLASH 4670886 4670886 0 0.0
RAM 218416 218416 0 0.0
chip-tool debug unknown 5984 5984 0 0.0
FLASH 12872126 12872126 0 0.0
RAM 584466 584466 0 0.0
fabric-admin debug unknown 5808 5808 0 0.0
FLASH 11277209 11277209 0 0.0
RAM 584890 584890 0 0.0
fabric-bridge-app debug unknown 4688 4688 0 0.0
FLASH 4495154 4495154 0 0.0
RAM 205400 205400 0 0.0
fabric-sync debug unknown 4880 4880 0 0.0
FLASH 5309445 5450405 140960 2.7
RAM 466960 470824 3864 0.8
lighting-app debug+rpc+ui unknown 6096 6096 0 0.0
FLASH 5606817 5606817 0 0.0
RAM 228552 228552 0 0.0
lock-app debug unknown 5368 5368 0 0.0
FLASH 4720360 4720360 0 0.0
RAM 204552 204552 0 0.0
ota-provider-app debug unknown 4744 4744 0 0.0
FLASH 4345792 4345792 0 0.0
RAM 198272 198272 0 0.0
ota-requestor-app debug unknown 4680 4680 0 0.0
FLASH 4484194 4484194 0 0.0
RAM 202840 202840 0 0.0
shell debug unknown 4240 4240 0 0.0
FLASH 3011213 3011213 0 0.0
RAM 160368 160368 0 0.0
tv-app debug unknown 5648 5648 0 0.0
FLASH 5942501 5942501 0 0.0
RAM 596336 596336 0 0.0
tv-casting-app debug unknown 5232 5232 0 0.0
FLASH 11079325 11079325 0 0.0
RAM 694536 694536 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 915356 915356 0 0.0
RAM 143281 143281 0 0.0
nrf7002dk_nrf5340_cpuapp FLASH 887588 887588 0 0.0
RAM 141468 141468 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 847612 847612 0 0.0
RAM 142189 142189 0 0.0
nxp contact k32w0+release FLASH 584144 584144 0 0.0
RAM 71044 71044 0 0.0
mcxw71+release FLASH 598688 598688 0 0.0
RAM 63144 63144 0 0.0
light k32w0+release FLASH 611204 611204 0 0.0
RAM 70436 70436 0 0.0
k32w1+release FLASH 685000 685000 0 0.0
RAM 48776 48776 0 0.0
lock mcxw71+release FLASH 748528 748528 0 0.0
RAM 67300 67300 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 FLASH 1642964 1642964 0 0.0
RAM 212056 212056 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 FLASH 1550164 1550164 0 0.0
RAM 208856 208856 0 0.0
light cy8ckit_062s2_43012 FLASH 1466604 1466604 0 0.0
RAM 200840 200840 0 0.0
lock cy8ckit_062s2_43012 FLASH 1464780 1464780 0 0.0
RAM 225200 225200 0 0.0
qpg lighting-app qpg6105+debug FLASH 662168 662168 0 0.0
RAM 105380 105380 0 0.0
lock-app qpg6105+debug FLASH 620268 620268 0 0.0
RAM 99832 99832 0 0.0
stm32 light STM32WB5MM-DK FLASH 482968 482968 0 0.0
RAM 144844 144844 0 0.0
telink bridge-app tlsr9258a FLASH 684674 684674 0 0.0
RAM 91528 91528 0 0.0
contact-sensor-app tlsr9528a_retention FLASH 622686 622686 0 0.0
RAM 50464 50464 0 0.0
light-switch-app-ota-compress-lzma-shell-factory-data tlsr9528a FLASH 710612 710612 0 0.0
RAM 73804 73804 0 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 628980 628980 0 0.0
RAM 145384 145384 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 814868 814868 0 0.0
RAM 100004 100004 0 0.0
tizen all-clusters-app arm unknown 4948 4948 0 0.0
FLASH 1722540 1722540 0 0.0
RAM 90656 90656 0 0.0
chip-tool-ubsan arm unknown 10832 10832 0 0.0
FLASH 18007422 18007422 0 0.0
RAM 7857564 7857564 0 0.0

@mergify mergify bot added the conflict label Nov 8, 2024
Copy link

github-actions bot commented Nov 8, 2024

PR #36427: Size comparison from 18c0788 to dc958b1

Increases above 0.2%:

platform target config section 18c0788 dc958b1 change % change
linux fabric-sync debug FLASH 5309077 5450549 141472 2.7
RAM 466872 470824 3952 0.8
Full report (68 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
platform target config section 18c0788 dc958b1 change % change
bl602 lighting-app bl602+mfd+littlefs+rpc FLASH 1349426 1349426 0 0.0
RAM 104080 104080 0 0.0
bl702 lighting-app bl702+eth FLASH 649386 649386 0 0.0
RAM 25321 25321 0 0.0
bl702+wifi FLASH 826966 826966 0 0.0
RAM 14053 14053 0 0.0
bl706+mfd+rpc+littlefs FLASH 1055164 1055164 0 0.0
RAM 23893 23893 0 0.0
bl702l lighting-app bl702l+mfd+littlefs FLASH 976530 976530 0 0.0
RAM 16556 16556 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 837876 837876 0 0.0
RAM 123628 123628 0 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 823792 823792 0 0.0
RAM 125516 125516 0 0.0
pump-app LP_EM_CC1354P10_6 FLASH 770532 770532 0 0.0
RAM 113984 113984 0 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 754736 754736 0 0.0
RAM 114192 114192 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 625926 625926 0 0.0
RAM 205780 205780 0 0.0
lock CC3235SF_LAUNCHXL FLASH 667238 667238 0 0.0
RAM 205932 205932 0 0.0
cyw30739 light CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 679513 679513 0 0.0
RAM 78684 78684 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 699357 699357 0 0.0
RAM 81324 81324 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 699357 699357 0 0.0
RAM 81324 81324 0 0.0
CYW930739M2EVB-02 unknown 2040 2040 0 0.0
FLASH 656301 656301 0 0.0
RAM 73752 73752 0 0.0
light-switch CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 616361 616361 0 0.0
RAM 71644 71644 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 635989 635989 0 0.0
RAM 74188 74188 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 635989 635989 0 0.0
RAM 74188 74188 0 0.0
lock CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 635817 635817 0 0.0
RAM 74692 74692 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 655525 655525 0 0.0
RAM 77236 77236 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 655525 655525 0 0.0
RAM 77236 77236 0 0.0
thermostat CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 611213 611213 0 0.0
RAM 68772 68772 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 631065 631065 0 0.0
RAM 71412 71412 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 631065 631065 0 0.0
RAM 71412 71412 0 0.0
efr32 lock-app BRD4187C FLASH 927628 927628 0 0.0
RAM 160160 160160 0 0.0
BRD4338a FLASH 743696 743696 0 0.0
RAM 233288 233288 0 0.0
window-app BRD4187C FLASH 1018816 1018808 -8 -0.0
RAM 128260 128260 0 0.0
esp32 all-clusters-app c3devkit DRAM 95328 95328 0 0.0
FLASH 1540466 1540466 0 0.0
IRAM 82542 82542 0 0.0
m5stack DRAM 116256 116256 0 0.0
FLASH 1549282 1549282 0 0.0
IRAM 117039 117039 0 0.0
linux air-purifier-app debug unknown 4712 4712 0 0.0
FLASH 2698879 2698879 0 0.0
RAM 129648 129648 0 0.0
all-clusters-app debug unknown 5552 5552 0 0.0
FLASH 5992608 5992608 0 0.0
RAM 523968 523968 0 0.0
all-clusters-minimal-app debug unknown 5448 5448 0 0.0
FLASH 5328216 5328216 0 0.0
RAM 242480 242480 0 0.0
bridge-app debug unknown 5432 5432 0 0.0
FLASH 4670954 4670954 0 0.0
RAM 218416 218416 0 0.0
chip-tool debug unknown 5984 5984 0 0.0
FLASH 12872194 12872194 0 0.0
RAM 584466 584466 0 0.0
chip-tool-ipv6only arm64 unknown 21400 21400 0 0.0
FLASH 11002704 11002704 0 0.0
RAM 635368 635368 0 0.0
fabric-admin debug unknown 5808 5808 0 0.0
FLASH 11277239 11277239 0 0.0
RAM 584826 584826 0 0.0
fabric-bridge-app debug unknown 4688 4688 0 0.0
FLASH 4495222 4495222 0 0.0
RAM 205400 205400 0 0.0
fabric-sync debug unknown 4880 4880 0 0.0
FLASH 5309077 5450549 141472 2.7
RAM 466872 470824 3952 0.8
lighting-app debug+rpc+ui unknown 6096 6096 0 0.0
FLASH 5606881 5606881 0 0.0
RAM 228552 228552 0 0.0
lock-app debug unknown 5368 5368 0 0.0
FLASH 4720428 4720428 0 0.0
RAM 204552 204552 0 0.0
ota-provider-app debug unknown 4744 4744 0 0.0
FLASH 4345860 4345860 0 0.0
RAM 198272 198272 0 0.0
ota-requestor-app debug unknown 4680 4680 0 0.0
FLASH 4484262 4484262 0 0.0
RAM 202840 202840 0 0.0
shell debug unknown 4240 4240 0 0.0
FLASH 3011293 3011293 0 0.0
RAM 160368 160368 0 0.0
thermostat-no-ble arm64 unknown 9440 9440 0 0.0
FLASH 4089952 4089952 0 0.0
RAM 242936 242936 0 0.0
tv-app debug unknown 5648 5648 0 0.0
FLASH 5942565 5942565 0 0.0
RAM 596336 596336 0 0.0
tv-casting-app debug unknown 5232 5232 0 0.0
FLASH 11079389 11079389 0 0.0
RAM 694536 694536 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 915372 915372 0 0.0
RAM 143281 143281 0 0.0
nrf7002dk_nrf5340_cpuapp FLASH 887588 887588 0 0.0
RAM 141468 141468 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 847628 847628 0 0.0
RAM 142189 142189 0 0.0
nxp contact k32w0+release FLASH 584144 584144 0 0.0
RAM 71044 71044 0 0.0
mcxw71+release FLASH 598696 598696 0 0.0
RAM 63144 63144 0 0.0
light k32w0+release FLASH 611220 611220 0 0.0
RAM 70436 70436 0 0.0
k32w1+release FLASH 685016 685016 0 0.0
RAM 48776 48776 0 0.0
lock mcxw71+release FLASH 748544 748544 0 0.0
RAM 67300 67300 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 FLASH 1642980 1642980 0 0.0
RAM 212056 212056 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 FLASH 1550180 1550180 0 0.0
RAM 208856 208856 0 0.0
light cy8ckit_062s2_43012 FLASH 1466620 1466620 0 0.0
RAM 200840 200840 0 0.0
lock cy8ckit_062s2_43012 FLASH 1464796 1464796 0 0.0
RAM 225200 225200 0 0.0
qpg lighting-app qpg6105+debug FLASH 662184 662184 0 0.0
RAM 105380 105380 0 0.0
lock-app qpg6105+debug FLASH 620284 620284 0 0.0
RAM 99832 99832 0 0.0
stm32 light STM32WB5MM-DK FLASH 482984 482984 0 0.0
RAM 144844 144844 0 0.0
telink bridge-app tlsr9258a FLASH 684684 684684 0 0.0
RAM 91528 91528 0 0.0
contact-sensor-app tlsr9528a_retention FLASH 622696 622696 0 0.0
RAM 50464 50464 0 0.0
light-switch-app-ota-compress-lzma-shell-factory-data tlsr9528a FLASH 710622 710622 0 0.0
RAM 73804 73804 0 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 628990 628990 0 0.0
RAM 145384 145384 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 814878 814878 0 0.0
RAM 100004 100004 0 0.0
tizen all-clusters-app arm unknown 4948 4948 0 0.0
FLASH 1722580 1722580 0 0.0
RAM 90656 90656 0 0.0
chip-tool-ubsan arm unknown 10832 10832 0 0.0
FLASH 18007550 18007550 0 0.0
RAM 7857676 7857676 0 0.0

Copy link
Contributor

@tehampson tehampson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still have to review two more files, examples/fabric-sync/admin/DeviceManager.cpp and it's header. But might not get around to it until this evening, so just giving the comments I have thus far.

Mainly just nits

examples/fabric-sync/admin/DeviceSubscription.cpp Outdated Show resolved Hide resolved
examples/fabric-sync/admin/DeviceSynchronization.cpp Outdated Show resolved Hide resolved
examples/fabric-sync/bridge/include/Bridge.h Outdated Show resolved Hide resolved
examples/fabric-sync/bridge/include/FabricAdminDelegate.h Outdated Show resolved Hide resolved
examples/fabric-sync/bridge/include/FabricBridge.h Outdated Show resolved Hide resolved
examples/fabric-sync/bridge/src/Bridge.cpp Outdated Show resolved Hide resolved
Copy link

github-actions bot commented Nov 12, 2024

PR #36427: Size comparison from 18c0788 to 5da2dda

Increases above 0.2%:

platform target config section 18c0788 5da2dda change % change
linux fabric-sync debug unknown 4880 4896 16 0.3
FLASH 5309077 5451317 142240 2.7
RAM 466872 470424 3552 0.8
thermostat-no-ble arm64 unknown 9440 9464 24 0.3
tv-app debug unknown 5648 5664 16 0.3
Full report (68 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
platform target config section 18c0788 5da2dda change % change
bl602 lighting-app bl602+mfd+littlefs+rpc FLASH 1349426 1349448 22 0.0
RAM 104080 104080 0 0.0
bl702 lighting-app bl702+eth FLASH 649386 649346 -40 -0.0
RAM 25321 25313 -8 -0.0
bl702+wifi FLASH 826966 826926 -40 -0.0
RAM 14053 14053 0 0.0
bl706+mfd+rpc+littlefs FLASH 1055164 1055124 -40 -0.0
RAM 23893 23893 0 0.0
bl702l lighting-app bl702l+mfd+littlefs FLASH 976530 976486 -44 -0.0
RAM 16556 16556 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 837876 837852 -24 -0.0
RAM 123628 123632 4 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 823792 823768 -24 -0.0
RAM 125516 125520 4 0.0
pump-app LP_EM_CC1354P10_6 FLASH 770532 770492 -40 -0.0
RAM 113984 113988 4 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 754736 754704 -32 -0.0
RAM 114192 114196 4 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 625926 625930 4 0.0
RAM 205780 205784 4 0.0
lock CC3235SF_LAUNCHXL FLASH 667238 667242 4 0.0
RAM 205932 205936 4 0.0
cyw30739 light CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 679513 679489 -24 -0.0
RAM 78684 78692 8 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 699357 699341 -16 -0.0
RAM 81324 81324 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 699357 699341 -16 -0.0
RAM 81324 81324 0 0.0
CYW930739M2EVB-02 unknown 2040 2040 0 0.0
FLASH 656301 656277 -24 -0.0
RAM 73752 73760 8 0.0
light-switch CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 616361 616345 -16 -0.0
RAM 71644 71644 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 635989 635973 -16 -0.0
RAM 74188 74196 8 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 635989 635973 -16 -0.0
RAM 74188 74196 8 0.0
lock CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 635817 635801 -16 -0.0
RAM 74692 74692 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 655525 655517 -8 -0.0
RAM 77236 77244 8 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 655525 655517 -8 -0.0
RAM 77236 77244 8 0.0
thermostat CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 611213 611197 -16 -0.0
RAM 68772 68780 8 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 631065 631049 -16 -0.0
RAM 71412 71412 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 631065 631049 -16 -0.0
RAM 71412 71412 0 0.0
efr32 lock-app BRD4187C FLASH 927628 927596 -32 -0.0
RAM 160160 160164 4 0.0
BRD4338a FLASH 743696 743712 16 0.0
RAM 233288 233296 8 0.0
window-app BRD4187C FLASH 1018816 1018808 -8 -0.0
RAM 128260 128264 4 0.0
esp32 all-clusters-app c3devkit DRAM 95328 95336 8 0.0
FLASH 1540466 1540460 -6 -0.0
IRAM 82542 82542 0 0.0
m5stack DRAM 116256 116264 8 0.0
FLASH 1549282 1549294 12 0.0
IRAM 117039 117039 0 0.0
linux air-purifier-app debug unknown 4712 4712 0 0.0
FLASH 2698879 2699169 290 0.0
RAM 129648 129696 48 0.0
all-clusters-app debug unknown 5552 5552 0 0.0
FLASH 5992608 5992706 98 0.0
RAM 523968 523552 -416 -0.1
all-clusters-minimal-app debug unknown 5448 5448 0 0.0
FLASH 5328216 5328474 258 0.0
RAM 242480 242512 32 0.0
bridge-app debug unknown 5432 5432 0 0.0
FLASH 4670954 4671244 290 0.0
RAM 218416 218432 16 0.0
chip-tool debug unknown 5984 5984 0 0.0
FLASH 12872194 12854058 -18136 -0.1
RAM 584466 583122 -1344 -0.2
chip-tool-ipv6only arm64 unknown 21400 21336 -64 -0.3
FLASH 11002704 10989008 -13696 -0.1
RAM 635368 634024 -1344 -0.2
fabric-admin debug unknown 5808 5808 0 0.0
FLASH 11277239 11260215 -17024 -0.2
RAM 584826 583474 -1352 -0.2
fabric-bridge-app debug unknown 4688 4688 0 0.0
FLASH 4495222 4495514 292 0.0
RAM 205400 205408 8 0.0
fabric-sync debug unknown 4880 4896 16 0.3
FLASH 5309077 5451317 142240 2.7
RAM 466872 470424 3552 0.8
lighting-app debug+rpc+ui unknown 6096 6096 0 0.0
FLASH 5606881 5607169 288 0.0
RAM 228552 228600 48 0.0
lock-app debug unknown 5368 5368 0 0.0
FLASH 4720428 4720718 290 0.0
RAM 204552 204600 48 0.0
ota-provider-app debug unknown 4744 4744 0 0.0
FLASH 4345860 4346150 290 0.0
RAM 198272 198272 0 0.0
ota-requestor-app debug unknown 4680 4680 0 0.0
FLASH 4484262 4484552 290 0.0
RAM 202840 202856 16 0.0
shell debug unknown 4240 4240 0 0.0
FLASH 3011293 3011565 272 0.0
RAM 160368 160344 -24 -0.0
thermostat-no-ble arm64 unknown 9440 9464 24 0.3
FLASH 4089952 4090256 304 0.0
RAM 242936 242984 48 0.0
tv-app debug unknown 5648 5664 16 0.3
FLASH 5942565 5943781 1216 0.0
RAM 596336 595920 -416 -0.1
tv-casting-app debug unknown 5232 5232 0 0.0
FLASH 11079389 11060765 -18624 -0.2
RAM 694536 693032 -1504 -0.2
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 915372 915332 -40 -0.0
RAM 143281 143289 8 0.0
nrf7002dk_nrf5340_cpuapp FLASH 887588 887596 8 0.0
RAM 141468 141476 8 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 847628 847588 -40 -0.0
RAM 142189 142197 8 0.0
nxp contact k32w0+release FLASH 584144 584128 -16 -0.0
RAM 71044 71048 4 0.0
mcxw71+release FLASH 598696 598664 -32 -0.0
RAM 63144 63144 0 0.0
light k32w0+release FLASH 611220 611172 -48 -0.0
RAM 70436 70440 4 0.0
k32w1+release FLASH 685016 684976 -40 -0.0
RAM 48776 48776 0 0.0
lock mcxw71+release FLASH 748544 748512 -32 -0.0
RAM 67300 67300 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 FLASH 1642980 1642980 0 0.0
RAM 212056 212064 8 0.0
all-clusters-minimal cy8ckit_062s2_43012 FLASH 1550180 1550196 16 0.0
RAM 208856 208864 8 0.0
light cy8ckit_062s2_43012 FLASH 1466620 1466620 0 0.0
RAM 200840 200848 8 0.0
lock cy8ckit_062s2_43012 FLASH 1464796 1464796 0 0.0
RAM 225200 225208 8 0.0
qpg lighting-app qpg6105+debug FLASH 662184 662144 -40 -0.0
RAM 105380 105384 4 0.0
lock-app qpg6105+debug FLASH 620284 620244 -40 -0.0
RAM 99832 99836 4 0.0
stm32 light STM32WB5MM-DK FLASH 482984 482944 -40 -0.0
RAM 144844 144848 4 0.0
telink bridge-app tlsr9258a FLASH 684684 684686 2 0.0
RAM 91528 91536 8 0.0
contact-sensor-app tlsr9528a_retention FLASH 622696 622694 -2 -0.0
RAM 50464 50472 8 0.0
light-switch-app-ota-compress-lzma-shell-factory-data tlsr9528a FLASH 710622 710624 2 0.0
RAM 73804 73812 8 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 628990 628992 2 0.0
RAM 145384 145392 8 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 814878 814880 2 0.0
RAM 100004 100012 8 0.0
tizen all-clusters-app arm unknown 4948 4956 8 0.2
FLASH 1722580 1722680 100 0.0
RAM 90656 90660 4 0.0
chip-tool-ubsan arm unknown 10832 10800 -32 -0.3
FLASH 18007550 17979646 -27904 -0.2
RAM 7857676 7844172 -13504 -0.2

Copy link

github-actions bot commented Nov 12, 2024

PR #36427: Size comparison from 18c0788 to 641726e

Increases above 0.2%:

platform target config section 18c0788 641726e change % change
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 625926 627498 1572 0.3
Full report (14 builds for cc13x4_26x4, cc32xx, nrfconnect, qpg, stm32, tizen)
platform target config section 18c0788 641726e change % change
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 837876 837852 -24 -0.0
RAM 123628 123632 4 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 823792 823768 -24 -0.0
RAM 125516 125520 4 0.0
pump-app LP_EM_CC1354P10_6 FLASH 770532 770492 -40 -0.0
RAM 113984 113988 4 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 754736 754704 -32 -0.0
RAM 114192 114196 4 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 625926 627498 1572 0.3
RAM 205780 205784 4 0.0
lock CC3235SF_LAUNCHXL FLASH 667238 667242 4 0.0
RAM 205932 205936 4 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 915372 914884 -488 -0.1
RAM 143281 143289 8 0.0
nrf7002dk_nrf5340_cpuapp FLASH 887588 887340 -248 -0.0
RAM 141468 141476 8 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 847628 849080 1452 0.2
RAM 142189 142197 8 0.0
qpg lighting-app qpg6105+debug FLASH 662184 662144 -40 -0.0
RAM 105380 105384 4 0.0
lock-app qpg6105+debug FLASH 620284 620244 -40 -0.0
RAM 99832 99836 4 0.0
stm32 light STM32WB5MM-DK FLASH 482984 482944 -40 -0.0
RAM 144844 144848 4 0.0
tizen all-clusters-app arm unknown 4948 4952 4 0.1
FLASH 1722580 1721608 -972 -0.1
RAM 90656 90628 -28 -0.0
chip-tool-ubsan arm unknown 10832 10800 -32 -0.3
FLASH 18007550 17979646 -27904 -0.2
RAM 7857676 7844172 -13504 -0.2

Copy link

github-actions bot commented Nov 12, 2024

PR #36427: Size comparison from 18c0788 to 73fb325

Increases above 0.2%:

platform target config section 18c0788 73fb325 change % change
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 625926 627498 1572 0.3
linux fabric-sync debug unknown 4880 4896 16 0.3
FLASH 5309077 5455621 146544 2.8
RAM 466872 470520 3648 0.8
thermostat-no-ble arm64 unknown 9440 9464 24 0.3
tv-app debug unknown 5648 5664 16 0.3
Full report (68 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
platform target config section 18c0788 73fb325 change % change
bl602 lighting-app bl602+mfd+littlefs+rpc FLASH 1349426 1349448 22 0.0
RAM 104080 104080 0 0.0
bl702 lighting-app bl702+eth FLASH 649386 649346 -40 -0.0
RAM 25321 25313 -8 -0.0
bl702+wifi FLASH 826966 826926 -40 -0.0
RAM 14053 14053 0 0.0
bl706+mfd+rpc+littlefs FLASH 1055164 1055124 -40 -0.0
RAM 23893 23893 0 0.0
bl702l lighting-app bl702l+mfd+littlefs FLASH 976530 976486 -44 -0.0
RAM 16556 16556 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 837876 837852 -24 -0.0
RAM 123628 123632 4 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 823792 823768 -24 -0.0
RAM 125516 125520 4 0.0
pump-app LP_EM_CC1354P10_6 FLASH 770532 770492 -40 -0.0
RAM 113984 113988 4 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 754736 754704 -32 -0.0
RAM 114192 114196 4 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 625926 627498 1572 0.3
RAM 205780 205784 4 0.0
lock CC3235SF_LAUNCHXL FLASH 667238 667242 4 0.0
RAM 205932 205936 4 0.0
cyw30739 light CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 679513 679489 -24 -0.0
RAM 78684 78692 8 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 699357 699341 -16 -0.0
RAM 81324 81324 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 699357 699341 -16 -0.0
RAM 81324 81324 0 0.0
CYW930739M2EVB-02 unknown 2040 2040 0 0.0
FLASH 656301 656277 -24 -0.0
RAM 73752 73760 8 0.0
light-switch CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 616361 616345 -16 -0.0
RAM 71644 71644 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 635989 635973 -16 -0.0
RAM 74188 74196 8 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 635989 635973 -16 -0.0
RAM 74188 74196 8 0.0
lock CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 635817 635801 -16 -0.0
RAM 74692 74692 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 655525 655517 -8 -0.0
RAM 77236 77244 8 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 655525 655517 -8 -0.0
RAM 77236 77244 8 0.0
thermostat CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 611213 611197 -16 -0.0
RAM 68772 68780 8 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 631065 631049 -16 -0.0
RAM 71412 71412 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 631065 631049 -16 -0.0
RAM 71412 71412 0 0.0
efr32 lock-app BRD4187C FLASH 927628 927596 -32 -0.0
RAM 160160 160164 4 0.0
BRD4338a FLASH 743696 743712 16 0.0
RAM 233288 233296 8 0.0
window-app BRD4187C FLASH 1018816 1018808 -8 -0.0
RAM 128260 128264 4 0.0
esp32 all-clusters-app c3devkit DRAM 95328 95336 8 0.0
FLASH 1540466 1540014 -452 -0.0
IRAM 82542 82542 0 0.0
m5stack DRAM 116256 116264 8 0.0
FLASH 1549282 1548882 -400 -0.0
IRAM 117039 117039 0 0.0
linux air-purifier-app debug unknown 4712 4712 0 0.0
FLASH 2698879 2699379 500 0.0
RAM 129648 129696 48 0.0
all-clusters-app debug unknown 5552 5552 0 0.0
FLASH 5992608 5990598 -2010 -0.0
RAM 523968 523488 -480 -0.1
all-clusters-minimal-app debug unknown 5448 5448 0 0.0
FLASH 5328216 5328802 586 0.0
RAM 242480 242512 32 0.0
bridge-app debug unknown 5432 5432 0 0.0
FLASH 4670954 4671244 290 0.0
RAM 218416 218432 16 0.0
chip-tool debug unknown 5984 5984 0 0.0
FLASH 12872194 12854058 -18136 -0.1
RAM 584466 583122 -1344 -0.2
chip-tool-ipv6only arm64 unknown 21400 21336 -64 -0.3
FLASH 11002704 10989008 -13696 -0.1
RAM 635368 634024 -1344 -0.2
fabric-admin debug unknown 5808 5808 0 0.0
FLASH 11277239 11260215 -17024 -0.2
RAM 584826 583474 -1352 -0.2
fabric-bridge-app debug unknown 4688 4688 0 0.0
FLASH 4495222 4495514 292 0.0
RAM 205400 205408 8 0.0
fabric-sync debug unknown 4880 4896 16 0.3
FLASH 5309077 5455621 146544 2.8
RAM 466872 470520 3648 0.8
lighting-app debug+rpc+ui unknown 6096 6096 0 0.0
FLASH 5606881 5607169 288 0.0
RAM 228552 228600 48 0.0
lock-app debug unknown 5368 5368 0 0.0
FLASH 4720428 4720718 290 0.0
RAM 204552 204600 48 0.0
ota-provider-app debug unknown 4744 4744 0 0.0
FLASH 4345860 4346150 290 0.0
RAM 198272 198272 0 0.0
ota-requestor-app debug unknown 4680 4680 0 0.0
FLASH 4484262 4484552 290 0.0
RAM 202840 202856 16 0.0
shell debug unknown 4240 4240 0 0.0
FLASH 3011293 3013837 2544 0.1
RAM 160368 160344 -24 -0.0
thermostat-no-ble arm64 unknown 9440 9464 24 0.3
FLASH 4089952 4090256 304 0.0
RAM 242936 242984 48 0.0
tv-app debug unknown 5648 5664 16 0.3
FLASH 5942565 5943781 1216 0.0
RAM 596336 595920 -416 -0.1
tv-casting-app debug unknown 5232 5232 0 0.0
FLASH 11079389 11060765 -18624 -0.2
RAM 694536 693032 -1504 -0.2
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 915372 914884 -488 -0.1
RAM 143281 143289 8 0.0
nrf7002dk_nrf5340_cpuapp FLASH 887588 887340 -248 -0.0
RAM 141468 141476 8 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 847628 849080 1452 0.2
RAM 142189 142197 8 0.0
nxp contact k32w0+release FLASH 584144 584128 -16 -0.0
RAM 71044 71048 4 0.0
mcxw71+release FLASH 598696 598664 -32 -0.0
RAM 63144 63144 0 0.0
light k32w0+release FLASH 611220 611172 -48 -0.0
RAM 70436 70440 4 0.0
k32w1+release FLASH 685016 684976 -40 -0.0
RAM 48776 48776 0 0.0
lock mcxw71+release FLASH 748544 748512 -32 -0.0
RAM 67300 67300 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 FLASH 1642980 1642476 -504 -0.0
RAM 212056 212064 8 0.0
all-clusters-minimal cy8ckit_062s2_43012 FLASH 1550180 1550468 288 0.0
RAM 208856 208864 8 0.0
light cy8ckit_062s2_43012 FLASH 1466620 1466620 0 0.0
RAM 200840 200848 8 0.0
lock cy8ckit_062s2_43012 FLASH 1464796 1464796 0 0.0
RAM 225200 225208 8 0.0
qpg lighting-app qpg6105+debug FLASH 662184 662144 -40 -0.0
RAM 105380 105384 4 0.0
lock-app qpg6105+debug FLASH 620284 620244 -40 -0.0
RAM 99832 99836 4 0.0
stm32 light STM32WB5MM-DK FLASH 482984 482944 -40 -0.0
RAM 144844 144848 4 0.0
telink bridge-app tlsr9258a FLASH 684684 684686 2 0.0
RAM 91528 91536 8 0.0
contact-sensor-app tlsr9528a_retention FLASH 622696 622694 -2 -0.0
RAM 50464 50472 8 0.0
light-switch-app-ota-compress-lzma-shell-factory-data tlsr9528a FLASH 710622 710624 2 0.0
RAM 73804 73812 8 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 628990 628992 2 0.0
RAM 145384 145392 8 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 814878 814880 2 0.0
RAM 100004 100012 8 0.0
tizen all-clusters-app arm unknown 4948 4952 4 0.1
FLASH 1722580 1721608 -972 -0.1
RAM 90656 90628 -28 -0.0
chip-tool-ubsan arm unknown 10832 10800 -32 -0.3
FLASH 18007550 17979646 -27904 -0.2
RAM 7857676 7844172 -13504 -0.2

@mergify mergify bot merged commit dac4a53 into project-chip:master Nov 14, 2024
67 checks passed
@yufengwangca yufengwangca deleted the pr/fs/cctrl branch November 14, 2024 02:45
yyzhong-g pushed a commit to yyzhong-g/connectedhomeip that referenced this pull request Dec 12, 2024
…ect-chip#36427)

* [Fabric-Sync] Port commissioner control protocol

* Add device synchronization

* Fix compile error

* Update examples/fabric-sync/admin/DeviceSubscription.cpp

Co-authored-by: Terence Hampson <[email protected]>

* Address review comments

* Add debug log

* Update examples/fabric-sync/admin/DeviceManager.cpp

Co-authored-by: Terence Hampson <[email protected]>

* Address review comments

* Add endpoint check

---------

Co-authored-by: Terence Hampson <[email protected]>
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.

3 participants