sometime fds_gc() does not call the user handler.

RE: sometime fds_gc() does not call the user handler.

masataka hasegawa — Mon, 03 Sep 2018 01:26:15 GMT

Hi,

I checked modules/nrfx/mdk/system_nrf52.c,
and I found the workaround code of errata 108.

if (errata_108()){
*(volatile uint32_t *)0x40000EE4 = *(volatile uint32_t *)0x10000258 & 0x0000004F;
}

And we are discussing Zou-san.
We are trying to reproduce it at simple projects.

Thanks,

RE: sometime fds_gc() does not call the user handler.

Kenneth — Wed, 29 Aug 2018 09:19:29 GMT

Hello,

From my understand you have also been in contact with my colleague Zou-san. We are not able to recreate your issue at the moment.

We will need a (simplified test) project from you that we can run on a nRF52840-DK to recreate your issue here, can you please arrange that for us?

Our FreeRTOS expert is back on Monday, he can look at your project then. Also please send us your system_nrf52.c file in your project so we can check that you have a workaround for errata_108().

Best regards,
Kenneth

RE: sometime fds_gc() does not call the user handler.

masataka hasegawa — Wed, 29 Aug 2018 01:38:52 GMT

Hi,

Yes, I know the meaning of NRF_ERROR_BUSY.
My experiment is below.

1. our firmware calls fds_gc.
2. fds_gc calls some fstorage APIs.
:
3. fds_gc calls sd_flash_write.
4. no event and no call user handler.
5. check timeout.
6. if timeout is detected, f/w calls event handler itself.
7. fds_gc goes to next step.
8. fds_gc calls some sd APIs (sd_flash_write...)
9. sd_flash_write return NRF_ERROR_BUSY.

1 ~ 4, 7 ~ 9 = original SDK code
5, 6, = added my experiment code

This result means that sd_flash_write of No 3 have not complete.

Thanks,

RE: sometime fds_gc() does not call the user handler.

Kenneth — Tue, 28 Aug 2018 13:52:14 GMT

Hello,

* @retval ::NRF_ERROR_BUSY The previous command has not yet completed.

This means that there is already an on-going flash operation, you will have to wait until NRF_EVT_FLASH_OPERATION_SUCCESS or NRF_EVT_FLASH_OPERATION_ERROR events and try again.

There is no way to reset the flash module in the softdevice no.

Best regards,
Kenneth

RE: sometime fds_gc() does not call the user handler.

masataka hasegawa — Tue, 28 Aug 2018 09:49:29 GMT

Hi,

I want to reset flash module in softdevice.
Are there the reset API in softdevice?

Thanks,

RE: sometime fds_gc() does not call the user handler.

masataka hasegawa — Tue, 28 Aug 2018 09:07:20 GMT

Hi,

I added debug code (detect timeout and retry sd_flash_write.)
sd_flash_write returns NRF_ERROR_BUSY.

Thanks,

RE: sometime fds_gc() does not call the user handler.

masataka hasegawa — Tue, 28 Aug 2018 05:36:58 GMT

Hi,

I am using SDK15.
and My projects is based on ANCS example.

Thanks,

RE: sometime fds_gc() does not call the user handler.

Kenneth — Mon, 27 Aug 2018 11:23:23 GMT

Hello,

Can you provide which SDK you are using? For instance is it nRF5 SDK v14.2? That will help me find internally if there are any known issues.

Best regards,
Kenneth

RE: sometime fds_gc() does not call the user handler.

masataka hasegawa — Mon, 27 Aug 2018 04:33:17 GMT

Hi,

I added log code and followings is the log.

call fds_record_delete().
> delete_execute()
> sd_flash_write(0x000fea20, 0x00000001)
> destination flash value = 0x00240051 (not 0xFFFFFFFF)
result == FDS_OP_EXECUTING

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> delete_execute()
call fds_record_write().
> write_execute()
> sd_flash_write(0x000fbf0c, 0x00000001)
> destination flash value = 0xffffffff
result == FDS_OP_EXECUTING

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> write_execute()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fbf14, 0x00000001)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> write_execute()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fbf18, 0x00000024)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> write_execute()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fbf10, 0x00000001)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> write_execute()

call fds_record_delete().
> delete_execute()
> sd_flash_write(0x000feabc, 0x00000001)
> destination flash value = 0x00ad0002 (not 0xFFFFFFFF)
result == FDS_OP_EXECUTING

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> delete_execute()

call fds_record_write().
call fds_gc().
> gc_execute()
> gc_init()
> gc_next_page();
> sd_flash_write(0x000ff008, 0x00000 093)
> destination flash value = 0xffffffff
result == FDS_OP_EXECUTING

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_update_swap_offset()
> gc_record_find_next()
result == FDS_OP_EXECUTING
> sd_flash_page_erase(0xfe)

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_swap_pages()
> gc_swap_promote()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000ff000, 0x00000002)
> destination flash value = 0xdeadc0de (not 0xFFFFFFFF)

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> case GC_PROMOTE_SWAP: m_gc.state = GC_TAG_NEW_SWAP;
> gc_tag_new_swap()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fe000, 0x00000002)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> change m_gc.state = GC_NEXT_PAGE
> gc_next_page()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fe008, 0x000000B0)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_update_swap_offset()
> gc_record_find_next()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fe2c8, 0x000000C5)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_update_swap_offset()
> gc_record_find_next()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fe5dc, 0x00000093)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_update_swap_offset()
> gc_record_find_next()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fe828, 0x00000093)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_update_swap_offset()
> gc_record_find_next()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fea74, 0x00000093)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_update_swap_offset()
> gc_record_find_next()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fecc0, 0x00000093)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_update_swap_offset()
> gc_record_find_next()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fef0c, 0x0000009c)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_update_swap_offset()
> gc_record_find_next()
result == FDS_OP_EXECUTING
> sd_flash_page_erase(0xfb)

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_swap_pages()
> gc_swap_promote()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fe000, 0x00000002)
> destination flash value = 0xdeadc0de (not 0xFFFFFFFF)

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> case GC_PROMOTE_SWAP: m_gc.state = GC_TAG_NEW_SWAP;
> gc_tag_new_swap()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fb000, 0x00000002)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> change m_gc.state = GC_NEXT_PAGE
> gc_next_page()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fb008, 0x000000B0)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_update_swap_offset()
> gc_record_find_next()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fb2c8, 0x00000093)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_update_swap_offset()
> gc_record_find_next()
result == FDS_OP_EXECUTING
> sd_flash_page_erase(0xfc)

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_swap_pages()
> gc_swap_promote()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fb000, 0x00000002)
> destination flash value = 0xdeadc0de (not 0xFFFFFFFF)

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> case GC_PROMOTE_SWAP: m_gc.state = GC_TAG_NEW_SWAP;
> gc_tag_new_swap()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fc000, 0x00000002)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> change m_gc.state = GC_NEXT_PAGE
> gc_next_page()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fc008, 0x00000093)
> destination flash value = 0xffffffff

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_update_swap_offset()
> gc_record_find_next()
result == FDS_OP_EXECUTING
> sd_flash_page_erase(0xfd)

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> gc_swap_pages()
> gc_swap_promote()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fc000, 0x00000002)
> destination flash value = 0xdeadc0de (not 0xFFFFFFFF)

> nrf_fstorage_sys_evt_handler()
> fs_event_handler()
> gc_execute()
> case GC_PROMOTE_SWAP: m_gc.state = GC_TAG_NEW_SWAP;
> gc_tag_new_swap()
result == FDS_OP_EXECUTING
> sd_flash_write(0x000fd000, 0x00000002)
> destination flash value = 0xffffffff

no nrf_fstorage_sys_evt_handler() event of sd_flash_write !!!

Thanks,