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

Upgrade to platform-espressif32 6.0.0 (after 5.3.0) #862

Closed
proddy opened this issue Dec 30, 2022 · 6 comments
Closed

Upgrade to platform-espressif32 6.0.0 (after 5.3.0) #862

proddy opened this issue Dec 30, 2022 · 6 comments
Labels
technical Technical enhancement, or tech-debt issue
Milestone

Comments

@proddy
Copy link
Contributor

proddy commented Dec 30, 2022

platform-espressif32 5.3.0 was just released (https://github.com/platformio/platform-espressif32/releases/tag/v5.3.0)
and compared to 5.2.0 both RAM and Flash have gone up quite considerably. Shall we hardcode EMS-ESP to use 5.2.0 ?

Originally posted by @proddy in #388 (comment)

@proddy proddy added the enhancement New feature or request label Dec 30, 2022
@proddy proddy added this to the v3.6.0 milestone Dec 30, 2022
@proddy
Copy link
Contributor Author

proddy commented Dec 30, 2022

this needs some investigation. Also look into moving from C++11 to C++17

@proddy
Copy link
Contributor Author

proddy commented Dec 30, 2022

In the firmware build the RAM went up by 5768 bytes and the Flash up by 24260 bytes. When running EMS-ESP (with MQTT and HA enabled) the available heap went down by 7KB. This is using v3.5.0-dev.14.

For EMS-ESP v3.5.0 I suggest stick to espressif32 5.2.0 until we figure out some useful switches or work arounds.

proddy added a commit to proddy/EMS-ESP32 that referenced this issue Dec 30, 2022
@MichaelDvP
Copy link
Contributor

My testbuild with some extra debug messages compiled with c++17 and platform 5.3.0:
ESP32 fits in partition with 627 bytes left. (S3 is 16M flash, C3 binary is larger than reported by build, can't upload to symetric ota1/ota2 partitions)

ESP32
RAM:   [==        ]  17.5% (used 57288 bytes from 327680 bytes)
Flash: [==========]  100.0% (used 2030989 bytes from 2031616 bytes)

ESP32 C3
RAM:   [==        ]  15.6% (used 51276 bytes from 327680 bytes)
Flash: [==========]  95.1% (used 1932090 bytes from 2031616 bytes)

ESP32 S2
RAM:   [==        ]  17.8% (used 58304 bytes from 327680 bytes)
Flash: [==========]  96.0% (used 1950086 bytes from 2031616 bytes)

ESP32 S3
RAM:   [==        ]  17.1% (used 55952 bytes from 327680 bytes)
Flash: [==        ]  23.4% (used 1946669 bytes from 8323072 bytes)

ESP32 flashed to MH-ET32 with factory partitions:
grafik

@proddy proddy added technical Technical enhancement, or tech-debt issue and removed enhancement New feature or request labels Dec 31, 2022
@proddy
Copy link
Contributor Author

proddy commented Jan 5, 2023

I've created a new branch tech-upgrade for testing these changes and monitoring memory usage and performance. These changes will most likely end up in EMS-ESP 3.6.0.

@proddy proddy mentioned this issue Jan 6, 2023
13 tasks
@proddy
Copy link
Contributor Author

proddy commented Jan 16, 2023

and before you know it v6 is out with C+11 as default

https://github.com/platformio/platform-espressif32/releases/tag/v6.0.0

@proddy proddy changed the title Upgrade to platform-espressif32 5.3.0 Upgrade to platform-espressif32 6.0.0 (after 5.3.0) Jan 19, 2023
@proddy
Copy link
Contributor Author

proddy commented Jan 22, 2023

with 4MB

5.2.0 it uses 15.7% RAM and 99.6% flash
5.3.0 and the latest 6.0.0 its 17.5% RAM and 100.8% flash (which doesn't fir the 4MB)

I still can't find why it the library uses so much more memory. I've been browsing the forums but haven't spotted anything yet

@MichaelDvP any ideas?

@proddy proddy closed this as completed Feb 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
technical Technical enhancement, or tech-debt issue
Projects
None yet
Development

No branches or pull requests

2 participants