Add back a feature that was available in the old heap implementation in release/v2.1 and earlier: keep track of which task allocates each block from the heap. The task handle is conditionally added as another word in the heap poisoning header under this configuration option CONFIG_HEAP_TASK_TRACKING. To allow custom monitoring and debugging code to be added, add helper functions in multi_heap.c and multi_heap_poisoning.c to provide access to information in the block headers. Add esp_heap_debug_dump_totals() to monitor heap usage esp_heap_debug_dump_totals() dumps into a user-provided data structure a summary of the amound of heap memory in region type that is used by each task. Optionally it will also dump into another data structure the metadata about each allocated block for a given list of tasks or for all tasks (limited by available space). Address change requests on PR #1498 This set of changes fixes the files in e3b702c to just add the CONFIG_HEAP_TASK_TRACKING option without adding the new function heap_caps_get_per_task_info() in case that is the only portion of the PR that will be accepted. Part of the change is to remove the new .c and .h files containing that function and to remove the line to compile it from components/heap/component.mk since it should not have been included in e3b702c. One or more additional commits to add the new function will follow. The other changes here: - uint32_t get_all_caps() moves to heap_private.h - replace "void* foo" with "void *foo" - add braces around single-line "if" blocks - replace tab characters with spaces Address change requests on PR #1498, part 2 This set of changes fixes the files in cdf32aa to add the new function heap_caps_get_per_task_info() with its new name and to add the line to compile it in components/heap/component.mk. This does not address all the suggested changes because there are some needing further discussion. This commit does not include the suggested change to move the declaration of the new function into esp_heap_caps.h because the new function references TaskHandle_t so esp_heap_caps.h would have to include freertos/FreeRTOS.h and freertos/task.h, but FreeRTOS.h includes esp_heap_caps.h through two other header files which results in compilation errors because not all of FreeRTOS.h has been read yet. Change heap_caps_get_per_task_info() to take struct of params In addition to moving the large number of function parameters into a struct as the single parameter, the following changes were made: - Clear out the totals for any prepopulated tasks so the app code doesn't have to do it. - Rather than partitioning the per-task totals into a predetermined set of heap capabilities, take a list of <caps,mask> pairs to compare the caps to the heap capabilities as masked. This lets the caller configure the desired partitioning, or none. - Allow the totals array pointer or the blocks array pointer to be NULL to indicate not to collect that part of the information. - In addition to returning the total space allocated by each task, return the number of blocks allocated by each task. - Don't need to return the heap capabilities as part of the details for each block since the heap region (and therefore its capabilities) can be determined from the block address. - Renamed heap_task_info.h to esp_heap_task_info.h to fit the naming convention, and renamed the structs for totals and block details to better fit the revised function name. - Provide full Doxygen commenting for the function and parameter structs. Add copyright header to new files Merges https://github.com/espressif/esp-idf/pull/1498
129 lines
4.4 KiB
C
129 lines
4.4 KiB
C
// Copyright 2018 Espressif Systems (Shanghai) PTE LTD
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
#include <freertos/FreeRTOS.h>
|
|
#include <freertos/task.h>
|
|
#include <multi_heap.h>
|
|
#include "multi_heap_internal.h"
|
|
#include "heap_private.h"
|
|
#include "esp_heap_task_info.h"
|
|
|
|
#ifdef CONFIG_HEAP_TASK_TRACKING
|
|
|
|
/*
|
|
* Return per-task heap allocation totals and lists of blocks.
|
|
*
|
|
* For each task that has allocated memory from the heap, return totals for
|
|
* allocations within regions matching one or more sets of capabilities.
|
|
*
|
|
* Optionally also return an array of structs providing details about each
|
|
* block allocated by one or more requested tasks, or by all tasks.
|
|
*
|
|
* Returns the number of block detail structs returned.
|
|
*/
|
|
size_t heap_caps_get_per_task_info(heap_task_info_params_t *params)
|
|
{
|
|
heap_t *reg;
|
|
heap_task_block_t *blocks = params->blocks;
|
|
size_t count = *params->num_totals;
|
|
size_t remaining = params->max_blocks;
|
|
|
|
// Clear out totals for any prepopulated tasks.
|
|
if (params->totals) {
|
|
for (size_t i = 0; i < count; ++i) {
|
|
for (size_t type = 0; type < NUM_HEAP_TASK_CAPS; ++type) {
|
|
params->totals[i].size[type] = 0;
|
|
params->totals[i].count[type] = 0;
|
|
}
|
|
}
|
|
}
|
|
|
|
SLIST_FOREACH(reg, ®istered_heaps, next) {
|
|
multi_heap_handle_t heap = reg->heap;
|
|
if (heap == NULL) {
|
|
continue;
|
|
}
|
|
|
|
// Find if the capabilities of this heap region match on of the desired
|
|
// sets of capabilities.
|
|
uint32_t caps = get_all_caps(reg);
|
|
uint32_t type;
|
|
for (type = 0; type < NUM_HEAP_TASK_CAPS; ++type) {
|
|
if ((caps & params->mask[type]) == params->caps[type]) {
|
|
break;
|
|
}
|
|
}
|
|
if (type == NUM_HEAP_TASK_CAPS) {
|
|
continue;
|
|
}
|
|
|
|
multi_heap_block_handle_t b = multi_heap_get_first_block(heap);
|
|
multi_heap_internal_lock(heap);
|
|
for ( ; b ; b = multi_heap_get_next_block(heap, b)) {
|
|
if (multi_heap_is_free(b)) {
|
|
continue;
|
|
}
|
|
void *p = multi_heap_get_block_address(b); // Safe, only arithmetic
|
|
size_t bsize = multi_heap_get_allocated_size(heap, p); // Validates
|
|
TaskHandle_t btask = (TaskHandle_t)multi_heap_get_block_owner(b);
|
|
|
|
// Accumulate per-task allocation totals.
|
|
if (params->totals) {
|
|
size_t i;
|
|
for (i = 0; i < count; ++i) {
|
|
if (params->totals[i].task == btask) {
|
|
break;
|
|
}
|
|
}
|
|
if (i < count) {
|
|
params->totals[i].size[type] += bsize;
|
|
params->totals[i].count[type] += 1;
|
|
}
|
|
else {
|
|
if (count < params->max_totals) {
|
|
params->totals[count].task = btask;
|
|
params->totals[count].size[type] = bsize;
|
|
params->totals[i].count[type] = 1;
|
|
++count;
|
|
}
|
|
}
|
|
}
|
|
|
|
// Return details about allocated blocks for selected tasks.
|
|
if (blocks && remaining > 0) {
|
|
if (params->tasks) {
|
|
size_t i;
|
|
for (i = 0; i < params->num_tasks; ++i) {
|
|
if (btask == params->tasks[i]) {
|
|
break;
|
|
}
|
|
}
|
|
if (i == params->num_tasks) {
|
|
continue;
|
|
}
|
|
}
|
|
blocks->task = btask;
|
|
blocks->address = p;
|
|
blocks->size = bsize;
|
|
++blocks;
|
|
--remaining;
|
|
}
|
|
}
|
|
multi_heap_internal_unlock(heap);
|
|
}
|
|
*params->num_totals = count;
|
|
return params->max_blocks - remaining;
|
|
}
|
|
|
|
#endif // CONFIG_HEAP_TASK_TRACKING
|