Compile error with enabling crash report

[masoudr]

Description

Hi,
I'm trying to set auto reboot for fault errors with help of this document. When I try to compile my code, I get the following compile error which seems to generate when enabling crash-capture-enabled in configuration.
I tested with mbed-os 5.11.5 and 5.12.0. My target platform is an STM32F407VG.
mbed_app.json:

    "target_overrides": {
        "*": {
            "platform.crash-capture-enabled": true,
            "platform.fatal-error-auto-reboot-enabled": true,
            "platform.error-reboot-max": 10,
        }

c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/cmsis/TARGET_CORTEX_M/mbed_fault_handler.o:../.\mbed-os\cmsis\TARGET_CORTEX_M/mbed_fault_handler.c:36: undefined reference to __CRASH_DATA_RAM_START__' c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/cmsis/TARGET_CORTEX_M/mbed_fault_handler.o: in function mbed_fault_handler':
../.\mbed-os\cmsis\TARGET_CORTEX_M/mbed_fault_handler.c:79: undefined reference to __CRASH_DATA_RAM_START__' c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/cmsis/TARGET_CORTEX_M/mbed_fault_handler.o: in function print_context_info':
../.\mbed-os\cmsis\TARGET_CORTEX_M/mbed_fault_handler.c:142: undefined reference to __CRASH_DATA_RAM_START__' c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/platform/mbed_error.o: in function mbed_error_initialize':
.././mbed-os/platform/mbed_error.c:235: undefined reference to __CRASH_DATA_RAM_START__' c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/platform/mbed_error.o: in function mbed_error':
.././mbed-os/platform/mbed_error.c:302: undefined reference to __CRASH_DATA_RAM_START__' c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/platform/mbed_error.o:.././mbed-os/platform/mbed_error.c:348: more undefined references to CRASH_DATA_RAM_START' follow
collect2.exe: error: ld returned 1 exit status
[ERROR] c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/cmsis/TARGET_CORTEX_M/mbed_fault_handler.o../.\mbed-os\cmsis\TARGET_CORTEX_M/mbed_fault_handler.c:36: undefined reference to __CRASH_DATA_RAM_START__' c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/cmsis/TARGET_CORTEX_M/mbed_fault_handler.o: in function mbed_fault_handler':
../.\mbed-os\cmsis\TARGET_CORTEX_M/mbed_fault_handler.c:79: undefined reference to __CRASH_DATA_RAM_START__' c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/cmsis/TARGET_CORTEX_M/mbed_fault_handler.o: in function print_context_info':
../.\mbed-os\cmsis\TARGET_CORTEX_M/mbed_fault_handler.c:142: undefined reference to __CRASH_DATA_RAM_START__' c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/platform/mbed_error.o: in function mbed_error_initialize':
.././mbed-os/platform/mbed_error.c:235: undefined reference to __CRASH_DATA_RAM_START__' c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/platform/mbed_error.o: in function mbed_error':
.././mbed-os/platform/mbed_error.c:302: undefined reference to __CRASH_DATA_RAM_START__' c:/program files (x86)/gnu tools arm embedded/8 2018-q4-major/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld.exe: BUILD/DISCO_F407VG/GCC_ARM-DEBUG/mbed-os/platform/mbed_error.o:.././mbed-os/platform/mbed_error.c:348: more undefined references to CRASH_DATA_RAM_START' follow
collect2.exe: error: ld returned 1 exit status

Issue request type

[x] Question
[ ] Enhancement
[x] Bug

[cmonr]

@SenRamakri

[ciarmcom]

Internal Jira reference: https://jira.arm.com/browse/MBOCUSTRIA-1065

[SenRamakri]

@masoudr - Have you made corresponding changes to your linker scripts according to section "Adding the Crash-data-RAM region for crash reporting" in https://os.mbed.com/docs/mbed-os/v5.11/apis/error-handling.html?

[masoudr]

@SenRamakri Thanks, I didn't know that.
Here is the ...\targets\TARGET_STM\TARGET_STM32F4\TARGET_STM32F407xG\device\TOOLCHAIN_GCC_ARM\STM32F407XG.ld `:

/* Linker script to configure memory regions. */

#if !defined(MBED_BOOT_STACK_SIZE)
    #define MBED_BOOT_STACK_SIZE 0x400
#endif

STACK_SIZE = MBED_BOOT_STACK_SIZE;

M_CRASH_DATA_RAM_SIZE = 0x100;

MEMORY
{ 
  FLASH (rx) : ORIGIN = 0x08000000, LENGTH = 1024K
  CCM (rwx) : ORIGIN = 0x10000000, LENGTH = 64K
  RAM (rwx) : ORIGIN = 0x20000188, LENGTH = 128k - 0x188 
}

/* Linker script to place sections and symbol values. Should be used together
 * with other linker script that defines memory regions FLASH and RAM.
 * It references following symbols, which must be defined in code:
 *   Reset_Handler : Entry of reset handler
 * 
 * It defines following symbols, which code can use without definition:
 *   __exidx_start
 *   __exidx_end
 *   __etext
 *   __data_start__
 *   __preinit_array_start
 *   __preinit_array_end
 *   __init_array_start
 *   __init_array_end
 *   __fini_array_start
 *   __fini_array_end
 *   __data_end__
 *   __bss_start__
 *   __bss_end__
 *   __end__
 *   end
 *   __HeapLimit
 *   __StackLimit
 *   __StackTop
 *   __stack
 *   _estack
 */
ENTRY(Reset_Handler)

SECTIONS
{
    .text :
    {
        KEEP(*(.isr_vector))
        *(.text*)
        KEEP(*(.init))
        KEEP(*(.fini))

        /* .ctors */
        *crtbegin.o(.ctors)
        *crtbegin?.o(.ctors)
        *(EXCLUDE_FILE(*crtend?.o *crtend.o) .ctors)
        *(SORT(.ctors.*))
        *(.ctors)

        /* .dtors */
        *crtbegin.o(.dtors)
        *crtbegin?.o(.dtors)
        *(EXCLUDE_FILE(*crtend?.o *crtend.o) .dtors)
        *(SORT(.dtors.*))
        *(.dtors)

        *(.rodata*)

        KEEP(*(.eh_frame*))
    } > FLASH

    .ARM.extab :
    {
        *(.ARM.extab* .gnu.linkonce.armextab.*)
    } > FLASH

    __exidx_start = .;
    .ARM.exidx :
    {
        *(.ARM.exidx* .gnu.linkonce.armexidx.*)
    } > FLASH
    __exidx_end = .;

    __etext = .;
    _sidata = .;

    .crash_data_ram :
    {
        . = ALIGN(8);
        __CRASH_DATA_RAM__ = .;
        __CRASH_DATA_RAM_START__ = .; /* Create a global symbol at data start */
        KEEP(*(.keep.crash_data_ram))
        *(.m_crash_data_ram)     /* This is a user defined section */
        . += M_CRASH_DATA_RAM_SIZE;
        . = ALIGN(8);
        __CRASH_DATA_RAM_END__ = .; /* Define a global symbol at data end */
    } > RAM

    .data : AT (__etext)
    {
        __data_start__ = .;
        _sdata = .;
        *(vtable)
        *(.data*)

        . = ALIGN(8);
        /* preinit data */
        PROVIDE_HIDDEN (__preinit_array_start = .);
        KEEP(*(.preinit_array))
        PROVIDE_HIDDEN (__preinit_array_end = .);

        . = ALIGN(8);
        /* init data */
        PROVIDE_HIDDEN (__init_array_start = .);
        KEEP(*(SORT(.init_array.*)))
        KEEP(*(.init_array))
        PROVIDE_HIDDEN (__init_array_end = .);


        . = ALIGN(8);
        /* finit data */
        PROVIDE_HIDDEN (__fini_array_start = .);
        KEEP(*(SORT(.fini_array.*)))
        KEEP(*(.fini_array))
        PROVIDE_HIDDEN (__fini_array_end = .);

        KEEP(*(.jcr*))
        . = ALIGN(8);
        /* All data end */
        __data_end__ = .;
        _edata = .;

    } > RAM

    .bss :
    {
        . = ALIGN(8);
        __bss_start__ = .;
        _sbss = .;
        *(.bss*)
        *(COMMON)
        . = ALIGN(8);
        __bss_end__ = .;
        _ebss = .;
    } > RAM

    .heap (COPY):
    {
        __end__ = .;
        end = __end__;
        *(.heap*)
        . = ORIGIN(RAM) + LENGTH(RAM) - STACK_SIZE;
        __HeapLimit = .;
    } > RAM

    /* .stack_dummy section doesn't contains any symbols. It is only
     * used for linker to calculate size of stack sections, and assign
     * values to stack symbols later */
    .stack_dummy (COPY):
    {
        *(.stack*)
    } > RAM

    /* Set stack top to end of RAM, and stack limit move down by
     * size of stack_dummy section */
    __StackTop = ORIGIN(RAM) + LENGTH(RAM);
    _estack = __StackTop;
    __StackLimit = __StackTop - STACK_SIZE;
    PROVIDE(__stack = __StackTop);

    /* Check if data + heap + stack exceeds RAM limit */
    ASSERT(__StackLimit >= __HeapLimit, "region RAM overflowed with stack")
}

[masoudr]

I compiled this sample code successfully but the device gets freezes and isn't rebooted. I also don't see any information about the exception. My aim is to just prevent the device from getting stuck on fault exceptions:

#include "mbed.h"
#include "mbed_error.h"
#include "mbed_fault_handler.h"

void generate_bus_fault_unaligned_access()
{
    SCB->CCR |= SCB_CCR_UNALIGN_TRP_Msk;
    uint32_t inv_addr = 0xAAA3;
    uint32_t val = *(uint32_t *)inv_addr;

    printf("\nval= %X", val);

    return;
}

int main()
{
    wait(2);
    generate_bus_fault_unaligned_access();
}

[bulislaw]

@masoudr did you manage to fix that issue? I'm afraid we won't be able to investigate it anytime soon. If you have time and it's an important issue for you please consider contributing a fix.

[masoudr]

@bulislaw Yes, I get it working by increasing the rtos.main-thread-stack-size. I will make a PR for it.

[jeromecoutant]

FYI: #10365