RE: MCUboot loses pending update image in slot 1 after reset

Voxorin — Sat, 15 Jun 2024 16:15:13 GMT

I have confirmed the issue was due to an improperly signed image generated by the CI.

The image wasn't signed correctly because I forgot to ensure the correct NRF python dependencies were installed.

I'm using the zephyrproject-rtos/ci-base which has the Zephyr python dependencies preinstalled but not the dependencies required for Nordic's version of MCUboot.

Adding this to the image solved my issue:

- name: Update Python dependencies
    run: |
      pip install -r zephyr/scripts/requirements.txt
      pip install -r nrf/scripts/requirements.txt
      pip install -r bootloader/mcuboot/scripts/requirements.txt

RE: MCUboot loses pending update image in slot 1 after reset

Voxorin — Fri, 14 Jun 2024 21:20:16 GMT

It turns out the image actually isn't valid. Something must be different about the CI build env.

RE: MCUboot loses pending update image in slot 1 after reset

Voxorin — Fri, 14 Jun 2024 15:05:35 GMT

Here's a less verbose log:

<inf> mcuboot: Starting bootloader
<inf> mcuboot: Primary image: magic=unset, swap_type=0x1, copy_done=0x3, image_ok=0x3
<inf> mcuboot: Secondary image: magic=good, swap_type=0x3, copy_done=0x3, image_ok=0x1
<inf> mcuboot: Boot source: none
<inf> mcuboot: Image index: 0, Swap type: perm
<err> mcuboot: Image in the secondary slot is not valid!
<inf> mcuboot: Bootloader chainload address offset: 0x20000
<inf> mcuboot: Jumping to the first image slot

RE: MCUboot loses pending update image in slot 1 after reset

Voxorin — Fri, 14 Jun 2024 14:27:25 GMT

It took me a bit to get MCUboot logging working, but I finally saw a useful message: mcuboot: Image in the secondary slot is not valid!

It isn't obvious to me why the image isn't valid.

Here's (part of) the stack trace:

[00:00:10.471,740] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: tx_bufs 0x20005130 - rx_bufs 0x20005138 - 1␛[0m
[00:00:10.473,358] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: current_tx 0x20005140 (2), current_rx 0x20005140 (2), tx buf/len 0x20005128/1, rx buf/len 0x20005128/1␛[0m
[00:00:10.475,769] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_tx: tx buf/len 0x20005177/1␛[0m
[00:00:10.476,989] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_rx: rx buf/len 0x20005177/1␛[0m
[00:00:10.478,179] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_tx: tx buf/len (nil)/0␛[0m
[00:00:10.479,339] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_rx: rx buf/len (nil)/0␛[0m
[00:00:10.480,468] ␛[0m<dbg> spi_nrfx_spim: finish_transaction: Transaction finished with status 0␛[0m
[00:00:10.481,781] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: tx_bufs 0x20005130 - rx_bufs 0x20005138 - 1␛[0m
[00:00:10.483,428] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: current_tx 0x20005140 (2), current_rx 0x20005140 (2), tx buf/len 0x20005128/1, rx buf/len 0x20005128/1␛[0m
[00:00:10.485,839] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_tx: tx buf/len 0x20005177/1␛[0m
[00:00:10.487,030] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_rx: rx buf/len 0x20005177/1␛[0m
[00:00:10.488,250] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_tx: tx buf/len (nil)/0␛[0m
[00:00:10.489,379] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_rx: rx buf/len (nil)/0␛[0m
[00:00:10.490,539] ␛[0m<dbg> spi_nrfx_spim: finish_transaction: Transaction finished with status 0␛[0m
[00:00:10.491,851] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: tx_bufs 0x20005130 - rx_bufs 0x20005138 - 1␛[0m
[00:00:10.493,499] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: current_tx 0x20005140 (2), current_rx 0x20005140 (2), tx buf/len 0x20005128/1, rx buf/len 0x20005128/1␛[0m
[00:00:10.495,910] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_tx: tx buf/len 0x20005177/1␛[0m
[00:00:10.497,100] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_rx: rx buf/len 0x20005177/1␛[0m
[00:00:10.498,321] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_tx: tx buf/len (nil)/0␛[0m
[00:00:10.499,450] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_rx: rx buf/len (nil)/0␛[0m
[00:00:10.500,610] ␛[0m<dbg> spi_nrfx_spim: finish_transaction: Transaction finished with status 0␛[0m
[00:00:10.501,922] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: tx_bufs 0x20005130 - rx_bufs 0x20005138 - 1␛[0m
[00:00:10.503,570] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: current_tx 0x20005140 (2), current_rx 0x20005140 (2), tx buf/len 0x20005128/1, rx buf/len 0x20005128/1␛[0m
[00:00:10.505,950] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_tx: tx buf/len 0x20005177/1␛[0m
[00:00:10.507,171] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_rx: rx buf/len 0x20005177/1␛[0m
[00:00:10.508,392] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_tx: tx buf/len (nil)/0␛[0m
[00:00:10.509,521] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_rx: rx buf/len (nil)/0␛[0m
[00:00:10.510,681] ␛[0m<dbg> spi_nrfx_spim: finish_transaction: Transaction finished with status 0␛[0m
[00:00:10.511,993] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: tx_bufs 0x20005130 - rx_bufs 0x20005138 - 1␛[0m
[00:00:10.513,610] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: current_tx 0x20005140 (2), current_rx 0x20005140 (2), tx buf/len 0x20005128/1, rx buf/len 0x20005128/1␛[0m
[00:00:10.516,021] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_tx: tx buf/len 0x20005177/1␛[0m
[00:00:10.517,211] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_rx: rx buf/len 0x20005177/1␛[0m
[00:00:10.518,432] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_tx: tx buf/len (nil)/0␛[0m
[00:00:10.519,592] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_rx: rx buf/len (nil)/0␛[0m
[00:00:10.520,721] ␛[0m<dbg> spi_nrfx_spim: finish_transaction: Transaction finished with status 0␛[0m
[00:00:10.522,033] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: tx_bufs 0x20005140 - rx_bufs (nil) - 1␛[0m
[00:00:10.523,620] ␛[0m<dbg> spi_nrfx_spim: spi_context_buffers_setup: current_tx 0x20005150 (1), current_rx (nil) (0), tx buf/len 0x20005138/1, rx buf/len (nil)/0␛[0m
[00:00:10.525,878] ␛[0m<dbg> spi_nrfx_spim: spi_context_update_tx: tx buf/len (nil)/0␛[0m
[00:00:10.527,038] ␛[0m<dbg> spi_nrfx_spim: finish_transaction: Transaction finished with status 0␛[0m
[00:00:10.528,320] ␛[1;31m<err> mcuboot: Image in the secondary slot is not valid!␛[0m
[00:00:10.583,068] ␛[0m<inf> mcuboot: Bootloader chainload address offset: 0x20000␛[0m
[00:00:10.584,259] ␛[0m<inf> mcuboot: Jumping to the first image slot␛[0m

RE: MCUboot loses pending update image in slot 1 after reset

Sigurd — Wed, 12 Jun 2024 12:24:02 GMT

Hi!

I would have expected MCUboot to swap here.

[quote user=""]My `mcuboot.conf`:[/quote]

Could you try to enable logging for MCUboot ? The MCUboot log should hopefully be able to tell us more about what is happening.

# Enable logging
CONFIG_LOG=y
CONFIG_LOG_DEFAULT_LEVEL=4