This is an automated email from the ASF dual-hosted git repository. xiaoxiang pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/nuttx.git
commit f2bdfa43e13c15cac6ab6d43fdc4867ff09a2983 Author: anjiahao <[email protected]> AuthorDate: Thu Jul 4 15:02:17 2024 +0800 coredump:Add a coreudmp api to watch custom variables Signed-off-by: anjiahao <[email protected]> --- include/nuttx/coredump.h | 10 +++++++ sched/misc/coredump.c | 75 ++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 85 insertions(+) diff --git a/include/nuttx/coredump.h b/include/nuttx/coredump.h index b664074544..db41e769b2 100644 --- a/include/nuttx/coredump.h +++ b/include/nuttx/coredump.h @@ -42,6 +42,16 @@ int coredump_set_memory_region(FAR const struct memory_region_s *region); +/**************************************************************************** + * Name: coredump_add_memory_region + * + * Description: + * Use coredump to dump the memory of the specified area. + * + ****************************************************************************/ + +int coredump_add_memory_region(FAR const void *ptr, size_t size); + /**************************************************************************** * Name: coredump_initialize * diff --git a/sched/misc/coredump.c b/sched/misc/coredump.c index 3602cada7d..ae5e09e154 100644 --- a/sched/misc/coredump.c +++ b/sched/misc/coredump.c @@ -181,6 +181,81 @@ int coredump_set_memory_region(FAR const struct memory_region_s *region) return 0; } +/**************************************************************************** + * Name: coredump_add_memory_region + * + * Description: + * Use coredump to dump the memory of the specified area. + * + ****************************************************************************/ + +int coredump_add_memory_region(FAR const void *ptr, size_t size) +{ + FAR struct memory_region_s *region; + size_t count = 1; /* 1 for end flag */ + + if (g_regions != NULL) + { + region = (FAR struct memory_region_s *)g_regions; + + while (region->start != 0) + { + if ((uintptr_t)ptr >= region->start && + (uintptr_t)ptr + size < region->end) + { + /* Already watched */ + + return 0; + } + else if ((uintptr_t)ptr < region->end && + (uintptr_t)ptr + size >= region->end) + { + /* start in region, end out of region */ + + region->end = (uintptr_t)ptr + size; + return 0; + } + else if ((uintptr_t)ptr < region->start && + (uintptr_t)ptr + size >= region->start) + { + /* start out of region, end in region */ + + region->start = (uintptr_t)ptr; + return 0; + } + else if ((uintptr_t)ptr < region->start && + (uintptr_t)ptr + size >= region->end) + { + /* start out of region, end out of region */ + + region->start = (uintptr_t)ptr; + region->end = (uintptr_t)ptr + size; + return 0; + } + + count++; + region++; + } + + /* Need a new region */ + } + + region = lib_realloc((FAR void *)g_regions, + sizeof(struct memory_region_s) * (count + 1)); + if (region == NULL) + { + return -ENOMEM; + } + + region[count - 1].start = (uintptr_t)ptr; + region[count - 1].end = (uintptr_t)ptr + size; + region[count - 1].flags = 0; + region[count].start = 0; + + g_regions = region; + return 0; +} + /**************************************************************************** * Name: coredump_initialize *
