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

Build time optimization #37433

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Build time optimization #37433

wants to merge 6 commits into from

Conversation

adigie
Copy link
Contributor

@adigie adigie commented Feb 6, 2025 •
edited
Loading

Changes

  • Remove dependency to cluster-objects.h in Accessors.h
  • Remove cluster-objects.cpp from chip_data_model.cmake

This should improve build times.

Testing

Tested by building nrfconnect lock-app example. Build time improved by 5.69%.

Before After
Total 34.95s 32.96s
Accessors.cpp 6.47s 5.18s
cluster-objects.cpp 7.40s 0.00s
door-lock-server-callback.cpp 3.24s 2.69s
ZclCallbacks.cpp 2.94s 2.18s

cluster-objects.cpp is now compiled only once

Traces generated with ninjatracing:

@semanticdiff-com SemanticDiff.com
Copy link

Review changes with  SemanticDiff

@github-actions github-actions bot added the app label Feb 6, 2025
@adigie adigie force-pushed the accessors-optimization branch from afed4dd to ed2e356 Compare February 6, 2025 18:41
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 6, 2025 •
edited
Loading

PR #37433: Size comparison from 35781e5 to ed2e356

Full report (52 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, esp32, linux, nrfconnect, nxp, qpg, stm32, telink, tizen)
platform target config section 35781e5 ed2e356 change % change
bl602 lighting-app bl602+mfd+littlefs+rpc FLASH 1094024 1094024 0 0.0
RAM 103314 103314 0 0.0
bl702 lighting-app bl702+eth FLASH 650542 650542 0 0.0
RAM 25265 25265 0 0.0
bl702+wifi FLASH 828506 828506 0 0.0
RAM 13981 13981 0 0.0
bl706+mfd+rpc+littlefs FLASH 1057066 1057066 0 0.0
RAM 23861 23861 0 0.0
bl702l contact-sensor-app bl702l+mfd+littlefs FLASH 888444 888444 0 0.0
RAM 18504 18504 0 0.0
lighting-app bl702l+mfd+littlefs FLASH 971486 971486 0 0.0
RAM 16368 16368 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 838240 838240 0 0.0
RAM 123480 123480 0 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 823644 823644 0 0.0
RAM 125360 125360 0 0.0
pump-app LP_EM_CC1354P10_6 FLASH 770820 770820 0 0.0
RAM 113820 113820 0 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 755088 755088 0 0.0
RAM 114028 114028 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 538621 538621 0 0.0
RAM 205208 205208 0 0.0
lock CC3235SF_LAUNCHXL FLASH 572493 572493 0 0.0
RAM 205360 205360 0 0.0
esp32 all-clusters-app c3devkit DRAM 97312 97312 0 0.0
FLASH 1577420 1577420 0 0.0
IRAM 83820 83820 0 0.0
m5stack DRAM 116100 116100 0 0.0
FLASH 1545326 1545326 0 0.0
IRAM 117039 117039 0 0.0
linux air-purifier-app debug unknown 4760 4760 0 0.0
FLASH 2709171 2709169 -2 -0.0
RAM 132784 132784 0 0.0
all-clusters-app debug unknown 5568 5568 0 0.0
FLASH 5975584 5975552 -32 -0.0
RAM 531600 531600 0 0.0
all-clusters-minimal-app debug unknown 5464 5464 0 0.0
FLASH 5323948 5323948 0 0.0
RAM 242712 242712 0 0.0
bridge-app debug unknown 5480 5480 0 0.0
FLASH 4682560 4682526 -34 -0.0
RAM 221448 221448 0 0.0
chip-tool debug unknown 6120 6120 0 0.0
FLASH 13099268 13099268 0 0.0
RAM 596578 596578 0 0.0
chip-tool-ipv6only arm64 unknown 21816 21816 0 0.0
FLASH 11162800 11162800 0 0.0
RAM 648256 648256 0 0.0
fabric-admin debug unknown 5808 5808 0 0.0
FLASH 11388297 11388297 0 0.0
RAM 596362 596362 0 0.0
fabric-bridge-app debug unknown 4736 4736 0 0.0
FLASH 4507818 4507816 -2 -0.0
RAM 208632 208632 0 0.0
fabric-sync debug unknown 4976 4976 0 0.0
FLASH 5614085 5614053 -32 -0.0
RAM 483504 483504 0 0.0
lighting-app debug+rpc+ui unknown 6144 6144 0 0.0
FLASH 5625217 5625217 0 0.0
RAM 231728 231728 0 0.0
lock-app debug unknown 5416 5416 0 0.0
FLASH 4731186 4731184 -2 -0.0
RAM 207696 207696 0 0.0
ota-provider-app debug unknown 4776 4776 0 0.0
FLASH 4360136 4360134 -2 -0.0
RAM 201336 201336 0 0.0
ota-requestor-app debug unknown 4728 4728 0 0.0
FLASH 4497640 4497638 -2 -0.0
RAM 205920 205920 0 0.0
shell debug unknown 4256 4256 0 0.0
FLASH 3005709 3005709 0 0.0
RAM 160472 160472 0 0.0
thermostat-no-ble arm64 unknown 9512 9512 0 0.0
FLASH 4097192 4097192 0 0.0
RAM 246024 246024 0 0.0
tv-app debug unknown 5744 5744 0 0.0
FLASH 5952965 5952933 -32 -0.0
RAM 606904 606904 0 0.0
tv-casting-app debug unknown 5320 5320 0 0.0
FLASH 11272797 11272765 -32 -0.0
RAM 710864 710864 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 907432 907432 0 0.0
RAM 142411 142411 0 0.0
nrf7002dk_nrf5340_cpuapp FLASH 901980 901980 0 0.0
RAM 124755 124755 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 846040 846040 0 0.0
RAM 141339 141339 0 0.0
nxp contact k32w0+release FLASH 584624 584624 0 0.0
RAM 70876 70876 0 0.0
mcxw71+release FLASH 599936 599936 0 0.0
RAM 63096 63096 0 0.0
light k32w0+release FLASH 610932 610932 0 0.0
RAM 70268 70268 0 0.0
k32w1+release FLASH 685336 685336 0 0.0
RAM 48680 48680 0 0.0
lock mcxw71+release FLASH 748832 748832 0 0.0
RAM 67492 67492 0 0.0
qpg lighting-app qpg6105+debug FLASH 662216 662216 0 0.0
RAM 105220 105220 0 0.0
lock-app qpg6105+debug FLASH 619996 619996 0 0.0
RAM 99664 99664 0 0.0
stm32 light STM32WB5MM-DK FLASH 482872 482872 0 0.0
RAM 144688 144688 0 0.0
telink bridge-app tl7218x FLASH 665258 665258 0 0.0
RAM 90828 90828 0 0.0
contact-sensor-app tlsr9528a_retention FLASH 622020 622020 0 0.0
RAM 31500 31500 0 0.0
light-app-ota-shell-factory-data tl3218x FLASH 770804 770804 0 0.0
RAM 43568 43568 0 0.0
tl7218x FLASH 779306 779306 0 0.0
RAM 98704 98704 0 0.0
light-switch-app-ota-compress-lzma-factory-data tl7218x_retention FLASH 680618 680618 0 0.0
RAM 52192 52192 0 0.0
light-switch-app-ota-compress-lzma-shell-factory-data tlsr9528a FLASH 709172 709172 0 0.0
RAM 73400 73400 0 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 626108 626108 0 0.0
RAM 142032 142032 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 813492 813492 0 0.0
RAM 99576 99576 0 0.0
tizen all-clusters-app arm unknown 5104 5104 0 0.0
FLASH 1752232 1752232 0 0.0
RAM 93508 93508 0 0.0
chip-tool-ubsan arm unknown 11396 11396 0 0.0
FLASH 18696694 18696726 32 0.0
RAM 8182536 8182536 0 0.0

@adigie adigie force-pushed the accessors-optimization branch from ed2e356 to 065a90a Compare February 6, 2025 20:04
@adigie adigie force-pushed the accessors-optimization branch from 065a90a to c8e1550 Compare February 6, 2025 20:05
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 6, 2025 •
edited
Loading

PR #37433: Size comparison from 2cf6dd8 to c8e1550

Full report (36 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, efr32, nrfconnect, nxp, qpg, stm32, telink, tizen)
platform target config section 2cf6dd8 c8e1550 change % change
bl602 lighting-app bl602+mfd+littlefs+rpc FLASH 1094024 1094024 0 0.0
RAM 103314 103314 0 0.0
bl702 lighting-app bl702+eth FLASH 650542 650542 0 0.0
RAM 25265 25265 0 0.0
bl702+wifi FLASH 828506 828506 0 0.0
RAM 13981 13981 0 0.0
bl706+mfd+rpc+littlefs FLASH 1057066 1057066 0 0.0
RAM 23861 23861 0 0.0
bl702l contact-sensor-app bl702l+mfd+littlefs FLASH 888444 888444 0 0.0
RAM 18504 18504 0 0.0
lighting-app bl702l+mfd+littlefs FLASH 971486 971486 0 0.0
RAM 16368 16368 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 838240 838240 0 0.0
RAM 123480 123480 0 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 823644 823644 0 0.0
RAM 125360 125360 0 0.0
pump-app LP_EM_CC1354P10_6 FLASH 770820 770820 0 0.0
RAM 113820 113820 0 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 755088 755088 0 0.0
RAM 114028 114028 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 538621 538621 0 0.0
RAM 205208 205208 0 0.0
lock CC3235SF_LAUNCHXL FLASH 572493 572493 0 0.0
RAM 205360 205360 0 0.0
efr32 lock-app BRD4187C FLASH 936408 936408 0 0.0
RAM 159872 159872 0 0.0
BRD4338a FLASH 730348 730348 0 0.0
RAM 234716 234716 0 0.0
window-app BRD4187C FLASH 1029656 1029656 0 0.0
RAM 128008 128008 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 907432 907432 0 0.0
RAM 142411 142411 0 0.0
nrf7002dk_nrf5340_cpuapp FLASH 901980 901980 0 0.0
RAM 124755 124755 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 846040 846040 0 0.0
RAM 141339 141339 0 0.0
nxp contact k32w0+release FLASH 584624 584624 0 0.0
RAM 70876 70876 0 0.0
mcxw71+release FLASH 599936 599936 0 0.0
RAM 63096 63096 0 0.0
light k32w0+release FLASH 610932 610932 0 0.0
RAM 70268 70268 0 0.0
k32w1+release FLASH 685336 685336 0 0.0
RAM 48680 48680 0 0.0
lock mcxw71+release FLASH 748832 748832 0 0.0
RAM 67492 67492 0 0.0
qpg lighting-app qpg6105+debug FLASH 662216 662216 0 0.0
RAM 105220 105220 0 0.0
lock-app qpg6105+debug FLASH 619996 619996 0 0.0
RAM 99664 99664 0 0.0
stm32 light STM32WB5MM-DK FLASH 482872 482872 0 0.0
RAM 144688 144688 0 0.0
telink bridge-app tl7218x FLASH 665258 665258 0 0.0
RAM 90828 90828 0 0.0
contact-sensor-app tlsr9528a_retention FLASH 622020 622020 0 0.0
RAM 31500 31500 0 0.0
light-app-ota-shell-factory-data tl3218x FLASH 770804 770804 0 0.0
RAM 43568 43568 0 0.0
tl7218x FLASH 779306 779306 0 0.0
RAM 98704 98704 0 0.0
light-switch-app-ota-compress-lzma-factory-data tl7218x_retention FLASH 680618 680618 0 0.0
RAM 52192 52192 0 0.0
light-switch-app-ota-compress-lzma-shell-factory-data tlsr9528a FLASH 709172 709172 0 0.0
RAM 73400 73400 0 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 626108 626108 0 0.0
RAM 142032 142032 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 813492 813492 0 0.0
RAM 99576 99576 0 0.0
tizen all-clusters-app arm unknown 5104 5104 0 0.0
FLASH 1752232 1752232 0 0.0
RAM 93508 93508 0 0.0
chip-tool-ubsan arm unknown 11396 11396 0 0.0
FLASH 18696694 18696726 32 0.0
RAM 8182536 8182536 0 0.0

@adigie adigie force-pushed the accessors-optimization branch from c8e1550 to 49eb455 Compare February 6, 2025 21:12
adigie added 6 commits February 6, 2025 22:12
@adigie
Move some classes out of af-types.h
514805a 
Move AttributesChangedListener and MarkAttributeDirty out of af-types.h

Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie
Remove dependency to cluster-objects.h
604c9ba 
Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie
Remove cluster-objects from chip_data_model.cmake
6eb10fd 
Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie
[Chef] Fix includes
503179b 
Fix missing includes after changes in Accessors.h

Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie
[Infineon] Fix includes
cabae47 
Fix missing includes after changes in Accessors.h

Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie
[Silabs] Fix includes
bbccf31 
Fix missing includes after changes in Accessors.h

Signed-off-by: Adrian Gielniewski <[email protected]>
@adigie adigie force-pushed the accessors-optimization branch from 49eb455 to bbccf31 Compare February 6, 2025 21:12
@adigie adigie marked this pull request as ready for review February 6, 2025 21:14
@adigie adigie requested a review from a team as a code owner February 6, 2025 21:14
@andy31415
Copy link
Contributor

@adigie could you share build time deltas in the PR summary?

andy31415
andy31415 requested changes Feb 7, 2025
View reviewed changes
Copy link
Contributor

@andy31415 andy31415 left a comment

Choose a reason for hiding this comment

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

The Testing section should contain information how we tested that this actually does what it claims (improve build times). Things still compiling is ok and will be checked by CI. However we also claim this makes builds faster, so we should figure out how much faster

@adigie adigie requested a review from andy31415 February 7, 2025 16:05
@adigie
Copy link
Contributor Author

adigie commented Feb 12, 2025

@andy31415 I updated the description, could you take a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

@andy31415 andy31415 Awaiting requested review from andy31415

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
app examples review - pending
Projects
None yet
Milestone
No milestone
Development

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