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

ESP32 builds are massive #2819

Open
mikee47 opened this issue Jun 19, 2024 · 4 comments
Open

ESP32 builds are massive #2819

mikee47 opened this issue Jun 19, 2024 · 4 comments

Comments

@mikee47
Copy link
Contributor

mikee47 commented Jun 19, 2024

HostTests builds for esp32 look suspicously large. These are all default builds. Needs further investigation.

esp32: app.bin 1,173,952 bytes

   Section|                   Description| Start (hex)|   End (hex)|Used space
------------------------------------------------------------------------------
      data|        Initialized Data (RAM)|    3FFBDB60|    3FFC0B18|   12216
    rodata|           ReadOnly Data (SPI)|    3F400020|    3F447910|  293104
       bss|      Uninitialized Data (RAM)|    3FFC0B18|    3FFC4F48|   17456
 iram_text|            Cached Code (IRAM)|    40080400|    40094148|   81224
      text|           Uncached Code (SPI)|    400D0020|    4018FF84|  786276

esp32c3: app.bin 1,207,664 bytes

   Section|                   Description| Start (hex)|   End (hex)|Used space
------------------------------------------------------------------------------
      data|        Initialized Data (RAM)|    3FC8FE00|    3FC926E8|   10472
    rodata|           ReadOnly Data (SPI)|    3C0D0020|    3C116AE0|  289472
       bss|      Uninitialized Data (RAM)|    3FC926F0|    3FC96C50|   17760
 iram_text|            Cached Code (IRAM)|    40380250|    4038FE00|   64432
      text|           Uncached Code (SPI)|    42000020|    420CD1BC|  840092

esp8266: rom0.bin 771,584 bytes

   Section|                   Description| Start (hex)|   End (hex)|Used space
------------------------------------------------------------------------------
      data|        Initialized Data (RAM)|    3FFE8000|    3FFE8794|    1940
    rodata|           ReadOnly Data (RAM)|    3FFE87A0|    3FFE9AF0|    4944
       bss|      Uninitialized Data (RAM)|    3FFE9AF0|    3FFEFFC8|   25816
      text|            Cached Code (IRAM)|    40100000|    40106B8F|   27535
irom0_text|           Uncached Code (SPI)|    40202010|    402B5F58|  737096

rp2040: app.bin 710,172 bytes

     Section|                   Description| Start (hex)|   End (hex)|Used space
------------------------------------------------------------------------------
        data|        Initialized Data (RAM)|    200000C0|    200015E4|    5412
         bss|      Uninitialized Data (RAM)|    20001608|    2000CB48|   46400
flash_binary|           Uncached Code (SPI)|    10000000|    100AD61C|  710172
@mikee47
Copy link
Contributor Author

mikee47 commented Jun 23, 2024

Comparing build logs between IDF 4.4 and IDF 5.2:

tests/HostTests
  log                data   rodata  bss    iram_text  text     Total Used RAM  Free RAM  Free IRam
  -----------------  -----  ------  -----  ---------  -------  --------------  --------  ---------
  log-esp32-4.4.txt  10592  248088  16336  78800      683690   26928           308944    52272    
  log-esp32-5.2.txt  12216  293572  17472  81224      788196   29688           306184    49848    
  Difference         +1624  +45484  +1136  +2424      +104506  +2760           -2760     -2424    

@mikee47
Copy link
Contributor Author

mikee47 commented Jun 24, 2024

IDF 5 adds esp_newlib_init_global_stdio which results in various I/O code from newlib being linked. Not a lot though.

@mikee47
Copy link
Contributor Author

mikee47 commented Jun 24, 2024

Building with WPA3 disabled reduces size:

IDF 4.4

ESP_WIFI_ENABLE_WPA3_SAE=n

app.bin: 1,017,424 bytes

HostTests: Linking out/Esp32/esp32/debug/build/app.out
   Section|                   Description| Start (hex)|   End (hex)|Used space
------------------------------------------------------------------------------
      data|        Initialized Data (RAM)|    3FFBDB60|    3FFC04C0|   10592
    rodata|           ReadOnly Data (SPI)|    3F400020|    3F43C25C|  246332
       bss|      Uninitialized Data (RAM)|    3FFC04C0|    3FFC4490|   16336
 iram_text|            Cached Code (IRAM)|    40080400|    400937D0|   78800
      text|           Uncached Code (SPI)|    400D0020|    40174A5E|  674366

IDF 5.2

ESP_WIFI_ENABLE_WPA3_OWE_STA=n
ESP_WIFI_ENABLE_WPA3_SAE=n
ESP_WIFI_SOFTAP_SAW_SUPPORT=n

app.bin: 1,077,520 bytes

HostTests: Linking out/Esp32/esp32/debug/build/app.out
   Section|                   Description| Start (hex)|   End (hex)|Used space
------------------------------------------------------------------------------
      data|        Initialized Data (RAM)|    3FFBDB60|    3FFC0AF8|   12184
    rodata|           ReadOnly Data (SPI)|    3F400020|    3F43E734|  255764
       bss|      Uninitialized Data (RAM)|    3FFC0AF8|    3FFC4E28|   17200
 iram_text|            Cached Code (IRAM)|    40080400|    400940F0|   81136
      text|           Uncached Code (SPI)|    400D0020|    4017F16C|  717132

@mikee47
Copy link
Contributor Author

mikee47 commented Jun 24, 2024

Release builds are very similar between IDF 4.4 and 5.2. Conclusion: More debug code/text in IDF 5.2.

IDF 4.4

HostTests: Linking out/Esp32/esp32/release/build/app.out
   Section|                   Description| Start (hex)|   End (hex)|Used space
------------------------------------------------------------------------------
      data|        Initialized Data (RAM)|    3FFBDB60|    3FFC04B8|   10584
    rodata|           ReadOnly Data (SPI)|    3F400020|    3F435EF8|  220888
       bss|      Uninitialized Data (RAM)|    3FFC04B8|    3FFC4338|   16000
 iram_text|            Cached Code (IRAM)|    40080400|    40093008|   76808
      text|           Uncached Code (SPI)|    400D0020|    40171DDE|  662974

IDF 5.2

HostTests: Linking out/Esp32/esp32/release/build/app.out
   Section|                   Description| Start (hex)|   End (hex)|Used space
------------------------------------------------------------------------------
      data|        Initialized Data (RAM)|    3FFBDB60|    3FFC0AB8|   12120
    rodata|           ReadOnly Data (SPI)|    3F400020|    3F438034|  229396
       bss|      Uninitialized Data (RAM)|    3FFC0AB8|    3FFC4C98|   16864
 iram_text|            Cached Code (IRAM)|    40080400|    40093A9C|   79516
      text|           Uncached Code (SPI)|    400D0020|    40172B24|  666372

slaff pushed a commit that referenced this issue Jun 25, 2024
Further to #2819 the size mainly seems to relate to two things:

1. WPA3 support is enabled by default, but probably isn't necessary for most applications (could be wrong).
2. More debug stuff in IDF 5.2; release builds are comparable to IDF 4.4.

## Bug fixes

- esp32 timer definitions missing in RELEASE build
- sdkconfig definitions not picked up

## Size fixes

**Disable WPA3 by default**

This is also enabled by default in IDF 4.4 but can always be re-enabled if required.
It adds quite a lot to the image size.

**Don't pull in stdio code from newlib**

Sming doesn't use newlib stdio streams


## Others

- Revise CI tool `scanlog.py` to include missing builds (HostTests)
- Make `checksdk` a pre-requisite
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant