Skip to content

Add LOG_HEAP log type, and more heap debug messages#7937

Merged
jp-bennett merged 1 commit into
masterfrom
LOG_HEAP
Sep 9, 2025
Merged

Add LOG_HEAP log type, and more heap debug messages#7937
jp-bennett merged 1 commit into
masterfrom
LOG_HEAP

Conversation

@jp-bennett

Copy link
Copy Markdown
Collaborator

Add the LOG_HEAP log call, and Add some debugging around memory allocation and freeing

@jp-bennett jp-bennett added the enhancement New feature or request label Sep 9, 2025
@jp-bennett jp-bennett merged commit e7741c2 into master Sep 9, 2025
78 of 79 checks passed
@jp-bennett jp-bennett deleted the LOG_HEAP branch September 9, 2025 15:29
alexl83 pushed a commit to alexl83/firmware that referenced this pull request Sep 10, 2025
Comment thread src/mesh/MeshService.cpp
// (so we update our nodedb for the local node)

// Send the packet into the mesh
auto heapBefore = memGet.getFreeHeap();

@mverch67 mverch67 Sep 10, 2025

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should go into a #ifdef DEBUG_HEAP as these look like costly operations. Not sure if the compiler will optimize these away when not used (they are not declared const).

@mverch67 mverch67 Sep 10, 2025

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also saw crashes on esp32 when I used these in MUI periodically. So I switched the heap icon off by default.

jeek pushed a commit to jeek/Meshtastic-Exploiteers-Hacker-Pager that referenced this pull request Jun 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants