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

Latest commit has issues #355

Open
eriksl opened this issue Jan 3, 2022 · 1 comment
Open

Latest commit has issues #355

eriksl opened this issue Jan 3, 2022 · 1 comment

Comments

@eriksl
Copy link

eriksl commented Jan 3, 2022

feat(core): Updated all core version to 16407d55

This wastes a vast amount of precious IRAM memory and also it tends to crash frequently, in SDK code itself (not my code) with an LoadStoreException.

I really like the previous version "feat(core): update net802.11 version to 5d71a456" where the amount of used IRAM was minimal and there were no issues that bothered me.

@kriegste
Copy link

kriegste commented Jan 20, 2022

Have you tried the latest commit? I lose about 1.3 kB of heap.

I took a close look and compared the .a files. None were suspicious except for one. Part of libmain.a ended up in my crosshairs. These are the sizes of the sections inside libmain.a(app_main.o), comparing the old working to the new commit:

.comment:                           old:86    new:60
.data:                              old:1
.data.cache_map:                              new:1
.rodata.str1.4:                     old:56
.rodata.str1.1:                               new:52
.sdk.version:                       old:19    new:19
.text.Cache_Read_Enable_New:        old:80    new:73
.text.call_user_start_local:        old:32    new:33
.text.esp_crc8:                     old:55    new:54
.text.get_bin_len:                  old:221
.text.irom_printf:                  old:47    new:47
.text.NMI_Handler:                  old:98    new:95
.text.NmiTimSetFunc:                old:40    new:38
.text.spi_flash_clk_sel:            old:219
.text.system_correct_flash_map:     old:4     new:4
.text.user_fatal_exception_handler: old:61    new:562
.text.user_iram_memory_is_enabled:  old:4     new:4
.text.user_start:                   old:1283  new:4644
.text.user_uart_tx_one_char:        old:113   new:97
.text.user_uart_write_char:         old:57    new:51
.text.userbin_check:                old:22
.text.wdt_feed:                     old:115   new:123
.xt.lit:                            old:192   new:120
.xt.prop:                           old:5184  new:3144
.xtensa.info:                       old:440   new:56

To obtain the info
xtensa-lx106-elf-objdump -s
was used.

Some sections vanished in the new commit, some became smaller, some slightly bigger.
.text.user_start and .text.user_fatal_exception_handler became way bigger.
I don't know what it means and if anything can be done about it. But overall, the change in size reflects the shrunken amount of free heap.

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

2 participants