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

[ESP32S3] Incomprehensible offset if "CONFIG_ESP32S3_USE_FIXED_STATIC_RAM_SIZE" is used (IDFGH-8850) #10271

Closed
3 tasks done
sbithinx opened this issue Nov 29, 2022 · 2 comments
Assignees
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally

Comments

@sbithinx
Copy link

Answers checklist.

  • I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
  • I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report

Hi,
if the KConfig option "CONFIG_ESP32S3_USE_FIXED_STATIC_RAM_SIZE" is used later in the memory.ld file the following lines can be found:

#if CONFIG_ESP32S3_USE_FIXED_STATIC_RAM_SIZE
/* static data ends at defined address */
_static_data_end = 0x3FCA0000 + DRAM0_0_SEG_LEN;
#else
_static_data_end = _bss_end;
#endif // CONFIG_ESP32S3_USE_FIXED_STATIC_RAM_SIZE

The used offset 0x3FCA0000 is incomprehensible. This not the beginning of any memory region of the ESP32S3. Could you please explain the used address? If not this seems to be a bug.

This may also related to the issue #10270. At least it can be analyzed of fixed together.

Greetings
Sebastian

@Alvin1Zhang
Copy link
Collaborator

Thanks for reporting.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Nov 30, 2022
@github-actions github-actions bot changed the title [ESP32S3] Incomprehensible offset if "CONFIG_ESP32S3_USE_FIXED_STATIC_RAM_SIZE" is used [ESP32S3] Incomprehensible offset if "CONFIG_ESP32S3_USE_FIXED_STATIC_RAM_SIZE" is used (IDFGH-8850) Nov 30, 2022
@ESP-Marius
Copy link
Collaborator

Yeah, I think you are right. Seems like we have done a poor job of updating this option for new chips.

Thanks for reporting it, we'll take a look.

@espressif-bot espressif-bot added Status: Selected for Development Issue is selected for development Status: In Progress Work is in progress and removed Status: Opened Issue is new Status: Selected for Development Issue is selected for development labels Nov 30, 2022
espressif-bot pushed a commit to espressif/esp-hal-components that referenced this issue Mar 20, 2023
The USE_FIXED_STATIC_RAM_SIZE was not actually causing the heap to start
at a fixed address.

Closes espressif/esp-idf#10270
Closes espressif/esp-idf#10271
espressif-bot pushed a commit to espressif/esp-hal-components that referenced this issue Mar 20, 2023
The USE_FIXED_STATIC_RAM_SIZE was not actually causing the heap to start
at a fixed address.

Closes espressif/esp-idf#10270
Closes espressif/esp-idf#10271
@espressif-bot espressif-bot added Resolution: Done Issue is done internally Status: Done Issue is done internally and removed Status: In Progress Work is in progress labels Mar 23, 2023
espressif-bot pushed a commit that referenced this issue May 14, 2023
The USE_FIXED_STATIC_RAM_SIZE was not actually causing the heap to start
at a fixed address.

Closes #10270
Closes #10271
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

No branches or pull requests

4 participants