Commit 713b98b6 authored by Andrzej Puzdrowski's avatar Andrzej Puzdrowski
Browse files

boot/boot_serial: build-time skip of the erasing of img status page 



For MCUboot's MCUBOOT_SINGLE_APPLICATION_SLOT, MCUBOOT_FIRMWARE_LOADER,
MCUBOOT_SINGLE_APPLICATION_SLOT_RAM_LOAD
modes there is not image run-time status data at the end on the image.
Erase of that was done as the last image page erase - and was no harm
before by luck.

Following commit just reveled the issue:
 #35536633

Signed-off-by: default avatarAndrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no>
parent bf5321bb

boot/boot_serial/src/boot_serial.c

+10 −2
Original line number Diff line number Diff line
@@ -77,6 +77,12 @@ 


BOOT_LOG_MODULE_DECLARE(mcuboot);



#if !(defined(MCUBOOT_SINGLE_APPLICATION_SLOT) || \

    defined(MCUBOOT_FIRMWARE_LOADER) ||           \

    defined(MCUBOOT_SINGLE_APPLICATION_SLOT_RAM_LOAD))

#define BOOT_IMAGE_HAS_STATUS_FIELDS

#endif



#ifndef ARRAY_SIZE

#define ARRAY_SIZE ZCBOR_ARRAY_SIZE

#endif
@@ -913,8 +919,10 @@ bs_upload(char *buf, int len) 
                                         * erase has stopped to let us know whether erase

                                         * is needed to be able to write current chunk.

                                         */

#ifdef BOOT_IMAGE_HAS_STATUS_FIELDS

    static struct flash_sector status_sector;

#endif

#endif /* MCUBOOT_ERASE_PROGRESSIVELY */

#ifdef MCUBOOT_SWAP_USING_OFFSET

    static uint32_t start_off = 0;

#endif
@@ -987,7 +995,7 @@ bs_upload(char *buf, int len) 
#endif



        curr_off = 0;

#ifdef MCUBOOT_ERASE_PROGRESSIVELY

#if defined(MCUBOOT_ERASE_PROGRESSIVELY) && defined(BOOT_IMAGE_HAS_STATUS_FIELDS)

        /* Get trailer sector information; this is done early because inability to get

         * that sector information means that upload will not work anyway.

         * TODO: This is single occurrence issue, it should get detected during tests
@@ -1154,7 +1162,7 @@ bs_upload(char *buf, int len) 
    if (rc == 0) {

        curr_off += img_chunk_len + rem_bytes;

        if (curr_off == img_size) {

#ifdef MCUBOOT_ERASE_PROGRESSIVELY

#if defined(MCUBOOT_ERASE_PROGRESSIVELY) && defined(BOOT_IMAGE_HAS_STATUS_FIELDS)

            /* Assure that sector for image trailer was erased. */

            /* Check whether it was erased during previous upload. */

            off_t start = flash_sector_get_off(&status_sector);

CRA Git | Maintained and supported by SUSTech CRA and CCSE