This is an automated email from the ASF dual-hosted git repository. pkarashchenko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git
commit 3edff9d32349585438f279f45e9261911c43714d Author: Xiang Xiao <xiaoxi...@xiaomi.com> AuthorDate: Sun May 28 19:40:46 2023 +0800 mm/tlsf: Change "memdump_backtrace_s *dump" to memdump_backtrace_s *buf" avoid the conflict with "mm_memdump_s *dump" Signed-off-by: Xiang Xiao <xiaoxi...@xiaomi.com> --- mm/tlsf/mm_tlsf.c | 70 +++++++++++++++++++++++++++---------------------------- 1 file changed, 35 insertions(+), 35 deletions(-) diff --git a/mm/tlsf/mm_tlsf.c b/mm/tlsf/mm_tlsf.c index 63b892d279..3a3b495db9 100644 --- a/mm/tlsf/mm_tlsf.c +++ b/mm/tlsf/mm_tlsf.c @@ -138,23 +138,23 @@ struct mm_mallinfo_handler_s ****************************************************************************/ static void memdump_backtrace(FAR struct mm_heap_s *heap, - FAR struct memdump_backtrace_s *dump) + FAR struct memdump_backtrace_s *buf) { # if CONFIG_MM_BACKTRACE > 0 FAR struct tcb_s *tcb; # endif - dump->pid = _SCHED_GETTID(); - dump->seqno = g_mm_seqno++; + buf->pid = _SCHED_GETTID(); + buf->seqno = g_mm_seqno++; # if CONFIG_MM_BACKTRACE > 0 - tcb = nxsched_get_tcb(dump->pid); + tcb = nxsched_get_tcb(buf->pid); if (heap->mm_procfs.backtrace || (tcb && tcb->flags & TCB_FLAG_HEAPDUMP)) { - int ret = backtrace(dump->backtrace, CONFIG_MM_BACKTRACE); + int ret = backtrace(buf->backtrace, CONFIG_MM_BACKTRACE); if (ret < CONFIG_MM_BACKTRACE) { - dump->backtrace[ret] = NULL; + buf->backtrace[ret] = NULL; } } # endif @@ -234,14 +234,14 @@ static void free_delaylist(FAR struct mm_heap_s *heap) static FAR void *mempool_memalign(FAR void *arg, size_t alignment, size_t size) { - FAR struct memdump_backtrace_s *dump; + FAR struct memdump_backtrace_s *buf; FAR void *ret; ret = mm_memalign(arg, alignment, size); if (ret) { - dump = ret + mm_malloc_size(arg, ret); - dump->pid = MM_BACKTRACE_MEMPOOL_PID; + buf = ret + mm_malloc_size(arg, ret); + buf->pid = MM_BACKTRACE_MEMPOOL_PID; } return ret; @@ -282,13 +282,13 @@ static void mallinfo_task_handler(FAR void *ptr, size_t size, int used, FAR void *user) { #if CONFIG_MM_BACKTRACE >= 0 - FAR struct memdump_backtrace_s *dump; + FAR struct memdump_backtrace_s *buf; #endif FAR struct mm_mallinfo_handler_s *handler = user; #if CONFIG_MM_BACKTRACE >= 0 size -= sizeof(struct memdump_backtrace_s); - dump = ptr + size; + buf = ptr + size; if (used) { @@ -300,12 +300,12 @@ static void mallinfo_task_handler(FAR void *ptr, size_t size, int used, } #else if (handler->dump->pid == MM_BACKTRACE_ALLOC_PID || - handler->dump->pid == dump->pid || + handler->dump->pid == buf->pid || (handler->dump->pid == MM_BACKTRACE_INVALID_PID && - nxsched_get_tcb(dump->pid) == NULL)) + nxsched_get_tcb(buf->pid) == NULL)) { - if (dump->seqno >= handler->dump->seqmin && - dump->seqno <= handler->dump->seqmax) + if (buf->seqno >= handler->dump->seqmin && + buf->seqno <= handler->dump->seqmax) { handler->info->aordblks++; handler->info->uordblks += size; @@ -409,10 +409,10 @@ static void memdump_handler(FAR void *ptr, size_t size, int used, { FAR const struct mm_memdump_s *dump = user; #if CONFIG_MM_BACKTRACE >= 0 - FAR struct memdump_backtrace_s *bt; + FAR struct memdump_backtrace_s *buf; size -= sizeof(struct memdump_backtrace_s); - bt = ptr + size; + buf = ptr + size; #endif if (used) @@ -421,9 +421,9 @@ static void memdump_handler(FAR void *ptr, size_t size, int used, if (pid == MM_BACKTRACE_ALLOC_PID) #else if ((dump->pid == MM_BACKTRACE_ALLOC_PID || - bt->pid == dump->pid) && - bt->seqno >= dump->seqmin && - bt->seqno <= dump->seqmax) + buf->pid == dump->pid) && + buf->seqno >= dump->seqmin && + buf->seqno <= dump->seqmax) #endif { #if CONFIG_MM_BACKTRACE < 0 @@ -433,21 +433,21 @@ static void memdump_handler(FAR void *ptr, size_t size, int used, int i; FAR const char *format = " %0*p"; # endif - char buf[CONFIG_MM_BACKTRACE * MM_PTR_FMT_WIDTH + 1]; + char tmp[CONFIG_MM_BACKTRACE * MM_PTR_FMT_WIDTH + 1]; - buf[0] = '\0'; + tmp[0] = '\0'; # if CONFIG_MM_BACKTRACE > 0 - for (i = 0; i < CONFIG_MM_BACKTRACE && dump->backtrace[i]; i++) + for (i = 0; i < CONFIG_MM_BACKTRACE && buf->backtrace[i]; i++) { - snprintf(buf + i * MM_PTR_FMT_WIDTH, - sizeof(buf) - i * MM_PTR_FMT_WIDTH, - format, MM_PTR_FMT_WIDTH - 1, dump->backtrace[i]); + snprintf(tmp + i * MM_PTR_FMT_WIDTH, + sizeof(tmp) - i * MM_PTR_FMT_WIDTH, + format, MM_PTR_FMT_WIDTH - 1, buf->backtrace[i]); } # endif syslog(LOG_INFO, "%6d%12zu%12lu%*p%s\n", - (int)bt->pid, size, bt->seqno, MM_PTR_FMT_WIDTH, - ptr, buf); + (int)buf->pid, size, buf->seqno, MM_PTR_FMT_WIDTH, + ptr, tmp); #endif } } @@ -1058,9 +1058,9 @@ FAR void *mm_malloc(FAR struct mm_heap_s *heap, size_t size) if (ret) { #if CONFIG_MM_BACKTRACE >= 0 - FAR struct memdump_backtrace_s *dump = ret + mm_malloc_size(heap, ret); + FAR struct memdump_backtrace_s *buf = ret + mm_malloc_size(heap, ret); - memdump_backtrace(heap, dump); + memdump_backtrace(heap, buf); #endif kasan_unpoison(ret, mm_malloc_size(heap, ret)); } @@ -1112,9 +1112,9 @@ FAR void *mm_memalign(FAR struct mm_heap_s *heap, size_t alignment, if (ret) { #if CONFIG_MM_BACKTRACE >= 0 - FAR struct memdump_backtrace_s *dump = ret + mm_malloc_size(heap, ret); + FAR struct memdump_backtrace_s *buf = ret + mm_malloc_size(heap, ret); - memdump_backtrace(heap, dump); + memdump_backtrace(heap, buf); #endif kasan_unpoison(ret, mm_malloc_size(heap, ret)); } @@ -1216,10 +1216,10 @@ FAR void *mm_realloc(FAR struct mm_heap_s *heap, FAR void *oldmem, #if CONFIG_MM_BACKTRACE >= 0 if (newmem) { - FAR struct memdump_backtrace_s *dump = newmem + - mm_malloc_size(heap, newmem); + FAR struct memdump_backtrace_s *buf = + newmem + mm_malloc_size(heap, newmem); - memdump_backtrace(heap, dump); + memdump_backtrace(heap, buf); } #endif