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

[ESP32] Details for using ccache for idf builds #21340

Merged
merged 2 commits into from
Jul 29, 2022
Merged

[ESP32] Details for using ccache for idf builds #21340

merged 2 commits into from
Jul 29, 2022

Conversation

shubhamdp
Copy link
Contributor

@shubhamdp shubhamdp commented Jul 28, 2022 •
edited
Loading

Problem

  • Building matter examples for ESP32 takes a lot of time

Change overview

  • Details for using ccache with IDF builds
  • Removed the stale menuconfig option details

Testing

  • First local build took around ~6min and subsequent build was just ~2min

@github-actions
Copy link

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

PR #21340: Size comparison from e758eb2 to eea8294

Increases (1 build for cc13x2_26x2)
platform target config section e758eb2 eea8294 change % change
cc13x2_26x2 pump-app LP_CC2652R7 (read/write) 161648 161656 8 0.0
Decreases (1 build for cc13x2_26x2)
platform target config section e758eb2 eea8294 change % change
cc13x2_26x2 pump-app LP_CC2652R7 (read only) 680743 680735 -8 -0.0
.text 591396 591388 -8 -0.0
Full report (32 builds for bl602, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section e758eb2 eea8294 change % change
bl602 lighting-app bl602 (read/write) 1381330 1381330 0 0.0
.bss 117618 117618 0 0.0
.data 4480 4480 0 0.0
.text 1051624 1051624 0 0.0
bl602+rpc (read/write) 1426746 1426746 0 0.0
.bss 125058 125058 0 0.0
.data 4600 4600 0 0.0
.text 1083292 1083292 0 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 668299 668299 0 0.0
(read/write) 183068 183068 0 0.0
.bss 74260 74260 0 0.0
.data 3356 3356 0 0.0
.rodata 88195 88195 0 0.0
.text 579788 579788 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 633875 633875 0 0.0
(read/write) 157828 157828 0 0.0
.bss 73556 73556 0 0.0
.data 3356 3356 0 0.0
.rodata 77411 77411 0 0.0
.text 556140 556140 0 0.0
lock-ftd LP_CC2652R7 (read only) 671455 671455 0 0.0
(read/write) 170104 170104 0 0.0
.bss 71340 71340 0 0.0
.data 3280 3280 0 0.0
.rodata 76263 76263 0 0.0
.text 594712 594712 0 0.0
lock-mtd LP_CC2652R7 (read only) 653555 653555 0 0.0
(read/write) 183692 183692 0 0.0
.bss 67028 67028 0 0.0
.data 3280 3280 0 0.0
.rodata 100875 100875 0 0.0
.text 552200 552200 0 0.0
pump-app LP_CC2652R7 (read only) 680743 680735 -8 -0.0
(read/write) 161648 161656 8 0.0
.bss 71404 71404 0 0.0
.data 3280 3280 0 0.0
.rodata 88863 88863 0 0.0
.text 591396 591388 -8 -0.0
pump-controller-app LP_CC2652R7 (read only) 666487 666487 0 0.0
(read/write) 176040 176040 0 0.0
.bss 71540 71540 0 0.0
.data 3276 3276 0 0.0
.rodata 84679 84679 0 0.0
.text 581328 581328 0 0.0
shell LP_CC2652R7 (read only) 661006 661006 0 0.0
(read/write) 185880 185880 0 0.0
.bss 76580 76580 0 0.0
.data 3360 3360 0 0.0
.rodata 85166 85166 0 0.0
.text 575524 575524 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 583206 583206 0 0.0
.app_xip_area 460016 460016 0 0.0
.bss 65648 65648 0 0.0
.data 728 728 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 589126 589126 0 0.0
.app_xip_area 461208 461208 0 0.0
.bss 70376 70376 0 0.0
.data 732 732 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 588938 588938 0 0.0
.app_xip_area 466564 466564 0 0.0
.bss 64888 64888 0 0.0
.data 672 672 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1087832 1087832 0 0.0
.bss 133284 133284 0 0.0
.data 2048 2048 0 0.0
.text 952480 952480 0 0.0
BRD4161A+rpc (read/write) 1142124 1142124 0 0.0
.bss 149964 149964 0 0.0
.data 2260 2260 0 0.0
.text 989880 989880 0 0.0
BRD4161A+rs911x (read/write) 973656 973656 0 0.0
.bss 161736 161736 0 0.0
.data 2048 2048 0 0.0
.text 809852 809852 0 0.0
lock-app BRD4161A+wf200 (read/write) 1128328 1128328 0 0.0
.bss 144368 144368 0 0.0
.data 2056 2056 0 0.0
.text 981884 981884 0 0.0
window-app BRD4161A (read/write) 1081308 1081308 0 0.0
.bss 134756 134756 0 0.0
.data 2076 2076 0 0.0
.text 944452 944452 0 0.0
esp32 all-clusters-app c3devkit (read only) 1022460 1022460 0 0.0
(read/write) 1486274 1486274 0 0.0
.dram0.bss 70296 70296 0 0.0
.dram0.data 14600 14600 0 0.0
.flash.rodata 215936 215936 0 0.0
.flash.text 1022460 1022460 0 0.0
.iram0.text 62902 62902 0 0.0
m5stack (read only) 1076107 1076107 0 0.0
(read/write) 488312 488312 0 0.0
.dram0.bss 75808 75808 0 0.0
.dram0.data 34144 34144 0 0.0
.flash.rodata 246364 246364 0 0.0
.flash.text 1070723 1070723 0 0.0
.iram0.text 123267 123267 0 0.0
k32w light k32w0+release (read/write) 642600 642600 0 0.0
.bss 69736 69736 0 0.0
.data 2028 2028 0 0.0
.text 568108 568108 0 0.0
lock k32w0+release (read/write) 699832 699832 0 0.0
.bss 70176 70176 0 0.0
.data 2036 2036 0 0.0
.text 624892 624892 0 0.0
linux chip-tool-ipv6only arm64 (read only) 9829588 9829588 0 0.0
(read/write) 675729 675729 0 0.0
.bss 32897 32897 0 0.0
.data 3272 3272 0 0.0
.data.rel.ro 621080 621080 0 0.0
.dynamic 560 560 0 0.0
.got 13528 13528 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 467492 467492 0 0.0
.text 7786516 7786516 0 0.0
thermostat-no-ble arm64 (read only) 2343388 2343388 0 0.0
(read/write) 141601 141601 0 0.0
.bss 55313 55313 0 0.0
.data 1672 1672 0 0.0
.data.rel.ro 75856 75856 0 0.0
.dynamic 560 560 0 0.0
.got 4992 4992 0 0.0
.init 24 24 0 0.0
.init_array 408 408 0 0.0
.rodata 138964 138964 0 0.0
.text 1967200 1967200 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2450384 2450384 0 0.0
.bss 214516 214516 0 0.0
.data 5872 5872 0 0.0
.text 1413028 1413028 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1172691 1172691 0 0.0
bss 143140 143140 0 0.0
rodata 141828 141828 0 0.0
text 808808 808808 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1152743 1152743 0 0.0
bss 142376 142376 0 0.0
rodata 133360 133360 0 0.0
text 798120 798120 0 0.0
p6 all-clusters-app default (read only) 881560 881560 0 0.0
(read/write) 1686700 1686700 0 0.0
.bss 149136 149136 0 0.0
.data 2648 2648 0 0.0
.text 1526528 1526528 0 0.0
all-clusters-minimal-app default (read only) 882280 882280 0 0.0
(read/write) 1630796 1630796 0 0.0
.bss 148416 148416 0 0.0
.data 2648 2648 0 0.0
.text 1471344 1471344 0 0.0
light-app default (read only) 890584 890584 0 0.0
(read/write) 1552220 1552220 0 0.0
.bss 140320 140320 0 0.0
.data 2440 2440 0 0.0
.text 1401072 1401072 0 0.0
lock-app default (read only) 886112 886112 0 0.0
(read/write) 1589828 1589828 0 0.0
.bss 144776 144776 0 0.0
.data 2456 2456 0 0.0
.text 1434208 1434208 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 799600 799600 0 0.0
bss 70816 70816 0 0.0
noinit 40416 40416 0 0.0
text 567240 567240 0 0.0
lighting-app tlsr9518adk80d (read/write) 819708 819708 0 0.0
bss 71660 71660 0 0.0
noinit 40416 40416 0 0.0
text 583812 583812 0 0.0

@github-actions github-actions bot removed the platform label Jul 29, 2022
@shubhamdp shubhamdp mentioned this pull request Jul 29, 2022
Closed
@dhrishi
Copy link
Contributor

dhrishi commented Jul 29, 2022

@woody-apple @bzbarsky-apple Can we merge this?

@bzbarsky-apple bzbarsky-apple merged commit 4c52019 into project-chip:master Jul 29, 2022
github-actions bot pushed a commit that referenced this pull request Jul 29, 2022
@shubhamdp @web-flow
[ESP32] Details for using ccache for idf builds (#21340)
2de614f 
* [ESP32] Details for using ccache for idf builds

* Remove unavailable menuconfig options readmes
@shubhamdp shubhamdp deleted the esp32_ccache branch July 29, 2022 13:46
woody-apple added a commit that referenced this pull request Jul 29, 2022
@woody-apple @shubhamdp
[ESP32] Details for using ccache for idf builds (#21340) (#21411)
c7dc530 
* [ESP32] Details for using ccache for idf builds

* Remove unavailable menuconfig options readmes

Co-authored-by: Shubham Patil <[email protected]>
isiu-apple pushed a commit to isiu-apple/connectedhomeip that referenced this pull request Sep 16, 2022
@shubhamdp @isiu-apple
[ESP32] Details for using ccache for idf builds (project-chip#21340)
a438110 
* [ESP32] Details for using ccache for idf builds

* Remove unavailable menuconfig options readmes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bzbarsky-apple bzbarsky-apple bzbarsky-apple approved these changes

@arkq arkq arkq approved these changes

@dhrishi dhrishi dhrishi approved these changes

@anush-apple anush-apple Awaiting requested review from anush-apple

@Byungjoo-Lee Byungjoo-Lee Awaiting requested review from Byungjoo-Lee

@carol-apple carol-apple Awaiting requested review from carol-apple

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@Damian-Nordic Damian-Nordic Awaiting requested review from Damian-Nordic

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@emargolis emargolis Awaiting requested review from emargolis

@erjiaqing erjiaqing Awaiting requested review from erjiaqing

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

@gjc13 gjc13 Awaiting requested review from gjc13

@harsha-rajendran harsha-rajendran Awaiting requested review from harsha-rajendran

@hawk248 hawk248 Awaiting requested review from hawk248

@isiu-apple isiu-apple Awaiting requested review from isiu-apple

@jelderton jelderton Awaiting requested review from jelderton

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

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

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

@kghost kghost Awaiting requested review from kghost

@kpschoedel kpschoedel Awaiting requested review from kpschoedel

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

@mrjerryjohns mrjerryjohns Awaiting requested review from mrjerryjohns

@msandstedt msandstedt Awaiting requested review from msandstedt

@rgoliver rgoliver Awaiting requested review from rgoliver

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

@tecimovic tecimovic Awaiting requested review from tecimovic

@tehampson tehampson Awaiting requested review from tehampson

@turon turon Awaiting requested review from turon

@vijs vijs Awaiting requested review from vijs

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

@wbschiller wbschiller Awaiting requested review from wbschiller

@woody-apple woody-apple Awaiting requested review from woody-apple

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

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

Successfully merging this pull request may close these issues.
4 participants
@shubhamdp @dhrishi @arkq @bzbarsky-apple