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

mm: Log name of the Heap to which the new region is being added #6385

Merged

Conversation

gustavonihei
Copy link
Contributor

Summary

This PR intends to improve the logging of mm_addregion function by informing the name of the Heap to which the new region is being added.

Impact

Increase in debug information, should not have any real impact.

Testing

esp32c3-devkit:knsh

Before

mm_initialize: Heap: name=Umem, start=0x3fc905ac size=322228
mm_addregion: Region 1: base=0x3fc90704 size=321872
up_allocate_kheap: Heap: start=3fc83068 end=3fc90000 size=53144
mm_initialize: Heap: name=Kmem, start=0x3fc83068 size=53144
mm_addregion: Region 1: base=0x3fc831c4 size=52784

###After

mm_initialize: Heap: name=Umem, start=0x3fc905ac size=322228
mm_addregion: [Umem] Region 1: base=0x3fc90704 size=321872
up_allocate_kheap: Heap: start=3fc83068 end=3fc90000 size=53144
mm_initialize: Heap: name=Kmem, start=0x3fc83068 size=53144
mm_addregion: [Kmem] Region 1: base=0x3fc831c4 size=52784

Signed-off-by: Gustavo Henrique Nihei <gustavo.nihei@espressif.com>
@xiaoxiang781216 xiaoxiang781216 merged commit 183da24 into apache:master Jun 7, 2022
@gustavonihei gustavonihei deleted the feature/mm_addregion_name branch June 7, 2022 17:35
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.

3 participants