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

CONFIG_ESP_SYSTEM_PANIC_GDBSTUB suppress CONFIG_ESP_COREDUMP_ENABLE in panic.c but the menuconfig does not specify the conflict (IDFGH-8567) #10017

Closed
3 tasks done
wuyuanyi135 opened this issue Oct 20, 2022 · 1 comment
Assignees
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally

Comments

@wuyuanyi135
Copy link
Contributor

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

I just realized that when CONFIG_ESP_SYSTEM_PANIC_GDBSTUB is checked (Under Component config → ESP System Settings → Panic handler behaviour) the core-dump-to-flash did not generate any data.

#else
#if CONFIG_ESP_COREDUMP_ENABLE

Clearly, core dump is not enabled when GDB Stub is in use. I think those two options should be linked with Kconfig dependencies.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Oct 20, 2022
@github-actions github-actions bot changed the title CONFIG_ESP_SYSTEM_PANIC_GDBSTUB suppress CONFIG_ESP_COREDUMP_ENABLE in panic.c but the menuconfig does not specify the conflict CONFIG_ESP_SYSTEM_PANIC_GDBSTUB suppress CONFIG_ESP_COREDUMP_ENABLE in panic.c but the menuconfig does not specify the conflict (IDFGH-8567) Oct 20, 2022
@espressif-bot espressif-bot added Status: Selected for Development Issue is selected for development and removed Status: Opened Issue is new labels Oct 26, 2022
@espressif-bot espressif-bot added Status: Reviewing Issue is being reviewed and removed Status: Selected for Development Issue is selected for development labels Jan 20, 2023
@espressif-bot espressif-bot added Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Reviewing Issue is being reviewed Resolution: NA Issue resolution is unavailable labels Feb 20, 2023
@ESP-Marius
Copy link
Collaborator

Thanks for reporting this.

Actually, I cant think of a good reason why you shouldnt be allowed to use both. We have merged an internal MR to allow these features to be used together now.

espressif-bot pushed a commit to espressif/esp-hal-components that referenced this issue Feb 21, 2023
espressif-bot pushed a commit to espressif/esp-hal-components that referenced this issue Feb 21, 2023
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

3 participants