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

The newer ESP32 IDF can do core dumps in the coredump partition #561

Merged
merged 4 commits into from
May 10, 2023

Conversation

jeremypoulter
Copy link
Collaborator

No description provided.

@KipK
Copy link
Collaborator

KipK commented Mar 8, 2023

edit: interesting, what would be the best way to use this if it boot loop and no access to serial port?

I guess not much, but I can display in the Dev sections the last recorded coredump, still usefull for single crashes
Could you add a method to read it in evse_man ?

@jeremypoulter
Copy link
Collaborator Author

jeremypoulter commented Mar 9, 2023

Not really looked into it, just noticed a warning during crashes.

@KipK
Copy link
Collaborator

KipK commented Mar 22, 2023

in the case of bootloop, wouldn't it write forever on the partition ?

@jeremypoulter
Copy link
Collaborator Author

@KipK
Copy link
Collaborator

KipK commented Apr 11, 2023

We should add a new partition scheme too for the upcoming 16mb flash wifi modules.
I did that for now but not sure it's necessary to allocate this much for the app.

# Name,   Type, SubType, Offset,  Size, Flags
nvs,      data, nvs,     0x009000, 0x005000,
otadata,  data, ota,     0x00e000, 0x002000,
app0,     app,  ota_0,   0x010000, 0x7F0000,
app1,     app,  ota_1,   0x800000, 0x7F0000,
spiffs,   data, spiffs,  0xFF0000, 0x00F000,
coredump, data, coredump,0xFFF000, 0x10000

@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2023

Test Results

12 tests  ±0   12 ✔️ ±0   0s ⏱️ ±0s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit 6877db0. ± Comparison against base commit 98e2273.

@KipK
Copy link
Collaborator

KipK commented May 9, 2023

do you want this one to be merged ?

@jeremypoulter jeremypoulter merged commit 8170b69 into master May 10, 2023
@jeremypoulter jeremypoulter deleted the jeremypoulter/core_partition branch May 12, 2023 18:18
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

Successfully merging this pull request may close these issues.

2 participants